From 3a388b98a9aa6a14b1c7f7dc909c109cf9837800 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 16 Aug 2026 00:59:54 +0900 Subject: 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) --- crates/shirabe/src/command/base_command.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'crates/shirabe/src/command/base_command.rs') 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>, - pub(crate) io: std::cell::RefCell>>>, + composer: std::cell::RefCell>, + io: std::cell::RefCell>>>, /// 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>>, - pub(crate) composer_options: + composer_options: std::cell::RefCell>>, } @@ -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 } } -- cgit v1.3.1-4-g156e