Skip to content

chore(core): update models pricing snapshot - #490

Merged
szanata merged 1 commit into
mainfrom
chore/take-models-pricing-snapshot
Nov 2, 2026
Merged

szanata merged 1 commit into
mainfrom
chore/take-models-pricing-snapshot

Conversation

@GrowthX-Team

Copy link
Copy Markdown
Collaborator

Automated changes by create-pull-request GitHub action

@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

PR review

Verdict

✅ PASS

Findings

  1. Nice-to-have (Quality): This PR's only change is _meta.createdAt; no pricing data moved. buildPricing in sdk/core/scripts/take_models_pricing_snapshot.js always stamps a fresh timestamp, so the weekly cron opens a review-requiring PR even when models.dev is unchanged. Consider comparing the non-_meta content against the committed snapshot and skipping the write when identical.

Categories

  • Design: ✅ PASS
  • Quality: ✅ PASS
  • Correctness: ✅ PASS
  • Documentation: ✅ PASS
  • Changeset: ✅ PASS
  • Tests: ✅ PASS
  • Security: ✅ PASS
  • Compatibility: ✅ PASS

@szanata
szanata merged commit 06b6297 into main Nov 2, 2026
7 checks passed
@szanata
szanata deleted the chore/take-models-pricing-snapshot branch November 2, 2026 11:17
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.

2 participants