Skip to content

feat: add health-check and sync-check CI scripts for all commits#110

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/add-health-check-and-sync-check-scripts
Draft

feat: add health-check and sync-check CI scripts for all commits#110
Copilot wants to merge 2 commits into
mainfrom
copilot/add-health-check-and-sync-check-scripts

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 5, 2026

No CI ran on non-main branches, and the health / sync:check / test:brain npm scripts were missing despite their backing scripts already existing in scripts/.

Changes

package.json

  • Registers three new npm scripts: health, test:brain, sync:check

scripts/sync-check.js (new)

  • Verifies Supabase REST API reachability (200/401 = up), Edge Function deployment (non-404 = deployed), and Shopify Storefront API connectivity via a minimal { shop { name } } GraphQL probe
  • Shopify check skipped when VITE_SHOPIFY_STOREFRONT_TOKEN is unset — CI-safe
  • Respects VITE_SHOPIFY_API_VERSION env var (defaults 2025-07)

.github/workflows/ci-all-commits.yml (new)

  • Triggers on every push to any branch and on all PRs
  • Three jobs:
    • lint-and-typecheck — always; runs lint + typecheck
    • sync-check — always; runs npm run sync:check against Supabase/Shopify public endpoints (no secrets required)
    • health-checkmain only, runs after the above two pass; pings the live Lovable deployment via npm run health
  • Workflow-level permissions: contents: read for least-privilege token scope

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • qqceibvalkoytafynwoc.supabase.co
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node node --input-type=module (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Add npm scripts: sync:check, health, test:brain to package.json
- Add scripts/sync-check.js: verifies Supabase REST API, Edge Functions,
  and Shopify Storefront API connectivity; CI-safe (exits 0 when creds absent)
- Add .github/workflows/ci-all-commits.yml: runs lint+typecheck+sync-check
  on every push to any branch; runs health-check on main after other jobs pass

Co-authored-by: asperpharma <252395498+asperpharma@users.noreply.github.com>
Copilot AI changed the title [WIP] Add health check and sync check scripts for project feat: add health-check and sync-check CI scripts for all commits Mar 5, 2026
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.

2 participants