Skip to content

Fix get ans tool#107

Merged
swati510 merged 3 commits intomainfrom
fix-get-ans-tool
Apr 28, 2026
Merged

Fix get ans tool#107
swati510 merged 3 commits intomainfrom
fix-get-ans-tool

Conversation

@swati510
Copy link
Copy Markdown
Collaborator

Upgrade get_answer tool with question-aware symbol promotion: extracts identifiers
from the question, promotes matching symbols to the top of retrieval results, and
attaches source-body excerpts so the LLM can answer specific-method questions without
hedging

  • Add hedge detection to downgrade confidence when the LLM admits it can't answer,
    and bypass stale hedged cache entries for re-synthesis
  • Add identifier-citation gate to downgrade confidence when named symbols aren't
    found in top retrieval hits

Related Issues

Fixes #84

Test Plan

  • Tests pass (pytest)
  • Lint passes (ruff check .)
  • Web build passes (npm run build) (if frontend changes)

Checklist

  • My code follows the project's code style
  • I have added tests for new functionality
  • All existing tests still pass
  • I have updated documentation if needed

Copy link
Copy Markdown
Collaborator

@RaghavChamadiya RaghavChamadiya left a comment

Choose a reason for hiding this comment

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

LGTM

@swati510 swati510 merged commit b90aca4 into main Apr 28, 2026
5 checks passed
@swati510 swati510 deleted the fix-get-ans-tool branch April 28, 2026 04:37
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