Merge branch 'issue-6-list-scaffold-r060-stack' into release/0.6.0

Author: blackopsrepl

crates/solverforge-cli/WIREFRAME.md

@@ -87,7 +87,6 @@ Scaffolds a complete SolverForge project from embedded templates.
 |------|----------|----------------|
 | `--basic` | Generic skeleton | Standard (basic) variable |
 | `--basic=employee-scheduling` | Pre-built employee shift scheduling | Standard (basic) variable |
-| `--list` | Generic list-variable skeleton | List variable |
 | `--list=vehicle-routing` | Pre-built CVRP domain | List variable |
 
 **Optional flags:**

crates/solverforge-cli/src/commands/new.rs

@@ -20,7 +20,6 @@ const AVAILABLE_TEMPLATES: &str = "
     --basic=employee-scheduling     — assign employees to shifts
 
   List Variable (each entity owns an ordered sequence):
-    --list                          — generic list-variable skeleton  (coming soon)
     --list=vehicle-routing          — capacitated vehicle routing (CVRP)";
 
 pub fn run(
@@ -54,10 +53,6 @@ pub fn run(
             skip_readme,
             quiet,
         ),
-        Template::List => Err(CliError::with_hint(
-            "the generic list-variable skeleton is not yet available",
-            format!("Available templates:{AVAILABLE_TEMPLATES}"),
-        )),
         Template::ListVehicleRouting => scaffold(
             name,
             &crate_name,
@@ -350,7 +345,6 @@ fn generate_readme(project_name: &str, _crate_name: &str, label: &str) -> String
 pub enum Template {
     Basic,
     BasicEmployeeScheduling,
-    List,
     ListVehicleRouting,
 }
 
@@ -359,7 +353,10 @@ impl Template {
         match (basic, list, specialization) {
             (true, false, None) => Ok(Template::Basic),
             (true, false, Some("employee-scheduling")) => Ok(Template::BasicEmployeeScheduling),
-            (false, true, None) => Ok(Template::List),
+            (false, true, None) => Err(CliError::with_hint(
+                "the --list template requires a specialization",
+                "Use --list=vehicle-routing".to_string(),
+            )),
             (false, true, Some("vehicle-routing")) => Ok(Template::ListVehicleRouting),
             (false, false, None) => Err(CliError::with_hint(
                 "specify a template flag",

crates/solverforge-cli/src/main.rs

@@ -54,7 +54,7 @@ enum Command {
     /// Variable class (required, mutually exclusive):
     ///
     ///   --basic     Standard variable — each entity holds one assigned value
-    ///   --list      List variable     — each entity owns an ordered sequence
+    ///   --list=...  List variable     — each entity owns an ordered sequence
     ///
     /// Specializations (append after the flag with =):
     ///
@@ -71,7 +71,7 @@ enum Command {
         #[arg(long = "basic", value_name = "SPECIALIZATION", num_args = 0..=1, require_equals = true)]
         basic: Option<Option<String>>,
 
-        /// Scaffold a list-variable project (optionally: --list=vehicle-routing)
+        /// Scaffold a list-variable project specialization (currently: --list=vehicle-routing)
         #[arg(long = "list", value_name = "SPECIALIZATION", num_args = 0..=1, require_equals = true)]
         list: Option<Option<String>>,
 

crates/solverforge-cli/tests/scaffold_test.rs

@@ -131,6 +131,33 @@ fn test_new_vehicle_routing_creates_domain() {
     );
 }
 
+// Bare --list should be rejected because the generic list scaffold is not supported.
+#[test]
+fn test_new_list_requires_specialization() {
+    let tmp = tempfile::tempdir().expect("failed to create temp dir");
+
+    let output = Command::new(cli_bin())
+        .args(["new", "test_list_project", "--list"])
+        .current_dir(tmp.path())
+        .output()
+        .expect("failed to run solverforge new");
+
+    assert!(
+        !output.status.success(),
+        "solverforge new --list unexpectedly succeeded"
+    );
+
+    let stderr = String::from_utf8_lossy(&output.stderr);
+    assert!(
+        stderr.contains("the --list template requires a specialization"),
+        "unexpected stderr: {stderr}"
+    );
+    assert!(
+        stderr.contains("Use --list=vehicle-routing"),
+        "unexpected stderr: {stderr}"
+    );
+}
+
 // Full cargo check on a scaffolded basic project.
 // Requires network access and a full Rust toolchain.
 // Run with: cargo test -p solverforge-cli -- --ignored