Skip to content

Improved trait prompt#246

Merged
sfreeman422 merged 1 commit into
masterfrom
feature/duplicative-traits
Apr 29, 2026
Merged

Improved trait prompt#246
sfreeman422 merged 1 commit into
masterfrom
feature/duplicative-traits

Conversation

@sfreeman422
Copy link
Copy Markdown
Collaborator

@sfreeman422 sfreeman422 commented Apr 29, 2026

This pull request updates the deduplication and phrasing guidelines for user traits in the ai.constants.ts file. The main focus is to ensure trait lists are both clear and non-redundant by treating similar meanings as duplicates and preferring the best possible phrasing.

Trait deduplication and phrasing improvements:

  • Added instructions to treat semantically similar phrasings as duplicates, not just exact wording matches.
  • Specified that only one canonical version of a trait should be kept, and the rest should be dropped.
  • Clarified that the canonical trait should be the clearest, most specific, and most durable version.
  • Required a final deduplication pass to remove any overlapping traits before finalizing the list.

Copilot AI review requested due to automatic review settings April 29, 2026 12:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refines the TRAIT_EXTRACTION_PROMPT guidance to more explicitly prevent duplicate/overlapping traits by adding clearer semantic de-duplication instructions.

Changes:

  • Adds explicit rules to treat semantically similar traits as duplicates (even with different wording).
  • Instructs the model to select a single canonical phrasing per underlying trait and run a final dedupe pass.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sfreeman422 sfreeman422 merged commit de6e6b2 into master Apr 29, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants