fix: address Copilot review feedback on OpenAPI transformer skill

  - Use curl -fsSL with retry for fail-fast HTTP error handling
  - Pin swagger2openapi@7.0.8 with --yes for reproducible transforms
  - Add monitor and zoneconcierge module mappings to tag config
  - Add explicit fallback strategy for unmapped Babylon modules

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

Author: web3jenks

.claude/skills/openapi-spec-transformer/SKILL.md

@@ -44,11 +44,11 @@ Download Swagger 2.0 specs from source repositories into `/tmp/`:
 
 ```bash
 # Babylon gRPC spec
-curl -sL -o /tmp/babylon-swagger2.yaml \
+curl -fsSL --retry 3 --retry-delay 5 -o /tmp/babylon-swagger2.yaml \
   https://raw.githubusercontent.com/babylonlabs-io/babylon/main/client/docs/swagger-ui/swagger.yaml
 
 # Staking API spec
-curl -sL -o /tmp/staking-swagger2.yaml \
+curl -fsSL --retry 3 --retry-delay 5 -o /tmp/staking-swagger2.yaml \
   https://raw.githubusercontent.com/babylonlabs-io/staking-api-service/main/docs/swagger.yaml
 ```
 
@@ -83,8 +83,8 @@ grep -c "operationId:" static/swagger/babylon-staking-api-openapi3.yaml
 Use `swagger2openapi` for mechanical Swagger 2.0 → OpenAPI 3.0 conversion:
 
 ```bash
-npx swagger2openapi /tmp/babylon-swagger2.yaml -o /tmp/babylon-openapi3-base.yaml
-npx swagger2openapi /tmp/staking-swagger2.yaml -o /tmp/staking-openapi3-base.yaml
+npx --yes swagger2openapi@7.0.8 /tmp/babylon-swagger2.yaml -o /tmp/babylon-openapi3-base.yaml
+npx --yes swagger2openapi@7.0.8 /tmp/staking-swagger2.yaml -o /tmp/staking-openapi3-base.yaml
 ```
 
 ### 3b. Babylon gRPC Enhancements
@@ -158,6 +158,10 @@ tags:
     description: Reward calculation and distribution
   - name: mint
     description: Token inflation and minting parameters
+  - name: monitor
+    description: Monitor module for checkpoint and epoch verification
+  - name: zoneconcierge
+    description: Zone Concierge module for cross-chain header and finalization queries
 ```
 
 **5. Add x-tagGroups extension** (see references/tag-mapping.md for full structure).
@@ -173,7 +177,10 @@ tags:
 - `/babylon/finality/` → `finalityprovider`
 - `/babylon/incentive/` → `incentive`
 - `/babylon/mint/` → `mint`
+- `/babylon/monitor/` → `monitor`
+- `/babylon/zoneconcierge/` → `zoneconcierge`
 - `/cosmos/` → Keep original tag or assign `CometBFT`
+- Any unmapped `/babylon/<module>/` prefix → Log a warning and assign `other`
 
 ### 3c. Staking API Enhancements
 

.claude/skills/openapi-spec-transformer/references/tag-mapping.md

@@ -16,8 +16,12 @@
 | `/babylon/finality/` | `finalityprovider` | Finality provider registration and status |
 | `/babylon/incentive/` | `incentive` | Reward calculation and distribution |
 | `/babylon/mint/` | `mint` | Token inflation and minting parameters |
+| `/babylon/monitor/` | `monitor` | Monitor module for checkpoint and epoch verification |
+| `/babylon/zoneconcierge/` | `zoneconcierge` | Zone Concierge cross-chain header and finalization queries |
 | `/cosmos/` | Keep original | Standard Cosmos SDK endpoints |
 
+> **Fallback**: Any unmapped `/babylon/<module>/` prefix should be logged as a warning and assigned to an `other` tag. Update this table when new modules appear in the source spec.
+
 ### x-tagGroups Structure
 
 ```yaml
@@ -77,6 +81,17 @@ x-tagGroups:
       - mint-params
       - inflation
     description: Mint queries
+  - name: monitor
+    tags:
+      - monitor-checkpoints
+      - monitor-epochs
+    description: Monitor queries
+  - name: zoneconcierge
+    tags:
+      - zone-chain-info
+      - zone-headers
+      - zone-finalization
+    description: Zone Concierge queries
 ```
 
 ---
@@ -122,13 +137,19 @@ function assignTag(path) {
     '/babylon/finality/': 'finalityprovider',
     '/babylon/incentive/': 'incentive',
     '/babylon/mint/': 'mint',
+    '/babylon/monitor/': 'monitor',
+    '/babylon/zoneconcierge/': 'zoneconcierge',
   };
 
   for (const [prefix, tag] of Object.entries(tagMap)) {
     if (path.startsWith(prefix)) {
       return tag;
     }
   }
+  // Log unmapped Babylon modules so they can be added to this mapping
+  if (path.startsWith('/babylon/')) {
+    console.warn(`Unmapped Babylon module path: ${path}`);
+  }
   return 'other';
 }
 ```