@gramatr/client 0.5.1 → 0.5.2

package/bin/install.ts

@@ -284,19 +284,43 @@ function installClientFiles(): void {
   copyFileIfExists(join(SCRIPT_DIR, 'bin/render-claude-hooks.ts'), join(CLIENT_DIR, 'bin/render-claude-hooks.ts'), true);
 
   // Bin lib helpers (statusline shim depends on these — see #495)
-  // Copies git.ts, config.ts, stdin.ts and any future helpers without enumerating each file.
+  // Walks the directory but filters out test files / __tests__ subdir so
+  // we don't ship .test.ts to user installs.
   const binLibSrc = join(SCRIPT_DIR, 'bin', 'lib');
   if (existsSync(binLibSrc)) {
-    copyDir(binLibSrc, join(CLIENT_DIR, 'bin', 'lib'));
-    log('OK Installed bin/lib (statusline shim helpers)');
+    mkdirSync(join(CLIENT_DIR, 'bin', 'lib'), { recursive: true });
+    let binLibCount = 0;
+    for (const f of readdirSync(binLibSrc)) {
+      const src = join(binLibSrc, f);
+      if (!statSync(src).isFile()) continue;
+      if (!f.endsWith('.ts')) continue;
+      if (f.endsWith('.test.ts')) continue;
+      copyFileIfExists(src, join(CLIENT_DIR, 'bin', 'lib', f));
+      binLibCount++;
+    }
+    log(`OK Installed bin/lib (${binLibCount} statusline shim helpers)`);
   }
   log('OK Installed bin (statusline, gmtr-login, render-claude-hooks)');
 
-  // Core dependencies (routing.ts required by GMTRPromptEnricher hook)
-  for (const f of ['install.ts', 'version.ts', 'types.ts', 'routing.ts']) {
-    copyFileIfExists(join(SCRIPT_DIR, 'core', f), join(CLIENT_DIR, 'core', f));
+  // Core dependencies — directory walk so new hook deps don't silently break.
+  // Hook imports include: routing.ts (enricher), session.ts (session-start),
+  // version-check.ts (session-start), version.ts (session-start), types.ts (transitive).
+  // Past bugs (#495 bin/lib, this same class repeated): hardcoded file lists drift
+  // when a hook grows a new core import. Walking the directory + filtering tests
+  // makes the installer self-healing.
+  const coreSrc = join(SCRIPT_DIR, 'core');
+  if (existsSync(coreSrc)) {
+    let coreCount = 0;
+    for (const f of readdirSync(coreSrc)) {
+      const src = join(coreSrc, f);
+      if (!statSync(src).isFile()) continue;
+      if (!f.endsWith('.ts')) continue;
+      if (f.endsWith('.test.ts')) continue;
+      copyFileIfExists(src, join(CLIENT_DIR, 'core', f));
+      coreCount++;
+    }
+    log(`OK Installed ${coreCount} core modules`);
   }
-  log('OK Installed core modules');
 
   // package.json — copied so core/version.ts can resolve the installed
   // version at runtime. Single source of truth; see core/version.ts.
@@ -594,7 +618,11 @@ function verify(url: string, token: string): boolean {
     }
   };
 
-  // Critical files
+  // Critical files — every runtime hook + bin import target must be present.
+  // If you add a new ../core/X.ts or ./lib/X.ts import in hooks/ or bin/, add the
+  // corresponding entry here so a partial install fails loudly instead of
+  // exploding at hook execution time. The hook-imports-shipped.test.ts
+  // regression guard catches missing entries at PR review.
   for (const f of [
     'hooks/GMTRToolTracker.hook.ts',
     'hooks/GMTRPromptEnricher.hook.ts',
@@ -602,8 +630,14 @@ function verify(url: string, token: string): boolean {
     'hooks/GMTRSecurityValidator.hook.ts',
     'hooks/lib/notify.ts',
     'core/routing.ts',
+    'core/session.ts',
+    'core/version.ts',
+    'core/version-check.ts',
+    'core/types.ts',
+    'core/install.ts',
     'bin/statusline.ts',
     'bin/gmtr-login.ts',
+    'bin/render-claude-hooks.ts',
     'bin/lib/git.ts',
     'bin/lib/config.ts',
     'bin/lib/stdin.ts',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gramatr/client",
-  "version": "0.5.1",
+  "version": "0.5.2",
   "publishConfig": {
     "access": "public"
   },