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>

8 files changed, 67 insertions, 67 deletions

diff --git a/crates/shirabe/src/package/alias_package.rs b/crates/shirabe/src/package/alias_package.rs
index 52b6efa4..9df804f1 100644
--- a/crates/shirabe/src/package/alias_package.rs
+++ b/crates/shirabe/src/package/alias_package.rs
@@ -23,18 +23,18 @@ pub struct AliasPackage {
     repository: Option<RepositoryInterfaceWeakHandle>,
 
     /// @var string
-    pub(crate) version: String,
+    version: String,
     /// @var string
-    pub(crate) pretty_version: String,
+    pretty_version: String,
     /// @var bool
-    pub(crate) dev: bool,
+    dev: bool,
     /// @var bool
-    pub(crate) root_package_alias: bool,
+    root_package_alias: bool,
     /// @var string
     /// @phpstan-var 'stable'|'RC'|'beta'|'alpha'|'dev'
-    pub(crate) stability: String,
+    stability: String,
     /// @var bool
-    pub(crate) has_self_version_requires: bool,
+    has_self_version_requires: bool,
 
     /// @var BasePackage
     pub(crate) alias_of: PackageHandle,
diff --git a/crates/shirabe/src/package/archiver/archivable_files_finder.rs b/crates/shirabe/src/package/archiver/archivable_files_finder.rs
index 72fe0646..8fba18cb 100644
--- a/crates/shirabe/src/package/archiver/archivable_files_finder.rs
+++ b/crates/shirabe/src/package/archiver/archivable_files_finder.rs
@@ -10,7 +10,7 @@ use shirabe_symfony_finder::Finder;
 use std::path::{Path, PathBuf};
 
 pub struct ArchivableFilesFinder {
-    pub(crate) finder: Finder,
+    finder: Finder,
     inner_iter: Box<dyn Iterator<Item = PathBuf>>,
 }
 
diff --git a/crates/shirabe/src/package/archiver/archive_manager.rs b/crates/shirabe/src/package/archiver/archive_manager.rs
index 0d99d301..a0216d88 100644
--- a/crates/shirabe/src/package/archiver/archive_manager.rs
+++ b/crates/shirabe/src/package/archiver/archive_manager.rs
@@ -17,10 +17,10 @@ use shirabe_php_shim::{
 };
 
 pub struct ArchiveManager {
-    pub(crate) download_manager: std::rc::Rc<std::cell::RefCell<DownloadManager>>,
-    pub(crate) r#loop: std::rc::Rc<std::cell::RefCell<Loop>>,
-    pub(crate) archivers: Vec<Box<dyn ArchiverInterface>>,
-    pub(crate) overwrite_files: bool,
+    download_manager: std::rc::Rc<std::cell::RefCell<DownloadManager>>,
+    r#loop: std::rc::Rc<std::cell::RefCell<Loop>>,
+    archivers: Vec<Box<dyn ArchiverInterface>>,
+    overwrite_files: bool,
 }
 
 impl std::fmt::Debug for ArchiveManager {
diff --git a/crates/shirabe/src/package/complete_package.rs b/crates/shirabe/src/package/complete_package.rs
index b2c3978c..00b4a67c 100644
--- a/crates/shirabe/src/package/complete_package.rs
+++ b/crates/shirabe/src/package/complete_package.rs
@@ -11,18 +11,18 @@ use shirabe_php_shim::PhpMixed;
 #[derive(Debug, Clone)]
 pub struct CompletePackage {
     pub(crate) inner: Package,
-    pub(crate) repositories: IndexMap<String, PhpMixed>,
-    pub(crate) license: Vec<String>,
-    pub(crate) keywords: Vec<String>,
-    pub(crate) authors: Vec<IndexMap<String, String>>,
-    pub(crate) description: Option<String>,
-    pub(crate) homepage: Option<String>,
-    pub(crate) scripts: IndexMap<String, Vec<String>>,
-    pub(crate) support: IndexMap<String, String>,
-    pub(crate) funding: Vec<IndexMap<String, PhpMixed>>,
-    pub(crate) abandoned: PhpMixed,
-    pub(crate) archive_name: Option<String>,
-    pub(crate) archive_excludes: Vec<String>,
+    repositories: IndexMap<String, PhpMixed>,
+    license: Vec<String>,
+    keywords: Vec<String>,
+    authors: Vec<IndexMap<String, String>>,
+    description: Option<String>,
+    homepage: Option<String>,
+    scripts: IndexMap<String, Vec<String>>,
+    support: IndexMap<String, String>,
+    funding: Vec<IndexMap<String, PhpMixed>>,
+    abandoned: PhpMixed,
+    archive_name: Option<String>,
+    archive_excludes: Vec<String>,
 }
 
 impl CompletePackage {
diff --git a/crates/shirabe/src/package/link.rs b/crates/shirabe/src/package/link.rs
index 027fa009..fc0a8063 100644
--- a/crates/shirabe/src/package/link.rs
+++ b/crates/shirabe/src/package/link.rs
@@ -5,11 +5,11 @@ use shirabe_semver::constraint::AnyConstraint;
 
 #[derive(Clone, Debug)]
 pub struct Link {
-    pub(crate) source: String,
-    pub(crate) target: String,
-    pub(crate) constraint: AnyConstraint,
-    pub(crate) description: String,
-    pub(crate) pretty_constraint: String,
+    source: String,
+    target: String,
+    constraint: AnyConstraint,
+    description: String,
+    pretty_constraint: String,
 }
 
 impl Link {
diff --git a/crates/shirabe/src/package/loader/array_loader.rs b/crates/shirabe/src/package/loader/array_loader.rs
index df588cf9..21811f14 100644
--- a/crates/shirabe/src/package/loader/array_loader.rs
+++ b/crates/shirabe/src/package/loader/array_loader.rs
@@ -29,7 +29,7 @@ pub struct ArrayLoader {
     /// @var VersionParser
     pub(crate) version_parser: VersionParser,
     /// @var bool
-    pub(crate) load_options: bool,
+    load_options: bool,
 }
 
 impl ArrayLoader {
diff --git a/crates/shirabe/src/package/package.rs b/crates/shirabe/src/package/package.rs
index ee1a5e10..4e35ee55 100644
--- a/crates/shirabe/src/package/package.rs
+++ b/crates/shirabe/src/package/package.rs
@@ -31,41 +31,41 @@ pub struct Package {
     /// Back-reference to the owning repository. `Weak` breaks the repository -> packages cycle.
     repository: Option<RepositoryInterfaceWeakHandle>,
 
-    pub(crate) r#type: Option<String>,
-    pub(crate) target_dir: Option<String>,
+    r#type: Option<String>,
+    target_dir: Option<String>,
     /// `'source'` | `'dist'` | `null`
-    pub(crate) installation_source: Option<String>,
-    pub(crate) source_type: Option<String>,
-    pub(crate) source_url: Option<String>,
-    pub(crate) source_reference: Option<String>,
-    pub(crate) source_mirrors: Option<Vec<Mirror>>,
-    pub(crate) dist_type: Option<String>,
-    pub(crate) dist_url: Option<String>,
-    pub(crate) dist_reference: Option<String>,
-    pub(crate) dist_sha1_checksum: Option<String>,
-    pub(crate) dist_mirrors: Option<Vec<Mirror>>,
-    pub(crate) version: String,
-    pub(crate) pretty_version: String,
-    pub(crate) release_date: Option<chrono::DateTime<chrono::Utc>>,
-    pub(crate) extra: IndexMap<String, PhpMixed>,
-    pub(crate) binaries: Vec<String>,
-    pub(crate) dev: bool,
+    installation_source: Option<String>,
+    source_type: Option<String>,
+    source_url: Option<String>,
+    source_reference: Option<String>,
+    source_mirrors: Option<Vec<Mirror>>,
+    dist_type: Option<String>,
+    dist_url: Option<String>,
+    dist_reference: Option<String>,
+    dist_sha1_checksum: Option<String>,
+    dist_mirrors: Option<Vec<Mirror>>,
+    version: String,
+    pretty_version: String,
+    release_date: Option<chrono::DateTime<chrono::Utc>>,
+    extra: IndexMap<String, PhpMixed>,
+    binaries: Vec<String>,
+    dev: bool,
     /// `'stable'` | `'RC'` | `'beta'` | `'alpha'` | `'dev'`
-    pub(crate) stability: String,
-    pub(crate) notification_url: Option<String>,
+    stability: String,
+    notification_url: Option<String>,
 
-    pub(crate) requires: IndexMap<String, Link>,
-    pub(crate) conflicts: IndexMap<String, Link>,
-    pub(crate) provides: IndexMap<String, Link>,
-    pub(crate) replaces: IndexMap<String, Link>,
-    pub(crate) dev_requires: IndexMap<String, Link>,
-    pub(crate) suggests: IndexMap<String, String>,
-    pub(crate) autoload: IndexMap<String, PhpMixed>,
-    pub(crate) dev_autoload: IndexMap<String, PhpMixed>,
-    pub(crate) include_paths: Vec<String>,
-    pub(crate) is_default_branch: bool,
-    pub(crate) transport_options: IndexMap<String, PhpMixed>,
-    pub(crate) php_ext: Option<IndexMap<String, PhpMixed>>,
+    requires: IndexMap<String, Link>,
+    conflicts: IndexMap<String, Link>,
+    provides: IndexMap<String, Link>,
+    replaces: IndexMap<String, Link>,
+    dev_requires: IndexMap<String, Link>,
+    suggests: IndexMap<String, String>,
+    autoload: IndexMap<String, PhpMixed>,
+    dev_autoload: IndexMap<String, PhpMixed>,
+    include_paths: Vec<String>,
+    is_default_branch: bool,
+    transport_options: IndexMap<String, PhpMixed>,
+    php_ext: Option<IndexMap<String, PhpMixed>>,
 }
 
 impl Package {
diff --git a/crates/shirabe/src/package/root_package.rs b/crates/shirabe/src/package/root_package.rs
index 2c2b87ea..0807d15b 100644
--- a/crates/shirabe/src/package/root_package.rs
+++ b/crates/shirabe/src/package/root_package.rs
@@ -15,12 +15,12 @@ use shirabe_php_shim::PhpMixed;
 #[derive(Debug, Clone)]
 pub struct RootPackage {
     pub(crate) inner: CompletePackage,
-    pub(crate) minimum_stability: String,
-    pub(crate) prefer_stable: bool,
-    pub(crate) stability_flags: IndexMap<String, i64>,
-    pub(crate) config: IndexMap<String, PhpMixed>,
-    pub(crate) references: IndexMap<String, String>,
-    pub(crate) aliases: Vec<IndexMap<String, String>>,
+    minimum_stability: String,
+    prefer_stable: bool,
+    stability_flags: IndexMap<String, i64>,
+    config: IndexMap<String, PhpMixed>,
+    references: IndexMap<String, String>,
+    aliases: Vec<IndexMap<String, String>>,
 }
 
 impl RootPackage {