Skip to content

fix(extraction): Report malformed model output accurately - #461

Open
sentry-junior[bot] wants to merge 5 commits into
mainfrom
fix/extraction-failure-diagnostics
Open

fix(extraction): Report malformed model output accurately#461
sentry-junior[bot] wants to merge 5 commits into
mainfrom
fix/extraction-failure-diagnostics

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Classify malformed model output as an extraction failure instead of suggesting authentication problems. Failed GitHub checks now retain sanitized per-hunk diagnostics so transient provider or parser failures can be diagnosed from the check itself.

Extraction recovery

The structured fallback now accepts any non-empty model response and retries once when extraction fails, aggregating auxiliary usage across both attempts.

Failure reporting

All-extraction failures preserve their specific extraction error code, and the action carries sanitized filename, line range, code, message, and preview details into the failed check summary.

Requested by David Cramer.

--

View Junior Session Sentry

Co-Authored-By: David Cramer <david@sentry.io>
@dcramer
dcramer marked this pull request as ready for review August 1, 2026 01:03
Comment thread packages/warden/src/cli/output/tasks.ts
Comment thread packages/warden/src/output/github-checks.ts
Comment thread packages/warden/src/sdk/analyze.test.ts

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 985e2a8. Configure here.

Comment thread packages/warden/src/output/github-checks.ts Outdated
Comment thread packages/warden/src/output/github-checks.ts
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.

0 participants