Make verify-workflows.sh idempotent by reusing already-deployed workflows

Fusion has no delete API, so once verify deploys a workflow it stays in
the CID and a second run fails re-importing the same name — which made
verify effectively single-use. The duplicate check already returns the
existing definition's id; use it: when a workflow is already deployed,
mark deploy PASS and reuse that id for the execute/browser phases
instead of attempting a doomed re-import. Re-running verify now proves
the deployed state rather than false-failing on the duplicate.

Author: mraible

verify-workflows.sh

@@ -252,18 +252,29 @@ for wf_file in "${YAML_FILES[@]}"; do
     D_STATUS="SKIP"
     say "  deploy:    ${YELLOW}SKIP${RESET} (validate failed)"
   else
-    # Informational duplicate check (recorded in notes, not its own column).
+    # Duplicate check. Fusion has no delete API, so once a workflow is deployed
+    # (e.g. by a previous verify run) it stays in the CID and re-importing the
+    # same name fails. That is not a verify failure: the definition IS deployed.
+    # If the check reports an existing_id, treat deploy as PASS and reuse that
+    # id for the execute/browser phases instead of attempting a doomed import.
+    EXISTING_ID=""
     if DUP_OUT="$("$PYTHON" "$QUERY_PY" --check-yaml "$wf_file" --json 2>/dev/null)"; then
       DUP_COUNT="$(printf '%s' "$DUP_OUT" | jq '.duplicates | length' 2>/dev/null || echo 0)"
       if [ "${DUP_COUNT:-0}" -gt 0 ]; then
-        NOTES="${NOTES:+$NOTES; }duplicate name already deployed"
-        say "  (dup-check: ${YELLOW}already exists${RESET})"
+        EXISTING_ID="$(printf '%s' "$DUP_OUT" | jq -r '.duplicates[0].existing_id // empty' 2>/dev/null)"
+        say "  (dup-check: ${YELLOW}already exists${RESET}${EXISTING_ID:+ — reusing ID $EXISTING_ID})"
       fi
     fi
 
+    if [ -n "$EXISTING_ID" ]; then
+      # Already deployed on a prior run — reuse it, don't re-import.
+      DEF_ID="$EXISTING_ID"
+      D_STATUS="PASS"
+      NOTES="${NOTES:+$NOTES; }already deployed (reused existing ID)"
+      say "  deploy:    ${GREEN}PASS${RESET} (already deployed, ID: $DEF_ID)"
     # Import. validate + duplicate checks are already separate concerns here,
     # so skip them inside the importer to keep each scorecard column distinct.
-    if DEPLOY_OUT="$("$PYTHON" "$IMPORT_PY" --skip-validate --skip-duplicate-check "$wf_file" 2>&1)"; then
+    elif DEPLOY_OUT="$("$PYTHON" "$IMPORT_PY" --skip-validate --skip-duplicate-check "$wf_file" 2>&1)"; then
       DEF_ID="$(printf '%s\n' "$DEPLOY_OUT" | grep -oE 'ID: [0-9a-f]{32}' | head -1 | sed 's/ID: //')"
       D_STATUS="PASS"
       say "  deploy:    ${GREEN}PASS${RESET} (ID: ${DEF_ID:-unknown})"