@geekbeer/minion 3.55.1 → 3.58.0

package/roles/engineer.md

@@ -28,7 +28,9 @@
 
 1. **まずプロジェクトメモリーを検索** — `GET /api/project-memories?project_id=...&search=キーワード` で過去の知見を確認
 2. **自己解決不可なら即ヘルプスレッドを起票** — `POST /api/threads` で `thread_type: "help"` を作成。`attempted_resolution` に試したことを記載
-3. **人間にしか解決できない問題は `mentions: ["user"]` で即エスカレーション** — 認証コード、パスワード、外部承認など
+3. **人間にしか解決できない問題は即エスカレーション** — 認証コード、パスワード、外部承認など
+   - 当事者が特定できる場合は **`GET /api/minion/me/project/<project_id>/members` で `user_id` を引いて `mentions: ["user:<auth_user_id>"]` で個別指名** (display_name 重複時の誤通知を避けるため)
+   - 誰でも対応可能な問題なら `mentions: ["user"]` (generic)
 4. **解決後はスレッドを resolve し、知見をプロジェクトメモリーに保存**
 
 **スキルを失敗終了させる前に、必ずヘルプスレッドでエスカレーションすること。**

package/roles/pm.md

@@ -48,14 +48,18 @@ PMとして、ブロッカーの解決を主導する責務がある:
 
 ### 自分がブロッカーに遭遇した場合
 1. **プロジェクトメモリーを検索** — 過去の知見で解決できないか確認
-2. **自己解決不可なら `mentions: ["user"]` でヘルプスレッドを起票** — PMが解決できない問題は人間にエスカレーション
+2. **自己解決不可ならヘルプスレッドを起票** — PMが解決できない問題は人間にエスカレーション
+   - 特定の人にしか分からない問題なら **`GET /api/minion/me/project/<project_id>/members` で `user_id` を引いて `mentions: ["user:<auth_user_id>"]` で個別指名**
+   - 誰でも良い性質の問題なら `mentions: ["user"]` (generic)
 3. **解決後はスレッドを resolve し、知見をプロジェクトメモリーに保存**
 
 ### エンジニアからのヘルプスレッドに対応する場合
 1. **thread_watcher が自動で通知するので、`role:pm` 宛のスレッドに回答する**
-2. **自分で解決できない場合は `@user` メンション付きで人間にエスカレーション**
+2. **自分で解決できない場合は人間にエスカレーション** — 当事者が特定できるなら個別指名 (`user:<auth_user_id>`)、不明なら `user` (generic)
 3. **解決策が汎用的なら、プロジェクトメモリーへの保存を促す**
 
+display_name はワークスペース内で一意とは限らない。同名問題を避けるため、**generic `user` より個別指名 (`user:<id>`) を優先する**こと。`user_id` の取得方法は `~/.minion/docs/api-reference.md` の「Project Members / Workspace Members」セクションを参照。
+
 詳細は `~/.minion/rules/core.md` の「Blocker Handling」セクションを参照。
 
 ## Routine (従来方式)

package/rules/core.md

@@ -220,10 +220,11 @@ Routine 実行中は以下もtmuxセッション環境で利用可能:
 - `MINION_EXECUTION_ID` — 実行UUID
 - `MINION_ROUTINE_ID` — ルーティンUUID
 - `MINION_ROUTINE_NAME` — ルーティン名
+- `MINION_ROUTINE_WORKSPACE_ID` — ルーティンが特定ワークスペースにバインドされている場合のワークスペースUUID（未バインドなら空文字）
 
