Skip to content

fix(health): warn only for skills and gateway failures - #1236

Merged
gewenyu99 merged 4 commits into
mainfrom
codex/wizard-health-dependencies
Sep 10, 2026
Merged

fix(health): warn only for skills and gateway failures#1236
gewenyu99 merged 4 commits into
mainfrom
codex/wizard-health-dependencies

Conversation

@gewenyu99

@gewenyu99 gewenyu99 commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

What it blocks on

  • Both GitHub and AWS skills sources fail, in interactive runs.
  • The minted gateway’s /readyz check fails after retries, including connection failures and timeouts, in all runs.

What it warns on

  • Both skills sources failing is advisory in CI.
  • No warning-only cases in interactive runs. A single skills source failing or an unrelated service outage produces no warning.

@gewenyu99
gewenyu99 requested a review from a team as a code owner September 9, 2026 14:34
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci ai-observability
  • /wizard-ci basic-integration
  • /wizard-ci mcp-analytics
  • /wizard-ci replay-vision
  • /wizard-ci revenue
  • /wizard-ci self-driving
  • /wizard-ci warehouse
  • /wizard-ci warehouse-seeded

Test an individual app:

  • /wizard-ci ai-observability/anthropic
  • /wizard-ci ai-observability/google-adk
  • /wizard-ci ai-observability/groq
Show more apps
  • /wizard-ci ai-observability/manual-capture
  • /wizard-ci ai-observability/openai
  • /wizard-ci ai-observability/openai-agents
  • /wizard-ci ai-observability/opentelemetry
  • /wizard-ci ai-observability/vercel-ai
  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/flutter
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci mcp-analytics/custom-dispatcher
  • /wizard-ci mcp-analytics/typescript-sdk
  • /wizard-ci replay-vision/javascript-node
  • /wizard-ci replay-vision/next-js
  • /wizard-ci replay-vision/react-vite
  • /wizard-ci revenue/stripe
  • /wizard-ci self-driving/astro
  • /wizard-ci self-driving/fastapi
  • /wizard-ci self-driving/nuxt
  • /wizard-ci self-driving/react-router
  • /wizard-ci self-driving/sveltekit
  • /wizard-ci warehouse/monorepo-env
  • /wizard-ci warehouse/multi-source-next
  • /wizard-ci warehouse/stripe-node
  • /wizard-ci warehouse/zero-source
  • /wizard-ci warehouse-seeded/next-stripe
  • /wizard-ci warehouse-seeded/next-stripe-declined

Test against a Context Mill branch:

  • /wizard-ci all context-mill:my-branch

Add context-mill:<branch> to any command above to pin the Context Mill branch. It defaults to main.

Results will be posted here when complete.

@gewenyu99 gewenyu99 changed the title fix(health): check only wizard dependencies fix(health): warn only for skills and gateway failures Sep 9, 2026
Keep the CI legacy-gateway fallback from main and run the minted gateway's
readiness check after a successful mint.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@gewenyu99
gewenyu99 enabled auto-merge (squash) September 10, 2026 14:48
@gewenyu99
gewenyu99 disabled auto-merge September 10, 2026 14:49
@gewenyu99
gewenyu99 merged commit d2d757d into main Sep 10, 2026
18 checks passed
@gewenyu99
gewenyu99 deleted the codex/wizard-health-dependencies branch September 10, 2026 14: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.

2 participants