chore: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false #659

ryoppippi · 2025-09-21T16:49:38Z

Summary

Updated @ryoppippi/eslint-config from previous version to 0.4.0
Removed svelte: false from all eslint.config.js files since svelte is now disabled by default

Changes

Updated package dependency to latest version (0.4.0)
Removed redundant svelte: false configuration from:
- Root eslint.config.js
- apps/ccusage/eslint.config.js
- apps/mcp/eslint.config.js
- apps/codex/eslint.config.js
- docs/eslint.config.js
- packages/terminal/eslint.config.js
- packages/internal/eslint.config.js

Test plan

All eslint configurations updated
Run format command to verify configuration works
CI tests pass

Summary by CodeRabbit

Chores
- Updated lint configuration to @ryoppippi/eslint-config v0.4.0 and aligned workspace catalog references.
- Simplified ESLint setups across apps, packages, and docs by relying on default settings; removed explicit Svelte disable flags.
- No runtime or user-facing behavior changes; developer linting experience becomes more consistent.

coderabbitai · 2025-09-21T16:49:46Z

Warning

Rate limit exceeded

@ryoppippi has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 23 minutes and 56 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 9342a5e and e58050c.

⛔ Files ignored due to path filters (1)

pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (9)

apps/ccusage/eslint.config.js (0 hunks)
apps/codex/eslint.config.js (0 hunks)
apps/mcp/eslint.config.js (0 hunks)
docs/eslint.config.js (0 hunks)
eslint.config.js (0 hunks)
package.json (1 hunks)
packages/internal/eslint.config.js (0 hunks)
packages/terminal/eslint.config.js (0 hunks)
pnpm-workspace.yaml (1 hunks)

Note

Other AI code review bot(s) detected

CodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review.

Walkthrough

Removed the explicit svelte: false option from multiple ESLint config files and updated the workspace catalog to @ryoppippi/eslint-config ^0.4.0, with a corresponding devDependency alias tweak in package.json. No other configuration structures or exports were changed.

Changes

Cohort / File(s)	Summary of Changes
ESLint configs: remove svelte option `apps/ccusage/eslint.config.js`, `apps/codex/eslint.config.js`, `apps/mcp/eslint.config.js`, `docs/eslint.config.js`, `packages/internal/eslint.config.js`, `packages/terminal/eslint.config.js`, `eslint.config.js`	Removed `svelte: false` from the object passed to `ryoppippi`; other config content unchanged.
Workspace catalog bump `pnpm-workspace.yaml`	Updated catalogs.lint entry for `@ryoppippi/eslint-config` from `^0.3.7` to `^0.4.0`.
DevDependency alias tweak `package.json`	Changed `@ryoppippi/eslint-config` devDependency value from `"catalog:lint"` to `"catalog:"`.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

feat(docs): add ESLint configuration for documentation linting #644 — Introduced docs ESLint config with svelte option; this PR removes that option, touching the same file/setting.
chore(docs): eslint config #652 — Adjusted ryoppippi options in docs/eslint.config.js; related due to modifying the same configuration field now removed.
refactor(terminal): extract terminal utilities to dedicated @ccusage/terminal package #640 — Modified packages/terminal/eslint.config.js to include { svelte: false }; this PR removes that property in the same file.

Poem

I nibbled configs, neat and small,
Plucked “svelte: false” from every wall.
Hopped to the workspace, bumped the tag,
Catalog trails in tidy swag.
With lint winds calm and carrots bright,
This bunny’s build now squeaks just right. 🥕✨

✨ Finishing touches

📝 Generate Docstrings

🧪 Generate unit tests

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch update-eslint-config

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.}

gemini-code-assist · 2025-09-21T16:49:56Z

Summary of Changes

Hello @ryoppippi, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request focuses on modernizing the project's ESLint setup by upgrading the primary configuration package and removing unnecessary Svelte-related settings. This streamlines the linting process and ensures the project benefits from the latest features and fixes in the ESLint ecosystem. Additionally, a range of other development dependencies have been brought up to date.

Highlights

ESLint Configuration Update: The core @ryoppippi/eslint-config package has been updated to version 0.4.0.
Svelte Configuration Simplification: Redundant svelte: false configurations have been removed from all eslint.config.js files, as Svelte support is now disabled by default in the updated ESLint configuration.
Dependency Updates: Several other development dependencies, including various @typescript-eslint packages, @stylistic/eslint-plugin, @vitest/eslint-plugin, and browser-related packages like browserslist and caniuse-lite, have been updated to their latest versions.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

