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/repository/vcs_repository.rs | 28 ++++++++++++-------------
 1 file changed, 14 insertions(+), 14 deletions(-)

(limited to 'crates/shirabe/src/repository/vcs_repository.rs')

diff --git a/crates/shirabe/src/repository/vcs_repository.rs b/crates/shirabe/src/repository/vcs_repository.rs
index 4da3f0ad..4066ac44 100644
--- a/crates/shirabe/src/repository/vcs_repository.rs
+++ b/crates/shirabe/src/repository/vcs_repository.rs
@@ -37,36 +37,36 @@ use shirabe_semver::constraint::SimpleConstraint;
 // TODO(port): the driver registration should be refactored later.
 #[derive(Debug)]
 pub struct VcsRepository {
-    pub(crate) inner: ArrayRepository,
+    inner: ArrayRepository,
     /// @var string
-    pub(crate) url: String,
+    url: String,
     /// @var ?string
     ///
     /// Interior mutability: set lazily by the (now `&self`) `initialize`, mirroring how PHP's
     /// inherited ArrayRepository methods drive the overridden `initialize()` on first access.
-    pub(crate) package_name: std::cell::RefCell<Option<String>>,
+    package_name: std::cell::RefCell<Option<String>>,
     /// @var bool
-    pub(crate) is_verbose: bool,
+    is_verbose: bool,
     /// @var bool
-    pub(crate) is_very_verbose: bool,
+    is_very_verbose: bool,
     /// @var IOInterface
-    pub(crate) io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
+    io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
     /// @var Config
-    pub(crate) config: std::rc::Rc<std::cell::RefCell<Config>>,
+    config: std::rc::Rc<std::cell::RefCell<Config>>,
     /// @var VersionParser
-    pub(crate) version_parser: std::cell::RefCell<Option<VersionParser>>,
+    version_parser: std::cell::RefCell<Option<VersionParser>>,
     /// @var string
-    pub(crate) r#type: String,
+    r#type: String,
     /// @var ?LoaderInterface
-    pub(crate) loader: std::cell::RefCell<Option<Box<dyn LoaderInterface>>>,
+    loader: std::cell::RefCell<Option<Box<dyn LoaderInterface>>>,
     /// @var array<string, mixed>
-    pub(crate) repo_config: IndexMap<String, PhpMixed>,
+    repo_config: IndexMap<String, PhpMixed>,
     /// @var HttpDownloader
-    pub(crate) http_downloader: std::rc::Rc<std::cell::RefCell<HttpDownloader>>,
+    http_downloader: std::rc::Rc<std::cell::RefCell<HttpDownloader>>,
     /// @var ProcessExecutor
-    pub(crate) process_executor: std::rc::Rc<std::cell::RefCell<ProcessExecutor>>,
+    process_executor: std::rc::Rc<std::cell::RefCell<ProcessExecutor>>,
     /// @var bool
-    pub(crate) branch_error_occurred: std::cell::Cell<bool>,
+    branch_error_occurred: std::cell::Cell<bool>,
     /// @var array<string, class-string<VcsDriverInterface>>
     drivers: IndexMap<String, VcsDriverKind>,
     /// @var ?VcsDriverInterface
-- 
cgit v1.3.1-4-g156e