@turtton/oh-my-openagent 3.12.3-copilot.2 → 3.12.3-copilot.3

package/README.ja.md

@@ -25,12 +25,9 @@ OpenCodeの設定ファイルの `"plugin"` 配列にパッケージ名を追加
 
 ## 適用パッチ
 各パッチの説明:
-- **001-background-output-block-true**: `background_output`ツールのガイダンスを6つのソースファイルで変更します。upstreamのデフォルトではバックグラウンドタスク完了時に「レスポンスを終了してシステム通知を待つ」ことを推奨しており、各通知が新しいエージェントターンをトリガーしてPremium Requestを消費します。このパッチは代わりに`block=true`で現在のターン内で結果を待つことを推奨し、通知トリガーによる余分なPremium Request消費を回避します。
-    - 対象ファイル: `constants.ts`, `background-executor.ts`, `background-agent-executor.ts`, `create-background-task.ts`, `dynamic-agent-prompt-builder.ts`, `sisyphus.ts`
+- **001-block-true-waiting**: バックグラウンドタスクの結果取得時に`block=true`の使用を推奨し、不要なPremium Request消費を削減します。upstreamのデフォルトではバックグラウンドタスク完了時に「レスポンスを終了してシステム通知を待つ」ことを推奨しており、各通知が新しいエージェントターンをトリガーしてPremium Requestを消費します。このパッチは代わりに`block=true`で現在のターン内で結果を待つことを推奨し、ランタイム側もそれを適切にサポートするよう変更します: アクティブにポーリング中のタスクを追跡して冗長な通知を抑制（`noReply: true`）、`block=true`のタイムアウト制限を撤廃（無期限待機、`timeout`パラメータはdeprecatedとなり無視されます）、エージェントプロンプトとツール説明のガイダンステキストを全面更新。ポーリングされていないタスクについては、元の通知駆動の`noReply: !allComplete`ロジックが維持されます。
+    - 対象ファイル: `src/tools/background-task/constants.ts`, `src/tools/call-omo-agent/background-executor.ts`, `src/tools/call-omo-agent/background-agent-executor.ts`, `src/tools/background-task/create-background-task.ts`, `src/agents/dynamic-agent-prompt-builder.ts`, `src/agents/sisyphus.ts`, `src/tools/background-task/clients.ts`, `src/tools/background-task/types.ts`, `src/features/background-agent/manager.ts`, `src/tools/background-task/create-background-output.ts`
 - **002-disable-todo-continuation-enforcer**: `todo-continuation-enforcer`フックをデフォルトで無効化します。このフックはTODOリストに基づいて自動的に作業を継続し、意図しない追加のエージェントターン（とPremium Request）をトリガーする場合があります。設定初期化時のデフォルト`disabled_hooks`リストに追加することで無効化しています。
-- **003-noreply-true**: バックグラウンドタスク完了通知を常に`noReply: true`に強制します。このパッチがない場合、すべてのバックグラウンドタスクが完了したときに`noReply: false`の通知が不要な追加エージェントターンをトリガーします — `block=true`（パッチ001）による同一ターン内の結果取得で既に結果を受け取っていても発生します。このパッチは通知トリガーによる追加ターンを抑制し、そのターンが消費するPremium Requestを回避します。
-    - 対象ファイル: `manager.ts`（2箇所）
-    - **⚠️ 注意**: このパッチ適用後、`block=true`を使用しないエージェントはバックグラウンドタスク完了時に自動再開しなくなります。パッチ001がエージェントプロンプトで`block=true`を推奨することで緩和していますが、一部のエッジケース（例: 長時間タスクでの`block=true`タイムアウト）では手動介入が必要になる場合があります。
 
 ## 無効化されたフックの再有効化
 `todo-continuation-enforcer`フックを再有効化するには、プロジェクトルートに`opencode.json`を作成します:

package/README.md

@@ -29,11 +29,9 @@ OpenCode will automatically install and load the plugin on next startup.
 
 ## Patches Applied
 
-- **001-background-output-block-true**: Changes `background_output` tool guidance across 6 source files. The upstream default recommends agents "end their response and wait for system notifications" when background tasks complete — each such notification triggers a new agent turn, consuming a premium request. This patch instead recommends `block=true` to wait for results within the current turn, avoiding extra premium request consumption from notification-triggered responses.
-    - Files affected: `constants.ts`, `background-executor.ts`, `background-agent-executor.ts`, `create-background-task.ts`, `dynamic-agent-prompt-builder.ts`, `sisyphus.ts`
+- **001-block-true-waiting**: Encourages agents to use `block=true` when collecting background task results, reducing unnecessary premium request consumption. The upstream default recommends agents "end their response and wait for system notifications" when background tasks complete — each notification triggers a new agent turn, consuming a premium request. This patch instead recommends `block=true` to wait within the current turn, and makes the runtime support it properly: tracking which tasks are being actively polled to suppress redundant notifications (`noReply: true`), removing the `block=true` timeout limit (now waits indefinitely, with the `timeout` parameter deprecated and ignored), and updating all guidance text across agent prompts and tool descriptions. For tasks not being actively polled, the original notification-driven `noReply: !allComplete` logic is preserved.
+    - Files affected: `src/tools/background-task/constants.ts`, `src/tools/call-omo-agent/background-executor.ts`, `src/tools/call-omo-agent/background-agent-executor.ts`, `src/tools/background-task/create-background-task.ts`, `src/agents/dynamic-agent-prompt-builder.ts`, `src/agents/sisyphus.ts`, `src/tools/background-task/clients.ts`, `src/tools/background-task/types.ts`, `src/features/background-agent/manager.ts`, `src/tools/background-task/create-background-output.ts`
 - **002-disable-todo-continuation-enforcer**: Disables the `todo-continuation-enforcer` hook by default. This hook automatically continues work based on todo items, triggering additional agent turns (and premium requests) that may not be intended. Disabled by adding it to the default `disabled_hooks` list in the config initialization.
