fix: address PR #562 review feedback

Author: web-flow

.gitignore

@@ -30,9 +30,6 @@ node_modules
 
 # Codex CLI
 .codex/
-# AGENTS.md is committed as the canonical context file (per OpenAI harness engineering).
-# Codex/Cursor/OpenCode read it directly; Claude Code reads it via @AGENTS.md import
-# in CLAUDE.md; Gemini CLI reads it via .gemini/settings.json `context.fileName`.
 
 # Cursor
 .cursor/
@@ -42,23 +39,18 @@ node_modules
 .opencode/
 opencode.json
 
-.codex/
-.agents/
-
-# OpenCode runtime (skills + workflows)
-.agents/
-
-# Cursor agent workflows
-.agent/
+# Copilot
+.copilot/
 
 # Gemini CLI (legacy paths kept for compatibility)
 commands/**/*.toml
 /commands/
 /agents/
 /skills/
 
-# Local agent caches
-.agent/
-.agents/
+# Antigravity CLI
 .antigravity/
 
+# Local agent runtime caches (IDE/2.0 discovery)
+.agent/
+.agents/

AGENTS.md

@@ -45,8 +45,8 @@ make generate HARNESS=codex      # emits .codex/skills, .codex/agents
 make generate HARNESS=cursor     # emits .cursor-plugin/, .cursor/rules/
 make generate HARNESS=opencode   # emits .opencode/agents/, .opencode/commands/, .opencode/skills/
 make generate HARNESS=gemini     # emits skills/, agents/, commands/ at extension root
-make generate HARNESS=antigravity # emits .antigravity/agents/, .antigravity/skills/
-make generate-all                # all five
+make generate HARNESS=antigravity # emits .antigravity/agents/, .antigravity/skills/, .antigravity/workflows/
+make generate-all                # all six
 make install-opencode            # symlink generated OpenCode artifacts into global config
 ```
 
@@ -61,7 +61,7 @@ Source-of-truth lives only under `plugins/`. Generated artifacts are gitignored
 - **OpenCode**: mirrored to `.opencode/skills/<plugin>-<skill>/` using hyphenated names for global install
 - **Cursor**: reads `.claude/skills/` directly (no re-emit)
 - **Gemini CLI**: native skills at `skills/<plugin>__<skill>/SKILL.md`
-- **Antigravity CLI**: `.antigravity/skills/<plugin>-<skill>/SKILL.md` (lowercase, hyphen-separated names)
+- **Antigravity CLI**: `.antigravity/skills/<plugin>__<skill>/SKILL.md` (lowercase, double-underscore namespacing)
 
 Top-level `skills/` is Gemini output; do not use it for OpenCode installs.
 
@@ -73,7 +73,7 @@ Top-level `skills/` is Gemini output; do not use it for OpenCode installs.
 - **OpenCode**: `.opencode/agents/<plugin>__<agent>.md` with `mode: subagent` + `permission:` block (locked agents — those with source `tools: []` — get deny-everything except base `skill`/`task`)
 - **Gemini**: `agents/<plugin>__<agent>.md` (April 2026 subagent spec)
 - **Cursor**: reads `.claude/agents/` directly
-- **Antigravity**: `.antigravity/agents/<agent>/agent.json` (Gemini model aliases, lowercase tool names)
+- **Antigravity**: `.antigravity/agents/<plugin>__<agent>/agent.json` (Gemini model aliases, lowercase tool names)
 
 ## Why this file is short
 

Makefile

@@ -22,7 +22,7 @@ help:
 	@echo "claude-agents — multi-harness plugin marketplace"
 	@echo "================================================="
 	@echo ""
-	@echo "Multi-harness adapter (Codex / Cursor / OpenCode / Gemini):"
+	@echo "Multi-harness adapter (Codex / Cursor / OpenCode / Gemini / Copilot / Antigravity):"
 	@echo "  make generate HARNESS=<h> [PLUGIN=<p>]           Generate per-harness artifacts (defaults to all plugins)"
 	@echo "  make generate-all                                Generate for ALL harnesses + ALL plugins"
 	@echo "  make clean-generated [HARNESS=<h>]               Remove generated artifacts"

README.md

@@ -84,7 +84,7 @@ emits harness-native artifacts (not lowest-common-denominator translations):
 | **OpenCode** | `.opencode/agents/`, `.opencode/commands/`, `.opencode/skills/` | `permission:` block from `tools:` allowlist; OpenCode-safe skill names |
 | **Gemini CLI** | `skills/`, `agents/`, `commands/` (TOML) | Native skills + subagents (April 2026 spec) |
 | **Copilot** | `.copilot/agents/`, `.copilot/skills/`, `.copilot/commands/` | Markdown agent profiles + SKILL.md skills + commands-as-skills; model maps to GPT-5 family |
-| **Antigravity CLI** | `.antigravity/agents/`, `.antigravity/skills/`, `.antigravity/commands/` | Lowercase agent.json per agent + SKILL.md skills + commands-as-skills; model maps to Gemini 2.5 family |
+| **Antigravity CLI** | `.antigravity/agents/`, `.antigravity/skills/`, `.antigravity/workflows/` | Lowercase agent.json per agent + SKILL.md skills + workflows (commands); model maps to Gemini 2.5 family |
 
 ```bash
 make generate-all                        # all six
