fix(container): Codex review 指摘の HIGH/MED 対応

Codex 第二意見レビューで指摘された 4 件に対応:

[1.1 HIGH] 公開イメージの定期 re-pull を廃止
  docker image inspect の Created はイメージのビルド日時で
  ローカル pull 日時ではないため、age 比較で再 pull すると
  毎回 pull がループする欠陥があった。image-only サービス
  (build: なし) は不在時のみ pull し、定期再 pull は行わない
  方針に変更。「image: のみで定期 pull が必要」というユース
  ケースが出てきたら別 PR で digest 比較等を実装する。

[1.2 HIGH] DEV_SERVICE_NAME 連動の build に修正
  Python 側 _ensure_images() は get_dev_service_name() で対象
  サービスを解決していたが、Bash 側 cmd_build は
  'docker compose build dev' を固定実行していた。
  DEV_SERVICE_NAME != dev なプロジェクトで自動再ビルド経路に
  入ると存在しないサービスをビルドしようとして失敗する回帰。
  Bash 側を 'docker compose build "${DEV_SERVICE_NAME:-dev}"'
  に変更してチェックと実行のサービス名解決を整合化。

[1.4 MED] エラーメッセージを build/pull 双方を案内する文面に
  _ensure_images() 失敗時に常に "devbase container build" を
  案内していたが、image-only サービスでは build は不適切。
  build と docker pull の両方を案内する文面に変更。

[2.1 MED] cli-reference.md に自動 build/pull 動作を明記
  devbase up が条件次第で build/pull を自動実行する旨と、
  DEVBASE_IMAGE_MAX_AGE_DAYS 環境変数の存在を追記。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: takemi-ohama

bin/devbase

@@ -92,7 +92,7 @@ cmd_build() {
 
         echo ""
         echo "[2/2] Building project image..."
-        if docker compose build dev "$@"; then
+        if docker compose build "${DEV_SERVICE_NAME:-dev}" "$@"; then
             echo ""
             echo "✓ All images built successfully"
         else
@@ -126,7 +126,7 @@ cmd_build() {
 
     echo ""
     echo "[2/2] Building project image..."
-    if docker compose build dev "$@"; then
+    if docker compose build "${DEV_SERVICE_NAME:-dev}" "$@"; then
         echo ""
         echo "✓ All images built successfully"
     else

docs/user/cli-reference.md

@@ -101,6 +101,11 @@ devbase up
 
 - 起動時にスナップショットを自動作成（新世代 or 差分追加）
 - `CONTAINER_SCALE` の値に基づいてコンテナ数を決定
+- イメージの自動準備:
+  - `build:` 定義あり、イメージ未存在 → `devbase build` を自動実行
+  - `build:` 定義あり、イメージが7日以上古い → `devbase build --no-cache` で再ビルド
+  - `image:` のみ（公開イメージ）、未存在 → `docker pull` を自動実行
+  - 閾値は `DEVBASE_IMAGE_MAX_AGE_DAYS` 環境変数で上書き可能（既定 7、不正値は警告して既定値）
 
 ### `devbase container down`
 

lib/devbase/commands/container.py

@@ -105,7 +105,11 @@ def cmd_up(project_name: str = None, scale: int = None) -> int:
 
     # Pre-check 2: Ensure container images exist
     if not _ensure_images():
-        logger.error("Failed to build container images. Please run 'devbase container build' manually.")
+        logger.error(
+            "Failed to ensure container images. "
+            "Run 'devbase container build' for build-based services, "
+            "or 'docker pull <image>' for image-only services."
+        )
         return 1
 
     # Pre-step: Auto snapshot（差分世代数ベース世代管理）
@@ -440,9 +444,13 @@ def _ensure_images() -> bool:
       - Image missing + image-only (no build:) → run `docker pull`
       - Image present and >= threshold days old + has build:
         → rebuild with `--no-cache`
-      - Image present and >= threshold days old + image-only
-        → re-pull
+      - Image present + image-only → nothing to do
+        (Created reflects upstream build time, not local pull time, so we
+        cannot derive a meaningful freshness signal here. Users who want
+        public images refreshed should run `docker pull` explicitly.)
       - Otherwise: nothing to do
+
+    Returns True on success or no-op, False on failure.
     """
     compose_file = Path('compose.yml')
     if not compose_file.exists():
@@ -489,6 +497,11 @@ def _ensure_images() -> bool:
             logger.info("Container image '%s' not found, pulling...", image_name)
             return _run_pull(image_name)
 
+        # Image-only services: 'Created' reflects upstream build time, not
+        # local pull time, so age-based re-pull is not meaningful. Skip.
+        if not has_build:
+            return True
+
         max_age = _image_max_age_days()
         age_days = _get_image_age_days(inspect.stdout)
         if age_days is None or age_days < max_age:
@@ -498,11 +511,8 @@ def _ensure_images() -> bool:
             "Container image '%s' is %d days old (>= %d days threshold)",
             image_name, age_days, max_age
         )
-        if has_build:
-            logger.info("Rebuilding with --no-cache...")
-            return _run_build(no_cache=True)
-        logger.info("Re-pulling latest image...")
-        return _run_pull(image_name)
+        logger.info("Rebuilding with --no-cache...")
+        return _run_build(no_cache=True)
 
     except Exception as e:
         logger.warning("Error checking image: %s", e)