socket-security · 2025-09-21T16:50:00Z

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	@ryoppippi/eslint-config@0.3.7 ⏵ 0.4.0	^-1		⁺²	⁺⁷

View full report

cloudflare-workers-and-pages · 2025-09-21T16:50:04Z

Deploying with Cloudflare Workers

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

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	ccusage-guide	`e58050c`	Sep 21 2025, 04:53 PM

pkg-pr-new · 2025-09-21T16:50:29Z

Open in StackBlitz

npm i https://pkg.pr.new/ryoppippi/ccusage@659

npm i https://pkg.pr.new/ryoppippi/ccusage/@ccusage/codex@659

npm i https://pkg.pr.new/ryoppippi/ccusage/@ccusage/mcp@659

commit: e58050c

pkg-pr-new · 2025-09-21T16:50:29Z

Open in StackBlitz

npm i https://pkg.pr.new/ryoppippi/ccusage@659

npm i https://pkg.pr.new/ryoppippi/ccusage/@ccusage/codex@659

npm i https://pkg.pr.new/ryoppippi/ccusage/@ccusage/mcp@659

commit: d92b5ac

gemini-code-assist

Code Review

This pull request updates the @ryoppippi/eslint-config dependency to version 0.4.0 and cleans up the ESLint configuration files by removing the svelte: false option, which is now the default. The changes are clear and align with the stated goals. I've identified a minor issue with a redundant dependency definition in pnpm-workspace.yaml that should be addressed to improve maintainability.

gemini-code-assist · 2025-09-21T16:51:25Z

pnpm-workspace.yaml

@@ -4,6 +4,7 @@ packages:
  - packages/*

 catalog:
+  '@ryoppippi/eslint-config': 0.4.0


By moving @ryoppippi/eslint-config to the default catalog, the old entry in the lint catalog (on line 28) has become redundant. To avoid confusion and keep the configuration clean, please remove the old entry.

…te=false - Updated @ryoppippi/eslint-config in lint catalog from ^0.3.7 to ^0.4.0 - Removed svelte: false from all eslint.config.js files as it's now default false - Removed unused @ryoppippi/eslint-config entry from default catalog - Changes affect root, apps/ccusage, apps/mcp, apps/codex, docs, packages/terminal, and packages/internal

ryoppippi force-pushed the update-eslint-config branch from d109ffe to d92b5ac Compare September 21, 2025 16:49

gemini-code-assist bot reviewed Sep 21, 2025

View reviewed changes

ryoppippi force-pushed the update-eslint-config branch from d92b5ac to bd1d1f8 Compare September 21, 2025 16:52

ryoppippi force-pushed the update-eslint-config branch from bd1d1f8 to e58050c Compare September 21, 2025 16:52

ryoppippi changed the title ~~deps: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false~~ chore: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false Sep 21, 2025

ryoppippi merged commit 3ed63f9 into main Sep 21, 2025
14 of 16 checks passed

ryoppippi deleted the update-eslint-config branch September 21, 2025 16:53

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

chore: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false #659

chore: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false #659

Uh oh!

ryoppippi commented Sep 21, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Sep 21, 2025 •

edited

Loading

Rate limit exceeded

Other AI code review bot(s) detected

Uh oh!

gemini-code-assist bot commented Sep 21, 2025

Uh oh!

socket-security bot commented Sep 21, 2025 •

edited

Loading

Uh oh!

cloudflare-workers-and-pages bot commented Sep 21, 2025 •

edited

Loading

Uh oh!

pkg-pr-new bot commented Sep 21, 2025 •

edited

Loading

Uh oh!

pkg-pr-new bot commented Sep 21, 2025

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

gemini-code-assist bot Sep 21, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

chore: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false #659

chore: update @ryoppippi/eslint-config to 0.4.0 and remove svelte=false #659

Uh oh!

Conversation

ryoppippi commented Sep 21, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Test plan

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Sep 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rate limit exceeded

Other AI code review bot(s) detected

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

gemini-code-assist bot commented Sep 21, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

socket-security bot commented Sep 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cloudflare-workers-and-pages bot commented Sep 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Deploying with Cloudflare Workers

Uh oh!

pkg-pr-new bot commented Sep 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pkg-pr-new bot commented Sep 21, 2025

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Sep 21, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ryoppippi commented Sep 21, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Sep 21, 2025 •

edited

Loading

socket-security bot commented Sep 21, 2025 •

edited

Loading

cloudflare-workers-and-pages bot commented Sep 21, 2025 •

edited

Loading

pkg-pr-new bot commented Sep 21, 2025 •

edited

Loading