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/dependency_resolver/solver.rs | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

(limited to 'crates/shirabe/src/dependency_resolver/solver.rs')

diff --git a/crates/shirabe/src/dependency_resolver/solver.rs b/crates/shirabe/src/dependency_resolver/solver.rs
index a9453a3e..cb398a0c 100644
--- a/crates/shirabe/src/dependency_resolver/solver.rs
+++ b/crates/shirabe/src/dependency_resolver/solver.rs
@@ -26,25 +26,25 @@ use shirabe_semver::constraint::AnyConstraint;
 
 #[derive(Debug)]
 pub struct Solver {
-    pub(crate) policy: std::rc::Rc<dyn PolicyInterface>,
-    pub(crate) pool: std::rc::Rc<std::cell::RefCell<Pool>>,
+    policy: std::rc::Rc<dyn PolicyInterface>,
+    pool: std::rc::Rc<std::cell::RefCell<Pool>>,
 
-    pub(crate) rules: RuleSet,
+    rules: RuleSet,
 
-    pub(crate) watch_graph: RuleWatchGraph,
-    pub(crate) decisions: Decisions,
-    pub(crate) fixed_map: IndexMap<i64, BasePackageHandle>,
+    watch_graph: RuleWatchGraph,
+    decisions: Decisions,
+    fixed_map: IndexMap<i64, BasePackageHandle>,
 
-    pub(crate) propagate_index: i64,
+    propagate_index: i64,
     /// Pairs of `(literals, level)` — PHP indexes into these with the BRANCH_* constants.
-    pub(crate) branches: Vec<(Vec<i64>, i64)>,
-    pub(crate) problems: Vec<Problem>,
-    pub(crate) learned_pool: Vec<Vec<std::rc::Rc<std::cell::RefCell<Rule>>>>,
-    pub(crate) learned_why: IndexMap<String, i64>,
+    branches: Vec<(Vec<i64>, i64)>,
+    problems: Vec<Problem>,
+    learned_pool: Vec<Vec<std::rc::Rc<std::cell::RefCell<Rule>>>>,
+    learned_why: IndexMap<String, i64>,
 
     pub test_flag_learned_positive_literal: bool,
 
-    pub(crate) io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
+    io: std::rc::Rc<std::cell::RefCell<dyn IOInterface>>,
 }
 
 impl Solver {
-- 
cgit v1.3.1-4-g156e