Skip to content

chore(deps): upgrade Rslint to v0.8.0 - #252

Open
chenjiahan wants to merge 1 commit into
mainfrom
chore/upgrade-rslint-0.8
Open

chore(deps): upgrade Rslint to v0.8.0#252
chenjiahan wants to merge 1 commit into
mainfrom
chore/upgrade-rslint-0.8

Conversation

@chenjiahan

Copy link
Copy Markdown
Member

Summary

  • Upgrade @rslint/core to v0.8.0 and refresh the lockfile where applicable.
  • Preserve the existing recommended-rule coverage under Rslint 0.8.
  • Resolve newly surfaced lint diagnostics where needed.
  • Verify the relevant lint checks pass.

Related Links

Copilot AI lite review requested due to automatic review settings August 12, 2026 10:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Upgrades the repo’s linting toolchain to Rslint v0.8.0 and adjusts configuration/source to preserve existing recommended-rule coverage while addressing newly surfaced diagnostics.

Changes:

  • Bump @rslint/core to ^0.8.0 and update pnpm-lock.yaml accordingly.
  • Expand rslint.config.ts to include JS recommended rules and add targeted overrides for tests/** and example/**.
  • Minor TypeScript changes to align with updated lint expectations (e.g., Error cause propagation; ESC handler refactor).

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/server.ts Adds Error { cause } metadata when rethrowing missing optional dependency errors.
rslint.config.ts Includes js.configs.recommended and adds per-folder rule overrides for tests/examples.
pnpm-lock.yaml Refreshes lockfile for @rslint/core@0.8.0 and related dependency updates.
package.json Updates @rslint/core dependency range to ^0.8.0.
client-src/overlay.ts Refactors ESC key handler into a function declaration to satisfy linting/usage order.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants