@aion0/forge 0.10.29 → 0.10.31

package/RELEASE_NOTES.md

@@ -1,20 +1,11 @@
-# Forge v0.10.29
+# Forge v0.10.31
 
 Released: 2026-06-02
 
-## Changes since v0.10.28
+## Changes since v0.10.30
 
 ### Other
-- feat(ssh): template-expand spec.timeout_sec — lets tools surface as arg
-- fix(help-content): drop import.meta.url branch — Turbopack rejects webpackIgnore
-- fix(help-content): suppress Turbopack 'cannot resolve ./help-docs' warning
-- fix(http): empty-string body resolves to no-body
-- fix(http-auth): bearer-token-exchange honours connector http.verify_tls
-- feat(connector-test): add body_match regex for response-body validation
-- fix(http): reorder method-template expand to AFTER argsWithDefaults
-- fix(http): expand templates in spec.method before uppercase
-- docs(connector-authoring): add bearer-token-exchange section to 21-build-connector
-- feat(http-auth): bearer-token-exchange supports body-mode + bare format
+- fix(watch): allow builtin tool names (no connector prefix) in start_watch
 
 
-**Full Changelog**: https://github.com/aiwatching/forge/compare/v0.10.28...v0.10.29
+**Full Changelog**: https://github.com/aiwatching/forge/compare/v0.10.30...v0.10.31

package/lib/chat/agent-loop.ts

@@ -41,7 +41,11 @@ import type {
   ToolUseBlock,
 } from './types';
 
-const MAX_ITERATIONS = 6;
+// Set to 24: enough headroom for typical multi-step tool chains
+// (search → paginate → drill → summarise can be 8-15; complex audits
+// 15-20) without inviting runaway loops. Hitting it surfaces the
+// sentinel message below so the user knows why the turn stopped.
+const MAX_ITERATIONS = 24;
 const MAX_TOKENS = 16000;
 // Working-window budgets for the LLM history. Capped by message count
 // AND by token estimate (whichever hits first), see design §8. Older
@@ -596,6 +600,23 @@ export async function runTurn(args: RunTurnArgs): Promise<{ ok: boolean; error?:
       cb({ type: 'message_saved', message_id: toolResultMsg.id, data: toolResultMsg });
     }
 
