[codex] Add Sakana AI usage provider#1728
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 24, 2026, 9:19 AM ET / 13:19 UTC. Summary Reproducibility: not applicable. as a bug reproduction; this is a feature PR. The changed parser/fetcher behavior is source-checkable from the PR head, and the contributor supplied redacted live Sakana billing output for the central runtime path. Review metrics: 2 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land the disabled-by-default first-class provider only after maintainer sign-off on the manual-cookie and HTML-parser boundary; otherwise defer Sakana to a broader custom-provider direction. Do we have a high-confidence way to reproduce the issue? Not applicable as a bug reproduction; this is a feature PR. The changed parser/fetcher behavior is source-checkable from the PR head, and the contributor supplied redacted live Sakana billing output for the central runtime path. Is this the best way to solve the issue? Yes for the implementation shape: it follows the existing descriptor, strategy, settings, config, and test patterns and keeps the provider disabled by default. The unresolved question is product/auth acceptance of a first-class provider based on manual cookies and console HTML parsing. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against ada3660e9d61. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e9d4fd63b0
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Addressed the current P2 in
Validation:
Proof note: I checked local env/config presence without printing secret values. No Sakana cookie is available here, so I still cannot produce a real billing-page fetch proof from this machine. @codex review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
Codex Review: Didn't find any major issues. Another round soon, please! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
Addressed the live Sakana proof gap in
Redacted live behavior proofRan a temporary local proof harness against the current PR head ( No cookie values, account identifiers, invoice URLs, raw HTML, raw response bodies, IP addresses, API keys, or private endpoints are included below. {
"provider": "sakana",
"fetchSource": "web",
"billingURL": "https://console.sakana.ai/billing",
"cookieSource": "Chrome profile",
"cookieHeaderPresent": true,
"cookieCount": 3,
"planNamePresent": true,
"priceLabelPresent": true,
"fiveHourRendered": true,
"fiveHourWindowMinutes": 300,
"fiveHourUsedPercent": 0,
"fiveHourResetPresent": false,
"fiveHourResetDescriptionNil": true,
"weeklyRendered": true,
"weeklyWindowMinutes": 10080,
"weeklyUsedPercent": 32,
"weeklyResetPresent": true,
"weeklyResetDescriptionNil": true,
"identityLoginMethodPresent": true,
"timestamp": "2026-06-24T12:56:24Z"
}Validation:
Maintainer sign-off still requested: this provider depends on Sakana console HTML/Next markup rather than a documented API, and uses a manual Cookie header path. @codex review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f23516ec71
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Addressed the two new P2s in
Validation:
@codex review |
|
🦞🧹 I asked ClawSweeper to review this item again. |
|
Codex Review: Didn't find any major issues. Delightful! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
SAKANA_COOKIEand configcookieHeadersupporthttps://console.sakana.ai/billingand parse the billing page's plan label plus 5-hour and Weekly usage limit windowsOpenCLI notes
Validation
make checkswift test --filter 'Sakana|ProviderConfigEnvironmentTests|ConfigurationDocsProviderIDTests|ProviderIconResourcesTests'Risk