@geekbeer/minion 3.60.1 → 3.60.5

package/docs/api-reference.md

@@ -2084,9 +2084,9 @@ POST レスポンス例:
 
 | Method | Endpoint | Description |
 |--------|----------|-------------|
-| POST | `/api/minion/workspaces/:id/drive/decks/:deckId/slides` | スライド追加。Body: `{position?, background?, speaker_notes?}` (`position` 省略時は末尾) |
+| POST | `/api/minion/workspaces/:id/drive/decks/:deckId/slides` | スライド追加。Body: `{position?, background?, speaker_notes?}` (`position` 省略時は末尾)。`background` の形式は下記参照 |
 | GET | `/api/minion/workspaces/:id/drive/slides/:slideId` | スライド + 全要素 (z-order 昇順) |
-| PATCH | `/api/minion/workspaces/:id/drive/slides/:slideId` | スライド更新。Body: `{version, background?, speaker_notes?, order_key?, is_skipped?}`。`version` 必須。`is_skipped=true` でプレゼン時に飛ばされる(順序は保持) |
+| PATCH | `/api/minion/workspaces/:id/drive/slides/:slideId` | スライド更新。Body: `{version, background?, speaker_notes?, order_key?, is_skipped?}`。`version` 必須。`background` の形式は下記参照。`is_skipped=true` でプレゼン時に飛ばされる(順序は保持) |
 | DELETE | `/api/minion/workspaces/:id/drive/slides/:slideId` | スライド削除 (要素は CASCADE) |
 
 #### 要素操作 (text / image)
@@ -2097,6 +2097,23 @@ POST レスポンス例:
 | PATCH | `/api/minion/workspaces/:id/drive/elements/:elementId` | 要素更新。Body: `{version, x?, y?, width?, height?, rotation?, data?, order_key?}`。`version` 必須、`element_type` は不変 |
 | DELETE | `/api/minion/workspaces/:id/drive/elements/:elementId` | 要素削除 |
 
+#### background ペイロード形式
+
+`slides.background` および POST/PATCH の `background` フィールドは **オブジェクト必須** (文字列 `"#fff"` 等は 400 で弾かれる)。形式は以下の2種類のみ:
+
+```json
+{ "type": "color", "value": "#ffffff" }
+```
+
+```json
+{ "type": "image", "src": "deck:<uuid>.<ext>" }
+```
+
+- `type` は `"color"` または `"image"` のいずれか必須
+- `color` の場合 `value` (任意の CSS color 文字列、通常は `#rrggbb`) が必須
+- `image` の場合 `src` (非空文字列) が必須。形式は下記「image data の src」と同じルール
+- 400 `invalid_background` が返るのは、オブジェクトでない / `type` が不明 / 必須フィールド欠落のいずれか
+
 #### data ペイロード形式
 
 **text:**
@@ -2115,12 +2132,63 @@ POST レスポンス例:
 }
 ```
 
+`style` 内の **全フィールドは必須** (省略すると 400)。値の制約:
+
+| フィールド | 型 | 許容値 |
+|-----------|----|-------|
+| `fontFamily` | string | 任意の CSS font-family 文字列 (空文字不可) |
+| `fontSize` | number | `> 0` |
+| `fontWeight` | number | `400` または `700` のみ |
+| `fontStyle` | string | `"normal"` または `"italic"` のみ |
+| `color` | string | 任意の CSS color 文字列 |
+| `textAlign` | string | `"left"` / `"center"` / `"right"` のみ |
+| `lineHeight` | number | `> 0` |
+
+`content` は HTML を含めて良い。サーバ側 (`src/lib/workspace/slide-html.ts`) で allowlist サニタイズを通り、許可外のタグはテキストとして escape され、許可外の属性・スタイルプロパティは破棄される。
+
+**許可タグ一覧:**
+
+| タグ | 用途 |
+|------|------|
+| `<p>` | 段落 (Tiptap が改行ごとに自動付与) |
+| `<br>` | 強制改行 (void タグ) |
+| `<strong>` | 太字 |
+| `<em>` | 斜体 |
+| `<u>` | 下線 |
+| `<s>` | 打ち消し線 |
+| `<ul>` `<ol>` `<li>` | 箇条書き / 番号付きリスト |
+| `<span style="...">` | インライン書式 (下記スタイル限定) |
+
+**許可属性:** `span` の `style` のみ。`class` `id` `data-*` `href` 等は全て破棄される。
+
+**許可スタイルプロパティ:**
+
+| プロパティ | 許容値 |
+|-----------|-------|
+| `color` | `#rgb` / `#rrggbb` / `#rrggbbaa` (3〜8桁の hex)、`rgb(...)` / `rgba(...)`、`transparent`、CSS 色名 (`red` 等、英字1〜30文字) |
+| `background-color` | 同上 (ハイライトに使用) |
+| `font-size` | 数値 + 単位 (`px` / `em` / `rem` / `pt` / `%`) |
+
+スタイル値に `url(...)` `expression(...)` `javascript:` `<` `>` を含むものは破棄される。`<span style="...">` で書式を組み合わせるときは `;` 区切りで複数プロパティを並べる:
+
+```html
+<p>通常 <strong>太字</strong> <em>斜体</em> <span style="color: #c00">赤字</span></p>
+<ul><li>項目1</li><li>項目2 <span style="background-color: yellow">ハイライト</span></li></ul>
+<ol><li>番号付き</li></ol>
+```
+
+サニタイザは未閉じタグを末尾で自動クローズし、`<` `>` `&` を含む生テキストは entity に escape する。生 JSON で `content` を組み立てるときは、引用符の escape (`\"`) と HTML entity の二重エスケープに注意。
+
 **image:**
 ```json
-{ "src": "<workspace_files.id>", "fit": "contain" }
+{ "src": "deck:<image_uuid>.<ext>", "fit": "contain" }
 ```
 
