Skip to content

fix(github-orchestration): Detect Cloudflare dashboard URLs and filter Supabase ignored comments - #339

Merged
sylvansys merged 1 commit into
mainfrom
338-fix/pr-created-httpsgithubcomconstellosconst
Jan 31, 2026
Merged

fix(github-orchestration): Detect Cloudflare dashboard URLs and filter Supabase ignored comments#339
sylvansys merged 1 commit into
mainfrom
338-fix/pr-created-httpsgithubcomconstellosconst

Conversation

@sylvansys

Copy link
Copy Markdown
Collaborator

Summary

  • Add dash.cloudflare.com URL extraction to extractCloudflarePreviewUrls() so Cloudflare bot comments with dashboard log links are detected
  • Fix .workers.dev/.pages.dev regex to support multi-subdomain URLs (e.g. abc123.my-project.pages.dev)
  • Filter out Supabase bot "ignored" comments in extractSupabasePreviewBranches() to prevent showing the main project ref as a preview branch when no preview was created

Fixes #338

Test plan

🤖 Generated with Claude Code

…ss.\n\nSession-ID: 67900f0e-cd80-4102-88b4-aa9bac8a3416\nSession-Timestamp: 2026-01-31T04:44:32.809Z\nBranch: 338-fix/pr-created-httpsgithubcomconstellosconst\n\n🤖 Generated with [Claude Code](https://claude.com/claude-code)\n\nCo-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@constellos

constellos Bot commented Jan 31, 2026

Copy link
Copy Markdown
Contributor

Constellos Review - PR feedback from your reviewer agents.

Requirements: ❌ 0 • ❌ 0 • ⏭️ 0

View agent prompt

Code Quality: ✅ 4 • ❌ 0 • ⏭️ 0

View agent prompt

Check Status Reasoning
undefined The ci-status.ts file effectively consolidates shared CI utilities into reusable functions. Functions like fetchPRCommentBodies(), extractPreviewUrls(), and execCommand() are well-factored to avoid duplication across multiple hook implementations.
undefined All functionality in ci-status.ts directly supports documented use cases (commit-task-await-ci-status, await-pr-status, commit-session-await-ci-status hooks). Features like fail-fast CI checking, preview URL extraction, and merge conflict detection solve real problems without speculative additions.
undefined Code demonstrates excellent separation of concerns with clear single-responsibility functions (e.g., checkPRExists, waitForCIChecks, extractPreviewUrls). Each function has a focused purpose, comprehensive JSDoc documentation, and well-defined interfaces. The module exports cohesive utilities for CI/PR operations.
undefined Code is highly maintainable with extensive JSDoc comments, clear TypeScript interfaces, descriptive function/variable names, and consistent error handling patterns. The file includes helpful examples in documentation and uses semantic constants (MAX_OUTPUT_CHARS, DEFAULT_TIMEOUT_MS) for magic numbers.

@sylvansys
sylvansys merged commit 67af237 into main Jan 31, 2026
9 of 10 checks passed
@sylvansys
sylvansys deleted the 338-fix/pr-created-httpsgithubcomconstellosconst branch January 31, 2026 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: Cloudflare deployments missing + Supabase URL points to main project

1 participant