Repository navigation
build(nix): add ccusage to the dev shell - #592
Conversation
`quest usage` prices sessions with ccusage and reports accuracy alone without it, so the model check had no cost data. Pinning it with the lock keeps its JSON in step with the parser. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 26 minutes. View limit details
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. |
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed head: 425152b3bd491f0968364d2a4c95e4fe7033c7e8.
No actionable defects found in the two-line flake.nix change. Adding ccusage to the existing locked dev shell fits quest usage's current PATH-based invocation and keeps the dependency scoped to development tooling.
Verification: inspected the complete diff, flake configuration, lockfile, and runCcusage/parseCcusage. This was a GitHub-only static review; Nix evaluation and runtime/tests were not run. The upstream nixpkgs package-definition read returned HTTP 429, so its version and platform support were not independently verified. Current Check/Queue runs are skipped; the PR body's reported local checks were not reproduced.
(written by unknown)

quest usageprices agent sessions with ccusage fromPATH, and without it reports accuracy alone, so m0's model check (moq-dev/quest#584) had no cost data. Anthropic's model guide compares models by cost per finished task, which m1's check now needs to retune the routing tables (moq-dev/quest#590).Changes
ccusage(20.0.37 at the locked nixpkgs), so the model check, which runs in this checkout, gets it. Pinning it withflake.lockkeeps itssession --jsonformat in step withparseCcusage, which fails on any field it lacks; a globalnpm install -g ccusagefloats.Public API
None.
quest usage's install hint for hosts withoutccusageis unchanged.Validation
nix develop --command ccusage --versionprintsccusage 20.0.37.nix develop --command bun run quest usage --since 2026-11-08 --pr 584parses and prices its sessions.just checkandjust testpass in the new shell (1183 passed, 5 skipped). No test readsccusagefromPATH; the usage tests inject it.Limitations
Models
Size: XS
Implementer: Claude Code, claude-opus-5-5, xhigh effort
🤖 Generated with Claude Code
(Written by Claude Opus 5.5 with xhigh)
Review
Rewritten by
quest reviewafter each round.01a1c261-a4af-7512-8f45-c25730b7575a, round 1The change is small and focused with no actionable defects identified by code inspection, pending runtime validation because builds and tests were not run as instructed.