feat(targets): add Cursor 2.4 skills and subagent support

Update CursorTarget for Cursor 2.4 release features:
- Skills now use Agent Skills standard (.cursor/skills/<name>/SKILL.md)
  instead of MDC rules format (.cursor/rules/<name>.mdc)
- Add subagent support (.cursor/agents/<name>.md) with model: inherit
- Remove unused _rewrite_relative_paths helper
- Simplify implementation by reusing BaseAssistantTarget.remove_skill

Based on: https://cursor.com/changelog/2-4

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: 2 people

AGENTS.md

@@ -105,12 +105,13 @@ Defined in `targets.py` TARGETS dict. Each assistant has different output format
 | Assistant | Skills | Commands | Agents |
 |-----------|--------|----------|--------|
 | claude-code | `.claude/skills/<module>-<skill>/SKILL.md` | `.claude/commands/<module>-<cmd>.md` | `.claude/agents/<module>-<agent>.md` |
-| cursor | `.cursor/rules/<module>-<skill>.mdc` | `.cursor/commands/<module>-<cmd>.md` | N/A |
+| cursor | `.cursor/skills/<module>-<skill>/SKILL.md` | `.cursor/commands/<module>-<cmd>.md` | `.cursor/agents/<module>-<agent>.md` |
 | gemini-cli | `GEMINI.md` (managed section) | `.gemini/commands/<module>-<cmd>.toml` | N/A |
 | opencode | `AGENTS.md` (managed section) | `.opencode/commands/<module>-<cmd>.md` | `.opencode/agent/<module>-<agent>.md` |
 
 Agent frontmatter is modified during generation:
-- Claude Code: `model: inherit` is added
+- Claude Code: `name` and `model: inherit` are added
+- Cursor: `name` and `model: inherit` are added
 - OpenCode: `mode: subagent` is added
 
 ### Source Handlers

src/lola/targets/__init__.py

@@ -29,7 +29,7 @@
 
 # Concrete target implementations
 from lola.targets.claude_code import ClaudeCodeTarget
-from lola.targets.cursor import CursorTarget, _rewrite_relative_paths
+from lola.targets.cursor import CursorTarget
 from lola.targets.gemini import GeminiTarget, _convert_to_gemini_args
 from lola.targets.opencode import OpenCodeTarget
 
@@ -90,7 +90,6 @@ def get_target(assistant: str) -> AssistantTarget:
     "_get_content_path",
     "_get_skill_description",
     "_skill_source_dir",
-    "_rewrite_relative_paths",
     "_convert_to_gemini_args",
     "_generate_passthrough_command",
     "_generate_agent_with_frontmatter",

src/lola/targets/cursor.py

@@ -1,49 +1,41 @@
 """
 Cursor target implementation for lola.
 
-Cursor uses .mdc rule files with alwaysApply: true for instructions,
-avoiding inconsistent AGENTS.md loading behavior.
+Cursor 2.4+ supports:
+- Skills in .cursor/skills/<skill-name>/SKILL.md (Agent Skills standard)
+- Subagents in .cursor/agents/<name>.md
+- Rules in .cursor/rules/*.mdc for always-on instructions
 """
 
 from __future__ import annotations
 
-import re
+import shutil
 from pathlib import Path
 
 import lola.config as config
-import lola.frontmatter as fm
-from .base import MCPSupportMixin, BaseAssistantTarget, _generate_passthrough_command
-
-
-def _rewrite_relative_paths(content: str, assets_path: str) -> str:
-    """Rewrite relative paths in content to point to the assets location."""
-    patterns = [
-        (r'(\s|^|"|\x27|\(|`)(\.\./[^\s"\x27)\]`]+)', r"\1" + assets_path + r"/\2"),
-        (r'(\s|^|"|\x27|\(|`)(\./([^\s"\x27)\]`]+))', r"\1" + assets_path + r"/\3"),
-    ]
-    result = content
-    for pattern, replacement in patterns:
-        result = re.sub(pattern, replacement, result)
-    result = re.sub(r"(?<!:)//+", "/", result)
-    return result
+from .base import (
+    MCPSupportMixin,
+    BaseAssistantTarget,
+    _generate_passthrough_command,
+    _generate_agent_with_frontmatter,
+)
 
 
 class CursorTarget(MCPSupportMixin, BaseAssistantTarget):
