Skip to content

fix: update development tools - #727

Merged
keito4 merged 6 commits into
mainfrom
fix/update-dev-tools
May 10, 2026
Merged

fix: update development tools#727
keito4 merged 6 commits into
mainfrom
fix/update-dev-tools

Conversation

@claude

@claude claude Bot commented May 9, 2026

Copy link
Copy Markdown
Contributor

開発ツールのバージョン更新

  • GitHub CLI: `2.91.0` → `2.92.0`
  • pnpm: `10.33.2` → `11.0.9`
  • Claude Code: `2.1.121` → `2.1.137`

📝 リリースノート


Auto-generated by: update-dev-tools.yml

⚠️ Note: Update Development Tools ワークフローが PR 作成に失敗したため、手動で作成しました。
根本原因: リポジトリ設定の "Allow GitHub Actions to create and approve pull requests" が無効になっています。
修正手順: GitHub → Settings → Actions → General → Workflow permissions → "Allow GitHub Actions to create and approve pull requests" を有効化

@keito4

keito4 commented May 10, 2026

Copy link
Copy Markdown
Owner

(aside)
@claude PRがbaseにマージする際にCIが落ちています。
修正してください。

@claude

claude Bot commented May 10, 2026

Copy link
Copy Markdown
Contributor Author

Claude finished @keito4's task in 3m 19s —— View job


CI修正完了

  • CIの失敗状況を確認
  • 原因特定:pnpm v11のbreaking change
  • Dockerfileを修正
  • 修正をプッシュ

🔍 原因

Container Security Scan ジョブが Docker ビルドの step #13 で失敗していました。

エラーメッセージ:

[ERROR] The configured global bin directory "/home/vscode/.local/share/pnpm/bin" is not in PATH

根本原因: pnpm v10 → v11 でグローバル bin ディレクトリが変更されました。

  • v10: $PNPM_HOME (例: /home/vscode/.local/share/pnpm)
  • v11: $PNPM_HOME/bin (例: /home/vscode/.local/share/pnpm/bin)

🛠 修正内容

.devcontainer/Dockerfile の2箇所を修正しました:

# Before
ENV PATH="${PNPM_HOME}:${PATH}"
...
echo "export PATH=\"\$PNPM_HOME:\$PATH\"" >> /home/vscode/.bashrc

# After
ENV PATH="${PNPM_HOME}/bin:${PATH}"
...
echo "export PATH=\"\$PNPM_HOME/bin:\$PATH\"" >> /home/vscode/.bashrc

pnpm setup が非インタラクティブシェルでスキップされた場合でも、Docker ENV が正しい bin パスを保証します。

pnpm v10→v11でグローバルbinディレクトリが変更された。
- v10: $PNPM_HOME
- v11: $PNPM_HOME/bin

ENV PATHおよび.bashrcへのexportを$PNPM_HOME/binに更新。

Co-authored-by: keito4 <keito4@users.noreply.github.com>
@keito4

keito4 commented May 10, 2026

Copy link
Copy Markdown
Owner

(aside)
@claude PRがbaseにマージする際にCIが落ちています。
修正してください。

@claude

claude Bot commented May 10, 2026

Copy link
Copy Markdown
Contributor Author

Claude finished @keito4's task in 16m 2s —— View job


CI修正完了

  • タスクリスト作成
  • 現在のCI失敗状況を確認
  • 失敗原因を特定
  • 追加改善を実施してプッシュ

🔍 調査結果

Container Security Scan が commit cb9837b に対して失敗していた根本原因を特定しました。

エラーログ(旧コミット):

[ERROR] The configured global bin directory "/home/vscode/.local/share/pnpm/bin" is not in PATH
Run "pnpm setup" to update your shell configuration.
exit code: 1

根本原因(2層構造):

  1. pnpm v11のPATH変更: v10は $PNPM_HOME、v11は $PNPM_HOME/bin を bin ディレクトリとして使用
  2. 非インタラクティブシェルの問題: bash -c "source /home/vscode/.bashrc && pnpm add" では .bashrc の interactive guard により PATH 更新が無効化される

