From 3a388b98a9aa6a14b1c7f7dc909c109cf9837800 Mon Sep 17 00:00:00 2001
From: nsfisis <nsfisis@gmail.com>
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) <noreply@anthropic.com>
---
 crates/shirabe/src/installer/library_installer.rs | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

(limited to 'crates/shirabe/src/installer/library_installer.rs')

diff --git a/crates/shirabe/src/installer/library_installer.rs b/crates/shirabe/src/installer/library_installer.rs
index cab1e206..5f3b2054 100644
--- a/crates/shirabe/src/installer/library_installer.rs
+++ b/crates/shirabe/src/installer/library_installer.rs
@@ -24,13 +24,12 @@ pub struct LibraryInstaller {
     pub(crate) composer: PartialComposerWeakHandle,
     /// Behind a RefCell so initialize_vendor_dir can canonicalize it through `&self` (the
     /// installer instance is shared between concurrent package operations).
-    pub(crate) vendor_dir: std::cell::RefCell<String>,
-    pub(crate) download_manager:
-        Option<std::rc::Rc<std::cell::RefCell<dyn DownloadManagerInterface>>>,
+    vendor_dir: std::cell::RefCell<String>,
+    download_manager: Option<std::rc::Rc<std::cell::RefCell<dyn DownloadManagerInterface>>>,
     pub(crate) io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
-    pub(crate) r#type: Option<String>,
-    pub(crate) filesystem: std::rc::Rc<std::cell::RefCell<Filesystem>>,
-    pub(crate) binary_installer: std::rc::Rc<std::cell::RefCell<dyn BinaryInstallerInterface>>,
+    r#type: Option<String>,
+    filesystem: std::rc::Rc<std::cell::RefCell<Filesystem>>,
+    binary_installer: std::rc::Rc<std::cell::RefCell<dyn BinaryInstallerInterface>>,
 }
 
 impl LibraryInstaller {
@@ -121,7 +120,7 @@ impl LibraryInstaller {
     ///
     /// It is used for BC as getInstallPath tends to be overridden by
     /// installer plugins but not getPackageBasePath
-    pub(crate) fn get_package_base_path(&self, package: PackageInterfaceHandle) -> String {
+    fn get_package_base_path(&self, package: PackageInterfaceHandle) -> String {
         let install_path = self.get_install_path(package.clone()).unwrap();
         let target_dir = package.get_target_dir();
 
@@ -142,7 +141,7 @@ impl LibraryInstaller {
         install_path
     }
 
-    pub(crate) async fn install_code(
+    async fn install_code(
         &self,
         package: PackageInterfaceHandle,
     ) -> anyhow::Result<Option<PhpMixed>> {
@@ -154,7 +153,7 @@ impl LibraryInstaller {
             .await
     }
 
-    pub(crate) async fn update_code(
+    async fn update_code(
         &self,
         initial: PackageInterfaceHandle,
         target: PackageInterfaceHandle,
@@ -183,7 +182,7 @@ impl LibraryInstaller {
             .await
     }
 
-    pub(crate) async fn remove_code(
+    async fn remove_code(
         &self,
         package: PackageInterfaceHandle,
     ) -> anyhow::Result<Option<PhpMixed>> {
@@ -195,7 +194,7 @@ impl LibraryInstaller {
             .await
     }
 
-    pub(crate) fn initialize_vendor_dir(&self) {
+    fn initialize_vendor_dir(&self) {
         self.filesystem
             .borrow_mut()
             .ensure_directory_exists(&self.vendor_dir.borrow());
@@ -203,7 +202,7 @@ impl LibraryInstaller {
         *self.vendor_dir.borrow_mut() = realpath;
     }
 
-    pub(crate) fn get_download_manager(
+    fn get_download_manager(
         &self,
     ) -> &std::rc::Rc<std::cell::RefCell<dyn DownloadManagerInterface>> {
         // PHP: assert($this->downloadManager instanceof DownloadManager, new \LogicException(...))
-- 
cgit v1.3.1-4-g156e