@schoolai/shipyard-mcp 0.1.3-next.451 → 0.1.3-next.453

package/apps/hook/dist/index.js

@@ -28479,7 +28479,7 @@ init_cjs_shims();
 // ../../packages/schema/dist/index.mjs
 init_cjs_shims();
 
-// ../../packages/schema/dist/yjs-helpers-xzZyVQl7.mjs
+// ../../packages/schema/dist/yjs-helpers-CkZ1Z-ZW.mjs
 init_cjs_shims();
 
 // ../../packages/schema/dist/plan.mjs
@@ -42650,7 +42650,7 @@ var PRReviewCommentSchema = external_exports.object({
   resolved: external_exports.boolean().optional()
 });
 
-// ../../packages/schema/dist/yjs-helpers-xzZyVQl7.mjs
+// ../../packages/schema/dist/yjs-helpers-CkZ1Z-ZW.mjs
 function assertNever2(value) {
   throw new Error(`Unhandled discriminated union member: ${JSON.stringify(value)}`);
 }

package/apps/server/dist/{chunk-N44DCU4J.js → chunk-LD5NBKJ5.js}

@@ -3,7 +3,7 @@ import {
   YDOC_KEYS,
   createInputRequest,
   logPlanEvent
-} from "./chunk-LTC26IRQ.js";
+} from "./chunk-UN6RJ77A.js";
 import {
   logger
 } from "./chunk-GSGLHRWX.js";
