@tenonhq/dovetail-core 0.0.112 → 0.0.114

package/dist/appUtils.js

@@ -36,7 +36,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.createAndAssignUpdateSet = exports.swapScope = exports.buildFiles = exports.summarizeRecord = exports.pushFiles = exports.getAppFileList = exports.groupAppFiles = exports.processMissingFiles = exports.refreshAllFiles = exports.findMissingFiles = exports.syncManifest = exports.narrowManifestToTables = exports.processManifest = exports.normalizeManifestKeys = exports.toSafeFolderName = exports.stampMetadataContent = void 0;
+exports.createAndAssignUpdateSet = exports.swapScope = exports.buildFiles = exports.summarizeRecord = exports.pushFiles = exports.getAppFileList = exports.groupAppFiles = exports.processMissingFiles = exports.refreshAllFiles = exports.findMissingFiles = exports.syncManifest = exports.narrowManifestToRecord = exports.narrowManifestToTables = exports.processManifest = exports.normalizeManifestKeys = exports.toSafeFolderName = exports.stampMetadataContent = void 0;
 const path_1 = __importDefault(require("path"));
 const progress_1 = __importDefault(require("progress"));
 const fUtils = __importStar(require("./FileUtils"));
@@ -298,6 +298,44 @@ const narrowManifestToTables = (manifest, tables) => {
     return Object.assign({}, manifest, { tables: wantedTables });
 };
 exports.narrowManifestToTables = narrowManifestToTables;
+/**
+ * Narrow a manifest's table map to a SINGLE record (matched by `table` +
+ * `sysId`), for the FILE refresh only.
+ *
+ * Returns a NEW manifest and NEVER mutates its input — same safety contract as
+ * `narrowManifestToTables`: the caller keeps writing the original, full manifest
+ * to disk, so the on-disk `dove.manifest.<scope>.json` never loses its other
+ * tables/records. Only the file download is scoped to this one record.
+ *
+ * If the table or a record with that sys_id is absent from the manifest, the
+ * returned manifest has an empty table map — refreshAllFiles no-ops on it, so no
+ * files are written (the create flow warns and points the user at `dove refresh`).
+ */
+const narrowManifestToRecord = (manifest, table, sysId) => {
+    var empty = Object.assign({}, manifest, { tables: {} });
+    var tables = manifest.tables || {};
+    var tableEntry = tables[table];
+    if (!tableEntry || !tableEntry.records)
+        return empty;
+    var records = tableEntry.records;
+    var recKeys = Object.keys(records);
+    var wantedRecords = {};
+    // sys_id is unique per table, so stop at the first match — the record set for
+    // a large table can be sizable and this is the exact scenario the feature targets.
+    for (var i = 0; i < recKeys.length; i++) {
+        var rec = records[recKeys[i]];
+        if (rec && rec.sys_id === sysId) {
+            wantedRecords[recKeys[i]] = rec;
+            break;
+        }
+    }
+    if (Object.keys(wantedRecords).length === 0)
+        return empty;
+    var narrowedTables = {};
+    narrowedTables[table] = Object.assign({}, tableEntry, { records: wantedRecords });
+    return Object.assign({}, manifest, { tables: narrowedTables });
+};
+exports.narrowManifestToRecord = narrowManifestToRecord;
 const syncManifest = async (scope, options = {}) => {
     // Top-level entry owns the collector lifecycle. Recursive calls (all-scopes
     // → per-scope) inherit the collector via options._benchmarkCollector.
@@ -368,6 +406,23 @@ const syncManifest = async (scope, options = {}) => {
             // so intersecting the manifest is enough. `newManifest` is left intact —
             // see narrowManifestToTables.
             var refreshManifest = (0, exports.narrowManifestToTables)(newManifest, options.tables);
+            // Single-record narrowing (dove create round-trip). Applied after the
+            // --table narrowing so the two compose; the full manifest was already
+            // written above, so only the file download is scoped to the one record.
+            if (options.record) {
+                refreshManifest = (0, exports.narrowManifestToRecord)(refreshManifest, options.record.table, options.record.sysId);
+                if (Object.keys(refreshManifest.tables).length === 0) {
+                    Logger_1.logger.warn("Created record " + options.record.sysId + " (" + options.record.table +
+                        ") not found in the refreshed scope manifest — local files not written. " +
+                        "If '" + options.record.table + "' is missing from this scope's `_tables` " +
+                        "whitelist in dove.config.js, add it there first; otherwise run " +
+                        "'npx dove refresh' to pull it.");
+                }
+                else {
+                    FileLogger_1.fileLogger.debug("syncManifest: narrowed file refresh to single record " +
+                        options.record.sysId + " (" + options.record.table + ")");
+                }
+            }
             if (options.tables && options.tables.length > 0) {
                 var wantedNames = Object.keys(refreshManifest.tables);
                 if (wantedNames.length === 0) {

package/dist/commander.js

@@ -228,6 +228,11 @@ function configureCli(cli) {
                 default: false,
                 describe: "Skip interactive prompts",
             },
+            refresh: {
+                type: "boolean",
+                default: true,
+                describe: "Pull the created record's files to disk afterward (default: true; --no-refresh skips the local sync)",
+            },
         });
         return cmdArgs;
     }, createRecordCommand_1.createRecordCommand)

