@geekbeer/minion 4.2.1 → 4.3.3

package/skills/accounting-bookkeeping/references/troubleshooting.md

@@ -2,19 +2,35 @@
 
 会計APIで遭遇しやすいエラーパターンと対処。
 
+> パスはすべて **ミニオン用** `$HQ_URL/api/minion/workspaces/:id/accounting/` を基準とした相対表記。
+
 ## エラーレスポンスの構造
 
 ```json
 {
   "error": "missing_default_payable_account",
   "message": "取引先「山田太郎」に立替用の貸方科目が未設定です",
-  "next_action": "PATCH /api/accounting/counterparties/<id> で...",
+  "next_action": "PATCH /counterparties/<id> で default_payable_account_id を設定してください",
   "context": { "counterparty_id": "..." }
 }
 ```
 
 **必ず `next_action` を読み、その指示に従うこと。** 指示が曖昧な場合のみ自己判断する。
 
+## アクセス・認証まわり
+
+### `accountant_role_required` (403)
+
+**原因:** 呼び出しミニオンが workspace 内のどのプロジェクトでも accountant ロールを持っていない。
+
+**対処:** 自己解決不可。`POST /api/threads` で `@user` メンションし、accountant ロール付与を依頼する。
+
+### `feature_disabled` (403)
+
+**原因:** workspace で `experimental_accounting` フラグが OFF。
+
+**対処:** 自己解決不可。人間にフラグ有効化を依頼する。
+
 ## 立替経費まわり
 
 ### `missing_default_payable_account` (422)
@@ -23,9 +39,9 @@
 
 **対処:**
 1. liability 型の科目「役員借入金」「従業員未払金」などがあるか確認
-   - `GET /api/accounting/accounts` で `type=liability` をフィルタ
-2. 無ければ作成: `POST /api/accounting/accounts` で `{ name: "役員借入金", type: "liability" }`
-3. counterparty に紐付け: `PATCH /api/accounting/counterparties/<id>` で `{ default_payable_account_id: "..." }`
+   - `GET /accounts?type=liability`
+2. 無ければ作成: `POST /accounts` で `{ name: "役員借入金", type: "liability" }`
+3. counterparty に紐付け: `PATCH /counterparties/<id>` で `{ default_payable_account_id: "..." }`
 4. 立替記帳をリトライ
 
 ### `counterparty_not_found` (404)
@@ -33,8 +49,8 @@
 **原因:** 立替者の counterparty が未登録。
 
 **対処:**
-1. `GET /api/accounting/counterparties?q=<name>` で typo の可能性を確認
-2. 該当者がいなければ `POST /api/accounting/counterparties` で作成
+1. `GET /counterparties?kind=director,employee` で typo の可能性を確認
+2. 該当者がいなければ `POST /counterparties` で作成
    - 役員なら `kind: "director"`、従業員なら `kind: "employee"`
    - 可能なら `default_payable_account_id` も同時に設定
 
@@ -47,12 +63,6 @@
 - 同一人物なら既存IDを使う
 - 別人なら名前を変える(例: "山田太郎(役員)" vs "山田太郎(従業員)")
 
-### `already_settled` (409)
-
-**原因:** 精算しようとした立替が既に精算済み。
-
-**対処:** `context.settled_at` と `context.settlement_journal_entry_id` を確認し、二重精算を回避
-
 ## 期間まわり
 
 ### `period_closed` (409)
@@ -60,7 +70,7 @@
 **原因:** 指定日付が締め済期間内。
 
 **対処:**
-- 締めはPMの権限。勝手に再オープンしない
+- 締めはPMの権限。勝手に再オープンしない(再オープンは人間専用)
 - 締め後の取引なら日付を調整(後日付に変更)、または PM にエスカレーション
 - threadで `mentions: ["role:pm"]` を付けて「<DATE> の取引が締め後だが記帳が必要」と相談
 
@@ -70,13 +80,7 @@
 
 **原因:** 立替経費の費用科目に expense 型でない科目を指定した。
 
-**対処:** `GET /api/accounting/accounts` で type=expense の科目を取得して使用
-
-### `not_a_wallet_account` (400)
-
-**原因:** 精算時の支払口座に `is_wallet=false` の科目を指定した。
-
-**対処:** `is_wallet=true` の科目(現金/普通預金)を選択
+**対処:** `GET /accounts?type=expense` の科目を取得して使用
 
 ### `invalid_default_payable_account` (400)
 
