Skip to content

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

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

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

Conversation

@VickyXAI

@VickyXAI VickyXAI commented Aug 17, 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 model visibility counts to reflect one additional visible model.
    • Increased the visible video model count from 7 to 8.

…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 17, 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: 40ac71c9-4d8f-4bb8-bd9e-871220e83315

📥 Commits

Reviewing files that changed from the base of the PR and between 334bd97 and b304223.

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

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


📝 Walkthrough

Walkthrough

The pull request updates brand-numbers.json. It increases totalVisible from 93 to 94 and video from 7 to 8. Other model visibility counts remain unchanged.

Changes

Model visibility counts

Layer / File(s) Summary
Refresh visibility totals
brand-numbers.json
Updates totalVisible and video counts. Keeps free, freeWithheld, and image unchanged.

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

Merge Risk: ⚪ Minimal · up to b3042

This is a narrowly scoped snapshot refresh with no actionable merge-blocking risk remaining after normal checks and review.

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 refreshing the brand snapshot to correct stale catalog data, which matches the pull request objectives.
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
🧪 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.

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