feat: revamp min init output - #773

crates/op/src/project/init.rs (1)

329-346: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

Assert the complete session package behavior.

This test passes even if base or vim disappears, and no test covers adding git when .git exists. Assert the exact package list for both repository states, preferably by parsing the generated TOML.

Proposed coverage

     assert!(out.contains("[session]"));
+    assert!(out.contains(r#"packages = ["base", "vim"]"#));
 }
+
+#[test]
+fn generate_mfile_adds_git_to_session_for_git_repository() {
+    let origin = SpecOrigin::Repo(Repo::Git {
+        url: "u".to_string(),
+        rev: "r".to_string(),
+        tracking: None,
+    });
+    let cwd = TempDir::new().unwrap();
+    std::fs::create_dir(cwd.path().join(".git")).unwrap();
+
+    let out = generate_mfile(None, &origin, cwd.path());
+
+    assert!(out.contains(r#"packages = ["base", "vim", "git"]"#));
+}

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/op/src/project/init.rs` around lines 329 - 346, Strengthen
generate_mfile_falls_back_to_upstream_with_session by parsing the generated TOML
and asserting the complete session package list, including base, vim, and git
when the temporary directory contains a .git directory. Add coverage for both
repository states: without .git, verify the expected packages excluding git;
with .git, verify git is included, while preserving the existing assertions for
absent stack, branch, and package sections.