Skip to content

fix(ai): admit bounded warnings while resuming - #52

Merged
first-assist merged 1 commit into
mainfrom
fix/retained-resume-warning-interleaving-20260901
Sep 1, 2026
Merged

fix(ai): admit bounded warnings while resuming#52
first-assist merged 1 commit into
mainfrom
fix/retained-resume-warning-interleaving-20260901

Conversation

@first-assist

@first-assist first-assist commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Outcome

The strict Codex app-server actor now admits the documented generic warning notification during an exact retained thread/resume handshake, in addition to the existing correlated-turn window.

Warning text is validated and discarded. Admission remains thread-correlated, count- and byte-bounded, and cannot advance resume readiness. New-thread creation, deletion, state-index scans, idle periods, malformed frames, mismatched thread IDs, and flooding remain rejected.

Compatibility

  • graphql-orm-ai: 0.95.14
  • AI schema module: unchanged at 0.64.0
  • no database, GraphQL SDL, protected-payload, backup, restore, or data migration
  • hosts should continue reading after RuntimeWarning until exact resume readiness is proven

Verification

  • focused warning and resume interleaving tests
  • full provider-enabled graphql-orm-ai unit, integration, and doc tests
  • warnings-denied Clippy and Rustdoc
  • PascalCase contract and documentation
  • PostgreSQL and MSSQL compile-only lanes
  • documentation, workspace dependency, release state, package release, deterministic manifest, formatting, diff, and SemVer gates

Documentation impact

  • Documentation updated

README, changelog, migration guide, implementation status, and generated workspace inventory are updated.

@first-assist first-assist reopened this Sep 1, 2026
@first-assist
first-assist merged commit e139dc6 into main Sep 1, 2026
35 of 38 checks passed
@first-assist
first-assist deleted the fix/retained-resume-warning-interleaving-20260901 branch September 1, 2026 05:09
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