-
Notifications
You must be signed in to change notification settings - Fork 3.1k
fix(runtime): preserve Hermes activation transport response #10272
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
50fdb15
test(runtime): expose Hermes activation transport bound
rsliter 1397355
fix(runtime): preserve activation transport response
rsliter 4cab78f
merge: refresh Hermes transport fix from main
rsliter 8c3518e
fix(runtime): keep release transport bound
rsliter e89ddd5
merge: refresh Hermes transport fix from main
rsliter e50c63e
test(runtime): parse activation window portably
rsliter 577bd04
merge: refresh Hermes transport fix from main
rsliter efc5b2a
fix(runtime): acknowledge Hermes activation release
rsliter 4c9fd8b
fix(onboard): cover activation replay deadline
rsliter d22dff5
fix(runtime): cover retained activation deadline
cjagwani 079b9b1
fix(runtime): use full response allowance
cjagwani 897d867
merge: refresh Hermes Shields fix from main
cjagwani 68343f9
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
cjagwani 60c2567
fix(runtime): make release acknowledgement failure explicit
rsliter f9e7e51
fix(runtime): resolve release acknowledgement feedback
rsliter df1c5b3
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
rsliter 83519cf
fix(runtime): make acknowledgement failure return explicit
rsliter 8ece9a0
test(runtime): diagnose repeated Shields transitions
cjagwani c141740
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
rsliter f50ff0f
test(ci): refresh Hermes Shields validation
rsliter f6a02e5
fix(onboard): wait for verified create handoff
cjagwani 77550ea
fix(onboard): wait for exact sandbox publication
cjagwani 33112b2
Merge remote-tracking branch 'origin/main' into codex/fix-hermes-shie…
cjagwani 24c8a0d
test(onboard): cover sandbox publication rejection
cjagwani 4ee5047
fix(hermes): preserve startup identity for release ack
cjagwani 45e7e98
merge: resolve conflicts with main
github-actions[bot] d0743bb
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
sandl99 7f91eb7
test(runtime): refresh Shields merge coverage
sandl99 c203f10
test(runtime): cover release acknowledgement order
sandl99 1bec8ca
fix(runtime): rescan replaced Hermes writers
sandl99 c63f95c
fix(runtime): wait for Hermes release publisher
sandl99 e74baa6
fix(runtime): stabilize Hermes release observation
sandl99 098abe9
test(runtime): stabilize helper timeout allowance
sandl99 76ce503
fix(runtime): require Hermes parent release acknowledgement
sandl99 338257c
fix(runtime): rescan vanished mutation processes
sandl99 d647734
fix(runtime): recapture exact mutation processes
sandl99 eb6856a
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
cjagwani cceb629
fix(runtime): retry raced mutation signals
cjagwani 05aed98
fix(shields): observe settled Hermes lock
cjagwani f0d17d2
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
cv 9637671
Merge branch 'main' into codex/fix-hermes-shields-transport-10155
cv 369bfa3
ci: retrigger PR review advisor
cv 15141cd
fix(shields): use read-only Hermes timer confirmation
rsliter 30e78c8
fix(shields): confirm fresh Hermes lockdown
rsliter 6e66aba
fix(runtime): retain Hermes transport recovery
rsliter 69f4a41
merge: refresh Hermes transport repair from main
prekshivyas 3667b6d
fix(runtime): bind pidfd to immutable task identity
prekshivyas 52a3c43
merge: refresh Hermes pidfd repair from main
prekshivyas 425301f
fix(runtime): retain the pinned OpenShell supervisor
prekshivyas a4f1e66
fix(shields): preserve the production settle window
prekshivyas e51aa98
merge: refresh Hermes repair from main
prekshivyas 6628681
test(shields): reuse the Hermes publisher harness
prekshivyas 3361440
test(shields): close advisor feedback
prekshivyas 99c4c72
fix(runtime): bind supervisor executable identity
prekshivyas a424235
fix(runtime): distinguish invalid startup gate state
prekshivyas 7649bfb
refactor(runtime): install fixed transport broker
prekshivyas e2b8c81
refactor(runtime): close broker review findings
prekshivyas 565569c
Merge remote-tracking branch 'origin/main' into codex/refresh-pr-10272
prekshivyas 702f72a
Merge remote-tracking branch 'origin/main' into codex/refresh-pr-10272
prekshivyas a2a59e2
refactor(runtime): close final advisor findings
prekshivyas 24ca693
Merge remote-tracking branch 'origin/main' into codex/refresh-pr-10272
prekshivyas 4bfd3f6
fix(runtime): bind broker interpreter identity
prekshivyas 8f10786
docs(shields): clarify immutable lock fallback
prekshivyas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[P1] Keep the stale-base replay fixture complete
Adding the broker to this unconditional ownership and mode replay breaks the extracted Dockerfile fixture in test/runtime/sandbox/sandbox-rlimit-hooks.test.ts because that fixture never stages the new path. Required CLI shard 2 is red, and the focused local test reproduces the same missing-file failure at line 770. Stage the broker in that fixture and assert its root-only mode so the stale-base replay remains executable and the required suite returns green.