claude-code-conductor 2.29.1__tar.gz → 2.29.2__tar.gz

{claude_code_conductor-2.29.1 → claude_code_conductor-2.29.2}/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## [2.29.2] - 2026-05-28
+
+**内部品質リファクタ（振る舞い不変・公開 API 不変・破壊的変更なし）**: v2.29.1 以降に進めたコード負債返済をまとめてリリース。新機能・バグ修正・破壊的変更はなく、`c3.db` などの公開 import は従来どおり動作する。観測可能な変化は「想定内のテーブル未作成（`sqlite3.OperationalError`）のログレベルが warning → debug に下がる」のみ。各リファクタは全テスト緑（1339 passed）・0 regression・全 Python（3.10/3.11/3.12）CI green を確認済み。
+
+### 変更
+
+- **`db.py`: 環境変数解決 3 関数を `_resolve_float_env` に集約（DRY）**: `resolve_cost_lambda` / `resolve_epsilon` / `resolve_escalation_threshold` のほぼ同一だった実装（env 名・デフォルト・有効域だけが違う）を共通ヘルパーに統合。型ナローイングは `typing.cast` で表現。
+- **`db.py`: tier-routing パラメータを `_db_params.py` へ分離**: SSOT 定数（`LEARNING_THRESHOLD` / `EPSILON_TIEBREAK` / `COST_LAMBDA_*` / `ESCALATION_THRESHOLD_DEFAULT`）と `resolve_*` を新規モジュールへ。`db.py` は後方互換のため re-export を維持し、`from c3.db import ...` / `from c3 import db; db.X` の両形式が従来どおり動作する（`dir(c3.db)` の公開シンボル集合は分割前と一致）。
+- **`usage_ingester.py`: パス traversal 検証の重複を `_safe_resolved_file` に集約（DRY）**: `_ingest_jsonl` / `_read_agent_meta` で重複していた symlink + resolve + project_dir 配下検証を 1 ヘルパーに統合。`cli_update` / `mcp_server` のパス検証は意味（段階別エラー返却・strict 存在必須）が異なるため共通化対象外とした。
+- **`db.py`: 例外分類を 10 関数で統一**: 想定内の `sqlite3.OperationalError`（テーブル未作成）は debug、想定外の `Exception` は warning に分類（既存 4 関数のパターンに合わせる）。graceful degradation の catch-all は意図的に維持（狭めない）。情報漏洩防止のため型名のみログ（生メッセージは出さない・既存方針を踏襲）。
+
+### ドキュメント
+
+- **`/ARCHITECTURE.md` を新設（リポジトリルート・配布対象外）**: `c3` パッケージと `.claude/` フレームワークの二層構造・ランタイムのオーケストレーション・hook ライフサイクル（settings.json と照合済み）・知能基盤（c3.db / recall）・ビルド/配布パイプラインを 1 枚に集約。既存の taxonomy.md / config-policy.md への索引役に徹する。
+
+### 後方互換
+
+- 公開 API（`c3.db` の import）・CLI・DB スキーマに変更なし。migration 不要。**破壊的変更なし**。
+- 唯一の観測可能な差分: 想定内 missing-table のログレベルが warning → debug（ログノイズ削減）。
+
 ## [2.29.1] - 2026-05-27
 
 **`deletions.txt` 遡及追記（rules→references 移動 3 ファイル）**: v2.15.0 で `.claude/rules/` から `.claude/skills/dev-workflow/references/` へ移動した 3 つのチェックリストが `deletions.txt` に未記載だった漏れを修正。配布先が次回 `c3 update` で旧 `.claude/rules/` の常時ロード残骸を除去できるようになる。**コード変更なし・破壊的変更なし・migration なし**。

