chore(hermes): drop the unreachable review mode

The Hermes plugin only ever builds context from `_current_mode` or the
default, and both accept off/lite/full/ultra only, so the review branch,
CONFIG_MODES, _normalize_config_mode and REVIEW_SKILL never ran outside a
test. Removed them and the test that exercised only that code.

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

Author: manuvarghese456-collab

__init__.py

@@ -10,7 +10,6 @@
 
 DEFAULT_MODE = "full"
 RUNTIME_MODES = {"off", "lite", "full", "ultra"}
-CONFIG_MODES = RUNTIME_MODES | {"review"}
 SKILL_COMMANDS = {
     "ponytail-review": "Review the current diff or provided target: bugs, security, load, missing tests, speed, and what to cut.",
     "ponytail-audit": "Audit the whole repo: bugs, security, load, missing tests, speed, and what to cut.",
@@ -22,7 +21,6 @@
 ROOT = Path(__file__).resolve().parent
 SKILLS_DIR = ROOT / "skills"
 PONYTAIL_SKILL = SKILLS_DIR / "ponytail" / "SKILL.md"
-REVIEW_SKILL = SKILLS_DIR / "ponytail-review" / "SKILL.md"
 
 _current_mode = None
 
@@ -34,13 +32,6 @@ def _normalize_runtime_mode(mode: str | None) -> str | None:
     return mode if mode in RUNTIME_MODES else None
 
 
-def _normalize_config_mode(mode: str | None) -> str | None:
-    if not isinstance(mode, str):
-        return None
-    mode = mode.strip().lower()
-    return mode if mode in CONFIG_MODES else None
-
-
 def _config_dir() -> Path:
     if os.environ.get("XDG_CONFIG_HOME"):
         return Path(os.environ["XDG_CONFIG_HOME"]) / "ponytail"
@@ -110,17 +101,9 @@ def _fallback_instructions(mode: str) -> str:
 
 def build_injected_context(mode: str | None = None) -> str:
     """Return the mode-filtered Ponytail context injected before LLM turns."""
-    configured = _normalize_config_mode(mode) or _default_mode()
-    if configured == "off":
+    effective = _normalize_runtime_mode(mode) or _default_mode()
+    if effective == "off":
         return ""
-    if configured == "review":
-        try:
-            body = REVIEW_SKILL.read_text(encoding="utf-8")
-            return f"PONYTAIL MODE ACTIVE — level: review\n\n{_strip_frontmatter(body)}"
-        except OSError:
-            return "PONYTAIL MODE ACTIVE — level: review. Review the diff for bugs, risks, load, missing tests, speed and bloat; explain each finding in plain English."
-
-    effective = _normalize_runtime_mode(configured) or DEFAULT_MODE
     try:
         body = PONYTAIL_SKILL.read_text(encoding="utf-8")
         return f"PONYTAIL MODE ACTIVE — level: {effective}\n\n{_filter_skill_body_for_mode(body, effective)}"

tests/hermes-plugin.test.js

@@ -177,7 +177,7 @@ print(json.dumps(mod._pre_llm_call()))
   }
 });
 
-test('Hermes rejects review as a default while keeping explicit review mode', () => {
+test('Hermes treats review as a command, never a level', () => {
   const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'ponytail-review-default-'));
   fs.mkdirSync(path.join(tmp, 'ponytail'), { recursive: true });
   fs.writeFileSync(path.join(tmp, 'ponytail', 'config.json'), JSON.stringify({ defaultMode: 'review' }));
@@ -199,23 +199,7 @@ print(json.dumps({
   const data = JSON.parse(output);
   assert.match(data.env_default, /PONYTAIL MODE ACTIVE — level: full/);
   assert.match(data.file_default, /PONYTAIL MODE ACTIVE — level: full/);
-  assert.match(data.explicit_review, /PONYTAIL MODE ACTIVE — level: review/);
-});
-
-test('Hermes plugin review mode injects the real review skill body', () => {
-  const output = python(String.raw`
-import importlib.util, json
-spec = importlib.util.spec_from_file_location('ponytail_hermes_plugin', '__init__.py')
-mod = importlib.util.module_from_spec(spec)
-spec.loader.exec_module(mod)
-ctx = mod.build_injected_context('review')
-print(json.dumps({'ctx': ctx}))
-`);
-  const { ctx } = JSON.parse(output);
-  assert.match(ctx, /PONYTAIL MODE ACTIVE — level: review/);
-  assert.match(ctx, /Review a change like the senior developer/);
-  assert.match(ctx, /Verdict: Ship\./);
-  assert.doesNotMatch(ctx, /^---/);
+  assert.match(data.explicit_review, /PONYTAIL MODE ACTIVE — level: full/);
 });
 
 test('Hermes /ponytail command changes mode and pre_llm_call injects current context', () => {