-    """Target for Cursor assistant.
-
-    Cursor uses .mdc rule files with alwaysApply: true for instructions,
-    avoiding inconsistent AGENTS.md loading behavior.
-    """
+    """Target for Cursor assistant."""
 
     name = "cursor"
-    supports_agents = False
+    supports_agents = True
 
     def get_skill_path(self, project_path: str) -> Path:
-        return Path(project_path) / ".cursor" / "rules"
+        return Path(project_path) / ".cursor" / "skills"
 
     def get_command_path(self, project_path: str) -> Path:
         return Path(project_path) / ".cursor" / "commands"
 
+    def get_agent_path(self, project_path: str) -> Path:
+        return Path(project_path) / ".cursor" / "agents"
+
     def get_instructions_path(self, project_path: str) -> Path:
         return Path(project_path) / ".cursor" / "rules"
 
@@ -55,44 +47,36 @@ def generate_skill(
         source_path: Path,
         dest_path: Path,
         skill_name: str,
-        project_path: str | None = None,
+        project_path: str | None = None,  # noqa: ARG002
     ) -> bool:
-        """Convert skill to Cursor MDC format."""
+        """Copy skill directory with SKILL.md and supporting files.
+
+        Cursor 2.4+ uses the Agent Skills standard with SKILL.md files.
+        """
         if not source_path.exists():
             return False
 
-        dest_path.mkdir(parents=True, exist_ok=True)
+        skill_dest = dest_path / skill_name
+        skill_dest.mkdir(parents=True, exist_ok=True)
 
-        # Calculate assets path for relative path rewriting
-        if project_path:
-            try:
-                relative_source = source_path.relative_to(Path(project_path))
-                assets_path = str(relative_source)
-            except ValueError:
-                assets_path = str(source_path)
-        else:
-            assets_path = str(source_path)
-
-        # Convert SKILL.md to MDC format
+        # Copy SKILL.md
         skill_file = source_path / config.SKILL_FILE
         if not skill_file.exists():
             return False
 
