atris 3.37.0 → 3.38.0

package/lib/wish-delegate.js

@@ -919,9 +919,13 @@ function startWishDelegation(wish, audit, root, options = {}) {
   const reused = missionPayload && missionPayload.action === 'mission_reused';
   // A delegated wish must have a live worker, not just a filed mission. Start
   // the detached driver for any runner that can work headlessly; runners that
-  // need a live session keep an honest not-started marker instead.
+  // need a live session keep an honest not-started marker instead. A one-lap
+  // intake is the exception: the calling lap dispatches in-process through the
+  // sealed review sandbox, so a detached driver would race it with a second,
+  // ungated dispatch of the same ask.
   let driver = null;
-  if (mission && !missionRunnerNeedsLiveSession(mission.runner) && process.env.ATRIS_WISH_NO_DRIVER !== '1') {
+  const lapOwnsDispatch = options.oneLap === true;
+  if (mission && !lapOwnsDispatch && !missionRunnerNeedsLiveSession(mission.runner) && process.env.ATRIS_WISH_NO_DRIVER !== '1') {
     driver = spawnMissionDriver(mission, root);
   }
   const workerStarted = Boolean(driver && driver.ok && (driver.pid || driver.already_running));
@@ -940,9 +944,11 @@ function startWishDelegation(wish, audit, root, options = {}) {
     worker_started: workerStarted,
     ...(driver && driver.pid ? { driver_pid: driver.pid } : {}),
     ...(mission && !workerStarted ? {
-      worker_note: missionRunnerNeedsLiveSession(mission.runner)
-        ? `waiting for a live ${mission.runner || 'session'} to pick this up`
-        : `worker did not start; run: atris mission run ${mission.id}`,
+      worker_note: lapOwnsDispatch
+        ? 'the one-lap dispatches this in-process'
+        : missionRunnerNeedsLiveSession(mission.runner)
+          ? `waiting for a live ${mission.runner || 'session'} to pick this up`
+          : `worker did not start; run: atris mission run ${mission.id}`,
     } : {}),
     engine: reused ? (mission?.runner || engine) : engine,
     validator: audit.validator.id,
@@ -1422,7 +1428,12 @@ function dispatchWishHeadlessly(wish, root, reason) {
       questions: audit.questions,
     };
   }
-  const { record } = appendDelegatedWishRecord(wish, audit, root);
+  // Preserve one-lap ownership through sweep/headless grant: without this,
+  // a later atris wish sweep would spawn the detached mission driver and race
+  // the lap's sealed in-process dispatch (CLI-1170).
+  const { record } = appendDelegatedWishRecord(wish, audit, root, {
+    oneLap: wish.one_lap === true,
+  });
   return {
     id: wish.id,
     status: 'delegated',
@@ -1531,6 +1542,9 @@ function runCapturedWish(text, root = process.cwd(), options = {}) {
     status: 'captured',
   };
   if (options.parentId) wish.parent_id = String(options.parentId);
+  // One-lap provenance lives on the record so a later `wish answer` also
+  // skips the detached driver: the lap owns dispatch in-process (80de4ae1).
+  if (options.singleTask === true) wish.one_lap = true;
   if (engineOverride) wish.requested_engine = engineOverride;
   if (options.metric && options.metric.ok) wish.metric = options.metric.expression;
   captureWishToJournal(text, root);
@@ -1539,7 +1553,10 @@ function runCapturedWish(text, root = process.cwd(), options = {}) {
   const taskText = wishBodyText(text);
   const verifyPlan = options.verifyPlan || deriveVerifyPlan(text);
   const decompositionVerifyPlan = options.verifyPlan || (verifyPlan.status === 'explicit' ? verifyPlan : null);
-  const audit = auditWish(taskText, root, { engineOverride });
+  // A one-lap intake owns dispatch and validation policy in-process, so its
+  // route must stay deterministic across machines: never interview about the
+  // reviewer just because no validator binary is installed here.
+  const audit = auditWish(taskText, root, { engineOverride, assumeValidator: options.singleTask === true });
   if (builderMode) return builderWish(wish, audit, root, asJson);
   const parts = options.singleTask ? null : analyzeWishParts(text, root);
   if (parts && noMission) return captureOnlyWish(wish, null, root, asJson);
@@ -1552,6 +1569,7 @@ function runCapturedWish(text, root = process.cwd(), options = {}) {
   return delegateWish(wish, audit, root, asJson, {
     taskText,
     recordText: wish.text,
+    oneLap: options.singleTask === true,
     ...(options.metric ? { metric: options.metric } : {}),
     verifyPlan,
     userVerifyProvided: options.userVerifyProvided === true || verifyPlan.status === 'explicit',
@@ -1622,7 +1640,7 @@ function applyAnswer(wish, answer, root, options = {}) {
   const audit = bestGuessAfterRepeatedQuestion(answeredWish, audited, root);
   if (!audit.ok) return askForInput(answeredWish, audit, root, asJson);
   if (noMission) return captureOnlyWish(answeredWish, audit, root, asJson);
-  return delegateWish(answeredWish, audit, root, asJson, {});
+  return delegateWish(answeredWish, audit, root, asJson, { oneLap: wish.one_lap === true });
 }
 
 function sameWaitingPart(left, right) {
@@ -1811,25 +1829,13 @@ function grantWish(positionals, root = process.cwd(), options = {}) {
 module.exports = {
   REVIEW_WORD_SCORES,
   answerWish,
-  appendDelegatedWishRecord,
-  appendNeedsInputRecord,
-  askForInput,
-  captureOnlyWish,
   captureWishToJournal,
-  delegateWish,
-  dispatchWishHeadlessly,
-  fulfillLandedWishes,
   grantWish,
-  metricVerifierCommand,
-  listedWishes,
-  openWishes,
   parseMetricExpression,
   printBoard,
   printList,
   printReviewNudges,
   printRewards,
-  reviewNudgeWishes,
-  reviewWordScore,
   reviewWish,
   runAgainWish,
   runCapturedWish,

package/lib/wish-stats.js

@@ -173,11 +173,6 @@ function printWishStats(root = process.cwd(), options = {}) {
 }
 
 module.exports = {
-  buildWishStats,
-  haikuRates,
-  missionReceiptFiles,
   printWishStats,
   verifierPassed,
-  wishStatsLogPath,
-  writeWishStats,
 };

package/lib/wish-store.js

@@ -338,7 +338,6 @@ module.exports = {
   improveWishes,
   lessonsFile,
   pruneWishLessons,
-  readWishLessonLines,
   wishLessonsBrief,
   wishLessonsSummary,
   latestWishEventMap,

package/lib/workspace-safety.js

@@ -80,8 +80,6 @@ function assertSafeWorkspaceRoot(dir, { slug, op } = {}) {
 }
 
 module.exports = {
-  dangerousWorkspaceReason,
-  safeFallbackDir,
   resolveSafeOutputDir,
   assertSafeWorkspaceRoot,
 };

package/lib/zip.js

@@ -217,5 +217,4 @@ module.exports = {
   writeZipFile,
   readZipBuffer,
   readZipFile,
-  crc32,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "atris",
-  "version": "3.37.0",
+  "version": "3.38.0",
   "description": "you say what you want in plain words. atris builds it, checks it, and shows you proof.",
   "main": "bin/atris.js",
   "bin": {