-**変数**（ミニオン変数・プロジェクト変数）はスキル本文の `{{VAR_NAME}}` テンプレートとして実行時に展開される。スキル作成時にパラメータ化したい値は `{{変数名}}` で記述すること。展開優先順位: ミニオン変数 < プロジェクト変数（後者が優先）。
+**変数**（ワークスペース変数・プロジェクト変数・ワークフロー変数・ミニオン変数）はスキル本文の `{{VAR_NAME}}` テンプレートとして実行時に展開される。スキル作成時にパラメータ化したい値は `{{変数名}}` で記述すること。展開優先順位: ワークスペース < プロジェクト < ワークフロー < ミニオン（後者が優先）。ミニオンが最終オーバーライドとなる設計で、ミニオン運用者がHQ側のデフォルトを差し替えられる経路を保証する。ミニオン変数はさらにミニオン全体スコープとワークスペース別スコープに分かれ、当該ワークスペースのコンテキストで動く実行時はWS別がミニオン全体を上書きする。`/api/variables/*` は `?workspace_id=<uuid>` でスコープ指定（省略時はミニオン全体）。
 
-**シークレット**（ミニオンシークレット）はサーバー起動時に `process.env` にロードされ、全子プロセスで環境変数 `$SECRET_NAME` として利用可能。APIキーやパスワード等の機密情報に使用する。シークレットは `{{VAR}}` テンプレートでは展開されない。
+**シークレット**はワークスペース別にスコープされ、ミニオンローカルのSQLite (`secrets(workspace_id, key, value)`) に保存される。ワークスペース未指定（`workspace_id=''`）はミニオン全体のスコープで、サーバー起動時に `process.env` にロードされ全子プロセスで `$SECRET_NAME` として利用可能。ワークスペース別シークレット（`workspace_id=<uuid>`）は `process.env` にはロードされず、当該ワークスペースのコンテキストで動くランナー（ワークフロー/WSバインドされたルーティン/チャットセッション）にのみ実行時注入される。同名キーが両スコープに存在する場合はワークスペース別が優先。スキル側は常に `$KEY` で参照すればよく、どちらのスコープから来た値かを意識する必要はない。シークレット値はHQ DBに保存されることはなく、HQはpass-throughとして中継するのみ。
 
 デイリーログやメモリーから変数・シークレットの値を推測して使用してはならない。変数は `{{VAR_NAME}}` テンプレートとして定義し、シークレットは環境変数として参照すること。
 
@@ -321,7 +322,8 @@ API詳細は `~/.minion/docs/api-reference.md` の「Todos」セクションを
      "thread_type": "help",
      "title": "問題の要約（1行）",
      "content": "状況の詳細説明",
