Skip to content

chore(deps): update dependency @eslint/js to v10 - #28

Merged
stevendborrelli merged 2 commits into
mainfrom
ci/eslint-js-v10
Aug 27, 2026
Merged

chore(deps): update dependency @eslint/js to v10#28
stevendborrelli merged 2 commits into
mainfrom
ci/eslint-js-v10

Conversation

@stevendborrelli

Copy link
Copy Markdown
Member

Replaces #21, which was red on lint.

The bump itself is unchanged, rebased on current main. ESLint 10 promotes preserve-caught-error into the core rule set, and it flags the two rethrow sites in src/resource/resource.tsmustStructObject and mustStructJSON — for building a new Error from the caught one's message and then discarding the original.

Fixed rather than disabled: passing { cause: error } keeps the original stack attached, which is strictly better than the string interpolation that was there.

Verified locally

npm ci, npm run build, npm run typecheck, npm run lint, npm run format:check, npm test (116 passed), and protobuf-check all pass.

🤖 Generated with Claude Code

renovate Bot and others added 2 commits August 27, 2026 14:25
ESLint 10 promotes `preserve-caught-error` into core, which flags the two
rethrow sites in mustStructObject and mustStructJSON for discarding the
original error. Pass the caught error through as `{ cause }` so the stack
survives.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Signed-off-by: Steven Borrelli <steve@borrelli.org>
@stevendborrelli
stevendborrelli merged commit ec56ec1 into main Aug 27, 2026
7 checks passed
@stevendborrelli
stevendborrelli deleted the ci/eslint-js-v10 branch August 27, 2026 13:27
@stevendborrelli stevendborrelli mentioned this pull request Aug 27, 2026
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.

1 participant