@cybernetyx1/atlasflow-runtime 0.1.0 → 0.1.2

package/README.md

@@ -5,7 +5,7 @@ The portable TypeScript runtime and SDK used inside AtlasFlow agent code — def
 ## Install
 
 ```sh
-pnpm add @cybernetyx1/atlasflow-runtime
+npm install @cybernetyx1/atlasflow-runtime
 ```
 
 Part of the AtlasFlow monorepo. Proprietary.

package/dist/{channel-Dv3Hv1ee.d.ts → channel-CRAw_ZfB.d.ts}

@@ -457,6 +457,13 @@ declare const WorkflowStepSchema: v.VariantSchema<"kind", [v.ObjectSchema<{
     readonly kind: v.LiteralSchema<"gate", undefined>;
     readonly title: v.StringSchema<undefined>;
     readonly description: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
+    /**
+     * On rejection, loop back to an earlier step instead of failing the run —
+     * the Brain's "Review → Build" rework loop. Omit for a terminal gate.
+     */
+    readonly onReject: v.OptionalSchema<v.ObjectSchema<{
+        readonly goto: v.SchemaWithPipe<readonly [v.StringSchema<undefined>, v.MinLengthAction<string, 1, undefined>]>;
+    }, undefined>, undefined>;
 }, undefined>], undefined>;
 declare const WorkflowDefSchema: v.ObjectSchema<{
     readonly name: v.SchemaWithPipe<readonly [v.StringSchema<undefined>, v.MinLengthAction<string, 1, undefined>]>;
@@ -482,6 +489,13 @@ declare const WorkflowDefSchema: v.ObjectSchema<{
         readonly kind: v.LiteralSchema<"gate", undefined>;
         readonly title: v.StringSchema<undefined>;
         readonly description: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
+        /**
+         * On rejection, loop back to an earlier step instead of failing the run —
+         * the Brain's "Review → Build" rework loop. Omit for a terminal gate.
+         */
+        readonly onReject: v.OptionalSchema<v.ObjectSchema<{
+            readonly goto: v.SchemaWithPipe<readonly [v.StringSchema<undefined>, v.MinLengthAction<string, 1, undefined>]>;
+        }, undefined>, undefined>;
     }, undefined>], undefined>, undefined>, v.MinLengthAction<({
         id: string;
         kind: "prompt";
@@ -499,6 +513,9 @@ declare const WorkflowDefSchema: v.ObjectSchema<{
         kind: "gate";
         title: string;
         description?: string | undefined;
+        onReject?: {
+            goto: string;
+        } | undefined;
     })[], 1, undefined>]>;
     /** Role slot → default persona name; rebindable at deploy. */
     readonly roleSlots: v.OptionalSchema<v.RecordSchema<v.StringSchema<undefined>, v.StringSchema<undefined>, undefined>, undefined>;
@@ -517,6 +534,8 @@ interface WorkflowState {
         note?: string;
         at: number;
     }>;
