@indigoai-us/hq-cli 5.345.97 → 5.345.98

package/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## [Unreleased]
 
+## [5.345.98] — 2026-10-12
+- Session-start health no longer files a leftover personal-files journal plus
+  an ordinary HQ Core upgrade as failed doctor repairs. `hq doctor --fix`
+  still refreshes personal files; the upgrade warning stays in `hq doctor`.
+
 ## [5.345.97] — 2026-10-12
 - Session-start health no longer files a leftover company journal plus an
   ordinary HQ Core upgrade as failed doctor repairs. `hq doctor --fix` still

package/dist/lib/doctor/checks/sync-health.js

@@ -392,26 +392,75 @@ function journalAndManifestResults(deps) {
     }
     return [...journalResults(deps, journals), ...manifestResults(deps, journals)];
 }
+function leftoverPersonalJournalResult(entry) {
+    return {
+        status: "NA",
+        checkId: `sync.journal.${journalSlugForCheckId(PERSONAL_SCOPE_SLUG)}`,
+        reasonCode: "legacy-personal-journal",
+        target: entry.path,
+        message: "Sync journal 'personal' is unused on this machine — personal files now sync through the personal vault journal, so this leftover shard is not a live sync target.",
+    };
+}
+function journalFreshnessResult(entry, slug, nowMs) {
+    const lastSync = entry.journal?.lastSync;
+    const checkId = `sync.journal.${journalSlugForCheckId(slug)}`;
+    if (typeof lastSync !== "string" || lastSync.length === 0) {
+        return {
+            status: "WARN",
+            checkId,
+            reasonCode: "never-synced",
+            target: entry.path,
+            message: `Sync journal '${slug}' exists but has never recorded a successful sync.`,
+            remediation: "Run /hq-sync (or `hq sync`) to complete a first sync.",
+        };
+    }
+    const parsed = Date.parse(lastSync);
+    if (!Number.isFinite(parsed)) {
+        return {
+            status: "UNKNOWN",
+            checkId,
+            reasonCode: "invalid-last-sync",
+            target: entry.path,
+            message: `Sync journal '${slug}' has an unparseable lastSync (${lastSync}).`,
+        };
+    }
+    const ageMs = nowMs - parsed;
+    if (ageMs > STALE_JOURNAL_THRESHOLD_MS) {
+        const days = Math.floor(ageMs / 86_400_000);
+        return {
+            status: "WARN",
+            checkId,
+            reasonCode: "stale-threshold",
+            target: entry.path,
+            message: `Sync journal '${slug}' is stale: last successful sync ${days} day${days === 1 ? "" : "s"} ago (${lastSync}).`,
+            remediation: "Run /hq-sync (or `hq sync`) and check the sync runner.",
+        };
+    }
+    return {
+        status: "PASS",
+        checkId,
+        target: entry.path,
+        message: `Sync journal '${slug}' is fresh (last sync ${lastSync}).`,
+    };
+}
 function journalResults(deps, journals) {
     const nowMs = deps.now().getTime();
     const hasPersonalVaultJournal = journals.some((entry) => entry.slug === PERSONAL_VAULT_SCOPE_SLUG);
-    return journals.map((entry) => {
-        const lastSync = entry.journal?.lastSync;
-        const checkId = `sync.journal.${journalSlugForCheckId(entry.slug)}`;
+    const results = [];
+    for (const entry of journals) {
         // The legacy `personal` shard and `__hq_personal_vault__` name the same
-        // tree. Once the vault journal exists, the old file is leftover
-        // bookkeeping: WARN would make `hq doctor --fix` look like it failed to
-        // repair a row it cannot refresh (a personal pull writes the vault
-        // journal, not this one).
-        if (entry.slug === PERSONAL_SCOPE_SLUG && hasPersonalVaultJournal) {
-            return {
-                status: "NA",
-                checkId,
-                reasonCode: "legacy-personal-journal",
-                target: entry.path,
-                message: "Sync journal 'personal' is unused on this machine — personal files now sync through the personal vault journal, so this leftover shard is not a live sync target.",
-            };
+        // tree. A personal pull writes the vault journal, not this leftover file,
+        // so WARN on `sync.journal.personal` makes `hq doctor --fix` look like it
+        // failed to repair a row it cannot refresh — including when the vault
+        // journal has not been created yet.
+        if (entry.slug === PERSONAL_SCOPE_SLUG) {
+            results.push(leftoverPersonalJournalResult(entry));
+            if (!hasPersonalVaultJournal) {
+                results.push(journalFreshnessResult(entry, PERSONAL_VAULT_SCOPE_SLUG, nowMs));
+            }
+            continue;
         }
+        const checkId = `sync.journal.${journalSlugForCheckId(entry.slug)}`;
         // A company journal without a local tree is leftover membership state,
         // not a live sync target. WARN would make `hq doctor --fix` look like it
         // failed to repair something it was never going to touch (US-015 health
@@ -427,54 +476,19 @@ function journalResults(deps, journals) {
                 local = true;
             }
             if (!local) {
-                return {
+                results.push({
                     status: "NA",
                     checkId,
                     reasonCode: "no-local-tree",
                     target: entry.path,
                     message: `Sync journal '${entry.slug}' is unused on this machine — there is no companies/${entry.slug} directory, so this journal is not a live sync target.`,
-                };
+                });
+                continue;
             }
         }
-        if (typeof lastSync !== "string" || lastSync.length === 0) {
-            return {
-                status: "WARN",
-                checkId,
-                reasonCode: "never-synced",
-                target: entry.path,
-                message: `Sync journal '${entry.slug}' exists but has never recorded a successful sync.`,
-                remediation: "Run /hq-sync (or `hq sync`) to complete a first sync.",
-            };
-        }
-        const parsed = Date.parse(lastSync);
-        if (!Number.isFinite(parsed)) {
-            return {
-                status: "UNKNOWN",
-                checkId,
-                reasonCode: "invalid-last-sync",
-                target: entry.path,
-                message: `Sync journal '${entry.slug}' has an unparseable lastSync (${lastSync}).`,
-            };
-        }
-        const ageMs = nowMs - parsed;
-        if (ageMs > STALE_JOURNAL_THRESHOLD_MS) {
-            const days = Math.floor(ageMs / 86_400_000);
-            return {
-                status: "WARN",
-                checkId,
-                reasonCode: "stale-threshold",
-                target: entry.path,
-                message: `Sync journal '${entry.slug}' is stale: last successful sync ${days} day${days === 1 ? "" : "s"} ago (${lastSync}).`,
-                remediation: "Run /hq-sync (or `hq sync`) and check the sync runner.",
-            };
-        }
-        return {
-            status: "PASS",
-            checkId,
-            target: entry.path,
-            message: `Sync journal '${entry.slug}' is fresh (last sync ${lastSync}).`,
-        };
-    });
+        results.push(journalFreshnessResult(entry, entry.slug, nowMs));
+    }
+    return results;
 }
 // ─── Manifest upload freshness (sync-reconciliation-audit US-004) ────────────
 /**

package/dist/lib/doctor/fix/remediation.js

@@ -130,8 +130,9 @@ export function deriveRemediation(result) {
         };
     }
     // refresh-sync — a company journal that is stale or never-synced, or a
-    // live personal / personal-vault journal. Leftover shards are NA at the
-    // check layer, so they never reach this classifier. A SessionStart
+    // live personal / personal-vault journal. Leftover `personal` shards are
+    // NA at the check layer (including when the vault journal is not on disk
+    // yet), so they never reach this classifier. A SessionStart
     // `--fix --yes` pull is keep-conflicts and pull-only, matching the
     // company repair: leaving personal journals WARN made the health hook
     // report an unrepaired failure for a row doctor refused to touch.

package/dist/lib/doctor/json-output.d.ts

@@ -106,11 +106,13 @@ export interface BuildDoctorJsonInput {
     clientHealthCheckStatusEnabledCheckIds?: ReadonlySet<string>;
     /**
      * JSON consumed by the SessionStart health hook (`--client-health-report`).
-     * Ordinary hq-core upgrade WARNs stay in human `hq doctor` output and in
-     * default `--json`, but they are omitted here: `--fix` cannot apply a
-     * release, and older hooks treated the leftover WARN as an unrepaired
-     * failure. Company journals this `--fix` pass did not try to refresh are
-     * omitted the same way: the repair cap leaves them for a later session.
+     * Ordinary hq-core upgrade WARNs are omitted from every JSON document
+     * (`--json` and `--client-health-report`): `--fix` cannot apply a release,
+     * and both current hooks and older ones that only call `hq doctor --json`
+     * treated the leftover WARN as an unrepaired failure. Human `hq doctor`
+     * still prints the upgrade. Company journals this `--fix` pass did not try
+     * to refresh are omitted only from `--client-health-report`: the repair cap
+     * leaves them for a later session.
      */
     clientHealthReport?: boolean;
     /** Clock for the leftover-journal freshness window. Default: `Date.now`. */
