Skip to content

fix(analysis): surface recovered tool errors - #56

Merged
drewstone merged 1 commit into
mainfrom
fix/tool-error-findings
Jul 30, 2026
Merged

fix(analysis): surface recovered tool errors#56
drewstone merged 1 commit into
mainfrom
fix/tool-error-findings

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

What changed

  • emit one deterministic finding for recovered tool errors with exact span citations
  • retain root and failed-tool evidence on terminal failures without duplicate findings
  • cap cited errors at three and report omitted counts
  • correct CodeTraceBench clean-control documentation

Proof

  • pnpm test: 381 passed
  • pnpm typecheck: passed
  • pnpm build: passed
  • pnpm check:package: package ships the CLI and two skills
  • HALO bad fixture: 1 finding citing 3/3 failed calls
  • HALO clean fixture: 0 recovered-error findings
  • git merge-tree --write-tree origin/main HEAD: clean

@drewstone
drewstone merged commit 166f285 into main Jul 30, 2026
1 check 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.

1 participant