fix: improve setup command and ignore metadata on repairs

- Removed the optional private key from the setup command options, using global options instead.
- Enhanced the private key validation message to specify the required format (64 hex characters starting with 0x).
- Updated logic in add-pieces to improve readability by introducing a variable for repair state.

Author: hugomrdias

packages/repair-cli/src/commands/setup.ts

@@ -22,17 +22,15 @@ function validatePostgresUrl(value: string) {
 
 export const setup = Cli.create('setup', {
   description: 'Setup the CLI',
-  options: globalOptions.extend({
-    privateKey: z.string().optional().describe('Private key to use'),
-  }),
+  options: globalOptions,
   run: async (c) => {
     try {
       // Private key
       const pk = await p.text({
         message: 'Enter your private key',
         validate(value) {
           if (!value || !/^0x[a-fA-F0-9]{64}$/.test(value)) {
-            return `Invalid private key!`
+            return `Invalid private key! (must be 64 hex characters) starting with 0x`
           }
         },
         initialValue: config.get('privateKey'),

packages/repair-cli/src/pipeline/add-pieces.ts

@@ -47,14 +47,15 @@ function createAddPiecesWorker({ localDb, indexerDb, repair, client, state, log
     let completedOps = 0
     let failedOps = 0
     let operations: OperationSelect[] = options.operations
+    const isRepair = repair.repairDataSetId == null
 
     const group = log.group(`Batch ${options.batchNumber}/${state.totalBatches}`)
 
     try {
       const dataset = await getTargetDataset({ localDb, repairId: repair.id, client })
 
       // dedupe operations by CID on the target dataset for repairs jobs
-      if (repair.repairDataSetId == null) {
+      if (isRepair) {
         operations = await dedupeCids({ indexerDb, localDb, dataSetId: dataset.dataSetId, operations })
       }
       group.message(`Pulling ${operations.length} pieces...`)
@@ -101,7 +102,7 @@ function createAddPiecesWorker({ localDb, indexerDb, repair, client, state, log
           clientDataSetId: dataset.clientDataSetId,
           pieces: operations.map((operation) => ({
             pieceCid: Piece.from(operation.cid),
-            metadata: operation.metadata,
+            metadata: isRepair ? undefined : operation.metadata,
           })),
         })