opendoor-plugin 0.2.0 → 0.5.0

package/dist/package.json

@@ -1,20 +1,20 @@
 {
   "name": "opendoor-plugin",
-  "version": "0.2.0",
+  "version": "0.5.0",
   "description": "Opendoor — a pi-web browser plugin that surfaces Doorstop requirements stored in the workspace: the document tree, per-item state (reviewed/unreviewed changes, suspect links), validation findings, and one-click doorstop CLI actions.",
   "type": "module",
   "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/kairibu/opendoor.git"
-  },
   "keywords": [
     "pi",
+    "pi-package",
     "pi-web",
     "plugin",
+    "pi-web.dev-plugin",
+    "pi-web-plugin",
     "doorstop",
     "requirements"
   ],
+  "main": "browser/pi-web-plugin.js",
   "piWeb": {
     "plugins": [
       {
@@ -25,30 +25,7 @@
       }
     ]
   },
-  "files": [
-    "dist"
-  ],
-  "main": "dist/browser/pi-web-plugin.js",
-  "scripts": {
-    "build": "node scripts/build-plugin.mjs",
-    "dev": "node scripts/build-plugin.mjs --watch",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
-    "typecheck": "tsc --noEmit"
-  },
   "peerDependencies": {
-    "@jmfederico/pi-web": ">=1.202608.1"
-  },
-  "devDependencies": {
-    "@jmfederico/pi-web": "^1.202608.2",
-    "@types/js-yaml": "^4.0.9",
-    "@types/node": "^24.0.0",
-    "@vitest/coverage-v8": "^4.1.11",
-    "esbuild": "^0.27.7",
-    "happy-dom": "^20.11.1",
-    "js-yaml": "^4.3.2",
-    "lit": "^3.3.3",
-    "typescript": "^6.0.3",
-    "vitest": "^4.1.10"
+    "@jmfederico/pi-web": ">=1.202611.1"
   }
 }

package/dist/pi-extension/index.js