@@ -142,7 +142,7 @@ integrations for this marketplace's other supported harnesses.
 | OpenCode | [integrations/opencode-plugin](https://github.com/major7apps/pensyve/tree/main/integrations/opencode-plugin) |
 | Gemini CLI | `gemini extensions install https://github.com/major7apps/pensyve` |
 | Copilot | `.copilot/` in repo root or `~/.copilot/` via `make install-copilot` |
-| Antigravity CLI | `.antigravity/` in repo root or `~/.antigravity/` via `make install-antigravity` |
+| Antigravity CLI | `.antigravity/` in repo root or `~/.gemini/antigravity-cli/` via `make install-antigravity` |
 
 ## License
 

docs/authoring.md

@@ -106,7 +106,7 @@ clean naming — pick distinct names for skill/command pairs within a plugin.
 ### Model aliases
 
 | Source field | Codex | Cursor | OpenCode | Gemini | Copilot | Antigravity CLI |
-|---|---|---|---|---|---|---|---|---|
+|---|---|---|---|---|---|---|---|
 | `model: opus` | `gpt-5` | `inherit` | `anthropic/claude-opus-4-7` | `gemini-2.5-pro` | `gpt-5` | `gemini-2.5-pro` |
 | `model: sonnet` | `gpt-5-mini` | `inherit` | `anthropic/claude-sonnet-4-6` | `gemini-2.5-pro` | `gpt-5-mini` | `gemini-2.5-pro` |
 | `model: haiku` | `gpt-5-nano` | `inherit` | `anthropic/claude-haiku-4-5-20251001` | `gemini-2.5-flash` | `gpt-5-nano` | `gemini-2.5-flash` |

docs/harnesses.md

@@ -23,7 +23,7 @@ as Claude Code markdown. Per-harness artifacts are generated by adapters under `
 |:---|---|---|---|---|---|---|
 | Skills (SKILL.md native) | ✅ | ✅ | ✅ via `.claude/` | ✅ via `.opencode/skills/` | ✅ (auto-discovered) | ✅ |
 | Subagents (markdown native) | ✅ | TOML format | ✅ via `.claude/` | ✅ (different frontmatter) | ✅ (April 2026 spec) | JSON format |
-| Slash commands | ✅ | converted to skills | ✅ | ✅ | TOML at `commands/` | — |
+| Slash commands | ✅ | converted to skills | ✅ | ✅ | TOML at `commands/` | ✅ via `.antigravity/workflows/` |
 | Plugin marketplace | ✅ | — | ✅ (2.5+) | — | — (direct URL install) | — |
 | Parallel subagents | ✅ | ✅ | ✅ | ✅ | ✅ (April 2026) | ✅ |
 | Per-agent tool allowlist | ✅ (`tools:`) | only `sandbox_mode` | only `readonly:` | ✅ (`permission:` block) | ✅ (`tools:`) | ✅ (`tools:`) |
@@ -50,7 +50,7 @@ rules to write portable content.
 | `TodoWrite` in body | no equivalent — leave as-is | no equivalent — leave as-is | works as-is | no equivalent | no equivalent |
 | Skill body > 8 KB | split into `references/details.md` | passed through | passed through | passed through | passed through |
 | Agent named `worker` | namespaced to `<plugin>__worker` | passed through | passed through | passed through | namespaced to `<plugin>__<agent>` |
-| Slash command (`commands/<x>.md`) | converted to skill | passed through | rewritten to `.opencode/commands/` | TOML at `commands/<plugin>/<x>.toml` | converted to skill |
+| Slash command (`commands/<x>.md`) | converted to skill | passed through | rewritten to `.opencode/commands/` | TOML at `commands/<plugin>/<x>.toml` | rewritten to `.antigravity/workflows/<plugin>-<x>.md` |
 
 ## Output paths (all gitignored)
 

tools/adapters/antigravity.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import json
+import re
 from pathlib import Path
 
 from tools.adapters.base import (
@@ -17,28 +18,64 @@
 from tools.adapters.capabilities import TOOL_NAME_MAPS, resolve_model
 
 
+_YAML_NEEDS_QUOTING = re.compile(r"[:#{}[\]&*?|<>!=%@`,\n]|^\s+|\s+$|^- |^[>|]")
+
+
+def _needs_yaml_quoting(value: str) -> bool:
+    """True when a scalar value must be quoted to remain valid YAML."""
+    if not isinstance(value, str):
+        return True
+    if not value:
+        return True
+    if value.lower() in ("true", "false", "yes", "no", "on", "off", "null", "~"):
+        return True
+    try:
+        int(value)
+        return True
+    except ValueError:
+        pass
+    return bool(_YAML_NEEDS_QUOTING.search(value))
+
+
+def _yaml_value(value: str) -> str:
+    """Serialize a scalar value as safe YAML (quoting if necessary)."""
+    if _needs_yaml_quoting(value):
+        escaped = value.replace("\\", "\\\\").replace('"', '\\"')
+        return f'"{escaped}"'
+    return value
+
+
 def _antigravity_frontmatter(fm: dict) -> str:
     """Format YAML frontmatter for Antigravity skill files."""
     lines = ["---"]
     for k, v in fm.items():
         if isinstance(v, list):