package/dist/createRecordCommand.js

@@ -281,20 +281,33 @@ async function createRecordCommand(args) {
             " (" +
             newSysId +
             ")");
-        // 7. Full round-trip: pull the record back locally
-        Logger_1.logger.info("Syncing record to local files...");
-        FileLogger_1.fileLogger.debug("Starting manifest sync for scope:", scope);
-        try {
-            await AppUtils.syncManifest(scope);
-            var sourcePath = ConfigManager.getSourcePath();
-            var localPath = path_1.default.join(sourcePath, table, recordName);
-            Logger_1.logger.success(chalk_1.default.green("Local files created at: ") + localPath);
+        // 7. Round-trip: pull ONLY the new record's files back locally. Scoped to
+        // { table, sysId } so creating one record no longer refreshes the entire
+        // scope (the whole-scope-churn footgun). --no-refresh skips it entirely.
+        if (typedArgs.refresh === false) {
+            Logger_1.logger.info("Skipping local sync (--no-refresh). Run 'npx dove refresh' to pull the record locally.");
         }
-        catch (syncErr) {
-            Logger_1.logger.warn("Record created on instance but local sync failed.");
-            Logger_1.logger.warn("Run 'npx dove refresh' to pull the record locally.");
-            if (syncErr instanceof Error) {
-                FileLogger_1.fileLogger.error("Sync error:", syncErr.message);
+        else {
+            Logger_1.logger.info("Syncing record to local files...");
+            FileLogger_1.fileLogger.debug("Starting single-record sync for scope:", scope);
+            try {
+                await AppUtils.syncManifest(scope, {
+                    record: { table: table, sysId: newSysId },
+                });
+                // Resolve the scope's own source directory — syncManifest writes into
+                // getSourcePathForScope(scope), which a scope config can override. Using
+                // the top-level getSourcePath() here would print a wrong path in
+                // multi-scope setups even though the files landed correctly.
+                var sourcePath = ConfigManager.getSourcePathForScope(scope);
+                var localPath = path_1.default.join(sourcePath, table, recordName);
+                Logger_1.logger.success(chalk_1.default.green("Local files created at: ") + localPath);
+            }
+            catch (syncErr) {
+                Logger_1.logger.warn("Record created on instance but local sync failed.");
+                Logger_1.logger.warn("Run 'npx dove refresh' to pull the record locally.");
+                if (syncErr instanceof Error) {
+                    FileLogger_1.fileLogger.error("Sync error:", syncErr.message);
+                }
             }
         }
     }

package/dist/tests/dovetailUtilsMSCollision.test.js

@@ -0,0 +1,232 @@
+"use strict";
+/**
+ * Tests the SERVER half of the sync engine — servicenow/dovetail/sys_script_include/
+ * DovetailUtilsMS.js — against the record-name collision that silently dropped records.
+ *
+ * Before the fix, `records` was keyed by display name, so two records sharing a name
+ * collapsed into one entry: one vanished from the manifest with no warning, and because
+ * the survivor kept the shared folder, a later push to that folder wrote to the WRONG
+ * record.
+ *
+ * This loads the real script include into a sandbox with minimal ServiceNow stubs
+ * (Class / GlideRecord / gs) rather than re-implementing the logic, so the test fails if
+ * the deployed source regresses. There is no ATF/SN harness in this repo, and this code
+ * is deployed to a shared instance — so this is the only gate it gets before deploy.
+ *
+ * getFileMap() short-circuits when `includes[table]` carries explicit field overrides,
+ * which is why no sys_dictionary / TableUtils stub is needed here.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const fs_1 = __importDefault(require("fs"));
+const path_1 = __importDefault(require("path"));
+const vm_1 = __importDefault(require("vm"));
+const SOURCE = path_1.default.join(__dirname, "../../../../servicenow/dovetail/sys_script_include/DovetailUtilsMS.js");
+/** Minimal GlideRecord over a fixture list. Only what buildTableMap and processMissingFiles touch. */
+function makeGlideRecord(rows) {
+    return function GlideRecordStub(_table) {
+        var i = -1;
+        this.get = function (sysId) {
+            i = rows.findIndex(function (r) {
+                return r.sys_id === sysId;
+            });
+            return i !== -1;
+        };
+        this.addQuery = function () {
+            return { addOrCondition: function () { } };
+        };
+        this.addEncodedQuery = function () { };
+        this.query = function () {
+            i = -1;
+        };
+        this.next = function () {
+            i += 1;
+            return i < rows.length;
+        };
+        this.getValue = function (field) {
+            return rows[i][field];
+        };
+        this.getDisplayValue = function (field) {
+            // No arg → the record's display value (the name). This is what
+            // generateRecordName() uses, and it is where the collision comes from.
+            return field ? rows[i][field] : rows[i].name;
+        };
+        this.getElements = function () {
+            return null; // metaData capture is skipped; getContents is false below
+        };
+    };
+}
+function loadUtils(rows) {
+    const warnings = [];
+    const sandbox = {
+        Class: {
+            create: function () {
+                return function () {
+                    if (this.initialize)
+                        this.initialize.apply(this, arguments);
+                };
+            },
+        },
+        GlideRecord: makeGlideRecord(rows),
+        gs: {
+            warn: function (msg) {
+                warnings.push(msg);
+            },
+            getProperty: function () {
+                return "https://example.service-now.com/";
+            },
+        },
+    };
+    vm_1.default.createContext(sandbox);
+    vm_1.default.runInContext(fs_1.default.readFileSync(SOURCE, "utf8"), sandbox);
+    return { utils: new sandbox.DovetailUtilsMS(), warnings: warnings };
+}
+function buildMap(rows) {
+    const loaded = loadUtils(rows);
+    const result = loaded.utils.buildTableMap({
+        tableName: "x_cadso_journey_action",
+        scopeId: "scope1",
+        getContents: false,
+        // Explicit field override → getFileMap short-circuits (no sys_dictionary stub).
+        includes: { x_cadso_journey_action: { script: { type: "js" } } },
+        excludes: {},
+        tableOptions: {},
+    });
+    return { records: result.records, warnings: loaded.warnings };
+}
+describe("DovetailUtilsMS.buildTableMap — duplicate display names", () => {
+    it("keeps BOTH records when two share a display name (regression: one was silently dropped)", () => {
+        const out = buildMap([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint" },
+        ]);
+        const keys = Object.keys(out.records);
+        expect(keys).toHaveLength(2); // was 1 — the second overwrote the first
+        const sysIds = keys.map((k) => out.records[k].sys_id).sort();
+        expect(sysIds).toEqual([
+            "1607d7f0c373f2d0d4ddf1db05013101",
+            "46ae2decc33f32d0d4ddf1db050131d9",
+        ]);
+    });
+    it("suffixes EVERY member of a colliding set, so no record arbitrarily keeps the bare name", () => {
+        // If only the later duplicate were suffixed, which record kept "Blueprint" would
+        // depend on query order — and folder names would churn between refreshes.
+        const out = buildMap([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint" },
+        ]);
+        expect(Object.keys(out.records).sort()).toEqual([
+            "Blueprint (1607d7f0)",
+            "Blueprint (46ae2dec)",
+        ]);
+        expect(out.records["Blueprint"]).toBeUndefined();
+    });
+    it("keeps record.name identical to the map key (writers build the folder path from it)", () => {
+        const out = buildMap([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint" },
+        ]);
+        Object.keys(out.records).forEach((key) => {
+            expect(out.records[key].name).toBe(key);
+        });
+    });
+    it("warns on a collision instead of dropping silently", () => {
+        const out = buildMap([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint" },
+        ]);
+        expect(out.warnings).toHaveLength(2);
+        expect(out.warnings[0]).toContain("duplicate display name");
+        expect(out.warnings[0]).toContain("Blueprint");
+    });
+    it("leaves unique names completely untouched (no suffix, no warning)", () => {
+        const out = buildMap([
+            { sys_id: "dc2ef56133a0e2507b18bc534d5c7bf5", name: "Send Email" },
+            { sys_id: "0d4626f933e826507b18bc534d5c7b37", name: "Send Text" },
+        ]);
+        expect(Object.keys(out.records).sort()).toEqual(["Send Email", "Send Text"]);
+        expect(out.warnings).toHaveLength(0);
+    });
+    it("handles three-way collisions", () => {
+        const out = buildMap([
+            { sys_id: "aaaaaaaa11111111", name: "Dup" },
+            { sys_id: "bbbbbbbb22222222", name: "Dup" },
+            { sys_id: "cccccccc33333333", name: "Dup" },
+        ]);
+        expect(Object.keys(out.records).sort()).toEqual([
+            "Dup (aaaaaaaa)",
+            "Dup (bbbbbbbb)",
+            "Dup (cccccccc)",
+        ]);
+    });
+    it("disambiguates only the colliding set, not the whole table", () => {
+        const out = buildMap([
+            { sys_id: "aaaaaaaa11111111", name: "Blueprint" },
+            { sys_id: "bbbbbbbb22222222", name: "Blueprint" },
+            { sys_id: "cccccccc33333333", name: "Send Email" },
+        ]);
+        expect(Object.keys(out.records).sort()).toEqual([
+            "Blueprint (aaaaaaaa)",
+            "Blueprint (bbbbbbbb)",
+            "Send Email", // untouched — this is why differentiatorField was the wrong tool
+        ]);
+    });
+});
+/**
+ * bulkDownload keys its response by display name too — the same collapse dropped the
+ * file CONTENT for one of a colliding pair even once the manifest listed both.
+ */
+function processMissing(rows) {
+    const loaded = loadUtils(rows);
+    const recordMap = {};
+    rows.forEach((r) => {
+        recordMap[r.sys_id] = [{ name: "script", type: "js" }];
+    });
+    const result = loaded.utils.processMissingFiles({ x_cadso_journey_action: recordMap }, {});
+    return {
+        records: result.x_cadso_journey_action.records,
+        warnings: loaded.warnings,
+    };
+}
+describe("DovetailUtilsMS.processMissingFiles — duplicate display names", () => {
+    it("keeps BOTH records' content when two share a display name (regression: one payload was silently dropped)", () => {
+        const out = processMissing([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint", script: "var a = 1;" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint", script: "var b = 2;" },
+        ]);
+        const keys = Object.keys(out.records).sort();
+        expect(keys).toEqual(["Blueprint (1607d7f0)", "Blueprint (46ae2dec)"]);
+        expect(out.records["Blueprint"]).toBeUndefined();
+        // The actual bulkDownload payload — each record keeps its OWN file content.
+        expect(out.records["Blueprint (1607d7f0)"].files[0].content).toBe("var a = 1;");
+        expect(out.records["Blueprint (46ae2dec)"].files[0].content).toBe("var b = 2;");
+    });
+    it("keeps record.name identical to the map key (writers build the folder path from it)", () => {
+        const out = processMissing([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint", script: "" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint", script: "" },
+        ]);
+        Object.keys(out.records).forEach((key) => {
+            expect(out.records[key].name).toBe(key);
+        });
+    });
+    it("warns on a collision instead of staying silent (parity with buildTableMap)", () => {
+        const out = processMissing([
+            { sys_id: "1607d7f0c373f2d0d4ddf1db05013101", name: "Blueprint", script: "" },
+            { sys_id: "46ae2decc33f32d0d4ddf1db050131d9", name: "Blueprint", script: "" },
+        ]);
+        expect(out.warnings).toHaveLength(2);
+        expect(out.warnings[0]).toContain("duplicate display name");
+        expect(out.warnings[0]).toContain("bulkDownload");
+    });
+    it("leaves unique names completely untouched (no suffix, no warning)", () => {
+        const out = processMissing([
+            { sys_id: "dc2ef56133a0e2507b18bc534d5c7bf5", name: "Send Email", script: "" },
+            { sys_id: "0d4626f933e826507b18bc534d5c7b37", name: "Send Text", script: "" },
+        ]);
+        expect(Object.keys(out.records).sort()).toEqual(["Send Email", "Send Text"]);
+        expect(out.warnings).toHaveLength(0);
+    });
+});

