feat(export): opt-in APA .docx export of agentic answers#24
Merged
Conversation
Generalizable rework of the approach in #20 (closed as draft): - OFF by default via config.rag_modes.agentic.export_apa_docx (+ _dir). - python-docx is an optional [docx] extra (uv sync --extra docx), imported lazily; absent extra raises a clear ImportError, swallowed off the answer path. - Exporter tolerates Paper objects, dicts, and Author objects/strings (the original treated papers/authors as dicts/strings and silently failed). - No debug prints; export failures never break the answer; references deduped. - Hermetic tests: APA formatting, config default-off, disabled-helper no-op, and a real .docx write (skipped without the extra). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Generalizable rework of #20 (closed as draft). Same capability — export an agentic answer + APA references to a
.docx— but safe and opt-in.What changed vs #20
config.rag_modes.agentic.export_apa_docx(+export_apa_docx_dir). No behaviour change unless enabled. (Add APA DOCX manuscript export with reference deduplication #20 wrote a.docxon every agentic answer.)[docx]extra:python-docxis no longer a core dependency;uv sync --extra docx. Imported lazily; absent extra raises a clearImportErrorthat's caught off the answer path.Paperobjects, dicts, andAuthorobjects/strings. (Add APA DOCX manuscript export with reference deduplication #20 treated papers/authors as dicts/strings → would have silently produced blank refs on realPaperobjects.)prints; export failures are logged, never break the answer; references are deduplicated.Files
rag/export/apa_docx_exporter.py(new),rag/export/__init__.pyrag/agentic/orchestrator.py:_maybe_export_apa_docxgate + two call sitesconfig/schema.py,web/state.py,config.example.yml,pyproject.tomlTest Plan
tests/unit/test_apa_docx_export.py— APA formatting (Author objects/dicts/strings, 1/2/3+), config default-off, disabled-helper no-op, real.docxwrite (skipped without[docx]). 6 passed; full config suite green (32 passed).Supersedes #20.
🤖 Generated with Claude Code