@@ -119,6 +121,8 @@ export interface BuildDoctorJsonInput {
 /**
  * True when a sync finding is an ordinary, non-repairable hq-core upgrade.
  * FAIL stays visible: that is a broken local core, not "a newer release exists".
+ * JSON omits the WARN so SessionStart health (including core 15.x hooks that
+ * only call `hq doctor --json`) does not file it as a failed repair.
  */
 export declare function isOrdinaryCoreUpdateWarn(result: {
     checkId: string;

package/dist/lib/doctor/json-output.js

@@ -33,6 +33,8 @@ export const DOCTOR_JSON_SCHEMA_VERSION = 2;
 /**
  * True when a sync finding is an ordinary, non-repairable hq-core upgrade.
  * FAIL stays visible: that is a broken local core, not "a newer release exists".
+ * JSON omits the WARN so SessionStart health (including core 15.x hooks that
+ * only call `hq doctor --json`) does not file it as a failed repair.
  */
 export function isOrdinaryCoreUpdateWarn(result) {
     return result.checkId === "sync.update.core" && result.status === "WARN";
@@ -45,7 +47,7 @@ export function buildDoctorJson(input) {
     const results = [];
     for (const run of input.families) {
         for (const result of run.results) {
-            if (input.clientHealthReport === true && isOrdinaryCoreUpdateWarn(result)) {
+            if (isOrdinaryCoreUpdateWarn(result)) {
                 continue;
             }
             if (input.clientHealthReport === true &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@indigoai-us/hq-cli",
-  "version": "5.345.97",
+  "version": "5.345.98",
   "description": "HQ by Indigo management CLI — modules and cloud sync",
   "main": "dist/index.js",
   "bin": {