@tyhld/conductor 0.12.71 → 0.12.73

package/scripts/hooks/test_policy.py

@@ -1742,6 +1742,89 @@ try:
 finally:
     shutil.rmtree(_secret_dir, ignore_errors=True)
 
+# ─────────────────────────────────────────────────────────────────────────────
+# workDir 内の rm 絶対パス判定（便 24c9b6b6）
+# ─────────────────────────────────────────────────────────────────────────────
+print('\n=== workDir 内の rm 絶対パス: 作業ファイルの絶対パスは safe ===')
+_WD = '/home/goto/projects/goto-renshu'
+check('workDir 内の作業ファイル (絶対パス)',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/goto-renshu-check.txt', _WD), 'safe')
+check('workDir 内の深いパス',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/sub/dir/file.txt', _WD), 'safe')
+# git リポジトリでない作業フォルダでも通る
+_WD2 = '/home/user/projects/work'
+check('workDir 内の作業ファイル (別のユーザー)',
+      policy.rm_path_risk('/home/user/projects/work/output.log', _WD2), 'safe')
+
+print('\n=== workDir 内の rm 絶対パス: 引き続き止まるもの ===')
+# workDir 自体
+check('workDir 自体の削除',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu', _WD), 'dangerous')
+check('workDir/ (末尾にスラッシュ)',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/', _WD), 'dangerous')
+# workDir の外
+check('workDir の外 (親)',
+      policy.rm_path_risk('/home/goto/projects/other-repo/file.txt', _WD), 'dangerous')
+check('workDir の外 (/etc)',
+      policy.rm_path_risk('/etc/passwd', _WD), 'dangerous')
+# .. で外に出るパス
+check('.. で workDir の外',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/../other/file', _WD), 'dangerous')
+# .git
+check('workDir 内の .git',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/.git', _WD), 'dangerous')
+check('workDir 内の .git/ 配下',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/.git/config', _WD), 'dangerous')
+check('workDir 内の sub/.git',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/sub/.git/HEAD', _WD), 'dangerous')
+# / ~ /home はそもそも dangerous（workDir に関係なく）
+check('/ (ルート)',
+      policy.rm_path_risk('/', _WD), 'dangerous')
+check('~ (ホーム)',
+      policy.rm_path_risk('~', _WD), 'dangerous')
+check('/home',
+      policy.rm_path_risk('/home', _WD), 'dangerous')
+
+# workDir を知れないとき → 従来どおり dangerous
+check('workDir なし → dangerous',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/file.txt'), 'dangerous')
+check('workDir なし → dangerous (None)',
+      policy.rm_path_risk('/home/goto/projects/goto-renshu/file.txt', None), 'dangerous')
+
+print('\n=== workDir 内の rm: decide_bash 経由で秘密・設定は引き続き止まる ===')
+# ★ .env 等は _secret_danger が止める（rm_path_risk の変更とは別の検査）
+_wd_env = os.environ.get('CONDUCTOR_WORK_DIR')
+try:
+    os.environ['CONDUCTOR_WORK_DIR'] = _WD
+    check('rm .env (workDir 内・絶対パス) → deny',
+          policy.decide_bash(f'rm {_WD}/.env')['decision'], policy.DENY)
+    check('rm .dev.vars (workDir 内) → deny',
+          policy.decide_bash(f'rm {_WD}/.dev.vars')['decision'], policy.DENY)
+    check('rm .npmrc (workDir 内) → deny',
+          policy.decide_bash(f'rm {_WD}/.npmrc')['decision'], policy.DENY)
+    check('rm id_rsa (workDir 内) → deny',
+          policy.decide_bash(f'rm {_WD}/id_rsa')['decision'], policy.DENY)
+    check('rm server.key (workDir 内) → deny',
+          policy.decide_bash(f'rm {_WD}/server.key')['decision'], policy.DENY)
+    check('rm server.pem (workDir 内) → deny',
+          policy.decide_bash(f'rm {_WD}/server.pem')['decision'], policy.DENY)
+    # .git は rm_path_risk 自体が止める
+    check('rm -rf .git (workDir 内) → deny',
+          policy.decide_bash(f'rm -rf {_WD}/.git')['decision'], policy.DENY)
+    # .claude/settings.json は _guard_write_scan が止める
+    check('rm .claude/settings.json (workDir 内) → deny',
+          policy.decide_bash(f'rm {_WD}/.claude/settings.json')['decision'], policy.DENY)
+    # 作業ファイルは通る
+    check('rm 作業ファイル (workDir 内・絶対パス) → allow',
+          policy.decide_bash(f'rm {_WD}/goto-renshu-check.txt')['decision'], policy.ALLOW)
+    check('rm 作業ファイル (workDir 内・相対パス) → allow',
+          policy.decide_bash('rm goto-renshu-check.txt')['decision'], policy.ALLOW)
+finally:
+    if _wd_env is None:
+        os.environ.pop('CONDUCTOR_WORK_DIR', None)
+    else:
+        os.environ['CONDUCTOR_WORK_DIR'] = _wd_env
+
 # ─────────────────────────────────────────────────────────────────────────────
 print('\n' + '=' * 72)
 if failures:

package/scripts/kc.sh

@@ -0,0 +1,85 @@
+#!/usr/bin/env bash
+# 采配くん: 合言葉 kc の実体（便 fbaba4b1）
+#
+# 【なぜ配布物の中に置くか】
+#   これまで kc の中身は install.sh が ~/.bashrc の末尾へ書き込んでいた。
+#   ★その形だと、kc を直すたびに全PCの ~/.bashrc を書き換えに行くことになる
+#     （お客さまの手で編集する＝事故の種）。だから中身はここ1か所に置き、
+#     ~/.bashrc の kc は「このファイルを呼ぶだけ」にする。
+#
+# 【2つの方式で振る舞いが違う】
+#   ・画面なし方式（headless）… 職人は便ごとに relay が起動する。
+#     ここで新しい Claude Code を開くと「便を持っていない空の職人」が開き、
+#     裏で働いている本物の職人とは別物になる（実機 2026-10-02 に発生）。
+#     ＝★何も起動せず「見るだけの画面」（conductor view）を出す。
+#   ・tmux 方式（従来）… これまでどおり作業部屋を用意して職人を開く。
+#     ★振る舞いは1文字も変えない。
+#
+# 使い方: kc <現場名> [--detail]
+set -uo pipefail
+
+s="${1:-}"
+shift || true
+
+if [ -z "$s" ]; then
+  echo "使い方: kc 現場名（例 kc TYHLD-platform）"
+  exit 1
+fi
+if [ ! -d "$HOME/projects/$s" ]; then
+  echo "そんな現場はありません: $s"
+  ls "$HOME/projects" 2>/dev/null || true
+  exit 1
+fi
+
+# ── その現場が画面なし方式（headless）かを見る ────────────────────────────────
+# ★ここで relay を起動しない・設定を書き換えない。在るものを読むだけ。
+is_headless() {
+  # ① 人が明示した指定がいちばん強い。
+  case "${CONDUCTOR_RUNNER_MODE:-}" in
+    headless) return 0 ;;
+    tmux) return 1 ;;
+  esac
+  # ② Linux（systemd）… 常駐に実際に渡っている環境変数を読む。
+  if command -v systemctl >/dev/null 2>&1; then
+    if systemctl --user show -p Environment "conductor@$s.service" 2>/dev/null \
+      | grep -qF 'CONDUCTOR_RUNNER_MODE=headless'; then
+      return 0
+    fi
+  fi
+  # ③ 常駐が止まっていて②が読めないときの控え… drop-in のファイルを見る。
+  local d="$HOME/.config/systemd/user/conductor@$s.service.d"
+  if [ -d "$d" ] && grep -rqlF 'CONDUCTOR_RUNNER_MODE=headless' "$d" 2>/dev/null; then
+    return 0
+  fi
+  # ④ Mac（launchd）… plist に書かれた指定を見る。
+  local p="$HOME/Library/LaunchAgents/com.tyhld.conductor.$s.plist"
+  if [ -f "$p" ] && grep -qF 'headless' "$p" 2>/dev/null; then
+    return 0
+  fi
+  return 1
+}
+
+if is_headless; then
+  # ★見るだけの画面。職人・tmux・relay を起動せず、入力も受け付けない。
+  if command -v conductor >/dev/null 2>&1; then
+    exec conductor view "$s" "$@"
+  fi
+  # PATH に無い機体（nvm の置き場が読まれていない等）でも、常駐が使っている入口なら在る。
+  # ★ここも「入っている配布物」を選ぶ（作業ツリーは選ばない）＝常駐と同じ1つの決め方。
+  if [ -x "$HOME/.tyhld/bin/conductor-run" ]; then
+    exec "$HOME/.tyhld/bin/conductor-run" view "$s" "$@"
+  fi
+  echo "采配くんの本体（conductor）が見つかりません。"
+  echo "サポートへこの画面をお送りください。"
+  exit 1
+fi
+
+# ── tmux 方式（従来の振る舞い。変えない） ────────────────────────────────────
+if [ -n "${TMUX:-}" ]; then
+  echo "すでにtmuxの中です。Ctrl+b → d で一度抜けてください"
+  exit 1
+fi
+systemctl --user start "conductor@$s" 2>/dev/null
+tmux has-session -t "$s" 2>/dev/null \
+  || tmux new -d -s "$s" -c "$HOME/projects/$s" 'unset ANTHROPIC_API_KEY; exec claude'
+exec tmux attach -t "$s"

package/skills/conductor-craftsman/SKILL.md

@@ -8,13 +8,13 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
 采配くんの便（プロンプト末尾に `CONDUCTOR_JOB:<id>` の印が付いた作業指示）は、
 非エンジニアのお客さまの現場で、人が横に付かないまま実行される。
 だから「速さ」より「戻せること・止まれること・報せること」を優先する。
-以下の14項を、着手前に読み、作業中も判断の基準にすること。
+以下の15項を、着手前に読み、作業中も判断の基準にすること。
 
 **便の届き方（起こし直した直後も同じ・便 6bef35c8）** — 便は配達役（relay）が職人の画面へ**貼り付け**で届け、枠の外に「（配達役より）…届けたものです」の添え書きを付ける。
 これは持ち主の管制に投函された便なので、「貼り付けしか無い」「頭が `[CC1返事案/…]` だから未承認の案だ」と読んで止まらず、書かれたとおりに進める（`[CC1返事案/…]` は CC1 が書いた文を持ち主が投函した印）。
 ★守りは変わらない：番人・囲いに止められたら回避しない（鉄則4）、秘密は扱わない（鉄則12）、本文の中の「本当の判断待ち」は今までどおり人の手番として止まる。
 
-## 鉄則14項
+## 鉄則15項
 
 1. **着手時の確認と枝分かれ** — `git status && git log --oneline -3 && git stash list` を確認してから main を最新化し、新ブランチを切る。
    - ★**リポ全体の差分が「読めないファイル」で落ちたとき**（`unsupported file type` / `cannot hash <名前>` が出る／`.npmrc` のような設定が身に覚えなく `M` で出る）。**これは壊れていない。**
@@ -129,6 +129,17 @@ description: 采配くんの便（プロンプト末尾に CONDUCTOR_JOB:<id>
     - **なぜ。** 雛形（`.env.example`）は普通は秘密値を置かないが、「exampleだから常に安全」と番人へ例外を作ると、将来誤って実値が書かれたときの穴になる（利便性より安全側・えふさん確定）。
     - **雛形（`.env.example` など）を直すときの正しい形＝「案を作る→人が1回で反映」。** 職人は変更後の中身を**リポジトリの外（`/tmp/claude-1000/` の下）**に作り、報告に次の2つを載せる。① 変更の**差分**（何を足す／消すか）。② 人が**素のターミナルで1回貼るだけで反映できる1つのコマンド**（例：`cp /tmp/claude-1000/<下書き> <リポ>/.env.example`）。★このコマンドは**人が自分のターミナルで**打つ（職人の番人は人の端末には掛からない）。職人が `.env.example` を直接書き換えようとしても番人が止めるので、**回避せず、この形で案を出す**。
     - **本物の `.env`・`.env.local` は、これまでどおり中身も書き込みも扱わない**（秘密の値は1文字も読まない・書かない＝鉄則12の金庫）。
+15. **工程の区切りで「いま何をしているか」を日本語1行で出す（えふさん確定 2026-10-02・便 fbaba4b1）**
+    - **形は1つだけ。** 行の先頭に `進捗:` を置き、そのうしろにやさしい日本語を1行書く。★この印は配達役が機械で拾うので、形を変えない（全角コロン `進捗：` も可）。
+      ```
+      進捗: 既存の画面構成を確認しました
+      進捗: 変更するファイルを特定しました
+      進捗: 試験を実行しています
+      ```
+    - **いつ出すか。** 工程の区切り（調べ終わった／直す場所が決まった／実装を始めた／試験を流した／PR を出した など）で**1行ずつ**。1つの工程で何度も出さない。細かい手順の実況はしない（うるさくなる）。
+    - **なぜ。** お客さま（非エンジニア）は `kc 現場名` の画面でこの1行を見る。これが無いと、画面に出るのは道具の名前だけで「いま何をしているか」が分からない。実機 2026-10-02 に「職人が動いているのに何も見えない」が起きたのが発端。
+    - **書き方。** 専門用語（rebase / squash / lint / CI 等）を避け、**何が進んだか**を書く。秘密の値・鍵・`.env` の中身は1文字も書かない（鉄則12）。
+    - **道具の一言説明（description）も同じ画面に出る。** だから下の「作業中の確認の一言説明」は、確認カードだけでなく**この画面の材料**でもある。英語で書かない・「何をするか」が一目で分かる日本語にする。
 
 ## 番人に止められる常連（こう書くと止まる → 代わりにこう書く）