fix: remove anvil from inspect command and add support for target string

.github/workflows/commitlint-pr.yml

@@ -1,11 +1,6 @@
 name: 'Semantic Commit'
 
 on:
-  pull_request:
-    types:
-      - opened
-      - edited
-      - synchronize
   pull_request_target:
     types:
       - opened

packages/builder/src/schemas.zod.ts

@@ -24,6 +24,21 @@ const jsonAbiPathRegex = RegExp(/^(?!.*\.d?$).*\.json?$/, 'i');
 const artifactNameRegex = RegExp(/^[A-Z]{1}[\w]+$/, 'i');
 const artifactPathRegex = RegExp(/^.*\.sol:\w+/, 'i');
 
+// Invoke target string schema
+const targetString = z.string().refine(
+  (val) =>
+    ethers.utils.isAddress(val) ||
+    !!val.match(interpolatedRegex) ||
+    !!val.match(stepRegex) ||
+    !!val.match(artifactNameRegex) ||
+    !!val.match(artifactPathRegex),
+  (val) => ({
+    message: `"${val}" must be a valid ethereum address, existing contract step name, contract artifact name or filepath`,
+  })
+);
+
+const targetSchema = targetString.or(z.array(targetString).nonempty());
+
 export const contractSchema = z
   .object({
     /**
@@ -171,21 +186,7 @@ export const invokeSchema = z
     /**
      *  Names of the contract to call or contract action that deployed the contract to call
      */
-    target: z
-      .array(
-        z.string().refine(
-          (val) =>
-            ethers.utils.isAddress(val) ||
-            !!val.match(interpolatedRegex) ||
-            !!val.match(stepRegex) ||
-            !!val.match(artifactNameRegex) ||
-            !!val.match(artifactPathRegex),
-          (val) => ({
-            message: `"${val}" must be a valid ethereum address, previously defined contract step name or contract artifact name or path`,
-          })
-        )
-      )
-      .nonempty(),
+    target: targetSchema,
     /**
      *  Name of the function to call on the contract
      */

packages/builder/src/steps/invoke.ts

@@ -190,6 +190,10 @@ export default {
   async getState(_runtime: ChainBuilderRuntimeInfo, ctx: ChainBuilderContextWithHelpers, config: Config) {
     const cfg = this.configInject(ctx, config);
 
+    if (typeof cfg.target === 'string') {
+      cfg.target = [cfg.target as string];
+    }
+
     return {
       to: cfg.target?.map((t) => getContractFromPath(ctx, t)?.address),
       func: cfg.func,
@@ -201,6 +205,10 @@ export default {
   configInject(ctx: ChainBuilderContextWithHelpers, config: Config) {
     config = _.cloneDeep(config);
 
+    if (typeof config.target === 'string') {
+      config.target = [config.target as string];
+    }
+
     if (config.target) {
       // [string, ...string[]] refers to a nonempty array
       config.target = config.target.map((v) => _.template(v)(ctx)) as [string, ...string[]];

packages/cli/src/commands/inspect.ts

@@ -5,11 +5,9 @@ import { createDefaultReadRegistry } from '../registry';
 import { resolveCliSettings } from '../settings';
 import fs from 'fs-extra';
 import path from 'path';
-import { setupAnvil } from '../helpers';
 import { getMainLoader } from '../loader';
 
 export async function inspect(packageRef: string, chainId: number, preset: string, json: boolean, writeDeployments: string) {
-  await setupAnvil();
   const { name, version } = parsePackageRef(packageRef);
 
   const resolver = await createDefaultReadRegistry(resolveCliSettings());
@@ -24,6 +22,15 @@ export async function inspect(packageRef: string, chainId: number, preset: strin
     );
   }
 
+  if (!chainId) {
+    console.warn(
+      yellow(
+        "The deployment data for the latest local version of this package (which runs with 'cannon PACKAGE_NAME') was exported. \
+      Specify the --chain-id parameter to retrieve the addresses/ABIs for other deployments."
+      )
+    );
+  }
+
   const deployData = await loader[deployUrl.split(':')[0] as 'ipfs' | 'file'].read(deployUrl);
 
   if (!deployData) {