vendored-sync: reference the local claw checkout (claw private)#181
Merged
Conversation
The canonical source moved to culturebotai-claw (claw#18, shared/idlabel/), but claw is private, so the tokenless raw-fetch model 404s. This points the drift check at a LOCAL sibling claw checkout instead of the network. check_vendored_sync.sh now: - locates claw via $CLAW_ROOT or sibling auto-discovery (handles a flat Mech and CommunityMech's nested path); - diffs the 5 vendored files against $claw/shared/idlabel/ and fails on drift; - SKIPS (exit 0) with a loud notice when no claw checkout is present -- so it does not false-fail on a CI runner that lacks it. Enforcement is LOCAL while claw is private; anyone doing cross-Mech work with the sibling checkout gets it. .vendored_canon_ref (bumped to claw 1ad5d40) records the intended claw commit and drives a soft HEAD-mismatch warning. Trade-off: mech CI (no claw checkout) no longer enforces this -- it skips. The durable CI path is claw-side: claw's cross-repo-validation already checks out the Mechs and can diff them against shared/idlabel/. Restoring a pinned raw-fetch (CI-enforceable) just needs claw to be public. Vendored file content is unchanged -- only the checker and the ref. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the vendored-sync drift checker to compare against a local culturebotai-claw checkout (since claw is private and raw GitHub fetches 404), and bumps the pinned canonical commit reference.
Changes:
- Reworked
scripts/check_vendored_sync.shto locate a local claw checkout ($CLAW_ROOTor sibling auto-discovery) andcmpthe vendored files againstclaw/shared/idlabel/, skipping (exit 0) when claw isn’t available. - Updated
scripts/.vendored_canon_refto pin claw commit1ad5d40….
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| scripts/check_vendored_sync.sh | Switches canonical comparison from network raw-fetch to local claw checkout discovery + byte-compare, with skip behavior when claw is absent. |
| scripts/.vendored_canon_ref | Updates the pinned canonical claw commit hash used for informational reporting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # sibling checkout. It is NOT enforced in CI while claw is private. Making claw | ||
| # public again lets this revert to a pinned raw-fetch that CI can run. | ||
| # | ||
| # Dependency-free: bash + diff only. |
Comment on lines
+31
to
+45
| canon_dir="" | ||
| if [ -n "${CLAW_ROOT:-}" ] && [ -d "${CLAW_ROOT}/shared/idlabel" ]; then | ||
| canon_dir="${CLAW_ROOT}/shared/idlabel" | ||
| else | ||
| for cand in ../culturebotai-claw ../../culturebotai-claw ../../../culturebotai-claw; do | ||
| if [ -d "$cand/shared/idlabel" ]; then canon_dir="$cand/shared/idlabel"; break; fi | ||
| done | ||
| fi | ||
|
|
||
| if [ -z "$canon_dir" ]; then | ||
| echo "SKIP: culturebotai-claw checkout not found (set CLAW_ROOT to its path)." | ||
| echo " Cross-repo vendored-drift is NOT enforced here (claw is private)." | ||
| echo " Run locally alongside a claw checkout to enforce." | ||
| exit 0 | ||
| fi |
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.
The canonical id-label source moved to claw (claw#18,
shared/idlabel/), but claw is private, so the tokenlessraw.githubusercontentfetch 404s. Per direction, this references a local claw checkout instead of the network.check_vendored_sync.shnow:$CLAW_ROOTor sibling auto-discovery (handles flat Mechs and CommunityMech's nested path);$claw/shared/idlabel/, fails on drift (verified: catches an edit, exit 1);Trade-off (important): mech CI has no claw checkout, so this job now skips in CI — enforcement is local (anyone with the sibling claw checkout). The durable CI path is claw-side: claw's
cross-repo-validationalready checks out the Mechs and can diff them againstshared/idlabel/. Making claw public later restores a CI-enforceable pinned raw-fetch with a one-line revert.Vendored file content is unchanged — only the checker +
.vendored_canon_ref(bumped to claw1ad5d40).🤖 Generated with Claude Code