@enactprotocol/cli 2.1.7 → 2.1.10

package/tests/commands/{get.test.ts → info.test.ts}

@@ -1,64 +1,66 @@
 /**
- * Tests for the get command
+ * Tests for the info command
  */
 
 import { describe, expect, test } from "bun:test";
 import type { ToolVersionInfo } from "@enactprotocol/api";
 import { Command } from "commander";
-import { configureGetCommand } from "../../src/commands/get";
+import { configureInfoCommand } from "../../src/commands/info";
 
-describe("get command", () => {
+describe("info command", () => {
   describe("command configuration", () => {
-    test("configures get command on program", () => {
+    test("configures info command on program", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      expect(getCmd).toBeDefined();
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      expect(infoCmd).toBeDefined();
     });
 
     test("has correct description", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      expect(getCmd?.description()).toBe("Show detailed information about a tool");
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      expect(infoCmd?.description()).toBe(
+        "Show detailed information about a tool (local path or registry)"
+      );
     });
 
-    test("has info as alias", () => {
+    test("has get as alias", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      expect(getCmd?.aliases()).toContain("info");
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      expect(infoCmd?.aliases()).toContain("get");
     });
 
     test("accepts tool argument", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      const args = getCmd?.registeredArguments ?? [];
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      const args = infoCmd?.registeredArguments ?? [];
       expect(args.length).toBeGreaterThan(0);
       expect(args[0]?.name()).toBe("tool");
     });
 
     test("has --ver option for specifying version", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      const opts = getCmd?.options ?? [];
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      const opts = infoCmd?.options ?? [];
       const verOpt = opts.find((o) => o.long === "--ver");
       expect(verOpt).toBeDefined();
     });
 
     test("has -v short option for verbose (not version)", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      const opts = getCmd?.options ?? [];
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      const opts = infoCmd?.options ?? [];
       // -v is for verbose, not version (--ver is for version)
       const verboseOpt = opts.find((o) => o.short === "-v");
       expect(verboseOpt).toBeDefined();
@@ -67,20 +69,20 @@ describe("get command", () => {
 
     test("has --json option", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      const opts = getCmd?.options ?? [];
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      const opts = infoCmd?.options ?? [];
       const jsonOpt = opts.find((o) => o.long === "--json");
       expect(jsonOpt).toBeDefined();
     });
 
     test("has --verbose option", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      const opts = getCmd?.options ?? [];
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      const opts = infoCmd?.options ?? [];
       const verboseOpt = opts.find((o) => o.long === "--verbose");
       expect(verboseOpt).toBeDefined();
     });
@@ -249,12 +251,12 @@ Documentation here.`;
       expect(enactMdContent).toContain("Documentation here.");
     });
 
-    test("verbose option is available on get command", () => {
+    test("verbose option is available on info command", () => {
       const program = new Command();
-      configureGetCommand(program);
+      configureInfoCommand(program);
 
-      const getCmd = program.commands.find((cmd) => cmd.name() === "get");
-      const opts = getCmd?.options ?? [];
+      const infoCmd = program.commands.find((cmd) => cmd.name() === "info");
+      const opts = infoCmd?.options ?? [];
 
       // Check both short and long form exist
       const verboseOpt = opts.find((o) => o.long === "--verbose");

package/tests/commands/init.test.ts

@@ -159,7 +159,7 @@ describe("init command", () => {
       expect(content).toEqual({ tools: {} });
     });
 
-    test("--tool mode creates enact.md", async () => {
+    test("--tool mode creates SKILL.md", async () => {
       const program = new Command();
       program.exitOverride(); // Prevent process.exit
       configureInitCommand(program);
@@ -176,7 +176,7 @@ describe("init command", () => {
         process.chdir(originalCwd);
       }
 
-      const manifestPath = join(testDir, "enact.md");
+      const manifestPath = join(testDir, "SKILL.md");
       expect(existsSync(manifestPath)).toBe(true);
 
       const content = readFileSync(manifestPath, "utf-8");
@@ -206,7 +206,7 @@ describe("init command", () => {
 
       const content = readFileSync(agentsPath, "utf-8");
       expect(content).toContain("enact run");
-      expect(content).toContain("enact.md");
+      expect(content).toContain("SKILL.md");
       expect(content).toContain("Parameter Substitution");
     });
 
@@ -395,7 +395,7 @@ describe("init command", () => {
       expect(existsSync(toolsJsonPath)).toBe(false);
     });
 
-    test("enact.md contains valid YAML frontmatter", async () => {
+    test("SKILL.md contains valid YAML frontmatter", async () => {
       const program = new Command();
       program.exitOverride();
       configureInitCommand(program);
@@ -418,7 +418,7 @@ describe("init command", () => {
         process.chdir(originalCwd);
       }
 
-      const content = readFileSync(join(testDir, "enact.md"), "utf-8");
+      const content = readFileSync(join(testDir, "SKILL.md"), "utf-8");
 
       // Check frontmatter structure
       expect(content.startsWith("---")).toBe(true);
@@ -522,7 +522,7 @@ describe("init command", () => {
         process.chdir(originalCwd);
       }
 
-      const content = readFileSync(join(testDir, "enact.md"), "utf-8");
+      const content = readFileSync(join(testDir, "SKILL.md"), "utf-8");
 
       // Required fields per spec
       expect(content).toContain("name:");

package/tests/commands/learn.test.ts

@@ -55,14 +55,14 @@ describe("learn command", () => {
       expect(jsonOpt).toBeDefined();
     });
 
-    test("does not have --verbose option (always shows full docs)", () => {
+    test("has --verbose option for showing attestation details", () => {
       const program = new Command();
       configureLearnCommand(program);
 
       const learnCmd = program.commands.find((cmd) => cmd.name() === "learn");
       const opts = learnCmd?.options ?? [];
       const verboseOpt = opts.find((o) => o.long === "--verbose");
-      expect(verboseOpt).toBeUndefined();
+      expect(verboseOpt).toBeDefined();
     });
   });
 

package/tests/e2e.test.ts

@@ -139,7 +139,7 @@ describe("E2E: Tool Installation Flow", () => {
     const { manifest, destPath } = installTool(ECHO_TOOL, destBase);
 
     expect(manifest.name).toBe("test/echo-tool");
-    expect(existsSync(join(destPath, "enact.md"))).toBe(true);
+    expect(existsSync(join(destPath, "SKILL.md"))).toBe(true);
   });
 
   test("installs multiple tools without conflict", () => {