Skip to content

Fix post-session processing retry storm on unparseable AI responses - #123

Merged
MikeAlhayek merged 1 commit into
mainfrom
ma/fix-post-session-processing
Aug 7, 2026
Merged

Fix post-session processing retry storm on unparseable AI responses#123
MikeAlhayek merged 1 commit into
mainfrom
ma/fix-post-session-processing

Conversation

@MikeAlhayek

Copy link
Copy Markdown
Member

The no-tools structured output path returned null when a successful (HTTP 200) AI response could not be parsed into structured task results (empty completion, safety refusal, truncation, or non-JSON prose). The post-close processor recorded this as a generic "produced no result", retried until the attempt cap, and marked the whole cycle failed.

  • Structured no-tools path now records an explicit Failed result with a diagnostic message instead of silently returning null.
  • Log the unparseable-but-successful response at Warning (with a preview of the raw response) instead of Debug-only.
  • Improve the processor's fallback error message to explain the AI produced no parseable result or there was no content to evaluate.
  • Skip the post-session AI request entirely when there is no meaningful user content (empty, whitespace-only, or system-generated prompts).
  • Add tests and update the 1.1.0 changelog and chat docs.

The no-tools structured output path returned null when a successful
(HTTP 200) AI response could not be parsed into structured task results
(empty completion, safety refusal, truncation, or non-JSON prose). The
post-close processor recorded this as a generic "produced no result",
retried until the attempt cap, and marked the whole cycle failed.

- Structured no-tools path now records an explicit Failed result with a
  diagnostic message instead of silently returning null.
- Log the unparseable-but-successful response at Warning (with a preview
  of the raw response) instead of Debug-only.
- Improve the processor's fallback error message to explain the AI
  produced no parseable result or there was no content to evaluate.
- Skip the post-session AI request entirely when there is no meaningful
  user content (empty, whitespace-only, or system-generated prompts).
- Add tests and update the 1.1.0 changelog and chat docs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@MikeAlhayek
MikeAlhayek merged commit 2f5cf45 into main Aug 7, 2026
10 checks passed
@MikeAlhayek
MikeAlhayek deleted the ma/fix-post-session-processing branch August 7, 2026 19:10
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