Skip to content

AI CLI設定ディレクトリの権限を修正する - #157

Merged
takemi-ohama merged 2 commits into
mainfrom
fix/etc-devbase-permissions
Sep 4, 2026
Merged

takemi-ohama merged 2 commits into
mainfrom
fix/etc-devbase-permissions

Conversation

@takemi-ohama

@takemi-ohama takemi-ohama commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

Pull Request

概要

no-cacheでbaseイメージを構築すると/etc/devbaseが0644になり、ubuntuユーザーのbash起動時にai-cli-aliases.shを読めない問題を修正します。

関連 Issue

変更点

  • aliasファイルのCOPY前に/etc/devbaseを0755で明示作成
  • BuildKitが親ディレクトリへファイル用0644を適用する退行を再現テストで固定
  • 実装計画をdocs/specifications/ai-cli-alias-loading.mdの確定仕様へ変換

動作確認

  • 修正前の再現テストがRUN ... /etc/devbase不在で失敗
  • uv run pytest -q tests/containers/test_ai_cli_aliases.py(38 passed)
  • uv run pytest -q(1793 passed)
  • Ruff、compileall、bash構文、git diff --check
  • devbase-base:hotfix-156を実ビルド
  • 実イメージで/etc/devbase=0755、aliasファイル0644、ubuntuユーザーのsource成功
  • 最新HEADのCIがgreenである

補足

稼働中ai-plugins-dev-1には暫定措置としてchmod 0755 /etc/devbaseを適用済みです。恒久反映には本PRマージ後のbaseイメージ再ビルドとコンテナ再作成が必要です。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🤖 cross-review | round 1 | codex | APPROVE

修正必須の指摘はありません。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🤖 cross-review | round 1 | agy | APPROVE

変更内容、パーミッション設定および回帰テストを確認しました。修正アクションはありません。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🤖 cross-review | round 2 | agy | APPROVE

確定仕様(docs/specifications/ai-cli-alias-loading.md)の追加および回帰テスト・実装内容を確認しました。修正が必要な点はありません。

@takemi-ohama takemi-ohama left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🤖 cross-review | round 2 | codex | APPROVE

修正アクションが必要な指摘はありません。

@takemi-ohama
takemi-ohama merged commit 84d3197 into main Sep 4, 2026
5 checks passed
@takemi-ohama takemi-ohama mentioned this pull request Sep 4, 2026
7 tasks
@takemi-ohama

Copy link
Copy Markdown
Contributor Author

リリース後テスト

対象の版: v3.2.2(2026-09-04 12:30 JST公開)
導入経路: 公開タグ v3.2.2 を取得し、devbase build base --no-cache と devbase build ai-plugins --no-cache の後に devbase up ai-plugins で再作成

受け入れ条件 実行したこと 実行時刻 結果
公開版からno-cacheビルドできる ./bin/devbase build base --no-cache 2026-09-04 12:31 JST 合格 / exit=0
/etc/devbase が 0755 新baseイメージと再作成後コンテナで stat 2026-09-04 12:39 JST 合格 / 755 root:root
aliasファイルが 0644 stat /etc/devbase/ai-cli-aliases.sh 2026-09-04 12:39 JST 合格 / 644 root:root
ubuntuユーザーがaliasを読める source /etc/devbase/ai-cli-aliases.sh と type claude 2026-09-04 12:39 JST 合格 / exit=0
Kiro認証が再作成後も残る readlink -f ~/.local/share/kiro-cli と kiro-cli whoami 2026-09-04 12:39 JST 合格 / group volume、exit=0
更新イメージで稼働する コンテナID更新と実行image IDを比較 2026-09-04 12:39 JST 合格 / running

合否: 合格(6件中6件)
起票したもの: なし

@takemi-ohama
takemi-ohama deleted the fix/etc-devbase-permissions branch September 4, 2026 03:53
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.

baseイメージの /etc/devbase が0644になりシェル起動時にPermission deniedとなる

1 participant