refactor: narrow pub(crate) items to private

Porting mapped every PHP `protected` member onto `pub(crate)`, which is
wider than nearly all of them need. Each item demoted here is reached
only from the module that defines it, so the crate-wide visibility
conveyed nothing.

Every `pub(crate)` that survives has at least one reader in another
module of the same crate.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

1 files changed, 5 insertions, 5 deletions

diff --git a/crates/shirabe/src/command/base_command.rs b/crates/shirabe/src/command/base_command.rs
index 9eb28dd8..d7f935c1 100644
--- a/crates/shirabe/src/command/base_command.rs
+++ b/crates/shirabe/src/command/base_command.rs
@@ -38,16 +38,16 @@ pub const INVALID: i64 = 2;
 #[derive(Debug)]
 pub struct BaseCommandData {
     inner: CommandData,
-    pub(crate) composer: std::cell::RefCell<Option<PartialComposerHandle>>,
-    pub(crate) io: std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<dyn IOInterface>>>>,
+    composer: std::cell::RefCell<Option<PartialComposerHandle>>,
+    io: std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<dyn IOInterface>>>>,
     /// The definition entries that were registered through the Composer-typed
     /// InputArgument/InputOption (which carry the suggested-values backport). PHP checks
     /// `$definition->getArgument($name) instanceof Composer\Console\Input\InputArgument`; in
     /// this port `set_definition` converts the entries to the Symfony types for storage, so
     /// the Composer-typedness is tracked by name in these side maps instead.
-    pub(crate) composer_arguments:
+    composer_arguments:
         std::cell::RefCell<IndexMap<String, std::rc::Rc<crate::console::input::InputArgument>>>,
-    pub(crate) composer_options:
+    composer_options:
         std::cell::RefCell<IndexMap<String, std::rc::Rc<crate::console::input::InputOption>>>,
 }
 
@@ -65,7 +65,7 @@ impl BaseCommandData {
     /// Access to the embedded Symfony command state, used by the Composer-typed definition
     /// builders to forward to `CommandData`'s Symfony-typed entry points. `CommandData` is
     /// interior-mutable, so a shared reference is enough.
-    pub(crate) fn command_data(&self) -> &CommandData {
+    fn command_data(&self) -> &CommandData {
         &self.inner
     }
 }