change visibility on structs/methods

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased changes
+
+* mark some internal functionality public for the darklua library (`Configuration::location()`) ([#328](https://github.com/seaofvoices/darklua/pull/328))
+
 ## 0.17.3
 
 * fix `inject_global_value` rule to not override environment values with the default value when they're defined ([#324](https://github.com/seaofvoices/darklua/pull/324))

src/frontend/configuration.rs

@@ -122,7 +122,7 @@ impl Configuration {
     }
 
     #[inline]
-    pub(crate) fn location(&self) -> Option<&Path> {
+    pub fn location(&self) -> Option<&Path> {
         self.location.as_deref()
     }
 }

src/lib.rs

@@ -67,3 +67,4 @@ pub use frontend::{
     Options, Resources, WorkerTree,
 };
 pub use parser::{Parser, ParserError};
+pub use utils::ScopedHashMap;

src/rules/mod.rs

@@ -72,7 +72,7 @@ pub use remove_types::*;
 pub use remove_unused_variable::*;
 pub use rename_variables::*;
 pub(crate) use replace_referenced_tokens::*;
-pub use require::PathRequireMode;
+pub use require::{PathRequireMode, RequirePathLocator, PathLocator};
 pub use rule_property::*;
 pub(crate) use shift_token_line::*;
 pub use unused_if_branch::*;

src/rules/require/mod.rs

@@ -11,12 +11,12 @@ use std::path::{Path, PathBuf};
 pub(crate) use luau_path_locator::LuauPathLocator;
 pub use luau_require_mode::LuauRequireMode;
 pub(crate) use match_require::{is_require_call, match_path_require_call};
-pub(crate) use path_locator::RequirePathLocator;
+pub use path_locator::RequirePathLocator;
 pub use path_require_mode::PathRequireMode;
 
 use crate::DarkluaError;
 
-pub(crate) trait PathLocator {
+pub trait PathLocator {
     fn find_require_path(
         &self,
         path: impl Into<PathBuf>,

src/rules/require/path_locator.rs

@@ -4,14 +4,14 @@ use super::{path_iterator, PathRequireMode};
 use crate::{rules::require::path_utils::is_require_relative, utils, DarkluaError, Resources};
 
 #[derive(Debug)]
-pub(crate) struct RequirePathLocator<'a, 'b, 'resources> {
+pub struct RequirePathLocator<'a, 'b, 'resources> {
     path_require_mode: &'a PathRequireMode,
     extra_module_relative_location: &'b Path,
     resources: &'resources Resources,
 }
 
 impl<'a, 'b, 'c> RequirePathLocator<'a, 'b, 'c> {
-    pub(crate) fn new(
+    pub fn new(
         path_require_mode: &'a PathRequireMode,
         extra_module_relative_location: &'b Path,
         resources: &'c Resources,

src/rules/require/path_require_mode.rs

@@ -68,7 +68,7 @@ impl PathRequireMode {
         }
     }
 
-    pub(crate) fn initialize(&mut self, context: &Context) -> Result<(), DarkluaError> {
+    pub fn initialize(&mut self, context: &Context) -> Result<(), DarkluaError> {
         if !self.use_luau_configuration {
             self.luau_rc_aliases.take();
             return Ok(());

src/utils/mod.rs

@@ -9,7 +9,7 @@ mod timer;
 pub(crate) use expressions_as_statement::{expressions_as_expression, expressions_as_statement};
 pub(crate) use luau_config::{clear_luau_configuration_cache, find_luau_configuration};
 pub(crate) use preserve_arguments_side_effects::preserve_arguments_side_effects;
-pub(crate) use scoped_hash_map::ScopedHashMap;
+pub use scoped_hash_map::ScopedHashMap;
 pub(crate) use serde_string_or_struct::string_or_struct;
 use std::{
     ffi::OsStr,

src/utils/scoped_hash_map.rs

@@ -2,16 +2,16 @@ use std::collections::HashMap;
 use std::hash::Hash;
 
 #[derive(Debug, Default, Clone)]
-pub(crate) struct ScopedHashMap<Key, Value> {
+pub struct ScopedHashMap<Key, Value> {
     stack: Vec<HashMap<Key, Value>>,
 }
 
 impl<Key: Eq + Hash, Value> ScopedHashMap<Key, Value> {
-    pub(crate) fn get(&self, key: &Key) -> Option<&Value> {
+    pub fn get(&self, key: &Key) -> Option<&Value> {
         self.stack.iter().rev().find_map(|map| map.get(key))
     }
 
-    pub(crate) fn insert(&mut self, key: Key, value: Value) {
+    pub fn insert(&mut self, key: Key, value: Value) {
         if let Some(hash_map) = self.stack.last_mut() {
             hash_map.insert(key, value);
         } else {
@@ -21,11 +21,11 @@ impl<Key: Eq + Hash, Value> ScopedHashMap<Key, Value> {
         }
     }
 
-    pub(crate) fn push(&mut self) {
+    pub fn push(&mut self) {
         self.stack.push(Default::default())
     }
 
-    pub(crate) fn pop(&mut self) {
+    pub fn pop(&mut self) {
         self.stack.pop();
     }
 }