@@ -156,10 +156,9 @@ var InputRequestManager = class {
       const handleTimeout = () => {
         if (resolved) return;
         logger.warn({ requestId, timeout: effectiveTimeout }, "Input request timed out");
-        markRequestAsCancelled();
-        if (resolved) return;
         resolved = true;
         cleanup();
+        markRequestAsCancelled();
         const timeStr = formatDuration(effectiveTimeout);
         resolve({
           success: false,
@@ -169,7 +168,6 @@ var InputRequestManager = class {
       };
       const markRequestAsCancelled = () => {
         ydoc.transact(() => {
-          if (resolved) return;
           const currentRequest = findRequest();
           if (!currentRequest || currentRequest.status !== "pending") {
             return;

package/apps/server/dist/{chunk-LTC26IRQ.js → chunk-UN6RJ77A.js}

@@ -863,7 +863,7 @@ function createHandedOffConversationVersion(params) {
   return ConversationVersionSchema.parse(version);
 }
 
-// ../../packages/schema/dist/yjs-helpers-xzZyVQl7.mjs
+// ../../packages/schema/dist/yjs-helpers-CkZ1Z-ZW.mjs
 import { z as z2 } from "zod";
 import { nanoid as nanoid2 } from "nanoid";
 import * as Y from "yjs";
@@ -959,6 +959,7 @@ var CreateSubscriptionRequestSchema = z2.object({
   threshold: z2.number().positive().optional()
 });
 var CreateSubscriptionResponseSchema = z2.object({ clientId: z2.string() });
+var DEFAULT_INPUT_REQUEST_TIMEOUT_SECONDS = 300;
 var InputRequestTypeValues = [
   "text",
   "multiline",
@@ -2770,6 +2771,7 @@ export {
   UnregisterServerResponseSchema,
   CreateSubscriptionRequestSchema,
   CreateSubscriptionResponseSchema,
+  DEFAULT_INPUT_REQUEST_TIMEOUT_SECONDS,
   InputRequestTypeValues,
   InputRequestStatusValues,
   InputRequestSchema,

package/apps/server/dist/{dist-LQBXUHLR.js → dist-4IAHBBMC.js}

@@ -23,6 +23,7 @@ import {
   CreateSubscriptionRequestSchema,
   CreateSubscriptionResponseSchema,
   CursorOriginMetadataSchema,
+  DEFAULT_INPUT_REQUEST_TIMEOUT_SECONDS,
   DeleteSubscriptionResponseSchema,
   DeliverableSchema,
   DevinOriginMetadataSchema,
@@ -201,7 +202,7 @@ import {
   updateLinkedPRStatus,
   updatePlanIndexViewedBy,
   validateA2AMessages
-} from "./chunk-LTC26IRQ.js";
+} from "./chunk-UN6RJ77A.js";
 import "./chunk-JSBRDJBE.js";
 export {
   A2ADataPartSchema,
@@ -228,6 +229,7 @@ export {
   CreateSubscriptionRequestSchema,
   CreateSubscriptionResponseSchema,
   CursorOriginMetadataSchema,
+  DEFAULT_INPUT_REQUEST_TIMEOUT_SECONDS,
   DeleteSubscriptionResponseSchema,
   DeliverableSchema,
   DevinOriginMetadataSchema,

package/apps/server/dist/index.js

@@ -18,7 +18,7 @@ import {
 } from "./chunk-EBNL5ZX7.js";
 import {
   InputRequestManager
-} from "./chunk-N44DCU4J.js";
+} from "./chunk-LD5NBKJ5.js";
 import {
   ArtifactSchema,
   DeliverableSchema,
@@ -67,7 +67,7 @@ import {
   setPlanMetadata,
   touchPlanIndexEntry,
   transitionPlanStatus
-} from "./chunk-LTC26IRQ.js";
+} from "./chunk-UN6RJ77A.js";
 import {
   loadEnv,
   logger
@@ -1027,16 +1027,14 @@ async function waitForApprovalHandler(planId, _reviewRequestIdParam, ctx) {
     if (!reviewedBy) {
       throw new Error(`Invalid session state transition: missing reviewedBy for changes_requested`);
     }
-    if (!syncedFields) {
-      throw new Error(
-        `Invalid session state transition: changes_requested requires synced fields (contentHash, sessionToken)`
-      );
-    }
     const deliverables = extraData.deliverables || (isSessionStateApproved(session) || isSessionStateReviewed(session) || isSessionStateApprovedAwaitingToken(session) ? session.deliverables : []);
+    const webUrl = webConfig.SHIPYARD_WEB_URL;
     setSessionState(sessionId, {
       lifecycle: "reviewed",
       ...baseState,
-      ...syncedFields,
+      contentHash: syncedFields?.contentHash ?? "",
+      sessionToken: syncedFields?.sessionToken ?? "",
+      url: syncedFields?.url ?? `${webUrl}/plan/${baseState.planId}`,
       deliverables,
       reviewComment: reviewComment || "",
       reviewedBy,
@@ -5064,7 +5062,7 @@ async function setupReviewNotification(planId, pollIntervalSeconds) {
   return { script, fullResponse: text };
 }
 async function requestUserInput(opts) {
-  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-MVKPYLFW.js");
+  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-RO5G36ST.js");
   const ydoc = await getOrCreateDoc3(PLAN_INDEX_DOC_NAME);
   const manager = new InputRequestManager2();
   const params = opts.type === "choice" ? {
@@ -5108,7 +5106,7 @@ async function requestUserInput(opts) {
   };
 }
 async function postActivityUpdate(opts) {
-  const { logPlanEvent: logPlanEvent2 } = await import("./dist-LQBXUHLR.js");
+  const { logPlanEvent: logPlanEvent2 } = await import("./dist-4IAHBBMC.js");
   const { getGitHubUsername: getGitHubUsername2 } = await import("./server-identity-KUXYHULN.js");
   const { nanoid: nanoid7 } = await import("nanoid");
   const doc = await getOrCreateDoc3(opts.planId);
@@ -5131,7 +5129,7 @@ async function postActivityUpdate(opts) {
   return { success: true, eventId, requestId };
 }
 async function resolveActivityRequest(opts) {
-  const { logPlanEvent: logPlanEvent2, getPlanEvents } = await import("./dist-LQBXUHLR.js");
+  const { logPlanEvent: logPlanEvent2, getPlanEvents } = await import("./dist-4IAHBBMC.js");
   const { getGitHubUsername: getGitHubUsername2 } = await import("./server-identity-KUXYHULN.js");
   const doc = await getOrCreateDoc3(opts.planId);
   const actorName = await getGitHubUsername2();

package/apps/server/dist/{input-request-manager-MVKPYLFW.js → input-request-manager-RO5G36ST.js}

@@ -1,7 +1,7 @@
 import {
   InputRequestManager
-} from "./chunk-N44DCU4J.js";
-import "./chunk-LTC26IRQ.js";
+} from "./chunk-LD5NBKJ5.js";
+import "./chunk-UN6RJ77A.js";
 import "./chunk-GSGLHRWX.js";
 import "./chunk-JSBRDJBE.js";
 export {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@schoolai/shipyard-mcp",
-  "version": "0.1.3-next.451",
+  "version": "0.1.3-next.453",
   "description": "Shipyard MCP server and CLI tools for distributed planning with CRDTs",
   "type": "module",
   "bin": {