fix: preserve monitor fallback during migration

Co-Authored-By: Codex <noreply@openai.com>

Author: Pigbibi

.github/workflows/sync-cloud-run-env.yml

@@ -861,8 +861,6 @@ jobs:
             remove_env_vars+=("STRATEGY_PLUGIN_ALERT_TELEGRAM_BODY_MAX_CHARS")
           fi
 
-          remove_env_vars+=("MONITOR_DISPATCH_TARGETS_JSON" "LONGBRIDGE_MONITOR_DISPATCH_TARGETS_JSON")
-
           gcloud_args=(
             run services update "${CLOUD_RUN_SERVICE}"
             --region "${CLOUD_RUN_REGION}"
@@ -1315,6 +1313,8 @@ jobs:
                   or scheduler.get("location")
                   or target.get("region")
                   or target.get("cloud_run_region")
+                  or os.environ.get("CLOUD_SCHEDULER_LOCATION")
+                  or os.environ.get("CLOUD_RUN_REGION")
                   or ""
               ).strip()
               if service_name:

tests/test_scheduler_workflow.py

@@ -13,10 +13,21 @@ def test_replacement_probe_and_precheck_exist_before_shared_dispatcher_cleanup()
     assert precheck < cleanup_job < cleanup
 
     cleanup_section = workflow[cleanup_job:]
+    sync_section = workflow[:cleanup_job]
     assert "needs: sync" in cleanup_section
     assert "environment: longbridge-sg" in cleanup_section
     assert 'replacement_jobs=("${service_name}-probe-scheduler" "${service_name}-precheck-scheduler")' in cleanup_section
     assert "if: steps.replacements.outputs.ready == 'true'" in cleanup_section
+    assert "MONITOR_DISPATCH_TARGETS_JSON" not in sync_section
+    assert "LONGBRIDGE_MONITOR_DISPATCH_TARGETS_JSON" not in sync_section
+
+
+def test_replacement_verifier_falls_back_to_global_scheduler_location() -> None:
+    workflow = Path(".github/workflows/sync-cloud-run-env.yml").read_text(encoding="utf-8")
+    cleanup_section = workflow[workflow.index("cleanup-shared-monitor:") :]
+
+    assert 'or os.environ.get("CLOUD_SCHEDULER_LOCATION")' in cleanup_section
+    assert 'or os.environ.get("CLOUD_RUN_REGION")' in cleanup_section
 
 
 def test_existing_scheduler_cron_rejects_ambiguous_day_fields() -> None:

tests/test_sync_cloud_run_env_workflow.sh

@@ -209,7 +209,6 @@ grep -Fq 'join_by_delimiter()' "$workflow_file"
 grep -Fq 'gcloud_args+=(--remove-secrets "$(IFS=,; echo "${remove_secret_vars[*]}")")' "$workflow_file"
 grep -Fq 'gcloud_args+=(--update-secrets "$(IFS=,; echo "${secret_pairs[*]}")")' "$workflow_file"
 grep -Fq -- '--update-env-vars "^|^$(join_by_delimiter "|" "${env_pairs[@]}")"' "$workflow_file"
-grep -Fq 'remove_env_vars+=("MONITOR_DISPATCH_TARGETS_JSON" "LONGBRIDGE_MONITOR_DISPATCH_TARGETS_JSON")' "$workflow_file"
 grep -Fq 'Sync Cloud Scheduler schedule' "$workflow_file"
 grep -Fq 'scheduler_location="${CLOUD_SCHEDULER_LOCATION:-${CLOUD_RUN_REGION}}"' "$workflow_file"
 grep -Fq 'target_env = target.get("env") or {}' "$workflow_file"