@enactprotocol/cli 2.3.5 → 2.3.8

package/tests/commands/publish.test.ts

@@ -110,7 +110,8 @@ describe("publish command", () => {
     test("identifies manifest files", () => {
       const isManifest = (filename: string): boolean => {
         return (
-          filename === "skill.yaml" ||
+          filename === "skill.package.yaml" ||
+          filename === "skill.package.yml" ||
           filename === "skill.yml" ||
           filename === "enact.yaml" ||
           filename === "enact.yml" ||
@@ -118,7 +119,8 @@ describe("publish command", () => {
         );
       };
 
-      expect(isManifest("skill.yaml")).toBe(true);
+      expect(isManifest("skill.package.yaml")).toBe(true);
+      expect(isManifest("skill.package.yml")).toBe(true);
       expect(isManifest("skill.yml")).toBe(true);
       expect(isManifest("enact.yaml")).toBe(true);
       expect(isManifest("enact.yml")).toBe(true);
@@ -132,7 +134,7 @@ describe("publish command", () => {
         return lastDot === -1 ? "" : filename.slice(lastDot + 1);
       };
 
-      expect(getExtension("skill.yaml")).toBe("yaml");
+      expect(getExtension("skill.package.yaml")).toBe("yaml");
       expect(getExtension("skill.yml")).toBe("yml");
       expect(getExtension("enact.yaml")).toBe("yaml");
       expect(getExtension("enact.yml")).toBe("yml");

package/tests/commands/sign.test.ts

@@ -17,7 +17,7 @@ describe("sign command", () => {
 
     // Create a test manifest
     writeFileSync(
-      join(FIXTURES_DIR, "skill.yaml"),
+      join(FIXTURES_DIR, "skill.package.yml"),
       `enact: "2.0.0"
 name: "test/sign-tool"
 version: "1.0.0"

package/tests/e2e.test.ts

@@ -119,14 +119,16 @@ describe("E2E: Tool Installation Flow", () => {
     }
   });
 
-  test("installs tool to project .enact/tools directory", () => {
-    const destBase = join(tempDir, ".enact", "tools");
+  test("installs tool to project agents/skills directory", () => {
+    const destBase = join(tempDir, "agents", "skills");
     const { manifest, destPath } = installTool(GREETER_TOOL, destBase);
 
     expect(manifest.name).toBe("test/greeter");
     expect(existsSync(destPath)).toBe(true);
     expect(
-      existsSync(join(destPath, "enact.yaml")) || existsSync(join(destPath, "skill.yaml"))
+      existsSync(join(destPath, "enact.yaml")) ||
+        existsSync(join(destPath, "skill.package.yaml")) ||
+        existsSync(join(destPath, "skill.package.yml"))
     ).toBe(true);
 
     // Verify installed manifest can be loaded
@@ -136,7 +138,7 @@ describe("E2E: Tool Installation Flow", () => {
   });
 
   test("installs markdown tool correctly", () => {
-    const destBase = join(tempDir, ".enact", "tools");
+    const destBase = join(tempDir, "agents", "skills");
     const { manifest, destPath } = installTool(ECHO_TOOL, destBase);
 
     expect(manifest.name).toBe("test/echo-tool");
@@ -144,7 +146,7 @@ describe("E2E: Tool Installation Flow", () => {
   });
 
   test("installs multiple tools without conflict", () => {
-    const destBase = join(tempDir, ".enact", "tools");
+    const destBase = join(tempDir, "agents", "skills");
 
     const result1 = installTool(GREETER_TOOL, destBase);
     const result2 = installTool(ECHO_TOOL, destBase);
@@ -161,7 +163,7 @@ describe("E2E: Tool Installation Flow", () => {
   });
 
   test("overwrites existing tool on reinstall", () => {
-    const destBase = join(tempDir, ".enact", "tools");
+    const destBase = join(tempDir, "agents", "skills");
 
     // First install
     const result1 = installTool(GREETER_TOOL, destBase);
@@ -190,7 +192,7 @@ describe("E2E: Tool Resolution Flow", () => {
     mkdirSync(resolveTempDir, { recursive: true });
 
     // Install tools
-    const destBase = join(resolveTempDir, ".enact", "tools");
+    const destBase = join(resolveTempDir, "agents", "skills");
     installTool(GREETER_TOOL, destBase);
     installTool(ECHO_TOOL, destBase);
     installTool(CALCULATOR_TOOL, destBase);
@@ -210,7 +212,7 @@ describe("E2E: Tool Resolution Flow", () => {
 
   test("resolves tool by path", () => {
     // Resolve from installed location
-    const toolPath = join(resolveTempDir, ".enact", "tools", "test", "greeter");
+    const toolPath = join(resolveTempDir, "agents", "skills", "test", "greeter");
     const resolution = tryResolveTool(toolPath);
     expect(resolution).not.toBeNull();
     expect(resolution?.manifest.name).toBe("test/greeter");
@@ -481,7 +483,7 @@ describe("E2E: Full Workflow", () => {
 
   test("complete install -> resolve -> validate -> prepare flow (scripts)", () => {
     // 1. Install tool
-    const destBase = join(tempDir, ".enact", "tools");
+    const destBase = join(tempDir, "agents", "skills");
     const { manifest } = installTool(GREETER_TOOL, destBase);
     expect(manifest.name).toBe("test/greeter");
 
@@ -507,7 +509,7 @@ describe("E2E: Full Workflow", () => {
 
   test("complete calculator workflow (scripts)", () => {
     // 1. Install tool
-    const destBase = join(tempDir, ".enact", "tools");
+    const destBase = join(tempDir, "agents", "skills");
     const { manifest } = installTool(CALCULATOR_TOOL, destBase);
 
     // 2. Resolve tool
@@ -542,7 +544,7 @@ describe("E2E: Full Workflow", () => {
 
   test("handles markdown tool workflow (scripts)", () => {
     // Install markdown-based tool
-    const destBase = join(tempDir, ".enact", "tools");
+    const destBase = join(tempDir, "agents", "skills");
     const { manifest } = installTool(ECHO_TOOL, destBase);
     expect(manifest.name).toBe("test/echo-tool");
 

package/tests/utils/ignore.test.ts

@@ -281,7 +281,8 @@ dist   `;
       });
 
       test("allows manifest files", () => {
-        expect(shouldIgnore("skill.yaml", "skill.yaml")).toBe(false);
+        expect(shouldIgnore("skill.package.yaml", "skill.package.yaml")).toBe(false);
+        expect(shouldIgnore("skill.package.yml", "skill.package.yml")).toBe(false);
         expect(shouldIgnore("skill.yml", "skill.yml")).toBe(false);
         expect(shouldIgnore("enact.md", "enact.md")).toBe(false);
         expect(shouldIgnore("enact.yaml", "enact.yaml")).toBe(false);