feat(tests): configuration management system

tom-sapletta-com · tom-sapletta-com · commit d5a95ba022b3 · 2026-02-26T19:01:45.000+01:00
changes:
  - file: parsers.py
    area: analyzer
    added: [_walk, _extract_js_require_imports, _extract_from_expression_stmt, _extract_js_var_fn, _add_func, _walk_nested_functions]
    modified: [_parse_js_ts, _extract_js_arrow_fn, UniversalParser, TreeSitterParser]
  - file: sample_javascript_advanced.js
    area: test
    added: [result, innerSort, middleHelper, files, items, fetchResource, +22 more]
  - file: test_parser_integrity.py
    area: analyzer
    added: [parse_js, TestJavaScriptFunctionExtraction]
    new_tests: 15

testing:
  new_tests: 15
  scenarios:
    - var_function_expression
    - class_with_methods
    - no_duplicate_functions
    - module_exports_shorthand
    - const_arrow_function
    - commonjs_require_imports
    - regular_function_declaration
    - const_function_expression
    - export_clause
    - iife_named_function
    # +5 more

stats:
  lines: "+608/-57 (net +551)"
  files: 3
  complexity: "Large structural change (normalized)"
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,19 @@
+## [1.0.48] - 2026-02-26
+
+### Summary
+
+feat(tests): configuration management system
+
+### Test
+
+- update tests/samples/sample_javascript_advanced.js
+- update tests/test_parser_integrity.py
+
+### Other
+
+- update code2logic/parsers.py
+
+
 ## [1.0.47] - 2026-02-26
 
 ### Summary
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-1.0.47
+1.0.48
diff --git a/code2logic/__init__.py b/code2logic/__init__.py
@@ -18,7 +18,7 @@
     >>> print(output)
 """
 
-__version__ = "1.0.47"
+__version__ = "1.0.48"
 __author__ = "Softreck"
 __email__ = "info@softreck.dev"
 __license__ = "MIT"
diff --git a/code2logic/parsers.py b/code2logic/parsers.py
diff --git a/logic2code/__init__.py b/logic2code/__init__.py
@@ -14,5 +14,5 @@
 from .generator import CodeGenerator, GeneratorConfig, GenerationResult
 from .renderers import PythonRenderer
 
-__version__ = '1.0.47'
+__version__ = '1.0.48'
 __all__ = ['CodeGenerator', 'GeneratorConfig', 'GenerationResult', 'PythonRenderer']
diff --git a/logic2test/__init__.py b/logic2test/__init__.py
@@ -15,5 +15,5 @@
 from .parsers import LogicParser
 from .templates import TestTemplate
 
-__version__ = '1.0.47'
+__version__ = '1.0.48'
 __all__ = ['TestGenerator', 'GeneratorConfig', 'GenerationResult', 'LogicParser', 'TestTemplate']
diff --git a/lolm/__init__.py b/lolm/__init__.py
@@ -76,7 +76,7 @@
 )
 from .clients import LLMRateLimitError
 
-__version__ = '1.0.47'
+__version__ = '1.0.48'
 __all__ = [
     # Config
     'LLMConfig',
diff --git a/pyproject.toml b/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "code2logic"
-version = "1.0.47"
+version = "1.0.48"
 description = "Code2Logic - Source code to logical representation converter for LLM analysis, featuring Tree-sitter parsing, dependency graph analysis, and multi-language support."
 readme = "README.md"
 license = "Apache-2.0"
diff --git a/tests/samples/sample_javascript_advanced.js b/tests/samples/sample_javascript_advanced.js
@@ -0,0 +1,143 @@
+/**
+ * Advanced JavaScript sample for testing TOON parser coverage.
+ * Covers: IIFEs, nested functions, var declarations, function expressions,
+ * module.exports, CommonJS require, arrow functions with params.
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { EventEmitter } = require('events');
+
+// Arrow function assigned to const
+const getArg = (name, def) => {
+  const idx = process.argv.indexOf(`--${name}`);
+  if (idx !== -1) return process.argv[idx + 1];
+  return def;
+};
+
+// Arrow function assigned to let
+let processItem = (item) => {
+  return item.trim().toLowerCase();
+};
+
+// Function expression assigned to const
+const validate = function(input) {
+  if (!input || typeof input !== 'string') return false;
+  return input.length > 0;
+};
+
+// Async function expression assigned to const
+const fetchResource = async function(url, options) {
+  const response = await fetch(url, options);
+  return response.json();
+};
+
+// var arrow function
+var shouldIgnore = (filePath) => {
+  return filePath.includes('node_modules');
+};
+
+// var function expression
+var formatOutput = function(data, indent) {
+  return JSON.stringify(data, null, indent || 2);
+};
+
+// Regular top-level function
+function walk(dir, onFile) {
+  const entries = fs.readdirSync(dir, { withFileTypes: true });
+  for (const e of entries) {
+    const full = path.join(dir, e.name);
+    if (e.isDirectory()) walk(full, onFile);
+    else onFile(full);
+  }
+}
+
+// Function with nested function inside
+function findFiles(dir, extensions) {
+  const files = [];
+
+  function traverse(currentDir) {
+    const items = fs.readdirSync(currentDir);
+    for (const item of items) {
+      const fullPath = path.join(currentDir, item);
+      if (fs.statSync(fullPath).isDirectory()) {
+        traverse(fullPath);
+      } else if (extensions.includes(path.extname(item))) {
+        files.push(fullPath);
+      }
+    }
+  }
+
+  traverse(dir);
+  return files;
+}
+
+// Async top-level function
+async function processFiles(pattern) {
+  const files = findFiles('.', ['.js', '.ts']);
+  for (const file of files) {
+    const content = fs.readFileSync(file, 'utf8');
+    console.log(`Processing: ${file}`);
+  }
+}
+
+// Class with methods
+class FileProcessor extends EventEmitter {
+  constructor(rootDir) {
+    super();
+    this.rootDir = rootDir;
+    this.results = [];
+  }
+
+  async analyze(filePath) {
+    const content = fs.readFileSync(filePath, 'utf8');
+    const result = { path: filePath, lines: content.split('\n').length };
+    this.results.push(result);
+    this.emit('analyzed', result);
+    return result;
+  }
+
+  static fromConfig(configPath) {
+    const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
+    return new FileProcessor(config.rootDir);
+  }
+
+  getResults() {
+    return [...this.results];
+  }
+}
+
+// Deeply nested function
+function outerFunction(data) {
+  function middleHelper(items) {
+    function innerSort(a, b) {
+      return a.localeCompare(b);
+    }
+    return items.sort(innerSort);
+  }
+  return middleHelper(data);
+}
+
+// IIFE with named function
+(function main() {
+  const root = getArg('root', '.');
+  console.log(`Scanning ${root}...`);
+  walk(root, (file) => {
+    console.log(file);
+  });
+})();
+
+// module.exports with shorthand properties
+module.exports = {
+  getArg,
+  processItem,
+  validate,
+  fetchResource,
+  shouldIgnore,
+  formatOutput,
+  walk,
+  findFiles,
+  processFiles,
+  FileProcessor,
+  outerFunction
+};
diff --git a/tests/samples/sample_reexport/__init__.py b/tests/samples/sample_reexport/__init__.py
@@ -18,4 +18,4 @@
     "ProcessingError",
 ]
 
-__version__ = "1.0.47"
+__version__ = "1.0.48"
diff --git a/tests/test_parser_integrity.py b/tests/test_parser_integrity.py
@@ -285,6 +285,152 @@ def function_{i:04d}(param_{i}: int) -> str:
         assert result.functions[49].name == "function_0049"
 
 
+def parse_js(parser, code: str):
+    """Helper to parse JavaScript code."""
+    return parser.parse('test.js', code, 'javascript')
+
+
+class TestJavaScriptFunctionExtraction:
+    """Test JS function extraction: arrow fns, function expressions, IIFEs, nested, etc."""
+
+    def test_regular_function_declaration(self, parser):
+        code = 'function walk(dir, onFile) {\n  console.log(dir);\n}\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'walk' in names
+
+    def test_async_function_declaration(self, parser):
+        code = 'async function fetchData(url) {\n  return await fetch(url);\n}\n'
+        result = parse_js(parser, code)
+        fn = next(f for f in result.functions if f.name == 'fetchData')
+        assert fn.is_async
+
+    def test_const_arrow_function(self, parser):
+        code = 'const getArg = (name, def) => {\n  return name;\n};\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'getArg' in names
+        fn = next(f for f in result.functions if f.name == 'getArg')
+        assert len(fn.params) == 2
+
+    def test_let_arrow_function(self, parser):
+        code = 'let processItem = (item) => {\n  return item;\n};\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'processItem' in names
+
+    def test_var_arrow_function(self, parser):
+        code = 'var shouldIgnore = (filePath) => {\n  return false;\n};\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'shouldIgnore' in names
+
+    def test_const_function_expression(self, parser):
+        code = 'const validate = function(input) {\n  return !!input;\n};\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'validate' in names
+
+    def test_var_function_expression(self, parser):
+        code = 'var formatOutput = function(data, indent) {\n  return JSON.stringify(data);\n};\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'formatOutput' in names
+
+    def test_iife_named_function(self, parser):
+        code = '(function main() {\n  console.log("hello");\n})();\n'
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'main' in names
+
+    def test_nested_function_in_body(self, parser):
+        code = '''function findFiles(dir) {
+  const files = [];
+  function traverse(currentDir) {
+    console.log(currentDir);
+  }
+  traverse(dir);
+  return files;
+}
+'''
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'findFiles' in names
+        assert 'traverse' in names
+
+    def test_deeply_nested_functions(self, parser):
+        code = '''function outer(data) {
+  function middle(items) {
+    function inner(a, b) {
+      return a - b;
+    }
+    return items.sort(inner);
+  }
+  return middle(data);
+}
+'''
+        result = parse_js(parser, code)
+        names = [f.name for f in result.functions]
+        assert 'outer' in names
+        assert 'middle' in names
+        assert 'inner' in names
+
+    def test_module_exports_shorthand(self, parser):
+        code = '''function foo() {}
+function bar() {}
+module.exports = { foo, bar };
+'''
+        result = parse_js(parser, code)
+        assert 'foo' in result.exports
+        assert 'bar' in result.exports
+
+    def test_commonjs_require_imports(self, parser):
+        code = "const fs = require('fs');\nconst path = require('path');\n"
+        result = parse_js(parser, code)
+        assert 'fs' in result.imports
+        assert 'path' in result.imports
+
+    def test_class_with_methods(self, parser):
+        code = '''class FileProcessor {
+  constructor(rootDir) {
+    this.rootDir = rootDir;
+  }
+  async analyze(filePath) {
+    return {};
+  }
+  static fromConfig(configPath) {
+    return new FileProcessor('.');
+  }
+}
+'''
+        result = parse_js(parser, code)
+        assert len(result.classes) == 1
+        cls = result.classes[0]
+        assert cls.name == 'FileProcessor'
+        method_names = [m.name for m in cls.methods]
+        assert 'constructor' in method_names
+        assert 'analyze' in method_names
+        assert 'fromConfig' in method_names
+
+    def test_export_clause(self, parser):
+        code = '''function foo() {}
+const bar = () => {};
+export { foo, bar };
+'''
+        result = parse_js(parser, code)
+        assert 'foo' in result.exports
+        assert 'bar' in result.exports
+
+    def test_no_duplicate_functions(self, parser):
+        """Ensure same function isn't listed twice."""
+        code = '''function walk(dir) {}
+module.exports = { walk };
+'''
+        result = parse_js(parser, code)
+        walk_fns = [f for f in result.functions if f.name == 'walk']
+        assert len(walk_fns) == 1
+
+
 class TestMethodSignatureIntegrity:
     """Additional tests for method signature integrity."""
     

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@`
`76`	`76`	`)`
`77`	`77`	`from .clients import LLMRateLimitError`
`78`	`78`
`79`		`-__version__ = '1.0.47'`
	`79`	`+__version__ = '1.0.48'`
`80`	`80`	`__all__ = [`
`81`	`81`	`# Config`
`82`	`82`	`'LLMConfig',`
Original file line number	Diff line number	Diff line change
`@@ -18,4 +18,4 @@`
`18`	`18`	`"ProcessingError",`
`19`	`19`	`]`
`20`	`20`
`21`		`-__version__ = "1.0.47"`
	`21`	`+__version__ = "1.0.48"`