@schoolai/shipyard-mcp 0.2.2-next.485 → 0.2.2-next.489

package/apps/hook/dist/index.js

@@ -42603,10 +42603,9 @@ var PlanMetadataSchema = external_exports.discriminatedUnion("status", [
 var BaseArtifactSchema = external_exports.object({
   id: external_exports.string(),
   type: external_exports.enum([
-    "screenshot",
-    "video",
-    "test_results",
-    "diff"
+    "html",
+    "image",
+    "video"
   ]),
   filename: external_exports.string(),
   description: external_exports.string().optional(),
@@ -43793,6 +43792,29 @@ var GitHubPRResponseSchema = external_exports.object({
   merged: external_exports.boolean(),
   head: external_exports.object({ ref: external_exports.string() })
 });
+var ROUTES = {
+  REGISTRY_LIST: "/registry",
+  REGISTRY_REGISTER: "/register",
+  REGISTRY_UNREGISTER: "/unregister",
+  PLAN_STATUS: (planId) => `/api/plan/${planId}/status`,
+  PLAN_HAS_CONNECTIONS: (planId) => `/api/plan/${planId}/has-connections`,
+  PLAN_TRANSCRIPT: (planId) => `/api/plan/${planId}/transcript`,
+  PLAN_SUBSCRIBE: (planId) => `/api/plan/${planId}/subscribe`,
+  PLAN_CHANGES: (planId) => `/api/plan/${planId}/changes`,
+  PLAN_UNSUBSCRIBE: (planId) => `/api/plan/${planId}/unsubscribe`,
+  PLAN_PR_DIFF: (planId, prNumber) => `/api/plans/${planId}/pr-diff/${prNumber}`,
+  PLAN_PR_FILES: (planId, prNumber) => `/api/plans/${planId}/pr-files/${prNumber}`,
+  HOOK_SESSION: "/api/hook/session",
+  HOOK_CONTENT: (planId) => `/api/hook/plan/${planId}/content`,
+  HOOK_REVIEW: (planId) => `/api/hook/plan/${planId}/review`,
+  HOOK_SESSION_TOKEN: (planId) => `/api/hook/plan/${planId}/session-token`,
+  HOOK_PRESENCE: (planId) => `/api/hook/plan/${planId}/presence`,
+  CONVERSATION_IMPORT: "/api/conversation/import",
+  WEB_TASK: (planId) => `/task/${planId}`
+};
+function createPlanWebUrl(baseUrl, planId) {
+  return `${baseUrl.replace(/\/$/, "")}${ROUTES.WEB_TASK(planId)}`;
+}
 var InviteTokenSchema = external_exports.object({
   id: external_exports.string(),
   tokenHash: external_exports.string(),
@@ -45750,7 +45772,7 @@ async function handleUpdatedPlanReview(sessionId, planId, planContent, _originMe
   }
   const baseUrl = webConfig.SHIPYARD_WEB_URL;
   logger.info(
-    { planId, url: `${baseUrl}/plan/${planId}` },
+    { planId, url: createPlanWebUrl(baseUrl, planId) },
     "Content synced, browser already open. Waiting for server approval..."
   );
   const decision = await waitForReviewDecision(planId, "");
@@ -45907,7 +45929,7 @@ Reviewer comment: ${decision.reviewComment}` : "";
         allow: false,
         message: `Plan is pending review.
 
-Open: ${baseUrl}/plan/${planId}`,
+Open: ${createPlanWebUrl(baseUrl, planId)}`,
         planId
       };
     case "draft":
@@ -45915,7 +45937,7 @@ Open: ${baseUrl}/plan/${planId}`,
         allow: false,
         message: `Plan is still in draft.
 
-Submit for review at: ${baseUrl}/plan/${planId}`,
+Submit for review at: ${createPlanWebUrl(baseUrl, planId)}`,
         planId
       };
     case "in_progress":

package/apps/server/dist/{chunk-NH6GXGQO.js → chunk-7ISZ4RKB.js}

@@ -3,7 +3,7 @@ import {
   YDOC_KEYS,
   createInputRequest,
   logPlanEvent
-} from "./chunk-BBYZISY2.js";
+} from "./chunk-CYHEHTLS.js";
 import {
   logger
 } from "./chunk-GSGLHRWX.js";

package/apps/server/dist/{chunk-BBYZISY2.js → chunk-CYHEHTLS.js}

@@ -760,10 +760,9 @@ var PlanMetadataSchema = z.discriminatedUnion("status", [
 var BaseArtifactSchema = z.object({
   id: z.string(),
   type: z.enum([
-    "screenshot",
-    "video",
-    "test_results",
-    "diff"
+    "html",
+    "image",
+    "video"
   ]),
   filename: z.string(),
   description: z.string().optional(),
@@ -2274,6 +2273,29 @@ function asWebRTCPeerId(id) {
 function asGitHubUsername(username) {
   return username;
 }
+var ROUTES = {
+  REGISTRY_LIST: "/registry",
+  REGISTRY_REGISTER: "/register",
+  REGISTRY_UNREGISTER: "/unregister",
+  PLAN_STATUS: (planId) => `/api/plan/${planId}/status`,
+  PLAN_HAS_CONNECTIONS: (planId) => `/api/plan/${planId}/has-connections`,
+  PLAN_TRANSCRIPT: (planId) => `/api/plan/${planId}/transcript`,
+  PLAN_SUBSCRIBE: (planId) => `/api/plan/${planId}/subscribe`,
+  PLAN_CHANGES: (planId) => `/api/plan/${planId}/changes`,
+  PLAN_UNSUBSCRIBE: (planId) => `/api/plan/${planId}/unsubscribe`,
+  PLAN_PR_DIFF: (planId, prNumber) => `/api/plans/${planId}/pr-diff/${prNumber}`,
+  PLAN_PR_FILES: (planId, prNumber) => `/api/plans/${planId}/pr-files/${prNumber}`,
+  HOOK_SESSION: "/api/hook/session",
+  HOOK_CONTENT: (planId) => `/api/hook/plan/${planId}/content`,
+  HOOK_REVIEW: (planId) => `/api/hook/plan/${planId}/review`,
+  HOOK_SESSION_TOKEN: (planId) => `/api/hook/plan/${planId}/session-token`,
+  HOOK_PRESENCE: (planId) => `/api/hook/plan/${planId}/presence`,
+  CONVERSATION_IMPORT: "/api/conversation/import",
+  WEB_TASK: (planId) => `/task/${planId}`
+};
+function createPlanWebUrl(baseUrl, planId) {
+  return `${baseUrl.replace(/\/$/, "")}${ROUTES.WEB_TASK(planId)}`;
+}
 var InviteTokenSchema = z3.object({
   id: z3.string(),
   tokenHash: z3.string(),
@@ -2307,7 +2329,7 @@ function parseInviteFromUrl(url) {
 }
 function buildInviteUrl(baseUrl, planId, tokenId, tokenValue) {
   const normalizedBase = baseUrl.endsWith("/") ? baseUrl.slice(0, -1) : baseUrl;
-  const url = new URL(`${normalizedBase}/task/${planId}`);
+  const url = new URL(`${normalizedBase}${ROUTES.WEB_TASK(planId)}`);
   url.searchParams.set("invite", `${tokenId}:${tokenValue}`);
   return url.toString();
 }
@@ -2588,25 +2610,6 @@ function getAllEventViewedByForPlan(ydoc, planId) {
   }
   return result;
 }
-var ROUTES = {
-  REGISTRY_LIST: "/registry",
-  REGISTRY_REGISTER: "/register",
-  REGISTRY_UNREGISTER: "/unregister",
-  PLAN_STATUS: (planId) => `/api/plan/${planId}/status`,
-  PLAN_HAS_CONNECTIONS: (planId) => `/api/plan/${planId}/has-connections`,
-  PLAN_TRANSCRIPT: (planId) => `/api/plan/${planId}/transcript`,
-  PLAN_SUBSCRIBE: (planId) => `/api/plan/${planId}/subscribe`,
-  PLAN_CHANGES: (planId) => `/api/plan/${planId}/changes`,
-  PLAN_UNSUBSCRIBE: (planId) => `/api/plan/${planId}/unsubscribe`,
-  PLAN_PR_DIFF: (planId, prNumber) => `/api/plans/${planId}/pr-diff/${prNumber}`,
-  PLAN_PR_FILES: (planId, prNumber) => `/api/plans/${planId}/pr-files/${prNumber}`,
-  HOOK_SESSION: "/api/hook/session",
-  HOOK_CONTENT: (planId) => `/api/hook/plan/${planId}/content`,
-  HOOK_REVIEW: (planId) => `/api/hook/plan/${planId}/review`,
-  HOOK_SESSION_TOKEN: (planId) => `/api/hook/plan/${planId}/session-token`,
-  HOOK_PRESENCE: (planId) => `/api/hook/plan/${planId}/presence`,
-  CONVERSATION_IMPORT: "/api/conversation/import"
-};
 function formatThreadsForLLM(threads, options = {}) {
   const { includeResolved = false, selectedTextMaxLength = 100, resolveUser } = options;
   const unresolvedThreads = threads.filter((t$1) => !t$1.resolved);
@@ -2947,6 +2950,8 @@ export {
   asAwarenessClientId,
   asWebRTCPeerId,
   asGitHubUsername,
+  ROUTES,
+  createPlanWebUrl,
   InviteTokenSchema,
   InviteRedemptionSchema,
   parseInviteFromUrl,
@@ -2987,7 +2992,6 @@ export {
   clearEventViewedBy,
   isEventUnread,
   getAllEventViewedByForPlan,
-  ROUTES,
   formatThreadsForLLM,
   PlanIdSchema,
   PlanStatusResponseSchema,

package/apps/server/dist/{dist-D4DG2R4R.js → dist-HCVQNHBC.js}

@@ -109,6 +109,7 @@ import {
   createPlanSnapshot,
   createPlanUrl,
   createPlanUrlWithHistory,
+  createPlanWebUrl,
   createUserResolver,
   declineInputRequest,
   decodeChunkMessage,
@@ -205,7 +206,7 @@ import {
   updateLinkedPRStatus,
   updatePlanIndexViewedBy,
   validateA2AMessages
-} from "./chunk-BBYZISY2.js";
+} from "./chunk-CYHEHTLS.js";
 import "./chunk-JSBRDJBE.js";
 export {
   A2ADataPartSchema,
@@ -318,6 +319,7 @@ export {
   createPlanSnapshot,
   createPlanUrl,
   createPlanUrlWithHistory,
+  createPlanWebUrl,
   createUserResolver,
   declineInputRequest,
   decodeChunkMessage,

package/apps/server/dist/index.js

@@ -20,7 +20,7 @@ import {
 } from "./chunk-EBNL5ZX7.js";
 import {
   InputRequestManager
-} from "./chunk-NH6GXGQO.js";
+} from "./chunk-7ISZ4RKB.js";
 import {
   ArtifactSchema,
   DeliverableSchema,
@@ -46,6 +46,7 @@ import {
   createLinkedPR,
   createPlanSnapshot,
   createPlanUrlWithHistory,
+  createPlanWebUrl,
   createUserResolver,
   extractDeliverables,
   extractMentions,
@@ -71,7 +72,7 @@ import {
   setPlanMetadata,
   touchPlanIndexEntry,
   transitionPlanStatus
-} from "./chunk-BBYZISY2.js";
+} from "./chunk-CYHEHTLS.js";
 import {
   loadEnv,
   logger
@@ -592,8 +593,7 @@ function extractTitleFromBlocks(blocks) {
 async function createSessionHandler(input, ctx) {
   const existingSession = getSessionState(input.sessionId);
   if (existingSession) {
-    const webUrl2 = webConfig.SHIPYARD_WEB_URL;
-    const url2 = `${webUrl2}/plan/${existingSession.planId}`;
+    const url2 = createPlanWebUrl(webConfig.SHIPYARD_WEB_URL, existingSession.planId);
     ctx.logger.info(
       { planId: existingSession.planId, sessionId: input.sessionId },
       "Returning existing session (idempotent)"
@@ -658,8 +658,7 @@ async function createSessionHandler(input, ctx) {
     ownerId,
     deleted: false
   });
-  const webUrl = webConfig.SHIPYARD_WEB_URL;
-  const url = `${webUrl}/plan/${planId}`;
+  const url = createPlanWebUrl(webConfig.SHIPYARD_WEB_URL, planId);
   ctx.logger.info({ url }, "Plan URL generated");
   setSessionState(input.sessionId, {
     lifecycle: "created",
@@ -838,8 +837,7 @@ async function setSessionTokenHandler(planId, sessionTokenHash, ctx) {
   setPlanMetadata(ydoc, {
     sessionTokenHash
   });
-  const webUrl = webConfig.SHIPYARD_WEB_URL;
-  const url = `${webUrl}/plan/${planId}`;
+  const url = createPlanWebUrl(webConfig.SHIPYARD_WEB_URL, planId);
   const session = getSessionStateByPlanId(planId);
   const sessionId = getSessionIdByPlanId(planId);
   if (session && sessionId) {
@@ -1019,7 +1017,7 @@ async function waitForApprovalHandler(planId, _reviewRequestIdParam, ctx) {
       setSessionState(sessionId, {
         lifecycle: "approved_awaiting_token",
         ...baseState,
-        url: `${webUrl}/plan/${baseState.planId}`,
+        url: createPlanWebUrl(webUrl, baseState.planId),
         approvedAt: extraData.approvedAt,
         deliverables: extraData.deliverables,
         reviewComment,
@@ -1038,7 +1036,7 @@ async function waitForApprovalHandler(planId, _reviewRequestIdParam, ctx) {
       ...baseState,
       contentHash: syncedFields?.contentHash ?? "",
       sessionToken: syncedFields?.sessionToken ?? "",
-      url: syncedFields?.url ?? `${webUrl}/plan/${baseState.planId}`,
+      url: syncedFields?.url ?? createPlanWebUrl(webUrl, baseState.planId),
       deliverables,
       reviewComment: reviewComment || "",
       reviewedBy,
@@ -1231,8 +1229,7 @@ async function getDeliverableContextHandler(planId, sessionToken, ctx) {
     });
   }
   const deliverables = getDeliverables(ydoc);
-  const webUrl = webConfig.SHIPYARD_WEB_URL;
-  const url = `${webUrl}/plan/${planId}`;
+  const url = createPlanWebUrl(webConfig.SHIPYARD_WEB_URL, planId);
   let deliverablesSection = "";
   if (deliverables.length > 0) {
     deliverablesSection = `
@@ -2551,7 +2548,7 @@ var TOOL_NAMES = {
 var AddArtifactInputBase = z3.object({
   planId: z3.string().describe("The plan ID to add artifact to"),
   sessionToken: z3.string().describe("Session token from create_plan"),
-  type: z3.enum(["screenshot", "video", "test_results", "diff"]).describe("Artifact type"),
+  type: z3.enum(["html", "image", "video"]).describe("Artifact type"),
   filename: z3.string().describe("Filename for the artifact"),
   description: z3.string().optional().describe("What this artifact proves (deliverable name)"),
   deliverableId: z3.string().optional().describe("ID of the deliverable this artifact fulfills")
@@ -2570,6 +2567,30 @@ var AddArtifactInput = z3.discriminatedUnion("source", [
     content: z3.string().describe("Base64 encoded file content")
   })
 ]);
+function validateArtifactType(type, filename) {
+  const ext = filename.split(".").pop()?.toLowerCase();
+  const validExtensions = {
+    html: ["html", "htm"],
+    image: ["png", "jpg", "jpeg", "gif", "webp", "svg"],
+    video: ["mp4", "webm", "mov", "avi"]
+  };
+  const valid = validExtensions[type];
+  if (!valid || !ext || !valid.includes(ext)) {
+    const suggestions = {
+      html: "HTML is the primary format for test results, terminal output, code reviews, and structured data. Use self-contained HTML with inline CSS and base64 images.",
+      image: 'Images are for actual UI screenshots only. For terminal output or test results, use type: "html" instead.',
+      video: 'Videos are for browser automation flows and complex interactions. For static content, use type: "image" or "html".'
+    };
+    throw new Error(
+      `Invalid file extension for artifact type '${type}'.
+
+Expected: ${valid?.join(", ") || "unknown"}
+Got: ${ext || "no extension"}
+
+Tip: ${suggestions[type]}`
+    );
+  }
+}
 var addArtifactTool = {
   definition: {
     name: TOOL_NAMES.ADD_ARTIFACT,
@@ -2613,7 +2634,7 @@ ARTIFACT TYPES:
         sessionToken: { type: "string", description: "Session token from create_plan" },
         type: {
           type: "string",
-          enum: ["screenshot", "video", "test_results", "diff"],
+          enum: ["html", "image", "video"],
           description: "Artifact type for rendering"
         },
         filename: {
@@ -2653,6 +2674,7 @@ ARTIFACT TYPES:
   handler: async (args) => {
     const input = AddArtifactInput.parse(args);
     const { planId, sessionToken, type, filename } = input;
+    validateArtifactType(type, filename);
     const actorName = await getGitHubUsername();
     logger.info({ planId, type, filename }, "Adding artifact");
     let content;
@@ -3630,7 +3652,7 @@ Bad deliverables (not provable):
       deleted: false
     });
     logger.info({ planId }, "Plan index updated");
-    const url = `${webConfig.SHIPYARD_WEB_URL}/plan/${planId}`;
+    const url = createPlanWebUrl(webConfig.SHIPYARD_WEB_URL, planId);
     await openPlanInBrowser(planId, url);
     const repoInfo = repo ? `Repo: ${repo}${!input.repo ? " (auto-detected)" : ""}` : "Repo: Not set (provide repo and prNumber for artifact uploads)";
     return {
@@ -4966,7 +4988,7 @@ Upload proof-of-work artifact.
 Parameters:
 - planId (string): The plan ID
 - sessionToken (string): Session token
-- type (string): 'screenshot' | 'video' | 'test_results' | 'diff'
+- type (string): 'html' | 'image' | 'video'
 - filename (string): e.g., "screenshot.png"
 - source (string): Content source type - 'file' | 'url' | 'base64'
 - filePath (string): Local file path (required when source='file') - RECOMMENDED
@@ -5386,7 +5408,7 @@ async function setupReviewNotification(planId, pollIntervalSeconds) {
   return { script, fullResponse: text };
 }
 async function requestUserInput(opts) {
-  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-3STPPPYU.js");
+  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-YOBSG7RN.js");
   const ydoc = await getOrCreateDoc3(PLAN_INDEX_DOC_NAME);
   const manager = new InputRequestManager2();
   const params = opts.type === "choice" ? {
@@ -5430,7 +5452,7 @@ async function requestUserInput(opts) {
   };
 }
 async function postActivityUpdate(opts) {
-  const { logPlanEvent: logPlanEvent2 } = await import("./dist-D4DG2R4R.js");
+  const { logPlanEvent: logPlanEvent2 } = await import("./dist-HCVQNHBC.js");
   const { getGitHubUsername: getGitHubUsername2 } = await import("./server-identity-LSZ4CZRK.js");
   const { nanoid: nanoid8 } = await import("nanoid");
   const doc = await getOrCreateDoc3(opts.planId);
@@ -5453,7 +5475,7 @@ async function postActivityUpdate(opts) {
   return { success: true, eventId, requestId };
 }
 async function resolveActivityRequest(opts) {
-  const { logPlanEvent: logPlanEvent2, getPlanEvents } = await import("./dist-D4DG2R4R.js");
+  const { logPlanEvent: logPlanEvent2, getPlanEvents } = await import("./dist-HCVQNHBC.js");
   const { getGitHubUsername: getGitHubUsername2 } = await import("./server-identity-LSZ4CZRK.js");
   const doc = await getOrCreateDoc3(opts.planId);
   const actorName = await getGitHubUsername2();

package/apps/server/dist/{input-request-manager-3STPPPYU.js → input-request-manager-YOBSG7RN.js}

@@ -1,7 +1,7 @@
 import {
   InputRequestManager
-} from "./chunk-NH6GXGQO.js";
-import "./chunk-BBYZISY2.js";
+} from "./chunk-7ISZ4RKB.js";
+import "./chunk-CYHEHTLS.js";
 import "./chunk-GSGLHRWX.js";
 import "./chunk-JSBRDJBE.js";
 export {

package/package.json

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