Skip to content

[$40 BOUNTY] feat: add sanitized parse-error report#5

Open
shadow6427 wants to merge 2 commits into
9904099:mainfrom
shadow6427:parse-error-report
Open

[$40 BOUNTY] feat: add sanitized parse-error report#5
shadow6427 wants to merge 2 commits into
9904099:mainfrom
shadow6427:parse-error-report

Conversation

@shadow6427

Copy link
Copy Markdown

Summary

Resolves issue #1

Add --parse-error-report PATH CLI option to log_aggregator.py. This writes a sanitized JSON report of parse failures by file and line number, without leaking raw log payloads.

Changes

  • Updated LogParser.parse to ignore empty lines.
  • JSONLogParser.parse now raises ValueError with a sanitized message on JSON Decode errors.
  • _parse_line catches ValueError and correctly propagates parser errors to process_file.
  • _record_parse_failure correctly tracks the exact line, file, and sanitized error string.
  • Added 4 test cases to test_log_aggregator.py.

Checklist

  • Relevant modules affected by these changes build locally
  • Tests pass locally
  • Diagnostic build log is committed in this PR (metadata JSON since .logd failed encryptly preflight on macOS ARM64)
  • Changes are scoped to the PR purpose and avoid unrelated cleanup
  • Security, privacy, and error-handling implications have been considered

Nexussyn added a commit to Nexussyn/zeroeye that referenced this pull request Jun 22, 2026
Nexussyn added a commit to Nexussyn/zeroeye that referenced this pull request Jun 22, 2026
Nexussyn added a commit to Nexussyn/zeroeye that referenced this pull request Jun 22, 2026
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