fix(classifier): prefer feature intent for ambiguous prompts#229
Open
ozymandiashh wants to merge 2 commits intogetagentseal:mainfrom
Open
fix(classifier): prefer feature intent for ambiguous prompts#229ozymandiashh wants to merge 2 commits intogetagentseal:mainfrom
ozymandiashh wants to merge 2 commits intogetagentseal:mainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CodeBurn was over-counting normal feature work as Debugging because the classifier checked broad debug keywords before feature intent. Words like
error,issue, andfixare not always bug work: users often ask agents to add error handling, create issue trackers, or adjust layout for a new feature.This PR narrows that behavior for the specific feature-style contexts called out in #196. It keeps explicit bugs, failures, crashes, status-code errors, and broken behavior in Debugging, but lets feature construction win for ambiguous debug-like phrases.
What changed
error handling,issue tracker, andnew featurecontexts.fix the login bugorfix the 500 error in the new feature.Partially addresses #196.
Why this is safe
The change is limited to activity classification. It does not touch provider discovery, token parsing, pricing, session aggregation, or the separate provider-attribution/Zed gaps discussed in #196. Explicit debug signals still win when the prompt clearly describes a bug or failure.
Validation
npx tsc --noEmitnpx vitest run tests/classifier.test.tsnpx vitest runnpm run build