{claude_code_conductor-2.29.1 → claude_code_conductor-2.29.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-code-conductor
-Version: 2.29.1
+Version: 2.29.2
 Summary: Multi-agent orchestration framework for Claude Code with Codex/Cursor adapters (C3)
 Project-URL: Homepage, https://github.com/satoh-y-0323/claude-code-conductor
 Project-URL: Repository, https://github.com/satoh-y-0323/claude-code-conductor

{claude_code_conductor-2.29.1 → claude_code_conductor-2.29.2}/src/c3/__init__.py

@@ -1,3 +1,3 @@
 """Claude Code Conductor (C3) - multi-agent orchestration framework for Claude Code."""
 
-__version__ = "2.29.1"
+__version__ = "2.29.2"

claude_code_conductor-2.29.2/src/c3/_db_params.py

@@ -0,0 +1,143 @@
+"""C3 tier-routing パラメータ解決（env override + SSOT 定数）。
+
+``db.py``（SQLite read/write helpers）から分離した tier-routing の tunable 群。
+``LEARNING_THRESHOLD`` / ``EPSILON_TIEBREAK`` / ``COST_LAMBDA_*`` /
+``ESCALATION_THRESHOLD_DEFAULT`` の SSOT 定数と、それらを環境変数で上書き解決する
+``resolve_*`` を提供する。DB I/O には依存しない（純粋な env パース）。
+
+後方互換のため、これらは ``c3.db`` からも re-export される。cli_tier.py /
+select_tier.py は従来どおり ``c3.db`` 経由でも、本モジュール直接でも参照できる。
+"""
+
+from __future__ import annotations
+
+import math
+import os
+import sys
+from typing import cast
+
+# tier-routing: 学習データ収集期の閾値（合計試行数がこの値未満なら uniform 選択）。
+# SSOT: cli_tier.py / select_tier.py はここから参照する（CR-M-002）。
+LEARNING_THRESHOLD = 30
+# cost-aware tie-break の拮抗判定閾値。Beta サンプルは 0〜1 スケールで、
+# 成功率 5pt（=0.05）以内を拮抗とみなす。本定数が SSOT。
+# 過大にすると成功率を犠牲にするリスク、過小にすると無発動になる。
+# C3_TIER_EPSILON 環境変数で上書き可（v2.25.0）。
+EPSILON_TIEBREAK = 0.05
+# cost-weighted Thompson の重み係数 λ の既定値。
+# None = v2.25.0 互換モード（ε tie-break を維持し全 tier weighting を発動しない）。
+# C3_TIER_COST_LAMBDA 環境変数で上書き可（v2.26.0）。
+# λ>0 で全 tier の score=sample-λ*cost_norm weighting が発動、λ=0 明示で cost 無視（純 Thompson）。
+# 本定数が SSOT。
+COST_LAMBDA_DEFAULT = None
+# failure rate がこの値以上で 1 段上位 tier へ escalation する閾値。
+# C3_ESCALATION_THRESHOLD 環境変数で上書き可（v2.26.0）。
+# 本定数が SSOT（select_tier.py はここから参照）。
+ESCALATION_THRESHOLD_DEFAULT = 0.5
+
+# cost-weighted Thompson の λ 有効範囲（v2.27.0: 上限を 1.0→5.0 に拡張）。
+# cost を成功率より強く効かせる余地を確保するため上限を 5.0 に設定。
+# select_tier.py の _resolve_cost_lambda はここを SSOT として参照する。
+COST_LAMBDA_MIN = 0.0
+COST_LAMBDA_MAX = 5.0
+
+
+def _resolve_float_env(
+    env_key: str,
+    default: float | None,
+    *,
+    min_val: float,
+    max_val: float,
+    min_inclusive: bool,
+    log_prefix: str,
+) -> float | None:
+    """env 変数を float として安全に解決する共通ヘルパー（resolve_* の SSOT 実体）。
+
+    挙動（resolve_cost_lambda / resolve_epsilon / resolve_escalation_threshold で共通）:
+    - 未設定 / 空文字 → 無警告で ``default`` を返す。
+    - 非数値 / NaN / 範囲外 → stderr に env 名入りの警告を出し ``default`` に戻す。
+    - 妥当域: 上限は常に閉区間 ``<= max_val``。下限は ``min_inclusive`` で開閉を切替
+      （True なら ``>= min_val`` の閉区間、False なら ``> min_val`` の半開区間）。
+    """
+    raw = os.environ.get(env_key)
+    if raw is None or raw == "":
+        return default
+    bracket = (
+        f"[{min_val}, {max_val}]" if min_inclusive else f"({min_val}, {max_val}]"
+    )
+    try:
+        x = float(raw)
+    except ValueError:
+        print(
+            f"{log_prefix} invalid {env_key}={raw!r}, using default {default}",
+            file=sys.stderr,
+        )
+        return default
+    if math.isnan(x):
+        print(
+            f"{log_prefix} {env_key}={raw!r} is NaN, using default {default}",
+            file=sys.stderr,
+        )
+        return default
+    low_ok = x >= min_val if min_inclusive else x > min_val
+    if not low_ok or x > max_val:
+        print(
+            f"{log_prefix} {env_key}={x!r} out of range {bracket}, "
+            f"using default {default}",
+            file=sys.stderr,
+        )
+        return default
+    return x
+
+
+def resolve_cost_lambda() -> float | None:
+    """``C3_TIER_COST_LAMBDA`` を安全に解決する（cli_tier 用 SSOT）。
+
+    妥当域: [COST_LAMBDA_MIN, COST_LAMBDA_MAX]（x=0 許容の閉区間）。
+    戻り値が None の場合は v2.25.0 互換の ε tie-break 経路を維持する（センチネル）。
+    詳細な共通挙動は :func:`_resolve_float_env` を参照。
+    """
+    return _resolve_float_env(
+        "C3_TIER_COST_LAMBDA",
+        COST_LAMBDA_DEFAULT,
+        min_val=COST_LAMBDA_MIN,
+        max_val=COST_LAMBDA_MAX,
+        min_inclusive=True,
+        log_prefix="[c3:cost_lambda]",
+    )
+
+
+def resolve_epsilon() -> float:
+    """``C3_TIER_EPSILON`` を安全に解決する（cli_tier 用 SSOT）。
+
+    妥当域: (0, 1]（x=0 拒否の半開区間）。default が float のため戻り値は常に float。
+    詳細な共通挙動は :func:`_resolve_float_env` を参照。
+    """
+    value = _resolve_float_env(
+        "C3_TIER_EPSILON",
+        EPSILON_TIEBREAK,
+        min_val=0.0,
+        max_val=1.0,
+        min_inclusive=False,
+        log_prefix="[c3:epsilon]",
+    )
+    # default=EPSILON_TIEBREAK のため None になり得ない（戻り値型を float に絞る）
+    return cast(float, value)
+
+
+def resolve_escalation_threshold() -> float:
+    """``C3_ESCALATION_THRESHOLD`` を安全に解決する（cli_tier 用 SSOT）。
+
+    妥当域: (0, 1]（x=0 拒否の半開区間）。default が float のため戻り値は常に float。
+    詳細な共通挙動は :func:`_resolve_float_env` を参照。
+    """
+    value = _resolve_float_env(
+        "C3_ESCALATION_THRESHOLD",
+        ESCALATION_THRESHOLD_DEFAULT,
+        min_val=0.0,
+        max_val=1.0,
+        min_inclusive=False,
+        log_prefix="[c3:escalation]",
+    )
+    # default=ESCALATION_THRESHOLD_DEFAULT のため None になり得ない（戻り値型を float に絞る）
+    return cast(float, value)

{claude_code_conductor-2.29.1 → claude_code_conductor-2.29.2}/src/c3/db.py

@@ -9,6 +9,9 @@ DB が見つからない場合・書き込みエラー時は静かにスキッ
 書き込みは Python 標準の `sqlite3` で行う（WAL モード）。
 読み・分析は別途 DuckDB の sqlite_scanner で ATTACH する想定（duckdb-hybrid と整合）。
 
+tier-routing の tunable 定数・resolve_* は `_db_params` が SSOT。後方互換のため
+本モジュールからも re-export される（既存の `from c3.db import ...` を壊さないため）。
+
 履歴: v1.11.0 までは `src/parallel_orchestra/c3_db.py` に置かれていたが、
 PO 廃止計画（plan: atomic-foraging-sprout）の Step 1 で本ファイルに物理移動し、
 v2.0.0 で PO 専用ヘルパー（record_task_results / fetch_po_results /
@@ -18,13 +21,25 @@ upsert_po_status / fetch_po_status）も同時に削除した。
 from __future__ import annotations
 
 import logging
-import math
 import os
 import sqlite3
-import sys
 from datetime import datetime
 from pathlib import Path
 
+# tier-routing パラメータ（定数 + env 解決）は _db_params.py が SSOT。
+# 後方互換のため c3.db からも参照可能にする（cli_tier.py / select_tier.py 等）。
+from c3._db_params import (
+    COST_LAMBDA_DEFAULT as COST_LAMBDA_DEFAULT,
+    COST_LAMBDA_MAX as COST_LAMBDA_MAX,
+    COST_LAMBDA_MIN as COST_LAMBDA_MIN,
+    EPSILON_TIEBREAK as EPSILON_TIEBREAK,
+    ESCALATION_THRESHOLD_DEFAULT as ESCALATION_THRESHOLD_DEFAULT,
+    LEARNING_THRESHOLD as LEARNING_THRESHOLD,
+    resolve_cost_lambda as resolve_cost_lambda,
+    resolve_epsilon as resolve_epsilon,
+    resolve_escalation_threshold as resolve_escalation_threshold,
+)
+
 logger = logging.getLogger(__name__)
 
 
@@ -34,144 +49,6 @@ logger = logging.getLogger(__name__)
 BUSY_TIMEOUT_MS = 5000
 _BUSY_TIMEOUT_MS = BUSY_TIMEOUT_MS  # 内部互換エイリアス（既存コードへの影響なし）
 
-# tier-routing: 学習データ収集期の閾値（合計試行数がこの値未満なら uniform 選択）。
-# SSOT: cli_tier.py / select_tier.py はここから参照する（CR-M-002）。
-LEARNING_THRESHOLD = 30
-# cost-aware tie-break の拮抗判定閾値。Beta サンプルは 0〜1 スケールで、
-# 成功率 5pt（=0.05）以内を拮抗とみなす。本定数が SSOT。
-# 過大にすると成功率を犠牲にするリスク、過小にすると無発動になる。
-# C3_TIER_EPSILON 環境変数で上書き可（v2.25.0）。
-EPSILON_TIEBREAK = 0.05
-# cost-weighted Thompson の重み係数 λ の既定値。
-# None = v2.25.0 互換モード（ε tie-break を維持し全 tier weighting を発動しない）。
-# C3_TIER_COST_LAMBDA 環境変数で上書き可（v2.26.0）。
-# λ>0 で全 tier の score=sample-λ*cost_norm weighting が発動、λ=0 明示で cost 無視（純 Thompson）。
-# 本定数が SSOT。
-COST_LAMBDA_DEFAULT = None
-# failure rate がこの値以上で 1 段上位 tier へ escalation する閾値。
-# C3_ESCALATION_THRESHOLD 環境変数で上書き可（v2.26.0）。
-# 本定数が SSOT（select_tier.py はここから参照）。
-ESCALATION_THRESHOLD_DEFAULT = 0.5
-
-# cost-weighted Thompson の λ 有効範囲（v2.27.0: 上限を 1.0→5.0 に拡張）。
-# cost を成功率より強く効かせる余地を確保するため上限を 5.0 に設定。
-# select_tier.py の _resolve_cost_lambda はここを SSOT として参照する。
-COST_LAMBDA_MIN = 0.0
-COST_LAMBDA_MAX = 5.0
-
-
-def resolve_cost_lambda() -> float | None:
-    """``C3_TIER_COST_LAMBDA`` を安全に解決する（cli_tier 用 SSOT）。
-
-    不正値（非数値 / 0 未満 / COST_LAMBDA_MAX 超 / NaN）は受け付けず、
-    stderr 警告 + デフォルト（COST_LAMBDA_DEFAULT = None）に戻す。
-    未設定 / 空文字は無警告でデフォルト（None）を返す。
-    妥当域: [COST_LAMBDA_MIN, COST_LAMBDA_MAX]（x=0 許容の閉区間）。
-    戻り値が None の場合は v2.25.0 互換の ε tie-break 経路を維持する（センチネル）。
-    """
-    raw = os.environ.get("C3_TIER_COST_LAMBDA")
-    if raw is None or raw == "":
-        return COST_LAMBDA_DEFAULT
-    try:
-        x = float(raw)
-    except ValueError:
-        print(
-            f"[c3:cost_lambda] invalid C3_TIER_COST_LAMBDA={raw!r}, "
-            f"using default {COST_LAMBDA_DEFAULT}",
-            file=sys.stderr,
-        )
-        return COST_LAMBDA_DEFAULT
-    if math.isnan(x):
-        print(
-            f"[c3:cost_lambda] C3_TIER_COST_LAMBDA={raw!r} is NaN, "
-            f"using default {COST_LAMBDA_DEFAULT}",
-            file=sys.stderr,
-        )
-        return COST_LAMBDA_DEFAULT
-    if x < COST_LAMBDA_MIN or x > COST_LAMBDA_MAX:
-        print(
-            f"[c3:cost_lambda] C3_TIER_COST_LAMBDA={x!r} out of range "
-            f"[{COST_LAMBDA_MIN}, {COST_LAMBDA_MAX}], "
-            f"using default {COST_LAMBDA_DEFAULT}",
-            file=sys.stderr,
-        )
-        return COST_LAMBDA_DEFAULT
-    return x
-
-
-def resolve_epsilon() -> float:
-    """``C3_TIER_EPSILON`` を安全に解決する（cli_tier 用 SSOT）。
-
-    不正値（非数値 / 0 以下 / 1 超 / NaN）は受け付けず、
-    stderr 警告 + デフォルト（EPSILON_TIEBREAK）に戻す。
-    未設定 / 空文字は無警告でデフォルトを返す。
-    妥当域: (0, 1]（x=0 拒否の半開区間）。
-    """
-    raw = os.environ.get("C3_TIER_EPSILON")
-    if raw is None or raw == "":
-        return EPSILON_TIEBREAK
-    try:
-        x = float(raw)
-    except ValueError:
-        print(
-            f"[c3:epsilon] invalid C3_TIER_EPSILON={raw!r}, "
-            f"using default {EPSILON_TIEBREAK}",
-            file=sys.stderr,
-        )
-        return EPSILON_TIEBREAK
-    if math.isnan(x):
-        print(
-            f"[c3:epsilon] C3_TIER_EPSILON={raw!r} is NaN, "
-            f"using default {EPSILON_TIEBREAK}",
-            file=sys.stderr,
-        )
-        return EPSILON_TIEBREAK
-    if x <= 0 or x > 1:
-        print(
-            f"[c3:epsilon] C3_TIER_EPSILON={x!r} out of range (0, 1], "
-            f"using default {EPSILON_TIEBREAK}",
-            file=sys.stderr,
-        )
-        return EPSILON_TIEBREAK
-    return x
-
-
-def resolve_escalation_threshold() -> float:
-    """``C3_ESCALATION_THRESHOLD`` を安全に解決する（cli_tier 用 SSOT）。
-
-    不正値（非数値 / 0 以下 / 1 超 / NaN）は受け付けず、
-    stderr 警告 + デフォルト（ESCALATION_THRESHOLD_DEFAULT）に戻す。
-    未設定 / 空文字は無警告でデフォルトを返す。
-    妥当域: (0, 1]（x=0 拒否の半開区間）。
-    """
-    raw = os.environ.get("C3_ESCALATION_THRESHOLD")
-    if raw is None or raw == "":
-        return ESCALATION_THRESHOLD_DEFAULT
-    try:
-        x = float(raw)
-    except ValueError:
-        print(
-            f"[c3:escalation] invalid C3_ESCALATION_THRESHOLD={raw!r}, "
-            f"using default {ESCALATION_THRESHOLD_DEFAULT}",
-            file=sys.stderr,
-        )
-        return ESCALATION_THRESHOLD_DEFAULT
-    if math.isnan(x):
-        print(
-            f"[c3:escalation] C3_ESCALATION_THRESHOLD={raw!r} is NaN, "
-            f"using default {ESCALATION_THRESHOLD_DEFAULT}",
-            file=sys.stderr,
-        )
-        return ESCALATION_THRESHOLD_DEFAULT
-    if x <= 0 or x > 1:
-        print(
-            f"[c3:escalation] C3_ESCALATION_THRESHOLD={x!r} out of range (0, 1], "
-            f"using default {ESCALATION_THRESHOLD_DEFAULT}",
-            file=sys.stderr,
-        )
-        return ESCALATION_THRESHOLD_DEFAULT
-    return x
-
 
 def _apply_busy_timeout(conn: sqlite3.Connection) -> None:
     # PRAGMA はパラメータバインドできないため値が整数であることを int() で強制する。
@@ -274,6 +151,9 @@ def fetch_review_decisions(
             return [dict(r) for r in rows]
         finally:
             conn.close()
+    except sqlite3.OperationalError as exc:
+        logger.debug("fetch_review_decisions: table not found or inaccessible: %s", type(exc).__name__)
+        return []
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to fetch review_decisions: %s", type(exc).__name__)
         return []
@@ -339,6 +219,9 @@ def insert_review_decision(
         finally:
             conn.close()
         return True
+    except sqlite3.OperationalError as exc:
+        logger.debug("insert_review_decision: table not found or inaccessible: %s", type(exc).__name__)
+        return False
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to insert review_decision: %s", type(exc).__name__)
         return False
@@ -405,6 +288,9 @@ def read_tier_params(
             ).fetchall()
         finally:
             conn.close()
+    except sqlite3.OperationalError as exc:
+        logger.debug("read_tier_params: table not found or inaccessible: %s", type(exc).__name__)
+        return defaults
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to read tier_params: %s", type(exc).__name__)
         return defaults
@@ -477,6 +363,9 @@ def update_tier_params(
         finally:
             conn.close()
         return True
+    except sqlite3.OperationalError as exc:
+        logger.debug("update_tier_params: table not found or inaccessible: %s", type(exc).__name__)
+        return False
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to update tier_params: %s", type(exc).__name__)
         return False
@@ -534,6 +423,9 @@ def record_tier_recent_outcome(
         finally:
             conn.close()
         return True
+    except sqlite3.OperationalError as exc:
+        logger.debug("record_tier_recent_outcome: table not found or inaccessible: %s", type(exc).__name__)
+        return False
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to record tier_recent_outcome: %s", type(exc).__name__)
         return False
@@ -628,6 +520,9 @@ def read_tier_failure_rate(
             ).fetchall()
         finally:
             conn.close()
+    except sqlite3.OperationalError as exc:
+        logger.debug("read_tier_failure_rate: table not found or inaccessible: %s", type(exc).__name__)
+        return None, 0
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to read tier_failure_rate: %s", type(exc).__name__)
         return None, 0
@@ -723,6 +618,9 @@ def insert_agent_cost_run(
         finally:
             conn.close()
         return True
+    except sqlite3.OperationalError as exc:
+        logger.debug("insert_agent_cost_run: table not found or inaccessible: %s", type(exc).__name__)
+        return False
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to insert_agent_cost_run: %s", type(exc).__name__)
         return False
@@ -1243,6 +1141,9 @@ def set_ingest_offset(
         finally:
             conn.close()
         return True
+    except sqlite3.OperationalError as exc:
+        logger.debug("set_ingest_offset: table not found or inaccessible: %s", type(exc).__name__)
+        return False
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to set_ingest_offset: %s", type(exc).__name__)
         return False
@@ -1279,6 +1180,9 @@ def read_tier_bandit_cost(
             ).fetchall()
         finally:
             conn.close()
+    except sqlite3.OperationalError as exc:
+        logger.debug("read_tier_bandit_cost: table not found or inaccessible: %s", type(exc).__name__)
+        return {}
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to read tier_bandit_cost: %s", type(exc).__name__)
         return {}
@@ -1356,6 +1260,9 @@ def sync_tier_bandit_cost(
             conn.commit()
         finally:
             conn.close()
+    except sqlite3.OperationalError as exc:
+        logger.debug("sync_tier_bandit_cost: table not found or inaccessible: %s", type(exc).__name__)
+        return 0
     except Exception as exc:  # noqa: BLE001
         logger.warning("failed to sync_tier_bandit_cost: %s", type(exc).__name__)
         return 0

{claude_code_conductor-2.29.1 → claude_code_conductor-2.29.2}/src/c3/usage_ingester.py

@@ -21,7 +21,6 @@ import logging
 import re
 from dataclasses import dataclass, field
 from pathlib import Path
-from typing import Generator
 
 from c3.db import (
     get_ingest_offset,
@@ -133,6 +132,29 @@ def ingest_session(
     return result
 
 
+def _safe_resolved_file(
+    path: Path, project_dir: Path, *, log_label: str
+) -> Path | None:
+    """symlink でない実在ファイルを resolve し、project_dir 配下なら resolved Path を返す。
+
+    検証に失敗したら None を返す（symlink / 非ファイル / project_dir 外）。symlink・
+    範囲外は debug ログを出す。``is_symlink()`` / ``is_file()`` / ``resolve()`` の
+    例外（マウント切れ等の稀な OSError）は送出し、呼び出し側の try で扱う
+    （_ingest_jsonl は result.errors に記録、_read_agent_meta は debug ログ）。
+    SR-V-002 のパス traversal 対策を 1 箇所に集約する。
+    """
+    if path.is_symlink():
+        logger.debug("%s: symlink skipped", log_label)
+        return None
+    if not path.is_file():
+        return None
+    resolved = path.resolve()
+    if not resolved.is_relative_to(project_dir):
+        logger.debug("%s: path outside project_dir, skipped", log_label)
+        return None
+    return resolved
+
+
 def _ingest_jsonl(
     *,
     jsonl_path: Path,
@@ -148,24 +170,17 @@ def _ingest_jsonl(
     - 存在しない / symlink / project_dir 外のパスはスキップ
     - parse error なく読み切れた時のみ offset を更新（冪等性保証）
     """
-    # symlink チェック
-    if jsonl_path.is_symlink():
-        logger.debug("_ingest_jsonl: symlink skipped")
-        return
-
-    # ファイル存在チェック
-    if not jsonl_path.is_file():
-        return
-
-    # パス traversal チェック: resolve 後 project_dir 配下であることを確認
+    # symlink でない project_dir 配下の実在ファイルか検証し、resolved パスを得る。
+    # resolve() の例外のみ errors に記録する（symlink/非ファイル/範囲外は静かにスキップ）。
     try:
-        resolved_jsonl = jsonl_path.resolve()
-        if not resolved_jsonl.is_relative_to(project_dir):
-            logger.debug("_ingest_jsonl: path outside project_dir, skipped")
-            return
+        resolved_jsonl = _safe_resolved_file(
+            jsonl_path, project_dir, log_label="_ingest_jsonl"
+        )
     except Exception as exc:  # noqa: BLE001
         result.errors.append(type(exc).__name__)
         return
+    if resolved_jsonl is None:
+        return
 
     file_key = f"{session_id}:{agent_id}"
     start_offset = get_ingest_offset(file_key, db_path=db_path)
@@ -345,24 +360,17 @@ def _read_agent_meta(
         - parse エラー: (None, None)
         - agentType なし: (None, description_or_None)
     """
-    # symlink チェック
-    if meta_path.is_symlink():
-        logger.debug("_read_agent_meta: symlink skipped")
-        return None, None
-
-    # ファイル存在チェック
-    if not meta_path.is_file():
-        return None, None
-
-    # パス traversal チェック
+    # symlink でない project_dir 配下の実在ファイルか検証し、resolved パスを得る。
+    # resolve() の例外は debug ログのみ（symlink/非ファイル/範囲外は静かにスキップ）。
     try:
-        resolved_meta = meta_path.resolve()
-        if not resolved_meta.is_relative_to(project_dir):
-            logger.debug("_read_agent_meta: path outside project_dir, skipped")
-            return None, None
+        resolved_meta = _safe_resolved_file(
+            meta_path, project_dir, log_label="_read_agent_meta"
+        )
     except Exception as exc:  # noqa: BLE001
         logger.debug("_read_agent_meta: resolve error: %s", type(exc).__name__)
         return None, None
+    if resolved_meta is None:
+        return None, None
 
     try:
         data = json.loads(resolved_meta.read_text(encoding="utf-8"))