@indigoai-us/hq-cloud 6.14.9 → 6.14.11

package/dist/cli/share.test.js

@@ -62,6 +62,14 @@ function makeEntityContext(overrides = {}) {
         ...overrides,
     };
 }
+function deleteIntent(remoteEtag, localHash, localKind = "file") {
+    return {
+        version: 1,
+        remoteEtag,
+        localHash,
+        localKind,
+    };
+}
 describe("wrapFilterWithIgnoreVisibility", () => {
     it("records noteworthy base-ignore exclusions while counting all rejections", () => {
         const hqRoot = path.join("/tmp", "hqroot");
@@ -1613,6 +1621,8 @@ describe("share", () => {
                     syncedAt: new Date().toISOString(),
                     direction: "up",
                     remoteEtag: "knowledge-etag",
+                    kind: "symlink",
+                    localDeleteIntent: deleteIntent("knowledge-etag", "irrelevant-not-checked", "symlink"),
                 },
             },
         }));
@@ -1744,6 +1754,32 @@ describe("share", () => {
             delete process.env.HQ_STATE_DIR;
         }
     });
+    it("personalMode excludes generated namespaced skill wrappers while preserving local skills", async () => {
+        const generatedSource = path.join(tmpDir, "companies", "acme", "skills", "demo", "scripts");
+        const generatedWrapper = path.join(tmpDir, ".claude", "skills", "acme:demo");
+        const localSkill = path.join(tmpDir, ".claude", "skills", "local-skill");
+        fs.mkdirSync(generatedSource, { recursive: true });
+        fs.writeFileSync(path.join(generatedSource, "run.sh"), "#!/bin/sh\n");
+        fs.mkdirSync(generatedWrapper, { recursive: true });
+        fs.symlinkSync(generatedSource, path.join(generatedWrapper, "scripts"), "dir");
+        fs.mkdirSync(localSkill, { recursive: true });
+        fs.writeFileSync(path.join(localSkill, "SKILL.md"), "# local\n");
+        const result = await share({
+            paths: [path.join(tmpDir, ".claude")],
+            entityContext: makeEntityContext({
+                uid: "prs_01PERSONALTEST",
+                slug: "__hq_personal_vault__",
+                bucketName: "hq-vault-prs-01PERSONALTEST",
+            }),
+            hqRoot: tmpDir,
+            personalMode: true,
+            journalSlug: "personal-generated-wrapper",
+        });
+        expect(result.filesUploaded).toBe(1);
+        expect(result.filesExcludedByPolicy).toBeGreaterThanOrEqual(1);
+        expect(uploadSymlink).not.toHaveBeenCalled();
+        expect(uploadFile).toHaveBeenCalledWith(expect.anything(), path.join(localSkill, "SKILL.md"), ".claude/skills/local-skill/SKILL.md", undefined, expect.anything());
+    });
     it("personal-vault override does NOT apply to company vaults: `owned-only` on cmp_ entity still " +
         "refuses to tombstone a direction:'down' entry (multi-user curation intact)", async () => {
         // Sibling guard for the previous test: the override is scoped strictly
@@ -1808,6 +1844,8 @@ describe("share", () => {
                     syncedAt: new Date().toISOString(),
                     direction: "up",
                     remoteEtag: "gone-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("gone-etag", "irrelevant-not-checked-here"),
                 },
             },
         }));
@@ -1842,6 +1880,9 @@ describe("share", () => {
                     size: 42,
                     syncedAt: new Date().toISOString(),
                     direction: "up",
+                    remoteEtag: "removed-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("removed-etag", "h"),
                 },
             },
         }));
@@ -1912,6 +1953,9 @@ describe("share", () => {
                     size: 1,
                     syncedAt: new Date().toISOString(),
                     direction: "up",
+                    remoteEtag: "in-scope-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("in-scope-etag", "h"),
                 },
                 "other/also-gone.md": {
                     hash: "h",
@@ -1954,6 +1998,9 @@ describe("share", () => {
                     size: 5,
                     syncedAt: new Date().toISOString(),
                     direction: "up",
+                    remoteEtag: "flaky-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("flaky-etag", "h"),
                 },
             },
         }));
