feat: Implement YAML support for ARK configuration and synchronize BCASL plugin settings

Author: PyCompiler ARK++

Core/Compiler/ark_config_loader.py

@@ -105,10 +105,9 @@
         "include_modules": [],  # Modules to explicitly include
     },
 
-    # Plugins (BCASL/ACASL)
+    # Plugins (BCASL)
     "plugins": {
         "bcasl_enabled": True,  # Enable pre-compilation plugins
-        "acasl_enabled": True,  # Enable post-compilation plugins
         "plugin_timeout": 0,  # Plugin timeout in seconds (0 = unlimited)
     },
 
@@ -397,11 +396,10 @@ def create_default_ark_config(workspace_dir: str) -> bool:
     # - "hidden_module"
 
 # ───────────────────────────────────────────────────────────────
-# PLUGINS (BCASL/ACASL)
+# PLUGINS (BCASL)
 # ───────────────────────────────────────────────────────────────
 plugins:
   bcasl_enabled: true  # Enable pre-compilation plugins
-  acasl_enabled: true  # Enable post-compilation plugins
   plugin_timeout: 0  # Plugin timeout in seconds (0 = unlimited)
 
 # ───────────────────────────────────────────────────────────────

Tests/test_ark_config_loader.py

@@ -0,0 +1,107 @@
+# SPDX-License-Identifier: Apache-2.0
+# Tests for Core/Compiler/ark_config_loader.py behaviors
+
+from __future__ import annotations
+
+import os
+from pathlib import Path
+import importlib
+
+import pytest
+
+
+@pytest.fixture
+def mod():
+    return importlib.import_module("Core.Compiler.ark_config_loader")
+
+
+def test_load_defaults_when_no_file(tmp_path: Path, mod):
+    cfg = mod.load_ark_config(str(tmp_path))
+    # Basic structure keys
+    for key in ("exclusion_patterns", "inclusion_patterns", "pyinstaller", "nuitka", "output", "dependencies", "plugins", "advanced"):
+        assert key in cfg
+    # Inclusion defaults
+    assert isinstance(cfg["inclusion_patterns"], list)
+    assert any(p.endswith("*.py") for p in cfg["inclusion_patterns"]) 
+
+
+def test_create_default_config_file_idempotent(tmp_path: Path, mod):
+    # First call should create file
+    created_first = mod.create_default_ark_config(str(tmp_path))
+    assert created_first is True
+    conf_path = tmp_path / "ARK_Main_Config.yml"
+    assert conf_path.is_file()
+    # Second call should not overwrite and return False
+    created_second = mod.create_default_ark_config(str(tmp_path))
+    assert created_second is False
+
+
+def test_merge_user_config_and_normalization(tmp_path: Path, mod):
+    # Write a minimal user config overriding nested values and adding custom patterns
+    user_cfg = """
+pyinstaller:
+  onefile: false
+  windowed: true
+exclusion_patterns:
+  - "custom_ignore/**"
+  - "**/*.pyo"
+main_file_names:
+  - main.py
+  - start.py
+    """
+    (tmp_path / "ARK_Main_Config.yml").write_text(user_cfg, encoding="utf-8")
+
+    cfg = mod.load_ark_config(str(tmp_path))
+
+    # Nested overrides applied
+    assert cfg["pyinstaller"]["onefile"] is False
+    assert cfg["pyinstaller"]["windowed"] is True
+
+    # Exclusion patterns contain defaults plus user entries (deduped)
+    patterns = set(cfg["exclusion_patterns"])
+    assert "custom_ignore/**" in patterns
+    # Default like "**/*.pyc" should still be present
+    assert any(p.endswith("*.pyc") for p in patterns)
+
+    # Normalized main_file_names list
+    assert set(cfg["main_file_names"]) >= {"main.py", "start.py"}
+
+
+@pytest.mark.parametrize(
+    "compiler_key",
+    ["pyinstaller", "nuitka", "cx_freeze"],
+)
+def test_get_compiler_options_present(tmp_path: Path, mod, compiler_key: str):
+    cfg = mod.load_ark_config(str(tmp_path))
+    opts = mod.get_compiler_options(cfg, compiler_key)
+    assert isinstance(opts, dict)
+
+
+def test_option_accessors(tmp_path: Path, mod):
+    cfg = mod.load_ark_config(str(tmp_path))
+    assert isinstance(mod.get_output_options(cfg), dict)
+    assert isinstance(mod.get_dependency_options(cfg), dict)
+    assert isinstance(mod.get_plugin_options(cfg), dict)
+    assert isinstance(mod.get_advanced_options(cfg), dict)
+
+
+def test_should_exclude_file_logic(tmp_path: Path, mod):
+    # Create workspace and files
+    ws = tmp_path
+    (ws / "venv" / "lib").mkdir(parents=True)
+    inside_ok = ws / "src" / "app.py"
+    inside_ok.parent.mkdir(parents=True)
+    inside_ok.write_text("print('x')", encoding="utf-8")
+    excluded_by_pattern = ws / "venv" / "lib" / "x.py"
+    excluded_by_pattern.write_text("print('y')", encoding="utf-8")
+
+    patterns = mod.DEFAULT_EXCLUSION_PATTERNS
+
+    # File outside workspace must be excluded
+    assert mod.should_exclude_file("/tmp/somewhere_else.py", str(ws), patterns) is True
+
+    # File matching exclusion pattern is excluded
+    assert mod.should_exclude_file(str(excluded_by_pattern), str(ws), patterns) is True
+
+    # Regular file inside workspace not matching patterns is not excluded
+    assert mod.should_exclude_file(str(inside_ok), str(ws), patterns) is False

Tests/test_bcasl_loader_sync.py

@@ -0,0 +1,85 @@
+# SPDX-License-Identifier: Apache-2.0
+# Tests for BCASL <-> ARK config synchronization and YAML behavior
+
+from __future__ import annotations
+
+from pathlib import Path
+import importlib
+import yaml
+
+
+def write_yaml(path: Path, data: dict):
+    path.write_text(yaml.safe_dump(data, sort_keys=False, allow_unicode=True), encoding="utf-8")
+
+
+def read_yaml(path: Path) -> dict:
+    return yaml.safe_load(path.read_text(encoding="utf-8")) or {}
+
+
+def test_sync_ark_bcasl_enabled_creates_and_updates_yaml(tmp_path: Path):
+    mod = importlib.import_module("bcasl.Loader")
+
+    ws = tmp_path
+    ark_path = ws / "ARK_Main_Config.yml"
+    assert not ark_path.exists()
+
+    # Toggle to False -> should create the YAML and set plugins.bcasl_enabled False
+    mod._sync_ark_bcasl_enabled(ws, False)
+    assert ark_path.is_file()
+    cfg = read_yaml(ark_path)
+    assert cfg.get("plugins", {}).get("bcasl_enabled") is False
+
+    # Toggle to True -> update YAML accordingly
+    mod._sync_ark_bcasl_enabled(ws, True)
+    cfg2 = read_yaml(ark_path)
+    assert cfg2.get("plugins", {}).get("bcasl_enabled") is True
+
+
+def test_bcasl_loader_ignores_enabled_in_bcasl_yaml_and_uses_ark(tmp_path: Path):
+    mod = importlib.import_module("bcasl.Loader")
+
+    ws = tmp_path
+    ark_path = ws / "ARK_Main_Config.yml"
+    bcasl_path = ws / "bcasl.yml"
+
+    # Prepare ARK with bcasl disabled
+    write_yaml(ark_path, {"plugins": {"bcasl_enabled": False, "plugin_timeout": 0}})
+
+    # Prepare a legacy bcasl.yml that wrongly contains an 'enabled' field
+    write_yaml(bcasl_path, {
+        "options": {"enabled": True, "plugin_timeout_s": 0},
+        "plugins": {},
+        "plugin_order": []
+    })
+
+    cfg = mod._load_workspace_config(ws)
+    # In-memory config must reflect ARK: enabled False regardless of YAML content
+    opts = cfg.get("options", {})
+    assert isinstance(opts, dict)
+    assert opts.get("enabled") is False
+
+    # Persisted YAML must not be modified on read; still contains 'enabled' (legacy)
+    y = read_yaml(bcasl_path)
+    assert "enabled" in (y.get("options") or {})
+
+
+def test_bcasl_default_generation_omits_enabled_but_runtime_injects_from_ark(tmp_path: Path):
+    mod = importlib.import_module("bcasl.Loader")
+
+    ws = tmp_path
+    ark_path = ws / "ARK_Main_Config.yml"
+
+    # ARK enables BCASL
+    write_yaml(ark_path, {"plugins": {"bcasl_enabled": True, "plugin_timeout": 1}})
+
+    # No bcasl.yml present triggers default generation path
+    cfg = mod._load_workspace_config(ws)
+
+    # A bcasl.yml file should have been created without 'enabled' in options
+    bpath = ws / "bcasl.yml"
+    assert bpath.is_file()
+    persisted = read_yaml(bpath)
+    assert "enabled" not in (persisted.get("options") or {})
+
+    # But the in-memory config should include enabled: True
+    assert cfg.get("options", {}).get("enabled") is True