@fabiofiorita/porcelain 0.66.0 → 0.67.0

package/README.md

@@ -1,4 +1,4 @@
-# @fabiofiorita/porcelain (0.66.0)
+# @fabiofiorita/porcelain (0.67.0)
 
 The Porcelain review server and bundled web app for a plain Node installation.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fabiofiorita/porcelain",
-  "version": "0.66.0",
+  "version": "0.67.0",
   "description": "Review coding agents’ changes from a browser, Mac or phone",
   "license": "MIT",
   "repository": {

package/server/src/cli/main.mjs

@@ -2986,7 +2986,7 @@ import { NodeRuntime } from "@effect/platform-node";
 import {
   Cause as Cause3,
   ConfigProvider,
-  Console,
+  Console as Console2,
   Context as Context75,
   Effect as Effect58,
   Exit as Exit3,
@@ -7525,7 +7525,7 @@ var readCliVersion = Effect26.fn("Cli.readVersion")(function* (packageRoot) {
 });
 
 // apps/server/src/cli/cli-program.ts
-import { Effect as Effect57, Ref as Ref3, Result as Result6 } from "effect";
+import { Console, Effect as Effect57, Ref as Ref3, Result as Result6 } from "effect";
 import { CliError, Command as Command2 } from "effect/cli";
 
 // apps/server/src/cli/command-tree.ts
@@ -9385,6 +9385,9 @@ var openServiceUpdateRunner = Effect44.fn("openServiceUpdateRunner")(
         return answer2 === "active" || answer2 === "activating";
       }
     );
+    const updateRunning = Effect44.fn("ServiceUpdate.updateRunning")(function* (managed) {
+      return preparing || managed && ((yield* directoryLockIsHeld(`${paths.updater}.lock`)) || (yield* updaterActive()));
+    });
     const latestVersion = Effect44.fn("ServiceUpdate.latestVersion")(function* (input, running) {
       const known = latest;
       if (known !== void 0 && (running || known.checkedAt >= input.staleBefore))
@@ -9432,18 +9435,17 @@ var openServiceUpdateRunner = Effect44.fn("openServiceUpdateRunner")(
         function* (input) {
           const version = yield* runningVersion();
           const managed = version !== void 0 && options.packageRoot === pathApi.join(paths.runtime, "node_modules", PACKAGE_NAME) && (yield* exists(paths.installed));
-          const running = preparing || managed && ((yield* directoryLockIsHeld(`${paths.updater}.lock`)) || (yield* updaterActive()));
-          const published = managed ? yield* latestVersion(input, running) : void 0;
+          const initiallyRunning = yield* updateRunning(managed);
+          const published = managed ? yield* latestVersion(input, initiallyRunning) : void 0;
+          const record = yield* readUpdateRecord(paths.updateRecord);
+          const running = initiallyRunning || record !== void 0 && (yield* updateRunning(managed));
           return {
             managed,
             version,
             latest: published,
             available: published !== void 0 && version !== void 0 && (yield* compareVersions(published, version)) > 0,
             running,
-            last: presentedUpdate(
-              yield* readUpdateRecord(paths.updateRecord),
-              running
-            )
+            last: presentedUpdate(record, running)
           };
         },
         provideFiles,
@@ -10874,6 +10876,19 @@ var mcp = Command.make(
 ).pipe(
   Command.withDescription("Serve MCP over the local owner socket for an agent")
 );
+var desktopMcpCommand = Command.make(
+  "porcelain-mcp",
+  {
+    dataDirectory: Flag.String("data-directory").pipe(
+      Flag.withSchema(absolutePathSchema2),
+      Flag.withDescription("Server data directory of the running desktop app")
+    )
+  },
+  Effect56.fn("Cli.desktopMcpCommand")(function* (input) {
+    const operations2 = yield* CliOperations;
+    yield* operations2.mcp(input);
+  })
+).pipe(Command.withDescription("Connect an agent to the running desktop app"));
 var shareHandler = Effect56.fn("Cli.shareCommand")(function* (action) {
   const operations2 = yield* CliOperations;
   yield* finish(
@@ -11011,9 +11026,20 @@ var porcelainCommand = rootConfiguration.pipe(
 );
 
 // apps/server/src/cli/cli-program.ts
-var cliProgram = Effect57.fn("Cli.run")(function* (args, version) {
+var cliProgram = Effect57.fn("Cli.run")(function* (args, version, mode = "server") {
   const exitCode = yield* Ref3.make(0);
-  yield* Command2.runWith(porcelainCommand, { version })(args).pipe(
+  const program = mode === "desktop-mcp" ? Command2.runWith(desktopMcpCommand, { version })(args) : Command2.runWith(porcelainCommand, { version })(args);
+  const console2 = yield* Console.Console;
+  const presentation = new Proxy(console2, {
+    get(target, property, receiver) {
+      if (mode === "desktop-mcp" && property === "log")
+        return (...values) => target.error(...values);
+      const value2 = Reflect.get(target, property, receiver);
+      return value2;
+    }
+  });
+  yield* program.pipe(
+    Effect57.provideService(Console.Console, presentation),
     Effect57.provideService(CliExit, exitCode),
     Effect57.provideService(CliVersion, version),
     Effect57.catchFilter(
@@ -11088,11 +11114,17 @@ function createCliRunner(runtime, platform) {
     const program = Effect58.gen(function* () {
       const configured = yield* CliRuntime;
       return yield* Effect58.gen(function* () {
+        if (options.desktopMcpVersion !== void 0)
+          return yield* cliProgram(
+            args,
+            options.desktopMcpVersion,
+            "desktop-mcp"
+          );
         const packageRoot = yield* cliPackageRoot();
         const version = yield* readCliVersion(packageRoot).pipe(Effect58.orDie);
         return yield* cliProgram(args, version);
       }).pipe(
-        Effect58.provideService(Console.Console, output2),
+        Effect58.provideService(Console2.Console, output2),
         Effect58.provide(services),
         Effect58.catchCause(
           (cause) => Cause3.hasInterruptsOnly(cause) ? Effect58.interrupt : Effect58.sync(() => {
@@ -15853,6 +15885,20 @@ var events = channel("porcelain.http");
 function httpEvent(event) {
   events.publish(event);
 }
+function toolAudit(tool) {
+  return Effect152.flatMap(
+    Effect152.serviceOption(RequestContext),
+    (context) => Effect152.sync(() => {
+      if (context._tag === "Some" && context.value.principal?.kind === "owner")
+        httpEvent({
+          event: "tool-handled",
+          owner: true,
+          id: context.value.requestId,
+          tool
+        });
+    })
+  );
+}
 var handlerAudit = HttpRouter3.middleware(
   (app) => Effect152.flatMap(Effect152.serviceOption(RequestContext), (context) => {
     if (context._tag === "Some")
@@ -20863,7 +20909,7 @@ var agent = { kind: "agent" };
 var summaryWarnings = {
   "missing-style": "No authored CSS was detected in the summary HTML. The review was published. Add CSS and republish, matching the reviewed application\u2019s colors, background, typography and components where possible. Style the layer links and content hierarchy, then visually verify the result. If styles are generated at runtime, verify that they load correctly."
 };
-function invocation(operation) {
+function invocation(tool, operation) {
   return Effect228.gen(function* () {
     const request2 = yield* Effect228.serviceOption(
       HttpServerRequest2.HttpServerRequest
@@ -20876,6 +20922,7 @@ function invocation(operation) {
           message: "The x-porcelain-cwd header is required"
         })
       );
+    yield* toolAudit(tool);
     return yield* operation(cwd);
   }).pipe(
     Effect228.mapError(
@@ -20890,9 +20937,16 @@ function invocation(operation) {
 function reviewMcpHandlers(useCases) {
   return Layer152.mergeAll(
     McpServer.toolkit(ReviewToolkit).pipe(
+      Layer152.tap(
+        () => Effect228.sync(() => {
+          for (const tool of Object.keys(ReviewToolkit.tools))
+            httpEvent({ event: "tool-registered", owner: true, tool });
+        })
+      ),
       Layer152.provide(
         ReviewToolkit.toLayer({
           publish_review: ({ cwd, ...review }) => invocation(
+            "publish_review",
             (defaultCwd) => Effect228.gen(function* () {
               const published = yield* useCases.reviewTools.atWorktreePath.execute({
                 operation: useCases.reviewTools.publishReview,
@@ -20908,6 +20962,7 @@ function reviewMcpHandlers(useCases) {
             })
           ),
           delete_review: ({ cwd }) => invocation(
+            "delete_review",
             (defaultCwd) => useCases.reviewTools.atWorktreePath.execute({
               operation: useCases.reviewTools.deleteReview,
               cwd: cwd ?? defaultCwd,
@@ -20915,6 +20970,7 @@ function reviewMcpHandlers(useCases) {
             })
           ),
           read_review: ({ cwd }) => invocation(
+            "read_review",
             (defaultCwd) => useCases.reviewTools.atWorktreePath.execute({
               operation: useCases.reviewTools.readPublishedReview,
               cwd: cwd ?? defaultCwd,
@@ -20922,6 +20978,7 @@ function reviewMcpHandlers(useCases) {
             })
           ),
           list_comments: ({ cwd, scope }) => invocation(
+            "list_comments",
             (defaultCwd) => Effect228.map(
               useCases.reviewTools.atWorktreePath.execute({
                 operation: useCases.reviewTools.listCommentThreads,
@@ -20932,6 +20989,7 @@ function reviewMcpHandlers(useCases) {
             )
           ),
           create_comment: ({ cwd, ...input }) => invocation(
+            "create_comment",
             (defaultCwd) => useCases.reviewTools.atWorktreePath.execute({
               operation: useCases.reviewTools.createCommentThread,
               cwd: cwd ?? defaultCwd,
@@ -20939,6 +20997,7 @@ function reviewMcpHandlers(useCases) {
             })
           ),
           reply_to_comment: ({ cwd, ...input }) => invocation(
+            "reply_to_comment",
             (defaultCwd) => useCases.reviewTools.atWorktreePath.execute({
               operation: useCases.reviewTools.replyToComment,
               cwd: cwd ?? defaultCwd,
@@ -20946,6 +21005,7 @@ function reviewMcpHandlers(useCases) {
             })
           ),
           resolve_comment: ({ cwd, ...input }) => invocation(
+            "resolve_comment",
             (defaultCwd) => useCases.reviewTools.atWorktreePath.execute({
               operation: useCases.reviewTools.updateCommentThread,
               cwd: cwd ?? defaultCwd,

package/web/dist/assets/_paired-Cf9W7w-r.js

@@ -0,0 +1 @@
+import{t as e}from"./workspace-error-DZaV6KWv.js";var t=e;export{t as errorComponent};

package/web/dist/assets/_paired-DV6Uu96D.js

@@ -0,0 +1 @@
+import{r as e}from"./rolldown-runtime-hePW80VL.js";import{hi as t,li as n,mi as r}from"./files-De0CgA0d.js";import{Ct as i}from"./access-DKodjctH.js";import{r as a}from"./link-BtPdbBer.js";import{t as o}from"./useRouterState-B1-fsbiG.js";import{B as s,I as c,V as l,ln as u}from"./settings-page-BdwyGbHc.js";import{a as d,t as f}from"./desktop-actions-xMY_-vix.js";import{o as p}from"./index-DL7Akkem.js";var m=e(t(),1),h=n(),g=r();function _(e){let t=(0,h.c)(3),{children:n}=e,r;t[0]===Symbol.for(`react.memo_cache_sentinel`)?(r=(0,g.jsx)(u,{}),t[0]=r):r=t[0];let i;return t[1]===n?i=t[2]:(i=(0,g.jsxs)(g.Fragment,{children:[r,(0,g.jsx)(d,{children:n})]}),t[1]=n,t[2]=i),i}function v(e){let t=(0,h.c)(6),{children:n}=e,[r,a]=(0,m.useState)(!0),o=i(),u;t[0]===o?u=t[1]:(u=o&&(0,g.jsxs)(g.Fragment,{children:[(0,g.jsx)(s,{connection:o}),(0,g.jsx)(l,{connection:o})]}),t[0]=o,t[1]=u);let d;return t[2]!==n||t[3]!==r||t[4]!==u?(d=(0,g.jsx)(_,{children:(0,g.jsxs)(c,{open:r,onOpenChange:a,children:[n,u]})}),t[2]=n,t[3]=r,t[4]=u,t[5]=d):d=t[5],d}function y(){let e=(0,h.c)(4),t=i(),n;e[0]===Symbol.for(`react.memo_cache_sentinel`)?(n={select:b},e[0]=n):n=e[0];let r=o(n),s;return e[1]!==t||e[2]!==r?(s=t?(0,g.jsxs)(v,{children:[(0,g.jsx)(p,{}),(0,g.jsx)(f,{connection:t,empty:r})]}):(0,g.jsx)(a,{to:`/pair`,replace:!0}),e[1]=t,e[2]=r,e[3]=s):s=e[3],s}function b(e){return e.location.pathname===`/`}export{y as component};

package/web/dist/assets/_paired-DdiJtBi7.js

@@ -0,0 +1 @@
+import{li as e,mi as t}from"./files-De0CgA0d.js";import{St as n}from"./access-DKodjctH.js";import{r}from"./link-BtPdbBer.js";import{t as i}from"./useRouterState-B1-fsbiG.js";import{Et as a,Tt as o,xt as s}from"./settings-page-BdwyGbHc.js";import{i as c}from"./index-DL7Akkem.js";import{t as l}from"./connected-workspace-B0v2QBMa.js";var u=e(),d=t();function f(){let e=(0,u.c)(18),t=c.useSearch(),f,m;e[0]===t?(f=e[1],m=e[2]):({worktree:m,...f}=t,e[0]=t,e[1]=f,e[2]=m);let h;e[3]===Symbol.for(`react.memo_cache_sentinel`)?(h={select:p},e[3]=h):h=e[3];let g=i(h),{connection:_}=n(),v=s(_),y;e[4]!==v||e[5]!==m?(y=m===void 0?a(v,o(v)?.id):a(v,m),e[4]=v,e[5]=m,e[6]=y):y=e[6];let b=y;if(b&&g){let t;e[7]!==b.projectId||e[8]!==b.worktree.id?(t={projectId:b.projectId,worktreeId:b.worktree.id},e[7]=b.projectId,e[8]=b.worktree.id,e[9]=t):t=e[9];let n;e[10]!==f||e[11]!==m?(n=m===void 0?{}:f,e[10]=f,e[11]=m,e[12]=n):n=e[12];let i;return e[13]!==t||e[14]!==n?(i=(0,d.jsx)(r,{to:`/$projectId/$worktreeId`,params:t,search:n,replace:!0}),e[13]=t,e[14]=n,e[15]=i):i=e[15],i}let x=m!==void 0&&g,S;return e[16]===x?S=e[17]:(S=(0,d.jsx)(l,{missing:x}),e[16]=x,e[17]=S),S}function p(e){return e.status===`idle`&&e.location.pathname===`/`}export{f as component};

package/web/dist/assets/_paired-DtZr8WeA.js

@@ -0,0 +1 @@
+import{t as e}from"./workspace-error-DZaV6KWv.js";var t=e;export{t as errorComponent};

package/web/dist/assets/_section-BhA0KAGA.js

@@ -0,0 +1 @@
+import{li as e,mi as t}from"./files-De0CgA0d.js";import{t as n}from"./settings-page-BdwyGbHc.js";import{n as r}from"./index-DL7Akkem.js";var i=e(),a=t();function o(){let e=(0,i.c)(2),{section:t}=r.useParams(),o;return e[0]===t?o=e[1]:(o=(0,a.jsx)(n,{section:t}),e[0]=t,e[1]=o),o}export{o as component};

package/web/dist/assets/_worktreeId-Bn1qBq2T.js

@@ -0,0 +1 @@
+import{li as e,mi as t}from"./files-De0CgA0d.js";import{wt as n}from"./access-DKodjctH.js";import{r}from"./link-BtPdbBer.js";import"./settings-page-BdwyGbHc.js";import"./workspace-pending-BUZzjlSx.js";import{t as i}from"./index-DL7Akkem.js";import"./connected-workspace-B0v2QBMa.js";var a=e(),o=t();function s(){let e=(0,a.c)(9),{environmentId:t,projectId:s,worktreeId:c}=i.useParams(),l=i.useSearch();n(t);{let t;return e[0]!==l||e[1]!==c?(t=(0,o.jsx)(r,{to:`/`,search:{...l,worktree:c},replace:!0}),e[0]=l,e[1]=c,e[2]=t):t=e[2],t}}export{s as component};

package/web/dist/assets/_worktreeId-Cf9W7w-r.js

@@ -0,0 +1 @@
+import{t as e}from"./workspace-error-DZaV6KWv.js";var t=e;export{t as errorComponent};

package/web/dist/assets/_worktreeId-D0bWoqgW.js

@@ -0,0 +1 @@
+import{li as e,mi as t}from"./files-De0CgA0d.js";import{St as n}from"./access-DKodjctH.js";import{r}from"./link-BtPdbBer.js";import{Et as i,xt as a}from"./settings-page-BdwyGbHc.js";import{r as o}from"./index-DL7Akkem.js";import{t as s}from"./connected-workspace-B0v2QBMa.js";var c=e(),l=t();function u(){let e=(0,c.c)(10),{projectId:t,worktreeId:u}=o.useParams(),d=o.useSearch(),{connection:f}=n(),p=a(f),m;e[0]!==p||e[1]!==u?(m=i(p,u),e[0]=p,e[1]=u,e[2]=m):m=e[2];let h=m;if(h?.projectId!==t){let t;return e[3]!==d||e[4]!==u?(t=(0,l.jsx)(r,{to:`/`,search:{...d,worktree:u},replace:!0}),e[3]=d,e[4]=u,e[5]=t):t=e[5],t}let g;return e[6]!==p||e[7]!==d||e[8]!==h?(g=(0,l.jsx)(s,{review:{selection:h,search:d,inventory:p}}),e[6]=p,e[7]=d,e[8]=h,e[9]=g):g=e[9],g}export{u as component};

package/web/dist/assets/_worktreeId-DtZr8WeA.js

@@ -0,0 +1 @@
+import{t as e}from"./workspace-error-DZaV6KWv.js";var t=e;export{t as errorComponent};