fix(ci): restore runtime host validation - #3793
Closed
liuxiaocs7 wants to merge 1 commit into
Closed
Conversation
Repair the collided session metadata migration, update the remaining Runtime Host request aliases, and keep the cross-process storage fixture from treating Node's SQLite warning as a child failure. Generated-by: OpenAI Codex
Astro-Han
reviewed
Aug 25, 2026
Astro-Han
left a comment
Contributor
There was a problem hiding this comment.
I reviewed this head and found no blocking issues. The v30→v31 repeatable migration correctly converges both v30 DB shapes and adds the required admission/WorkHub unique indexes. No concurrency gaps.
Checks on 1517d18 are test: success.
简体中文
该头未发现阻断。
Member
Author
|
resolved by #3796 |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Restore the failing
mainCI path by replacing the remaining removed Runtime Host operation aliases with typedrequest(...)calls.Repair a session-metadata migration version collision introduced by consecutive merges that both claimed version 30. Migration 31 now converges both released v30 database variants by ensuring the durable message-admission tables and WorkHub coordination uniqueness index all exist.
Keep the managed-dependency cross-process fixture from treating Node 24's expected SQLite experimental warning as a child-process failure.
Verification
npm run lintnpm run format:checknpm run buildnpm run typecheckRoot cause
Two consecutive changes independently added SQLite session-metadata migration 30. Because migrations are held in a
Map, the later entry silently replaced the durable message-admission migration while the database was still recorded at version 30. Runtime Host startup then queried tables that had never been created.AI use
Select exactly one:
Tool(s) and scope: OpenAI Codex diagnosed the CI failures, implemented the fixes and regression coverage, and ran local verification.
Checklist
Does this PR entail a change in behavior?