Reordered to call registrar task with correct arg, renamed for clarity

tasks/deploy/integrations/helpers/fullStrategy.ts

@@ -56,28 +56,20 @@ export async function deployStrategySet(
     hre
   );
 
-  // establish registrar config on primary chain and this chain
-  await hre.run("manage:registrar:setStratParams", {
-    stratConfig: {
-      ...config,
-      params: {
-        approvalState: config.params.approvalState,
-        network: config.params.network,
-        lockedVaultAddr: Locked.contract.address,
-        liquidVaultAddr: Liquid.contract.address,
-      },
-    },
-    modifyExisting: true,
-    apTeamSignerPkey: signerPkey,
-  });
-
-  // Store addresses
+  // Store addresses - do this before updating registrar so that lookup is complete
   writeStrategyAddresses(strategyName, {
     locked: Locked.contract.address,
     liquid: Liquid.contract.address,
     strategy: Strategy.contract.address,
   });
 
+  // establish registrar config on primary chain and this chain
+  await hre.run("manage:registrar:setStratParams", {
+    stratName: strategyName,
+    modifyExisting: true,
+    apTeamSignerPkey: signerPkey,
+  });
+
   // Verify
   const deployments: Deployment<ContractFactory>[] = [Locked, Liquid, Strategy];
   for (const deployment of deployments) {

tasks/manage/registrar/setStratParams.ts

@@ -53,7 +53,7 @@ subtask(
   "Updates strat params on the network specified by the 'chainId' param"
 )
   .addParam(
-    "stratConfig",
+    "stratName",
     `The name of the strategy according to StratConfig, possible values: ${Object.keys(
       allStrategyConfigs
     ).join(", ")}`,