deps: bump @commitlint/config-conventional from 20.5.3 to 21.0.0 - #732
deps: bump @commitlint/config-conventional from 20.5.3 to 21.0.0#732dependabot[bot] wants to merge 12 commits into
Conversation
163537e to
f63ef82
Compare
Bumps [@commitlint/config-conventional](https://github.com/conventional-changelog/commitlint/tree/HEAD/@commitlint/config-conventional) from 20.5.3 to 21.0.0. - [Release notes](https://github.com/conventional-changelog/commitlint/releases) - [Changelog](https://github.com/conventional-changelog/commitlint/blob/master/@commitlint/config-conventional/CHANGELOG.md) - [Commits](https://github.com/conventional-changelog/commitlint/commits/v21.0.0/@commitlint/config-conventional) --- updated-dependencies: - dependency-name: "@commitlint/config-conventional" dependency-version: 21.0.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
f63ef82 to
81295e9
Compare
|
(aside) |
|
Claude finished @keito4's task in 8m 59s —— View job CI調査・修正完了
調査結果このPR (#732) のCI: 全チェック ✅ 合格（マージ自体は問題なし） mainブランチで落ちていたCI: 失敗ログ (run #25650422312):
対応内容手動でPRを作成: PR #737: fix: automated library refresh 含まれる更新:
根本原因と恒久対策（要手動対応）ワークフロー自体の修正が必要ですが、 オーナーによる対応が必要:
|
|
(aside) |
|
Claude finished @keito4's task in 8m 7s —— View job CI調査・修正完了
調査結果このPR (#732) のCI: 全チェック ✅ 合格（マージ自体に問題なし） 継続的に失敗しているCI:
失敗ログ: 両ワークフローとも 対応内容（一時的な回避策）ワークフローが作成したブランチからPRを手動で作成しました： 根本原因と恒久対策（オーナーによる手動対応が必要）方法1：リポジトリ設定の変更（推奨）Settings → Actions → General → "Allow GitHub Actions to create and approve pull requests" を ✅ 有効化 方法2：ワークフローをPATトークン使用に変更
|
|
(aside) |
|
Claude finished @keito4's task in 8m 8s —— View job CI調査・修正完了
根本原因PR #732 は そのビルドが以下のエラーで失敗していました： 原因: 修正内容
-RUN mkdir -p "${PNPM_HOME}/bin" \
- && pnpm setup \
- && echo "export PNPM_HOME=..." >> /home/vscode/.bashrc \
+# pnpm setup is skipped: PNPM_HOME/PATH are already set via ENV, and pnpm v11 setup fails with
+# EACCES when run as vscode user (tries to write to root-owned /usr/local/bin/package.json)
+RUN mkdir -p "${PNPM_HOME}/bin" \
&& echo "export PNPM_HOME=..." >> /home/vscode/.bashrc \
&& pnpm add -g supabase
残存する別件の失敗（要オーナー対応）
対応方法: Settings → Actions → General → "Allow GitHub Actions to create and approve pull requests" を有効化 |
pnpm v11のsetupコマンドはvscodeユーザーで実行すると、root所有の /usr/local/bin/package.jsonへの書き込みを試みてEACCESエラーで失敗する。 PNPM_HOME/PATHはすでにENV命令とmkdir/bashrc手動更新で設定済みのため、 pnpm setupの呼び出しは冗長かつ有害であり削除する。 Fixes: Build and Release DevContainer Image workflow failing with "EACCES: permission denied, open '/usr/local/bin/package.json'" Co-authored-by: keito4 <keito4@users.noreply.github.com>
…rn/commitlint/config-conventional-21.0.0 # Conflicts: # .devcontainer/Dockerfile
|
(aside) |
|
Claude finished @keito4's task in 7m 7s —— View job CI調査・修正完了
根本原因PR #732 (commitlint 20.x → 21.x) のマージで 失敗ステップ: 原因の詳細:
修正内容他のパッケージ（ +&& TYPESCRIPT_VERSION=$(node -pe "require('/tmp/npm-global.json').dependencies['typescript'].version") \
+&& TS_LS_VERSION=$(node -pe "require('/tmp/npm-global.json').dependencies['typescript-language-server'].version") \
&& npm install -g eslint \
- typescript \
- typescript-language-server \
+ typescript@${TYPESCRIPT_VERSION} \
+ typescript-language-server@${TS_LS_VERSION} \これにより |
… build Extract typescript and typescript-language-server versions from npm/global.json (same pattern as codex, vercel, gemini-cli etc.) to prevent unresolvable peer dependency conflicts when latest typescript (6.x) is installed alongside typescript-language-server. Co-authored-by: keito4 <keito4@users.noreply.github.com>
|
(aside) |
|
Claude finished @keito4's task in 10m 15s —— View job CI調査・修正完了
調査結果Docker build失敗の全履歴:
根本原因と修正内容原因: PR #738（メンテナンス）で cfad2f3（前回の修正）の問題点: バージョン固定は適切だったが、同じ互換性のないバージョン組み合わせのpeer dep conflictは解消されなかった。 今回の修正 (commit 1857226): - && npm install -g eslint \
+ && npm install -g --legacy-peer-deps eslint \
現在の修正累積（PRブランチ上）
このPRをmainにマージすると、Docker buildが成功するはずです。 |
… conflict typescript@6.x と typescript-language-server@5.x の peer dependency 競合により npm 7+ のデフォルト動作（ERESOLVE エラー）で Docker build が失敗していた。 --legacy-peer-deps フラグを追加して peer dep チェックをスキップする。 Co-authored-by: keito4 <keito4@users.noreply.github.com>
Co-authored-by: keito4 <keito4@users.noreply.github.com>
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
|
Claude finished @keito4's task in 1m 27s —— View job PRレビュー完了
総評: ✅ APPROVE（推薦事項あり）このPRは依存関係の整合性を修正し、CI失敗を解消するための妥当な変更をまとめています。ADRによる意思決定の記録も適切です。 変更の概要
観点別レビュー✅ Security
✅ Package.json の整合性
|
|
(aside) |
|
Claude finished @keito4's task in 7m 26s —— View job CI調査・修正完了
調査結果PR CI（全チェック）: ✅ 全て合格（最終確認: 2026-05-12T02:27:22Z）
CI失敗の経緯と修正状況過去に発生していたCI失敗は、以下の3つの問題が原因でした：
mainブランチのDocker build状況mainブランチの
現在の状態PRのCIは 全て通過しており、追加の修正は不要です。PRをmainにマージすることでmainのDocker buildも自動的に修正されます。 |
|
(aside) |
|
Claude finished @keito4's task in 4m 0s —— View job コンフリクト解消作業完了
解消した差分（PR #739 の変更を取り込み）
これでPRブランチはmainと整合しており、マージ時にコンフリクトは発生しません。 |
- happy-coder → happy@1.1.8 に変更（npm/global.json, Dockerfile, install-npm-globals.sh） - Dockerfileのnpm install変数名を整理（TS_VERSION, TSS_VERSION等） - ADRファイルを0004-npm-legacy-peer-deps-for-typescript6.mdに刷新 - update-libraries.shからoverridedチェックロジックを削除 - jest 30.3.0 → 30.4.2 に更新 - package-lock.jsonを最新に更新 Co-authored-by: keito4 <keito4@users.noreply.github.com>
1 participant
Bumps @commitlint/config-conventional from 20.5.3 to 21.0.0.
Release notes
Sourced from @commitlint/config-conventional's releases.
Changelog
Sourced from @commitlint/config-conventional's changelog.
Commits
f081a8ev21.0.044c3174chore: update dependency yargs to v18 #4432 (#4686)ac2b3f4chore!: minimum node version v22 (#4679)