-`src` が UUID 形式なら表示時に Drive 内画像として解決される。外部URLも指定可。
+`fit` は `"contain"` / `"cover"` / `"fill"` のみ。`src` の受け付け形式:
+
+- `deck:<image_uuid>.<ext>` — デッキ専用 Storage 上の画像 (HQ UI から `POST .../decks/:deckId/images` でアップロード済みのもの)。レンダリング時にデッキ画像エンドポイントに解決される
+- `https://…` などの外部 URL — そのまま `<img src>` に渡される (pass-through)
+- 旧形式の `<workspace_files.id>` (bare UUID) は **非対応** (v3.61.0 で廃止)。古いデッキは作り直しが必要
 
 #### 座標系
 
@@ -2132,9 +2200,28 @@ POST レスポンス例:
 
 | Code | 意味 |
 |------|------|
-| 400 | リクエスト不正 (`version` 欠落、`element_type` 不明、座標が数値でない等) |
+| 400 | リクエスト不正 (`version` 欠落、`element_type` 不明、座標が数値でない、`invalid_background` (オブジェクトでない / type 不明 / 必須フィールド欠落)、`style.*` 欠落・不正値等) |
 | 403 | `drive_not_enabled` (ワークスペースで Drive 未有効化) または `Minion is not a member` |
 | 404 | `not_found` (ID 間違いか別ワークスペースのリソース) |
 | 409 | `version_conflict` (他者の編集と衝突)、`name_conflict`、`order_key_collision` (同時挿入の競合、再送可) |
 | 410 | `deck_in_trash` (デッキがゴミ箱にある) |
 
+### Accounting Receipts 🧪 (HQ, experimental)
+
+`workspaces.feature_flags.experimental_accounting = true` のワークスペース向け。
+レシート (画像/PDF) の一覧取得・ダウンロード・取引紐付けをBearer認証で行える。
+**典型用途**: 人間が `/accounting/receipts` から大量アップロードした未仕訳レシートを
+ミニオン側で OCR → 仕訳作成 → 紐付け、という記帳ワークフロー。
+
+| Method | Endpoint | 説明 |
+|--------|----------|-------------|
+| GET | `/api/minion/workspaces/:id/accounting/receipts` | レシート一覧。Query: `?attached=true\|false\|all` (default `false`), `?limit=100` (max 500) |
+| GET | `/api/minion/workspaces/:id/accounting/receipts/:receiptId` | メタデータ |
+| GET | `/api/minion/workspaces/:id/accounting/receipts/:receiptId/download` | 60秒有効な signed URL を返す (`{ url, mime_type, file_name, expires_in_seconds }`) |
+| PATCH | `/api/minion/workspaces/:id/accounting/receipts/:receiptId` | 取引と紐付け / 解除。Body: `{ entry_id: string \| null }`。entry_id 指定時は同じ book に属する取引でないと 400 |
+
+レシートのMIME種別は `image/png`, `image/jpeg`, `image/webp`, `image/gif`, `image/heic`, `image/heif`, `application/pdf` のいずれか。
+
+**未実装**: ミニオン向け仕訳作成 API (`POST /api/minion/workspaces/:id/accounting/entries`) は現状未実装。
+OCR後の仕訳作成は、当面ユーザー操作で `/accounting/new` から行うか、HQ管理者向けの Supabase session 経由のみ。
+

