Skip to content

test(runtime): cover truncated notification resync - #28

Merged
m-mdy-m merged 3 commits into
mainfrom
test/runtime-notification-resync-conformance
Nov 13, 2026
Merged

m-mdy-m merged 3 commits into
mainfrom
test/runtime-notification-resync-conformance

Conversation

@m-mdy-m

@m-mdy-m m-mdy-m commented Nov 13, 2026

Copy link
Copy Markdown
Member

Summary

Add explicit Runtime integration coverage for incomplete Kernel notification history.

Coverage

  • An event with edits_truncated = true forces an authoritative metadata refresh.
  • A version-chain gap forces the same refresh instead of applying an incomplete edit transform.
  • The test verifies the refreshed version, byte length and exact document bytes.

Contract references

  • Spec RT-008: Runtime must resynchronize when notification history is incomplete or truncated.
  • Runtime Host API 1.0.0 §3 Notifications: truncated edit data or a version-chain gap must never be replayed as a partial edit list.

No production behavior is changed; this locks the existing fail-safe behavior with regression coverage.

@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-13T15:48:02.209017Z
We couldn’t complete this review. Comment @codex review to try again.
5b557c7 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@m-mdy-m m-mdy-m self-assigned this Nov 13, 2026
@m-mdy-m
m-mdy-m merged commit b4444cb into main Nov 13, 2026
10 checks passed
@m-mdy-m
m-mdy-m deleted the test/runtime-notification-resync-conformance branch November 13, 2026 15:54
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