-            value = ", ".join(str(x) for x in v)
-            lines.append(f"{k}: [{value}]")
+            items = ", ".join(_yaml_value(str(x)) for x in v)
+            lines.append(f"{k}: [{items}]")
         elif isinstance(v, bool):
             lines.append(f"{k}: {'true' if v else 'false'}")
         elif v is not None:
             value = str(v).replace("\n", " ").strip()
-            lines.append(f"{k}: {value}")
+            lines.append(f"{k}: {_yaml_value(value)}")
     lines.append("---")
     return "\n".join(lines)
 
 
+_REWRITE_TOOL_RE = re.compile(r"`(\w+)`|\b(Read|Write|Edit|Grep|Bash|Task|Glob)\b")
+
+
 def _rewrite_body_lowercase_tools(body: str) -> str:
-    """Rewrite Claude Code CamelCase tool names in backticked references to Antigravity names."""
-    out = body
-    for camel, replacement in TOOL_NAME_MAPS["antigravity"].items():
-        out = out.replace(f"`{camel}`", f"`{replacement}`")
-    return out
+    """Rewrite Claude Code CamelCase tool names to Antigravity lowercase names."""
+    tool_map = TOOL_NAME_MAPS["antigravity"]
+
+    def _replace(m: re.Match) -> str:
+        camel = m.group(1) or m.group(2)
+        lower = tool_map.get(camel, camel.lower())
+        if m.group(1):
+            return f"`{lower}`"
+        return lower
+
+    return _REWRITE_TOOL_RE.sub(_replace, body)
 
 
 class AntigravityAdapter(HarnessAdapter):
