fix(solidity/linter/core): now run foundry naming rule on public and …

…external functions only

toolchains/solidity/linter/core/solidhunter-lib/src/rules/naming/foundry_func_name.rs

@@ -51,6 +51,15 @@ impl RuleType for FoundryFuncName {
 
         for contract in contracts {
             for function in ast_extractor::retriever::retrieve_functions_nodes(&contract) {
+                let visibility = function.attributes.iter().find(|attr| matches!(attr, ast_extractor::FunctionAttribute::Visibility(_)));
+                let visibility = match visibility {
+                    Some(ast_extractor::FunctionAttribute::Visibility(visibility)) => visibility,
+                    _ => continue,
+                };
+
+                if !matches!(visibility, ast_extractor::Visibility::Public(_)) && !matches!(visibility, ast_extractor::Visibility::External(_)) {
+                    continue;
+                }
                 if let Some(name) = function.name {
                     if !re.is_match(&name.as_string()) && !self.excluded.contains(&name.as_string()) {
                         let span = name.span();

toolchains/solidity/linter/core/solidhunter-lib/testdata/FoundryFunctionName/file.t.sol

@@ -8,4 +8,6 @@ contract Test {
     function testFuzz_FuzzyTest() public {} // should pass
 
     function numberIs42() public {} // should fail
+
+    function YoloNamingBecauseSkipped() internal {} // should pass
 }