@indigoai-us/hq-cloud 6.11.12 → 6.11.13

package/src/watcher.ts

@@ -2,9 +2,8 @@
  * File watcher — monitors HQ directory for changes
  * Uses chokidar with debounced batching
  *
- * Day 1: not invoked by CLI surface; retained for future automatic-sync milestone.
- * When re-enabled, the constructor will need an EntityContext (or a context resolver)
- * to be passed in for entity-aware S3 operations.
+ * Active watcher path: TreeWatcher detects local changes, WatchPushDriver
+ * schedules pushes, and PushEventEmitter publishes typed push events.
  */
 
 import * as fs from "fs";
@@ -12,12 +11,7 @@ import { createHash } from "node:crypto";
 import { readFile, stat } from "node:fs/promises";
 import * as path from "path";
 import { watch } from "chokidar";
-import type { FSWatcher } from "chokidar";
-import type { EntityContext } from "./types.js";
-import { createIgnoreFilter, isWithinSizeLimit } from "./ignore.js";
-import { readJournal, writeJournal, hashFile, updateEntry } from "./journal.js";
-import { uploadFile, deleteRemoteFile, toPosixKey } from "./s3.js";
-import type { UploadAuthor } from "./s3.js";
+import { createIgnoreFilter } from "./ignore.js";
 import { isPersonalVaultExcluded } from "./personal-vault-exclusions.js";
 import {
   CONTINUITY_POINTER_REL,
@@ -205,141 +199,12 @@ export class WatchPushDriver {
   }
 }
 
-interface PendingChange {
-  type: "add" | "change" | "unlink";
-  absolutePath: string;
-  relativePath: string;
-}
-
-export class SyncWatcher {
-  private watcher: FSWatcher | null = null;
-  private hqRoot: string;
-  private ctx: EntityContext;
-  private author?: UploadAuthor;
-  private shouldSync: (filePath: string, isDir?: boolean) => boolean;
-  private pendingChanges = new Map<string, PendingChange>();
-  private debounceTimer: ReturnType<typeof setTimeout> | null = null;
-  private processing = false;
-
-  constructor(hqRoot: string, ctx: EntityContext, author?: UploadAuthor) {
-    this.hqRoot = hqRoot;
-    this.ctx = ctx;
-    this.author = author;
-    this.shouldSync = createIgnoreFilter(hqRoot);
-  }
-
-  start(): void {
-    if (this.watcher) return;
-
-    this.watcher = watch(this.hqRoot, {
-      // See toChokidarIgnored: chokidar's pre-stat descent probe has no stats
-      // hint, so a naive file-verdict prunes intermediate allowlist dirs
-      // before descending to their in-scope leaves. Keep a statless probe
-      // when EITHER the file or directory reading would survive the filter.
-      ignored: toChokidarIgnored(this.shouldSync, this.hqRoot),
-      persistent: true,
-      ignoreInitial: true,
-      awaitWriteFinish: {
-        stabilityThreshold: 500,
-        pollInterval: 100,
-      },
-    });
-
-    this.watcher
-      .on("add", (p) => this.queueChange("add", p))
-      .on("change", (p) => this.queueChange("change", p))
-      .on("unlink", (p) => this.queueChange("unlink", p))
-      .on("error", (err) => console.error("Watcher error:", err));
-  }
-
-  stop(): void {
-    if (this.watcher) {
-      this.watcher.close();
-      this.watcher = null;
-    }
-    if (this.debounceTimer) {
-      clearTimeout(this.debounceTimer);
-      this.debounceTimer = null;
-    }
-  }
-
-  private queueChange(type: "add" | "change" | "unlink", absolutePath: string): void {
-    const relativePath = toPosixKey(path.relative(this.hqRoot, absolutePath));
-
-    // Skip files that exceed size limit
-    if (type !== "unlink" && !isWithinSizeLimit(absolutePath)) {
-      return;
-    }
-
-    this.pendingChanges.set(relativePath, {
-      type,
-      absolutePath,
-      relativePath,
-    });
-
-    // Debounce: wait for DEBOUNCE_MS of quiet before processing
-    if (this.debounceTimer) {
-      clearTimeout(this.debounceTimer);
-    }
-    this.debounceTimer = setTimeout(() => this.flush(), DEBOUNCE_MS);
-  }
-
-  private async flush(): Promise<void> {
-    if (this.processing || this.pendingChanges.size === 0) return;
-    this.processing = true;
-
-    const batch = new Map(this.pendingChanges);
-    this.pendingChanges.clear();
-
-    const journal = readJournal(this.ctx.slug);
-
-    for (const [relativePath, change] of batch) {
-      try {
-        if (change.type === "unlink") {
-          await deleteRemoteFile(this.ctx, relativePath);
-          delete journal.files[relativePath];
-        } else {
-          const hash = hashFile(change.absolutePath);
-          const stat = fs.statSync(change.absolutePath);
-
-          // Skip if unchanged from last sync
-          const existing = journal.files[relativePath];
-          if (existing && existing.hash === hash) continue;
-
-          const { etag } = this.author
-            ? await uploadFile(this.ctx, change.absolutePath, relativePath, this.author)
-            : await uploadFile(this.ctx, change.absolutePath, relativePath);
-          updateEntry(journal, relativePath, hash, stat.size, "up", etag);
-        }
-      } catch (err) {
-        console.error(
-          `Sync error [${relativePath}]:`,
-          err instanceof Error ? err.message : err
-        );
-        // Re-queue failed changes
-        this.pendingChanges.set(relativePath, change);
-      }
-    }
-
-    // See cli/sync.ts: stamp lastSync on every flush so the indicator
-    // ticks even when all changes were re-queued or no-op.
-    journal.lastSync = new Date().toISOString();
-    writeJournal(this.ctx.slug, journal);
-    this.processing = false;
-
-    // Process any changes that came in while we were flushing
-    if (this.pendingChanges.size > 0) {
-      this.debounceTimer = setTimeout(() => this.flush(), DEBOUNCE_MS);
-    }
-  }
-}
-
 // ---------------------------------------------------------------------------
 // US-002 — debounced, ignore-aware, exclusion-aware tree watcher.
 //
-// Unlike SyncWatcher (which does S3 work inline), TreeWatcher is a pure change
-// detector: it emits a single debounced `changed` callback after a quiet window
-// and never touches S3 itself. US-003 wires that callback to a targeted push.
+// TreeWatcher is a pure change detector: it emits a single debounced `changed`
+// callback after a quiet window and never touches S3 itself. US-003 wires that
+// callback to a targeted push.
 //
 // The emit decision composes the SAME filter stack the push walk uses, so a
 // path that the push would skip never wakes the watcher: