Repository navigation
chore(vendor): adopt interchange 4cc6972 drift in vendored trees - #1431
Merged
TheGreatAxios merged 4 commits intoOct 15, 2026
Merged
Conversation
Contributor
|
Triage: Fail @TheGreatAxios, please address the following:
|
TheGreatAxios
changed the base branch from
cl-10651-intx-re-sync-vendor-snapshot-to-040-re-carry-local-patches
to
main
October 15, 2026 19:46
Verbatim adoption of the drift ranges triaged in docs/VENDORING.md; carried patches and ledgers land next.
Verbatim adoptions of mime/mailbox/harness and workflow-host's narrow subset; types runtime split carried with ledger markers; inference's 9 commits merged with 3-way-resolved carries; storage-isogit e75eee2 verified safe (no first-party sync.request/statePushed/createStatePack/undeploy use); reasoning-effort c8be7e9 adopted; crypto/author/log unchanged. formatSafetyRatingText gains a ledgered compat re-export for the published @corbits/openai-responses dist, which still imports it from @intx/types/runtime.
TheGreatAxios
force-pushed
the
cl-10656-intx-triage-adopt-upstream-head-drift-mimemail
branch
from
October 15, 2026 20:17
d829f46 to
e4630cd
Compare
Static analysis on PR 1431 failed bunx oxfmt --check on this file.
storage-isogit is 5 commits, not 9 files. Agent work is d1835be, 660ef81, and d1e05e9. runtime.ts is the split re-exports plus the compat re-export, not a two-line barrel.
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
5453d0b0→4cc6972) per vendored area and adopts the vendored surface: mime, mailbox, harness verbatim; types carries the runtime split (runtime.ts → runtime-core.ts/inference-events.ts, 33-entry export map mirrored); inference takes its 9 commits with three-way merges preserving all 24 ledgered patches incl.ephemeralTurnsand the atomic-acceptance try/finally; storage-isogit lands the verified-safee75eee2removal (sync.request/statePushedhad zero local, vendored, and vendorable references); reasoning-effort mappingc8be7e9adopted (no wire change for first-party operators withouteffort); workflow-host narrows to exactly the 3 files we vendor; crypto/npm-dist and authz/log/tools-posix need nothing.formatSafetyRatingTextfrom the./runtimesubpath while@corbits/openai-responses@0.2.1(installed) imports it from there — one compat re-export in vendoredruntime.ts, independently ledgered with its removal path (the openai-responses release that adopts the new subpath). Zero ledger entries dropped; the divergent src surface stays bijective with the ledger anchors (inference 1324+/230−, types 43+/6−, storage-isogit 73+/1−, agent 382+/2−, tools-posix 13+/1−re-verified at HEAD).AdapterManifestimport moves to@intx/types(upstream dropped the inference barrel re-export), one narrowedmessageIdreuse in submit.ts, two test guards.6db66b56, patches + adaptation7406c34c), split re-balanced so both commits compile independently (verified with true-isolation typecheck+build, not just hooks).Verification
bun run checkexit 0: 8938 pass / 0 fail across 580 files;bun test e2e/vendored-carry.test.ts2 pass (both carried-patch integration tests);vendor-patch-ledger2 pass.4cc6972byte-for-byte (285/285 verified); the ledger set is bijective with the divergence surface at HEAD; e75eee2 and c8be7e9 verdicts independently reproduced from the upstream clone;sync.request/statePushedhave zero references anywhere in the repo.substrate-mailbox-store.test.tsfailure (documented, unchanged, out of gate).Fixes CL-10656