package/dist/tests/syncManifestRecord.test.js

@@ -0,0 +1,234 @@
+"use strict";
+/**
+ * Tests for the single-record narrowing of syncManifest — the `record` option
+ * used by `dove create`'s post-create round-trip.
+ *
+ * Regression target: `dove create <table>` previously called syncManifest(scope)
+ * with no narrowing, which re-downloaded and rewrote EVERY record in the scope.
+ * Creating one record churned ~1,892 files in a scope like x_cadso_text_spoke.
+ * The fix scopes the file refresh to just the created record.
+ *
+ * These tests assert:
+ *   1. narrowManifestToRecord returns only the matching record, never mutating input
+ *   2. narrowManifestToRecord returns an empty table map when the record is absent
+ *   3. syncManifest({ record }) fetches + writes ONLY the target record's files
+ *   4. sibling records already on disk are left untouched (never re-fetched)
+ *   5. a record absent from the fetched manifest → warn, no fetch, no throw
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+const fs = __importStar(require("fs"));
+const os = __importStar(require("os"));
+const path = __importStar(require("path"));
+// ---------- mocks (must come before importing the module under test) ----------
+var mockLogger = {
+    info: jest.fn(),
+    debug: jest.fn(),
+    warn: jest.fn(),
+    error: jest.fn(),
+    success: jest.fn(),
+    getLogLevel: function () { return "warn"; },
+};
+var mockFileLogger = {
+    debug: jest.fn(),
+    info: jest.fn(),
+    warn: jest.fn(),
+    error: jest.fn(),
+};
+var mockClient = {
+    getManifest: jest.fn(),
+    getMissingFiles: jest.fn(),
+};
+jest.mock("../Logger", function () { return { logger: mockLogger }; });
+jest.mock("../FileLogger", function () { return { fileLogger: mockFileLogger }; });
+jest.mock("../snClient", function () {
+    return {
+        defaultClient: function () { return mockClient; },
+        unwrapSNResponse: function (p) { return Promise.resolve(p).then(function (r) { return r; }); },
+        processPushResponse: jest.fn(),
+        retryOnErr: jest.fn(),
+        retryOnHttpErr: jest.fn(),
+        unwrapTableAPIFirstItem: jest.fn(),
+    };
+});
+var mockConfig = {
+    getConfig: jest.fn().mockReturnValue({
+        scopes: { x_cadso_core: {} },
+        tableOptions: {},
+    }),
+    getManifest: jest.fn().mockResolvedValue({
+        x_cadso_core: { scope: "x_cadso_core", tables: {} },
+    }),
+    getSourcePathForScope: jest.fn(),
+    getSourcePath: jest.fn(),
+    getManifestPath: jest.fn().mockReturnValue("/tmp/dove.manifest.json"),
+    resolveConfigForScope: jest.fn().mockImplementation(function () {
+        return {
+            tables: ["sys_script_include"],
+            fieldOverrides: {},
+            apiIncludes: {},
+            apiExcludes: {},
+        };
+    }),
+    isMultiScopeManifest: jest.fn().mockReturnValue(true),
+    updateManifest: jest.fn(),
+};
+jest.mock("../config", function () { return mockConfig; });
+jest.mock("progress", function () {
+    return jest.fn().mockImplementation(function () {
+        return { tick: jest.fn() };
+    });
+});
+// Stub writeScopeManifest so the test never writes a manifest file. Leave the
+// other FileUtils exports real so we can assert against the file system.
+jest.mock("../FileUtils", function () {
+    var actual = jest.requireActual("../FileUtils");
+    return Object.assign({}, actual, {
+        writeScopeManifest: jest.fn().mockResolvedValue(undefined),
+    });
+});
+const AppUtils = __importStar(require("../appUtils"));
+// ---------- fixtures ----------
+// A three-record scope manifest, all in sys_script_include, matching the shape
+// client.getManifest returns (files carry name+type only; no content).
+function threeRecordManifest() {
+    return {
+        scope: "x_cadso_core",
+        tables: {
+            sys_script_include: {
+                records: {
+                    New: { name: "New", sys_id: "sysid_New", files: [{ name: "script", type: "js" }] },
+                    Sib1: { name: "Sib1", sys_id: "sysid_Sib1", files: [{ name: "script", type: "js" }] },
+                    Sib2: { name: "Sib2", sys_id: "sysid_Sib2", files: [{ name: "script", type: "js" }] },
+                },
+            },
+        },
+    };
+}
+var tmpRoot;
+function writeLocal(record, name, type, content) {
+    var recDir = path.join(tmpRoot, "sys_script_include", record);
+    fs.mkdirSync(recDir, { recursive: true });
+    fs.writeFileSync(path.join(recDir, name + "." + type), content);
+}
+function readLocal(record, name, type) {
+    var p = path.join(tmpRoot, "sys_script_include", record, name + "." + type);
+    try {
+        return fs.readFileSync(p, "utf8");
+    }
+    catch (e) {
+        return null;
+    }
+}
+// ---------- pure helper ----------
+describe("narrowManifestToRecord", function () {
+    test("returns only the matching record and does NOT mutate the input", function () {
+        var manifest = threeRecordManifest();
+        var out = AppUtils.narrowManifestToRecord(manifest, "sys_script_include", "sysid_New");
+        // input untouched — all three records still present
+        expect(Object.keys(manifest.tables.sys_script_include.records).sort())
+            .toEqual(["New", "Sib1", "Sib2"]);
+        // output holds exactly the one record
+        expect(Object.keys(out.tables)).toEqual(["sys_script_include"]);
+        expect(Object.keys(out.tables.sys_script_include.records)).toEqual(["New"]);
+        expect(out.tables.sys_script_include.records.New.sys_id).toBe("sysid_New");
+    });
+    test("returns an empty table map when the sys_id is absent", function () {
+        var manifest = threeRecordManifest();
+        var out = AppUtils.narrowManifestToRecord(manifest, "sys_script_include", "sysid_Missing");
+        expect(Object.keys(out.tables)).toEqual([]);
+    });
+    test("returns an empty table map when the table is absent", function () {
+        var manifest = threeRecordManifest();
+        var out = AppUtils.narrowManifestToRecord(manifest, "sys_ui_action", "sysid_New");
+        expect(Object.keys(out.tables)).toEqual([]);
+    });
+});
+// ---------- syncManifest({ record }) ----------
+describe("syncManifest — single-record round-trip (dove create)", function () {
+    beforeEach(function () {
+        jest.clearAllMocks();
+        tmpRoot = fs.mkdtempSync(path.join(os.tmpdir(), "sinc-record-test-"));
+        mockConfig.getSourcePathForScope.mockReturnValue(tmpRoot);
+        mockConfig.getSourcePath.mockReturnValue(tmpRoot);
+    });
+    afterEach(function () {
+        try {
+            fs.rmSync(tmpRoot, { recursive: true, force: true });
+        }
+        catch (e) { }
+    });
+    test("fetches + writes ONLY the target record; siblings are never re-fetched", async function () {
+        // A stale sibling file already on disk — it must be left exactly as-is.
+        writeLocal("Sib1", "script", "js", "STALE SIBLING — must not change");
+        mockClient.getManifest.mockResolvedValue(threeRecordManifest());
+        // getMissingFiles should be asked for the new record ONLY; return its content.
+        mockClient.getMissingFiles.mockResolvedValue({
+            sys_script_include: {
+                records: {
+                    New: {
+                        name: "New", sys_id: "sysid_New",
+                        files: [{ name: "script", type: "js", content: "// fresh created record" }],
+                    },
+                },
+            },
+        });
+        await AppUtils.syncManifest("x_cadso_core", {
+            record: { table: "sys_script_include", sysId: "sysid_New" },
+        });
+        // exactly one content fetch, scoped to the one record
+        expect(mockClient.getMissingFiles).toHaveBeenCalledTimes(1);
+        var missingArg = mockClient.getMissingFiles.mock.calls[0][0];
+        expect(Object.keys(missingArg.sys_script_include)).toEqual(["sysid_New"]);
+        // the new record landed
+        expect(readLocal("New", "script", "js")).toBe("// fresh created record");
+        // siblings untouched
+        expect(readLocal("Sib1", "script", "js")).toBe("STALE SIBLING — must not change");
+        expect(readLocal("Sib2", "script", "js")).toBeNull();
+    });
+    test("record absent from the fetched manifest → warn, no fetch, no throw", async function () {
+        mockClient.getManifest.mockResolvedValue(threeRecordManifest());
+        await AppUtils.syncManifest("x_cadso_core", {
+            record: { table: "sys_script_include", sysId: "sysid_DoesNotExist" },
+        });
+        // empty narrowed manifest → refreshAllFiles no-ops, so no content fetch
+        expect(mockClient.getMissingFiles).not.toHaveBeenCalled();
+        // and the user is told to run a full refresh
+        var warned = mockLogger.warn.mock.calls.some(function (c) {
+            return typeof c[0] === "string" && c[0].indexOf("sysid_DoesNotExist") !== -1;
+        });
+        expect(warned).toBe(true);
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tenonhq/dovetail-core",
-  "version": "0.0.112",
+  "version": "0.0.114",
   "description": "Next-gen file syncer",
   "license": "GPL-3.0",
   "main": "./dist/index.js",

package/release-manifest.json

@@ -1,7 +1,247 @@
 {
   "$schema": "dovetail-manifest/v1",
-  "generated_at": "2026-07-14T18:00:31.248Z",
+  "generated_at": "2026-07-15T02:38:50.750Z",
   "events": [
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-servicenow@0.0.30",
+      "package": "@tenonhq/dovetail-servicenow",
+      "version": "0.0.30",
+      "prev_version": "0.0.29",
+      "published_at": "2026-07-15T02:38:50.737Z",
+      "git": {
+        "sha": "2977ae7b00bf62482f12d7a6ccf7b87babdbae21",
+        "tag": "dovetail-servicenow@0.0.30"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "2977ae7b0",
+          "subject": "feat(servicenow): set-field/create-record accept --from-json for large fields (#224)",
+          "pr": 224
+        },
+        {
+          "sha": "e0c350259",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-mcp@0.0.27",
+      "package": "@tenonhq/dovetail-mcp",
+      "version": "0.0.27",
+      "prev_version": "0.0.26",
+      "published_at": "2026-07-15T02:38:50.737Z",
+      "git": {
+        "sha": "2977ae7b00bf62482f12d7a6ccf7b87babdbae21",
+        "tag": "dovetail-mcp@0.0.27"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "e0c350259",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-servicenow@0.0.29",
+      "package": "@tenonhq/dovetail-servicenow",
+      "version": "0.0.29",
+      "prev_version": "0.0.28",
+      "published_at": "2026-07-15T01:16:10.133Z",
+      "git": {
+        "sha": "6d7309949cc82ece927f9a4f61614ef4a23cc5ee",
+        "tag": "dovetail-servicenow@0.0.29"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "6d7309949",
+          "subject": "feat(servicenow): set-column verb + set_column MCP tool (DEV-509) (#217)",
+          "pr": 217
+        },
+        {
+          "sha": "6a4081d25",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-mcp@0.0.26",
+      "package": "@tenonhq/dovetail-mcp",
+      "version": "0.0.26",
+      "prev_version": "0.0.25",
+      "published_at": "2026-07-15T01:16:10.133Z",
+      "git": {
+        "sha": "6d7309949cc82ece927f9a4f61614ef4a23cc5ee",
+        "tag": "dovetail-mcp@0.0.26"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "6a4081d25",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-servicenow@0.0.28",
+      "package": "@tenonhq/dovetail-servicenow",
+      "version": "0.0.28",
+      "prev_version": "0.0.27",
+      "published_at": "2026-07-14T20:25:03.138Z",
+      "git": {
+        "sha": "a5aeaa3c3f56c0bb24de80f12808689c5987e457",
+        "tag": "dovetail-servicenow@0.0.28"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "a5aeaa3c3",
+          "subject": "feat(servicenow): edit a published action type's steps in one publish (#216) (#223)",
+          "pr": 216
+        },
+        {
+          "sha": "b8967c79d",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-mcp@0.0.25",
+      "package": "@tenonhq/dovetail-mcp",
+      "version": "0.0.25",
+      "prev_version": "0.0.24",
+      "published_at": "2026-07-14T20:25:03.138Z",
+      "git": {
+        "sha": "a5aeaa3c3f56c0bb24de80f12808689c5987e457",
+        "tag": "dovetail-mcp@0.0.25"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "b8967c79d",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-servicenow@0.0.27",
+      "package": "@tenonhq/dovetail-servicenow",
+      "version": "0.0.27",
+      "prev_version": "0.0.26",
+      "published_at": "2026-07-14T20:15:42.598Z",
+      "git": {
+        "sha": "43d6c5124bec7e79537d8ad6364749f36c601d9c",
+        "tag": "dovetail-servicenow@0.0.27"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "43d6c5124",
+          "subject": "feat(servicenow): add-column mandatory/default via scope-aware insert (#213)",
+          "pr": 213
+        },
+        {
+          "sha": "ef9e73505",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-mcp@0.0.24",
+      "package": "@tenonhq/dovetail-mcp",
+      "version": "0.0.24",
+      "prev_version": "0.0.23",
+      "published_at": "2026-07-14T20:15:42.598Z",
+      "git": {
+        "sha": "43d6c5124bec7e79537d8ad6364749f36c601d9c",
+        "tag": "dovetail-mcp@0.0.24"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "ef9e73505",
+          "subject": "chore(release): publish 2 packages [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-core@0.0.113",
+      "package": "@tenonhq/dovetail-core",
+      "version": "0.0.113",
+      "prev_version": "0.0.112",
+      "published_at": "2026-07-14T20:02:38.069Z",
+      "git": {
+        "sha": "99f064bf76f0fa4a2e049debff52be8303ebed24",
+        "tag": "dovetail-core@0.0.113"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "99f064bf7",
+          "subject": "fix(servicenow): stop getManifest silently dropping same-named records (#222)",
+          "pr": 222
+        },
+        {
+          "sha": "38ffe1e27",
+          "subject": "chore(release): @tenonhq/dovetail-core@0.0.112 [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
+    {
+      "$schema": "dovetail-release-event/v1",
+      "event_id": "dovetail-core@0.0.112",
+      "package": "@tenonhq/dovetail-core",
+      "version": "0.0.112",
+      "prev_version": "0.0.111",
+      "published_at": "2026-07-14T19:40:43.702Z",
+      "git": {
+        "sha": "a2777edcbd46ee69de13e73173132f9cd18aa618",
+        "tag": "dovetail-core@0.0.112"
+      },
+      "semver_bump": "patch",
+      "commits": [
+        {
+          "sha": "a2777edcb",
+          "subject": "fix: reject unknown dove commands and alias pull to refresh (#219) (#221)",
+          "pr": 219
+        },
+        {
+          "sha": "6a42ef546",
+          "subject": "chore(release): @tenonhq/dovetail-core@0.0.111 [skip ci]"
+        }
+      ],
+      "exports_delta": null,
+      "narrative": null
+    },
     {
       "$schema": "dovetail-release-event/v1",
       "event_id": "dovetail-core@0.0.111",
@@ -985,298 +1225,6 @@
       ],
       "exports_delta": null,
       "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-babel-plugin@0.0.10",
-      "package": "@tenonhq/dovetail-babel-plugin",
-      "version": "0.0.10",
-      "prev_version": "0.0.9",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-babel-plugin@0.0.10"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "9332fdf7a",
-          "subject": "chore(release): publish 20 packages [skip ci]"
-        },
-        {
-          "sha": "9046da774",
-          "subject": "chore(release): publish 19 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-babel-plugin-remove-modules@0.0.10",
-      "package": "@tenonhq/dovetail-babel-plugin-remove-modules",
-      "version": "0.0.10",
-      "prev_version": "0.0.9",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-babel-plugin-remove-modules@0.0.10"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "9332fdf7a",
-          "subject": "chore(release): publish 20 packages [skip ci]"
-        },
-        {
-          "sha": "9046da774",
-          "subject": "chore(release): publish 19 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-babel-preset-servicenow@0.0.10",
-      "package": "@tenonhq/dovetail-babel-preset-servicenow",
-      "version": "0.0.10",
-      "prev_version": "0.0.9",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-babel-preset-servicenow@0.0.10"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "9332fdf7a",
-          "subject": "chore(release): publish 20 packages [skip ci]"
-        },
-        {
-          "sha": "9046da774",
-          "subject": "chore(release): publish 19 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-schema@0.0.11",
-      "package": "@tenonhq/dovetail-schema",
-      "version": "0.0.11",
-      "prev_version": "0.0.10",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-schema@0.0.11"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "a8922d9d5",
-          "subject": "chore(release): @tenonhq/dovetail-schema@0.0.10 [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-core@0.0.103",
-      "package": "@tenonhq/dovetail-core",
-      "version": "0.0.103",
-      "prev_version": "0.0.102",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-core@0.0.103"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "d5e064760",
-          "subject": "chore(release): @tenonhq/dovetail-servicenow@0.0.15 [skip ci]"
-        },
-        {
-          "sha": "af1417796",
-          "subject": "chore(release): publish 3 packages [skip ci]"
-        },
-        {
-          "sha": "e25ae74c2",
-          "subject": "chore(release): @tenonhq/dovetail-core@0.0.102 [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-dashboard@0.0.37",
-      "package": "@tenonhq/dovetail-dashboard",
-      "version": "0.0.37",
-      "prev_version": "0.0.36",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-dashboard@0.0.37"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "ba1c67775",
-          "subject": "chore(release): @tenonhq/dovetail-dashboard@0.0.36 [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-eslint-plugin@0.0.10",
-      "package": "@tenonhq/dovetail-eslint-plugin",
-      "version": "0.0.10",
-      "prev_version": "0.0.9",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-eslint-plugin@0.0.10"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "9332fdf7a",
-          "subject": "chore(release): publish 20 packages [skip ci]"
-        },
-        {
-          "sha": "9046da774",
-          "subject": "chore(release): publish 19 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-gmail@0.0.11",
-      "package": "@tenonhq/dovetail-gmail",
-      "version": "0.0.11",
-      "prev_version": "0.0.10",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-gmail@0.0.11"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "9332fdf7a",
-          "subject": "chore(release): publish 20 packages [skip ci]"
-        },
-        {
-          "sha": "9046da774",
-          "subject": "chore(release): publish 19 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-google-calendar@0.0.11",
-      "package": "@tenonhq/dovetail-google-calendar",
-      "version": "0.0.11",
-      "prev_version": "0.0.10",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-google-calendar@0.0.11"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "9332fdf7a",
-          "subject": "chore(release): publish 20 packages [skip ci]"
-        },
-        {
-          "sha": "9046da774",
-          "subject": "chore(release): publish 19 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
-    },
-    {
-      "$schema": "dovetail-release-event/v1",
-      "event_id": "dovetail-mcp@0.0.9",
-      "package": "@tenonhq/dovetail-mcp",
-      "version": "0.0.9",
-      "prev_version": "0.0.8",
-      "published_at": "2026-06-05T06:43:18.252Z",
-      "git": {
-        "sha": "71bdf9bba8bd5e3849a212a18d9ebc9241232b32",
-        "tag": "dovetail-mcp@0.0.9"
-      },
-      "semver_bump": "patch",
-      "commits": [
-        {
-          "sha": "71bdf9bba",
-          "subject": "fix: float internal dep ranges to ~0.0.x so consumers need no overrides (#148)",
-          "pr": 148
-        },
-        {
-          "sha": "af1417796",
-          "subject": "chore(release): publish 3 packages [skip ci]"
-        }
-      ],
-      "exports_delta": null,
-      "narrative": null
     }
   ]
 }