@@ -71,7 +108,9 @@ def emit_global(self, plugins: list[PluginSource]) -> EmitResult:
         """No cross-plugin artifacts needed for Antigravity."""
         return EmitResult()
 
-    def _emit_agent(self, plugin: PluginSource, agent: AgentSource, result: EmitResult) -> None:
+    def _emit_agent(
+        self, plugin: PluginSource, agent: AgentSource, result: EmitResult
+    ) -> None:
         """Emit one custom agent configuration file (`agent.json`)."""
         agent_id = f"{plugin.name}__{agent.name}"
         agent_dir = Path(".antigravity") / "agents" / agent_id
@@ -105,15 +144,15 @@ def _emit_agent(self, plugin: PluginSource, agent: AgentSource, result: EmitResu
             "description": description,
             "hidden": False,
             "model": model,
-            "customAgentSpec": {
-                "customAgent": custom_agent
-            }
+            "customAgentSpec": {"customAgent": custom_agent},
         }
 
         content = json.dumps(agent_config, indent=2) + "\n"
         result.written.append(self.write(rel_file, content))
 
-    def _emit_skill(self, plugin: PluginSource, skill: SkillSource, result: EmitResult) -> None:
+    def _emit_skill(
+        self, plugin: PluginSource, skill: SkillSource, result: EmitResult
+    ) -> None:
         """Emit one skill into ``.antigravity/skills/<skill_id>/SKILL.md``."""
         skill_id = f"{plugin.name}__{skill.name}"
         skill_dir = Path(".antigravity") / "skills" / skill_id
