Skip to content

Add structured logging + CI on PRs - #1

Merged
AdrianBinDC merged 2 commits into
mainfrom
feat/logging
Jul 18, 2026
Merged

AdrianBinDC merged 2 commits into
mainfrom
feat/logging

Conversation

@AdrianBinDC

@AdrianBinDC AdrianBinDC commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add stdlib logging (wirecache.logutil) to stderr + data/wirecache.log by default
  • Wire fetch/store/docker/CLI through the logger; keep stdout for JSON/text/voice
  • Support -v / -q / --log-file and WIRECACHE_LOG_* env vars
  • CI: run tests on PRs (and pushes) to main

Test plan

  • uv run pytest -q locally (18 tests)
  • Confirm GitHub Actions CI is green on this PR
  • Spot-check: uv run wirecache -v status logs to stderr and data/wirecache.log

Made with Cursor

Adrian Bolinger and others added 2 commits July 18, 2026 15:12
Route operational logs to stderr and data/wirecache.log while keeping
stdout clean for CLI results; run pytest on pull requests.

Co-authored-by: Cursor <cursoragent@cursor.com>
Normalize published dates on ingest and ignore pre-1990 values in stats
so garbage like 0001-01-01 from Lil'Log no longer skews status.

Co-authored-by: Cursor <cursoragent@cursor.com>
@AdrianBinDC
AdrianBinDC merged commit f791989 into main Jul 18, 2026
1 check passed
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