Skip to content

pr-cleanup.yml のパッケージ名解決とエラーハンドリングを修正#10

Merged
uidev1116 merged 1 commit into
masterfrom
fix/pr-cleanup-package-name
Jul 3, 2026
Merged

pr-cleanup.yml のパッケージ名解決とエラーハンドリングを修正#10
uidev1116 merged 1 commit into
masterfrom
fix/pr-cleanup-package-name

Conversation

@uidev1116

Copy link
Copy Markdown
Contributor

問題

PR #9 のマージ時に pr-cleanup.yml が失敗した。

Package acms-dev のバージョンから suffix '-9' を検索します...
gh: Package not found. (HTTP 404)
jq: error (at <stdin>:5): Cannot index string with string "metadata"
Error: Process completed with exit code 5.

原因

  1. パッケージ名の誤り:GHCR にプッシュしているのは ghcr.io/<owner>/acms-development-docker-images/acms-dev なので、GHCR 上のパッケージ名は acms-dev 単体ではなく acms-development-docker-images/acms-dev(リポジトリ名込みのパス)。匿名レジストリAPIで実在パスを確認して特定した。
  2. エラーハンドリング不足gh api が 404 のエラー本文(JSON オブジェクト)をそのまま jq に渡していたため、.[] がオブジェクトの値(文字列)を返し、.metadata アクセスで意味不明なクラッシュになっていた。

修正内容

  • PACKAGE_NAME${{ github.event.repository.name }}/acms-dev に変更(リポジトリ改名にも自動追従)
  • パッケージ名の / を API パスで %2F にエンコード
  • gh api の exit code を明示チェックし、失敗時は ::warning:: を出して安全にスキップ(jq クラッシュを防止)

検証

  • actionlint 通過
  • 匿名 GHCR レジストリAPI で appleple/acms-development-docker-images/acms-dev が実在パス(UNAUTHORIZED = 存在するが非公開)であることを確認。appleple/acms-dev 単体は NAME_UNKNOWN(存在しない)だったことも確認済み

補足(別途要確認)

GHCR パッケージが private の可能性が高い(GitHub の既定挙動)。PR プレビューを外部レビュアーが docker pull できる想定なら、Package の Visibility を Public に変更する必要がある。

🤖 Generated with Claude Code

- GHCR のパッケージ名は ghcr.io/<owner>/ 以降のパス全体
  (<repo>/acms-dev)になる。acms-dev 単体では 404 になっていた
  (匿名レジストリAPIで実在パスを確認して特定)。
- パッケージ名の "/" は API パスで %2F にエンコードするよう修正。
- gh api の失敗(404等)をエラー本文のまま jq に流し込んでいたため
  jq: Cannot index string with string 'metadata' という分かりにくい
  クラッシュになっていた。exit code を明示チェックし、失敗時は
  ::warning:: で理由を出して安全にスキップするよう修正。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@uidev1116 uidev1116 merged commit afbafeb into master Jul 3, 2026
2 checks passed
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