openplanr 3.3.0 → 3.4.0

package/lib/host-packages/openai/openplanr/runtime/design/scripts/design.mjs

@@ -9,9 +9,18 @@ import {
   stopArtifactReviewServer
 } from "./design-review.mjs";
 import {
+  ARTIFACT_REVIEW_MAX_STATE_BYTES,
+  createReviewLedger,
+  decodeArtifactReviewSources,
+  mergeReviewLedger,
+  normalizeArtifactReview,
+  readArtifactReviewState,
   readDesignFeedback,
   readDesignHandoff,
-  updateDesignHandoff
+  resolveArtifactReviewDestination,
+  updateDesignHandoff,
+  withArtifactReviewLock,
+  writeArtifactReviewState
 } from "./design-handoff.mjs";
 import {
   inspectDesignDocument,
@@ -21,53 +30,39 @@ import {
   renderDesignDocument,
   standaloneDesignHtml
 } from "./design-document.mjs";
+import {
+  listDesignRevisions,
+  readDesignRevision
+} from "./design-shared-artifact-support-dependencies-design-support-protocol-contracts-d707e49b.mjs";
+import "./design-review-state-store.mjs";
+import "./design-parse5-parser.mjs";
+import "./design-parse5-tokenizer.mjs";
+import "./design-entities.mjs";
 import "./design-artifact-shell.mjs";
+import {
+  ARTIFACT_ERROR_CODES,
+  PipelineError,
+  digestArtifactEnvelope
+} from "./design-shared-artifact-support-protocol-contracts-9f6918b0.mjs";
 import "./design-sandbox-guards.mjs";
 import {
-  ARTIFACT_REVIEW_MAX_STATE_BYTES,
-  createReviewLedger,
-  decodeArtifactReviewSources,
   exportDesignShareRecovery,
   importDesignShareRecovery,
   manageDesignShare,
-  mergeReviewLedger,
-  normalizeArtifactReview,
   publishDesignShare,
-  readArtifactReviewState,
-  resolveArtifactReviewDestination,
   shareDesign,
-  syncDesignShare,
-  withArtifactReviewLock,
-  writeArtifactReviewState
-} from "./design-share.mjs";
+  syncDesignShare
+} from "./design-share-cli-bridge.mjs";
 import {
   atomicJson,
   currentDesign,
   hash,
-  listDesignRevisions,
-  readDesignRevision,
   readJson
-} from "./design-shared-artifact-support-dependencies-design-support-protocol-contracts-3d9b1a86.mjs";
-import "./design-encrypted-workspace-client.mjs";
-import "./design-upload-spool.mjs";
-import "./design-chunked-workspace-client.mjs";
-import "./design-resource-pack.mjs";
-import "./design-compression.mjs";
-import "./design-owner-custody.mjs";
+} from "./design-shared-design-support-0bac378d.mjs";
 import "./design-runtime-home.mjs";
 import {
   acquireStartLock
 } from "./design-loopback-server.mjs";
-import "./design-parse5-parser.mjs";
-import "./design-parse5-tokenizer.mjs";
-import "./design-entities.mjs";
-import {
-  ARTIFACT_ERROR_CODES,
-  PipelineError,
-  digestArtifactEnvelope
-} from "./design-shared-artifact-support-protocol-contracts-ea2cd15e.mjs";
-import "./design-shared-protocol-contracts-b5f28f94.mjs";
-import "./design-shared-protocol-contracts-75a938cc.mjs";
 
 // packages/design/lib/design/utility.mjs
 import { spawn } from "node:child_process";
@@ -352,12 +347,14 @@ import {
   statSync,
   writeFileSync
 } from "node:fs";
-import { dirname, join as join2, resolve as resolve2 } from "node:path";
+import { basename, dirname, join as join2, resolve as resolve2 } from "node:path";
 var MAX_FAMILIES = 2;
 var MAX_WEIGHTS = 3;
 var TIMEOUT_MS = 2e4;
 var MAX_FILE_BYTES = 15e5;
 var MAX_TOTAL_BYTES = 5e6;
