@goodfoot/agent-hooks 1.0.9 → 1.0.10

package/CHANGELOG.md

@@ -1,9 +1,13 @@
 # @goodfoot/agent-hooks npm package changelog
 
-## 1.0.9
-- Added support for Antigravity hooks, including event handling, output formatting, and transport
-- Added scaffolding for Antigravity hook projects via the CLI
-- Documented the Antigravity hook contract in the README
+## 1.0.10
+- Fixed scaffolded hook bundles to work when the install path is a symlink or otherwise differs from the project root, so hooks resolve correctly across repository layouts
+- Improved stack traces from hooks by making source map resolution reliable in relocated or symlinked installs
+- Applied the portability fixes consistently across the Claude Code, Codex, and Antigravity entry points
+
+## 1.0.10
+
+Makes compiled bundles portable across ordinary hoisted workspace layouts by anchoring esbuild's module-boundary comments and sourcemap source paths at the directory that owns the resolved `node_modules`. Inline sourcemaps are now disabled by default for smaller, reviewable build artifacts and remain available with the explicit `--sourcemap` flag.
 
 ## 1.0.9
 

package/README.md

@@ -2,6 +2,8 @@
 
 **Build AI coding agent hooks in TypeScript — one package, any supported agent.**
 
+Compiled bundles omit inline sourcemaps by default. Pass `--sourcemap` when original TypeScript locations are needed in runtime stack traces.
+
 `@goodfoot/agent-hooks` is the unified successor to `@goodfoot/claude-code-hooks` and
 `@goodfoot/codex-hooks`: a shared agent-neutral core (hook factory primitive, transport
 driver, `Logger`, env/stdin utilities) with a per-agent entry point on top. Write

package/dist/agents/antigravity/cli-support.js

@@ -22,7 +22,7 @@ import * as path from "node:path";
 import { fileURLToPath } from "node:url";
 import * as esbuild from "esbuild";
 import ts from "typescript";
-import { buildEsbuildLoaderMap, discoverHookSourceFiles, parseEsbuildLoaderFlag, pruneStaleHashedBundles, sha256Prefix8, symlinkVisiblePath, } from "../../cli.js";
+import { buildEsbuildLoaderMap, discoverHookSourceFiles, moduleWorkingDir, parseEsbuildLoaderFlag, pruneStaleHashedBundles, sha256Prefix8, symlinkVisiblePath, } from "../../cli.js";
 import { DEFAULT_TIMEOUT_SECONDS, GROUPED_EVENTS, HOOK_FACTORY_TO_EVENT } from "./constants.js";
 import { HOOK_EVENT_NAMES } from "./events.js";
 /** The matcher emitted when a grouped event's config names none. `"*"` matches every tool. */
