Skip to content

fix: CI env-dependent test, badge counter gaps, static MIT badge + visual /cx-status#82

Merged
fermonterom merged 1 commit into
mainfrom
fix/ci-badges-visual-status
Jul 2, 2026
Merged

fix: CI env-dependent test, badge counter gaps, static MIT badge + visual /cx-status#82
fermonterom merged 1 commit into
mainfrom
fix/ci-badges-visual-status

Conversation

@fermonterom

Copy link
Copy Markdown
Owner

Summary

  • Red tests badge root cause: test_injector_v4.sh Test 7 read $HOME/.claude/cortex/reflexes.json (the developer's real install state) instead of a fixture — green locally, red on clean CI checkouts. Now self-contained (mktemp fixture with the same condition/resetAt shapes).
  • Badge undercount: 3 suites printed non-standard result lines the CI "Count tests" step greps for (Results: N passed, M failed) and were counted as 0/0: test_v332_acceptance.sh, test_v329_5_safety.sh, test_install_downgrade.sh. Standard line added without touching their output or exit codes.
  • License badge: dynamic shields github/license rendered "invalid" despite public repo + MIT detected by the API; replaced with a static MIT badge.
  • Visual /cx-status: render section redesigned — global health semaphore, per-section traffic lights with documented deterministic thresholds, 14-char confidence bars, KPI header row. Same collector, same data, glanceable.

Test plan

  • test_injector_v4.sh 13/13 (env dependency removed; deterministic on any machine)
  • Normalized suites re-run green (11/11, 9/9, 5/5) and their Results: lines parse with the exact CI grep
  • test_cx_status.sh 3/3 · security 9/9 · dream_cycle 40/40 · shellcheck clean
  • Tests badge gist will self-correct to green on next main CI run

Closes the "696 of 697 passing" red badge and the "license invalid" badge from the README header.

🤖 Generated with Claude Code

…e + visual /cx-status

- test_injector_v4.sh Test 7 read $HOME/.claude/cortex/reflexes.json (real
  install state) — self-contained fixture now; was the '696 of 697' red badge
- 3 suites lacked the standard 'Results: N passed, M failed' line the CI
  badge counter greps (v332_acceptance, v329_5_safety, install_downgrade)
- README license badge static MIT (dynamic shields render broke despite
  public repo + MIT detected by API)
- /cx-status render redesigned: global health semaphore, per-section
  traffic lights with documented thresholds, confidence bars, KPI header —
  same collector, same data, glanceable

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@fermonterom
fermonterom merged commit 44de494 into main Jul 2, 2026
@fermonterom
fermonterom deleted the fix/ci-badges-visual-status branch July 2, 2026 17:37
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