+var STAGING_MAX_AGE_MS = 10 * 6e4;
+var STAGING_PREFIX = ".fonts-fetch-";
 var CSS_ENDPOINT = "https://fonts.googleapis.com/css2";
 var FONT_ORIGIN = "https://fonts.gstatic.com/";
 var CATALOG_ROOT = "https://raw.githubusercontent.com/google/fonts/main";
@@ -499,6 +496,10 @@ async function fetchDesignFonts({ families, dest, fetchImpl = fetch }) {
       `--family names a family twice: ${specs.map(({ family }) => family).join(", ")}.`
     );
   const target = resolve2(dest);
+  if (basename(target).startsWith(STAGING_PREFIX))
+    throw new Error(
+      `Font destination uses the reserved staging prefix ${STAGING_PREFIX}: ${target}`
+    );
   assertWritableDestination(target);
   const files = /* @__PURE__ */ new Map();
   const rules = [];
@@ -540,9 +541,29 @@ async function fetchDesignFonts({ families, dest, fetchImpl = fetch }) {
     });
   }
   add("fonts.css", Buffer.from(rules.join("")));
-  mkdirSync(dirname(target), { recursive: true });
-  const staging = mkdtempSync(join2(dirname(target), ".fonts-fetch-"));
+  const parent = dirname(target);
+  const missingParents = [];
+  for (let directory = parent; !existsSync(directory); directory = dirname(directory))
+    missingParents.push(directory);
+  const createdParents = [];
+  let staging;
   try {
+    for (const directory of missingParents.reverse()) {
+      mkdirSync(directory);
+      createdParents.push(directory);
+    }
+    const staleBefore = Date.now() - STAGING_MAX_AGE_MS;
+    for (const entry of readdirSync(parent, { withFileTypes: true })) {
+      if (!entry.isDirectory() || !entry.name.startsWith(STAGING_PREFIX)) continue;
+      const path = join2(parent, entry.name);
+      try {
+        if (statSync(path).mtimeMs < staleBefore) rmSync(path, { recursive: true });
+      } catch (error) {
+        if (error.code === "ENOENT") continue;
+        throw new Error(`Could not remove stale staging folder ${path}: ${error.message}`);
+      }
+    }
+    staging = mkdtempSync(join2(parent, STAGING_PREFIX));
     for (const [path, bytes] of files) {
       mkdirSync(dirname(join2(staging, path)), { recursive: true });
       writeFileSync(join2(staging, path), bytes, { flag: "wx" });
@@ -552,7 +573,19 @@ async function fetchDesignFonts({ families, dest, fetchImpl = fetch }) {
     chmodSync(staging, 493);
     renameSync(staging, target);
   } catch (error) {
-    rmSync(staging, { recursive: true, force: true });
+    const cleanupErrors = [];
+    for (const directory of [...staging ? [staging] : [], ...createdParents.reverse()]) {
+      try {
+        if (directory === staging) rmSync(directory, { recursive: true, force: true });
+        else if (readdirSync(directory).length === 0) rmdirSync(directory);
+      } catch (cleanupError) {
+        cleanupErrors.push({ path: directory, message: cleanupError.message });
+      }
+    }
+    if (cleanupErrors.length > 0) {
+      error.cleanupErrors = cleanupErrors;
+      error.message += cleanupErrors.map(({ path, message }) => ` (cleanup also failed for ${path}: ${message})`).join("");
+    }
     throw error;
   }
   return {
@@ -1149,6 +1182,34 @@ function verifyDesignDocument(file, report) {
   atomicJson(join4(current.root, ".design/verification", `${current.revision}.json`), saved);
   return saved;
 }
+var HELP_FLAGS = /* @__PURE__ */ new Set(["--help", "-h"]);
+var COMMAND_SUMMARIES = Object.freeze({
+  inspect: "Report the design document and its current render.",
+  validate: "Validate the design document without rendering.",
+  render: "Render the design document into its review revision.",
+  open: "Open the local review studio.",
+  studio: "Report or stop a running local studio (--action status|stop).",
+  "view-context": "Report what the studio is showing.",
+  export: "Export the design as html, json, or markdown.",
+  feedback: "Inspect, export, or import review feedback.",
+  verify: "Record a browser verification report.",
+  share: "Create a hosted review of the current revision.",
+  publish: "Publish the current revision to the existing hosted review.",
+  sync: "Retrieve hosted review feedback.",
+  manage: "Manage a hosted review (--action rotate|pause|resume|revoke|delete|recovery|restore).",
+  handoff: "Inspect or update the review handoff.",
+  fonts: "Download open-license fonts (fonts fetch --family ... --dest ...)."
+});
+function commandHelp(command) {
+  if (!Object.hasOwn(COMMAND_SUMMARIES, command))
+    throw new Error(`Unknown command: ${command}. Run design.mjs --help for the command list.`);
+  return {
+    command,
+    usage: command === "fonts" ? 'design.mjs fonts fetch --family "<Name>:<weights>" [--family ...] --dest <dir>' : `design.mjs ${command} <design-document.json> [options]`,
+    summary: COMMAND_SUMMARIES[command],
+    note: "Run design.mjs --help for every option. A help request never reads, renders, or sends anything."
+  };
+}
 async function designUtility(argv, {
   stdout = (value) => process.stdout.write(`${JSON.stringify(value)}
 `),
@@ -1156,7 +1217,7 @@ async function designUtility(argv, {
   env = process.env,
   fetchImpl = fetch
 } = {}) {
-  if (argv.length === 0 || argv[0] === "--help" || argv[0] === "help") {
+  if (argv.length === 0 || ["--help", "-h", "help"].includes(argv[0])) {
     const help = {
       usage: 'design.mjs inspect|validate|render|open|studio|view-context|export|feedback|verify|share|publish|sync|manage|handoff <design-document.json> | design.mjs fonts fetch --family "<Name>:<weights>" [--family ...] --dest <dir>',
       flags: [
@@ -1185,6 +1246,13 @@ async function designUtility(argv, {
     return help;
   }
   const [command, input, ...args] = argv;
+  if (argv.some((value) => HELP_FLAGS.has(value))) {
+    const help = commandHelp(command);
+    stdout(help);
+    return help;
+  }
+  if (typeof input === "string" && input.startsWith("-"))
+    throw new Error(`${input} must be a design document path, not an option.`);
   if (![
     "inspect",
     "validate",
@@ -1335,7 +1403,7 @@ async function designUtility(argv, {
   }
   if (command === "feedback") {
     const action = flags.action ?? "inspect";
-    if (action !== "import") await syncDesignShare(file, { env, fetchImpl });
+    const synced = action === "import" ? void 0 : await syncDesignShare(file, { env, fetchImpl, unavailable: "skip" });
     if (action === "import")
       result = await importDesignFeedback(file, {
         input: flags.input,
@@ -1395,6 +1463,8 @@ async function designUtility(argv, {
         revision: current.revision
       };
     } else throw new Error("Feedback action must be inspect, export, import, select, or resolve.");
+    if (synced?.skipped)
+      result = { ...result, sync: { skipped: synced.skipped, reason: synced.reason } };
   }
   stdout(result);
   return result;

package/lib/host-packages/openai/openplanr/runtime/design/scripts/runtime/notices/THIRD_PARTY_NOTICES.txt

@@ -1737,33 +1737,6 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 
 
-----------------
-
-pako@2.1.0
-
-(The MIT License)
-
-Copyright (C) 2014-2017 by Vitaly Puzrin and Andrei Tuputcyn
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.
-
-
 ----------------
 
 parse5@8.0.1