-     "mentions": ["role:pm"],
+     "mentions": ["user:<auth_user_id>"],  // 特定の人に聞くなら個別指名 (推奨)
+                                            // 誰でも良いなら ["user"] / ["role:pm"] 等
      "context": {
        "category": "auth|environment|external-service|information|approval",
        "attempted_resolution": "試行した内容"
@@ -335,7 +337,7 @@ API詳細は `~/.minion/docs/api-reference.md` の「Todos」セクションを
      "thread_type": "help",
      "title": "問題の要約（1行）",
      "content": "状況の詳細説明",
-     "mentions": ["user"],
+     "mentions": ["user:<auth_user_id>"],  // 個別指名 (推奨)。誰でも良いなら ["user"]
      "context": {
        "category": "auth|environment|external-service|information|approval",
        "attempted_resolution": "試行した内容"
@@ -365,18 +367,25 @@ API詳細は `~/.minion/docs/api-reference.md` の「Todos」セクションを
 
 ### メンションの使い分け
 
-| 状況 | メンション |
-|------|-----------|
-| PMに判断を仰ぐ | `["role:pm"]` |
-| エンジニアの知見が必要 | `["role:engineer"]` |
-| 特定ミニオンに聞く | `["minion:<minion_id>"]` |
-| 人間の介入が必要（認証コード、承認等） | `["user"]` |
-| チーム全体に共有 | `[]`（メンションなし） |
+| 状況 | メンション | 備考 |
+|------|-----------|------|
+| 特定の人間を指名（推奨） | `["user:<auth_user_id>"]` | 事前に Members API で `user_id` を解決すること |
+| PMに判断を仰ぐ | `["role:pm"]` | プロジェクト紐づきスレッドのみ有効 |
+| エンジニアの知見が必要 | `["role:engineer"]` | プロジェクト紐づきスレッドのみ有効 |
+| 特定ミニオンに聞く | `["minion:<minion_id>"]` | |
+| 誰でも良いから人間に聞く | `["user"]` | プロジェクト紐づきならプロジェクト人間メンバー、紐づきがなければワークスペース人間メンバー全員 |
+| チーム全体に共有 | `[]`（メンションなし） | |
+
+**人間メンバーの user_id 解決:**
+- プロジェクト紐づきスレッド: `GET /api/minion/me/project/<project_id>/members` で `humans[].user_id` を取得
+- ワークスペーススレッド: `GET /api/minion/workspaces/<workspace_id>/members` で `humans[].user_id` を取得
+
+display_name が同じユーザーが同一ワークスペースに複数いる可能性があるため、原則として個別指名 (`user:<id>`) を優先する。「誰でも良い」場合のみ generic `user` を使う。
 
 ### 重要なルール
 
 - **スレッド起票時は `attempted_resolution` を必ず含める。** 何を試したか不明だと、回答者が同じことを提案してしまう。
-- **人間にしか解決できない問題（認証コード入力、外部サービスのパスワード等）は即 `@user` メンション**で起票する。自己解決を試みて時間を浪費しないこと。
+- **人間にしか解決できない問題（認証コード入力、外部サービスのパスワード等）は即メンション付き** で起票する。自己解決を試みて時間を浪費しないこと。特定の人にしか分からないことなら個別指名 (`user:<id>`)、誰でも良い性質の問題なら `user`(generic) を使う。
 - **ブロッカーが解決したらスレッドを `resolve` する。** 放置しない。
 - **解決策をプロジェクトメモリーに保存する。** 同じブロッカーに再度遭遇する他のミニオンの助けになる。
 

package/win/routine-runner.js

@@ -12,12 +12,14 @@ const { stripAnsi } = require('../core/lib/strip-ansi')
 const fs = require('fs').promises
 const fsSync = require('fs')
 
-const { config } = require('../core/config')
+const { config, isHqConfigured } = require('../core/config')
+const api = require('../core/api')
 const executionStore = require('../core/stores/execution-store')
 const routineStore = require('../core/stores/routine-store')
 const logManager = require('../core/lib/log-manager')
 const { activeSessions } = require('./workflow-runner')
 const { expandSkillTemplates, restoreSkillTemplates } = require('../core/lib/template-expander')
+const { buildSpawnEnv } = require('../core/lib/session-env')
 const { getActivePrimary } = require('../core/llm-plugins/lib/active')
 const os = require('os')
 
@@ -28,6 +30,25 @@ function sleep(ms) {
   return new Promise((resolve) => setTimeout(resolve, ms))
 }
 
+/**
+ * Fetch workspace-scoped variables for a routine from HQ.
+ * Mirrors the Linux runner; see that file for full docs.
+ */
+async function fetchWorkspaceVars(workspaceId) {
+  if (!workspaceId || !isHqConfigured()) return {}
+  try {
+    const result = await api.request(`/workspaces/${workspaceId}/variables`)
+    const vars = {}
+    for (const v of (result?.variables || [])) {
+      if (v && typeof v.key === 'string') vars[v.key] = String(v.value ?? '')
+    }
+    return vars
+  } catch (err) {
+    console.error(`[RoutineRunner] Failed to fetch workspace vars (${workspaceId}): ${err.message}`)
+    return {}
+  }
+}
+
 function generateSessionName(routineId, executionId) {
   const routineShort = routineId ? routineId.substring(0, 8) : 'manual'
   const execShort = executionId ? executionId.substring(0, 4) : ''
@@ -67,10 +88,15 @@ async function executeRoutineSession(routine, executionId, skillNames) {
   console.log(`[RoutineRunner] Session: ${sessionName}`)
   console.log(`[RoutineRunner] Log file: ${logFile}`)
 
-  // Expand {{VAR}} templates in SKILL.md files with minion variables
+  // Fetch HQ workspace variables when the routine is bound to a workspace.
+  // Minion variables (minion-wide ∪ WS-scoped, resolved inside the expander
+  // via workspaceId) override these as the final layer.
+  const workspaceVars = await fetchWorkspaceVars(routine.workspace_id)
+
+  // Expand {{VAR}} templates in SKILL.md files
   let expandedOriginals = new Map()
   try {
-    expandedOriginals = await expandSkillTemplates(skillNames)
+    expandedOriginals = await expandSkillTemplates(skillNames, workspaceVars, routine.workspace_id || '')
     if (expandedOriginals.size > 0) {
       console.log(`[RoutineRunner] Expanded templates in ${expandedOriginals.size} skill(s)`)
     }
@@ -101,14 +127,15 @@ async function executeRoutineSession(routine, executionId, skillNames) {
       throw new Error('No LLM configured. Set a Primary plugin via /api/llm/config or LLM_COMMAND in minion.env')
     }
 
-    // PATH, HOME, USERPROFILE, and minion secrets are already set in
-    // process.env at server startup. Per-execution identifiers are added here.
-    const env = {
-      ...process.env,
+    // PATH, HOME, USERPROFILE are inherited; workspace-scoped secrets are
+    // merged in via buildSpawnEnv() so a routine only sees secrets relevant
+    // to the workspace it is bound to.
+    const env = buildSpawnEnv(routine.workspace_id || '', {
       MINION_EXECUTION_ID: executionId,
       MINION_ROUTINE_ID: routine.id,
       MINION_ROUTINE_NAME: routine.name,
-    }
+      MINION_ROUTINE_WORKSPACE_ID: routine.workspace_id || '',
+    })
 
     const logDir = path.dirname(logFile)
     await fs.mkdir(logDir, { recursive: true })

package/win/workflow-runner.js

@@ -24,6 +24,7 @@ const executionStore = require('../core/stores/execution-store')
 const workflowStore = require('../core/stores/workflow-store')
 const logManager = require('../core/lib/log-manager')
 const { expandSkillTemplates, restoreSkillTemplates } = require('../core/lib/template-expander')
+const { buildSpawnEnv } = require('../core/lib/session-env')
 const { getActivePrimary } = require('../core/llm-plugins/lib/active')
 const os = require('os')
 
@@ -142,10 +143,11 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
   console.log(`[WorkflowRunner] Log file: ${logFile}`)
   console.log(`[WorkflowRunner] HOME: ${homeDir}`)
 
-  // Expand {{VAR}} templates in SKILL.md files with minion variables
+  // Expand {{VAR}} templates in SKILL.md files with minion variables effective
+  // for this workflow's workspace (minion-wide ∪ WS-scoped, WS wins).
   let expandedOriginals = new Map()
   try {
-    expandedOriginals = await expandSkillTemplates(skillNames)
+    expandedOriginals = await expandSkillTemplates(skillNames, {}, workflow.workspace_id || '')
     if (expandedOriginals.size > 0) {
       console.log(`[WorkflowRunner] Expanded templates in ${expandedOriginals.size} skill(s)`)
     }
@@ -178,8 +180,9 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
       throw new Error('No LLM configured. Set a Primary plugin via /api/llm/config or LLM_COMMAND in minion.env')
     }
 
-    // PATH, HOME, USERPROFILE, and minion secrets are already set in
-    // process.env at server startup, so child processes inherit them automatically.
+    // PATH, HOME, USERPROFILE are inherited from process.env (set at startup);
+    // workspace-scoped secrets are merged in via buildSpawnEnv() so this
+    // session only sees secrets relevant to its workspace context.
 
     // Open log file for streaming writes
     const logDir = path.dirname(logFile)
@@ -195,7 +198,11 @@ async function executeWorkflowSession(workflow, executionId, skillNames, options
       cols: 200,
       rows: 50,
       cwd: homeDir,
-      env: process.env,
+      env: buildSpawnEnv(workflow.workspace_id || '', {
+        MINION_EXECUTION_ID: executionId,
+        MINION_WORKFLOW_ID: workflow.id || '',
+        MINION_WORKFLOW_WORKSPACE_ID: workflow.workspace_id || '',
+      }),
     })
 
     // Track session