Merge pull request #264 from PaulJPhilp/ep-cli/polish-fixes

fix(ep-cli): pre-release polish — tabs, dead code, error messages

Author: PaulJPhilp

packages/ep-cli/src/commands/install-commands.ts

@@ -2,22 +2,22 @@
  * Install Commands
  */
 
-import { Args, Command, Options, Prompt } from "@effect/cli";
+import { Command, Options, Prompt } from "@effect/cli";
 import { FileSystem } from "@effect/platform";
 import { Console, Effect, Option } from "effect";
 import path from "node:path";
 import { Display } from "../services/display/index.js";
 import { Install, InstalledRule } from "../services/install/index.js";
 import { colorize } from "../utils.js";
-import { DisabledFeatureError, UnsupportedToolError } from "../errors.js";
+import { UnsupportedToolError } from "../errors.js";
 
 /**
  * install:add - Add rules to AI tool configuration
  */
 export const installAddCommand = Command.make("add", {
   options: {
     tool: Options.text("tool").pipe(
-      Options.withDescription("The AI tool to add rules for (cursor, agents, etc.)")
+      Options.withDescription("The AI tool to install rules for (cursor, agents, vscode, windsurf)")
     ),
     skillLevel: Options.optional(
       Options.text("skill-level").pipe(
@@ -146,56 +146,6 @@ export const installAddCommand = Command.make("add", {
   )
 );
 
-/**
- * install:remove - Remove installed rules
- */
-export const installRemoveCommand = Command.make("remove", {
-  args: {
-    ruleId: Args.optional(Args.text({ name: "rule-id" }))
-  }
-}).pipe(
-  Command.withDescription("Temporarily disabled until server-backed tool config installation is implemented."),
-  Command.withHandler(({ args }) =>
-    Effect.gen(function* () {
-      yield* Display.showError(
-        "The install remove workflow is disabled for launch because tool-file injection is not implemented yet."
-      );
-      yield* Display.showInfo(
-        "Re-enable 'install remove' after server-backed injection and uninstall support is implemented."
-      );
-      return yield* Effect.fail(new DisabledFeatureError({
-        feature: "install remove",
-        reason: "Tool-file injection is not implemented yet",
-      }));
-    })
-  )
-);
-
-/**
- * install:diff - Compare installed rule with latest
- */
-export const installDiffCommand = Command.make("diff", {
-  args: {
-    ruleId: Args.text({ name: "rule-id" })
-  }
-}).pipe(
-  Command.withDescription("Temporarily disabled until server-backed tool config installation is implemented."),
-  Command.withHandler(({ args }) =>
-    Effect.gen(function* () {
-      yield* Display.showError(
-        "The install diff workflow is disabled for launch because tool-file injection is not implemented yet."
-      );
-      yield* Display.showInfo(
-        "Re-enable 'install diff' after server-backed source of truth and file-level diffing are implemented."
-      );
-      return yield* Effect.fail(new DisabledFeatureError({
-        feature: "install diff",
-        reason: "Server-backed source of truth and file-level diffing are not implemented yet",
-      }));
-    })
-  )
-);
-
 /**
  * Display installed rules or message if none exist
  */

packages/ep-cli/src/services/display/helpers.ts

@@ -11,31 +11,31 @@ import { Logger } from "../logger/index.js";
  * Icons for different display levels
  */
 export const ICONS = {
-	success: "✓",
-	error: "✖",
-	info: "ℹ",
-	warning: "⚠",
-	highlight: "📌",
+  success: "✓",
+  error: "✖",
+  info: "ℹ",
+  warning: "⚠",
+  highlight: "📌",
 } as const;
 
 /**
  * Get logger config from Logger service
  */
 export const getLoggerConfig = (): Effect.Effect<LoggerConfig, never, Logger> =>
-	Effect.gen(function* () {
-		const logger = yield* Logger;
-		const config = yield* logger.getConfig();
-		return config;
-	});
+  Effect.gen(function* () {
+    const logger = yield* Logger;
+    const config = yield* logger.getConfig();
+    return config;
+  });
 
 /**
  * Apply color if enabled in config (respects Logger config)
  */
 export const colorizeWithConfig = (
-	text: string,
-	color: keyof typeof colors,
-	config: LoggerConfig
+  text: string,
+  color: keyof typeof colors,
+  config: LoggerConfig
 ): string => {
-	if (!config.useColors) return text;
-	return `${colors[color]}${text}${colors.RESET}`;
+  if (!config.useColors) return text;
+  return `${colors[color]}${text}${colors.RESET}`;
 };