fix: enable/disable only change state, sync writes to files

- enable: only sets enabled=True and app_type, does NOT sync to file
- disable: only sets enabled=False, does NOT clear file
- sync: writes prompt content to app file
- unsync: clears app file content

Updated help text to clarify behavior and suggest using sync command.

Author: jzhu

code_assistant_manager/cli/prompts_commands.py

@@ -206,62 +206,51 @@ def enable_prompt(
         "-a",
         help="App type to enable for (claude, codex, gemini)",
     ),
-    level: str = typer.Option(
-        "user",
-        "--level",
-        "-l",
-        help="Enable level: 'user' or 'project'",
-    ),
-    project_dir: Optional[Path] = typer.Option(
-        None,
-        "--project-dir",
-        help="Project directory when using project level (defaults to current directory)",
-    ),
 ):
-    """Enable a prompt and sync it to the app file.
+    """Enable a prompt for an app type.
 
-    This will:
-    - Mark the prompt as enabled for the specified app
-    - Disable other prompts for the same app (user level only)
-    - Sync the prompt content to the app's prompt file
+    This only marks the prompt as enabled and associates it with the app.
+    It does NOT sync content to any files.
+    Use 'sync' to write content to prompt files.
     """
     if app_type not in USER_LEVEL_APPS:
         typer.echo(
             f"{Colors.RED}✗ Invalid app: {app_type}. Valid: {', '.join(USER_LEVEL_APPS)}{Colors.RESET}"
         )
         raise typer.Exit(1)
 
-    if level not in VALID_LEVELS:
-        typer.echo(
-            f"{Colors.RED}✗ Invalid level: {level}. Valid: {', '.join(VALID_LEVELS)}{Colors.RESET}"
-        )
-        raise typer.Exit(1)
-
     manager = _get_prompt_manager()
     prompt = manager.get(prompt_id)
 
     if not prompt:
         typer.echo(f"{Colors.RED}✗ Prompt '{prompt_id}' not found{Colors.RESET}")
         raise typer.Exit(1)
 
-    level_project_dir = (
-        ensure_project_dir(level, project_dir if level == "project" else None)
-        if level == "project"
-        else None
-    )
+    if prompt.enabled and prompt.app_type == app_type:
+        typer.echo(
+            f"{Colors.YELLOW}Prompt '{prompt.name}' is already enabled for {app_type}{Colors.RESET}"
+        )
+        return
 
     try:
-        manager.activate(
-            prompt_id,
-            app_type,
-            level=level,
-            project_dir=level_project_dir,
+        # Disable other prompts for this app type
+        prompts = manager.get_all()
+        for p in prompts.values():
+            if p.id != prompt_id and p.enabled and p.app_type == app_type:
+                p.enabled = False
+                manager.update(p)
+
+        # Enable this prompt
+        prompt.enabled = True
+        prompt.app_type = app_type
+        manager.update(prompt)
+
+        typer.echo(
+            f"{Colors.GREEN}✓ Enabled '{prompt.name}' for {app_type}{Colors.RESET}"
         )
         typer.echo(
-            f"{Colors.GREEN}✓ Enabled '{prompt.name}' for {app_type} ({level}){Colors.RESET}"
+            f"  {Colors.CYAN}Tip:{Colors.RESET} Use 'sync {prompt_id} --app {app_type}' to write to file"
         )
-        file_path = get_prompt_file_path(app_type, level, level_project_dir)
-        typer.echo(f"  {Colors.CYAN}File:{Colors.RESET} {file_path}")
     except Exception as e:
         typer.echo(f"{Colors.RED}✗ Error: {e}{Colors.RESET}")
         raise typer.Exit(1)

tests/unit/test_prompts_cli.py

@@ -234,19 +234,7 @@ def test_sync_prompt_with_enable(cli_manager, tmp_path, monkeypatch):
 
 
 def test_enable_prompt(cli_manager, tmp_path, monkeypatch):
-    """enable command activates a prompt and syncs it."""
-    from code_assistant_manager.prompts import PROMPT_HANDLERS
-
-    # Setup user-level file path
-    user_claude_file = tmp_path / "user_claude.md"
-
-    # Update manager's handler with the new path
-    for name, cls in PROMPT_HANDLERS.items():
-        overrides = {"user_path": user_claude_file} if name == "claude" else {}
-        cli_manager._handlers[name] = cls(
-            user_path_override=overrides.get("user_path"),
-        )
-
+    """enable command sets prompt state to enabled without syncing."""
     prompt = Prompt(id="test-enable", name="Test Enable", content="enable content")
     cli_manager.create(prompt)
 
@@ -258,17 +246,14 @@ def test_enable_prompt(cli_manager, tmp_path, monkeypatch):
     prompts_commands.enable_prompt(
         prompt_id="test-enable",
         app_type="claude",
-        level="user",
-        project_dir=None,
     )
 
-    assert user_claude_file.exists()
-    assert user_claude_file.read_text() == "enable content"
-
     stored = cli_manager.get("test-enable")
     assert stored.enabled is True
     assert stored.app_type == "claude"
     assert any("enabled" in msg.lower() for msg in outputs)
+    # Should suggest using sync
+    assert any("sync" in msg.lower() for msg in outputs)
 
 
 def test_disable_prompt(cli_manager, tmp_path, monkeypatch):
@@ -331,8 +316,6 @@ def test_enable_prompt_invalid_app(cli_manager, tmp_path, monkeypatch):
         prompts_commands.enable_prompt(
             prompt_id="test-invalid-app",
             app_type="copilot",  # copilot doesn't support enable
-            level="user",
-            project_dir=None,
         )
 
     assert any("invalid" in msg.lower() for msg in outputs)