@vornrun/mcp 0.7.3 → 0.7.5

package/dist/index.js

@@ -589,6 +589,7 @@ function createSchema() {
     CREATE INDEX IF NOT EXISTS idx_connector_inbox_connection
       ON connector_inbox(connection_id, created_at);
   `);
+  d.exec(ARTIFACT_DDL);
   migrateSchema(d);
   verifySchema(d);
   seedLegacyConnectorPollState(d);
@@ -1135,7 +1136,59 @@ function migrateSchema(d) {
     })();
     logger_default.info("[database] migrated schema to version 24 (the text a reviewer edited at a gate)");
   }
+  if (version < 25) {
+    d.transaction(() => {
+      d.exec(ARTIFACT_DDL);
+      d.prepare(
+        "INSERT OR REPLACE INTO schema_meta (key, value) VALUES ('schema_version', '25')"
+      ).run();
+    })();
+    logger_default.info("[database] migrated schema to version 25 (published artifacts and their comments)");
+  }
 }
+var ARTIFACT_DDL = `
+  CREATE TABLE IF NOT EXISTS artifacts (
+    id TEXT PRIMARY KEY,
+    kind TEXT NOT NULL,
+    title TEXT NOT NULL,
+    session_id TEXT,
+    project_name TEXT,
+    token TEXT NOT NULL,
+    latest_version INTEGER NOT NULL DEFAULT 0,
+    gate_run_id TEXT,
+    gate_node_id TEXT,
+    created_at TEXT NOT NULL,
+    updated_at TEXT NOT NULL
+  );
+
+  CREATE INDEX IF NOT EXISTS idx_artifacts_session ON artifacts(session_id, updated_at DESC);
+  CREATE INDEX IF NOT EXISTS idx_artifacts_project ON artifacts(project_name, updated_at DESC);
+
+  CREATE TABLE IF NOT EXISTS artifact_versions (
+    artifact_id TEXT NOT NULL REFERENCES artifacts(id) ON DELETE CASCADE,
+    version INTEGER NOT NULL,
+    author TEXT NOT NULL,
+    answers_batch_id TEXT,
+    created_at TEXT NOT NULL,
+    PRIMARY KEY (artifact_id, version)
+  );
+
+  CREATE TABLE IF NOT EXISTS artifact_comments (
+    id TEXT PRIMARY KEY,
+    artifact_id TEXT NOT NULL REFERENCES artifacts(id) ON DELETE CASCADE,
+    version INTEGER NOT NULL,
+    anchor TEXT,
+    body TEXT NOT NULL,
+    state TEXT NOT NULL DEFAULT 'draft',
+    batch_id TEXT,
+    created_at TEXT NOT NULL,
+    updated_at TEXT NOT NULL,
+    sent_at TEXT
+  );
+
+  CREATE INDEX IF NOT EXISTS idx_artifact_comments_artifact
+    ON artifact_comments(artifact_id, created_at);
+`;
 var GATE_COLUMNS = [
   ["message", "TEXT"],
   ["view_token", "TEXT"],
@@ -1258,7 +1311,7 @@ function verifySchema(d) {
       },
       // Which pass of a loop produced this row.
       { column: "iteration", ddl: "ALTER TABLE workflow_run_nodes ADD COLUMN iteration INTEGER" },
-      ...GATE_COLUMNS.map(([column, type]) => ({
+      ...[...GATE_COLUMNS, ...GATE_EDIT_COLUMNS].map(([column, type]) => ({
         column,
         ddl: `ALTER TABLE workflow_run_nodes ADD COLUMN ${column} ${type}`
       }))
@@ -4215,9 +4268,18 @@ The run is stopped by the instance holding it, so confirm with list_workflow_run
       edited: z6.string().max(2e5).optional().describe(
         "Your rewrite of the gate's editable text, when it has one. Kept beside the original, and read by later steps as {{steps.<gate>.text}}. Ignored on reject."
       ),
+      comments: z6.array(
+        z6.object({
+          quote: z6.string().max(2e3).optional().describe("The words on the review page it is about"),
+          comment: z6.string().min(1).max(4e3)
+        })
+      ).max(100).optional().describe(
+        "Comments on the review page, each on the words it quotes. Carried back with changes and read by the redone steps as {{steps.<gate>.comments}}; ignored on approve and reject."
+      ),
       node_id: V.id.optional().describe("The waiting node, when a run has more than one gate open")
     },
     async (args) => {
+      const comments = (args.comments ?? []).filter((c) => c.comment.trim());
       const run = await runById(args.run_id);
       if (!run) {
         return {
@@ -4247,7 +4309,7 @@ The run is stopped by the instance holding it, so confirm with list_workflow_run
           isError: true
         };
       }
-      if (args.decision === "changes" && !args.comment?.trim()) {
+      if (args.decision === "changes" && !args.comment?.trim() && comments.length === 0) {
         return {
           content: [
             { type: "text", text: "Error: changes needs a comment saying what to change." }
@@ -4266,7 +4328,8 @@ The run is stopped by the instance holding it, so confirm with list_workflow_run
             nodeId: target.nodeId,
             decision: args.decision,
             ...args.comment?.trim() && { comment: args.comment.trim() },
-            ...args.edited?.trim() && { edited: args.edited.trim() }
+            ...args.edited?.trim() && { edited: args.edited.trim() },
+            ...args.decision === "changes" && comments.length > 0 && { comments }
           }
         );
         if (answer?.accepted === false) {
@@ -4610,8 +4673,12 @@ var DOCS = {
         field: "items",
         description: "When the editable text is a JSON list of records: the rows the reviewer kept, as a list, ready for a forEach loop"
       },
-      { field: "feedback", description: "The reviewer's latest comment" },
-      { field: "feedbackAll", description: "Every comment, one line per round" },
+      {
+        field: "comments",
+        description: "Comments pinned to the review page in the latest round, as a JSON list of {quote, comment, anchored}"
+      },
+      { field: "feedback", description: "The general note from the latest round" },
+      { field: "feedbackAll", description: "Every note, one line per round" },
       { field: "round", description: "Which time the gate asked, from 1" },
       ...COMMON_OUTPUTS
     ],
@@ -5648,9 +5715,11 @@ function registerDeviceTools(server) {
   );
   server.tool(
     "device_interact",
-    'Act on your claimed simulator: tap, swipe, type, press a hardware button, or long-press. Address the target by "ref" from read_screen where possible. Coordinates are in POINTS. A ref from before an earlier interaction is refused rather than guessed at \u2014 read the screen again. A swipe starting at the very edge of the screen is refused too: iOS claims those as system gestures and swallows them, which looks to you like nothing happened.',
+    'Act on your claimed simulator: tap, swipe, type, press a hardware button, long-press, or rotate it. Address the target by "ref" from read_screen where possible. Coordinates are in POINTS. A ref from before an earlier interaction is refused rather than guessed at \u2014 read the screen again. A swipe starting at the very edge of the screen is refused too: iOS claims those as system gestures and swallows them, which looks to you like nothing happened.',
     {
-      action: z11.enum(["tap", "swipe", "type", "button", "press"]).describe('"press" is a long press; "button" takes a name in `text`'),
+      action: z11.enum(["tap", "swipe", "type", "button", "press", "rotate"]).describe(
+        '"press" is a long press; "button" takes a name in `text`; "rotate" takes `orientation`'
+      ),
       ref: required("Element ref from read_screen or device_find").optional(),
       x: z11.number().optional().describe("Screen x in points, when no ref is available"),
       y: z11.number().optional().describe("Screen y in points, when no ref is available"),
@@ -5659,6 +5728,7 @@ function registerDeviceTools(server) {
       text: V.description.optional().describe(
         'Text for "type", or the button name for "button" (HOME, LOCK, SIRI, SIDE_BUTTON)'
       ),
+      orientation: z11.enum(["portrait", "portrait-upside-down", "landscape-left", "landscape-right"]).optional().describe('Which way up to turn the device, for "rotate"'),
       duration: z11.number().min(0).max(30).optional().describe('Seconds to hold, for "press"'),
       system_gesture: z11.boolean().optional().describe("Allow a stroke starting in the bezel band, when a system gesture is the intent")
     },
@@ -5669,6 +5739,7 @@ function registerDeviceTools(server) {
         target: toDeviceTarget(args),
         to: typeof args.to_x === "number" && typeof args.to_y === "number" ? { x: args.to_x, y: args.to_y } : void 0,
         text: args.text,
+        orientation: args.orientation,
         duration: args.duration,
         systemGesture: args.system_gesture
       });
@@ -5763,6 +5834,98 @@ Screen is ${r.screen.width}x${r.screen.height} points. Image pixels are ${r.scal
   );
 }
 
+// src/tools/artifacts.ts
+import { z as z12 } from "zod";
+var MAX_CONTENT = 5 * 1024 * 1024;
+var text = (value) => ({ content: [{ type: "text", text: value }] });
+function registerArtifactTools(server) {
+  server.tool(
+    "publish_artifact",
+    "Publish a page, doc or design for the person to review. It opens in your session browser pane, where they can comment on the exact words and send their notes back to you as one message. Pass artifactId to publish the next version of an artifact after addressing comments. A page or design is self-contained HTML (no network: inline styles, scripts and images as data: URIs); a doc is Markdown.",
+    {
+      kind: z12.enum(["page", "doc", "design"]).describe("page: any HTML; doc: Markdown; design: HTML with a design manifest"),
+      title: V.shortText.min(1).describe("What the artifact is called in the pane"),
+      file: z12.string().min(1).max(1e3).optional().describe("A .html or .md file inside your project or worktree; give this or content"),
+      content: z12.string().max(MAX_CONTENT).optional().describe("The HTML or Markdown itself"),
+      artifactId: V.id.optional().describe("Publish the next version of this artifact"),
+      open: z12.boolean().optional().describe("Open it in the browser pane (default true)")
+    },
+    async (args) => withSession(async (id) => {
+      const result = await rpcCall("artifact:publish", { sessionId: id, ...args });
+      const lines = [
+        `Published ${result.artifact.kind} "${result.artifact.title}" v${result.version.version} (artifactId ${result.artifact.id}).`,
+        result.opened ? "It is open in your browser pane." : `Not opened in a pane. Served at ${result.url}`
+      ];
+      if (result.answered > 0) lines.push(`This version answers ${result.answered} comments.`);
+      return text(lines.join("\n"));
+    })
+  );
+  server.tool(
+    "list_artifacts",
+    "List the artifacts published by your session or in your project, newest first.",
+    { limit: z12.number().int().min(1).max(100).optional() },
+    async (args) => withSession(async (id) => {
+      const artifacts = await rpcCall("artifact:list", {
+        sessionId: id,
+        limit: args.limit
+      });
+      if (artifacts.length === 0) return text("No artifacts yet.");
+      return text(
+        artifacts.map(
+          (a) => `${a.id}  ${a.kind}  v${a.latestVersion}  "${a.title}"  updated ${a.updatedAt}`
+        ).join("\n")
+      );
+    })
+  );
+  server.tool(
+    "read_artifact_comments",
+    "Read the comments the person left on an artifact: the quoted words or point each one is about, the comment, which version it was written on, and whether it was sent to you.",
+    {
+      artifactId: V.id,
+      version: z12.number().int().min(1).optional().describe("Only comments written on this version")
+    },
+    async (args) => withSession(async (id) => {
+      const comments = await rpcCall("artifact:readComments", {
+        sessionId: id,
+        artifactId: args.artifactId,
+        version: args.version
+      });
+      return pageResult(
+        comments.map((c) => ({
+          version: c.version,
+          state: c.state,
+          anchor: c.anchor,
+          comment: c.body
+        })),
+        "ARTIFACT COMMENTS ON WEB PAGE CONTENT"
+      );
+    })
+  );
+  server.tool(
+    "read_artifact",
+    "Read the source of an artifact version: Markdown for a doc, HTML for a page or design. Use it before publishing the next version when the person saved edits of their own.",
+    {
+      artifactId: V.id,
+      version: z12.number().int().min(1).optional().describe("Defaults to the latest version")
+    },
+    async (args) => withSession(async (id) => {
+      const found = await rpcCall(
+        "artifact:readSource",
+        { sessionId: id, artifactId: args.artifactId, version: args.version }
+      );
+      if (!found) return text(`No such version of artifact ${args.artifactId}.`);
+      return pageResult(
+        {
+          version: found.version.version,
+          author: found.version.author === "user" ? "the person" : "an agent",
+          source: found.body
+        },
+        "WEB PAGE CONTENT: ARTIFACT SOURCE"
+      );
+    })
+  );
+}
+
 // src/server.ts
 function createMcpServer(version) {
   const server = new McpServer({ name: "vorn", version }, { capabilities: { tools: {} } });
@@ -5771,6 +5934,7 @@ function createMcpServer(version) {
   registerConnectorTools(server);
   registerBrowserTools(server);
   registerDeviceTools(server);
+  registerArtifactTools(server);
   registerProjectTools(server);
   registerTaskTools(server);
   registerWorkflowTools(server);
@@ -5788,7 +5952,7 @@ console.warn = (...args) => _origError("[mcp:warn]", ...args);
 console.error = (...args) => _origError("[mcp:error]", ...args);
 async function main() {
   configManager.init();
-  const version = true ? "0.7.3" : createRequire(import.meta.url)("../package.json").version;
+  const version = true ? "0.7.5" : createRequire(import.meta.url)("../package.json").version;
   const server = createMcpServer(version);
   const transport = new StdioServerTransport();
   await server.connect(transport);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vornrun/mcp",
-  "version": "0.7.3",
+  "version": "0.7.5",
   "description": "Vorn MCP server — task management, git, and workflow tools for AI coding agents",
   "type": "module",
   "license": "MIT",
@@ -38,8 +38,8 @@
     "zod": "^4.5.4"
   },
   "devDependencies": {
-    "@vornrun/server": "0.7.3",
-    "@vornrun/shared": "0.7.3",
+    "@vornrun/server": "0.7.5",
+    "@vornrun/shared": "0.7.5",
     "tsup": "^8.5.1",
     "tsx": "^4.23.1",
     "typescript": "^6.0.3"