Add Devin usage provider#1264
Conversation
|
Codex review: needs real behavior proof before merge. Reviewed June 11, 2026, 4:02 AM ET / 08:02 UTC. Summary Reproducibility: not applicable. This PR adds a new opt-in provider rather than reporting broken established behavior. Focused tests establish source behavior but do not replace proof from a real Devin account. Review metrics: 3 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:
Proof guidance:
Mantis proof suggestion Risk before merge
Maintainer options:
Next step before merge
Security Review findings
Review detailsBest possible solution: Keep the access-token-only, Chrome-only design, restore the normalized widget project reference, and merge only after maintainers accept the credential boundary and reviewers can inspect redacted live CLI or app proof from the final head. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR adds a new opt-in provider rather than reporting broken established behavior. Focused tests establish source behavior but do not replace proof from a real Devin account. Is this the best way to solve the issue? Mostly yes: the hardened access-token-only, Chrome-only path fits the existing browser-backed provider architecture, but unrelated widget metadata should be reverted and the credential boundary needs explicit maintainer approval. Full review comments:
Overall correctness: patch is correct AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 9015e94901c4. Label changesLabel changes:
Label justifications:
Evidence reviewedSecurity concerns:
What 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
|
25ebbb4 to
94a4a3e
Compare
|
Heads up on the (The Two small fixes that should clear CI and also resolve one of ClawSweeper's P2 findings:
The auth/privacy review and live proof are still on your side of the table, of course. Happy to recheck CI after you push a fix. |
Add a web-backed Devin provider with browser-session import, organization discovery, quota parsing, settings UI, widget/config registration, and regression coverage. Refresh expired Auth0 browser access tokens before retrying quota fetches and present Devin as a web source instead of a missing CLI. Closes: steipete#800
94a4a3e to
20bfc18
Compare
|
Landed as Verification:
|
Summary
auth1_sessionfrom Chrome local storage and infer the organization slug/internal ID from current Devin storage shapesFixes #800
Verification
swift test --filter DevinUsageFetcherTests(19 tests)swift test --filter ProviderIconResourcesTestsswift test --filter ProviderSettingsDescriptorTestsmake check/api/<redacted-org>/billing/quota/usagereturned HTTP 200CodexBar.app: Peekaboo selected Devin and verifiedUpdated just now, Daily, Weekly, and quota bars with no login or organization error