Skip to content

chore(brand): refresh the snapshot — the markers were rendering a stale catalog - #389

Open
VickyXAI wants to merge 1 commit into
mainfrom
chore/refresh-brand-numbers
Open

VickyXAI wants to merge 1 commit into
mainfrom
chore/refresh-brand-numbers

Conversation

@VickyXAI

@VickyXAI VickyXAI commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

brand-numbers.json in this repo was behind the published artifact, so every br: marker here rendered a stale number.

The markers worked correctly — they rendered the input they had. --check is offline by design so PR CI stays deterministic, which means it validates against this repo's own snapshot; only --refresh updates that snapshot.

Opened automatically by brand/fanout-brand-numbers.mjs. Produced by --refresh, no hand-edited digits.

Summary by CodeRabbit

  • Updates
    • Updated reported model counts to reflect the latest available data.

…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.
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: BlockRunAI/ClawRouter/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 9db28309-d4a7-481b-8545-9b2c443be8f2

📥 Commits

Reviewing files that changed from the base of the PR and between becb296 and 6abd4c9.

📒 Files selected for processing (1)
  • brand-numbers.json

Included review availability: Your plan provides up to 8 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The pull request updates two model counts in brand-numbers.json. withFallback changes from 33 to 23. withFallbackAllEntries changes from 74 to 62.

Changes

Brand model counts

Layer / File(s) Summary
Update brand model counts
brand-numbers.json
The withFallback count decreases from 33 to 23. The withFallbackAllEntries count decreases from 74 to 62.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: refreshing the brand snapshot to fix stale catalog data used by the markers.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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