diff --git a/cli/src/commands/branch/delete.rs b/cli/src/commands/branch/delete.rs
index 088393f10..b868c3506 100644
--- a/cli/src/commands/branch/delete.rs
+++ b/cli/src/commands/branch/delete.rs
@@ -30,11 +30,11 @@ pub struct BranchDeleteArgs {
     /// select branches by wildcard pattern. For details, see
     /// https://github.com/martinvonz/jj/blob/main/docs/revsets.md#string-patterns.
     #[arg(required_unless_present_any(&["glob"]), value_parser = StringPattern::parse)]
-    pub names: Vec<StringPattern>,
+    names: Vec<StringPattern>,
 
     /// Deprecated. Please prefix the pattern with `glob:` instead.
     #[arg(long, hide = true, value_parser = StringPattern::glob)]
-    pub glob: Vec<StringPattern>,
+    glob: Vec<StringPattern>,
 }
 
 pub fn cmd_branch_delete(
diff --git a/cli/src/commands/branch/forget.rs b/cli/src/commands/branch/forget.rs
index 375109b7a..91d9a82a1 100644
--- a/cli/src/commands/branch/forget.rs
+++ b/cli/src/commands/branch/forget.rs
@@ -33,11 +33,11 @@ pub struct BranchForgetArgs {
     /// select branches by wildcard pattern. For details, see
     /// https://github.com/martinvonz/jj/blob/main/docs/revsets.md#string-patterns.
     #[arg(required_unless_present_any(&["glob"]), value_parser = StringPattern::parse)]
-    pub names: Vec<StringPattern>,
+    names: Vec<StringPattern>,
 
     /// Deprecated. Please prefix the pattern with `glob:` instead.
     #[arg(long, hide = true, value_parser = StringPattern::glob)]
-    pub glob: Vec<StringPattern>,
+    glob: Vec<StringPattern>,
 }
 
 pub fn cmd_branch_forget(
diff --git a/cli/src/commands/branch/list.rs b/cli/src/commands/branch/list.rs
index a150bbd25..2cdc8fdf8 100644
--- a/cli/src/commands/branch/list.rs
+++ b/cli/src/commands/branch/list.rs
@@ -55,7 +55,7 @@ pub struct BranchListArgs {
     /// select branches by wildcard pattern. For details, see
     /// https://github.com/martinvonz/jj/blob/main/docs/revsets.md#string-patterns.
     #[arg(value_parser = StringPattern::parse)]
-    pub names: Vec<StringPattern>,
+    names: Vec<StringPattern>,
 
     /// Show branches whose local targets are in the given revisions.
     ///
diff --git a/cli/src/commands/branch/rename.rs b/cli/src/commands/branch/rename.rs
index 2082b68f7..8f81a4058 100644
--- a/cli/src/commands/branch/rename.rs
+++ b/cli/src/commands/branch/rename.rs
@@ -27,10 +27,10 @@ use crate::ui::Ui;
 #[derive(clap::Args, Clone, Debug)]
 pub struct BranchRenameArgs {
     /// The old name of the branch.
-    pub old: String,
+    old: String,
 
     /// The new name of the branch.
-    pub new: String,
+    new: String,
 }
 
 pub fn cmd_branch_rename(
diff --git a/cli/src/commands/branch/set.rs b/cli/src/commands/branch/set.rs
index 18ba43042..86e9ce841 100644
--- a/cli/src/commands/branch/set.rs
+++ b/cli/src/commands/branch/set.rs
@@ -25,15 +25,15 @@ use crate::ui::Ui;
 pub struct BranchSetArgs {
     /// The branch's target revision.
     #[arg(long, short)]
-    pub revision: Option<RevisionArg>,
+    revision: Option<RevisionArg>,
 
     /// Allow moving the branch backwards or sideways.
     #[arg(long, short = 'B')]
-    pub allow_backwards: bool,
+    allow_backwards: bool,
 
     /// The branches to update.
     #[arg(required = true)]
-    pub names: Vec<String>,
+    names: Vec<String>,
 }
 
 pub fn cmd_branch_set(
diff --git a/cli/src/commands/branch/track.rs b/cli/src/commands/branch/track.rs
index b392aba1b..3e0a4b77d 100644
--- a/cli/src/commands/branch/track.rs
+++ b/cli/src/commands/branch/track.rs
@@ -35,7 +35,7 @@ pub struct BranchTrackArgs {
     ///
     /// Examples: branch@remote, glob:main@*, glob:jjfan-*@upstream
     #[arg(required = true, value_name = "BRANCH@REMOTE")]
-    pub names: Vec<RemoteBranchNamePattern>,
+    names: Vec<RemoteBranchNamePattern>,
 }
 
 pub fn cmd_branch_track(
diff --git a/cli/src/commands/branch/untrack.rs b/cli/src/commands/branch/untrack.rs
index ca90650ba..44f5fe0fd 100644
--- a/cli/src/commands/branch/untrack.rs
+++ b/cli/src/commands/branch/untrack.rs
@@ -33,7 +33,7 @@ pub struct BranchUntrackArgs {
     ///
     /// Examples: branch@remote, glob:main@*, glob:jjfan-*@upstream
     #[arg(required = true, value_name = "BRANCH@REMOTE")]
-    pub names: Vec<RemoteBranchNamePattern>,
+    names: Vec<RemoteBranchNamePattern>,
 }
 
 pub fn cmd_branch_untrack(