@pellux/goodvibes-agent 1.0.33 → 1.0.34

package/src/tools/agent-harness-ui-surface-metadata.ts

@@ -11,6 +11,7 @@ export interface AgentHarnessUiSurfaceArgs {
   readonly target?: unknown;
   readonly key?: unknown;
   readonly prefix?: unknown;
+  readonly includeParameters?: unknown;
   readonly limit?: unknown;
   readonly pane?: unknown;
 }
@@ -727,22 +728,28 @@ function resolveHarnessUiSurface(args: AgentHarnessUiSurfaceArgs): UiSurfaceReso
   return null;
 }
 
-function describeSurface(context: CommandContext, surface: UiSurfaceDefinition, lookup?: UiSurfaceLookup): Record<string, unknown> {
+function describeSurface(
+  context: CommandContext,
+  surface: UiSurfaceDefinition,
+  options: { readonly includeParameters?: boolean; readonly lookup?: UiSurfaceLookup } = {},
+): Record<string, unknown> {
   return {
     id: surface.id,
     label: surface.label,
     kind: surface.kind,
     summary: surface.summary,
     command: surface.command,
-    ...(lookup ? { lookup } : {}),
+    ...(options.lookup ? { lookup: options.lookup } : {}),
     preferredModelRoute: surface.preferredModelRoute,
-    parameters: surface.parameters ?? [],
     available: surface.available(context),
-    policy: {
-      effect: 'visible-ui-navigation',
-      confirmation: 'agent_harness mode:"open_ui_surface" requires confirm:true and explicitUserRequest.',
-      boundary: 'UI surface routing opens the same visible Agent shell surface the user can open. Use first-class model tools, settings modes, workspace actions, or confirmed slash-command mirrors for actual operations.',
-    },
+    ...(options.includeParameters ? {
+      parameters: surface.parameters ?? [],
+      policy: {
+        effect: 'visible-ui-navigation',
+        confirmation: 'agent_harness mode:"open_ui_surface" requires confirm:true and explicitUserRequest.',
+        boundary: 'UI surface routing opens the same visible Agent shell surface the user can open. Use first-class model tools, settings modes, workspace actions, or confirmed slash-command mirrors for actual operations.',
+      },
+    } : {}),
   };
 }
 
@@ -753,15 +760,16 @@ export function totalHarnessUiSurfaces(): number {
 export function listHarnessUiSurfaces(context: CommandContext, args: AgentHarnessUiSurfaceArgs): readonly Record<string, unknown>[] {
   const query = readString(args.query);
   const limit = readLimit(args.limit, 200);
+  const includeParameters = args.includeParameters === true;
   return UI_SURFACES
-    .map((surface) => describeSurface(context, surface))
+    .map((surface) => describeSurface(context, surface, { includeParameters }))
     .filter((surface) => surfaceMatches(surface, query))
     .slice(0, limit);
 }
 
 export function describeHarnessUiSurface(context: CommandContext, args: AgentHarnessUiSurfaceArgs): Record<string, unknown> | null {
   const resolved = resolveHarnessUiSurface(args);
-  if (resolved?.status === 'found') return describeSurface(context, resolved.surface, resolved.lookup);
+  if (resolved?.status === 'found') return describeSurface(context, resolved.surface, { includeParameters: true, lookup: resolved.lookup });
   if (resolved?.status === 'ambiguous') {
     return { status: 'ambiguous', input: resolved.input, candidates: resolved.candidates };
   }
@@ -783,6 +791,6 @@ export async function openHarnessUiSurface(context: CommandContext, args: AgentH
   const routed = await resolved.surface.open(context, args);
   return {
     ...routed,
-    descriptor: describeSurface(context, resolved.surface, resolved.lookup),
+    descriptor: describeSurface(context, resolved.surface, { includeParameters: true, lookup: resolved.lookup }),
   };
 }

package/src/tools/agent-harness-workspace-actions.ts

@@ -97,6 +97,11 @@ export function describeWorkspaceEditor(editor: AgentWorkspaceLocalEditor): Reco
   };
 }
 
+function previewText(value: string, maxLength = 120): string {
+  const normalized = value.replace(/\s+/g, ' ').trim();
+  return normalized.length <= maxLength ? normalized : `${normalized.slice(0, maxLength - 1).trimEnd()}...`;
+}
+
 function selectedRoutineFromArgs(
   snapshot: AgentWorkspaceRuntimeSnapshot,
   args: AgentHarnessWorkspaceActionArgs,
@@ -163,6 +168,27 @@ export function describeWorkspaceAction(
   };
 }
 
+function describeWorkspaceActionSummary(
+  category: AgentWorkspaceCategory,
+  action: AgentWorkspaceAction,
+): Record<string, unknown> {
+  return {
+    id: action.id,
+    categoryId: category.id,
+    category: category.label,
+    group: category.group,
+    label: action.label,
+    summary: previewText(action.detail),
+    kind: action.kind,
+    safety: action.safety,
+    ...(action.command ? { command: action.command } : {}),
+    ...(action.targetCategoryId ? { targetCategoryId: action.targetCategoryId } : {}),
+    ...(action.editorKind ? { editorKind: action.editorKind } : {}),
+    ...(action.localKind ? { localKind: action.localKind } : {}),
+    ...(action.localOperation ? { localOperation: action.localOperation } : {}),
+  };
+}
+
 export function listWorkspaceActions(
   context: CommandContext,
   args: AgentHarnessWorkspaceActionArgs,
@@ -178,7 +204,9 @@ export function listWorkspaceActions(
   return source
     .filter((entry) => !categoryId || entry.category.id === categoryId)
     .slice(0, limit)
-    .map((entry) => describeWorkspaceAction(entry.category, entry.action, { includeEditor, editorContext }));
+    .map((entry) => includeEditor
+      ? describeWorkspaceAction(entry.category, entry.action, { includeEditor, editorContext })
+      : describeWorkspaceActionSummary(entry.category, entry.action));
 }
 
 function workspaceActionLookupFromArgs(args: AgentHarnessWorkspaceActionArgs): { readonly source: WorkspaceActionLookup['source']; readonly input: string } | null {

package/src/tools/agent-knowledge-ingest-tool.ts

@@ -261,11 +261,7 @@ export function createAgentKnowledgeIngestTool(
   return {
     definition: {
       name: 'agent_knowledge_ingest',
-      description: [
-        'Ingest one confirmed source into isolated Agent Knowledge.',
-        'Use only for explicit ingest/import requests.',
-        'Supports URL, file, bookmark, history, and connector sources.',
-      ].join(' '),
+      description: 'Ingest one confirmed source into isolated Agent Knowledge.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-knowledge-tool.ts

@@ -123,11 +123,7 @@ export function createAgentKnowledgeTool(
   return {
     definition: {
       name: 'agent_knowledge',
-      description: [
-        'Read isolated Agent Knowledge.',
-        'Use for status, ask/search, source/node/issue lists, item lookup, map summary, and connector inspection.',
-        'Read-only.',
-      ].join(' '),
+      description: 'Read isolated Agent Knowledge.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-local-registry-tool.ts

@@ -735,10 +735,7 @@ export function createAgentLocalRegistryTool(shellPaths: ShellPathService, memor
   return {
     definition: {
       name: 'agent_local_registry',
-      description: [
-        'Inspect or update Agent-local records: memory, notes, personas, skills, bundles, and routines.',
-        'Deletion requires confirm:true plus explicitUserRequest.',
-      ].join(' '),
+      description: 'Inspect or update Agent-local records.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-media-generate-tool.ts

@@ -52,11 +52,7 @@ export function createAgentMediaGenerateTool(
   return {
     definition: {
       name: 'agent_media_generate',
-      description: [
-        'Generate one confirmed image or video artifact.',
-        'Use only for explicit media generation requests.',
-        'Returns artifact ids, not inline base64.',
-      ].join(' '),
+      description: 'Generate one confirmed image or video artifact.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-notify-tool.ts

@@ -78,11 +78,7 @@ export function createAgentNotifyTool(
   return {
     definition: {
       name: 'agent_notify',
-      description: [
-        'Send one confirmed plain-text notification.',
-        'Use only for explicit notify/message/alert requests.',
-        'Uses existing targets only.',
-      ].join(' '),
+      description: 'Send one confirmed plain-text notification.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-operator-action-tool.ts

@@ -35,11 +35,7 @@ export function createAgentOperatorActionTool(
   return {
     definition: {
       name: 'agent_operator_action',
-      description: [
-        'Run one confirmed allowlisted operator action.',
-        'Use only for explicit approval, automation, or schedule requests.',
-        'No automation definition edits or host lifecycle.',
-      ].join(' '),
+      description: 'Run one confirmed allowlisted operator action.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-operator-briefing-tool.ts

@@ -175,11 +175,7 @@ export function createAgentOperatorBriefingTool(
   return {
     definition: {
       name: 'agent_operator_briefing',
-      description: [
-        'Read connected Agent operator state for a concise briefing.',
-        'Use for pending attention, approvals, automation, schedules, or status.',
-        'Read-only.',
-      ].join(' '),
+      description: 'Read connected Agent operator state for a concise briefing.',
       parameters: {
         type: 'object',
         properties: {},

package/src/tools/agent-read-policy.ts

@@ -126,10 +126,7 @@ export function isBlockedReadPath(path: string): boolean {
 }
 
 function narrowReadToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Read ordinary project files for GoodVibes Agent with bounded, non-secret, main-conversation policy.',
-    'Hidden paths, secret-looking files, broad batches, unoptimized image extraction, and oversized image reads are disabled.',
-  ].join(' ');
+  tool.definition.description = 'Read ordinary non-secret project files for GoodVibes Agent.';
   tool.definition.sideEffects = ['read_fs'];
   tool.definition.concurrency = 'serial';
 

package/src/tools/agent-reminder-schedule-tool.ts

@@ -120,11 +120,7 @@ export function createAgentReminderScheduleTool(
   return {
     definition: {
       name: 'agent_reminder_schedule',
-      description: [
-        'Schedule one confirmed GoodVibes Agent reminder.',
-        'Use only for an explicit reminder request.',
-        'No host lifecycle, background job, or Knowledge write.',
-      ].join(' '),
+      description: 'Schedule one confirmed GoodVibes Agent reminder.',
       parameters: {
         type: 'object',
         properties: {

package/src/tools/agent-tool-policy-guard.ts

@@ -309,11 +309,7 @@ export function normalizeAgentToolInvocationForAgentPolicy(args: AgentToolArgs):
 }
 
 export function wrapBlockedMainConversationToolForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    `Blocked in GoodVibes Agent main conversation: ${tool.definition.name}.`,
-    'Use explicit GoodVibes TUI build delegation for build/fix/review/code execution work.',
-    'Use Agent-owned local registries and isolated Agent Knowledge routes for Agent memory and knowledge work.',
-  ].join(' ');
+  tool.definition.description = `Blocked in GoodVibes Agent: ${tool.definition.name}.`;
   tool.definition.sideEffects = [];
   tool.execute = async () => ({ success: false, error: LOCAL_CODING_TOOL_DENIAL });
 }
@@ -349,11 +345,7 @@ export function wrapStateToolForAgentPolicy(tool: Tool): void {
 }
 
 export function wrapBlockedSettingsToolForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Blocked in GoodVibes Agent main conversation: configuration mutation.',
-    'Use explicit Agent CLI/slash settings commands for intentional config changes.',
-    'Connected-host lifecycle and service exposure remain externally managed outside GoodVibes Agent.',
-  ].join(' ');
+  tool.definition.description = 'Blocked in GoodVibes Agent: configuration mutation.';
   tool.definition.sideEffects = [];
   tool.definition.parameters = {
     type: 'object',
@@ -603,11 +595,7 @@ function isPresent(value: unknown): boolean {
 }
 
 function narrowAgentToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Read-only local Agent inspection for GoodVibes Agent.',
-    'This product does not create separate Agent jobs or run local delegated review chains.',
-    'For build/fix/review work, delegate to GoodVibes TUI through the explicit build-delegation path instead.',
-  ].join(' ');
+  tool.definition.description = 'Read-only local Agent inspection for GoodVibes Agent.';
   tool.definition.sideEffects = [];
 
   const properties = tool.definition.parameters.properties;
@@ -619,11 +607,7 @@ function narrowAgentToolDefinitionForAgentPolicy(tool: Tool): void {
 }
 
 function narrowExecToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Execute foreground shell commands serially for GoodVibes Agent main-conversation work.',
-    'Background processes, parallel batches, background process controls, and exec file_ops are disabled by Agent policy.',
-    'Delegate long-running build/fix/review execution to GoodVibes TUI instead.',
-  ].join(' ');
+  tool.definition.description = 'Execute foreground shell commands serially for GoodVibes Agent.';
 
   const properties = tool.definition.parameters.properties;
   if (!isRecord(properties)) return;
@@ -648,11 +632,7 @@ function narrowExecToolDefinitionForAgentPolicy(tool: Tool): void {
 }
 
 function narrowFetchToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Fetch public URLs for GoodVibes Agent with serial, read-only HTTP requests.',
-    'Only GET, HEAD, and OPTIONS are available in the main conversation.',
-    'Credentialed requests, request bodies, trust overrides, raw unsanitized responses, and parallel batches are disabled by Agent policy.',
-  ].join(' ');
+  tool.definition.description = 'Fetch public URLs with serial, read-only HTTP requests.';
 
   const properties = tool.definition.parameters.properties;
   if (!isRecord(properties)) return;
@@ -689,11 +669,7 @@ function narrowFetchToolDefinitionForAgentPolicy(tool: Tool): void {
 }
 
 function narrowStateToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Inspect runtime-owned state for GoodVibes Agent.',
-    'State mutation, runtime-owned memory writes, hook changes, output-mode changes, and analytics writes are disabled in the main conversation.',
-    'Use Agent-owned commands for intentional memory, skill, persona, and routine changes.',
-  ].join(' ');
+  tool.definition.description = 'Inspect runtime-owned state for GoodVibes Agent.';
 
   const properties = tool.definition.parameters.properties;
   if (!isRecord(properties)) return;
@@ -723,10 +699,7 @@ function narrowStateToolDefinitionForAgentPolicy(tool: Tool): void {
 }
 
 function narrowInspectToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Inspect and analyze project structure for GoodVibes Agent.',
-    'Scaffold mode is dry-run-only in the main conversation; code creation must be delegated to GoodVibes TUI.',
-  ].join(' ');
+  tool.definition.description = 'Inspect and analyze project structure for GoodVibes Agent.';
 
   const properties = tool.definition.parameters.properties;
   if (!isRecord(properties)) return;

package/src/tools/agent-web-search-policy.ts

@@ -73,10 +73,7 @@ export function normalizeWebSearchToolInvocationForAgentPolicy(args: WebSearchTo
 }
 
 function narrowWebSearchToolDefinitionForAgentPolicy(tool: Tool): void {
-  tool.definition.description = [
-    'Run bounded, read-only web research for GoodVibes Agent.',
-    'Full-page/raw/summary extraction, safe-search off, and high-fanout searches are disabled in the main conversation.',
-  ].join(' ');
+  tool.definition.description = 'Run bounded, read-only web research for GoodVibes Agent.';
   tool.definition.sideEffects = ['network'];
 
   const properties = tool.definition.parameters.properties;

package/src/tools/agent-work-plan-tool.ts

@@ -165,11 +165,7 @@ export function createAgentWorkPlanTool(store: WorkPlanStore): Tool {
   return {
     definition: {
       name: 'agent_work_plan',
-      description: [
-        'Inspect or update the visible Agent-local work plan.',
-        'Use for task/status tracking.',
-        'Destructive actions require confirm:true plus explicitUserRequest.',
-      ].join(' '),
+      description: 'Inspect or update the visible Agent-local work plan.',
       parameters: {
         type: 'object',
         properties: {

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 = '1.0.33';
+let _version = '1.0.34';
 let _sdkVersion = '0.33.36';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8')) as {