Skip to content

feat: preflight GitHub Check writeback#1

Merged
thebiglabasky merged 5 commits into
mainfrom
herve/github-test-session-metadata
Jul 4, 2026
Merged

feat: preflight GitHub Check writeback#1
thebiglabasky merged 5 commits into
mainfrom
herve/github-test-session-metadata

Conversation

@thebiglabasky

Copy link
Copy Markdown
Collaborator

Summary

Updates the packaged Checkly GitHub Action to preflight GitHub Check writeback before invoking the CLI.

This PR includes:

  • github-report fallback behavior: detached runs only when backend writeback is available
  • non-detached GitHub reporter fallback when the Checkly GitHub App is unavailable
  • CHECKLY_GITHUB_* metadata export for the CLI
  • deployment status environment_url propagation to ENVIRONMENT_URL when unset
  • README/input copy updates
  • dry-run tests for available/unavailable reporting paths

Linear

Validation

  • ./scripts/test.sh

thebiglabasky and others added 5 commits July 2, 2026 17:04
- An old pinned CLI no longer hard-fails the job: cli-version < 8.12.0
  now falls back to the non-detached mode with reason
  cli_version_too_old, skipping the preflight entirely.
- The preflight fetch gets a 10s AbortSignal timeout so a slow backend
  can never stall the customer's job before tests run.
- The preflight command substitution tolerates a non-zero node exit
  (OOM/kill) instead of aborting the whole action under set -e.
- The preflight-sourced reason is sanitized to [a-z0-9_] before being
  interpolated into ::warning:: workflow commands and the step summary,
  closing a workflow-command injection surface.
- resolve_github_repository/sha run once and feed both env configs,
  dropping four redundant node forks per run; configure_github_report
  loses its unreachable duplicate boolean validation.
- README/action.yml copy updated; tests cover the old-CLI fallback and
  the invalid-boolean error.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@thebiglabasky
thebiglabasky marked this pull request as ready for review July 4, 2026 08:38
@thebiglabasky
thebiglabasky merged commit 055016c into main Jul 4, 2026
1 check passed
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.

1 participant