@@ -0,0 +1,136 @@
+// Generated from src/pi-extension/opendoor-review.ts. Do not edit directly.
+
+// src/doorstop-session-metadata.ts
+var OPENDOOR_SESSION_METADATA_NAMESPACE = "opendoor";
+var DOORSTOP_SESSION_METADATA_SCHEMA = 1;
+var DOORSTOP_SESSION_METADATA_MAX_UIDS = 8;
+var DOORSTOP_SESSION_METADATA_MAX_UID_LENGTH = 32;
+var DOORSTOP_SESSION_METADATA_OPS = /* @__PURE__ */ new Set(["review"]);
+var DOORSTOP_SESSION_METADATA_UID_PATTERN = /^[\w.-]+$/u;
+function isMetadataUid(value) {
+  return typeof value === "string" && value !== "" && value.length <= DOORSTOP_SESSION_METADATA_MAX_UID_LENGTH && DOORSTOP_SESSION_METADATA_UID_PATTERN.test(value);
+}
+function buildDoorstopSessionMetadata(op, uids) {
+  if (!DOORSTOP_SESSION_METADATA_OPS.has(op)) {
+    throw new Error(`Unsupported doorstop session metadata op: ${JSON.stringify(op)}`);
+  }
+  if (uids.length === 0 || uids.length > DOORSTOP_SESSION_METADATA_MAX_UIDS) {
+    throw new Error(
+      `Doorstop session metadata requires 1..${String(DOORSTOP_SESSION_METADATA_MAX_UIDS)} UIDs`
+    );
+  }
+  const copied = [];
+  for (const uid of uids) {
+    if (!isMetadataUid(uid)) {
+      throw new Error(
+        `Invalid doorstop session metadata UID (expected a non-empty safe token of at most ${String(DOORSTOP_SESSION_METADATA_MAX_UID_LENGTH)} characters)`
+      );
+    }
+    copied.push(uid);
+  }
+  return {
+    schema: DOORSTOP_SESSION_METADATA_SCHEMA,
+    op,
+    uids: copied
+  };
+}
+
+// src/pi-extension/opendoor-review.ts
+var PI_WEB_SESSION_DISPATCH_CHANNEL = "pi-web:session-dispatch:v1";
+var OPENDOOR_REVIEW_COMMAND = "opendoor-review";
+var OPENDOOR_REVIEW_MESSAGE_CUSTOM_TYPE = "opendoor-review-dispatch";
+var OPENDOOR_REVIEW_SESSION_NAME_PREFIX = "Doorstop review ";
+function defaultReviewInstruction(uid) {
+  return `Check doorstop requirement ${uid}: run \`doorstop\` in the workspace root, inspect ${uid}'s child links and references, and when it is ready run \`doorstop review ${uid}\`.`;
+}
+function parseReviewCommandArgs(args) {
+  const trimmed = args.trim();
+  if (trimmed === "") return void 0;
+  const match = /^(\S+)(?:\s+([\s\S]*))?$/u.exec(trimmed);
+  if (match === null) return void 0;
+  const uid = match[1] ?? "";
+  if (uid === "") return void 0;
+  return { uid, notes: (match[2] ?? "").trim() };
+}
+function opendoorReviewExtension(pi) {
+  pi.registerCommand(OPENDOOR_REVIEW_COMMAND, {
+    description: "Start an independent Doorstop review conversation for a requirement UID (e.g. /opendoor-review REQ0001)",
+    handler: async (args) => {
+      await runOpendoorReview(pi, args);
+    }
+  });
+}
+async function runOpendoorReview(pi, args) {
+  const parsed = parseReviewCommandArgs(args);
+  if (parsed === void 0) {
+    report(
+      pi,
+      `Usage: /${OPENDOOR_REVIEW_COMMAND} <UID> [instruction\u2026] \u2014 the UID is a Doorstop requirement UID.`
+    );
+    return;
+  }
+  let metadata;
+  try {
+    metadata = {
+      [OPENDOOR_SESSION_METADATA_NAMESPACE]: buildDoorstopSessionMetadata("review", [parsed.uid])
+    };
+  } catch (error) {
+    report(pi, `Cannot start a Doorstop review session: ${formatError(error)}`);
+    return;
+  }
+  const prompt = parsed.notes === "" ? defaultReviewInstruction(parsed.uid) : parsed.notes;
+  const name = `${OPENDOOR_REVIEW_SESSION_NAME_PREFIX}${parsed.uid}`;
+  let accepted;
+  pi.events.emit(PI_WEB_SESSION_DISPATCH_CHANNEL, {
+    input: { prompt, name, metadata },
+    accept: (result) => {
+      accepted = result instanceof Promise ? result : Promise.resolve(result);
+    }
+  });
+  if (accepted === void 0) {
+    report(
+      pi,
+      "Doorstop review dispatch requires PI WEB's session dispatch bridge. Update PI WEB and restart the session daemon when safe; no session was started."
+    );
+    return;
+  }
+  try {
+    const result = await accepted;
+    report(pi, `Started an independent Doorstop review session for ${parsed.uid}${spawnSuffix(result)}.`);
+  } catch (error) {
+    report(
+      pi,
+      `The Doorstop review session for ${parsed.uid} was not started: ${formatError(error)}`
+    );
+  }
+}
+function report(pi, text) {
+  try {
+    pi.sendMessage({
+      customType: OPENDOOR_REVIEW_MESSAGE_CUSTOM_TYPE,
+      content: text,
+      display: true
+    });
+  } catch {
+  }
+}
+function spawnSuffix(result) {
+  if (typeof result !== "object" || result === null) return "";
+  const sessionId = result["sessionId"];
+  return typeof sessionId === "string" && sessionId !== "" ? ` (session ${sessionId})` : "";
+}
+function formatError(error) {
+  if (error instanceof Error && error.message !== "") return error.message;
+  const text = String(error);
+  return text === "" ? "unknown error" : text;
+}
+export {
+  OPENDOOR_REVIEW_COMMAND,
+  OPENDOOR_REVIEW_MESSAGE_CUSTOM_TYPE,
+  OPENDOOR_REVIEW_SESSION_NAME_PREFIX,
+  PI_WEB_SESSION_DISPATCH_CHANNEL,
+  opendoorReviewExtension as default,
+  defaultReviewInstruction,
+  parseReviewCommandArgs,
+  runOpendoorReview
+};

package/dist/server-plugin.js

@@ -993,10 +993,24 @@ async function probeContainsDoorstopMarker(root, signal) {
   return walk(root, 0);
 }
 var plugin = {
-  apiVersion: 1,
+  apiVersion: 5,
   name: "Opendoor",
+  // Permanently no `requires`/`start()` — see the module header: the review
+  // dispatch moved to the Pi extension, so this provider-contributing entry
+  // never approaches the host's provider-cycle rule (and never declares the
+  // host catalog capability, which is prohibited for provider-contributing
+  // entries).
   activate(context) {
-    return { workspaceProvider: createDoorstopWorkspaceProvider(context) };
+    return {
+      workspaceProvider: createDoorstopWorkspaceProvider(context),
+      // v3 onward (unchanged on v5): the six-operation dispatch lives at the
+      // activation level, not on the provider (`WorkspaceProvider.request`
+      // was removed). The v5 context carries a `scope` union; the dispatch
+      // accepts only the workspace scope and rejects the rest.
+      peer: {
+        request: (request) => dispatchDoorstopOperation(context, request)
+      }
+    };
   }
 };
 var server_plugin_default = plugin;
@@ -1020,28 +1034,47 @@ function createDoorstopWorkspaceProvider(context) {
           publicMetadata: { doorstop: true }
         }
       ];
