feat(impact): add reasoning handler for AI - #113
Merged
Conversation
andev0x
commented
Jul 31, 2026
andev0x
commented
Member
- update provider structs to include reasoningHandler field
- route reasoning content through handler in stream readers
- ensure reasoning content is not mixed with main response
- update provider structs to include reasoningHandler field - route reasoning content through handler in stream readers - ensure reasoning content is not mixed with main response
- update envVarForProvider to include new providers - extend ValidProviderNames with new provider names - modify buildProjectConfig to handle new providers - introduce NewNineRouterProvider for 9Router integration
- update patch application logic to re-read file context when hunk mismatches - implement forced full-content fallback for small files on patch apply failure
- modify IsTerminal to include TaskSkipped - implement MarkSkipped method for ledger - adjust tests to cover new behavior - update error handling in patch application logic
- apply semantic repository updates
- unified reasoning content handling across providers to support both "reasoning" and "reasoning_content" fields
- adjust max ledger chars to 4000 for local models - introduce first-token guard in plan synthesis timeout - deduplicate raw intent injection in bridge.go - refine problem selection in commands.go for synthesis prompt
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
1 participant
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.