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/config_command.rs | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'crates/shirabe/src/command/config_command.rs') diff --git a/crates/shirabe/src/command/config_command.rs b/crates/shirabe/src/command/config_command.rs index 3d92de4b..504a0f11 100644 --- a/crates/shirabe/src/command/config_command.rs +++ b/crates/shirabe/src/command/config_command.rs @@ -38,16 +38,15 @@ pub struct ConfigCommand { config_file: std::cell::RefCell>>>, config_source: std::cell::RefCell>, - pub(crate) auth_config_file: - std::cell::RefCell>>>, - pub(crate) auth_config_source: std::cell::RefCell>, + auth_config_file: std::cell::RefCell>>>, + auth_config_source: std::cell::RefCell>, } impl_php_class!(ConfigCommand, r"Composer\Command\ConfigCommand"); impl ConfigCommand { /// List of additional configurable package-properties - pub(crate) const CONFIGURABLE_PACKAGE_PROPERTIES: &'static [&'static str] = &[ + const CONFIGURABLE_PACKAGE_PROPERTIES: &'static [&'static str] = &[ "name", "type", "description", @@ -77,7 +76,7 @@ impl ConfigCommand { command } - pub(crate) fn handle_single_value( + fn handle_single_value( &self, key: &str, callbacks: &(ValidatorFn, NormalizerFn), @@ -143,7 +142,7 @@ impl ConfigCommand { Ok(()) } - pub(crate) fn handle_multi_value( + fn handle_multi_value( &self, key: &str, callbacks: &(ValidatorFn, NormalizerFn), @@ -181,7 +180,7 @@ impl ConfigCommand { } /// Display the contents of the file in a pretty formatted way - pub(crate) fn list_configuration( + fn list_configuration( &self, contents: PhpMixed, raw_contents: PhpMixed, -- cgit v1.3.1-4-g156e