Skip to content

chore(deps): @book000/eslint-config を v1.14.16 にアップグレード#2313

Merged
book000 merged 2 commits intomasterfrom
fix/upgrade-eslint-config-1-14-16
May 3, 2026
Merged

chore(deps): @book000/eslint-config を v1.14.16 にアップグレード#2313
book000 merged 2 commits intomasterfrom
fix/upgrade-eslint-config-1-14-16

Conversation

@akubiusa
Copy link
Copy Markdown
Member

@akubiusa akubiusa commented Apr 25, 2026

概要

@book000/eslint-config1.14.9 から 1.14.16 にアップグレードし、新バージョンで有効化された ESLint ルールに対応しました。

変更内容

パッケージアップデート

  • @book000/eslint-config: 1.14.91.14.16

コード修正

src/manager/position-load-manager.ts

v1.14.16 で有効化された以下の ESLint ルールへの対応:

  1. @typescript-eslint/no-unnecessary-type-assertion: reject(error as Error) の不要な型アサーション (as Error) を削除
  2. @typescript-eslint/prefer-promise-reject-errors: Promise の reject には Error インスタンスを渡すことが要求されるため、error instanceof Error でチェックし、Error でない場合は new Error(String(error)) でラップするように修正
  3. docstring の追加: load メソッドに日本語の JSDoc コメントを追加(CLAUDE.md 要件に対応)

背景

Renovate が作成した PR #2312 ("chore(deps): update dependency @book000/eslint-config to v1.14.16") にて、CI の node-ci ジョブが失敗していました。Renovate PR には直接変更を加えず、新規ブランチで対応しました。

テスト結果

ローカルで pnpm run lint を実行し、以下のすべての lint チェックが通過したことを確認済みです:

  • lint:prettier - Prettier フォーマットチェック: 通過
  • lint:eslint - ESLint チェック: 通過
  • lint:tsc - TypeScript 型チェック: 通過

GitHub Actions CI も全ジョブが通過しています。

@akubiusa
Copy link
Copy Markdown
Member Author

Code review

Found 1 issue and automatically fixed it in commit 825707a:

  1. load メソッドに docstring がなかった (CLAUDE.md says "関数やインターフェースには日本語で docstring を記載・更新する")

load(server: LockMoveChannelServer): Promise<ChangedChannelPosition[]> {
return new Promise((resolve, reject) => {
const id = server.id

Fixed: load メソッドに日本語の JSDoc docstring を追加しました (commit 825707a)

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

@book000/eslint-config を v1.14.16 へ更新し、新たに有効化された ESLint ルールに合わせてコードを調整する PR です。

Changes:

  • @book000/eslint-config1.14.91.14.16 にアップグレード
  • prefer-promise-reject-errors 等に対応するため、Promise の reject に渡す値を Error に正規化
  • 依存関係更新に伴う pnpm-lock.yaml の更新(typescript-eslint 系の解決バージョン更新等)

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/manager/position-load-manager.ts reject に渡す値を Error に統一し、ESLint ルール変更に追従
package.json @book000/eslint-config のバージョン更新
pnpm-lock.yaml ロックファイルの依存解決を v1.14.16 に合わせて更新
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

akubiusa and others added 2 commits May 3, 2026 17:24
- @book000/eslint-config を 1.14.9 から 1.14.16 に更新
- v1.14.16 で有効化された `@typescript-eslint/no-unnecessary-type-assertion` に対応するため、`reject(error as Error)` の不要な型アサーションを削除
- 同時に `@typescript-eslint/prefer-promise-reject-errors` に対応するため、`error instanceof Error` チェックを追加し、Error でない場合は `new Error(String(error))` でラップするよう修正

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- load メソッドに docstring がなかった問題を修正(CLAUDE.md 要件: 「関数やインターフェースには日本語で docstring を記載・更新する」)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@akubiusa akubiusa force-pushed the fix/upgrade-eslint-config-1-14-16 branch from 825707a to d1bdba1 Compare May 3, 2026 08:24
@book000 book000 merged commit c3e3db4 into master May 3, 2026
15 checks passed
@book000 book000 deleted the fix/upgrade-eslint-config-1-14-16 branch May 3, 2026 08:39
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.

3 participants