Skip to content

Add CI drift check for vendored Tiny Wire tokens - #7

Merged
LinzLos merged 1 commit into
mainfrom
claude/keen-mayer-9knmpc
Sep 28, 2026
Merged

LinzLos merged 1 commit into
mainfrom
claude/keen-mayer-9knmpc

Conversation

@LinzLos

@LinzLos LinzLos commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Adds .github/workflows/tinywire-drift.yml, which runs scripts/check-tinywire-drift.sh on every push and pull request so vendored-token drift is caught in CI instead of only by hand.

How it works:

  • Checks out this repo, then reads the pin from src/lib/.tinywire-version (where the script actually looks — this consumer vendors into src/lib/, and the pin sits next to tokens.css).
  • Checks out LinzLos/tiny-wire at the tag matching the pin (v<pin>; upstream tags releases as vX.Y — currently v1.6, matching the pin). A pin naming a version with no tag fails the checkout step, which is itself a drift signal. Tiny-wire is public, so no extra token is needed.
  • Runs the script with TW_REPO_DIR="$GITHUB_WORKSPACE/tiny-wire", since Actions can't place the source as a true sibling of the workspace.

Validated locally: cloning tiny-wire at v1.6 and running the script reports OK — vendored Tiny Wire tokens are in sync (v1.6). No vendored files or the pin were modified.

Note the tradeoff of pinning the checkout to the tag: CI verifies byte-for-byte integrity against the pinned release, but a new upstream release won't fail this check until the pin is bumped. Switching the ref to main would flip that: red on every upstream release until re-sync.

🤖 Generated with Claude Code

https://claude.ai/code/session_011wancyyu9ynb6atCpn7qt9


Generated by Claude Code

Runs scripts/check-tinywire-drift.sh on every push and PR: checks out
tiny-wire at the tag matching src/lib/.tinywire-version and verifies the
pin and vendored files match the pinned release byte for byte.
@LinzLos
LinzLos merged commit 0ce7197 into main Sep 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant