@schoolai/shipyard-mcp 0.1.0 → 0.1.1

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

@@ -9,6 +9,19 @@ import {
 } from "./chunk-GSGLHRWX.js";
 
 // src/services/input-request-manager.ts
+function formatDuration(totalSeconds) {
+  const minutes = Math.floor(totalSeconds / 60);
+  const seconds = totalSeconds % 60;
+  if (minutes === 0) {
+    return `${seconds} second${seconds === 1 ? "" : "s"}`;
+  }
+  const minutePart = `${minutes} minute${minutes === 1 ? "" : "s"}`;
+  if (seconds === 0) {
+    return minutePart;
+  }
+  const secondPart = `${seconds} second${seconds === 1 ? "" : "s"}`;
+  return `${minutePart} ${secondPart}`;
+}
 var InputRequestManager = class {
   /**
    * Create a new input request in the Y.Doc.
@@ -147,9 +160,7 @@ var InputRequestManager = class {
         if (resolved) return;
         resolved = true;
         cleanup();
-        const minutes = Math.floor(effectiveTimeout / 60);
-        const seconds = effectiveTimeout % 60;
-        const timeStr = minutes > 0 ? `${minutes} minute${minutes === 1 ? "" : "s"}${seconds > 0 ? ` ${seconds} second${seconds === 1 ? "" : "s"}` : ""}` : `${seconds} second${seconds === 1 ? "" : "s"}`;
+        const timeStr = formatDuration(effectiveTimeout);
         resolve({
           success: false,
           status: "cancelled",

package/apps/server/dist/index.js

@@ -18,7 +18,7 @@ import {
 } from "./chunk-EBNL5ZX7.js";
 import {
   InputRequestManager
-} from "./chunk-SIQIBECG.js";
+} from "./chunk-N44DCU4J.js";
 import {
   ArtifactSchema,
   DeliverableSchema,
@@ -899,15 +899,17 @@ async function waitForApprovalHandler(planId, _reviewRequestIdParam, ctx) {
     throw err;
   }
   const metadata = ydoc.getMap(YDOC_KEYS.METADATA);
-  const reviewRequestId = nanoid2();
   const planMetadata = getPlanMetadata(ydoc);
   const ownerId = planMetadata?.ownerId ?? "unknown";
-  if (planMetadata?.status === "pending_review") {
-    ctx.logger.warn(
-      { planId, currentStatus: planMetadata.status },
-      "Status already pending_review, another hook may be waiting. Skipping reviewRequestId update."
+  let reviewRequestId;
+  if (planMetadata?.status === "pending_review" && planMetadata.reviewRequestId) {
+    reviewRequestId = planMetadata.reviewRequestId;
+    ctx.logger.info(
+      { planId, currentStatus: planMetadata.status, reviewRequestId },
+      "Status already pending_review, reusing existing reviewRequestId for observer"
     );
   } else {
+    reviewRequestId = nanoid2();
     const result = transitionPlanStatus(
       ydoc,
       {
@@ -1174,11 +1176,15 @@ function extractFeedbackFromYDoc(ydoc, ctx) {
       return "Changes requested. Check the plan for reviewer comments.";
     }
     const contentFragment = ydoc.getXmlFragment(YDOC_KEYS.DOCUMENT_FRAGMENT);
-    const blocks = contentFragment.toJSON();
-    const planText = blocks.map((block) => {
-      if (!block.content || !Array.isArray(block.content)) return "";
-      return block.content.map((item) => typeof item === "object" && item && "text" in item ? item.text : "").join("");
-    }).filter(Boolean).join("\n");
+    const fragmentJson = contentFragment.toJSON();
+    let planText = "";
+    if (Array.isArray(fragmentJson)) {
+      const blocks = fragmentJson;
+      planText = blocks.map((block) => {
+        if (!block.content || !Array.isArray(block.content)) return "";
+        return block.content.map((item) => typeof item === "object" && item && "text" in item ? item.text : "").join("");
+      }).filter(Boolean).join("\n");
+    }
     const resolveUser = createUserResolver(ydoc);
     const feedbackText = formatThreadsForLLM(threads, {
       includeResolved: false,
@@ -5058,7 +5064,7 @@ async function setupReviewNotification(planId, pollIntervalSeconds) {
   return { script, fullResponse: text };
 }
 async function requestUserInput(opts) {
-  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-YVKAMQPF.js");
+  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-MVKPYLFW.js");
   const ydoc = await getOrCreateDoc3(PLAN_INDEX_DOC_NAME);
   const manager = new InputRequestManager2();
   const params = opts.type === "choice" ? {

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

@@ -1,6 +1,6 @@
 import {
   InputRequestManager
-} from "./chunk-SIQIBECG.js";
+} from "./chunk-N44DCU4J.js";
 import "./chunk-LTC26IRQ.js";
 import "./chunk-GSGLHRWX.js";
 import "./chunk-JSBRDJBE.js";

package/package.json

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