fix: use build_info_path from forge config instead of hardcoding (#638)

* fix: use build_info_path from forge config instead of hardcoding

Foundry projects can configure `build_info_path` separately from `out`.
For example, Optimism's contracts-bedrock uses:
- out: forge-artifacts
- build_info_path: artifacts/build-info

Previously crytic-compile assumed build-info was always at `{out}/build-info`,
which caused compilation failures for projects with custom build_info_path.

Changes:
- Add build_info_path field to PlatformConfig
- Read build_info_path from forge config --json
- Use build_info_path when set, fall back to {out}/build-info otherwise
- Fetch forge config before compile check (needed for ignore_compile mode)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: avoid calling forge config when using --ignore-compile

Address review feedback: calling config() early breaks --ignore-compile
because forge may not be available in CI environments without build tools.

Changes:
- Move config() call back inside the compile branch
- Add --foundry-build-info-directory CLI option for users who need custom
  build-info paths with --ignore-compile
- Priority order: CLI override > forge config > default ({out}/build-info)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix: use modern type union syntax for build_info_path

Replace `Optional[str]` with `str | None` to match the style of
other fields in PlatformConfig and fix the missing import error.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: Emilio López <2642849+elopez@users.noreply.github.com>

Author: 3 people

crytic_compile/cryticparser/cryticparser.py

@@ -450,6 +450,14 @@ def _init_foundry(parser: ArgumentParser) -> None:
         default=DEFAULTS_FLAG_IN_CONFIG["foundry_out_directory"],
     )
 
+    group_foundry.add_argument(
+        "--foundry-build-info-directory",
+        help="Use an alternative build-info directory (useful with --ignore-compile)",
+        action="store",
+        dest="foundry_build_info_directory",
+        default=DEFAULTS_FLAG_IN_CONFIG["foundry_build_info_directory"],
+    )
+
     group_foundry.add_argument(
         "--foundry-compile-all",
         help="Don't skip compiling test and script",

crytic_compile/cryticparser/defaults.py

@@ -44,6 +44,7 @@
     "hardhat_artifacts_directory": None,
     "foundry_ignore_compile": False,
     "foundry_out_directory": None,
+    "foundry_build_info_directory": None,
     "foundry_compile_all": False,
     "export_dir": "crytic-export",
     "compile_libraries": None,

crytic_compile/platform/abstract_platform.py

@@ -42,6 +42,7 @@ class PlatformConfig:
     libs_path: list[str] = field(default_factory=lambda: ["lib"])
     scripts_path: str = "script"
     out_path: str = "out"
+    build_info_path: str | None = None
 
 
 class AbstractPlatform(metaclass=abc.ABCMeta):

crytic_compile/platform/foundry.py

@@ -44,7 +44,8 @@ def compile(self, crytic_compile: "CryticCompile", **kwargs: str) -> None:
 
         Args:
             crytic_compile (CryticCompile): CryticCompile object to populate
-            **kwargs: optional arguments. Used: "foundry_ignore_compile", "foundry_out_directory"
+            **kwargs: optional arguments. Used: "foundry_ignore_compile", "foundry_out_directory",
+                "foundry_build_info_directory"
 
         """
 
@@ -92,11 +93,14 @@ def compile(self, crytic_compile: "CryticCompile", **kwargs: str) -> None:
         out_directory_config = kwargs.get("foundry_out_directory", None)
         out_directory = out_directory_config if out_directory_config else out_directory_detected
 
-        build_directory = Path(
-            self._project_root,
-            out_directory,
-            "build-info",
-        )
+        # Determine build-info directory: CLI override > forge config > default
+        build_info_override = kwargs.get("foundry_build_info_directory", None)
+        if build_info_override:
+            build_directory = Path(self._project_root, build_info_override)
+        elif foundry_config and foundry_config.build_info_path:
+            build_directory = Path(self._project_root, foundry_config.build_info_path)
+        else:
+            build_directory = Path(self._project_root, out_directory, "build-info")
 
         hardhat_like_parsing(
             crytic_compile, str(self._target), build_directory, str(self._project_root)
@@ -199,6 +203,7 @@ def config(working_dir: str | Path) -> PlatformConfig | None:
         result.libs_path = json_config.get("libs")
         result.scripts_path = json_config.get("script")
         result.out_path = json_config.get("out")
+        result.build_info_path = json_config.get("build_info_path")
 
         return result