-        content = skill_file.read_text()
-        frontmatter, body = fm.parse(content)
-
-        if assets_path:
-            body = _rewrite_relative_paths(body, assets_path)
-
-        mdc_lines = ["---"]
-        mdc_lines.append(f"description: {frontmatter.get('description', '')}")
-        mdc_lines.append("globs:")
-        mdc_lines.append("alwaysApply: false")
-        mdc_lines.append("---")
-        mdc_lines.append("")
-        mdc_lines.append(body)
-
-        (dest_path / f"{skill_name}.mdc").write_text("\n".join(mdc_lines))
+        (skill_dest / "SKILL.md").write_text(skill_file.read_text())
+
+        # Copy supporting files
+        for item in source_path.iterdir():
+            if item.name == "SKILL.md":
+                continue
+            dest_item = skill_dest / item.name
+            if item.is_dir():
+                if dest_item.exists():
+                    shutil.rmtree(dest_item)
+                shutil.copytree(item, dest_item)
+            else:
+                shutil.copy2(item, dest_item)
         return True
 
     def generate_command(
@@ -105,6 +89,29 @@ def generate_command(
         filename = self.get_command_filename(module_name, cmd_name)
         return _generate_passthrough_command(source_path, dest_dir, filename)
 
+    def generate_agent(
+        self,
+        source_path: Path,
+        dest_dir: Path,
+        agent_name: str,
+        module_name: str,
+    ) -> bool:
+        """Generate agent file with Cursor-compatible frontmatter.
+
+        Cursor subagents use:
+        - name: unique identifier (defaults to filename)
+        - description: when to use this agent
+        - model: "fast", "inherit", or specific model ID
+        """
+        filename = self.get_agent_filename(module_name, agent_name)
+        agent_full_name = filename.removesuffix(".md")
+        return _generate_agent_with_frontmatter(
+            source_path,
+            dest_dir,
+            filename,
+            {"name": agent_full_name, "model": "inherit"},
+        )
+
     def generate_instructions(
         self,
         source_path: Path,
@@ -135,14 +142,6 @@ def generate_instructions(
         mdc_file.write_text("\n".join(mdc_lines))
         return True
 
-    def remove_skill(self, dest_path: Path, skill_name: str) -> bool:
-        """Remove .mdc file instead of directory."""
-        mdc_file = dest_path / f"{skill_name}.mdc"
-        if mdc_file.exists():
-            mdc_file.unlink()
-            return True
-        return False
-
     def remove_instructions(self, dest_path: Path, module_name: str) -> bool:
         """Remove the module's instructions .mdc file."""
         mdc_file = dest_path / f"{module_name}-instructions.mdc"

tests/test_agents.py

@@ -163,10 +163,10 @@ def test_claude_code_supports_agents(self):
         target = get_target("claude-code")
         assert target.supports_agents is True
 
-    def test_cursor_does_not_support_agents(self):
-        """Cursor does not support agents."""
+    def test_cursor_supports_agents(self):
+        """Cursor 2.4+ supports subagents."""
         target = get_target("cursor")
-        assert target.supports_agents is False
+        assert target.supports_agents is True
 
     def test_gemini_does_not_support_agents(self):
         """Gemini doesn't support agents."""
@@ -179,6 +179,12 @@ def test_get_agent_path_claude_project(self, tmp_path):
         path = target.get_agent_path(str(tmp_path))
         assert path == tmp_path / ".claude" / "agents"
 
+    def test_get_agent_path_cursor_project(self, tmp_path):
+        """Get Cursor project agent path (2.4+)."""
+        target = get_target("cursor")
+        path = target.get_agent_path(str(tmp_path))
+        assert path == tmp_path / ".cursor" / "agents"
+
     def test_get_agent_path_gemini_returns_none(self):
         """Gemini's get_agent_path returns None."""
         target = get_target("gemini-cli")

tests/test_config.py

@@ -120,11 +120,11 @@ def test_gemini_cli_project(self, tmp_path):
         assert "GEMINI.md" in str(path)
 
     def test_cursor_project(self, tmp_path):
-        """Get cursor project skill path."""
+        """Get cursor project skill path (Cursor 2.4+)."""
         target = get_target("cursor")
         path = target.get_skill_path(str(tmp_path))
         assert isinstance(path, Path)
-        assert "rules" in str(path)
+        assert "skills" in str(path)
 
     def test_unknown_assistant(self):
         """Raise error for unknown assistant."""

tests/test_converters.py

@@ -51,30 +51,30 @@ def test_claude_skill_generation(self, tmp_path):
         assert result == skill_content
 
     def test_cursor_skill_generation(self, tmp_path):
-        """Generate skill for Cursor (MDC format)."""
+        """Generate skill for Cursor (2.4+ SKILL.md format)."""
         skill_dir = tmp_path / "myskill"
         skill_dir.mkdir()
-        (skill_dir / "SKILL.md").write_text("""---
+        skill_content = """---
 name: myskill
 description: Test skill for cursor
 ---
 
 # My Skill
 
 Content.
-""")
+"""
+        (skill_dir / "SKILL.md").write_text(skill_content)
 
         target = get_target("cursor")
         dest = tmp_path / "dest"
         success = target.generate_skill(skill_dir, dest, "test-myskill", str(tmp_path))
 
         assert success
-        mdc_file = dest / "test-myskill.mdc"
-        assert mdc_file.exists()
-        result = mdc_file.read_text()
-        assert "description: Test skill for cursor" in result
-        assert "globs:" in result
-        assert "alwaysApply: false" in result
+        skill_dest = dest / "test-myskill"
+        assert skill_dest.exists()
+        assert (skill_dest / "SKILL.md").exists()
+        result = (skill_dest / "SKILL.md").read_text()
+        assert result == skill_content
 
 
 class TestCommandConverters: