Skip to content

Add Phase 1 smoke test#27

Merged
enricopiovesan merged 1 commit into
mainfrom
claude/issue-5-phase1-smoke
Jul 1, 2026
Merged

Add Phase 1 smoke test#27
enricopiovesan merged 1 commit into
mainfrom
claude/issue-5-phase1-smoke

Conversation

@enricopiovesan

Copy link
Copy Markdown
Contributor

Summary

  • Makes scripts/ci/phase1_smoke.sh executable (was -rw-r--r--)
  • Changes no-runtime behavior from FAIL (exit 1) to SKIP (exit 0) — CI without a configured runtime now passes cleanly with a skip notice
  • Removes continue-on-error: true from nightly workflow — the step is now a real gate; set the TRAVERSE_RUNTIME_URL repo variable to activate it against a live runtime

Closes #5

Definition of Done

  • Smoke test starts or connects to local Traverse runtime
  • Verifies: execute → poll → output fields present
  • Output is concise and CI-friendly
  • Exits 0 on pass or when no runtime is configured (skip); exits 1 with detail on failure

Validation

# No runtime — clean skip
bash scripts/ci/phase1_smoke.sh  # → SKIP, exit 0

# With runtime
TRAVERSE_RUNTIME_URL=http://127.0.0.1:8787 bash scripts/ci/phase1_smoke.sh

- Makes phase1_smoke.sh executable
- Changes no-runtime behavior from FAIL to SKIP (exit 0) so nightly CI
  passes without a live runtime; set TRAVERSE_RUNTIME_URL to activate
- Removes continue-on-error from nightly workflow — script now self-manages
  the skip, making the step a real gate when a runtime URL is configured

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@enricopiovesan enricopiovesan merged commit f593f76 into main Jul 1, 2026
4 checks passed
@enricopiovesan enricopiovesan deleted the claude/issue-5-phase1-smoke branch July 1, 2026 18:14
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.

Add Phase 1 smoke test

1 participant