diff --git a/cli/src/cli_util.rs b/cli/src/cli_util.rs
index 273acc4faa3..64efc0cdd2d 100644
--- a/cli/src/cli_util.rs
+++ b/cli/src/cli_util.rs
@@ -522,6 +522,8 @@ impl ReadonlyUserRepo {
     }
 }
 
+/// Type-safe helper for conditionally advancing bookmarks
+///
 /// A bookmark that should be advanced to satisfy the "advance-bookmarks"
 /// feature. This is a helper for `WorkspaceCommandTransaction`. It provides a
 /// type-safe way to separate the work of checking whether a bookmark can be
@@ -1999,7 +2001,8 @@ Then run `jj squash` to move the resolution into the conflicted commit."#,
     }
 }
 
-/// A [`Transaction`] tied to a particular workspace.
+/// Ongoing [`Transaction`] tied to a particular workspace
+///
 /// `WorkspaceCommandTransaction`s are created with
 /// [`WorkspaceCommandHelper::start_transaction`] and committed with
 /// [`WorkspaceCommandTransaction::finish`]. The inner `Transaction` can also be
diff --git a/cli/src/config.rs b/cli/src/config.rs
index c22871dc234..f8cb46a31cf 100644
--- a/cli/src/config.rs
+++ b/cli/src/config.rs
@@ -455,10 +455,11 @@ pub fn existing_config_path() -> Result<Option<PathBuf>, ConfigError> {
     ConfigEnv::new().existing_config_path()
 }
 
-/// Returns a path to the user-specific config file. If no config file is found,
-/// tries to guess a reasonable new location for it. If a path to a new config
-/// file is returned, the parent directory may be created as a result of this
-/// call.
+/// Return a path to the user-specific config file
+///
+/// If no config file is found, tries to guess a reasonable new
+/// location for it. If a path to a new config file is returned, the
+/// parent directory may be created as a result of this call.
 pub fn new_config_path() -> Result<Option<PathBuf>, ConfigError> {
     ConfigEnv::new().new_config_path()
 }