+    // If we fell out of the while because we hit MAX_ITERATIONS (vs the
+    // model emitting a non-tool_use stopReason), the turn ends with no
+    // closing assistant text and the user sees a "no response" hang.
+    // Surface a sentinel assistant message so the cause is visible.
+    if (iter >= MAX_ITERATIONS && lastStop === 'tool_use') {
+      const sentinel = appendMessage({
+        session_id: args.sessionId,
+        role: 'assistant',
+        blocks: [{
+          type: 'text',
+          text: `⚠️ Hit the tool-call iteration limit (${MAX_ITERATIONS}) without finishing. Likely causes: a tool kept returning truncated/incomplete results so I kept retrying, or the task was too broad. Try narrowing the query or splitting it into smaller asks.`,
+        }],
+      });
+      cb({ type: 'message_saved', message_id: sentinel.id, data: sentinel });
+      cb({ type: 'error', data: { error: `iteration limit (${MAX_ITERATIONS}) exceeded` } });
+    }
+
     cb({ type: 'turn_done', data: { iterations: iter, stop_reason: lastStop } });
     return { ok: true };
   } catch (err) {

package/lib/watch/start-watch-tool.ts

@@ -35,12 +35,12 @@ export function buildStartWatchTool(sessionId: string | null): StartWatchTool {
   const def: BuiltinToolDef = {
     name: 'start_watch',
     description:
-      'Register a BACKGROUND WATCH that polls a tool until done, then posts the result back here — for long-running jobs you just kicked off (a Jenkins build, a test run, a device upgrade). Use this INSTEAD of polling in conversation: call the trigger tool, then call start_watch and STOP — Forge polls in the background and a completion message arrives in this chat. ' +
-      'Pick `poll` = the read tool that reports status (e.g. "jenkins.get_build") and `poll_args` to call it with (e.g. the build number you predicted via get_next_build_number). Give a done condition: `done_match` {path, equals} on the poll result (e.g. path "result" equals "SUCCESS"), or `done_path` (a result path that becomes truthy). You usually already saw the poll tool\'s output once, so you know the right field. Optional `fail_path` (truthy = failed). Tune `interval_sec`/`timeout_sec` to the job (build ≈ 60s / 1800s).',
+      'Register a BACKGROUND WATCH that polls a tool until done, then posts the result back here — for long-running jobs you just kicked off (a Forge pipeline, a Jenkins build, a test run, a device upgrade). Use this INSTEAD of polling in conversation: call the trigger tool, then call start_watch and STOP — Forge polls in the background and a completion message arrives in this chat. ' +
+      'Pick `poll` = the read tool that reports status. Two forms accepted: (a) connector tool "<connector>.<tool>" e.g. "jenkins.get_build", "gitlab.get_pipeline"; (b) BARE builtin name e.g. "get_pipeline_status" (Forge pipelines — pair with poll_args={pipeline_id} and done_match={path:"status",equals:"done"}). Set `poll_args` to call it with (e.g. the build number you predicted via get_next_build_number). Give a done condition: `done_match` {path, equals} on the poll result (e.g. path "result" equals "SUCCESS"), or `done_path` (a result path that becomes truthy). You usually already saw the poll tool\'s output once, so you know the right field. Optional `fail_path` (truthy = failed). Tune `interval_sec`/`timeout_sec` to the job (pipeline ≈ 30s / 1800s, build ≈ 60s / 1800s).',
     input_schema: {
       type: 'object',
       properties: {
-        poll: { type: 'string', description: 'Tool to poll, "<connector>.<tool>" e.g. "jenkins.get_build". Must be a read/status tool.' },
+        poll: { type: 'string', description: 'Tool to poll. Either "<connector>.<tool>" e.g. "jenkins.get_build", OR a bare builtin name e.g. "get_pipeline_status" (for Forge pipelines). Must be a read/status tool.' },
         poll_args: { type: 'object', description: 'Args to call the poll tool with each tick, e.g. {"job_path":"job/foo","build_number":18}. Concrete values, not templates.' },
         done_match: {
           type: 'object',
@@ -68,9 +68,16 @@ export function buildStartWatchTool(sessionId: string | null): StartWatchTool {
     const a = (input ?? {}) as Record<string, any>;
     const poll = String(a.poll || '').trim();
     const dot = poll.indexOf('.');
-    if (dot < 1) return JSON.stringify({ ok: false, error: 'poll must be "<connector>.<tool>", e.g. jenkins.get_build' });
-    const connectorId = poll.slice(0, dot);
-    const pollTool = poll.slice(dot + 1);
+    // Two forms accepted:
+    //   "connector.tool"  → connector_id="connector", poll_tool="tool"
+    //   "tool"            → builtin (e.g. get_pipeline_status), connector_id=""
+    // The watch-runner detects empty connector_id and dispatches as a
+    // bare builtin name instead of "connector.tool".
+    const connectorId = dot >= 1 ? poll.slice(0, dot) : '';
+    const pollTool = dot >= 1 ? poll.slice(dot + 1) : poll;
+    if (!pollTool) {
+      return JSON.stringify({ ok: false, error: 'poll is required, e.g. "jenkins.get_build" or builtin "get_pipeline_status"' });
+    }
 
     const doneMatch = a.done_match && typeof a.done_match === 'object' && a.done_match.path
       ? { path: String(a.done_match.path), ...(a.done_match.equals != null ? { equals: String(a.done_match.equals) } : {}), ...(a.done_match.contains != null ? { contains: String(a.done_match.contains) } : {}) }
@@ -83,8 +90,8 @@ export function buildStartWatchTool(sessionId: string | null): StartWatchTool {
       return JSON.stringify({ ok: false, error: `active watch limit reached (${MAX_ACTIVE_WATCHES})` });
     }
 
-    const hint = ['build_number', 'host', 'ip', 'lab', 'id', 'name'].map((k) => a.poll_args?.[k]).find((v) => v != null && v !== '');
-    const label = `${connectorId}.${pollTool}${hint != null ? ` ${hint}` : ''}`;
+    const hint = ['build_number', 'host', 'ip', 'lab', 'id', 'name', 'pipeline_id'].map((k) => a.poll_args?.[k]).find((v) => v != null && v !== '');
+    const label = `${connectorId ? `${connectorId}.${pollTool}` : pollTool}${hint != null ? ` ${hint}` : ''}`;
 
     const w = createWatch({
       session_id: sessionId,

package/lib/watch/watch-runner.ts

@@ -93,7 +93,10 @@ export function startWatchRunner(hooks: WatchRunnerHooks = {}): void {
       // preamble — the body comes back as raw JSON so done_path/done_match can
       // see it. (Without this, every http-protocol watch would silently never
       // hit its done condition, e.g. jenkins.get_build never resolving.)
-      res = await dispatchTool({ id: `watch-${w.id}-${w.polls}`, name: `${w.connector_id}.${w.poll_tool}`, input: w.poll_args }, { noTruncation: true } as any);
+      // Empty connector_id → builtin (e.g. get_pipeline_status); dispatch
+      // bare name so dispatchTool routes to the global BUILTINS table.
+      const dispatchName = w.connector_id ? `${w.connector_id}.${w.poll_tool}` : w.poll_tool;
+      res = await dispatchTool({ id: `watch-${w.id}-${w.polls}`, name: dispatchName, input: w.poll_args }, { noTruncation: true } as any);
     } catch (e) {
       res = { content: String(e), is_error: true };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aion0/forge",
-  "version": "0.10.29",
+  "version": "0.10.31",
   "description": "Unified AI workflow platform — multi-model task orchestration, persistent sessions, web terminal, remote access",
   "type": "module",
   "scripts": {