@@ -2020,6 +2067,9 @@ describe("share", () => {
                 "i-uploaded.md": {
                     hash: "h", size: 1, syncedAt: new Date().toISOString(),
                     direction: "up",
+                    remoteEtag: "uploaded-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("uploaded-etag", "h"),
                 },
             },
         }));
@@ -2050,6 +2100,9 @@ describe("share", () => {
                 "pulled.md": {
                     hash: "h", size: 1, syncedAt: new Date().toISOString(),
                     direction: "down",
+                    remoteEtag: "pulled-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("pulled-etag", "h"),
                 },
             },
         }));
@@ -2084,6 +2137,9 @@ describe("share", () => {
                 "active/current.md": {
                     hash: "h", size: 1, syncedAt: new Date().toISOString(),
                     direction: "up",
+                    remoteEtag: "active-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("active-etag", "h"),
                 },
             },
         }));
@@ -2137,6 +2193,8 @@ describe("share", () => {
                     hash: "h", size: 100, syncedAt: new Date().toISOString(),
                     direction: "down",
                     remoteEtag: "abc123",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("abc123", "h"),
                 },
             },
         }));
@@ -2178,6 +2236,8 @@ describe("share", () => {
                     syncedAt: new Date().toISOString(),
                     direction: "down",
                     remoteEtag: "abc123",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("abc123", "h"),
                     removedAt: new Date().toISOString(),
                     removedReason: "local-delete",
                 },
@@ -2219,6 +2279,8 @@ describe("share", () => {
                     hash: "h", size: 50, syncedAt: new Date().toISOString(),
                     direction: "down",
                     remoteEtag: "stale-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("stale-etag", "h"),
                 },
             },
         }));
@@ -2258,6 +2320,55 @@ describe("share", () => {
         expect(result.filesRefusedStale).toBe(1);
         expect(result.filesTombstoned).toBe(0);
     });
+    it("currency-gated: preserves a file recreated after the delete plan", async () => {
+        const companyRoot = path.join(tmpDir, "companies", "acme");
+        fs.mkdirSync(companyRoot, { recursive: true });
+        const recreatedPath = path.join(companyRoot, "racy.md");
+        const journalPath = path.join(stateDir, "sync-journal.acme.json");
+        fs.writeFileSync(journalPath, JSON.stringify({
+            version: "2",
+            lastSync: new Date().toISOString(),
+            files: {
+                "racy.md": {
+                    hash: "synced-hash",
+                    size: 10,
+                    syncedAt: new Date().toISOString(),
+                    direction: "up",
+                    remoteEtag: "racy-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("racy-etag", "synced-hash"),
+                },
+            },
+            pulls: [],
+        }));
+        // The currency HEAD establishes the remote revision, then a local writer
+        // recreates the file before the planned DeleteObject can run.
+        vi.mocked(headRemoteFile).mockImplementationOnce(async () => {
+            fs.writeFileSync(recreatedPath, "recreated after planning");
+            return { lastModified: new Date(), etag: '"racy-etag"', size: 10 };
+        });
+        const events = [];
+        const result = await share({
+            paths: [companyRoot],
+            company: "acme",
+            vaultConfig: mockConfig,
+            hqRoot: tmpDir,
+            skipUnchanged: true,
+            propagateDeletes: true,
+            propagateDeletePolicy: "currency-gated",
+            onEvent: (e) => events.push(e),
+        });
+        expect(result.filesDeleted).toBe(0);
+        expect(deleteRemoteFile).not.toHaveBeenCalled();
+        expect(fs.readFileSync(recreatedPath, "utf-8")).toBe("recreated after planning");
+        const journal = JSON.parse(fs.readFileSync(journalPath, "utf-8"));
+        expect(journal.files["racy.md"]).toBeDefined();
+        expect(events).toContainEqual(expect.objectContaining({
+            type: "delete-refused-stale-etag",
+            path: "racy.md",
+            reason: "intent-changed",
+        }));
+    });
     it("currency-gated: refuses delete for a divergent-local entry (journal-honesty — never delete a remote the local never genuinely matched)", async () => {
         const companyRoot = path.join(tmpDir, "companies", "acme");
         fs.mkdirSync(companyRoot, { recursive: true });
@@ -2328,6 +2439,8 @@ describe("share", () => {
                     hash: "h", size: 25, syncedAt: new Date().toISOString(),
                     direction: "down",
                     remoteEtag: "doesnt-matter",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("doesnt-matter", "h"),
                 },
             },
         }));
