Skip to content

Conversation

@anantham
Copy link
Owner

@anantham anantham commented Jan 8, 2026

Summary

Add structured export warnings for missing translations/image cache misses and package validation logs without changing EPUB output. Warnings are surfaced in progress messages and telemetry to improve diagnostics without altering EPUB contents.

Root Cause (if bug fix)

N/A (observability improvement)

Changes

  • store/slices/exportSlice.ts: record structured warnings and surface warning summary/telemetry.
  • services/epubService/packagers/epubPackager.ts: add package validation warnings.
  • tests/services/epubPackager.diagnostics.test.ts: verify warning emission.
  • docs/WORKLOG.md: session log updates.

Testing

  • npx vitest run tests/services/epubPackager.diagnostics.test.ts

Review Checklist

  • No direct commits to main
  • Follows commit format
  • Ready for Codex review
  • No unrelated changes mixed in

Context: Improve observability for missing translations, cache misses, and packaging validation without changing EPUB output.

Changes: Emit structured warnings in export and packager; add diagnostics test.

Impact: Users see warning summary; telemetry records warning context.

Tests: npx vitest run tests/services/epubPackager.diagnostics.test.ts

Docs: docs/WORKLOG.md

Fixes:

ADR:
@vercel
Copy link

vercel bot commented Jan 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
lexicon-forge Ready Ready Preview, Comment Jan 8, 2026 8:18pm

@anantham
Copy link
Owner Author

anantham commented Jan 9, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

@anantham anantham merged commit 75a35eb into main Jan 9, 2026
3 checks 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.

2 participants