@@ -84,6 +88,11 @@
 
 **対処:** liability 型(または equity 型)の科目を指定する。「立替金」(asset)は逆向きなので使わない
 
+### 科目の rename / アーカイブが必要なとき — **人間専用**
+
+ミニオンは `POST /accounts`(作成)のみ可能。誤って作った科目の修正・アーカイブ・削除はできない。
+`POST /api/threads` で `@user` メンションして HQ UI `/accounting/accounts` での操作を依頼する。
+
 ## 仕訳整合性
 
 ### `at least 2 lines required` / `debit/credit must balance`
@@ -95,6 +104,13 @@
 - 借方合計 = 貸方合計 を自前で計算してから POST
 - DB trigger は不整合を必ず検出するので、ローカル検算が大事
 
+### `human_owned_entry` (409)
+
+**原因:** 人間が作成した仕訳 (`source='manual'`/`'import'`) を PATCH しようとした。
+
+**対処:** ミニオンは自分が作成した仕訳 (`source='ai_generated'`) しか編集できない。
+人間作成仕訳の訂正が必要なら `POST /api/threads` で人間に依頼する。
+
 ## 取引先のアーカイブ
 
 ### `has_unsettled_reimbursements` (409)
@@ -102,9 +118,9 @@
 **原因:** 未精算の立替が残っている取引先をアーカイブしようとした。
 
 **対処:**
-1. `GET /api/accounting/expense-reimbursements?counterparty_id=<id>&settled=false` で未精算リスト取得
-2. 各立替を精算 (`POST /<id>/settle`)
-3. アーカイブを再実行
+1. `GET /reimbursements?counterparty_id=<id>&status=unsettled` で未精算リスト取得
+2. **精算は人間専用** — `POST /api/threads` で `@user` に精算を依頼する
+3. 全件精算後にアーカイブを再実行
 
 ## 自己解決できない場合
 
@@ -112,6 +128,7 @@
 
 1. プロジェクトメモリーで過去事例を検索
 2. なければ `POST /api/threads` でヘルプスレッド起票
-   - `thread_type: "help"`, `mentions: ["role:pm"]`
+   - `thread_type: "help"`, `mentions: ["role:pm"]` (人間判断が要るなら `["user"]`)
    - `attempted_resolution` に試したことを必ず書く
-3. **仕訳を中途半端な状態で残さない。** 失敗した立替記帳は `DELETE /api/accounting/expense-reimbursements/<id>` で取り消してからエスカレーション
+3. **仕訳を中途半端な状態で残さない。** ただしミニオンは仕訳・立替を DELETE できないため、
+   取消が必要な場合は状況を thread に明記して人間に削除を依頼する

package/win/server.js

@@ -33,6 +33,10 @@ const boardTaskRunner = require('./board-task-runner')
 // Config warnings (included in heartbeat)
 const { getConfigWarnings } = require('../core/lib/config-warnings')
 
+// Bundled skill deployment (version-gated, see core/lib/bundled-skills.js)
+const { syncBundledSkills } = require('../core/lib/bundled-skills')
+const { getActiveSkillDirs } = require('../core/llm-plugins/lib/skill-dirs')
+
 // Pull-model daemons (from core/)
 const stepPoller = require('../core/lib/step-poller')
 const dagStepPoller = require('../core/lib/dag-step-poller')
@@ -214,6 +218,25 @@ function syncBundledDocs() {
   }
 }
 
+/**
+ * (Re)deploy bundled skills to the active LLM skill directories when the package
+ * version changed. Lets `update-agent` (npm install + restart) propagate skill
+ * fixes without re-running configure, while preserving HQ-fetched customizations
+ * across ordinary restarts. See core/lib/bundled-skills.js.
+ */
+function syncBundledSkillsOnStartup() {
+  try {
+    syncBundledSkills({
+      packageRoot: path.join(__dirname, '..'),
+      version,
+      targetDirs: getActiveSkillDirs(),
+      markerPath: path.join(config.HOME_DIR, '.minion', 'skills-sync-version'),
+    })
+  } catch (err) {
+    console.error(`[Skills] Bundled skill sync skipped: ${err.message}`)
+  }
+}
+
 /**
  * Register all routes (compatible + Windows-specific)
  */
@@ -285,6 +308,7 @@ async function start() {
     syncBundledRules()
     syncBundledRoles()
     syncBundledDocs()
+    syncBundledSkillsOnStartup()
     syncPermissions()
     // Note: tmux.conf sync is skipped on Windows (not needed)