Improvements

- Remove the auto flag, crytic-compile will directly use the auto feature if its on .sol file + the working director is a known platform
- Remove abstract from config to reduce code footprint

Author: montyly

crytic_compile/crytic_compile.py

@@ -127,7 +127,7 @@ def __init__(self, target: Union[str, AbstractPlatform], **kwargs: str) -> None:
             # If the platform is Solc it means we are trying to compile a single
             # we try to see if we are in a known compilation framework to retrieve
             # information like remappings and solc version
-            if kwargs.get("auto_compile", False) and isinstance(platform, Solc):
+            if isinstance(platform, Solc):
                 # Try to get the platform of the current working directory
                 platform_wd = next(
                     (

crytic_compile/cryticparser/cryticparser.py

@@ -65,14 +65,6 @@ def init(parser: ArgumentParser) -> None:
         default=DEFAULTS_FLAG_IN_CONFIG["skip_clean"],
     )
 
-    group_compile.add_argument(
-        "--auto-compile",
-        help="Try to get the solc options automatically when compiling a single file",
-        action="store_true",
-        dest="auto_compile",
-        default=DEFAULTS_FLAG_IN_CONFIG["auto_compile"],
-    )
-
     _init_solc(parser)
     _init_truffle(parser)
     _init_embark(parser)

crytic_compile/platform/abstract_platform.py

@@ -177,7 +177,6 @@ def is_dependency(self, path: str) -> bool:
         return False
 
     @staticmethod
-    @abc.abstractmethod
     def config(working_dir: str) -> Optional[PlatformConfig]:
         """Return configuration data that should be passed to solc, such as version, remappings ecc.
 

crytic_compile/platform/archive.py

@@ -118,18 +118,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
             return False
         return Path(target).parts[-1].endswith("_export_archive.json")
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, _path: str) -> bool:
         """Check if the _path is a dependency. Always false
 

crytic_compile/platform/brownie.py

@@ -114,18 +114,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
             or os.path.isfile(os.path.join(target, "brownie-config.yml"))
         )
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, _path: str) -> bool:
         """Check if the path is a dependency (not supported for brownie)
 

crytic_compile/platform/buidler.py

@@ -193,18 +193,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
         is_typescript = os.path.isfile(os.path.join(target, "buidler.config.ts"))
         return is_javascript or is_typescript
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, path: str) -> bool:
         """Check if the path is a dependency
 

crytic_compile/platform/dapp.py

@@ -157,18 +157,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
                 return "dapp " in txt
         return False
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, path: str) -> bool:
         """Check if the path is a dependency (not supported for brownie)
 

crytic_compile/platform/embark.py

@@ -200,18 +200,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
             return False
         return os.path.isfile(os.path.join(target, "embark.json"))
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, path: str) -> bool:
         """Check if the path is a dependency
 

crytic_compile/platform/etherlime.py

@@ -192,18 +192,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
                 )
         return False
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, path: str) -> bool:
         """Check if the path is a dependency
 

crytic_compile/platform/etherscan.py

@@ -430,18 +430,6 @@ def is_supported(target: str, **kwargs: str) -> bool:
             target = target[target.find(":") + 1 :]
         return bool(re.match(r"^\s*0x[a-zA-Z0-9]{40}\s*$", target))
 
-    @staticmethod
-    def config(working_dir: str) -> Optional[PlatformConfig]:
-        """Return configuration data that should be passed to solc, such as remappings.
-
-        Args:
-            working_dir (str): path to the working directory
-
-        Returns:
-            Optional[PlatformConfig]: Platform configuration data such as optimization, remappings...
-        """
-        return None
-
     def is_dependency(self, _path: str) -> bool:
         """Check if the path is a dependency