@pellux/goodvibes-tui 0.19.86 → 0.19.87

package/CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to GoodVibes TUI.
 
 ---
 
+## [0.19.87] — 2026-05-09
+
+### Changes
+- e8f6269e fix: defer wrfc topology to sdk 0.33.22
+
 ## [0.19.86] — 2026-05-09
 
 ### Changes

package/README.md

@@ -2,7 +2,7 @@
 
 [![CI](https://github.com/mgd34msu/goodvibes-tui/actions/workflows/ci.yml/badge.svg)](https://github.com/mgd34msu/goodvibes-tui/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
-[![Version](https://img.shields.io/badge/version-0.19.86-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
+[![Version](https://img.shields.io/badge/version-0.19.87-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
 
 A terminal-native AI coding, operations, automation, knowledge, and integration console with a typed runtime, omnichannel surfaces, structured memory/knowledge, and a raw ANSI renderer.
 
@@ -18,7 +18,7 @@ GoodVibes is a Bun program. Install Bun first and make sure `bun` is on `PATH`,
 
 ```sh
 bun add -g @pellux/goodvibes-tui
-bun pm trust -g @pellux/goodvibes-tui core-js tree-sitter-css tree-sitter-javascript tree-sitter-json tree-sitter-python tree-sitter-typescript
+bun pm trust -g @pellux/goodvibes-tui @pellux/goodvibes-sdk core-js tree-sitter-css tree-sitter-javascript tree-sitter-json tree-sitter-python tree-sitter-typescript
 goodvibes
 ```
 
@@ -53,7 +53,7 @@ Release distribution:
 
 - GitHub Releases are the primary distribution path for compiled binaries
 - `bun add -g @pellux/goodvibes-tui` is the recommended global install path; the package is hosted on the npm registry and Bun installs from that registry directly
-- Bun global installs require trusting `@pellux/goodvibes-tui` so the package postinstall can download the matching TUI and daemon binaries; the dependency lifecycle packages currently requiring trust are `core-js`, `tree-sitter-css`, `tree-sitter-javascript`, `tree-sitter-json`, `tree-sitter-python`, and `tree-sitter-typescript`
+- Bun global installs require trusting `@pellux/goodvibes-tui` so the package postinstall can download the matching TUI and daemon binaries; the dependency lifecycle packages currently requiring trust are `@pellux/goodvibes-sdk`, `core-js`, `tree-sitter-css`, `tree-sitter-javascript`, `tree-sitter-json`, `tree-sitter-python`, and `tree-sitter-typescript`
 - `npm install -g @pellux/goodvibes-tui` is supported on Linux, macOS, and WSL when Bun is already installed; the preinstall check verifies Bun, and the install script downloads the matching TUI and daemon binaries for the current platform
 - native Windows is not supported; use WSL on Windows
 

package/docs/foundation-artifacts/operator-contract.json

@@ -3,7 +3,7 @@
   "product": {
     "id": "goodvibes",
     "surface": "operator",
-    "version": "0.33.21"
+    "version": "0.33.22"
   },
   "auth": {
     "modes": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-tui",
-  "version": "0.19.86",
+  "version": "0.19.87",
   "description": "Terminal-native GoodVibes product for coding, operations, automation, knowledge, channels, and daemon-backed control-plane workflows.",
   "type": "module",
   "main": "src/main.ts",
@@ -97,7 +97,7 @@
     "@anthropic-ai/vertex-sdk": "^0.16.0",
     "@ast-grep/napi": "^0.42.0",
     "@aws/bedrock-token-generator": "^1.1.0",
-    "@pellux/goodvibes-sdk": "0.33.21",
+    "@pellux/goodvibes-sdk": "0.33.22",
     "bash-language-server": "^5.6.0",
     "fuse.js": "^7.1.0",
     "graphql": "^16.13.2",

package/src/tools/wrfc-agent-guard.ts

@@ -19,19 +19,6 @@ type AgentTaskArgs = {
   readonly [key: string]: unknown;
 };
 
-const OWNER_BLOCKED_TEMPLATES = new Set(['reviewer', 'review', 'verifier', 'tester', 'test']);
-const OWNER_BLOCKED_TASK_PREFIXES = [
-  'review ',
-  'review:',
-  'review the ',
-  'verify ',
-  'verify:',
-  'verify the ',
-  'test ',
-  'test:',
-  'test the ',
-];
-
 export function installWrfcAgentToolGuard(registry: ToolRegistry): void {
   const agentTool = registry.list().find((tool) => tool.definition.name === 'agent');
   if (!agentTool) throw new Error('WRFC agent guard could not find the agent tool.');
@@ -48,17 +35,9 @@ export function wrapWrfcAgentTool(tool: Tool): void {
 }
 
 export function validateWrfcAgentToolInvocation(args: AgentToolArgs): string | null {
-  if (args.mode === 'spawn') {
-    if (isExplicitWrfcTask(args, args) && isBlockedRootTask(args, args)) {
-      return [
-        'WRFC spawn blocked: a WRFC root task must be an owner/engineer task, not a reviewer/verifier/tester task.',
-        'Spawn one engineer/general owner with reviewMode:"wrfc"; WRFC creates reviewer and fixer agents only after owner output exists.',
-      ].join(' ');
-    }
-    return null;
-  }
-
-  if (args.mode !== 'batch-spawn') return null;
+  if (args.mode !== 'spawn' && args.mode !== 'batch-spawn') return null;
+  // SDK owns WRFC topology enforcement. TUI must not block reviewer/tester/
+  // verifier root requests because the SDK normalizes those into owner chains.
   return null;
 }
 
@@ -81,23 +60,17 @@ export function normalizeWrfcAgentToolInvocation(args: AgentToolArgs): AgentTool
         : { ...task, reviewMode: 'none', dangerously_disable_wrfc: true }),
     };
   }
-  if (tasks.length === 1 && wrfcTasks.length === 1 && !isBlockedRootTask(tasks[0], args)) {
+  if (wrfcTasks.length > 0) {
     return {
       ...args,
       reviewMode: 'wrfc',
       dangerously_disable_wrfc: false,
-      tasks: [{ ...tasks[0], reviewMode: 'wrfc', dangerously_disable_wrfc: false }],
+      tasks: tasks.map((task) => isExplicitWrfcTask(task, args)
+        ? { ...task, reviewMode: 'wrfc', dangerously_disable_wrfc: false }
+        : task),
     };
   }
-
-  const ownerTask = buildCollapsedWrfcOwnerTask(args, tasks);
-  return {
-    ...args,
-    template: normalizeOwnerTemplate(args.template),
-    reviewMode: 'wrfc',
-    dangerously_disable_wrfc: false,
-    tasks: [ownerTask],
-  };
+  return args;
 }
 
 function isRecord(value: unknown): value is AgentTaskArgs {
@@ -113,53 +86,6 @@ function isExplicitWrfcTask(task: AgentTaskArgs, root: AgentToolArgs): boolean {
     || containsWrfcSignal(root.task);
 }
 
-function isBlockedOwnerTemplate(value: unknown): boolean {
-  if (typeof value !== 'string') return false;
-  return OWNER_BLOCKED_TEMPLATES.has(value.trim().toLowerCase());
-}
-
-function isBlockedRootTask(task: AgentTaskArgs, root: AgentToolArgs): boolean {
-  if (isBlockedOwnerTemplate(task.template ?? root.template)) return true;
-  if (typeof task.task !== 'string') return false;
-  const normalized = task.task.trim().toLowerCase();
-  return OWNER_BLOCKED_TASK_PREFIXES.some((prefix) => normalized.startsWith(prefix));
-}
-
-function normalizeOwnerTemplate(value: unknown): string {
-  if (isBlockedOwnerTemplate(value)) return 'engineer';
-  return typeof value === 'string' && value.trim().length > 0 ? value : 'engineer';
-}
-
-function buildCollapsedWrfcOwnerTask(root: AgentToolArgs, taskList: AgentTaskArgs[]): AgentTaskArgs {
-  const firstOwner = taskList.find((task) => isExplicitWrfcTask(task, root) && !isBlockedRootTask(task, root))
-    ?? taskList.find((task) => !isBlockedRootTask(task, root))
-    ?? taskList[0]
-    ?? {};
-  const taskLines = taskList.map((task, index) => {
-    const body = typeof task.task === 'string' && task.task.trim().length > 0 ? task.task.trim() : '(missing task text)';
-    const template = typeof task.template === 'string' && task.template.trim().length > 0 ? task.template.trim() : 'default';
-    return `${index + 1}. [${template}] ${body}`;
-  });
-  const rootTask = typeof root.task === 'string' && root.task.trim().length > 0
-    ? `\n\nRoot task:\n${root.task.trim()}`
-    : '';
-  return {
-    ...firstOwner,
-    task: [
-      'Complete the requested work as a single WRFC owner chain.',
-      'Do not spawn reviewer, verifier, tester, or parallel root WRFC agents yourself.',
-      'Use the attempted batch items below as context for one coherent owner deliverable; the WRFC controller will create review/fix agents after owner output exists.',
-      rootTask,
-      '',
-      'Attempted batch items:',
-      ...taskLines,
-    ].join('\n'),
-    template: normalizeOwnerTemplate(firstOwner.template ?? root.template),
-    reviewMode: 'wrfc',
-    dangerously_disable_wrfc: false,
-  };
-}
-
 function containsWrfcSignal(value: unknown): boolean {
   if (typeof value !== 'string') return false;
   return /\bwrfc\b|work[-\s]*review[-\s]*fix/i.test(value);

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '0.19.86';
+let _version = '0.19.87';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   _version = pkg.version ?? _version;