@@ -82,9 +82,10 @@ export function analyzeAntigravityHookFile(sourcePath) {
  * Bundles one hook source into a self-contained ESM executable whose entry
  * wrapper calls this agent's `execute`.
  */
-export async function compileAntigravityHook(sourcePath, loaders, sourcemap = true) {
+export async function compileAntigravityHook(sourcePath, loaders, sourcemap = false) {
     const resolveDir = path.dirname(sourcePath);
     const runtimeAbsolute = symlinkVisiblePath(path.resolve(path.dirname(fileURLToPath(import.meta.url)), "./transport.js"), resolveDir);
+    const absWorkingDir = moduleWorkingDir(runtimeAbsolute, resolveDir);
     const runtimeRelative = path.relative(resolveDir, runtimeAbsolute).replace(/\\/g, "/");
     const runtimeSpecifier = runtimeRelative.startsWith(".") ? runtimeRelative : `./${runtimeRelative}`;
     const entry = [
@@ -94,6 +95,7 @@ export async function compileAntigravityHook(sourcePath, loaders, sourcemap = tr
         ``,
     ].join("\n");
     const built = await esbuild.build({
+        absWorkingDir,
         stdin: {
             contents: entry,
             resolveDir,

package/dist/agents/antigravity/scaffold.js

@@ -50,7 +50,7 @@ function packageJson(projectName, outputPath) {
             lint: "biome check .",
             typecheck: "tsc --noEmit",
         },
-        dependencies: { [PACKAGE_NAME]: "^1.0.9" },
+        dependencies: { [PACKAGE_NAME]: "^1.0.10" },
         devDependencies: {
             "@biomejs/biome": "2.4.9",
             "@types/node": "^24.0.0",

package/dist/agents/codex/cli-support.js

@@ -15,7 +15,7 @@ import * as path from "node:path";
 import { fileURLToPath } from "node:url";
 import * as esbuild from "esbuild";
 import ts from "typescript";
-import { buildEsbuildLoaderMap, discoverHookSourceFiles, parseEsbuildLoaderFlag, pruneStaleHashedBundles, sha256Prefix8, symlinkVisiblePath, } from "../../cli.js";
+import { buildEsbuildLoaderMap, discoverHookSourceFiles, moduleWorkingDir, parseEsbuildLoaderFlag, pruneStaleHashedBundles, sha256Prefix8, symlinkVisiblePath, } from "../../cli.js";
 import { EVENTS_WITH_MATCHER, HOOK_FACTORY_TO_EVENT } from "./constants.js";
 /**
  * Extracts Codex hook metadata (event name, matcher, timeout in
@@ -88,9 +88,10 @@ export function analyzeCodexHookFile(sourcePath) {
  * Compiles one Codex hook to a self-contained ESM executable whose entry
  * wrapper imports this package's Codex runtime module (`execute`).
  */
-export async function compileCodexHook(sourcePath, loaders, sourcemap = true) {
+export async function compileCodexHook(sourcePath, loaders, sourcemap = false) {
     const resolveDir = path.dirname(sourcePath);
     const runtimePathAbsolute = symlinkVisiblePath(path.resolve(path.dirname(fileURLToPath(import.meta.url)), "./transport.js"), resolveDir);
+    const absWorkingDir = moduleWorkingDir(runtimePathAbsolute, resolveDir);
     let relativeRuntimePath = path.relative(resolveDir, runtimePathAbsolute).replace(/\\/g, "/");
     if (!relativeRuntimePath.startsWith(".")) {
         relativeRuntimePath = `./${relativeRuntimePath}`;
@@ -102,6 +103,7 @@ execute(hook);
 `;
     const baseName = path.basename(sourcePath, path.extname(sourcePath));
     const result = await esbuild.build({
+        absWorkingDir,
         stdin: {
             contents: wrapperContent,
             resolveDir,

package/dist/agents/codex/scaffold.js

@@ -55,7 +55,7 @@ function generatePackageJson(projectName, outputPath) {
             typecheck: "tsc --noEmit",
         },
         dependencies: {
-            [PACKAGE_NAME]: "^1.0.9",
+            [PACKAGE_NAME]: "^1.0.10",
         },
         devDependencies: {
             "@biomejs/biome": "2.4.9",

package/dist/agents/opencode/cli-support.js

@@ -23,7 +23,7 @@ import { buildEsbuildLoaderMap, discoverHookSourceFiles, parseEsbuildLoaderFlag
  * Bundles one OpenCode plugin entry to a self-contained ESM artifact. The
  * source file's own default export is preserved unchanged — no wrapper.
  */
-export async function compileOpenCodePlugin(sourcePath, loaders, sourcemap = true) {
+export async function compileOpenCodePlugin(sourcePath, loaders, sourcemap = false) {
     const result = await esbuild.build({
         entryPoints: [sourcePath],
         loader: loaders,

package/dist/cli.js

@@ -26,7 +26,7 @@ import { scaffoldProject } from "./scaffold.js";
 // ============================================================================
 // Constants
 // ============================================================================
-const VERSION = "1.0.9";
+const VERSION = "1.0.10";
 const DEFAULT_ESBUILD_LOADERS = {
     ".md": "text",
 };
@@ -142,13 +142,13 @@ Optional Arguments:
   --no-stable-names
       Restore the pre-1.7 behavior: hashed compiled bundles (<name>.<hash>.mjs).
 
-  --sourcemap (default)
+  --sourcemap
       Embed an inline sourcemap in each compiled bundle so stack traces show
       original TypeScript source locations.
 
-  --no-sourcemap
-      Emit bundles without the inline sourcemap, shrinking compiled output by
-      roughly 85-90%. The content hash is unaffected.
+  --no-sourcemap (default)
+      Emit bundles without the inline sourcemap, shrinking compiled output.
+      The content hash is unaffected.
 
   -h, --help
       Show this help message.
@@ -257,7 +257,7 @@ function parseArgs(argv) {
         // as true, while the Codex branch defaults by command context
         // (plugin mode → stable, otherwise hashed).
         stableNames: undefined,
-        sourcemap: true,
+        sourcemap: false,
     };
     for (let i = 0; i < argv.length; i++) {
         const arg = argv[i];
@@ -642,6 +642,28 @@ function symlinkVisiblePath(realPath, resolveDir) {
         dir = parent;
     }
 }
+/**
+ * Returns the stable project anchor for a module reached through node_modules.
+ *
+ * esbuild otherwise uses process.cwd() when rendering module-boundary comments
+ * and sourcemap source paths. A workspace script normally runs from the
+ * consuming package, while its dependencies may be hoisted several directories
+ * higher, making those emitted paths depend on the consumer's nesting depth.
+ * Anchoring at the directory which owns the visible node_modules keeps the
+ * canonical `node_modules/...` form in every such layout.
+ */
+function moduleWorkingDir(modulePath, fallback) {
+    for (let dir = path.dirname(modulePath);;) {
+        if (path.basename(dir) === "node_modules") {
+            return path.dirname(dir);
+        }
+        const parent = path.dirname(dir);
+        if (parent === dir) {
+            return fallback;
+        }
+        dir = parent;
+    }
+}
 /**
  * Compiles a TypeScript hook file to a self-contained ESM executable.
  *
@@ -672,6 +694,7 @@ async function compileHook(options) {
     // nesting depth.
     const resolveDir = path.dirname(sourcePath);
     const runtimePathAbsolute = symlinkVisiblePath(path.resolve(path.dirname(fileURLToPath(import.meta.url)), "./agents/claude-code/transport.js"), resolveDir);
+    const absWorkingDir = moduleWorkingDir(runtimePathAbsolute, resolveDir);
     // Compute relative paths from resolveDir to avoid absolute paths in source maps.
     // This ensures reproducible builds regardless of checkout directory.
     const relativeSourcePath = `./${path.basename(sourcePath)}`;
@@ -731,6 +754,7 @@ execute(hook);
     ].join("\n");
     // Common esbuild options
     const commonOptions = {
+        absWorkingDir,
         stdin: stdinOptions,
         loader: loaders,
         format: "esm",
@@ -784,7 +808,7 @@ execute(hook);
     // output minus the trailing sourceMappingURL comment (verified on esbuild
     // 0.24.2), so the sourcemap-free content is emitted directly. The content
     // hash is always derived from pass-1 output, so it is unchanged by the flag.
-    if (sourcemap === false) {
+    if (sourcemap !== true) {
         return { content: contentForHash, contentHash };
     }
     const resultWithSourcemap = await esbuild.build({
@@ -1447,4 +1471,4 @@ if (isDirectExecution) {
 // Shared build primitives reused by the per-agent CLI branches (single
 // source for driver logic — the duplication check forbids re-deriving these
 // inside agents/*).
-export { analyzeHookFile, buildLoaderMap, buildLoaderMap as buildEsbuildLoaderMap, compileHook, detectHookContext, discoverHookFiles, discoverHookFiles as discoverHookSourceFiles, extractPreservedHooks, generateCommandPath, generateContentHash, generateContentHash as sha256Prefix8, generateHooksJson, groupHooksByEventAndMatcher, HOOK_FACTORY_TO_EVENT, mergeHooksJson, parseArgs, parseLoaderFlag, parseLoaderFlag as parseEsbuildLoaderFlag, pruneStaleHashedBundles, readExistingHooksJson, removeOldGeneratedFiles, symlinkVisiblePath, validateArgs, };
+export { analyzeHookFile, buildLoaderMap, buildLoaderMap as buildEsbuildLoaderMap, compileHook, detectHookContext, discoverHookFiles, discoverHookFiles as discoverHookSourceFiles, extractPreservedHooks, generateCommandPath, generateContentHash, generateContentHash as sha256Prefix8, generateHooksJson, groupHooksByEventAndMatcher, HOOK_FACTORY_TO_EVENT, mergeHooksJson, moduleWorkingDir, parseArgs, parseLoaderFlag, parseLoaderFlag as parseEsbuildLoaderFlag, pruneStaleHashedBundles, readExistingHooksJson, removeOldGeneratedFiles, symlinkVisiblePath, validateArgs, };

package/dist/scaffold.js

@@ -131,7 +131,7 @@ function generatePackageJson(projectName, outputPath) {
             typecheck: "tsc --noEmit",
         },
         dependencies: {
-            "@goodfoot/agent-hooks": "^1.0.9",
+            "@goodfoot/agent-hooks": "^1.0.10",
         },
         devDependencies: {
             "@biomejs/biome": "2.4.16",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodfoot/agent-hooks",
-  "version": "1.0.9",
+  "version": "1.0.10",
   "description": "Unified agent hooks library: shared core runtime with per-agent entry points for Claude Code, Codex, and Antigravity",
   "homepage": "https://github.com/goodfoot-io/marketplace/tree/main/packages/agent-hooks",
   "repository": {

package/types/cli.d.ts

@@ -54,7 +54,7 @@ interface CliArgs {
     pluginRoot?: boolean;
     /** Emit hash-free `<name>.mjs` bundles. Defaults to true; set false with --no-stable-names. */
     stableNames?: boolean;
-    /** Embed an inline sourcemap in compiled bundles. Defaults to true; set false with --no-sourcemap. */
+    /** Embed an inline sourcemap in compiled bundles. Defaults to false; enable with --sourcemap. */
     sourcemap?: boolean;
 }
 type HookLoaderMap = Record<string, esbuild.Loader>;
@@ -173,14 +173,14 @@ interface CompileHookOptions {
     logEnvVar?: string;
     /** Explicit esbuild loaders for non-code imports. */
     loaders: HookLoaderMap;
-    /** Embed an inline sourcemap in the compiled content. Defaults to true; set false with --no-sourcemap. */
+    /** Embed an inline sourcemap in the compiled content. Defaults to false; enable with --sourcemap. */
     sourcemap?: boolean;
 }
 /**
  * Result of compiling a hook.
  */
 interface CompileHookResult {
-    /** Compiled content, with inline sourcemaps unless disabled. */
+    /** Compiled content, with inline sourcemaps when enabled. */
     content: string;
     /** Stable content hash generated from sourcemap-free output. */
     contentHash: string;
@@ -203,6 +203,17 @@ interface CompileHookResult {
  * checkout-local.
  */
 declare function symlinkVisiblePath(realPath: string, resolveDir: string): string;
+/**
+ * Returns the stable project anchor for a module reached through node_modules.
+ *
+ * esbuild otherwise uses process.cwd() when rendering module-boundary comments
+ * and sourcemap source paths. A workspace script normally runs from the
+ * consuming package, while its dependencies may be hoisted several directories
+ * higher, making those emitted paths depend on the consumer's nesting depth.
+ * Anchoring at the directory which owns the visible node_modules keeps the
+ * canonical `node_modules/...` form in every such layout.
+ */
+declare function moduleWorkingDir(modulePath: string, fallback: string): string;
 /**
  * Compiles a TypeScript hook file to a self-contained ESM executable.
  *
@@ -312,4 +323,4 @@ declare function extractPreservedHooks(existingHooksJson: HooksJson): Partial<Re
  */
 declare function mergeHooksJson(newHooksJson: HooksJson, preservedHooks: Partial<Record<HookEventName, MatcherEntry[]>>): HooksJson;
 export type { CliArgs, CompiledHook, HookConfig, HookLoaderMap, HookMetadata, HooksJson, MatcherEntry };
-export { analyzeHookFile, buildLoaderMap, buildLoaderMap as buildEsbuildLoaderMap, compileHook, detectHookContext, discoverHookFiles, discoverHookFiles as discoverHookSourceFiles, extractPreservedHooks, generateCommandPath, generateContentHash, generateContentHash as sha256Prefix8, generateHooksJson, groupHooksByEventAndMatcher, HOOK_FACTORY_TO_EVENT, mergeHooksJson, parseArgs, parseLoaderFlag, parseLoaderFlag as parseEsbuildLoaderFlag, pruneStaleHashedBundles, readExistingHooksJson, removeOldGeneratedFiles, symlinkVisiblePath, validateArgs, };
+export { analyzeHookFile, buildLoaderMap, buildLoaderMap as buildEsbuildLoaderMap, compileHook, detectHookContext, discoverHookFiles, discoverHookFiles as discoverHookSourceFiles, extractPreservedHooks, generateCommandPath, generateContentHash, generateContentHash as sha256Prefix8, generateHooksJson, groupHooksByEventAndMatcher, HOOK_FACTORY_TO_EVENT, mergeHooksJson, moduleWorkingDir, parseArgs, parseLoaderFlag, parseLoaderFlag as parseEsbuildLoaderFlag, pruneStaleHashedBundles, readExistingHooksJson, removeOldGeneratedFiles, symlinkVisiblePath, validateArgs, };