chore(brand): refresh the snapshot — the markers were rendering a stale catalog - #51
Closed
VickyXAI wants to merge 1 commit into
Closed
chore(brand): refresh the snapshot — the markers were rendering a stale catalog#51VickyXAI wants to merge 1 commit into
VickyXAI wants to merge 1 commit into
Conversation
…le catalog Opened by brand/fanout-brand-numbers.mjs. brand-numbers.json here was behind the published artifact, so every br: marker in this repo rendered a number that is no longer true. The markers did their job — they rendered the input they were given. --check is offline on purpose so PR CI stays deterministic, which means it validates against this repo's own snapshot, and only --refresh updates that snapshot. This job is what runs it. Produced by --refresh, not by hand.
VickyXAI
force-pushed
the
chore/refresh-brand-numbers
branch
from
August 31, 2026 06:02
bf33041 to
67e911f
Compare
VickyXAI
pushed a commit
that referenced
this pull request
Sep 5, 2026
`--check` was green because it only validates the keys a marker actually renders, and those five were current. The rest of the snapshot had drifted: withFallback 48 → 34, withFallbackAllEntries 89 → 73, aliases 229 → 259, plus three mcp context keys the artifact now carries. No rendered number changes, so this is not a docs fix — it stops the next marker added for one of those keys from rendering a stale value on day one. Produced by `scripts/sync-brand-numbers.mjs --refresh`, no hand-edited digits. Supersedes #51, which proposed the same refresh against a 26 Aug main. Its five rendered numbers have since landed independently, so the branch had nothing left to deliver and conflicted with the API-key documentation merged in #59/#60. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QMLNFbR6Jg2HBFpRUGBjyh
Contributor
Author
|
Superseded by b7e54a6 on main. The five rendered numbers this branch carried (chatVisible 76, models.free 7, autoVsBaselinePct 84) have already landed on main independently — Ran |
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
1 participant
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.
brand-numbers.jsonin this repo was behind the published artifact, so everybr:marker here rendered a stale number.The markers worked correctly — they rendered the input they had.
--checkis offline by design so PR CI stays deterministic, which means it validates against this repo's own snapshot; only--refreshupdates that snapshot.Opened automatically by
brand/fanout-brand-numbers.mjs. Produced by--refresh, no hand-edited digits.