+    /** Times each onReject gate has looped back — bounds the rework cycle. */
+    loops?: Record<string, number>;
 }
 interface WorkflowContext {
     /** Personas executing steps: registered agents win, then profiles. */

package/dist/{chunk-RF6W3TKJ.js → chunk-EU2FU6RF.js}

@@ -13,7 +13,7 @@ import {
 import {
   getRegisteredApiKey,
   resolveRegisteredModel
-} from "./chunk-HO6QHSUS.js";
+} from "./chunk-KY5APFKW.js";
 
 // src/agent.ts
 function createAgent(initialize) {
@@ -1336,6 +1336,10 @@ function resolveAnyTool(tool) {
   }
   return resolveTool(tool);
 }
+function sanitizeToolNamePart(value) {
+  const sanitized = value.replace(/[^A-Za-z0-9_-]/g, "_").replace(/^_+|_+$/g, "");
+  return sanitized || "unnamed";
+}
 
 // src/session.ts
 import { toJsonSchema as toJsonSchema2 } from "@valibot/to-json-schema";
@@ -1987,7 +1991,10 @@ async function reconcileRecoveredRunJournal(persistence, run, error) {
 function skillTool(skill, model, tools, sub) {
   const allowed = skill.allowedTools ? tools.filter((t) => skill.allowedTools.includes(t.name)) : tools;
   return {
-    name: `skill_${skill.name}`,
+    // Sanitize: skill display names ("Write & refactor services") would
+    // otherwise produce tool names that violate the model tool-name rule
+    // ^[a-zA-Z0-9_-]+$ (OpenAI rejects with 400). Caller dedupes collisions.
+    name: `skill_${sanitizeToolNamePart(skill.name)}`,
     description: `Run the "${skill.name}" skill. ${skill.description}`,
     parameters: {
       type: "object",
@@ -2318,9 +2325,17 @@ async function invokeAgent(options) {
       tools.push(wrapRuntimeTool(loadContextTool(contextPackMap)));
     }
     if (!lazySkillLoading) {
+      const usedToolNames = new Set(tools.map((t) => t.name));
       for (const skill of skills.values()) {
         if (isWorkspaceSkill(skill)) continue;
-        tools.push(wrapRuntimeTool(skillTool(skill, model, subcallTools, sub)));
+        const tool = skillTool(skill, model, subcallTools, sub);
+        if (usedToolNames.has(tool.name)) {
+          let n = 2;
+          while (usedToolNames.has(`${tool.name}_${n}`)) n++;
+          tool.name = `${tool.name}_${n}`;
+        }
+        usedToolNames.add(tool.name);
+        tools.push(wrapRuntimeTool(tool));
       }
     }
     const personas = [...options.sharedPersonas ?? options.sharedSubagents ?? [], ...config.personas ?? config.subagents ?? []];
@@ -2431,7 +2446,12 @@ var GateStepSchema = v3.object({
   id: v3.pipe(v3.string(), v3.minLength(1)),
   kind: v3.literal("gate"),
   title: v3.string(),
-  description: v3.optional(v3.string())
+  description: v3.optional(v3.string()),
+  /**
+   * On rejection, loop back to an earlier step instead of failing the run —
+   * the Brain's "Review → Build" rework loop. Omit for a terminal gate.
+   */
+  onReject: v3.optional(v3.object({ goto: v3.pipe(v3.string(), v3.minLength(1)) }))
 });
 var WorkflowStepSchema = v3.variant("kind", [PromptStepSchema, SkillStepSchema, GateStepSchema]);
 var WorkflowDefSchema = v3.object({
@@ -2481,20 +2501,27 @@ async function advanceWorkflow(def, ctx, runId) {
   if (!record2) throw new Error(`No workflow run "${runId}"`);
   const state = record2.state ?? { cursor: 0, results: {}, gates: {} };
   let lastText = "";
-  for (let i = state.cursor; i < def.steps.length; i++) {
-    const step = def.steps[i];
+  while (state.cursor < def.steps.length) {
+    const step = def.steps[state.cursor];
     if (step.kind === "gate") {
       const decision = state.gates[step.id];
       if (!decision) {
-        state.cursor = i;
         await persistence.runs.update(runId, { status: "waiting_approval", state });
         return { runId, status: "waiting_approval", gate: { id: step.id, title: step.title } };
       }
       if (!decision.approved) {
+        if (step.onReject) {
+          const result = loopBack(def, state, step, runId);
+          if (result) {
+            await persistence.runs.update(runId, result.update);
+            if (result.done) return result.done;
+            continue;
+          }
+        }
         await persistence.runs.update(runId, { status: "failed", error: { code: "gate_rejected", message: `Gate "${step.title}" rejected${decision.note ? `: ${decision.note}` : ""}` }, endedAt: Date.now(), state });
         return { runId, status: "failed" };
       }
-      state.cursor = i + 1;
+      state.cursor += 1;
       await persistence.runs.update(runId, { status: "running", state });
       continue;
     }
@@ -2506,12 +2533,36 @@ async function advanceWorkflow(def, ctx, runId) {
       return { runId, status: "failed" };
     }
     state.results[step.id] = { text: lastText };
-    state.cursor = i + 1;
+    state.cursor += 1;
     await persistence.runs.update(runId, { state });
   }
   await persistence.runs.update(runId, { status: "success", result: lastText, endedAt: Date.now(), state });
   return { runId, status: "success", text: lastText };
 }
+var MAX_LOOPBACKS = 10;
+function loopBack(def, state, gate, runId) {
+  const target = def.steps.findIndex((s) => s.id === gate.onReject.goto);
+  if (target === -1) {
+    return { update: { status: "failed", error: { code: "bad_workflow", message: `Gate "${gate.id}" onReject.goto "${gate.onReject.goto}" is not a step` }, endedAt: Date.now(), state }, done: { runId, status: "failed" } };
+  }
+  if (target >= state.cursor) {
+    return { update: { status: "failed", error: { code: "bad_workflow", message: `Gate "${gate.id}" onReject.goto "${gate.onReject.goto}" must target an earlier step (no forward or self jumps)` }, endedAt: Date.now(), state }, done: { runId, status: "failed" } };
+  }
+  const loops = state.loops ??= {};
+  const count = (loops[gate.id] ?? 0) + 1;
+  loops[gate.id] = count;
+  if (count > MAX_LOOPBACKS) {
+    return { update: { status: "failed", error: { code: "loop_limit_exceeded", message: `Gate "${gate.id}" looped back to "${gate.onReject.goto}" more than ${MAX_LOOPBACKS} times` }, endedAt: Date.now(), state }, done: { runId, status: "failed" } };
+  }
+  for (let i = target; i <= state.cursor; i++) {
+    const s = def.steps[i];
+    if (!s) continue;
+    delete state.results[s.id];
+    if (s.kind === "gate") delete state.gates[s.id];
+  }
+  state.cursor = target;
+  return { update: { status: "running", state } };
+}
 async function executeStep(step, def, ctx, state, parentRunId) {
   const personaName = resolvePersona(step, def, ctx);
   const registered = personaName ? ctx.agents?.[personaName] : void 0;
@@ -2738,6 +2789,7 @@ export {
   isRawTool,
   resolveTool,
   resolveAnyTool,
+  sanitizeToolNamePart,
   Session,
   Harness,
   defaultEngine,

package/dist/{chunk-HO6QHSUS.js → chunk-KY5APFKW.js}

@@ -52,7 +52,11 @@ function resolveRegisteredModel(providerId, modelId) {
     baseUrl,
     headers: base.headers || registration.headers ? { ...base.headers ?? {}, ...registration.headers ?? {} } : void 0,
     contextWindow: modelOverride?.contextWindow ?? registration.contextWindow ?? base.contextWindow,
-    maxTokens: modelOverride?.maxTokens ?? registration.maxTokens ?? base.maxTokens
+    maxTokens: modelOverride?.maxTokens ?? registration.maxTokens ?? base.maxTokens,
+    // A custom relay misses the pi-ai catalog → `base` is zeroMetadataModel (reasoning:false). Let the
+    // override restore the model's true reasoning shape so pi-ai inlines reasoning items correctly.
+    reasoning: modelOverride?.reasoning ?? base.reasoning,
+    ...modelOverride?.thinkingLevelMap ? { thinkingLevelMap: modelOverride.thinkingLevelMap } : base.thinkingLevelMap ? { thinkingLevelMap: base.thinkingLevelMap } : {}
   };
 }
 function zeroMetadataModel(providerId, modelId, api, baseUrl) {

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { c as AnyTool, d as AgentRuntimeConfig, C as CreatedAgent, W as WorkflowDef, E as EngineTool, R as Rule, e as AgentHooks, T as ToolDefinition, f as RawTool, S as Skill, g as ToolChoice, A as AgentEngine, h as CompactionConfig, i as EngineRunInput, j as EngineHooks, k as EngineRunResult } from './channel-Dv3Hv1ee.js';
-export { l as AgentCreateContext, m as AgentDefinition, a as AgentProfile, n as ChannelAccepted, o as ChannelContext, b as ChannelDefinition, p as ChannelDispatch, q as ChannelMode, r as ChannelResult, s as ContextPack, t as ContextPackItem, u as DurabilityConfig, v as DurableExecutionMeta, D as DurableExecutionRunner, H as HookDecision, w as InvokeOptions, I as InvokeResult, P as ProviderModelSettings, x as ProviderSettings, y as RuleEnforcePoint, z as SlackRequestSignatureOptions, B as ToolApprovalPolicy, F as WorkflowContext, G as WorkflowDefSchema, J as WorkflowRunResult, K as WorkflowState, L as WorkflowStep, M as advanceWorkflow, N as applyProfile, O as approveGate, Q as createAgent, U as defaultEngine, V as defineAgent, X as defineAgentProfile, Y as defineChannel, Z as defineTool, _ as dispatchWorkflow, $ as genId, a0 as getDefaultPersistence, a1 as invokeAgent, a2 as isChannelDefinition, a3 as isCreatedAgent, a4 as isRawTool, a5 as parseWorkflowDef, a6 as rawTool, a7 as recoverRuns, a8 as recoverWorkflows, a9 as resolveAnyTool, aa as resolveTool, ab as startWorkflow, ac as verifyGithubWebhookSignature, ad as verifySlackRequestSignature, ae as waitingGate } from './channel-Dv3Hv1ee.js';
+import { c as AnyTool, d as AgentRuntimeConfig, C as CreatedAgent, W as WorkflowDef, E as EngineTool, R as Rule, e as AgentHooks, T as ToolDefinition, f as RawTool, S as Skill, g as ToolChoice, A as AgentEngine, h as CompactionConfig, i as EngineRunInput, j as EngineHooks, k as EngineRunResult } from './channel-CRAw_ZfB.js';
+export { l as AgentCreateContext, m as AgentDefinition, a as AgentProfile, n as ChannelAccepted, o as ChannelContext, b as ChannelDefinition, p as ChannelDispatch, q as ChannelMode, r as ChannelResult, s as ContextPack, t as ContextPackItem, u as DurabilityConfig, v as DurableExecutionMeta, D as DurableExecutionRunner, H as HookDecision, w as InvokeOptions, I as InvokeResult, P as ProviderModelSettings, x as ProviderSettings, y as RuleEnforcePoint, z as SlackRequestSignatureOptions, B as ToolApprovalPolicy, F as WorkflowContext, G as WorkflowDefSchema, J as WorkflowRunResult, K as WorkflowState, L as WorkflowStep, M as advanceWorkflow, N as applyProfile, O as approveGate, Q as createAgent, U as defaultEngine, V as defineAgent, X as defineAgentProfile, Y as defineChannel, Z as defineTool, _ as dispatchWorkflow, $ as genId, a0 as getDefaultPersistence, a1 as invokeAgent, a2 as isChannelDefinition, a3 as isCreatedAgent, a4 as isRawTool, a5 as parseWorkflowDef, a6 as rawTool, a7 as recoverRuns, a8 as recoverWorkflows, a9 as resolveAnyTool, aa as resolveTool, ab as startWorkflow, ac as verifyGithubWebhookSignature, ad as verifySlackRequestSignature, ae as waitingGate } from './channel-CRAw_ZfB.js';
 import * as v from 'valibot';
 import { GenericSchema, InferOutput } from 'valibot';
 import { b as SandboxFactory, c as SessionEnv, e as ShellResult, S as SandboxNetworkPolicy, C as Command, A as AtlasFs } from './command-kxrqWIH7.js';
@@ -111,6 +111,9 @@ declare const PersonaManifestSchema: v.ObjectSchema<{
                 readonly kind: v.LiteralSchema<"gate", undefined>;
                 readonly title: v.StringSchema<undefined>;
                 readonly description: v.OptionalSchema<v.StringSchema<undefined>, undefined>;
+                readonly onReject: v.OptionalSchema<v.ObjectSchema<{
+                    readonly goto: v.SchemaWithPipe<readonly [v.StringSchema<undefined>, v.MinLengthAction<string, 1, undefined>]>;
+                }, undefined>, undefined>;
             }, undefined>], undefined>, undefined>, v.MinLengthAction<({
                 id: string;
                 kind: "prompt";
@@ -128,6 +131,9 @@ declare const PersonaManifestSchema: v.ObjectSchema<{
                 kind: "gate";
                 title: string;
                 description?: string | undefined;
+                onReject?: {
+                    goto: string;
+                } | undefined;
             })[], 1, undefined>]>;
             readonly roleSlots: v.OptionalSchema<v.RecordSchema<v.StringSchema<undefined>, v.StringSchema<undefined>, undefined>, undefined>;
         }, undefined>, undefined>, undefined>;

package/dist/index.js

@@ -44,12 +44,13 @@ import {
   recoverWorkflows,
   resolveAnyTool,
   resolveTool,
+  sanitizeToolNamePart,
   startCronScheduler,
   startWorkflow,
   verifyGithubWebhookSignature,
   verifySlackRequestSignature,
   waitingGate
-} from "./chunk-RF6W3TKJ.js";
+} from "./chunk-EU2FU6RF.js";
 import {
   BaseSandboxEnv,
   bash,
@@ -95,7 +96,7 @@ import {
   resetProvidersForTests,
   resolveRegisteredModel,
   unregisterApiProviders
-} from "./chunk-HO6QHSUS.js";
+} from "./chunk-KY5APFKW.js";
 
 // src/persona.ts
 import * as v from "valibot";
@@ -261,23 +262,41 @@ async function resolveToolsBinding(m, ctx, bindings) {
   const tools = [];
   const toolsBySlot = {};
   const cleanup = [];
+  const nameSource = /* @__PURE__ */ new Map();
   const collect = (slotName, resolved) => {
-    tools.push(...resolved.tools);
-    if (slotName) toolsBySlot[slotName] = resolved.tools;
+    const label = slotName ? `slot "${slotName}"` : "bindings.tools";
     const dispose = resolved.cleanup ?? resolved.close;
     if (dispose) cleanup.push(dispose);
+    for (const tool of resolved.tools) {
+      const prior = nameSource.get(tool.name);
+      if (prior) {
+        throw new Error(
+          `Persona "${m.name}" binds two tools named "${tool.name}" (${prior} and ${label}). Give each connector a distinct name, e.g. defineMemoryConnector({ name: "memory_search_<pack>" }).`
+        );
+      }
+      nameSource.set(tool.name, label);
+    }
+    tools.push(...resolved.tools);
+    if (slotName) toolsBySlot[slotName] = resolved.tools;
+  };
+  const disposeAll = async () => {
+    for (const dispose of cleanup.toReversed()) await dispose();
   };
-  collect(void 0, await resolveOneToolsBinding(bindings?.tools, { ...ctx, manifest: m }));
-  for (const [slotName, source] of Object.entries(bindings?.toolSlots ?? {})) {
-    const slot = (m.slots ?? []).find((s) => s.name === slotName);
-    collect(slotName, await resolveOneToolsBinding(source, { ...ctx, manifest: m, slot }));
+  try {
+    collect(void 0, await resolveOneToolsBinding(bindings?.tools, { ...ctx, manifest: m }));
+    for (const [slotName, source] of Object.entries(bindings?.toolSlots ?? {})) {
+      const slot = (m.slots ?? []).find((s) => s.name === slotName);
+      collect(slotName, await resolveOneToolsBinding(source, { ...ctx, manifest: m, slot }));
+    }
+  } catch (err) {
+    await disposeAll().catch(() => {
+    });
+    throw err;
   }
   return {
     tools,
     toolsBySlot,
-    cleanup: cleanup.length === 0 ? void 0 : async () => {
-      for (const dispose of cleanup.toReversed()) await dispose();
-    }
+    cleanup: cleanup.length === 0 ? void 0 : disposeAll
   };
 }
 function personaAgent(manifest, bindings) {
@@ -661,10 +680,6 @@ function mergeRequestInit(requestInit, headers) {
 function createToolName(serverName, toolName) {
   return `mcp__${sanitizeToolNamePart(serverName)}__${sanitizeToolNamePart(toolName)}`;
 }
-function sanitizeToolNamePart(value) {
-  const sanitized = value.replace(/[^A-Za-z0-9_-]/g, "_").replace(/^_+|_+$/g, "");
-  return sanitized || "unnamed";
-}
 function createToolDescription(serverName, tool) {
   const originalName = tool.name;
   const title = tool.title ?? tool.annotations?.title;

package/dist/providers.d.ts

@@ -18,11 +18,20 @@ interface ProviderRegistration {
     headers?: Record<string, string>;
     contextWindow?: number;
     maxTokens?: number;
-    /** Per-model metadata overrides keyed by model id. */
+    /**
+     * Per-model metadata overrides keyed by model id. `reasoning` MUST be set for reasoning models
+     * reached through a custom relay (e.g. gpt-5.x on an OpenAI-compatible relay): the pi-ai catalog
+     * lookup misses for a custom provider, so the model falls back to `zeroMetadataModel` (reasoning:
+     * false) and pi-ai then mishandles the model's reasoning items — referencing them by id instead of
+     * inlining the encrypted content, which 404s against store-less relays. Declaring `reasoning: true`
+     * makes pi-ai request `include:["reasoning.encrypted_content"]`.
+     */
     models?: Record<string, {
         contextWindow?: number;
         maxTokens?: number;
         name?: string;
+        reasoning?: boolean;
+        thinkingLevelMap?: Record<string, string>;
     }>;
     /** Reserved for OpenAI Responses-style providers that support remote item storage. */
     storeResponses?: boolean;

package/dist/providers.js

@@ -10,7 +10,7 @@ import {
   resetProvidersForTests,
   resolveRegisteredModel,
   unregisterApiProviders
-} from "./chunk-HO6QHSUS.js";
+} from "./chunk-KY5APFKW.js";
 export {
   ProviderRegistrationError,
   getApiProvider,

package/dist/routing/index.d.ts

@@ -1,5 +1,5 @@
 import { MiddlewareHandler, Hono } from 'hono';
-import { C as CreatedAgent, A as AgentEngine, a as AgentProfile, W as WorkflowDef, b as ChannelDefinition, I as InvokeResult, D as DurableExecutionRunner } from '../channel-Dv3Hv1ee.js';
+import { C as CreatedAgent, A as AgentEngine, a as AgentProfile, W as WorkflowDef, b as ChannelDefinition, I as InvokeResult, D as DurableExecutionRunner } from '../channel-CRAw_ZfB.js';
 import { P as PersistenceAdapter, A as AtlasEvent } from '../index-UFTgKRK4.js';
 import 'valibot';
 import '../command-kxrqWIH7.js';

package/dist/routing/index.js

@@ -13,7 +13,7 @@ import {
   startWorkflow,
   waitingGate,
   waitingToolApproval
-} from "../chunk-RF6W3TKJ.js";
+} from "../chunk-EU2FU6RF.js";
 import "../chunk-4DU4GJ2X.js";
 import {
   isJsonStreamContentType,
@@ -21,7 +21,7 @@ import {
   streamMessageBytes
 } from "../chunk-M4JW76IL.js";
 import "../chunk-S7RZJMCF.js";
-import "../chunk-HO6QHSUS.js";
+import "../chunk-KY5APFKW.js";
 
 // src/routing/index.ts
 import { Hono } from "hono";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cybernetyx1/atlasflow-runtime",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "Portable TypeScript runtime for AtlasFlow agents, sessions, tools, sandboxes, routing, and persistence.",
   "type": "module",
   "license": "SEE LICENSE IN LICENSE",