@schoolai/shipyard-mcp 0.2.2-next.487 → 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(),

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

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

package/apps/server/dist/{chunk-UFE5KX7E.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(),

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

@@ -206,7 +206,7 @@ import {
   updateLinkedPRStatus,
   updatePlanIndexViewedBy,
   validateA2AMessages
-} from "./chunk-UFE5KX7E.js";
+} from "./chunk-CYHEHTLS.js";
 import "./chunk-JSBRDJBE.js";
 export {
   A2ADataPartSchema,

package/apps/server/dist/index.js

@@ -20,7 +20,7 @@ import {
 } from "./chunk-EBNL5ZX7.js";
 import {
   InputRequestManager
-} from "./chunk-B4TQH7Q3.js";
+} from "./chunk-7ISZ4RKB.js";
 import {
   ArtifactSchema,
   DeliverableSchema,
@@ -72,7 +72,7 @@ import {
   setPlanMetadata,
   touchPlanIndexEntry,
   transitionPlanStatus
-} from "./chunk-UFE5KX7E.js";
+} from "./chunk-CYHEHTLS.js";
 import {
   loadEnv,
   logger
@@ -2548,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")
@@ -2567,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,
@@ -2610,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: {
@@ -2650,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;
@@ -4963,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
@@ -5383,7 +5408,7 @@ async function setupReviewNotification(planId, pollIntervalSeconds) {
   return { script, fullResponse: text };
 }
 async function requestUserInput(opts) {
-  const { InputRequestManager: InputRequestManager2 } = await import("./input-request-manager-IMVZDMUQ.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" ? {
@@ -5427,7 +5452,7 @@ async function requestUserInput(opts) {
   };
 }
 async function postActivityUpdate(opts) {
-  const { logPlanEvent: logPlanEvent2 } = await import("./dist-E4CPV3SO.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);
@@ -5450,7 +5475,7 @@ async function postActivityUpdate(opts) {
   return { success: true, eventId, requestId };
 }
 async function resolveActivityRequest(opts) {
-  const { logPlanEvent: logPlanEvent2, getPlanEvents } = await import("./dist-E4CPV3SO.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-IMVZDMUQ.js → input-request-manager-YOBSG7RN.js}

@@ -1,7 +1,7 @@
 import {
   InputRequestManager
-} from "./chunk-B4TQH7Q3.js";
-import "./chunk-UFE5KX7E.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.487",
+  "version": "0.2.2-next.489",
   "description": "Shipyard MCP server and CLI tools for distributed planning with CRDTs",
   "type": "module",
   "bin": {