Merge pull request #236 from PaulJPhilp/codex/ep-cli-pre-release-hardening

feat(ep-cli): harden public surface and API-backed pattern reads

Author: PaulJPhilp

packages/ep-cli/README.md

@@ -11,16 +11,12 @@ Search, browse, and install Effect-TS patterns directly from your terminal. Buil
 ## Installation
 
 ```bash
-# npm
-npm install -g @effect-patterns/ep-cli
-
 # bun
 bun add -g @effect-patterns/ep-cli
-
-# pnpm
-pnpm add -g @effect-patterns/ep-cli
 ```
 
+Requires Bun runtime.
+
 ## Commands
 
 ### Pattern Discovery
@@ -57,13 +53,6 @@ ep install list --installed
 
 Supported tools: `agents`, `cursor`, `vscode`, `windsurf`
 
-### Pattern Authoring
-
-```bash
-# Scaffold a new pattern (interactive wizard)
-ep pattern new
-```
-
 ### Skills Management
 
 Manage and validate Claude Skills built from patterns.
@@ -82,40 +71,17 @@ ep skills validate
 ep skills stats
 ```
 
-### Release Management
-
-```bash
-# Preview next release version and changelog
-ep release preview
-
-# Create a release (version bump, changelog, tag, push)
-ep release create
-```
-
-### Admin / Publishing Pipeline
-
-```bash
-# Lint patterns for Effect-TS best practices
-ep admin lint
-ep admin lint --fix
+## Maintainer Commands
 
-# Validate pattern files
-ep admin validate -v
-
-# Run example tests
-ep admin test
-
-# Generate documentation
-ep admin generate
-
-# Run the full publishing pipeline (test -> validate -> generate -> ingest)
-ep admin pipeline
-```
+Maintainer workflows (release, pattern authoring, publishing/admin commands) live in `@effect-patterns/ep-admin`.
 
 ## Environment Variables
 
 | Variable | Description | Default |
 |----------|-------------|---------|
+| `PATTERN_API_KEY` | API key for hosted Effect Patterns API (`x-api-key` header) | - |
+| `EFFECT_PATTERNS_API_URL` | Base URL for Effect Patterns API | `https://effect-patterns-mcp.vercel.app` |
+| `EP_API_TIMEOUT_MS` | HTTP timeout for API requests (milliseconds) | `10000` |
 | `LOG_LEVEL` | Set log level (`debug`, `info`, `warn`, `error`) | `info` |
 | `DEBUG` | Enable debug logging | - |
 | `VERBOSE` | Enable verbose logging | - |

packages/ep-cli/package.json

@@ -17,23 +17,13 @@
   },
   "dependencies": {
     "@effect-patterns/ep-shared-services": "workspace:*",
-    "@effect-patterns/pipeline-state": "workspace:*",
     "@effect-patterns/toolkit": "workspace:*",
-    "@effect/cli": "^0.73.0",
-    "@effect/platform": "^0.94.1",
-    "@effect/platform-node": "^0.104.0",
-    "conventional-commits-parser": "^6.2.1",
-    "conventional-recommended-bump": "^11.2.0",
-    "dotenv": "^17.2.3",
-    "effect": "^3.19.14",
+    "@effect/cli": "0.73.2",
+    "@effect/platform": "0.94.4",
+    "@effect/platform-node": "0.104.1",
+    "effect": "^3.19.16",
     "effect-cli-tui": "^2.2.0",
-    "effect-env": "^0.4.1",
-    "glob": "^13.0.1",
-    "js-yaml": "^4.1.1",
-    "liquidjs": "^10.24.0",
-    "ora": "^9.0.0",
-    "semver": "^7.7.3",
-    "yaml": "^2.8.2"
+    "effect-env": "^0.4.1"
   },
   "devDependencies": {
     "tsx": "^4.21.0",
@@ -57,7 +47,7 @@
   },
   "homepage": "https://github.com/PaulJPhilp/Effect-Patterns#readme",
   "engines": {
-    "node": ">=18.0.0"
+    "bun": ">=1.0.0"
   },
   "keywords": [
     "effect",

packages/ep-cli/src/__tests__/cli-surface.test.ts

@@ -0,0 +1,40 @@
+import { Effect, Exit } from "effect";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+import { runCli } from "../index.js";
+
+const run = (argv: ReadonlyArray<string>) => Effect.runPromiseExit(runCli(argv));
+
+describe("ep-cli command surface", () => {
+  beforeEach(() => {
+    vi.spyOn(console, "log").mockImplementation(() => undefined);
+    vi.spyOn(console, "error").mockImplementation(() => undefined);
+    vi.spyOn(console, "warn").mockImplementation(() => undefined);
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it("runs root help successfully", async () => {
+    const exit = await run(["bun", "ep", "--help"]);
+    expect(Exit.isSuccess(exit)).toBe(true);
+  });
+
+  it("exposes only public end-user commands", async () => {
+    const expectedCommands = ["search", "list", "show", "install", "skills"];
+
+    for (const command of expectedCommands) {
+      const exit = await run(["bun", "ep", command, "--help"]);
+      expect(Exit.isSuccess(exit)).toBe(true);
+    }
+  });
+
+  it("rejects maintainer-only commands", async () => {
+    const removedCommands = ["admin", "pattern", "release"];
+
+    for (const command of removedCommands) {
+      const exit = await run(["bun", "ep", command]);
+      expect(Exit.isFailure(exit)).toBe(true);
+    }
+  });
+});