Skip to content

feat(app): add document (PDF) memory support#104

Merged
comppaz merged 1 commit intomainfrom
fix/code-quality-improvements
Jan 26, 2026
Merged

feat(app): add document (PDF) memory support#104
comppaz merged 1 commit intomainfrom
fix/code-quality-improvements

Conversation

@comppaz
Copy link
Copy Markdown
Collaborator

@comppaz comppaz commented Jan 26, 2026

Summary

  • Add complete document (PDF) memory feature with encrypted storage and AI processing
  • Upload PDFs with text extraction and thumbnail generation
  • Encrypted document storage using Fernet encryption
  • AI-powered title, summary, and tag generation
  • PDF viewer with pagination in memory detail panel
  • Open documents in system default application

Code Quality Improvements

  • Extract Poppler version to workflow variable to prevent future CI/CD breakage
  • Create shared useDocumentUpload hook to reduce code duplication
  • Add error logging to Electron temp file cleanup
  • Add empty PDF validation to reject image-only/scanned PDFs
  • Simplify Vite PDF.js worker configuration

Test plan

  • Upload a PDF document from HomePage
  • Upload a PDF document from MemoriesPage
  • Verify thumbnail is generated and displayed
  • Verify text extraction works (check memory detail panel)
  • Verify AI processing generates title, summary, and tags
  • Open document in system viewer
  • Try uploading an image-only PDF (should be rejected)
  • Verify CI/CD workflow syntax is valid

Add complete document memory feature with encrypted storage and AI processing:
- Upload PDFs with text extraction and thumbnail generation
- Encrypted document storage using Fernet encryption
- AI-powered title, summary, and tag generation
- PDF viewer with pagination in memory detail panel
- Open documents in system default application

Code quality improvements:
- Extract Poppler version to workflow variable
- Create shared useDocumentUpload hook (DRY)
- Add error logging to Electron temp file cleanup
- Add empty PDF validation in backend
- Simplify Vite PDF.js worker configuration
@comppaz comppaz merged commit 7e96c33 into main Jan 26, 2026
1 check failed
@comppaz comppaz deleted the fix/code-quality-improvements branch January 26, 2026 12:41
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