feat: allow to add a query-planner-skip label to ignore engine checks

[alepane21]

Summary by CodeRabbit

• Chores
  • CI job renamed to "check-labels-and-engine-changes" and unified label/engine-change gating.
  • CI now detects a “query-planner-skip” label and exposes a skip flag (has_skip_label) to skip non-essential jobs.
  • Skip-aware gate propagated across build, test, release, and notification jobs.
  • Conditional checks updated to guard on has_skip_label and standardized boolean output handling; minor workflow formatting cleanup.

Checklist

• I have discussed my proposed changes in an issue and have received approval to proceed.
• I have followed the coding standards of the project.
• Tests or benchmarks have been added or updated.
• Documentation has been updated on https://github.com/wundergraph/cosmo-docs.
• I have read the Contributors Guide.

[coderabbitai]

Walkthrough

Renames the CI gating job to check-labels-and-engine-changes, adds a check_label_skip step that exposes has_skip_label, wires that output through dependent jobs' needs/outputs, and updates conditional expressions to skip steps when has_skip_label is 'true'. Minor YAML formatting tweaks.

Changes

Cohort / File(s)

Change summary

Workflow gating and downstream wiring .github/workflows/query-planner-ci.yaml

Renamed job check-label-and-engine-changes → check-labels-and-engine-changes; added check_label_skip step using github-script to detect query-planner-skip and expose has_skip_label; added has_skip_label to job outputs and propagated it to downstream needs/outputs; updated dependent jobs (build-router, build-cli, cli-release-url, build_test, get_pr_url, comment-on-failure, comment-on-success, etc.) to depend on the renamed job; adjusted if conditions to guard with has_skip_label != 'true' and standardized boolean output string checks; minor spacing/quote formatting changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• refactor(ci): force query-planner CI if engine changes #2165 — Modifies the same CI gating job and downstream wiring in .github/workflows/query-planner-ci.yaml, likely overlapping with the rename/outputs and conditional logic changes.

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly describes the primary feature introduced by this changeset, namely the ability to use a query-planner-skip label to bypass engine checks, matching both the intent and the modifications in the workflow configuration.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

.github/workflows/query-planner-ci.yaml (1)

156-156: Fix the stale needs reference to the renamed job.

check-label-and-engine-changes no longer exists, so the workflow will fail to even start. Update every needs entry that still uses the old id before merging.

-    needs: [ build-router, build-cli, cli-release-url, check-label-and-engine-changes ]
+    needs: [ build-router, build-cli, cli-release-url, check-labels-and-engine-changes ]
...
-    needs: [ build_test, check-label-and-engine-changes ]
+    needs: [ build_test, check-labels-and-engine-changes ]

Also applies to: 179-179

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 32483b8 and 015701b.

📒 Files selected for processing (1)

• .github/workflows/query-planner-ci.yaml (9 hunks)

🧰 Additional context used

🪛 actionlint (1.7.7)

.github/workflows/query-planner-ci.yaml

55-55: the runner of "actions/github-script@v6" action is too old to run on GitHub Actions. update the action's version to fix this issue

(action)

---

126-126: input "with_tests" is not defined in action "Build and Push Image (PR only)" defined at "./.github/actions/build-push-image". available inputs are "docker_build_target", "docker_context", "docker_password", "docker_username", "dockerfile", "image_description", "image_name", "image_platforms", "load_Image", "push", "token"

(action)

---

162-162: property "check-labels-and-engine-changes" is not defined in object type {build-cli: {outputs: {wgc: string}; result: string}; build-router: {outputs: {image_ref: string}; result: string}; cli-release-url: {outputs: {wgc: string}; result: string}}

(expression)

---

162-162: property "check-labels-and-engine-changes" is not defined in object type {build-cli: {outputs: {wgc: string}; result: string}; build-router: {outputs: {image_ref: string}; result: string}; cli-release-url: {outputs: {wgc: string}; result: string}}

(expression)

---

162-162: property "check-labels-and-engine-changes" is not defined in object type {build-cli: {outputs: {wgc: string}; result: string}; build-router: {outputs: {image_ref: string}; result: string}; cli-release-url: {outputs: {wgc: string}; result: string}}

(expression)

---

181-181: property "check-labels-and-engine-changes" is not defined in object type {build_test: {outputs: {workflow_id: string; workflow_url: string}; result: string}}

(expression)

---

181-181: property "check-labels-and-engine-changes" is not defined in object type {build_test: {outputs: {workflow_id: string; workflow_url: string}; result: string}}

(expression)

---

181-181: property "check-labels-and-engine-changes" is not defined in object type {build_test: {outputs: {workflow_id: string; workflow_url: string}; result: string}}

(expression)

[alepane21]

@coderabbitai full review

[coderabbitai]

✅ Actions performed

Full review triggered.

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 300ed0f and 58ab906.

📒 Files selected for processing (1)

• .github/workflows/query-planner-ci.yaml (9 hunks)