Skip to content

fix(session,runtime): tighten summary validation and compact parsing#9

Merged
wynxing merged 1 commit into
codex/taskstate-compact-prfrom
fork-pr-252-1776091508
Apr 14, 2026
Merged

fix(session,runtime): tighten summary validation and compact parsing#9
wynxing merged 1 commit into
codex/taskstate-compact-prfrom
fork-pr-252-1776091508

Conversation

@fennoai

@fennoai fennoai Bot commented Apr 13, 2026

Copy link
Copy Markdown

Requested by @Yumiue

Summary

  • Reject invalid payloads ( / non-object) in to keep list/load behavior consistent
  • Replace repeated-start JSON extraction with single-pass scanning in compact summary parsing
  • Add regression tests for both fixes and keep simplified test helpers

…rize compact JSON extraction

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: Yumiue <188874804+Yumiue@users.noreply.github.com>
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 89.28571% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/runtime/compact_generator.go 82.35% 2 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@wynxing wynxing merged commit e8ba566 into codex/taskstate-compact-pr Apr 14, 2026
1 check passed
@fennoai fennoai Bot deleted the fork-pr-252-1776091508 branch April 14, 2026 00:50
wynxing pushed a commit that referenced this pull request Apr 15, 2026
fix(runtime): simplify tool-call scheduling and align compact docs
wynxing pushed a commit that referenced this pull request Apr 24, 2026
fix(gateway): close unresolved review risks and simplify
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.

3 participants