@sema-agent/core 1.410.0 → 1.413.0

package/dist/core/runner/prepare-task.js

@@ -224,8 +224,8 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
         ? resolveModel(spec.compactionModel, deps.models)
         : roleModelIfSet("summarize", spec, deps);
     const toolEffects = new Map();
-    const toolNameForSafety = new Map();
-    const safetyKeyOf = (raw) => toolNameForSafety.get(raw) ?? canonicalToolName(raw);
+    const ownToolNames = new Set((spec.tools ?? []).map((t) => t.name));
+    const claimedAliases = new Set();
     const egressTools = new Set();
     const irreversibleTools = new Set();
     const irreversibilityTier = new Map();
@@ -241,8 +241,6 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
         if (t.effect) {
             toolEffects.set(t.name, t.effect);
         }
-        for (const alias of t.aliases ?? [])
-            toolNameForSafety.set(alias, t.name);
         const tier = t.irreversibility ?? (t.reversibilityProbe ? "maybe" : undefined);
         if (tier !== undefined) {
             irreversibilityTier.set(t.name, tier);
@@ -259,6 +257,24 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
             }
             egressTools.add(t.name);
         }
+        for (const alias of t.aliases ?? []) {
+            if (alias === t.name || ownToolNames.has(alias))
+                continue;
+            if (claimedAliases.has(alias))
+                continue;
+            claimedAliases.add(alias);
+            if (t.effect)
+                toolEffects.set(alias, t.effect);
+            if (tier !== undefined) {
+                irreversibilityTier.set(alias, tier);
+                if (tier === "always" || tier === "maybe")
+                    irreversibleTools.add(alias);
+            }
+            if (t.reversibilityProbe)
+                reversibilityProbes.set(alias, t.reversibilityProbe);
+            if (t.egress)
+                egressTools.add(alias);
+        }
     }
     toolEffects.set(OFFLOAD_TOOL_NAME, "read");
     if (deps.executionEnvFactory || deps.executionEnv) {
@@ -1772,6 +1788,7 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
             }
         }
         let centerAdoption;
+        let centerAdoptionFresh = false;
         {
             const failUnavailable = (digest, via) => {
                 const e = new Error(`prompt snapshot unavailable: ${via} center prompt artifact ${digest.slice(0, 23)}… has no verified resolvable copy (artifact store miss/corrupt${deps.promptSource === undefined ? "; RunnerDeps.promptSource is not configured" : ""}) — refusing to recompose a hybrid prompt (protocol §9.2)`);
@@ -1799,6 +1816,7 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
                     const candidate = deps.promptSource?.current();
                     if (candidate !== undefined && (await session.getBranch()).length === 0) {
                         centerAdoption = { artifact: candidate.artifact, sourceRevision: candidate.sourceRevision };
+                        centerAdoptionFresh = true;
                     }
                 }
             }
@@ -1828,9 +1846,19 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
         });
         const assembled = assemblePrompt(buildAssembleInputs(centerAdoption !== undefined ? artifactDeclarations(centerAdoption.artifact) : undefined));
         if (centerAdoption !== undefined && assembled.centerMounted !== true) {
-            const e = new Error(`center prompt artifact ${centerAdoption.artifact.artifactDigest.slice(0, 23)}… cannot mount: this PromptProvider owns the whole prompt (legacy system()/assembled-identity pass-through) — upgrade the provider to stableBlocks/stableSystem or remove RunnerDeps.promptSource for this deployment`);
-            e.code = "prompt.center_unmountable";
-            throw e;
+            const ownsWholePrompt = assembled.constitution === "legacy" || assembled.constitution === "provider-assembled";
+            const digest = centerAdoption.artifact.artifactDigest.slice(0, 23);
+            if (!ownsWholePrompt && centerAdoptionFresh) {
+                deps.onError?.(new Error(`prompt-constitution: center prompt artifact ${digest}… contributed NO section to this prompt — every section it declares collides with a PromptProvider stableBlocks declaration of the same id, and the deployment-local declaration wins. This session runs on the bundled prompt and is deliberately NOT pinned to the artifact (a pin claiming content the model never received would be durable and wrong). Rename the colliding section ids on either side to adopt it; the per-section collisions are reported individually on this channel.`), { phase: "prompt-constitution", sessionId });
+                centerAdoption = undefined;
+            }
+            else {
+                const e = new Error(ownsWholePrompt
+                    ? `center prompt artifact ${digest}… cannot mount: this PromptProvider owns the whole prompt (legacy system()/assembled-identity pass-through) — upgrade the provider to stableBlocks/stableSystem or remove RunnerDeps.promptSource for this deployment`
+                    : `center prompt artifact ${digest}… cannot mount: every one of its sections collides with a PromptProvider stableBlocks declaration of the same id, so zero center content would reach the model — but this session is ALREADY pinned to that artifact (or inherited it from a spawning parent), so quietly dropping it would rewrite what the session claims about itself. Rename the colliding section ids on either side, or start a fresh session`);
+                e.code = "prompt.center_unmountable";
+                throw e;
+            }
         }
         let systemPrompt = assembled.systemPrompt;
         const renderWithDate = assembled.renderWithDate
@@ -3069,8 +3097,8 @@ export async function prepareTask(spec, deps, sessions, resume, _memorySelector,
                             adjudicate,
                             resolveAsk: resolveAskBound,
                             suspendAsk,
-                            egress: egressTools.has(safetyKeyOf(e.toolName)),
-                            irreversibility: irreversibilityTier.get(safetyKeyOf(e.toolName)),
+                            egress: egressTools.has(e.toolName),
+                            irreversibility: irreversibilityTier.get(e.toolName),
                             reversibilityProbe: reversibilityProbes.get(e.toolName),
                             abortSignal: abortController.signal,
                             ...(notifyPermissionDenied ? { permissionDenied: notifyPermissionDenied } : {}),