show only diff stats in version review

Author: Thanga-Ganapathy

CHANGELOG.md

@@ -16,7 +16,7 @@ adheres to [Semantic Versioning](https://semver.org/). Work in progress lives un
 ### Fixed
 - **version** — Modified `git checkout -b` to `git checkout -B` so that release branch creation gracefully handles previously abandoned branches by resetting them instead of crashing.
 - **version** — Removed the startup `gh` confirmation prompt and moved confirmation to a final
-  review that shows the computed plan, changed files, and full git diff before commit/push/PR.
+  review that shows the computed plan and changed-file stats before commit/push/PR.
 - **init/npm** — npm workspace discovery now skips workspace manifests that are not release
   packages because they lack `name` or `version`, prints each skipped manifest with the reason,
   and still fails on malformed `package.json` files.

crates/cli/tests/publish_flow.rs

@@ -109,9 +109,6 @@ impl GitOps for FakeGit {
     fn diff_stat(&self) -> Result<String> {
         Ok(String::new())
     }
-    fn diff(&self) -> Result<String> {
-        Ok(String::new())
-    }
     fn reset_hard(&self) -> Result<()> {
         Ok(())
     }

crates/cli/tests/version_flow.rs

@@ -210,7 +210,7 @@ fn version_flow_releases_on_a_branch_and_never_touches_main() {
     let confirmations = prompt.confirmations.borrow();
     assert_eq!(confirmations.len(), 1);
     assert!(confirmations[0].contains("Changed Files:"));
-    assert!(confirmations[0].contains("Diff:"));
+    assert!(!confirmations[0].contains("Diff:"));
     assert!(confirmations[0].contains("packages/core/package.json"));
 
     // main is untouched: core is still 1.0.0 there.

crates/core/src/git.rs

@@ -91,7 +91,6 @@ pub trait GitOps {
     fn create_branch(&self, name: &str) -> Result<()>;
     fn checkout_branch(&self, name: &str) -> Result<()>;
     fn diff_stat(&self) -> Result<String>;
-    fn diff(&self) -> Result<String>;
     fn reset_hard(&self) -> Result<()>;
     fn add_all(&self) -> Result<()>;
     fn commit(&self, message: &str) -> Result<()>;
@@ -125,10 +124,6 @@ impl GitOps for GitRepo {
         run_git(&self.root, &["diff", "--stat"])
     }
 
-    fn diff(&self) -> Result<String> {
-        run_git(&self.root, &["diff"])
-    }
-
     fn reset_hard(&self) -> Result<()> {
         run_git(&self.root, &["reset", "--hard"]).map(|_| ())
     }

crates/core/src/version.rs

@@ -366,7 +366,7 @@ pub fn orchestrate_many(
 
     // 9. Final review: show the actual files and diff produced by the release edits. On cancel,
     // discard only the generated release-branch changes and return to the original branch.
-    let review_text = render_final_review(&summary_text, &git.diff_stat()?, &git.diff()?, opts);
+    let review_text = render_final_review(&summary_text, &git.diff_stat()?, opts);
     if !prompt.confirm(&review_text)? {
         git.reset_hard()?;
         git.checkout_branch(&branch)?;
@@ -395,12 +395,7 @@ pub fn orchestrate_many(
     Ok(())
 }
 
-fn render_final_review(
-    summary_text: &str,
-    diff_stat: &str,
-    diff: &str,
-    opts: &VersionOptions,
-) -> String {
+fn render_final_review(summary_text: &str, diff_stat: &str, opts: &VersionOptions) -> String {
     let mut out = String::new();
     out.push_str(summary_text);
     if opts.skip_pr {
@@ -418,15 +413,6 @@ fn render_final_review(
             out.push('\n');
         }
     }
-    out.push_str("\nDiff:\n");
-    if diff.trim().is_empty() {
-        out.push_str("  (empty diff)\n");
-    } else {
-        out.push_str(diff);
-        if !diff.ends_with('\n') {
-            out.push('\n');
-        }
-    }
     out.push('\n');
     out
 }
@@ -675,10 +661,6 @@ mod tests {
             Ok(" CHANGELOG.md | 2 ++\n 1 file changed, 2 insertions(+)\n".to_string())
         }
 
-        fn diff(&self) -> Result<String> {
-            Ok("diff --git a/CHANGELOG.md b/CHANGELOG.md\n".to_string())
-        }
-
         fn reset_hard(&self) -> Result<()> {
             Ok(())
         }

docs/commands/version.md

@@ -42,7 +42,7 @@ Implemented in `crates/core/src/version.rs`.
      stub `_Dependency updates._`. See [changelog-format.md](../changelog-format.md).
    - `adapter.update_lockfile` — refresh the lockfile in the **same commit**, or a CI install
      will drift.
-10. **Final review / confirm** — print the actual `git diff --stat` and `git diff`, then ask
+10. **Final review / confirm** — print the actual `git diff --stat`, then ask
     whether to commit, push, and open the PR. On cancel, generated release-branch changes are
     discarded and the command returns to the original branch.
 11. **Commit** (`chore(release): …`), **push**, and **open a PR** via `gh`.
@@ -66,9 +66,6 @@ Changed Files:
   Cargo.toml      | 2 +-
   CHANGELOG.md    | 8 +++++++-
 
-Diff:
-diff --git a/Cargo.toml b/Cargo.toml
-...
 ```
 
 Three blocks: explicitly **selected** packages, **auto-bumped** dependents (with the reason),