Skip to content

fix: run post-scan prompts after failed scans - #325

Draft
ianw-oai wants to merge 1 commit into
mainfrom
dev/ianw/post-scan-hooks-on-failure
Draft

fix: run post-scan prompts after failed scans#325
ianw-oai wants to merge 1 commit into
mainfrom
dev/ianw/post-scan-hooks-on-failure

Conversation

@ianw-oai

@ianw-oai ianw-oai commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Problem

The post-scan prompt only runs after a complete scan. Failed and incomplete scans skip cost accounting and other follow-up work.

Change

Run the prompt after failed or incomplete scans. Keep the original scan error, report follow-up errors as warnings, and do not start another model turn after cancellation or a cost limit.

Tests

  • 958 SDK tests passed before the update to current main.
  • Six focused tests passed on current main.
  • Type checks, formatting, and the package build passed.

@github-actions github-actions Bot added the bug Something isn't working label Aug 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant