Skip to content

chore: remove unused runtime catalog entries - #1039

Merged
ryoppippi merged 1 commit into
mainfrom
codex/remove-unused-catalog-deps
May 17, 2026
Merged

chore: remove unused runtime catalog entries#1039
ryoppippi merged 1 commit into
mainfrom
codex/remove-unused-catalog-deps

Conversation

@ryoppippi

@ryoppippi ryoppippi commented May 17, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Removes stale runtime catalog entries from pnpm-workspace.yaml that are no longer referenced by any package manifest. The lockfile was already aligned with the active catalog entries, so no lockfile update was needed.

Validation:

  • pnpm install --lockfile-only
  • pnpm run format
  • pnpm typecheck
  • pnpm run test

Summary by cubic

Removed unused runtime catalog entries from pnpm-workspace.yaml to match actual dependency usage and reduce noise. No lockfile changes needed.

Written for commit 8d896f4. Summary will update on new commits. Review in cubic

Summary by CodeRabbit

  • Chores
    • Updated workspace build catalog configuration, consolidating and reorganizing dependency versions.

Review Change Stack

@ryoppippi
chore: remove unused runtime catalog entries
8d896f4 
Remove stale runtime catalog entries from pnpm-workspace.yaml that are no longer referenced by any package manifest. The lockfile was already resolved without these catalog entries, so this keeps the workspace catalog aligned with actual dependency usage.
@coderabbitai

coderabbitai Bot commented May 17, 2026 •
edited
Loading

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2db51633-40f5-4933-a1f6-bf95c02820d7

📥 Commits

Reviewing files that changed from the base of the PR and between 5f7de81 and 8d896f4.

📒 Files selected for processing (1)
  • pnpm-workspace.yaml
💤 Files with no reviewable changes (1)
  • pnpm-workspace.yaml
📝 Walkthrough

Walkthrough

The PR updates the pnpm-workspace.yaml file to restructure the catalogs.build section. Several previously listed build-time packages are removed, while a smaller, focused set of runtime and CLI-related dependencies is retained and organized, including utilities for terminal control, regex handling, stdin input, CLI spinners, and type validation.

Changes

Build Catalog Restructuring

Layer / File(s) Summary
Build catalog restructuring
pnpm-workspace.yaml		 The catalogs.build dependency mappings are consolidated: multiple packages (@ryoppippi/limo, @std/async, cli-table3, consola, es-toolkit, fast-sort, etc.) are removed from the build catalog, while a focused set of runtime and CLI utilities (ansi-escapes, arkregex, get-stdin, gunshi, picospinner, type-fest, valibot, xdg-basedir) is retained and explicitly versioned.

Possibly related PRs

  • ryoppippi/ccusage#660: Modifies pnpm-workspace.yaml to remove consola from the top-level catalog, which overlaps with this PR's removal from catalogs.build.
  • ryoppippi/ccusage#631: Introduces and structures the workspace catalogs setup, providing the foundation for the catalog reorganization in this PR.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hopped through deps with care,
Pruning clutter here and there,
Build catalogs now lean and bright,
Essential tools kept in sight! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: removing unused runtime catalog entries from pnpm-workspace.yaml configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/remove-unused-catalog-deps

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented May 17, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs		 ccusage-guide 8d896f4 Commit Preview URL

Branch Preview URL		 May 17 2026, 06:16 PM

@pkg-pr-new

pkg-pr-new Bot commented May 17, 2026

Copy link
Copy Markdown

Open in StackBlitz

npx https://pkg.pr.new/ryoppippi/ccusage@1039

commit: 8d896f4

@github-actions

github-actions Bot commented May 17, 2026

Copy link
Copy Markdown
Contributor

ccusage performance comparison

This compares the PR build against the base branch build on the same CI runner.

Committed fixture performance

Committed small fixtures for stable PR-to-PR feedback and explicit Claude/Codex command coverage.

Fixtures: Claude apps/ccusage/test/fixtures/claude (0.00 MiB, 2 files), Codex apps/ccusage/test/fixtures/codex (0.00 MiB, 1 files)
Runtime: package ccusage bin from apps/ccusage/package.json through bun -b, --offline --json, measured by hyperfine with 2 warmups and 7 runs.

Command Input Base median PR median PR vs base Base throughput PR throughput
claude daily --offline --json 0.00 MiB 55.6ms 56.7ms 0.98x 0.03 MiB/s 0.03 MiB/s
claude session --offline --json 0.00 MiB 55.7ms 56.3ms 0.99x 0.03 MiB/s 0.03 MiB/s
codex daily --offline --json 0.00 MiB 56.2ms 56.3ms 1.00x 0.02 MiB/s 0.02 MiB/s
codex session --offline --json 0.00 MiB 55.5ms 56.7ms 0.98x 0.02 MiB/s 0.02 MiB/s

Large real-world-shaped fixture performance

Generated fixtures shaped from aggregate local log statistics: thousands of JSONL files, many small sessions, and a long tail of larger sessions. No real prompts, paths, or outputs are stored in the fixtures.

Fixtures: Claude /home/runner/work/_temp/ccusage-large-fixture (1.01 GiB, 2,597 files), Codex /home/runner/work/_temp/ccusage-large-codex-fixture (1.01 GiB, 2,597 files)
Runtime: package ccusage bin from apps/ccusage/package.json through bun -b, --offline --json, measured by hyperfine with 0 warmups and 1 runs.

Command Input Base median PR median PR vs base Base throughput PR throughput
claude --offline --json 1.01 GiB 1.417s 1.432s 0.99x 727.63 MiB/s 719.92 MiB/s
codex --offline --json 1.01 GiB 1.719s 1.755s 0.98x 599.72 MiB/s 587.44 MiB/s

Package size

Package artifact Base PR Delta Ratio
packed ccusage-*.tgz 67.75 KiB 67.75 KiB +0.00 KiB 1.00x

Lower medians and smaller packed package sizes are better. CI runner noise still applies; use same-run ratios as directional PR feedback, not release guarantees.

@ryoppippi
ryoppippi merged commit 4ef471e into main May 17, 2026
25 checks passed
@ryoppippi
ryoppippi deleted the codex/remove-unused-catalog-deps branch May 17, 2026 18:24
@coderabbitai coderabbitai Bot mentioned this pull request May 19, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

See More

Successfully merging this pull request may close these issues.

1 participant

@ryoppippi