refactor(tui): 第2弾 — 選択ヘルパの残ラダー解消とカテゴリ定義の SSoT 化

takemi-ohama · claude · takemi-ohama · commit b1b9ad66dae4 · 2026-06-11T23:51:46.000Z
- flow.back_as_cancel 新設: 選択ヘルパに残っていた None/MENU_BACK→ARG_CANCEL
  変換ラダー 4 箇所を 1 行化 (env/_select_project, project/_select_build_image,
  snapshot/_select_snapshot_name×2, plugin/_select_name)
- app.py: TOP_CATEGORIES / _LABELS / _route の if-elif で三重持ちだった
  カテゴリ定義を _CATEGORIES 単一 SSoT から導出。routing は module.run の
  遅延解決 dict に (monkeypatch 互換)
- actions_plugin: registry 名取得 2 関数を _registry_names(lister) に統合、
  空一覧の案内+中止を _select_name へ吸収し wrapper 2 関数を薄い委譲に
- actions_env / actions_project: 引数なし操作 (sync/edit/up/rebuild) を
  dict 直書き lambda に統一 (plugin/snapshot と同形)

公開契約・プロンプト文言・順序は不変。731 passed / ruff クリーン

Co-Authored-By: Claude Fable 5 &lt;noreply@anthropic.com&gt;
diff --git a/lib/devbase/tui/actions_env.py b/lib/devbase/tui/actions_env.py
@@ -100,11 +100,9 @@ def _select_project(devbase_root: Path):
     choices = [(entry, i) for i, entry in enumerate(entries)]
     idx = menu.select(f"対象プロジェクトを選択 {menu.HINT_SEARCH}:",
                       choices, back=True, search=True)
-    if idx is None:
-        return None                    # Ctrl-C → 全体中止 (ナビ規約)
-    if idx is menu.MENU_BACK:
-        return _ARG_CANCEL             # Esc → サブメニューを再表示
-    return rows[idx]["name"]
+    if isinstance(idx, int):
+        return rows[idx]["name"]
+    return flow.back_as_cancel(idx)    # None=Ctrl-C / MENU_BACK=Esc → 再表示
 
 
 def _run_in_project(devbase_root: Path, project_name: str, fn):
@@ -213,17 +211,6 @@ def _select_scoped_project(devbase_root: Path, message: str, choices):
 # 各操作の引数収集 + dispatch (plan 2.3 契約)
 # ---------------------------------------------------------------------------
 
