Skip to content

[bench] logging, test suite, and CI workflow#3

Merged
dburks-svg merged 2 commits into
mainfrom
bench/deepseek-timeout-increase
May 13, 2026
Merged

[bench] logging, test suite, and CI workflow#3
dburks-svg merged 2 commits into
mainfrom
bench/deepseek-timeout-increase

Conversation

@dburks-svg
Copy link
Copy Markdown
Contributor

Summary

  • Add util/logging.py — centralized governance logging with get_logger factory, correlation IDs, and 4 governance-specific helpers (log_evaluation, log_veto, log_chain, log_ledger_entry)
  • Add 9 new test files covering all previously untested modules: ledger integrity (chain, verify), pipeline stages (constitution, challenger, defender, oracle), orchestration (runner), hook entry point, and API client
  • Add GitHub Actions CI workflow running the full suite on Python 3.11–3.13 for every push/PR to main
  • Total suite: 264 tests across 14 files, all passing in ~0.3s

Test plan

  • py -m unittest discover -s tests -v — 264/264 pass
  • Each test file runs independently per CLAUDE.md rule 13
  • All API calls mocked — no network traffic, no secrets needed
  • Ledger tests use temp directories — no side effects on real ledger

Generated with Claude Code

dburks-svg and others added 2 commits May 13, 2026 00:12
- 9 new test files covering all previously untested modules:
  ledger (chain, verify), pipeline (constitution, challenger,
  defender, oracle, runner), hook entry point, and API client
- GitHub Actions CI workflow running on Python 3.11-3.13
- Total suite: 264 tests, 14 files, all passing

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
os.path.isabs() does not recognize Windows absolute paths on Linux,
causing test_windows_absolute_path_blocked to fail in CI.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dburks-svg dburks-svg merged commit b4a2bed into main May 13, 2026
3 checks passed
@dburks-svg dburks-svg deleted the bench/deepseek-timeout-increase branch May 13, 2026 07:21
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