aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/src/command/config_command.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-08-16 00:59:54 +0900
committernsfisis <nsfisis@gmail.com>2026-08-16 01:03:45 +0900
commit3a388b98a9aa6a14b1c7f7dc909c109cf9837800 (patch)
tree593e342313c6e6bd3ec2943a1690b8798e0a83ae /crates/shirabe/src/command/config_command.rs
parentaad468e8b75ffc3e87ea6dfa22c53a8299fc08da (diff)
downloadphp-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.tar.gz
php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.tar.zst
php-shirabe-3a388b98a9aa6a14b1c7f7dc909c109cf9837800.zip
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>
Diffstat (limited to 'crates/shirabe/src/command/config_command.rs')
-rw-r--r--crates/shirabe/src/command/config_command.rs13
1 files changed, 6 insertions, 7 deletions
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<Option<std::rc::Rc<std::cell::RefCell<JsonFile>>>>,
config_source: std::cell::RefCell<Option<JsonConfigSource>>,
- pub(crate) auth_config_file:
- std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<JsonFile>>>>,
- pub(crate) auth_config_source: std::cell::RefCell<Option<JsonConfigSource>>,
+ auth_config_file: std::cell::RefCell<Option<std::rc::Rc<std::cell::RefCell<JsonFile>>>>,
+ auth_config_source: std::cell::RefCell<Option<JsonConfigSource>>,
}
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,