@indigoai-us/hq-cloud 5.48.4 → 6.0.1

package/src/cli/sync.test.ts

@@ -36,8 +36,15 @@ vi.mock("../s3.js", async () => {
   };
 });
 
+// Mock the post-sync refresh so tests neither spawn the real reindex.sh nor
+// mutate the temp hqRoot, and we can assert the wiring directly.
+vi.mock("./reindex.js", () => ({
+  reindex: vi.fn(() => ({ status: 0 })),
+}));
+
 import { sync } from "./sync.js";
 import * as s3Module from "../s3.js";
+import { reindex } from "./reindex.js";
 
 const mockConfig: VaultServiceConfig = {
   apiUrl: "https://vault-api.test",
@@ -114,6 +121,22 @@ describe("sync", () => {
     delete process.env.HQ_STATE_DIR;
   });
 
+  it("runs reindex against hqRoot after a sync that downloaded files", async () => {
+    await sync({ company: "acme", vaultConfig: mockConfig, hqRoot: tmpDir });
+    expect(reindex).toHaveBeenCalledWith({ repoRoot: tmpDir });
+  });
+
+  it("skips reindex when skipReindex is set", async () => {
+    const result = await sync({
+      company: "acme",
+      vaultConfig: mockConfig,
+      hqRoot: tmpDir,
+      skipReindex: true,
+    });
+    expect(result.filesDownloaded).toBe(2);
+    expect(reindex).not.toHaveBeenCalled();
+  });
+
   it("downloads remote files under companies/{slug}/ so two companies don't collide", async () => {
     const result = await sync({
       company: "acme",

package/src/cli/sync.ts

@@ -49,6 +49,7 @@ import {
   readShortMachineId,
 } from "../lib/conflict-file.js";
 import { appendConflictEntry } from "../lib/conflict-index.js";
+import { reindex } from "./reindex.js";
 
 /**
  * Per-file events emitted by `sync()` as it progresses.
@@ -329,6 +330,15 @@ export interface SyncOptions {
    * tombstoned. Mirrors `hq sync narrow --force`.
    */
   forceScopeShrink?: boolean;
+  /**
+   * Skip the post-sync `reindex()` refresh (skill wrappers + personal overlay
+   * mirrors + workers registry). By default, when a sync changes on-disk
+   * sources (downloads, tombstones, or scope-orphan removals), `sync()`
+   * re-runs reindex so the generated `.claude/skills/<ns>:<skill>` wrappers
+   * stay in sync. An orchestrator syncing many companies in one pass can set
+   * this and run `reindex()` once itself instead of per-company.
+   */
+  skipReindex?: boolean;
 }
 
 export interface SyncResult {
@@ -1263,6 +1273,23 @@ export async function sync(options: SyncOptions): Promise<SyncResult> {
   journal.lastSync = new Date().toISOString();
   writeJournal(journalSlug, journal);
 
+  // When the pull actually changed on-disk sources (new files, tombstoned
+  // removals, or scope-orphan cleanups), refresh the generated skill wrappers,
+  // personal-overlay mirrors, and workers registry. reindex is idempotent and
+  // best-effort — it must never fail a sync, and is skipped on no-op syncs
+  // (the common daemon case) and when the caller opts out via skipReindex.
+  const changedOnDisk =
+    filesDownloaded > 0 ||
+    filesTombstoned > 0 ||
+    shrinkResult.cleanRemoved > 0;
+  if (!options.skipReindex && changedOnDisk) {
+    try {
+      reindex({ repoRoot: hqRoot });
+    } catch {
+      // best-effort: a post-sync refresh failure never fails the sync
+    }
+  }
+
   return {
     filesDownloaded,
     bytesDownloaded,

package/src/index.ts

@@ -212,9 +212,9 @@ export type { AcceptOptions, AcceptResult } from "./cli/index.js";
 export { promote } from "./cli/index.js";
 export type { PromoteOptions, PromoteResult } from "./cli/index.js";
 
-// Skill/personal-overlay mirroring + workers-registry regen (`hq master-sync`).
-export { masterSync, masterSyncScriptPath } from "./cli/index.js";
-export type { MasterSyncOptions, MasterSyncResult } from "./cli/index.js";
+// Skill/personal-overlay mirroring + workers-registry regen (`hq reindex`).
+export { reindex, reindexScriptPath } from "./cli/index.js";
+export type { ReindexOptions, ReindexResult } from "./cli/index.js";
 
 // Drift-preserving HQ-core re-sync — shared by the HQ Sync app and `hq rescue`.
 export { rescue, rescueScriptPath, buildRescueArgs } from "./cli/index.js";

package/dist/cli/master-sync.d.ts

@@ -1,22 +0,0 @@
-/**
- * Absolute path to the bundled master-sync.sh. From the compiled module at
- * dist/cli/master-sync.js, the package root is two levels up; the script lives
- * at <package-root>/scripts/master-sync.sh.
- */
-export declare function masterSyncScriptPath(): string;
-export interface MasterSyncOptions {
-    /** HQ root to operate on. Defaults to process.cwd(). */
-    repoRoot?: string;
-}
-export interface MasterSyncResult {
-    /** Exit status of the underlying script (0 = success). */
-    status: number;
-}
-/**
- * Run master-sync against an HQ root. Synchronous — the script is cheap and
- * idempotent, and callers (the hook shim, tests) want the exit status.
- * stdout/stderr from the script are forwarded to stderr so the caller's stdout
- * stays clean (hooks must not emit stdout that the agent interprets).
- */
-export declare function masterSync(opts?: MasterSyncOptions): MasterSyncResult;
-//# sourceMappingURL=master-sync.d.ts.map

package/dist/cli/master-sync.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"master-sync.d.ts","sourceRoot":"","sources":["../../src/cli/master-sync.ts"],"names":[],"mappings":"AAmBA;;;;GAIG;AACH,wBAAgB,oBAAoB,IAAI,MAAM,CAE7C;AAED,MAAM,WAAW,iBAAiB;IAChC,wDAAwD;IACxD,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,gBAAgB;IAC/B,0DAA0D;IAC1D,MAAM,EAAE,MAAM,CAAC;CAChB;AAED;;;;;GAKG;AACH,wBAAgB,UAAU,CAAC,IAAI,GAAE,iBAAsB,GAAG,gBAAgB,CAWzE"}

package/dist/cli/master-sync.js

@@ -1,44 +0,0 @@
-/**
- * hq master-sync — surfaces namespaced skills as Claude Code skill wrappers,
- * mirrors personal/{knowledge,policies,workers,settings} into core/, prunes
- * orphan wrappers, and regenerates the workers registry.
- *
- * The implementation lives in scripts/master-sync.sh, shipped with this
- * package. This module resolves that script relative to the package (dist/cli
- * → package root) and execs it against the caller's HQ root. Historically the
- * script ran directly as a Claude Code hook inside hq-core; it now lives here
- * so a single copy is maintained, and the hq-core hook is a thin shim over
- * `hq master-sync`.
- */
-import { spawnSync } from "child_process";
-import { fileURLToPath } from "url";
-import path from "path";
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(__filename);
-/**
- * Absolute path to the bundled master-sync.sh. From the compiled module at
- * dist/cli/master-sync.js, the package root is two levels up; the script lives
- * at <package-root>/scripts/master-sync.sh.
- */
-export function masterSyncScriptPath() {
-    return path.resolve(__dirname, "..", "..", "scripts", "master-sync.sh");
-}
-/**
- * Run master-sync against an HQ root. Synchronous — the script is cheap and
- * idempotent, and callers (the hook shim, tests) want the exit status.
- * stdout/stderr from the script are forwarded to stderr so the caller's stdout
- * stays clean (hooks must not emit stdout that the agent interprets).
- */
-export function masterSync(opts = {}) {
-    const repoRoot = opts.repoRoot ?? process.cwd();
-    const script = masterSyncScriptPath();
-    const res = spawnSync("bash", [script, repoRoot], {
-        stdio: ["ignore", "inherit", "inherit"],
-    });
-    if (res.error) {
-        process.stderr.write(`master-sync: failed to run ${script}: ${res.error.message}\n`);
-        return { status: 1 };
-    }
-    return { status: res.status ?? 1 };
-}
-//# sourceMappingURL=master-sync.js.map

package/dist/cli/master-sync.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"master-sync.js","sourceRoot":"","sources":["../../src/cli/master-sync.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AACH,OAAO,EAAE,SAAS,EAAE,MAAM,eAAe,CAAC;AAC1C,OAAO,EAAE,aAAa,EAAE,MAAM,KAAK,CAAC;AACpC,OAAO,IAAI,MAAM,MAAM,CAAC;AAExB,MAAM,UAAU,GAAG,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AAClD,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;AAE3C;;;;GAIG;AACH,MAAM,UAAU,oBAAoB;IAClC,OAAO,IAAI,CAAC,OAAO,CAAC,SAAS,EAAE,IAAI,EAAE,IAAI,EAAE,SAAS,EAAE,gBAAgB,CAAC,CAAC;AAC1E,CAAC;AAYD;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,OAA0B,EAAE;IACrD,MAAM,QAAQ,GAAG,IAAI,CAAC,QAAQ,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC;IAChD,MAAM,MAAM,GAAG,oBAAoB,EAAE,CAAC;IACtC,MAAM,GAAG,GAAG,SAAS,CAAC,MAAM,EAAE,CAAC,MAAM,EAAE,QAAQ,CAAC,EAAE;QAChD,KAAK,EAAE,CAAC,QAAQ,EAAE,SAAS,EAAE,SAAS,CAAC;KACxC,CAAC,CAAC;IACH,IAAI,GAAG,CAAC,KAAK,EAAE,CAAC;QACd,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,8BAA8B,MAAM,KAAK,GAAG,CAAC,KAAK,CAAC,OAAO,IAAI,CAAC,CAAC;QACrF,OAAO,EAAE,MAAM,EAAE,CAAC,EAAE,CAAC;IACvB,CAAC;IACD,OAAO,EAAE,MAAM,EAAE,GAAG,CAAC,MAAM,IAAI,CAAC,EAAE,CAAC;AACrC,CAAC"}

package/dist/cli/master-sync.test.d.ts

@@ -1,11 +0,0 @@
-/**
- * Unit tests for `hq master-sync` (the formerly-bash hq-core hook, now shipped
- * in this package and invoked via the CLI).
- *
- * The logic itself lives in scripts/master-sync.sh; these tests exercise it via
- * the masterSync() wrapper against a temp HQ tree, asserting the observable
- * filesystem outcomes (skill wrappers, personal-overlay mirroring) rather than
- * re-deriving the bash internals.
- */
-export {};
-//# sourceMappingURL=master-sync.test.d.ts.map

package/dist/cli/master-sync.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"master-sync.test.d.ts","sourceRoot":"","sources":["../../src/cli/master-sync.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG"}

package/dist/cli/master-sync.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"master-sync.test.js","sourceRoot":"","sources":["../../src/cli/master-sync.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,SAAS,EAAE,MAAM,QAAQ,CAAC;AACrE,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,KAAK,IAAI,MAAM,MAAM,CAAC;AAC7B,OAAO,KAAK,EAAE,MAAM,IAAI,CAAC;AACzB,OAAO,EAAE,UAAU,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAC;AAEpE,QAAQ,CAAC,sBAAsB,EAAE,GAAG,EAAE;IACpC,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC5D,MAAM,CAAC,GAAG,oBAAoB,EAAE,CAAC;QACjC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,gBAAgB,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACtE,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,YAAY,EAAE,GAAG,EAAE;IAC1B,IAAI,IAAY,CAAC;IAEjB,UAAU,CAAC,GAAG,EAAE;QACd,IAAI,GAAG,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,MAAM,EAAE,EAAE,UAAU,CAAC,CAAC,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,SAAS,CAAC,GAAG,EAAE;QACb,EAAE,CAAC,MAAM,CAAC,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;IACpD,CAAC,CAAC,CAAC;IAEH,SAAS,UAAU,CAAC,GAAW;QAC7B,MAAM,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;QACjC,EAAE,CAAC,SAAS,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACvC,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,UAAU,CAAC,EAAE,SAAS,CAAC,CAAC;IAC1D,CAAC;IAED,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC7E,UAAU,CAAC,kBAAkB,CAAC,CAAC;QAC/B,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,4BAA4B,CAAC,EAAE,KAAK,CAAC,CAAC;QACvE,UAAU,CAAC,8BAA8B,CAAC,CAAC;QAE3C,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;QAClD,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAEvB,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,0BAA0B,CAAC,CAAC;QAChE,MAAM,CAAC,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC,CAAC,cAAc,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACrF,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,IAAI,CAC9D,oCAAoC,CACrC,CAAC;QACF,oDAAoD;QACpD,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,WAAW,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAEtE,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,4BAA4B,CAAC,CAAC;QAClE,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC,CAAC,CAAC,IAAI,CAC9D,gDAAgD,CACjD,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACxD,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,mBAAmB,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACxE,EAAE,CAAC,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,6BAA6B,CAAC,EAAE,QAAQ,CAAC,CAAC;QAE3E,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;QAClD,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAEvB,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,yBAAyB,CAAC,CAAC;QACxD,MAAM,CAAC,EAAE,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,cAAc,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACvD,MAAM,CAAC,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,mCAAmC,CAAC,CAAC;IAC1E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,UAAU,CAAC,kBAAkB,CAAC,CAAC;QAC/B,UAAU,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/B,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,0BAA0B,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAE9E,EAAE,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,kBAAkB,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACjF,UAAU,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC;QAC/B,MAAM,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,0BAA0B,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACjF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,UAAU,CAAC,kBAAkB,CAAC,CAAC;QAC/B,MAAM,CAAC,UAAU,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACtD,MAAM,CAAC,UAAU,CAAC,EAAE,QAAQ,EAAE,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACtD,MAAM,CACJ,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,mCAAmC,CAAC,CAAC,CACtE,CAAC,IAAI,CAAC,oCAAoC,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/src/cli/master-sync.ts

@@ -1,56 +0,0 @@
-/**
- * hq master-sync — surfaces namespaced skills as Claude Code skill wrappers,
- * mirrors personal/{knowledge,policies,workers,settings} into core/, prunes
- * orphan wrappers, and regenerates the workers registry.
- *
- * The implementation lives in scripts/master-sync.sh, shipped with this
- * package. This module resolves that script relative to the package (dist/cli
- * → package root) and execs it against the caller's HQ root. Historically the
- * script ran directly as a Claude Code hook inside hq-core; it now lives here
- * so a single copy is maintained, and the hq-core hook is a thin shim over
- * `hq master-sync`.
- */
-import { spawnSync } from "child_process";
-import { fileURLToPath } from "url";
-import path from "path";
-
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(__filename);
-
-/**
- * Absolute path to the bundled master-sync.sh. From the compiled module at
- * dist/cli/master-sync.js, the package root is two levels up; the script lives
- * at <package-root>/scripts/master-sync.sh.
- */
-export function masterSyncScriptPath(): string {
-  return path.resolve(__dirname, "..", "..", "scripts", "master-sync.sh");
-}
-
-export interface MasterSyncOptions {
-  /** HQ root to operate on. Defaults to process.cwd(). */
-  repoRoot?: string;
-}
-
-export interface MasterSyncResult {
-  /** Exit status of the underlying script (0 = success). */
-  status: number;
-}
-
-/**
- * Run master-sync against an HQ root. Synchronous — the script is cheap and
- * idempotent, and callers (the hook shim, tests) want the exit status.
- * stdout/stderr from the script are forwarded to stderr so the caller's stdout
- * stays clean (hooks must not emit stdout that the agent interprets).
- */
-export function masterSync(opts: MasterSyncOptions = {}): MasterSyncResult {
-  const repoRoot = opts.repoRoot ?? process.cwd();
-  const script = masterSyncScriptPath();
-  const res = spawnSync("bash", [script, repoRoot], {
-    stdio: ["ignore", "inherit", "inherit"],
-  });
-  if (res.error) {
-    process.stderr.write(`master-sync: failed to run ${script}: ${res.error.message}\n`);
-    return { status: 1 };
-  }
-  return { status: res.status ?? 1 };
-}