@enactprotocol/shared 2.3.7 → 2.3.8

package/src/index.ts

@@ -24,6 +24,8 @@ export {
 export {
   getEnactHome,
   getProjectEnactDir,
+  getProjectAgentsDir,
+  getProjectRoot,
   getToolsDir,
   getSkillsDir,
   getCacheDir,

package/src/mcp-registry.ts

@@ -133,7 +133,7 @@ export function getMcpToolVersion(toolName: string): string | null {
 
 /**
  * Get the install path for an MCP tool
- * Skills are stored at ~/.agent/skills/{name}/ (flat, no version subdirectory)
+ * Skills are stored at ~/.agents/skills/{name}/ (flat, no version subdirectory)
  */
 function getMcpToolCachePath(toolName: string, _version: string): string {
   const skillsDir = getCacheDir();

package/src/paths.ts

@@ -4,7 +4,7 @@
 
 import { existsSync } from "node:fs";
 import { homedir } from "node:os";
-import { join, resolve } from "node:path";
+import { dirname, join, resolve } from "node:path";
 
 /**
  * Scope for tool directories
@@ -48,44 +48,81 @@ export function getProjectEnactDir(startDir?: string): string | null {
   return null;
 }
 
+/**
+ * Get the project-level agents directory
+ * Searches up from current working directory to find agents/
+ * @param startDir - Directory to start searching from (defaults to cwd)
+ * @returns Absolute path to agents/ or null if not found
+ */
+export function getProjectAgentsDir(startDir?: string): string | null {
+  let currentDir = resolve(startDir ?? process.cwd());
+  const root = resolve("/");
+
+  while (currentDir !== root) {
+    const agentsDir = join(currentDir, "agents");
+    if (existsSync(agentsDir)) {
+      return agentsDir;
+    }
+    const parentDir = resolve(currentDir, "..");
+    if (parentDir === currentDir) {
+      break;
+    }
+    currentDir = parentDir;
+  }
+
+  return null;
+}
+
+/**
+ * Get the project root directory by searching for agents/ directory
+ * @param startDir - Directory to start searching from (defaults to cwd)
+ * @returns Absolute path to the project root or the startDir/cwd as fallback
+ */
+export function getProjectRoot(startDir?: string): string {
+  const agentsDir = getProjectAgentsDir(startDir);
+  if (agentsDir) {
+    return dirname(agentsDir);
+  }
+  return resolve(startDir ?? process.cwd());
+}
+
 /**
  * Get the tools directory for specified scope
  *
  * NOTE: For global scope ("user"), this is DEPRECATED.
- * Global tools are now tracked in ~/.enact/tools.json and stored in cache.
+ * Global tools are now tracked in ~/.enact/tools.json and stored in cache (~/.agents/skills/).
  * Use getToolsJsonPath("global") and getToolCachePath() from ./registry instead.
  *
- * For project scope, this returns .enact/tools/ where project tools are copied.
+ * For project scope, this returns agents/skills/ where project tools are installed.
  *
- * @param scope - 'user' for ~/.enact/tools/ (deprecated) or 'project' for .enact/tools/
+ * @param scope - 'user' for ~/.agents/skills/ or 'project' for agents/skills/
  * @param startDir - For project scope, directory to start searching from
  * @returns Absolute path to tools directory or null if project scope and not found
  * @deprecated Use registry.ts functions for global tools
  */
 export function getToolsDir(scope: ToolScope, startDir?: string): string | null {
   if (scope === "user") {
-    // DEPRECATED: Global tools now use tools.json + cache
-    // This path is kept for backward compatibility during migration
-    return join(getEnactHome(), "tools");
+    // Global tools use ~/.agents/skills/
+    return getSkillsDir();
   }
 
-  const projectDir = getProjectEnactDir(startDir);
-  return projectDir ? join(projectDir, "tools") : null;
+  const agentsDir = getProjectAgentsDir(startDir);
+  return agentsDir ? join(agentsDir, "skills") : null;
 }
 
 /**
- * Get the skills directory (~/.agent/skills/)
+ * Get the skills directory (~/.agents/skills/)
  * This is the standard Agent Skills location for installed skills.
- * @returns Absolute path to ~/.agent/skills/
+ * @returns Absolute path to ~/.agents/skills/
  */
 export function getSkillsDir(): string {
-  return join(homedir(), ".agent", "skills");
+  return join(homedir(), ".agents", "skills");
 }
 
 /**
- * Get the cache directory (~/.agent/skills/)
+ * Get the cache directory (~/.agents/skills/)
  * @deprecated Use getSkillsDir() instead
- * @returns Absolute path to ~/.agent/skills/
+ * @returns Absolute path to ~/.agents/skills/
  */
 export function getCacheDir(): string {
   return getSkillsDir();

package/src/registry.ts

@@ -1,17 +1,16 @@
 /**
  * Local tool registry management
  *
- * Manages tools.json files for tracking installed tools:
+ * Manages skills.json files for tracking installed tools:
  * - Global: ~/.enact/tools.json (installed with -g)
- * - Project: .enact/tools.json (project dependencies)
+ * - Project: agents/skills.json (project dependencies)
  *
- * Tools are stored in cache and referenced by version in tools.json.
- * This eliminates the need for a separate ~/.enact/tools/ directory.
+ * Tools are stored in cache and referenced by version in the registry.
  */
 
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { getCacheDir, getEnactHome, getProjectEnactDir } from "./paths";
+import { dirname, join, resolve } from "node:path";
+import { getCacheDir, getEnactHome, getProjectAgentsDir } from "./paths";
 
 /**
  * Structure of tools.json file
@@ -39,15 +38,17 @@ export interface InstalledToolInfo {
 }
 
 /**
- * Get the path to tools.json for the specified scope
+ * Get the path to the registry file for the specified scope
+ * - Global: ~/.enact/tools.json
+ * - Project: agents/skills.json
  */
 export function getToolsJsonPath(scope: RegistryScope, startDir?: string): string | null {
   if (scope === "global") {
     return join(getEnactHome(), "tools.json");
   }
 
-  const projectDir = getProjectEnactDir(startDir);
-  return projectDir ? join(projectDir, "tools.json") : null;
+  const agentsDir = getProjectAgentsDir(startDir);
+  return agentsDir ? join(agentsDir, "skills.json") : null;
 }
 
 /**
@@ -84,12 +85,12 @@ export function saveToolsRegistry(
 ): void {
   let registryPath = getToolsJsonPath(scope, startDir);
 
-  // For project scope, create .enact/ directory if it doesn't exist
+  // For project scope, create agents/ directory if it doesn't exist
   if (!registryPath && scope === "project") {
-    const projectRoot = startDir ?? process.cwd();
-    const enactDir = join(projectRoot, ".enact");
-    mkdirSync(enactDir, { recursive: true });
-    registryPath = join(enactDir, "tools.json");
+    const projectRoot = resolve(startDir ?? process.cwd());
+    const agentsDir = join(projectRoot, "agents");
+    mkdirSync(agentsDir, { recursive: true });
+    registryPath = join(agentsDir, "skills.json");
   }
 
   if (!registryPath) {
@@ -166,7 +167,7 @@ export function getInstalledVersion(
 
 /**
  * Get the install path for a skill
- * Skills are stored at ~/.agent/skills/{name}/ (flat, no version subdirectory)
+ * Skills are stored at ~/.agents/skills/{name}/ (flat, no version subdirectory)
  */
 export function getToolCachePath(toolName: string, _version: string): string {
   const skillsDir = getCacheDir();

package/src/resolver.ts

@@ -3,9 +3,9 @@
  *
  * Resolution order:
  * 1. Direct file path (if provided path exists)
- * 2. Project tools (.enact/tools/{name}/)
- * 3. Global tools (via ~/.enact/tools.json → ~/.agent/skills/)
- * 4. Skills directory (~/.agent/skills/{name}/)
+ * 2. Project skills (agents/skills/{name}/)
+ * 3. Global tools (via ~/.enact/tools.json → ~/.agents/skills/)
+ * 4. Skills directory (~/.agents/skills/{name}/)
  */
 
 import { existsSync } from "node:fs";
@@ -18,7 +18,7 @@ import {
   loadManifestFromDir,
 } from "./manifest/loader";
 import { manifestScriptsToActionsManifest } from "./manifest/scripts";
-import { getProjectEnactDir, getSkillsDir } from "./paths";
+import { getProjectAgentsDir, getSkillsDir } from "./paths";
 import { getInstalledVersion, getToolCachePath, resolveAlias } from "./registry";
 import type { ToolLocation, ToolResolution } from "./types/manifest";
 
@@ -336,12 +336,12 @@ export function resolveTool(toolName: string, options: ResolveOptions = {}): Too
     }
   }
 
-  // 1. Try project tools (.enact/tools/{name}/)
+  // 1. Try project skills (agents/skills/{name}/)
   if (!options.skipProject) {
-    const projectDir = getProjectEnactDir(options.startDir);
-    if (projectDir) {
-      const projectToolsDir = join(projectDir, "tools");
-      const toolDir = getToolPath(projectToolsDir, normalizedName);
+    const agentsDir = getProjectAgentsDir(options.startDir);
+    if (agentsDir) {
+      const projectSkillsDir = join(agentsDir, "skills");
+      const toolDir = getToolPath(projectSkillsDir, normalizedName);
       searchedLocations.push(toolDir);
 
       const result = tryLoadFromDir(toolDir, "project");
@@ -351,7 +351,7 @@ export function resolveTool(toolName: string, options: ResolveOptions = {}): Too
     }
   }
 
-  // 2. Try global tools (via ~/.enact/tools.json → ~/.agent/skills/)
+  // 2. Try global tools (via ~/.enact/tools.json → ~/.agents/skills/)
   if (!options.skipUser) {
     const globalVersion = getInstalledVersion(normalizedName, "global");
     if (globalVersion) {
@@ -365,7 +365,7 @@ export function resolveTool(toolName: string, options: ResolveOptions = {}): Too
     }
   }
 
-  // 3. Try skills directory (~/.agent/skills/{name}/)
+  // 3. Try skills directory (~/.agents/skills/{name}/)
   if (!options.skipCache) {
     const skillsDir = getSkillsDir();
     const skillDir = getToolPath(skillsDir, normalizedName);
@@ -561,11 +561,11 @@ export function getToolSearchPaths(toolName: string, options: ResolveOptions = {
   const normalizedName = normalizeToolName(toolName);
   const paths: string[] = [];
 
-  // Project tools
+  // Project skills
   if (!options.skipProject) {
-    const projectDir = getProjectEnactDir(options.startDir);
-    if (projectDir) {
-      paths.push(join(projectDir, "tools", toolNameToPath(normalizedName)));
+    const agentsDir = getProjectAgentsDir(options.startDir);
+    if (agentsDir) {
+      paths.push(join(agentsDir, "skills", toolNameToPath(normalizedName)));
     }
   }
 

package/tests/config.test.ts

@@ -534,7 +534,7 @@ describe("configuration manager", () => {
       expect(existsSync(enactHome)).toBe(true);
     });
 
-    test("creates ~/.agent/skills/ directory", () => {
+    test("creates ~/.agents/skills/ directory", () => {
       const enactHome = getEnactHome();
       const cacheDir = getCacheDir();
 

package/tests/paths.test.ts

@@ -21,6 +21,7 @@ describe("path utilities", () => {
     // Create test directory structure
     mkdirSync(NESTED_DIR, { recursive: true });
     mkdirSync(join(TEST_DIR, ".enact"), { recursive: true });
+    mkdirSync(join(TEST_DIR, "agents"), { recursive: true });
   });
 
   afterAll(() => {
@@ -81,36 +82,35 @@ describe("path utilities", () => {
   });
 
   describe("getToolsDir", () => {
-    test("returns ~/.enact/tools/ for user scope", () => {
+    test("returns ~/.agents/skills/ for user scope", () => {
       const result = getToolsDir("user");
-      const expected = join(homedir(), ".enact", "tools");
+      const expected = join(homedir(), ".agents", "skills");
       expect(result).toBe(expected);
     });
 
-    test("returns .enact/tools/ for project scope", () => {
+    test("returns agents/skills/ for project scope", () => {
       const result = getToolsDir("project", TEST_DIR);
-      expect(result).toBe(join(TEST_DIR, ".enact", "tools"));
+      expect(result).toBe(join(TEST_DIR, "agents", "skills"));
     });
 
     test("finds project tools in parent directory", () => {
       const result = getToolsDir("project", NESTED_DIR);
-      expect(result).toBe(join(TEST_DIR, ".enact", "tools"));
+      expect(result).toBe(join(TEST_DIR, "agents", "skills"));
     });
 
-    test("returns null for project scope when .enact not found", () => {
-      // Similar to above - may find ~/.enact/ if it exists
-      const result = getToolsDir("project", "/tmp/no-enact-unlikely-path");
-      // Result will be null or a valid tools directory
+    test("returns null for project scope when agents/ not found", () => {
+      const result = getToolsDir("project", "/tmp/no-agents-unlikely-path");
+      // Result will be null or a valid skills directory
       if (result !== null) {
-        expect(result.endsWith("tools")).toBe(true);
+        expect(result.endsWith("skills")).toBe(true);
       }
     });
   });
 
   describe("getSkillsDir", () => {
-    test("returns ~/.agent/skills/ path", () => {
+    test("returns ~/.agents/skills/ path", () => {
       const result = getSkillsDir();
-      const expected = join(homedir(), ".agent", "skills");
+      const expected = join(homedir(), ".agents", "skills");
       expect(result).toBe(expected);
     });
   });

package/tests/registry.test.ts

@@ -24,12 +24,12 @@ import {
 
 const TEST_DIR = join(import.meta.dir, "temp-registry-test");
 const PROJECT_DIR = join(TEST_DIR, "project");
-const PROJECT_ENACT_DIR = join(PROJECT_DIR, ".enact");
+const PROJECT_AGENTS_DIR = join(PROJECT_DIR, "agents");
 
 describe("registry", () => {
   beforeAll(() => {
     // Create test directory structure
-    mkdirSync(PROJECT_ENACT_DIR, { recursive: true });
+    mkdirSync(PROJECT_AGENTS_DIR, { recursive: true });
   });
 
   afterAll(() => {
@@ -47,14 +47,14 @@ describe("registry", () => {
       expect(path).toEndWith("tools.json");
     });
 
-    test("returns path for project scope when .enact exists", () => {
+    test("returns path for project scope when agents/ exists", () => {
       const path = getToolsJsonPath("project", PROJECT_DIR);
       expect(path).not.toBeNull();
-      expect(path).toContain(PROJECT_ENACT_DIR);
-      expect(path).toEndWith("tools.json");
+      expect(path).toContain(PROJECT_AGENTS_DIR);
+      expect(path).toEndWith("skills.json");
     });
 
-    test("returns null for project scope when no .enact", () => {
+    test("returns null for project scope when no agents/", () => {
       const path = getToolsJsonPath("project", "/tmp/nonexistent-test-path");
       expect(path).toBeNull();
     });
@@ -63,7 +63,7 @@ describe("registry", () => {
   describe("loadToolsRegistry", () => {
     test("returns empty registry when file does not exist", () => {
       // Ensure tools.json doesn't exist from previous tests
-      const toolsJsonPath = join(PROJECT_ENACT_DIR, "tools.json");
+      const toolsJsonPath = join(PROJECT_AGENTS_DIR, "skills.json");
       if (existsSync(toolsJsonPath)) {
         rmSync(toolsJsonPath);
       }
@@ -73,7 +73,7 @@ describe("registry", () => {
 
     test("loads existing registry", () => {
       // Create a test registry file
-      const registryPath = join(PROJECT_ENACT_DIR, "tools.json");
+      const registryPath = join(PROJECT_AGENTS_DIR, "skills.json");
       writeFileSync(
         registryPath,
         JSON.stringify({
@@ -93,7 +93,7 @@ describe("registry", () => {
     });
 
     test("returns empty registry on parse error", () => {
-      const registryPath = join(PROJECT_ENACT_DIR, "tools.json");
+      const registryPath = join(PROJECT_AGENTS_DIR, "skills.json");
       writeFileSync(registryPath, "invalid json");
 
       const registry = loadToolsRegistry("project", PROJECT_DIR);
@@ -114,7 +114,7 @@ describe("registry", () => {
 
       saveToolsRegistry(registry, "project", PROJECT_DIR);
 
-      const registryPath = join(PROJECT_ENACT_DIR, "tools.json");
+      const registryPath = join(PROJECT_AGENTS_DIR, "skills.json");
       expect(existsSync(registryPath)).toBe(true);
 
       const loaded = loadToolsRegistry("project", PROJECT_DIR);
@@ -133,7 +133,7 @@ describe("registry", () => {
       expect(registry.tools["test/add"]).toBe("1.0.0");
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("updates existing tool version", () => {
@@ -144,7 +144,7 @@ describe("registry", () => {
       expect(registry.tools["test/update"]).toBe("2.0.0");
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
   });
 
@@ -159,7 +159,7 @@ describe("registry", () => {
       expect(registry.tools["test/remove"]).toBeUndefined();
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns false for non-existent tool", () => {
@@ -174,7 +174,7 @@ describe("registry", () => {
       expect(isToolInstalled("test/installed", "project", PROJECT_DIR)).toBe(true);
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns false for non-installed tool", () => {
@@ -189,7 +189,7 @@ describe("registry", () => {
       expect(version).toBe("3.0.0");
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns null for non-installed tool", () => {
@@ -199,9 +199,9 @@ describe("registry", () => {
   });
 
   describe("getToolCachePath", () => {
-    test("returns skill path under ~/.agent/skills/", () => {
+    test("returns skill path under ~/.agents/skills/", () => {
       const path = getToolCachePath("org/tool", "1.0.0");
-      expect(path).toContain(".agent");
+      expect(path).toContain(".agents");
       expect(path).toContain("skills");
       expect(path).toContain("org/tool");
       // No version subdirectory in new layout
@@ -226,7 +226,7 @@ describe("registry", () => {
       expect(tools.find((t) => t.name === "test/list2")).toBeTruthy();
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns empty list when no tools installed", () => {
@@ -244,7 +244,7 @@ describe("registry", () => {
       expect(registry.aliases?.mytool).toBe("test/aliased-tool");
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("throws error when alias already exists for different tool", () => {
@@ -257,7 +257,7 @@ describe("registry", () => {
       }).toThrow('Alias "shared" already exists for tool "test/tool1"');
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("allows adding same alias for same tool (idempotent)", () => {
@@ -270,7 +270,7 @@ describe("registry", () => {
       }).not.toThrow();
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
   });
 
@@ -286,7 +286,7 @@ describe("registry", () => {
       expect(registry.aliases?.removeme).toBeUndefined();
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns false for non-existent alias", () => {
@@ -304,7 +304,7 @@ describe("registry", () => {
       expect(resolved).toBe("org/full-name");
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns null for non-existent alias", () => {
@@ -325,7 +325,7 @@ describe("registry", () => {
       expect(aliases.length).toBe(2);
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns empty array for tool without aliases", () => {
@@ -335,7 +335,7 @@ describe("registry", () => {
       expect(aliases).toEqual([]);
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
   });
 
@@ -353,7 +353,7 @@ describe("registry", () => {
       expect(registry.aliases?.cleanup2).toBeUndefined();
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("returns 0 for tool without aliases", () => {
@@ -363,7 +363,7 @@ describe("registry", () => {
       expect(removed).toBe(0);
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
 
     test("does not remove aliases for other tools", () => {
@@ -379,13 +379,13 @@ describe("registry", () => {
       expect(registry.aliases?.removeme).toBeUndefined();
 
       // Clean up
-      rmSync(join(PROJECT_ENACT_DIR, "tools.json"));
+      rmSync(join(PROJECT_AGENTS_DIR, "skills.json"));
     });
   });
 
   describe("loadToolsRegistry with aliases", () => {
     test("loads existing registry with aliases", () => {
-      const registryPath = join(PROJECT_ENACT_DIR, "tools.json");
+      const registryPath = join(PROJECT_AGENTS_DIR, "skills.json");
       writeFileSync(
         registryPath,
         JSON.stringify({
@@ -407,7 +407,7 @@ describe("registry", () => {
     });
 
     test("returns empty aliases when not present in file", () => {
-      const registryPath = join(PROJECT_ENACT_DIR, "tools.json");
+      const registryPath = join(PROJECT_AGENTS_DIR, "skills.json");
       writeFileSync(
         registryPath,
         JSON.stringify({

package/tests/resolver.test.ts

@@ -16,16 +16,16 @@ import {
 
 const TEST_DIR = join(import.meta.dir, "temp-resolver-test");
 const PROJECT_DIR = join(TEST_DIR, "project");
-const PROJECT_ENACT_DIR = join(PROJECT_DIR, ".enact");
+const PROJECT_AGENTS_DIR = join(PROJECT_DIR, "agents");
 
 describe("tool resolver", () => {
   beforeAll(() => {
     // Create test directories
-    mkdirSync(join(PROJECT_ENACT_DIR, "tools", "test", "project-tool"), { recursive: true });
+    mkdirSync(join(PROJECT_AGENTS_DIR, "skills", "test", "project-tool"), { recursive: true });
 
     // Create a project-level tool
     writeFileSync(
-      join(PROJECT_ENACT_DIR, "tools", "test", "project-tool", "skill.package.yml"),
+      join(PROJECT_AGENTS_DIR, "skills", "test", "project-tool", "skill.package.yml"),
       `
 name: test/project-tool
 description: A project-level test tool
@@ -289,7 +289,7 @@ Documentation here.
       } finally {
         // Clean up
         removeAlias("pt", "project", PROJECT_DIR);
-        rmSync(join(PROJECT_ENACT_DIR, "tools.json"), { force: true });
+        rmSync(join(PROJECT_AGENTS_DIR, "skills.json"), { force: true });
       }
     });
 
@@ -311,7 +311,7 @@ Documentation here.
         expect(mixedResult.manifest.name).toBe("test/project-tool");
       } finally {
         removeAlias("mytool", "project", PROJECT_DIR);
-        rmSync(join(PROJECT_ENACT_DIR, "tools.json"), { force: true });
+        rmSync(join(PROJECT_AGENTS_DIR, "skills.json"), { force: true });
       }
     });
 
@@ -326,7 +326,7 @@ Documentation here.
         expect(result.manifest.name).toBe("test/project-tool");
       } finally {
         removeAlias("test/project-tool", "project", PROJECT_DIR);
-        rmSync(join(PROJECT_ENACT_DIR, "tools.json"), { force: true });
+        rmSync(join(PROJECT_AGENTS_DIR, "skills.json"), { force: true });
       }
     });
 
@@ -340,7 +340,7 @@ Documentation here.
         expect(result?.manifest.name).toBe("test/project-tool");
       } finally {
         removeAlias("try-alias", "project", PROJECT_DIR);
-        rmSync(join(PROJECT_ENACT_DIR, "tools.json"), { force: true });
+        rmSync(join(PROJECT_AGENTS_DIR, "skills.json"), { force: true });
       }
     });