test: prove composite host discovery isolation - #653
Merged
Conversation
🦋 Changeset detectedLatest commit: 2a9e921 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
ScriptedAlchemy
marked this pull request as ready for review
September 6, 2026 00:29
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
11 tasks
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
Host evidence
No adapter capability contract data changed. Existing pinned sources remain authoritative in
packages/agent-bundle/src/adapters/capabilities/{claude,codex,cursor,portable}.json.Local merge gate
Branch head
2a9e921a90contains currentorigin/main46e8e7d25a.pnpm build— passpnpm typecheck— passpnpm lint— pass (1,420 files, 88 rules)pnpm test:unit— pass (283 files; 4,223 passed, 6 skipped)pnpm build— pass (refreshed dist before integration pools)pnpm exec rstest --config rstest.config.ts packages/agent-bundle/tests/build-compose.test.ts packages/agent-bundle/tests/host-contract.test.ts packages/agent-bundle/tests/host-install-proof.test.ts packages/agent-bundle/tests/manifest-relocatable.test.ts packages/agent-bundle/tests/packed-host-install-proof.test.ts— pass (5 files; 47 passed, 1 opt-in host-contract check skipped)Deslop
Deslop: GPT-5.6 Sol, 3 edits (kept discovery assertions in existing helpers; reused canonical projection/executable records instead of adding a catalog; flattened the final inspect assertion path).
Self-review
Reviewer: Claude Fable 5.1 Thinking High. First pass findings and disposition:
build/inspect, and both install-proof suites exercise production composition. Registering two synthetic adapters is itself rejected by the existing AB4106 contract.inspectresult could skip assertions: fixed by failing unless the inspected manifest is valid.root manifest: fixed torootand aligned the diagnostic docs.nonCollidingFixturePairsand qualifying Claude/Cursor rejection to this fixture's frontmatter-bearing command.Second and post-main-refresh passes: no remaining merge risks. The sole cosmetic indentation note was fixed in
26f8e2f0b3; the complete local gate above passed on the branch containing current main.Closes #651