Skip to content

ci: add Pages healthcheck workflow (#278 Task 4) - #287

Merged
jsboige merged 1 commit into
masterfrom
ci/278-task4-healthcheck-pages
May 16, 2026
Merged

ci: add Pages healthcheck workflow (#278 Task 4)#287
jsboige merged 1 commit into
masterfrom
ci/278-task4-healthcheck-pages

Conversation

@jsboige

@jsboige jsboige commented May 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Verify workflow YAML is valid (syntax)
  • Manual trigger via gh workflow run healthcheck.yml after merge
  • Confirm curl -I returns 200 on the target URL

Part of Epic #278 (Task 4/7).

🤖 Generated with Claude Code

Runs after Pages deployment to verify CardPen URL returns 200 OK.
Also runs weekly (Monday) and on manual trigger. Prevents silent
404 regressions like the one fixed in #279.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatique — Hermes (myia-po-2026)

Verdict : ✅ COMMENT — Workflow CI propre et utile.

Workflow healthcheck.yml :

  • Déclenché sur workflow_run (après deploy Pages), workflow_dispatch, et schedule hebdomadaire (lundi 06:23 UTC) — bonne couverture.
  • Vérifie HTTP 200 sur la Pages URL avec curl, follow redirects (-L), timeout 15s — robuste.
  • Utilise les annotations GitHub Actions (::error::, ::notice::) — s'intègre bien dans l'UI.
  • Exit 1 si une URL échoue — CI sera rouge comme il se doit.

Point mineur : Le tableau URLS ne contient qu'une seule URL pour l'instant — facile à étendre. L'URL est codée en dur plutôt que via github.repository, mais c'est correct pour un repo spécifique.

Conclusion : Workflow simple, efficace, prêt à merger.

— Hermes (myia-po-2026) [CRON:review-pr 17:35Z]

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean CI workflow. Triggered on Pages deploy completion + weekly cron. Proper error handling (exit 1 on non-200, GitHub Actions ::error:: annotations). Curl with timeout and -L flag for redirects. Single URL check is appropriate for the current single Pages site.

@jsboige

jsboige commented May 16, 2026

Copy link
Copy Markdown
Contributor Author

LGTM ✅ — workflow propre et bien dimensionné.

Vérifications :

  • Trigger workflow_run cible "Deploy static content to Pages" → match exact avec name: dans .github/workflows/static.yml
  • Cron 23 6 * * 1 (Monday ~06:23 UTC) — bonne pratique d'éviter les minutes 0/30
  • Curl -L --max-time 15 + check 200 + exit 1 sur fail → CI rouge approprié
  • Annotations ::error:: / ::notice:: → bonne intégration UI
  • URLS array facilement extensible (futurs Release* paths CSV/JSON quand audit Release URL sera fait)

Couverture : aurait attrapé le bug original #279 (ReleaseCardpenUrl 404). Goal de Task 4 atteint.

Hermes a déjà approuvé. Merge.

@jsboige
jsboige merged commit 9b781b2 into master May 16, 2026
1 check passed
@jsboige
jsboige deleted the ci/278-task4-healthcheck-pages branch June 1, 2026 21:40
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