@tenonhq/dovetail-core 0.0.124 → 0.0.125

package/README.md

@@ -26,7 +26,8 @@ Quick reference:
 ```bash
 npx dove watch         # HUMAN-ONLY local dev: multi-scope watch + dashboard (hidden from --help; stop before git checkout — see #155)
 npx dove push          # build + push current files
-npx dove refresh       # pull manifest + new files (aliases: pull, r)
+npx dove refresh       # pull manifest + new files, scope-wide (alias: r; bare `dove pull` does the same)
+npx dove pull <table> <sys_id>   # mirror ONE record: its folder + its manifest key, nothing else (--dry-run, --sys-ids, --from-update-set)
 npx dove build         # local build only
 npx dove deploy        # deploy built artifacts
 npx dove dashboard     # update-set dashboard web UI

package/dist/commander.js

@@ -13,6 +13,7 @@ const schemaCommand_1 = require("./schemaCommand");
 const claudeCommand_1 = require("./claudeCommand");
 const createRecordCommand_1 = require("./createRecordCommand");
 const deleteRecordCommand_1 = require("./deleteRecordCommand");
+const pullRecordCommand_1 = require("./pullRecordCommand");
 const reconcileCommand_1 = require("./reconcileCommand");
 const migrateCommand_1 = require("./migrateCommand");
 const clickupCommands_1 = require("./clickupCommands");
@@ -87,10 +88,7 @@ function configureCli(cli) {
     }, async (args) => {
         await (0, allScopesCommands_1.watchAllScopesCommand)(args);
     })
-        .command(
-    // `pull` is the verb people reach for ("pull before push") and it appears
-    // in downstream docs and scripts — alias it rather than silently no-op.
-    ["refresh", "r", "pull"], "Pull latest manifest and file contents from the ServiceNow instance", (cmdArgs) => {
+        .command(["refresh", "r"], "Pull latest manifest and file contents from the ServiceNow instance (scope-wide; for one record see `dove pull <table> <sysId>`)", (cmdArgs) => {
         cmdArgs.options({
             ...sharedOptions,
             force: {
@@ -128,6 +126,75 @@ function configureCli(cli) {
         return cmdArgs;
     }, async (args) => {
         await (0, commands_1.refreshCommand)(args);
+    })
+        .command(
+    // Two commands in one verb, chosen by the positional:
+    //   dove pull                       → the historic alias of `dove refresh`
+    //                                     ("pull before push" is in dozens of
+    //                                     docs and scripts), scope-wide.
+    //   dove pull <table> <sysId...>    → per-record mirror (TenonHQ/Dovetail#319):
+    //                                     ONE record folder + ONE manifest entry,
+    //                                     every other entry byte-identical.
+    // The refresh-only flags (-t/--table, --metadata-only, --benchmark) are
+    // deliberately NOT declared here: -t would collide with the positional
+    // table, and strictCommands tolerates undeclared flags, so the handler
+    // turns a stray -t into an error instead of a silent no-op.
+    "pull [table] [sysId..]", "Mirror one or more records by sys_id into the repo without a scope refresh; with no table, same as `dove refresh`", (cmdArgs) => {
+        cmdArgs
+            .positional("table", {
+            type: "string",
+            describe: "ServiceNow table of the record(s), e.g. sys_security_acl",
+        })
+            // Singular on purpose: yargs camel-cases `--sys-ids` to `sysIds`, and a
+            // positional of the same name would swallow the flag.
+            .positional("sysId", {
+            type: "string",
+            describe: "One or more record sys_ids (space- or comma-separated)",
+        })
+            .options({
+            ...sharedOptions,
+            "sys-ids": {
+                type: "string",
+                describe: "Comma-separated sys_ids (alternative to the positional list)",
+            },
+            "from-update-set": {
+                type: "string",
+                describe: "Pull every per-record capture (<table>_<sys_id>) in this update set; schema/choice-set captures are reported and skipped",
+            },
+            scope: {
+                alias: "s",
+                type: "string",
+                describe: "Target scope. Per-record: every record must belong to it (a mismatch refuses the whole pull). Bare pull: refresh this scope only.",
+            },
+            force: {
+                alias: "f",
+                type: "boolean",
+                default: false,
+                describe: "Overwrite local files even when content matches the instance",
+            },
+            "dry-run": {
+                type: "boolean",
+                default: false,
+                describe: "Per-record only: list the files and manifest keys the pull would touch, write nothing",
+            },
+        });
+        return cmdArgs;
+    }, async (args) => {
+        const pullArgs = args;
+        if (!pullArgs.table && !pullArgs.fromUpdateSet) {
+            // Bare pull == refresh. The refresh-only flags arrive undeclared
+            // (strictCommands tolerates them, yargs still camel-cases them), so
+            // they pass straight through; a bare `-t <table>` is re-mapped to
+            // refresh's --table so the old `dove pull -t x` spelling keeps working.
+            const refreshArgs = args;
+            const stray = pullArgs.t;
+            if (typeof stray === "string" || Array.isArray(stray)) {
+                refreshArgs.table = stray;
+            }
+            await (0, commands_1.refreshCommand)(refreshArgs);
+            return;
+        }
+        await (0, pullRecordCommand_1.pullRecordCommand)(pullArgs);
     })
         .command(["push [target]"], "[DESTRUCTIVE] Push all files from current local files to ServiceNow instance.", (cmdArgs) => {
         cmdArgs.options({