Audit HPC-AI model catalogue - #1840
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
|
Warning Review limit reachedNext included review available in 54 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (26)
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. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8a1ebbb513
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "zero_data_retention": "optional", | ||
| "residency_source_url": "https://www.hpc-ai.com/agreement/privacy", | ||
| "prompt_training_policy": "no_train", | ||
| "prompt_training_notes": "Paid MaaS/API customer content is not used to train or fine-tune shared foundation models by default unless the customer opts in or law requires it. Operational logs may be retained for up to 30 days; eligible customers can request stricter controls including zero retention on supported endpoints.", |
There was a problem hiding this comment.
Populate the canonical retention-days field
Although this now records a 30-day operational retention limit, it leaves data_retention_days absent and places the fact only in training notes. The importer therefore writes null, and the provider table, retention filters, and model pricing cards classify HPC-AI's retention as unknown instead of 30 days; add "data_retention_days": 30 alongside the policy fields.
AGENTS.md reference: packages/data/catalog/AGENTS.md:L7-L10
Useful? React with 👍 / 👎.
| "note": null, | ||
| "match": [], | ||
| "priority": 100, | ||
| "effective_from": "2026-08-24T00:00:00Z", |
There was a problem hiding this comment.
Preserve the existing pricing window
For pricing lookups before 2026-08-24, this removes the previously available Opus 4.7 rates even though all three prices are unchanged: the importer formerly defaulted their absent effective_from to 1970, but now creates SKUs that start only on the audit date. The same regression occurs in the unchanged-rate MiniMax M2.5 and Kimi K2.5/K2.7 files, so historical catalogue queries between their availability and this audit return no applicable price; retain the original unbounded window when only provenance changed.
Useful? React with 👍 / 👎.
Summary
Validation
pnpm validate:datapnpm validate:pricingpnpm validate:gatewaygit diff --cached --checkCreated with Codex