@openscaffold/cli 0.1.3 → 0.1.5

package/dist/index.js

@@ -4,7 +4,7 @@ import { execFileSync, spawn } from "node:child_process";
 import { chmodSync, existsSync, lstatSync, mkdirSync, readdirSync, readFileSync, readlinkSync, rmSync, statSync, writeFileSync } from "node:fs";
 import { homedir, hostname } from "node:os";
 import { dirname, join, resolve } from "node:path";
-import { briefingFacts, briefingStatus, browserCommand, collectDiff, gitInfo, hasOpenScaffoldPlugin, hookBase, HOSTED, installedHooks, loadedEntry, loginResultLine, MCP_NAME, mcpText, nameFromDomain, callerSession, commandLine, rememberedCache, DEFAULT_REVIEW_TITLE, parseReviewArgs, parseSetupArgs, planSummary, pluginServes, preCommitOutput, REVIEW_HELP, REVIEW_WAIT, reviewTitle, selectFiles, SETUP_EXTENSIONS, setupPrompt, setupResultLine, skillFile, statusReport, stopOutput, switchAllowed, switchedTo, transcriptActivity, whichCustomerError, windowsCommandLine, withHooks, withoutHooks, } from "./install.js";
+import { briefingFacts, briefingStatus, browserCommand, collectDiff, gitInfo, hasOpenScaffoldPlugin, hookBase, HOSTED, installedHooks, loadedEntry, loginResultLine, MCP_NAME, mcpText, nameFromDomain, callerSession, commandLine, gitTracks, rememberedCache, DEFAULT_REVIEW_TITLE, parseReviewArgs, parseSetupArgs, planSummary, pluginServes, preCommitOutput, REVIEW_HELP, REVIEW_WAIT, reviewTitle, selectFiles, SETUP_EXTENSIONS, setupPrompt, setupResultLine, skillFile, statusReport, stopOutput, switchAllowed, switchedTo, transcriptActivity, whichCustomerError, windowsCommandLine, withHooks, withoutHooks, } from "./install.js";
 const CONFIG = join(homedir(), ".config", "openscaffold");
 const CREDENTIALS = join(CONFIG, "credentials.json");
 const CACHE = join(CONFIG, "cache");
@@ -443,7 +443,7 @@ async function useDeployment() {
  * get_review (which holds up to 50 s per call) and prints the verdict. Nothing to paste, nothing to poll.
  */
 async function review() {
-    const args = parseReviewArgs(process.argv.slice(3), existsSync);
+    const args = parseReviewArgs(process.argv.slice(3), (p) => existsSync(p) || gitTracks(p, (a) => execFileSync("git", a, { stdio: "ignore" })));
     if ("help" in args)
         return console.log(REVIEW_HELP);
     if ("error" in args)

package/dist/install.js

@@ -446,7 +446,7 @@ export function parseReviewArgs(args, exists = () => true) {
             return { error: `Unknown option ${flag}. ${REVIEW_USAGE}` };
         else if (/^req_\w+$/.test(args[i]) && !exists(args[i]))
             out.reqs.push(args[i]); // `--req req_1 req_2`
-        // ponytail: a deleted file named alone reads as the title; then the whole change is reviewed, which still includes it.
+        // `exists` also knows paths git tracks (a deleted file), so a listed deletion is a file, not the title.
         else if (!out.title && !exists(args[i]))
             out.title = args[i];
         else
@@ -454,8 +454,28 @@ export function parseReviewArgs(args, exists = () => true) {
     }
     return { ...out, title: out.title.trim() || DEFAULT_REVIEW_TITLE, reqs: [...new Set(out.reqs)] };
 }
-/** A CLI command line to copy and run: `npx -y @openscaffold/cli@latest <args>`, quoting what a shell would split. */
-export const commandLine = (args) => ["npx -y @openscaffold/cli@latest", ...args.map((a) => (/^[\w./:@=,-]+$/.test(a) ? a : JSON.stringify(a)))].join(" ");
+/**
+ * A CLI command line to copy and run: `npx -y @openscaffold/cli@latest <args>`, each argument single-quoted when a shell
+ * would split or expand it ($, backticks, spaces), so the copy keeps the exact text: PowerShell's form on Windows.
+ */
+export const commandLine = (args, platform = process.platform) => ["npx -y @openscaffold/cli@latest", ...args.map((a) => /^[\w./:@=,-]+$/.test(a) ? a
+        // PowerShell: single quotes are literal and a quote is doubled; elsewhere POSIX's '\'' escape.
+        : platform === "win32" ? `'${a.replace(/'/g, "''")}'` : `'${a.replace(/'/g, `'\\''`)}'`)].join(" ");
+/**
+ * Whether git tracks a path given relative to the current folder: in the index, or in HEAD (a staged deletion).
+ * `HEAD:./p` resolves against the current folder, not the repository root. `git` runs git and throws on failure.
+ */
+export function gitTracks(path, git) {
+    return [["ls-files", "--error-unmatch", "--", path], ["cat-file", "-e", `HEAD:./${path.replace(/^\.\//, "")}`]].some((a) => {
+        try {
+            git(a);
+            return true;
+        }
+        catch {
+            return false;
+        }
+    });
+}
 /** `review` with no --deployment and none cached for this session or folder: the same command with each cached customer's id. */
 export function whichCustomerError(args, known) {
     const head = "Which customer? This session has no OpenScaffold deployment cached here.";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openscaffold/cli",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Connect Claude Code to your OpenScaffold workspace: each session loads the right customer deployment.",
   "type": "module",
   "bin": {