@@ -2396,12 +2509,13 @@ describe("share", () => {
         });
         expect(result.filesDeleted).toBe(0);
         expect(deleteRemoteFile).not.toHaveBeenCalled();
-        // HEAD must NOT be called — we short-circuit on the missing journal etag.
+        // HEAD must NOT be called — the explicit-intent gate rejects legacy
+        // entries before any remote operation.
         expect(headRemoteFile).not.toHaveBeenCalled();
         const refused = events.find((e) => e.type === "delete-refused-stale-etag");
         expect(refused).toMatchObject({
-            journalEtag: "<legacy-no-etag>",
-            reason: "legacy-no-etag",
+            journalEtag: "<missing-delete-intent>",
+            reason: "missing-delete-intent",
         });
         expect(result.filesRefusedStale).toBe(1);
     });
@@ -2424,6 +2538,8 @@ describe("share", () => {
                     hash: "h", size: 200, syncedAt: new Date().toISOString(),
                     direction: "down",
                     remoteEtag: "upstream-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("upstream-etag", "h"),
                 },
             },
         }));
@@ -2525,6 +2641,8 @@ describe("share", () => {
                     hash: "h", size: 5, syncedAt: new Date().toISOString(),
                     direction: "up",
                     remoteEtag: "regular-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("regular-etag", "h"),
                 },
             },
         }));
@@ -2796,6 +2914,8 @@ describe("share", () => {
                     hash: "h", size: 1, syncedAt: new Date().toISOString(),
                     direction: "up",
                     remoteEtag: "abc",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("abc", "h"),
                 },
             },
         }));
@@ -3190,6 +3310,8 @@ describe("share", () => {
                     syncedAt: new Date().toISOString(),
                     direction,
                     remoteEtag: `etag-${i}`,
+                    kind: "file",
+                    localDeleteIntent: deleteIntent(`etag-${i}`, "h"),
                 };
                 if (i >= missing) {
                     fs.writeFileSync(path.join(companyRoot, key), "hi");
@@ -3394,11 +3516,15 @@ describe("share", () => {
                 files[`d-${i.toString().padStart(4, "0")}.md`] = {
                     hash: "h", size: 5, syncedAt: now,
                     direction: "down", remoteEtag: `down-etag-${i}`,
+                    kind: "file",
+                    localDeleteIntent: deleteIntent(`down-etag-${i}`, "h"),
                 };
             }
             files["only-up.md"] = {
                 hash: "h", size: 5, syncedAt: now,
                 direction: "up", remoteEtag: "up-etag",
+                kind: "file",
+                localDeleteIntent: deleteIntent("up-etag", "h"),
             };
             const journalPath = path.join(stateDir, "sync-journal.acme.json");
             fs.writeFileSync(journalPath, JSON.stringify({ version: "1", lastSync: now, files }));
@@ -3534,6 +3660,8 @@ describe("currency-gated: journal version 2 fixtures", () => {
                     syncedAt: new Date().toISOString(),
                     direction: "down",
                     remoteEtag: "v2-etag",
+                    kind: "file",
+                    localDeleteIntent: deleteIntent("v2-etag", "h"),
                 },
             },
             pulls: [],