Skip to content

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

Merged
VickyXAI merged 1 commit into
mainfrom
chore/refresh-brand-numbers
Aug 8, 2026
Merged

chore(brand): refresh the snapshot — the markers were rendering a stale catalog#239
VickyXAI merged 1 commit into
mainfrom
chore/refresh-brand-numbers

Conversation

@VickyXAI

@VickyXAI VickyXAI commented Aug 8, 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

  • Documentation
    • Updated automatic-routing savings claims from 87% to 88% across product documentation and comparisons.
    • Updated published brand metrics, including total visible brands and video-supported brands.
    • Revised inference-cost savings figures to reflect the latest 88% estimate.

…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 Aug 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 8fb05a0a-ebce-4684-8381-06e99244bb34

📥 Commits

Reviewing files that changed from the base of the PR and between 69c278d and 2a922d2.

📒 Files selected for processing (5)
  • README.md
  • brand-numbers.json
  • docs/anthropic-third-party-harness-changes.md
  • docs/clawrouter-vs-openrouter-llm-routing-comparison.md
  • skills/clawrouter/SKILL.md

📝 Walkthrough

Walkthrough

The pull request updates automatic-routing savings claims from 87% to 88%. It also updates visible model counts in brand-numbers.json and synchronizes the metric across the README and related documentation.

Changes

Metrics and documentation synchronization

Layer / File(s) Summary
Update source metrics
brand-numbers.json
The visible model count increases to 93, the video count increases to 6, and autoVsBaselinePct increases to 88.
Synchronize published references
README.md, docs/..., skills/clawrouter/SKILL.md
Automatic-routing savings references change from 87% to 88% across the README, comparison documentation, third-party harness documentation, and skill documentation.

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

Possibly related PRs

Suggested reviewers: 1bcmax

🚥 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 prevent markers from rendering stale catalog data.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/refresh-brand-numbers

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.

@VickyXAI
VickyXAI merged commit 4c28169 into main Aug 8, 2026
3 of 4 checks passed
@VickyXAI
VickyXAI deleted the chore/refresh-brand-numbers branch August 8, 2026 04:57
VickyXAI pushed a commit that referenced this pull request Aug 8, 2026
…xt surfaces

The brand fan-out PR (#239) updated every br:-marked occurrence of
savings.autoVsBaselinePct but left package.json's "description" and
skills/clawrouter/SKILL.md's frontmatter on 87%, because neither is
marker-wrapped. src/router/brand-numbers.test.ts pins exactly those two, so
main went red the moment that PR merged.

Same shape as the seed-oss regression in v0.12.241: the fan-out maintains
markers, and anything quoting a brand number OUTSIDE a marker is invisible to
it. The guard test is what catches the gap — it did its job here.

Both surfaces now read 88%. 716 tests pass, brand-numbers --check clean,
prettier clean.

No version bump: package.json's description ships in the npm metadata, but
0.12.243 is already published and this is a marketing string, not behaviour.
It rides along with the next release.
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