Skip to content
This repository was archived by the owner on Aug 7, 2026. It is now read-only.

fix(ci): root-repo KB checkout for intent-codec fixtures - #6

Merged
kpernyer merged 2 commits into
mainfrom
next
Jul 2, 2026
Merged

kpernyer merged 2 commits into
mainfrom
next

Conversation

@kpernyer

@kpernyer kpernyer commented Jul 2, 2026 •

Copy link
Copy Markdown
Contributor

Arena main CI failed: intent_codec_applets.rs includes applet intent JSON from the root repo's KB/02-product/applets/, invisible to CI. The sibling script now clones the root repo and symlinks KB at the expected relative location.

Follow-up worth considering: these fixtures are cross-repo contract artifacts — either move them into arena or publish them, so the test doesn't depend on a sibling checkout of the coordination repo.

🤖 Generated with Claude Code


Note

Low Risk
Test-only change: no runtime or production paths; main risk is stale fixtures if KB manifests change without re-copying.

Overview
Intent codec smoke tests no longer read applet JSON from a sibling root-repo KB/02-product/applets/ path (unavailable in Arena CI because that repo is private).

The manifests for activate-subscription and refill-prepaid-ai-credits are vendored under crates/cross-extension-smoke/tests/fixtures/, and intent_codec_applets.rs now include_str!s those local files. A comment documents that they are copies of the canonical KB manifests and must be re-synced manually if the source changes—drift should still fail the existing manifest-vs-arena_intent_cases assertions.

Reviewed by Cursor Bugbot for commit 7eef2ee. Bugbot is set up for automated code reviews on this repo. Configure here.

…/../KB

cross-extension-smoke's intent_codec_applets tests include_str! applet
intent JSON from the root coordination repo's KB, which CI never checked
out. Clone Reflective-Lab/reflective and symlink KB where the relative
include resolves. Cross-repo fixture reach noted as a smell — candidate
for moving the contract fixtures in-repo or publishing them.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cursor

cursor Bot commented Jul 2, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_238d2615-1424-4625-b251-80887b330f7e)

The previous attempt cloned Reflective-Lab/reflective for the KB
fixtures, but that repo is private and CI clones anonymously. Vendor
the two intent manifests into tests/fixtures/ with provenance noted;
codec checks fail loudly on divergence.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@cursor

cursor Bot commented Jul 2, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_4d4ceee2-9297-4243-a602-f77a9eb48026)

@kpernyer
kpernyer merged commit ec6be5c into main Jul 2, 2026
2 checks passed
@kpernyer
kpernyer deleted the next branch July 3, 2026 21:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant