feat: add the figures to project.zip

[maehr]

Replaces #55. A merge loop merged #55 into feat/map-explanations instead of main. The content is unchanged. It already includes the Copilot fixes.

Summary

Closes #50.

• exports.figure_files renders every chart of the result view. An Altair chart becomes a standalone HTML page with its data inside. A word cloud stays a PNG.
• The group, date, and position charts appear only when their data exists.
• Privacy: without Include the document text, the document map holds no snippet. The checkbox now governs the figures as it governs the CSV.
• project.zip writes the files under figures/. README.txt explains the folder and says that an HTML figure loads Vega from a CDN.
• The download button builds the package only on a click. The word clouds are slow under Pyodide.
• The export disables the Altair limit of 5,000 rows. A long text can exceed it.

An Altair PluginEnabler.__exit__ annotation rejects a valid with. One ty: ignore[invalid-context-manager] line covers it.

Test

• Unit tests cover the base figures, the group, date, and long-text branches, the snippet rule, and the row limit.
• Browser test on Pyodide 314: the newspaper demo gives 19 figures, the book demo gives 30.
• topic_map.html and document_map.html render outside the app.

🤖 Generated with Claude Code

https://claude.ai/code/session_01EsEUVKgZPLky1kHXnU2aBW

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 24 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4ab8c1b6-18ba-4cba-9181-e49d069759bd

📥 Commits

Reviewing files that changed from the base of the PR and between 27132db and 83791e9.

📒 Files selected for processing (6)

• README.md
• SPECS.md
• app.py
• simple_topic_modeling/exports.py
• simple_topic_modeling/plots.py
• tests/test_exports.py

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Copilot review overview

🟢 Approval recommended

The implementation matches the stated requirements and covers its conditional, privacy, security, and browser-specific behavior.

Review effort: Balanced
Findings: None

What changed in this PR

Adds all result-view charts to project.zip while preserving privacy controls and browser performance.

Changes:

• Exports Altair charts as standalone HTML and word clouds as PNG.
• Builds ZIP files lazily and conditionally includes metadata/position figures.
• Adds script-safe serialization, documentation, and comprehensive tests.

File	Description
simple_topic_modeling/​exports.py	Generates figure files and adds them to the ZIP.
simple_topic_modeling/​plots.py	Makes document snippets optional in tooltips.
app.py	Lazily builds the expanded package on download.
tests/​test_exports.py	Tests figure contents, conditions, privacy, limits, and escaping.
SPECS.md	Specifies figure export behavior.
README.md	Documents figures in project.zip.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.