-    },
-    request: (request) => {
-      switch (request.operation) {
-        case DOORSTOP_RUN_OPERATION:
-          return requestDoorstopBackend(context, request);
-        case DOORSTOP_BASELINE_OPERATION:
-          return requestDoorstopBaseline(context, request);
-        case DOORSTOP_GIT_STATUS_OPERATION:
-          return requestDoorstopGitStatus(context, request);
-        case DOORSTOP_GIT_STAGE_OPERATION:
-          return requestDoorstopGitStage(context, request);
-        case DOORSTOP_GIT_UNSTAGE_OPERATION:
-          return requestDoorstopGitUnstage(context, request);
-        case DOORSTOP_GIT_COMMIT_OPERATION:
-          return requestDoorstopGitCommit(context, request);
-        default:
-          return Promise.reject(unsupportedBackendOperationError(request.operation));
-      }
     }
   });
 }
+function dispatchDoorstopOperation(context, request) {
+  const { scope, operation, input, signal } = request;
+  if (scope.kind !== "workspace") {
+    return Promise.reject(unsupportedPeerScopeError(operation, scope.kind));
+  }
+  const handlerRequest = {
+    project: scope.project,
+    workspace: scope.workspace,
+    operation,
+    input,
+    signal
+  };
+  switch (operation) {
+    case DOORSTOP_RUN_OPERATION:
+      return requestDoorstopBackend(context, handlerRequest);
+    case DOORSTOP_BASELINE_OPERATION:
+      return requestDoorstopBaseline(context, handlerRequest);
+    case DOORSTOP_GIT_STATUS_OPERATION:
+      return requestDoorstopGitStatus(context, handlerRequest);
+    case DOORSTOP_GIT_STAGE_OPERATION:
+      return requestDoorstopGitStage(context, handlerRequest);
+    case DOORSTOP_GIT_UNSTAGE_OPERATION:
+      return requestDoorstopGitUnstage(context, handlerRequest);
+    case DOORSTOP_GIT_COMMIT_OPERATION:
+      return requestDoorstopGitCommit(context, handlerRequest);
+    default:
+      return Promise.reject(unsupportedBackendOperationError(operation));
+  }
+}
+var peerScopeKinds = /* @__PURE__ */ new Set(["machine", "project", "workspace"]);
+function unsupportedPeerScopeError(operation, scopeKind) {
+  const op = /^[A-Za-z0-9.-]+$/.test(operation) ? operation : "unknown";
+  const kind = peerScopeKinds.has(scopeKind) ? scopeKind : "unknown";
+  return new Error(`opendoor: ${op} peer requests require workspace scope (got ${kind})`);
+}
 export {
   createDoorstopWorkspaceProvider,
-  server_plugin_default as default
+  server_plugin_default as default,
+  dispatchDoorstopOperation,
+  unsupportedPeerScopeError
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opendoor-plugin",
-  "version": "0.2.0",
+  "version": "0.5.0",
   "description": "Opendoor — a pi-web browser plugin that surfaces Doorstop requirements stored in the workspace: the document tree, per-item state (reviewed/unreviewed changes, suspect links), validation findings, and one-click doorstop CLI actions.",
   "type": "module",
   "license": "MIT",
@@ -10,18 +10,26 @@
   },
   "keywords": [
     "pi",
+    "pi-package",
     "pi-web",
     "plugin",
+    "pi-web.dev-plugin",
+    "pi-web-plugin",
     "doorstop",
     "requirements"
   ],
+  "pi": {
+    "extensions": [
+      "./dist/pi-extension/index.js"
+    ]
+  },
   "piWeb": {
     "plugins": [
       {
         "id": "opendoor",
-        "browserRoot": "browser",
-        "module": "browser/pi-web-plugin.js",
-        "serverModule": "server-plugin.js"
+        "browserRoot": "dist/browser",
+        "module": "dist/browser/pi-web-plugin.js",
+        "serverModule": "dist/server-plugin.js"
       }
     ]
   },
@@ -37,18 +45,19 @@
     "typecheck": "tsc --noEmit"
   },
   "peerDependencies": {
-    "@jmfederico/pi-web": ">=1.202608.1"
+    "@earendil-works/pi-coding-agent": "*",
+    "@jmfederico/pi-web": ">=1.202611.1"
   },
   "devDependencies": {
-    "@jmfederico/pi-web": "^1.202608.2",
+    "@jmfederico/pi-web": "^1.202611.1",
     "@types/js-yaml": "^4.0.9",
-    "@types/node": "^24.0.0",
-    "@vitest/coverage-v8": "^4.1.11",
+    "@types/node": "^24.19.2",
+    "@vitest/coverage-v8": "^4.1.12",
     "esbuild": "^0.27.7",
-    "happy-dom": "^20.11.1",
+    "happy-dom": "^20.14.6",
     "js-yaml": "^4.3.2",
     "lit": "^3.3.3",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.10"
+    "vitest": "^4.1.12"
   }
 }