feat(record)!: use default commit message for --stash

But only when no message is explicitly supplied.

This is a breaking change. Previously, the user would be prompted for a message
if none was supplied.

Author: claytonrcarter

git-branchless-record/src/lib.rs

@@ -31,8 +31,8 @@ use lib::core::rewrite::{
     RepoResource,
 };
 use lib::git::{
-    process_diff_for_record, update_index, CategorizedReferenceName, FileMode, GitRunInfo,
-    MaybeZeroOid, NonZeroOid, Repo, ResolvedReferenceInfo, Stage, UpdateIndexCommand,
+    process_diff_for_record, update_index, CategorizedReferenceName, ConfigRead, FileMode,
+    GitRunInfo, MaybeZeroOid, NonZeroOid, Repo, ResolvedReferenceInfo, Stage, UpdateIndexCommand,
     WorkingCopyChangesType, WorkingCopySnapshot,
 };
 use lib::try_exit_code;
@@ -157,6 +157,16 @@ fn record(
             )?);
         }
     } else {
+        let config = repo.get_readonly_config()?;
+        let interactive_stash = config.get_or("branchless.record.interactiveStash", true)?;
+        let messages = if messages.is_empty() && stash && !interactive_stash {
+            vec![config.get_or(
+                "branchless.record.defautStashMessage",
+                "temp: stash".to_string(),
+            )?]
+        } else {
+            messages
+        };
         let args = {
             let mut args = vec!["commit"];
             args.extend(messages.iter().flat_map(|message| ["--message", message]));

git-branchless-record/tests/test_record.rs

@@ -342,6 +342,74 @@ fn test_record_stash() -> eyre::Result<()> {
     Ok(())
 }
 
+#[test]
+fn test_record_stash_default_message() -> eyre::Result<()> {
+    let git = make_git()?;
+
+    if !git.supports_reference_transactions()? {
+        return Ok(());
+    }
+    git.init_repo()?;
+
+    git.run(&["config", "branchless.record.interactiveStash", "false"])?;
+
+    git.commit_file("test1", 1)?;
+
+    {
+        git.write_file_txt("test1", "new test1 contents\n")?;
+
+        let (stdout, _stderr) = git.branchless("record", &["--stash"])?;
+        insta::assert_snapshot!(stdout, @r###"
+        [master 78c965b] temp: stash
+         1 file changed, 1 insertion(+), 1 deletion(-)
+        branchless: running command: <git-executable> branch -f master 62fc20d2a290daea0d52bdc2ed2ad4be6491010e
+        branchless: running command: <git-executable> checkout master
+        "###);
+    }
+
+    {
+        let stdout = git.smartlog()?;
+        insta::assert_snapshot!(stdout, @r###"
+        :
+        @ 62fc20d (> master) create test1.txt
+        |
+        o 78c965b temp: stash
+        "###);
+    }
+
+    {
+        git.run(&[
+            "config",
+            "branchless.record.defautStashMessage",
+            "foo(bar)!: baz",
+        ])?;
+
+        git.write_file_txt("test1", "more new test1 contents\n")?;
+
+        let (stdout, _stderr) = git.branchless("record", &["--stash"])?;
+        insta::assert_snapshot!(stdout, @r###"
+        [master e574dfa] foo(bar)!: baz
+         1 file changed, 1 insertion(+), 1 deletion(-)
+        branchless: running command: <git-executable> branch -f master 62fc20d2a290daea0d52bdc2ed2ad4be6491010e
+        branchless: running command: <git-executable> checkout master
+        "###);
+    }
+
+    {
+        let stdout = git.smartlog()?;
+        insta::assert_snapshot!(stdout, @r###"
+        :
+        @ 62fc20d (> master) create test1.txt
+        |\
+        | o 78c965b temp: stash
+        |
+        o e574dfa foo(bar)!: baz
+        "###);
+    }
+
+    Ok(())
+}
+
 #[test]
 fn test_record_create_branch() -> eyre::Result<()> {
     let git = make_git()?;