-def _op_sync(devbase_root: Path):
-    # 引数なし。ソースファイルから認証情報を再同期する。
-    return _dispatch(devbase_root, "sync")
-
-
-def _op_edit(devbase_root: Path):
-    # 引数なし。$DEVBASE_ROOT/.env を $EDITOR で開く (グローバル操作。
-    # plan 3.3 は CWD スコープとするが実装はグローバルのため chdir しない)。
-    return _dispatch(devbase_root, "edit")
-
-
 def _op_init(devbase_root: Path):
     # 既存設定がある場合は --reset でやり直し (既存はバックアップされる)。
     reset = flow.need(menu.confirm(
@@ -331,8 +318,11 @@ def _op_import(devbase_root: Path):
 
 
 _OP_HANDLERS = {
-    "sync": _op_sync,
-    "edit": _op_edit,
+    # sync は引数なしで即実行 (ソースファイルから認証情報を再同期する)。
+    # edit も引数なし。$DEVBASE_ROOT/.env を $EDITOR で開くグローバル操作のため
+    # chdir しない (plan 3.3 は CWD スコープとするが実装を正とする)。
+    "sync": lambda root: _dispatch(root, "sync"),
+    "edit": lambda root: _dispatch(root, "edit"),
     "init": _op_init,
     "list": _op_list,
     "set": _op_set,
diff --git a/lib/devbase/tui/actions_plugin.py b/lib/devbase/tui/actions_plugin.py
@@ -66,71 +66,59 @@ def _dispatch(devbase_root: Path, subcommand: str, **attrs) -> int:
 # 名前選択 (registry から一覧を取得して選ばせる)
 # ---------------------------------------------------------------------------
 
-def _installed_plugin_names(devbase_root: Path) -> list[str]:
-    """導入済み plugin 名の一覧を registry (plugins.yml) から取得する。"""
-    from devbase.plugin.registry import PluginRegistry
-
-    try:
-        return [p.name for p in PluginRegistry(Path(devbase_root)).list_installed()]
-    except DevbaseError as e:
-        logger.error("%s", e)
-        return []
-
+def _registry_names(devbase_root: Path, lister: str) -> list[str]:
+    """registry (plugins.yml) から名前一覧を取得する。
 
-def _repository_names(devbase_root: Path) -> list[str]:
-    """登録済みリポジトリ名の一覧を registry (plugins.yml) から取得する。"""
+    ``lister`` は ``PluginRegistry`` の一覧メソッド名 (``list_installed`` /
+    ``list_repositories``)。取得に失敗したら案内して空リストを返す。
+    """
     from devbase.plugin.registry import PluginRegistry
 
     try:
-        return [r.name for r in PluginRegistry(Path(devbase_root)).list_repositories()]
+        registry = PluginRegistry(Path(devbase_root))
+        return [item.name for item in getattr(registry, lister)()]
     except DevbaseError as e:
         logger.error("%s", e)
         return []
 
 
-def _select_name(message: str, names: list[str], *, all_label: str | None = None):
-    """名前一覧から 1 件選ばせる共通ヘルパ。
+def _select_name(message: str, names: list[str], *,
+                 all_label: str | None = None, empty_hint: str = "対象がありません。"):
+    """名前一覧から 1 件選ばせる共通ヘルパ。対象が無ければ案内して ``_ARG_CANCEL``。
 
     ``all_label`` 指定時は「全対象」(value="") を先頭に置く。選択メニューの ``None``
     (Ctrl-C → 全体中止) と衝突させないため空文字を番兵にし、``None`` への変換は
     呼び出し側で行う (_select_build_image と同じ流儀)。
 
     戻り値: 名前 (``str``) / ``""`` (all_label 選択 = 全対象。呼び出し側で ``None``
     へ変換) / ``None`` (Ctrl-C → 全体中止を呼び出し元へ伝搬) / ``_ARG_CANCEL``
-    (Esc・← → サブメニューへ戻る)。
+    (Esc・← → サブメニューへ戻る、または対象が 1 件もない)。
     """
-    choices: list[tuple[str, str]] = []
-    if all_label is not None:
-        choices.append((all_label, ""))
+    if not names:
+        logger.info("%s", empty_hint)
+        return _ARG_CANCEL
+    choices = ([(all_label, "")] if all_label is not None else [])
     choices += [(n, n) for n in names]
-    sel = menu.select(f"{message} {menu.HINT_BACK}:", choices, back=True, search=False)
-    if sel is None:
-        return None                    # Ctrl-C → 全体中止 (ナビ規約)
-    if sel is menu.MENU_BACK:
-        return _ARG_CANCEL             # Esc/← → サブメニューを再表示
-    return sel                         # "" = 全対象 (呼び出し側で None へ変換)
+    return flow.back_as_cancel(menu.select(
+        f"{message} {menu.HINT_BACK}:", choices, back=True, search=False))
 
 
 def _select_installed_plugin(devbase_root: Path, message: str, *,
                              all_label: str | None = None):
     """導入済み plugin から 1 件選ばせる。対象が無ければ案内して ``_ARG_CANCEL``。"""
-    names = _installed_plugin_names(devbase_root)
-    if not names:
-        logger.info("導入済みの plugin がありません。"
-                    "`plugin install` で導入してください。")
-        return _ARG_CANCEL
-    return _select_name(message, names, all_label=all_label)
+    return _select_name(
+        message, _registry_names(devbase_root, "list_installed"),
+        all_label=all_label,
+        empty_hint="導入済みの plugin がありません。`plugin install` で導入してください。")
 
 
 def _select_repository(devbase_root: Path, message: str, *,
                        all_label: str | None = None):
     """登録済みリポジトリから 1 件選ばせる。対象が無ければ案内して ``_ARG_CANCEL``。"""
-    names = _repository_names(devbase_root)
-    if not names:
-        logger.info("登録済みのリポジトリがありません。"
-                    "`plugin repo add` で登録してください。")
-        return _ARG_CANCEL
-    return _select_name(message, names, all_label=all_label)
+    return _select_name(
+        message, _registry_names(devbase_root, "list_repositories"),
+        all_label=all_label,
+        empty_hint="登録済みのリポジトリがありません。`plugin repo add` で登録してください。")
 
 
 # ---------------------------------------------------------------------------
diff --git a/lib/devbase/tui/actions_project.py b/lib/devbase/tui/actions_project.py
@@ -84,13 +84,9 @@ def _select_build_image(devbase_root: Path):
     # value="" を「compose.yml 全体」に割り当て、選択メニューの None (Ctrl-C =
     # 全体中止) と衝突させない。呼び出し側で空文字 → None へ変換する。
     choices = [("compose.yml 全体をビルド", "")] + [(img, img) for img in images]
-    sel = menu.select(f"ビルドするイメージを選択 {menu.HINT_BACK}:",
-                      choices, back=True, search=False)
-    if sel is None:
-        return None                    # Ctrl-C → 全体中止 (ナビ規約)
-    if sel is menu.MENU_BACK:
-        return _ARG_CANCEL             # Esc/← → サブメニューを再表示
-    return sel                         # "" = compose 全体 (呼び出し側で None へ変換)
+    return flow.back_as_cancel(menu.select(
+        f"ビルドするイメージを選択 {menu.HINT_BACK}:",
+        choices, back=True, search=False))
 
 
 # ---------------------------------------------------------------------------
@@ -133,6 +129,10 @@ def _op_build(devbase_root: Path, name: str):
 
 
 _OP_HANDLERS = {
+    # up/rebuild は引数なしで即実行。up は scale 属性を参照する (常に None。
+    # 他コマンドは無視する)。
+    "up": lambda root, name: dispatch_lifecycle("up", name, scale=None),
+    "rebuild": lambda root, name: dispatch_lifecycle("rebuild", name, scale=None),
     "down": _op_down,
     "login": _op_login,
     "ps": _op_ps,
@@ -149,10 +149,6 @@ def _run_operation(devbase_root: Path, name: str, op: str):
     戻り値: dispatch の rc (``int``) / ``_ARG_CANCEL`` (Esc・確認拒否で引数収集を
     中止 = サブメニューへ戻る) / ``None`` (選択・入力中の Ctrl-C → 全体中止)。
     """
-    if op in ("up", "rebuild"):
-        # 引数なしで即実行。up は scale 属性を参照する (常に None。他コマンドは無視)。
-        return dispatch_lifecycle(op, name, scale=None)
-
     handler = _OP_HANDLERS.get(op)
     if handler is None:
         # 到達しない (メニュー値は _RUNNING_OPS に限定される)。保守的に no-op。
diff --git a/lib/devbase/tui/actions_snapshot.py b/lib/devbase/tui/actions_snapshot.py
@@ -74,12 +74,7 @@ def _select_snapshot_name(devbase_root: Path, message: str):
 
     if snapshots is None:
         # 一覧が取れない環境では名前を直接入力させる。
-        name = menu.text(message, allow_empty=False)
-        if name is None:
-            return None                # Ctrl-C → 全体中止 (ナビ規約)
-        if name is menu.MENU_BACK:
-            return _ARG_CANCEL         # Esc → 操作メニューを再表示
-        return name
+        return flow.back_as_cancel(menu.text(message, allow_empty=False))
 
     if not snapshots:
         logger.info("スナップショットがありません。先に作成 (create) してください。")
@@ -93,13 +88,8 @@ def _select_snapshot_name(devbase_root: Path, message: str):
          s.get("name"))
         for s in snapshots
     ]
-    sel = menu.select(f"{message} {menu.HINT_SEARCH}:", choices,
-                      back=True, search=True)
-    if sel is None:
-        return None                    # Ctrl-C → 全体中止 (ナビ規約)
-    if sel is menu.MENU_BACK:
-        return _ARG_CANCEL             # Esc → 操作メニューを再表示
-    return sel
+    return flow.back_as_cancel(menu.select(
+        f"{message} {menu.HINT_SEARCH}:", choices, back=True, search=True))
 
 
 def _optional_point(message: str):
diff --git a/lib/devbase/tui/app.py b/lib/devbase/tui/app.py
@@ -35,16 +35,20 @@
 
 logger = get_logger(__name__)
 
-# プロジェクト一覧の末尾に並べるカテゴリ (表示順)。``(key, label)`` で保持し、
+# プロジェクト一覧の末尾に並べるカテゴリの SSoT (表示順 / ラベル / 実装モジュール)。
 # 一覧メニューには ``label (key)`` 形式で表示する (key 入力での絞り込みも効く)。
-TOP_CATEGORIES: list[tuple[str, str]] = [
-    ("env", "環境変数"),
-    ("plugin", "プラグイン"),
-    ("snapshot", "スナップショット"),
-    ("status", "ステータス"),
+# モジュール参照を保持し ``run`` の解決を呼び出し時まで遅らせる
+# (テストが ``actions_*.run`` を monkeypatch できるようにするため)。
+_CATEGORIES: list[tuple[str, str, object]] = [
+    ("env", "環境変数", actions_env),
+    ("plugin", "プラグイン", actions_plugin),
+    ("snapshot", "スナップショット", actions_snapshot),
+    ("status", "ステータス", actions_status),
 ]
 
+TOP_CATEGORIES: list[tuple[str, str]] = [(k, label) for k, label, _ in _CATEGORIES]
 _LABELS = dict(TOP_CATEGORIES)
+_CATEGORY_MODULES = {k: mod for k, _, mod in _CATEGORIES}
 
 
 def _route(category: str, devbase_root: Path):
@@ -55,16 +59,11 @@ def _route(category: str, devbase_root: Path):
     - 操作なしで一覧へ戻るときは ``menu.MENU_BACK``
     - Ctrl-C 全体中止のときは ``None``
     """
-    if category == "env":
-        return actions_env.run(devbase_root)
-    if category == "plugin":
-        return actions_plugin.run(devbase_root)
-    if category == "snapshot":
-        return actions_snapshot.run(devbase_root)
-    if category == "status":
-        return actions_status.run(devbase_root)
-    logger.error("未知のカテゴリです: %s", _LABELS.get(category, category))
-    return menu.MENU_BACK
+    module = _CATEGORY_MODULES.get(category)
+    if module is None:
+        logger.error("未知のカテゴリです: %s", _LABELS.get(category, category))
+        return menu.MENU_BACK
+    return module.run(devbase_root)
 
 
 def _select_top(rows: list[dict]):
diff --git a/lib/devbase/tui/flow.py b/lib/devbase/tui/flow.py
@@ -56,6 +56,16 @@ def need(value):
     return value
 
 
+def back_as_cancel(value):
+    """``MENU_BACK`` を ``ARG_CANCEL`` へ読み替える (選択ヘルパの番兵契約用)。
+
+    actions_* の選択ヘルパは「Esc = 呼び出し元メニューの再表示」を ``ARG_CANCEL``
+    で表現する契約を持つ (メニューループ自身の ``MENU_BACK`` = 1 つ上の階層へ、
+    と区別するため)。実値と ``None`` (Ctrl-C) はそのまま通す。
+    """
+    return ARG_CANCEL if value is menu.MENU_BACK else value
+
+
 def need_optional(value):
     """``optional_int`` の番兵 (``ABORT`` / ``ARG_CANCEL``) を例外へ変換する。
 
