Resolve Phase 17 review gate feedback

Author: ProfRandom92

PROJEKT.md

@@ -21,7 +21,7 @@ CompText CLI is an experimental terminal context client for building determinist
 ```text
 CURRENT_PHASE: 17
 CURRENT_TASK: Antigravity Plugin Bundle Skeleton
-LAST_GREEN_PHASE: 17
+LAST_GREEN_PHASE: 16
 STATUS: REVIEW-GATE
 NEXT_ALLOWED_ACTION: Await review gate confirmation
 ```

README.md

@@ -88,7 +88,7 @@ CompText is for developers who want AI-assisted workflows with stronger boundari
 Binary: ctxt
 Current phase: Phase 17
 Current task: Antigravity Plugin Bundle Skeleton
-Last green phase: Phase 17
+Last green phase: Phase 16
 Status: REVIEW-GATE
 Next allowed action: Await review gate confirmation
 ```

reports/phase_17_status.md

@@ -1,7 +1,7 @@
 # Phase 17 Status Report
 
 - **PHASE**: Phase 17 Antigravity Plugin Bundle
-- **STATUS**: success
+- **STATUS**: local-success / review-gate
 - **FILES_CHANGED**:
   - `.agent/skills/ctxt-phase-17-antigravity-plugin-bundle/SKILL.md`
   - `PROJEKT.md`
@@ -20,8 +20,10 @@
   - `templates/antigravity/skills/sample_skill/SKILL.md`
   - `tests/cli_smoke.rs`
 - **COMMANDS_RUN**:
+  - `cargo fmt --all --check`
   - `cargo check`
   - `cargo test`
+  - `cargo clippy -- -D warnings`
 - **VALIDATION**: All 34 unit tests and 9 integration tests passed. Compilation clean.
 - **ARTIFACTS**:
   - `templates/antigravity/plugin_manifest.json`
@@ -35,7 +37,11 @@
   - `templates/antigravity/README.md`
   - `docs/ANTIGRAVITY_PLUGIN_BUNDLE.md`
   - `.agent/skills/ctxt-phase-17-antigravity-plugin-bundle/SKILL.md`
-- **GIT**: Switched to main and checked out local branch `phase-17-antigravity-plugin-bundle`. Local commit prepared.
+- **GIT**:
+  - local commits prepared
+  - remote branch pushed
+  - PR #3 opened
+  - merge not performed
 - **NETWORK**: offline-only
 - **SECRETS**: No credentials accessed or exposed.
 - **POLICY_DECISIONS**: All design boundaries respected (no live runtime hooks, advisory-only subagents, MCP treated as untrusted, no LLM judge).

src/cli.rs

@@ -667,88 +667,34 @@ fn parse(argv: &[String]) -> Result<Command, String> {
                         action: None,
                     })
                 }
-                "skills" => {
+                "skills" | "agents" | "hooks" | "plugin" => {
+                    let expected_action = match sub {
+                        "skills" => "validate",
+                        "agents" => "export",
+                        "hooks" => "audit",
+                        "plugin" => "package",
+                        _ => unreachable!(),
+                    };
                     if argv.len() < 3 {
-                        return Err("missing action for 'antigravity skills'. Usage: ctxt antigravity skills validate".to_string());
-                    }
-                    if argv[2] != "validate" {
-                        return Err(format!(
-                            "unsupported action '{}' for 'antigravity skills'",
-                            argv[2]
-                        ));
-                    }
-                    if argv.len() > 3 {
-                        return Err(format!(
-                            "unexpected argument '{}' for 'antigravity skills validate'",
-                            argv[3]
-                        ));
-                    }
-                    Ok(Command::Antigravity {
-                        subcommand: "skills".to_string(),
-                        action: Some("validate".to_string()),
-                    })
-                }
-                "agents" => {
-                    if argv.len() < 3 {
-                        return Err("missing action for 'antigravity agents'. Usage: ctxt antigravity agents export".to_string());
-                    }
-                    if argv[2] != "export" {
-                        return Err(format!(
-                            "unsupported action '{}' for 'antigravity agents'",
-                            argv[2]
-                        ));
-                    }
-                    if argv.len() > 3 {
                         return Err(format!(
-                            "unexpected argument '{}' for 'antigravity agents export'",
-                            argv[3]
-                        ));
-                    }
-                    Ok(Command::Antigravity {
-                        subcommand: "agents".to_string(),
-                        action: Some("export".to_string()),
-                    })
-                }
-                "hooks" => {
-                    if argv.len() < 3 {
-                        return Err("missing action for 'antigravity hooks'. Usage: ctxt antigravity hooks audit".to_string());
-                    }
-                    if argv[2] != "audit" {
-                        return Err(format!(
-                            "unsupported action '{}' for 'antigravity hooks'",
-                            argv[2]
+                            "missing action for 'antigravity {sub}'. Usage: ctxt antigravity {sub} {expected_action}"
                         ));
                     }
-                    if argv.len() > 3 {
-                        return Err(format!(
-                            "unexpected argument '{}' for 'antigravity hooks audit'",
-                            argv[3]
-                        ));
-                    }
-                    Ok(Command::Antigravity {
-                        subcommand: "hooks".to_string(),
-                        action: Some("audit".to_string()),
-                    })
-                }
-                "plugin" => {
-                    if argv.len() < 3 {
-                        return Err("missing action for 'antigravity plugin'. Usage: ctxt antigravity plugin package".to_string());
-                    }
-                    if argv[2] != "package" {
+                    if argv[2] != expected_action {
                         return Err(format!(
-                            "unsupported action '{}' for 'antigravity plugin'",
+                            "unsupported action '{}' for 'antigravity {sub}'",
                             argv[2]
                         ));
                     }
                     if argv.len() > 3 {
                         return Err(format!(
-                            "unexpected argument '{}' for 'antigravity plugin package'",
+                            "unexpected argument '{}' for 'antigravity {sub} {expected_action}'",
                             argv[3]
                         ));
                     }
                     Ok(Command::Antigravity {
-                        subcommand: "plugin".to_string(),
-                        action: Some("package".to_string()),
+                        subcommand: sub.to_string(),
+                        action: Some(expected_action.to_string()),
                     })
                 }
                 other => Err(format!("unsupported antigravity subcommand '{}'", other)),