🛠 修正内容(全コミット合計)

mainとの差分:

-ARG PNPM_VERSION=10.33.2
+ARG PNPM_VERSION=11.0.9

-ENV PATH="${PNPM_HOME}:${PATH}"
+ENV PATH="${PNPM_HOME}/bin:${PATH}"   # ① v11対応: /bin 追加

-RUN bash -c "pnpm setup" \
+# pnpm v11: global bin dir changed to $PNPM_HOME/bin; create explicitly
+RUN mkdir -p "${PNPM_HOME}/bin" \     # ② bin ディレクトリを明示作成
+ && pnpm setup \                       # ③ bash -c ラッパー不要
  && echo "export PATH=\"\$PNPM_HOME/bin:\$PATH\"" >> .bashrc \
- && bash -c "source .bashrc && pnpm add -g supabase"
+ && pnpm add -g supabase               # ④ Docker ENV の PATH を直接使用

⚠️ CIについて

プッシュ後に Container Security Scan が自動トリガーされるか確認してください。GitHub Actions から push した場合に CI が自動トリガーされない制約があるため、必要に応じて以下を実施してください:

  • GitHub Actions タブから手動で "Container Security Scan" を再実行
  • または PR の "Re-run failed jobs" ボタンを使用
    | PR Branch

github-actions Bot and others added 2 commits May 10, 2026 03:11
非インタラクティブシェルで .bashrc を source すると interactive guard により
PATH が更新されず、pnpm add -g supabase が失敗していた。

bash -c "source .bashrc && pnpm add" の代わりに、Docker ENV で設定済みの
PATH を直接使用する方式に変更することで問題を解消する。

Co-authored-by: keito4 <keito4@users.noreply.github.com>
pnpm v11では $PNPM_HOME/bin がグローバルバイナリディレクトリとして
使用されるが、pnpm setupがシンボリックリンク作成に失敗した場合
ディレクトリが存在しない可能性がある。

- mkdir -p でビルド前にディレクトリを保証
- bash -c ラッパーを削除して直接コマンド実行に簡略化
- source .bashrc の不要な呼び出しを削除(Docker ENVで設定済み)

Co-authored-by: keito4 <keito4@users.noreply.github.com>

# Conflicts:
#	.devcontainer/Dockerfile
@keito4

keito4 commented May 10, 2026

Copy link
Copy Markdown
Owner

(aside)
@claude PRがbaseにマージする際にCIが落ちています。
修正してください。

@claude

claude Bot commented May 10, 2026

Copy link
Copy Markdown
Contributor Author

CI修正作業中

  • 現在のCI失敗状況を確認
  • 失敗原因を特定(重大な問題を発見)
  • リモートブランチの修正
  • CIを再トリガー
  • CI結果を確認

🔍 根本原因(3層構造)

  1. pnpm v11のPATH変更: $PNPM_HOME$PNPM_HOME/bin にグローバルbinディレクトリが変更
  2. CIが再実行されていなかった: claude[bot]のpushはCIをトリガーしない
  3. リモートブランチが別状態に強制上書きされていた(最重要):
    • Remote HEAD: c273054 (pnpm修正なしの旧Dockerfile)
    • Local HEAD: f55b31d (pnpm修正ありの正しいDockerfile)
    • → CIはc273054に対して実行されていたため、修正が適用されていなかった

🛠 対応中

リモートブランチを正しい状態(f55b31d)に更新してCIを再トリガー中...

View job run | PR Branch

@github-actions
github-actions Bot force-pushed the fix/update-dev-tools branch from f55b31d to c273054 Compare May 10, 2026 06:02
@claude claude Bot closed this May 10, 2026
@claude claude Bot reopened this May 10, 2026
@github-actions
github-actions Bot force-pushed the fix/update-dev-tools branch from c273054 to f55b31d Compare May 10, 2026 06:12
@claude claude Bot closed this May 10, 2026
@claude claude Bot reopened this May 10, 2026
@keito4

