Skip to content

Restructure examples directory and improve document storage#189

Merged
rejojer merged 5 commits intomainfrom
refactor/restructure-examples-clean
Mar 27, 2026
Merged

Restructure examples directory and improve document storage#189
rejojer merged 5 commits intomainfrom
refactor/restructure-examples-clean

Conversation

@rejojer
Copy link
Copy Markdown
Member

@rejojer rejojer commented Mar 27, 2026

Summary

  • Consolidate tests/ into examples/documents/, clean up .gitignore and README links
  • Add line_count for markdown documents and reorder structure keys for consistency
  • Lazy-load documents via _meta.json index with corruption recovery, cache PDF page text so queries don't need the original file
  • Update demo script with __file__-based paths and add pre-shipped workspace for out-of-the-box experience

Test plan

  • Run examples/openai_agents_demo.py and verify it works without downloading the PDF
  • Index a new PDF and confirm lazy-load round-trips correctly
  • Delete _meta.json and verify it rebuilds from individual doc JSONs

Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@rejojer
Copy link
Copy Markdown
Member Author

rejojer commented Mar 27, 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 👎.

@rejojer rejojer merged commit 7772283 into main Mar 27, 2026
4 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.

1 participant