fix(cli): drop unimplemented --brief/--fields/--count/--exists; fix install --verbose mock (final review)

- Remove four unimplemented flags (--brief, --fields, --count, --exists) from the
  common parent parser in jrag.py. These flags were registered but never read by
  any handler, violating the "inapplicable flags never silently ignored" principle.
- Remove references to these flags from skills/explore-codebase-cli/SKILL.md and
  agents/explorer-rag-cli.md documentation.
- Fix test_cmd_install_forwards_verbose_flag mock signature to accept surface="mcp"
  (matching run_install's new signature from PR-5), and assert the default forwarding.
- Run sync_agent_artifacts.py to propagate documentation changes to install_data.

Fixes #END_OF_PLAN_FULL_SUITE

Author: HumanBean17

agents/explorer-rag-cli.md

@@ -56,10 +56,6 @@ Run `jrag --help` for the canonical list. Groups:
 --module <name>       Filter by module
 --limit <N>           Cap on results (default 20; 10 for fan-out commands)
 --format text|json    Output format (default: text)
---brief               Compact output
---fields a,b,c        Field allowlist
---count               Return only the count
---exists              Return only an exists boolean (exit 0/2)
 --index-dir <path>    Index directory override
 ```
 

java_codebase_rag/install_data/agents/explorer-rag-cli.md

@@ -56,10 +56,6 @@ Run `jrag --help` for the canonical list. Groups:
 --module <name>       Filter by module
 --limit <N>           Cap on results (default 20; 10 for fan-out commands)
 --format text|json    Output format (default: text)
---brief               Compact output
---fields a,b,c        Field allowlist
---count               Return only the count
---exists              Return only an exists boolean (exit 0/2)
 --index-dir <path>    Index directory override
 ```
 

java_codebase_rag/install_data/skills/explore-codebase-cli/SKILL.md

@@ -62,10 +62,6 @@ Run `jrag --help` for the canonical list. Groups (PR-JRAG-1a..4):
 --module <name>       Filter by module
 --limit <N>           Cap on results (default 20; 10 for fan-out commands)
 --format text|json    Output format (default: text)
---brief               Compact output (fewer fields per node)
---fields a,b,c        Field allowlist for node projections
---count               Return only the count (no node rows)
---exists              Return only an exists boolean (exit 0/2)
 --index-dir <path>    Index directory override (default: discovered from cwd)
 ```
 

java_codebase_rag/jrag.py

@@ -154,21 +154,6 @@ def build_parser() -> argparse.ArgumentParser:
         default="text",
         help="Output format (default: text).",
     )
-    common.add_argument(
-        "--brief", action="store_true", help="Compact output (fewer fields per node)."
-    )
-    common.add_argument(
-        "--fields",
-        type=str,
-        default=None,
-        help="Comma-separated field allowlist for node projections.",
-    )
-    common.add_argument(
-        "--count", action="store_true", help="Return only the count (no node rows)."
-    )
-    common.add_argument(
-        "--exists", action="store_true", help="Return only an exists boolean (exit 0/2)."
-    )
 
     status = subparsers.add_parser(
         "status",

skills/explore-codebase-cli/SKILL.md

@@ -62,10 +62,6 @@ Run `jrag --help` for the canonical list. Groups (PR-JRAG-1a..4):
 --module <name>       Filter by module
 --limit <N>           Cap on results (default 20; 10 for fan-out commands)
 --format text|json    Output format (default: text)
---brief               Compact output (fewer fields per node)
---fields a,b,c        Field allowlist for node projections
---count               Return only the count (no node rows)
---exists              Return only an exists boolean (exit 0/2)
 --index-dir <path>    Index directory override (default: discovered from cwd)
 ```
 

tests/test_java_codebase_rag_cli.py

@@ -1511,10 +1511,11 @@ def test_cmd_install_forwards_verbose_flag(
     captured: dict = {}
 
     def _fake_run_install(*, non_interactive, agents, scope, model,
-                          source_root=None, quiet=False, verbose=False):
+                          source_root=None, quiet=False, verbose=False, surface="mcp"):
         captured["quiet"] = quiet
         captured["verbose"] = verbose
         captured["non_interactive"] = non_interactive
+        captured["surface"] = surface
         return 0
 
     monkeypatch.setattr(_installer, "run_install", _fake_run_install)
@@ -1525,6 +1526,7 @@ def _fake_run_install(*, non_interactive, agents, scope, model,
     )
     assert rc == 0
     assert captured["verbose"] is True
+    assert captured["surface"] == "mcp"
     # quiet still flows through too.
     rc2 = cli_mod.main(
         ["install", "--non-interactive", "--agent", "claude-code", "--quiet"]