Skip to content

Serialize logger error metadata safely#90

Merged
miyaontherelay merged 2 commits into
mainfrom
codex/issue-83-error-serialization
Jul 17, 2026
Merged

Serialize logger error metadata safely#90
miyaontherelay merged 2 commits into
mainfrom
codex/issue-83-error-serialization

Conversation

@miyaontherelay

Copy link
Copy Markdown
Contributor

Summary

  • normalize Error and unknown logger metadata without collapsing errors to empty objects
  • preserve bounded, redacted structured diagnostics and issue context across factory logger boundaries
  • contain getters, custom stack hooks, cycles, BigInt, and per-argument serialization failures

Validation

  • focused: 4 files, 385 tests passed
  • full: 39 files, 753 tests passed
  • npm run build
  • npm pack --dry-run
  • git diff --check
  • independent shadow audit: signed off with no blockers

Fixes #83

@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@miyaontherelay
miyaontherelay merged commit 38136e9 into main Jul 17, 2026
2 checks passed
@miyaontherelay
miyaontherelay deleted the codex/issue-83-error-serialization branch July 17, 2026 07:36
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.

[factory] error {} — raw Errors serialize to {} (message/stack are non-enumerable); describeError() already exists one line away

1 participant