package/docs/task-guides.md

@@ -918,7 +918,8 @@ meeting_end()  # DELETE /api/minion/meetings/:id を裏で叩く
 
 - **JSON 全体を送る方式ではない。** 1要素単位で PATCH するため、ミニオンが扱うコンテキストは常に小さい
 - **1テキスト要素 = 単一スタイル。** 部分太字などはできない。スタイルを変えたい区間は別要素として配置する
-- **画像は Drive にアップロード済みであることが前提。** `POST .../drive/files` で画像をアップロードした後、その `workspace_files.id` を `data.src` に貼る
+- **画像は外部 URL またはデッキ画像参照のいずれか。** `data.src` には `"https://..."` (passthrough) か `"deck:<uuid>.<ext>"` (HQ UI から `POST .../decks/:deckId/images` でアップロード済みの形式) を指定する。旧来の `workspace_files.id` 形式は v3.61.0 で廃止された。ミニオン API には現状デッキ画像アップロードのミラーが無いため、ミニオン主導でデッキ内画像を増やしたい場合は外部 URL を使うか、人間ユーザーに HQ から事前アップロードしてもらう
+- **背景 (`background`) はオブジェクト必須。** `{type: "color", value: "#..."}` または `{type: "image", src: "deck:..."}` のどちらか。文字列 `"#fff"` を直接渡すと 400 になる。詳細は `~/.minion/docs/api-reference.md` の `background ペイロード形式` を参照
 - **削除はソフト削除** (`DELETE .../decks/:deckId` で Drive ゴミ箱に入る)。誤削除は UI から復元できる
 - スライドマスター・テーマ・トランジション・アニメーションは MVP ではサポート外
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@geekbeer/minion",
-  "version": "3.60.1",
+  "version": "3.60.5",
   "description": "AI Agent runtime for Minion - manages status and skill deployment on VPS",
   "main": "linux/server.js",
   "bin": {

package/win/meeting-runner.js

@@ -1,65 +1,69 @@
 /**
- * Meeting Runner (Windows / WSL)
+ * Meeting Runner (Windows / native)
  *
- * Spawns a dedicated tmux session inside WSL per meeting invitation.
- * Mirrors linux/meeting-runner.js but routes commands through WSL since
- * Windows native tmux is not available.
+ * Spawns claude inside a node-pty session per meeting invitation. Mirrors
+ * linux/meeting-runner.js in behavior but uses node-pty + cmd.exe instead of
+ * tmux — the same pattern as win/workflow-runner.js.
  *
- * Session naming: `mt-{meetingId.slice(0,8)}`
+ * WSL is intentionally NOT used: meeting participation is a pure HTTP bridge
+ * via the meeting MCP server (see core/meetings/meeting-mcp-server.js) with
+ * no Linux toolchain requirement, and claude CLI runs natively on Windows.
+ *
+ * Session naming: `mt-{meetingId.slice(0,8)}` (same as Linux for parity)
+ *
+ * Lifecycle:
+ *   1. Receive invitation from HQ via routes/meetings POST /api/meetings/invitations.
+ *   2. If an active session for this meeting already exists in-process,
+ *      return early (idempotent: HQ may retry pushes).
+ *   3. Write temporary MCP config + kickoff prompt, then spawn claude via
+ *      cmd.exe with --mcp-config and --allowedTools flags.
+ *   4. pty exits when claude exits (after meeting_leave is called or the
+ *      meeting was deleted on HQ side); onExit cleans up tracking.
  */
 
-const { exec } = require('child_process')
-const { promisify } = require('util')
 const fs = require('fs').promises
+const fsSync = require('fs')
 const os = require('os')
 const path = require('path')
-const execAsync = promisify(exec)
 
 const { config } = require('../core/config')
 const runningTasks = require('../core/lib/running-tasks')
 const logManager = require('../core/lib/log-manager')
+const { stripAnsi } = require('../core/lib/strip-ansi')
 const { getActivePrimary } = require('../core/llm-plugins/lib/active')
 const { buildMeetingKickoffPrompt } = require('../core/meetings/meeting-prompt')
 
+/**
+ * Active pty sessions keyed by session name.
+ * @type {Map<string, {pty: object, logStream: object|null, meetingId: string, startedAt: string}>}
+ */
+const activeSessions = new Map()
+
 function generateSessionName(meetingId) {
   if (!meetingId) throw new Error('meetingId is required')
   return `mt-${String(meetingId).substring(0, 8)}`
 }
 
-function wslExec(cmd) {
-  return execAsync(`wsl bash -lc ${JSON.stringify(cmd)}`)
-}
-
-async function tmuxHasSession(sessionName) {
-  try {
-    await wslExec(`tmux has-session -t "${sessionName}" 2>/dev/null`)
-    return true
-  } catch {
-    return false
-  }
+function loadNodePty() {
+  try { return require('node-pty-prebuilt-multiarch') } catch {}
+  try { return require('node-pty') } catch {}
+  throw new Error(
+    'node-pty is required for Windows meeting participation. ' +
+    'Install with: npm install node-pty-prebuilt-multiarch'
+  )
 }
 
 async function listMeetingSessions() {
-  try {
-    const { stdout } = await wslExec(`tmux ls -F '#S' 2>/dev/null`)
-    return stdout
-      .split('\n')
-      .map((s) => s.trim())
-      .filter((s) => s.startsWith('mt-'))
-  } catch {
-    return []
-  }
+  return Array.from(activeSessions.keys())
 }
 
 async function killMeetingSession(meetingId) {
   const sessionName = generateSessionName(meetingId)
-  try {
-    await wslExec(`tmux kill-session -t "${sessionName}" 2>/dev/null`)
-    runningTasks.remove(sessionName)
-    return true
-  } catch {
-    return false
-  }
+  const session = activeSessions.get(sessionName)
+  if (!session) return false
+  try { session.pty.kill() } catch { /* ignore */ }
+  // onExit handler removes from activeSessions and runningTasks
+  return true
 }
 
 async function runMeeting({ meetingId, title, purpose, host, hqUrl, selfName, role }) {
@@ -69,17 +73,19 @@ async function runMeeting({ meetingId, title, purpose, host, hqUrl, selfName, ro
 
   const sessionName = generateSessionName(meetingId)
 
-  if (await tmuxHasSession(sessionName)) {
-    console.log(`[MeetingRunner-win] tmux session ${sessionName} already exists, skipping start`)
+  if (activeSessions.has(sessionName)) {
+    console.log(`[MeetingRunner-win] Session ${sessionName} already active, skipping start`)
     return { sessionName, started: false, success: true }
   }
 
   const tmpdir = os.tmpdir()
   const promptFile = path.join(tmpdir, `minion-meeting-prompt-${sessionName}.txt`)
   const mcpConfigFile = path.join(tmpdir, `minion-meeting-mcp-${sessionName}.json`)
-  const execScript = path.join(tmpdir, `minion-meeting-exec-${sessionName}.sh`)
   const mcpServerPath = path.join(__dirname, '..', 'core', 'meetings', 'meeting-mcp-server.js')
 
+  console.log(`[MeetingRunner-win] Starting meeting ${meetingId} (${title})`)
+  console.log(`[MeetingRunner-win] Session: ${sessionName}`)
+
   try {
     await logManager.ensureLogDir()
 
@@ -93,9 +99,8 @@ async function runMeeting({ meetingId, title, purpose, host, hqUrl, selfName, ro
     })
     await fs.writeFile(promptFile, prompt, 'utf-8')
 
-    // Use the minion's own config.HQ_URL when available. HQ-pushed hq_url is
-    // only a fallback because the minion already has the correct URL set at
-    // provisioning time.
+    // Use the minion's own config.HQ_URL — set correctly at provisioning time.
+    // Falling back to the URL pushed by HQ only if local config is missing.
     const effectiveHqUrl = config.HQ_URL || hqUrl
     if (!effectiveHqUrl) {
       throw new Error('No HQ_URL configured (neither minion config nor HQ push)')
@@ -119,12 +124,15 @@ async function runMeeting({ meetingId, title, purpose, host, hqUrl, selfName, ro
     const primary = getActivePrimary()
     if (!primary || primary.name !== 'claude') {
       throw new Error(
-        'Meeting participation currently requires the claude primary LLM (MCP support).',
+        'Meeting participation currently requires the claude primary LLM (MCP support). ' +
+          `Active primary: ${primary ? primary.name : 'none'}`,
       )
     }
 
-    // See linux/meeting-runner.js for why --allowedTools is needed (-p mode
-    // hangs on MCP permission prompts without this).
+    // --allowedTools: Claude Code defaults to prompting the user for every MCP
+    // tool call, which hangs in -p mode (no interactive UI). Pre-authorize the
+    // meeting MCP tools so claude can run the participation loop without
+    // approval prompts.
     const allowedTools = [
       'mcp__meeting__meeting_wait_for_next_message',
       'mcp__meeting__meeting_speak',
@@ -133,24 +141,67 @@ async function runMeeting({ meetingId, title, purpose, host, hqUrl, selfName, ro
       'mcp__meeting__meeting_end',
     ].join(',')
 
-    await fs.writeFile(
-      execScript,
-      `#!/bin/bash\nexport HQ_URL="${effectiveHqUrl}"\nexport API_TOKEN="${config.API_TOKEN}"\nexport MINION_MEETING_ID="${meetingId}"\nclaude -p --mcp-config "${mcpConfigFile}" --allowedTools "${allowedTools}" < "${promptFile}"\n`,
-      'utf-8',
-    )
+    // cmd.exe handles the `<` redirection for stdin from promptFile.
+    // `claude` resolves via PATH (claude.cmd shim on Windows).
+    const llmCommand = `claude -p --mcp-config "${mcpConfigFile}" --allowedTools "${allowedTools}" < "${promptFile}"`
+
+    const pty = loadNodePty()
+    const shell = process.env.COMSPEC || 'cmd.exe'
+    const shellArgs = ['/c', llmCommand]
+
+    const ptyProcess = pty.spawn(shell, shellArgs, {
+      name: 'xterm-256color',
+      cols: 200,
+      rows: 50,
+      cwd: config.HOME_DIR,
+      env: {
+        ...process.env,
+        HQ_URL: effectiveHqUrl,
+        API_TOKEN: config.API_TOKEN,
+        MINION_MEETING_ID: meetingId,
+      },
+    })
+
+    const logDir = logManager.LOG_DIR || path.join(config.HOME_DIR, '.minion', 'logs')
+    const logFile = path.join(logDir, `meeting-${meetingId}.log`)
+    let logStream = null
+    try {
+      logStream = fsSync.createWriteStream(logFile, { flags: 'a' })
+    } catch (err) {
+      console.error(`[MeetingRunner-win] Failed to open log file ${logFile}: ${err.message}`)
+    }
+
+    const session = {
+      pty: ptyProcess,
+      logStream,
+      meetingId,
+      startedAt: new Date().toISOString(),
+    }
+    activeSessions.set(sessionName, session)
+
+    ptyProcess.onData((data) => {
+      if (logStream) {
+        try { logStream.write(stripAnsi(data)) } catch { /* ignore */ }
+      }
+    })
 
-    await wslExec(`chmod +x "${execScript}"`)
-    await wslExec(`tmux new-session -d -s "${sessionName}" -x 200 -y 50`)
-    await wslExec(`tmux set-option -t "${sessionName}" remain-on-exit on`)
-    await wslExec(`tmux send-keys -t "${sessionName}" "bash ${execScript}" Enter`)
+    ptyProcess.onExit(({ exitCode }) => {
+      console.log(`[MeetingRunner-win] Session ${sessionName} exited (code: ${exitCode})`)
+      if (logStream) {
+        try { logStream.end() } catch { /* ignore */ }
+      }
+      activeSessions.delete(sessionName)
+      runningTasks.remove(sessionName)
+    })
 
     runningTasks.add({
       type: 'meeting',
       session_name: sessionName,
       meeting_id: meetingId,
-      started_at: new Date().toISOString(),
+      started_at: session.startedAt,
     })
 
+    console.log(`[MeetingRunner-win] Meeting ${meetingId} session started (PID: ${ptyProcess.pid})`)
     return { sessionName, started: true, success: true }
   } catch (err) {
     console.error(`[MeetingRunner-win] Failed to start meeting ${meetingId}: ${err.message}`)
@@ -163,4 +214,5 @@ module.exports = {
   killMeetingSession,
   listMeetingSessions,
   generateSessionName,
+  activeSessions,
 }