@@ -130,9 +169,13 @@ def _emit_skill(self, plugin: PluginSource, skill: SkillSource, result: EmitResu
             for ref in sorted(skill.references_dir.rglob("*")):
                 if ref.is_file():
                     rel = ref.relative_to(skill.references_dir)
-                    result.written.append(self.mirror_file(ref, skill_dir / "references" / rel))
+                    result.written.append(
+                        self.mirror_file(ref, skill_dir / "references" / rel)
+                    )
 
-    def _emit_command_as_skill(self, plugin: PluginSource, command: CommandSource, result: EmitResult) -> None:
+    def _emit_command_as_skill(
+        self, plugin: PluginSource, command: CommandSource, result: EmitResult
+    ) -> None:
         """Emit one command as an Antigravity skill."""
         skill_id = f"{plugin.name}-{command.name}"
         skill_dir = Path(".antigravity") / "skills" / skill_id
@@ -159,13 +202,19 @@ def _emit_command_as_skill(self, plugin: PluginSource, command: CommandSource, r
             extra_skill_dir = skill_parent / "skills" / skill_id
             result.written.append(self.write(extra_skill_dir / "SKILL.md", content))
 
-    def _emit_command_as_workflow(self, plugin: PluginSource, command: CommandSource, result: EmitResult) -> None:
+    def _emit_command_as_workflow(
+        self, plugin: PluginSource, command: CommandSource, result: EmitResult
+    ) -> None:
         """Emit one command as an Antigravity workflow (``.md``) for IDE/2.0 slash command support."""
         name = f"{plugin.name}-{command.name}"
         workflow_dir = Path(".antigravity") / "workflows"
         rel_file = workflow_dir / f"{name}.md"
 
-        description = command.description or h1_from_body(command.body) or name.replace("-", " ").title()
+        description = (
+            command.description
+            or h1_from_body(command.body)
+            or name.replace("-", " ").title()
+        )
         body = _rewrite_body_lowercase_tools(command.body).rstrip() + "\n"
 
         fm: dict = {"description": description}

tools/adapters/base.py

@@ -99,7 +99,9 @@ def parse_frontmatter(content: str) -> tuple[dict, str]:
             text = line.strip()
             if text:
                 existing = fields.get(current_key) or ""
-                fields[current_key] = (existing + " " + text).strip() if existing else text
+                fields[current_key] = (
+                    (existing + " " + text).strip() if existing else text
+                )
         elif in_list and (
             isinstance(fields.get(current_key), list)
             or (isinstance(fields.get(current_key), str) and fields[current_key] == "")
@@ -123,7 +125,11 @@ def parse_frontmatter(content: str) -> tuple[dict, str]:
                 item = stripped.strip('",[] ')
                 if item and item != "]":
                     fields[current_key].append(item)
-        elif current_key and isinstance(fields.get(current_key), str) and line.startswith("  "):
+        elif (
+            current_key
+            and isinstance(fields.get(current_key), str)
+            and line.startswith("  ")
+        ):
             fields[current_key] += " " + line.strip().strip('"')
 
     return fields, body
@@ -350,7 +356,9 @@ def load_plugin(plugin_name: str) -> PluginSource | None:
         for md in sorted(agents_dir.glob("*.md")):
             fm, body = parse_frontmatter(read_file(md))
             plugin.agents.append(
-                AgentSource(plugin=plugin_name, name=md.stem, path=md, frontmatter=fm, body=body)
+                AgentSource(
+                    plugin=plugin_name, name=md.stem, path=md, frontmatter=fm, body=body
+                )
             )
 
     skills_dir = plugin_dir / "skills"
@@ -361,15 +369,19 @@ def load_plugin(plugin_name: str) -> PluginSource | None:
                 continue
             fm, body = parse_frontmatter(read_file(skill_file))
             plugin.skills.append(
-                SkillSource(plugin=plugin_name, name=sd.name, dir=sd, frontmatter=fm, body=body)
+                SkillSource(
+                    plugin=plugin_name, name=sd.name, dir=sd, frontmatter=fm, body=body
+                )
             )
 
     commands_dir = plugin_dir / "commands"
     if commands_dir.is_dir():
         for md in sorted(commands_dir.glob("*.md")):
             fm, body = parse_frontmatter(read_file(md))
             plugin.commands.append(
-                CommandSource(plugin=plugin_name, name=md.stem, path=md, frontmatter=fm, body=body)
+                CommandSource(
+                    plugin=plugin_name, name=md.stem, path=md, frontmatter=fm, body=body
+                )
             )
 
     return plugin
@@ -437,7 +449,9 @@ def write(self, rel_path: str | Path, content: str) -> Path:
         target = (self.output_root / rel_path).resolve()
         root = self.output_root.resolve()
         if not target.is_relative_to(root):
-            raise ValueError(f"refusing to write outside output_root: {target} (root={root})")
+            raise ValueError(
+                f"refusing to write outside output_root: {target} (root={root})"
+            )
         target.parent.mkdir(parents=True, exist_ok=True)
         target.write_text(content, encoding="utf-8")
         return target
@@ -447,7 +461,9 @@ def write_bytes(self, rel_path: str | Path, content: bytes) -> Path:
         target = (self.output_root / rel_path).resolve()
         root = self.output_root.resolve()
         if not target.is_relative_to(root):
-            raise ValueError(f"refusing to write outside output_root: {target} (root={root})")
+            raise ValueError(
+                f"refusing to write outside output_root: {target} (root={root})"
+            )
         target.parent.mkdir(parents=True, exist_ok=True)
         target.write_bytes(content)
         return target