keito4 commented May 10, 2026

Copy link
Copy Markdown
Owner

(aside)
@claude PRがbaseにマージする際にコンフリクトが発生しています。最新のbaseからpullし、修正してください。

@claude

claude Bot commented May 10, 2026

Copy link
Copy Markdown
Contributor Author

Claude finished @keito4's task in 13m 24s —— View job


コンフリクト解消完了 ✅

  • タスクリスト作成
  • 現在のブランチ状況を確認
  • 最新の main をフェッチ(full history含む)
  • コンフリクト分析(根本原因特定)
  • mainブランチのdocsファイルを取り込み(6ba56a9)
  • git pull --no-rebase でマージを実行
  • Dockerfileのコンフリクトを手動解消
  • マージコミットをプッシュ(4df55ec)
  • GitHub マージ可能状態を確認:mergeable: true

🔍 コンフリクトの根本原因

PR #729(docs: ドキュメント最新化)が pnpmセクションの異なる修正を含んでいたため、本PRとの間でコンフリクトが発生していました。

PR #729 の変更(main) 本PR(fix/update-dev-tools)
pnpm PATH $PNPM_HOME(/binなし、v10用) $PNPM_HOME/bin(v11対応)
pnpm setup mkdir -p "${PNPM_HOME}" mkdir -p "${PNPM_HOME}/bin"
pnpm version 10.33.2 11.0.9

✅ 最終的なDockerfileの状態

ARG GH_CLI_VERSION=2.92.0        # 更新: 2.91.0 → 2.92.0
ARG PNPM_VERSION=11.0.9          # 更新: 10.33.2 → 11.0.9
ARG CLAUDE_CODE_VERSION=2.1.137  # 更新: 2.1.121 → 2.1.137

ENV PATH="${PNPM_HOME}/bin:${PATH}"  # pnpm v11対応
RUN mkdir -p "${PNPM_HOME}/bin" \    # v11: binディレクトリを明示作成
 && pnpm setup \
 && ...

PRはマージ可能な状態です(CIの通過を待っています)。
| PR Branch

github-actions Bot and others added 2 commits May 10, 2026 09:10
PR #729 (docs: ドキュメント最新化) でmainにマージされた変更を取り込む。
Dockerfileは本PRの変更(ツールバージョン更新 + pnpm v11対応)を保持。

Co-authored-by: keito4 <keito4@users.noreply.github.com>
コンフリクト箇所: .devcontainer/Dockerfile のpnpm設定セクション

解決方針:
- pnpm v11.0.9に対応するため、$PNPM_HOME/bin (PRブランチ版) を採用
- mainブランチの $PNPM_HOME (スラッシュなし) は pnpm v10用のため不採用
- ツールバージョンはPRブランチの更新版を維持:
  - GitHub CLI: 2.92.0
  - pnpm: 11.0.9
  - Claude Code: 2.1.137

Co-authored-by: keito4 <keito4@users.noreply.github.com>
@keito4
keito4 merged commit 9ed97e3 into main May 10, 2026
1 check passed
@keito4
keito4 deleted the fix/update-dev-tools branch May 10, 2026 09:35
github-actions Bot added a commit that referenced this pull request May 11, 2026
pnpm v11 の `pnpm setup` コマンドが vscode ユーザーで実行された際に
`/usr/local/bin/package.json` への書き込みを試みて EACCES エラーが発生する問題を修正。

- `pnpm setup` を削除(PNPM_HOME/PATH は ENV と bashrc で既に設定済み)
- PNPM_HOME ディレクトリの mkdir -p は維持(pnpm v11 で必要)

Fixes: docker-image.yml の断続的ビルド失敗 (PR #734, #727)

Co-authored-by: keito4 <keito4@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.112.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions Bot added the released リリース済み label May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released リリース済み

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant