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>

1 files changed, 3 insertions, 3 deletions

diff --git a/crates/shirabe/src/dependency_resolver/decisions.rs b/crates/shirabe/src/dependency_resolver/decisions.rs
index ac596054..3cf26ffb 100644
--- a/crates/shirabe/src/dependency_resolver/decisions.rs
+++ b/crates/shirabe/src/dependency_resolver/decisions.rs
@@ -8,8 +8,8 @@ use shirabe_php_shim::LogicException;
 use std::fmt;
 
 pub struct Decisions {
-    pub(crate) pool: std::rc::Rc<std::cell::RefCell<Pool>>,
-    pub(crate) decision_map: IndexMap<i64, i64>,
+    pool: std::rc::Rc<std::cell::RefCell<Pool>>,
+    decision_map: IndexMap<i64, i64>,
     pub(crate) decision_queue: Vec<(i64, std::rc::Rc<std::cell::RefCell<Rule>>)>,
 }
 
@@ -153,7 +153,7 @@ impl Decisions {
         self.decision_queue.is_empty()
     }
 
-    pub(crate) fn add_decision(&mut self, literal: i64, level: i64) {
+    fn add_decision(&mut self, literal: i64, level: i64) {
         let package_id = literal.abs();
 
         let previous_decision = self.decision_map.get(&package_id).copied().unwrap_or(0);