-- **003-noreply-true**: Intelligently suppresses background task completion notifications when the agent is already waiting via `block=true`. Instead of the previous blanket `noReply: true` approach, this patch tracks which tasks are being actively polled by `background_output(block=true)`. For those tasks, notifications use `noReply: true` to avoid triggering redundant agent turns. For tasks not being actively polled, the original `noReply: !allComplete` logic is preserved, allowing proper notification-driven resumption. Also removes the `block=true` timeout limit (now waits indefinitely) and registers/unregisters blocking state via `BackgroundManager`.
-    - Files affected: `clients.ts`, `types.ts`, `manager.ts`, `create-background-output.ts`
 
 ## Re-enabling Disabled Hooks
 

package/dist/features/background-agent/manager.d.ts

@@ -42,6 +42,7 @@ export declare class BackgroundManager {
     private idleDeferralTimers;
     private notificationQueueByParent;
     private blockingTaskRefCounts;
+    private wasBlockedTasks;
     private rootDescendantCounts;
     private preStartDescendantReservations;
     private enableParentSessionNotifications;

package/dist/index.js

@@ -78791,6 +78791,7 @@ class BackgroundManager {
   idleDeferralTimers = new Map;
   notificationQueueByParent = new Map;
   blockingTaskRefCounts = new Map;
+  wasBlockedTasks = new Set;
   rootDescendantCounts;
   preStartDescendantReservations;
   enableParentSessionNotifications;
@@ -78816,6 +78817,7 @@ class BackgroundManager {
   registerBlockingTask(taskId) {
     const current = this.blockingTaskRefCounts.get(taskId) ?? 0;
     this.blockingTaskRefCounts.set(taskId, current + 1);
+    this.wasBlockedTasks.add(taskId);
   }
   unregisterBlockingTask(taskId) {
     const current = this.blockingTaskRefCounts.get(taskId) ?? 0;
@@ -78826,7 +78828,7 @@ class BackgroundManager {
     }
   }
   isTaskBlocking(taskId) {
-    return (this.blockingTaskRefCounts.get(taskId) ?? 0) > 0;
+    return (this.blockingTaskRefCounts.get(taskId) ?? 0) > 0 || this.wasBlockedTasks.has(taskId);
   }
   async assertCanSpawn(parentSessionID) {
     const spawnContext = await resolveSubagentSpawnContext(this.client, parentSessionID);
@@ -79716,6 +79718,7 @@ ${originalText}`;
       }
       this.clearNotificationsForTask(taskId);
       this.tasks.delete(taskId);
+      this.wasBlockedTasks.delete(taskId);
       this.clearTaskHistoryWhenParentTasksGone(task.parentSessionID);
       if (task.sessionID) {
         subagentSessions.delete(task.sessionID);
@@ -79972,7 +79975,9 @@ Use \`background_output(task_id="${task.id}")\` to retrieve this result when rea
           noReply: shouldSuppressReply,
           isBlocking: this.isTaskBlocking(task.id)
         });
+        this.wasBlockedTasks.delete(task.id);
       } catch (error92) {
+        this.wasBlockedTasks.delete(task.id);
         if (isAbortedSessionError(error92)) {
           log("[background-agent] Parent session aborted while sending notification; continuing cleanup:", {
             taskId: task.id,
@@ -79984,6 +79989,7 @@ Use \`background_output(task_id="${task.id}")\` to retrieve this result when rea
         }
       }
     } else {
+      this.wasBlockedTasks.delete(task.id);
       log("[background-agent] Parent session notifications disabled, skipping prompt injection:", {
         taskId: task.id,
         parentSessionID: task.parentSessionID

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@turtton/oh-my-openagent",
-  "version": "3.12.3-copilot.2",
+  "version": "3.12.3-copilot.3",
   "description": "The Best AI Agent Harness - Batteries-Included OpenCode Plugin with Multi-Model Orchestration, Parallel Background Agents, and Crafted LSP/AST Tools (Copilot-compatible fork)",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",