Skip to content

[WIP] Bump @eslint/js from 9.39.2 to 10.0.1#534

Closed
lachieh with Copilot wants to merge 3 commits into
dependabot/npm_and_yarn/eslint/js-10.0.1from
copilot/sub-pr-533
Closed

[WIP] Bump @eslint/js from 9.39.2 to 10.0.1#534
lachieh with Copilot wants to merge 3 commits into
dependabot/npm_and_yarn/eslint/js-10.0.1from
copilot/sub-pr-533

Conversation

Copilot AI commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

Rebase on dependabot/npm_and_yarn/eslint/js-10.0.1

This PR successfully rebases onto the dependabot ESLint upgrade branch (commit 2417de5) and fixes the resulting linting errors.

Plan:

  • Fetch and analyze the dependabot branch
  • Understand current commits and their changes
  • Reset to dependabot base commit (2417de5)
  • Apply only linting fixes (add {cause: error} to 5 error throws)
  • Verify linting passes across entire project
  • Run type checking
  • Build and test the full project
  • Request code review
  • Run security checks

Changes Made:

Fixed 5 preserve-caught-error linting violations in packages/lattice-client-core/src/connection/nats-ws-lattice-connection.ts:

  1. Line 77: connect() - Added {cause: error} when throwing connection failure
  2. Line 95: request() - Added {cause: error} when throwing request failure
  3. Line 146: getBucketKeys() - Added {cause: error} when throwing bucket keys retrieval failure
  4. Line 168: getBucketEntry() - Added {cause: error} when throwing bucket entry retrieval failure
  5. Line 197: getBucketEntries() - Added {cause: error} when throwing bucket entries retrieval failure

Summary: Only 1 file changed, 5 insertions (+), 1 deletion (-)

This preserves the error chain for better debugging and satisfies the ESLint 10.0.1 preserve-caught-error rule.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 11, 2026 18:15
Co-authored-by: lachieh <1687902+lachieh@users.noreply.github.com>
…t default

Co-authored-by: lachieh <1687902+lachieh@users.noreply.github.com>
@lachieh

lachieh commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

@copilot try again

@lachieh

lachieh commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

Closing in favor of new attempt on #554

@lachieh lachieh closed this Feb 23, 2026
Copilot stopped work on behalf of lachieh due to an error February 23, 2026 16:57
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