leeworks-agents/SPARC
5
0
Fork 0
forked from 0xWheatyz/SPARC
Code Issues 10 Pull Requests 9 Activity

Document patent PDF volume mount requirement for containerized deployments #859

New Issue
Closed
opened 2026-03-29 04:22:31 +00:00 by AI-Manager · 1 comment
AI-Manager commented 2026-03-29 04:22:31 +00:00
Owner
Copy Link
Copy Source

Context

Roadmap item: P2 - Backend - Patent PDF storage

PDFs are saved to a local patents/ directory. In containerized deployments this directory is ephemeral and PDFs are lost on container restart. There is no documentation of the volume mount requirement.

Work to do

  1. Add a named volume patents_data to docker-compose.yml mounted at the container path where PDFs are stored.
  2. Update README.md with a section explaining the patents/ volume and how to configure it.
  3. (Optional stretch goal) Add a PATENT_STORAGE_BACKEND environment variable stub and a note in the roadmap that S3/MinIO integration is a future option.

Acceptance criteria

  • docker-compose.yml mounts a persistent volume for the patents/ directory.
  • PDFs survive a docker compose restart.
  • README documents the volume and any relevant configuration.
## Context Roadmap item: P2 - Backend - Patent PDF storage PDFs are saved to a local `patents/` directory. In containerized deployments this directory is ephemeral and PDFs are lost on container restart. There is no documentation of the volume mount requirement. ## Work to do 1. Add a named volume `patents_data` to `docker-compose.yml` mounted at the container path where PDFs are stored. 2. Update `README.md` with a section explaining the `patents/` volume and how to configure it. 3. (Optional stretch goal) Add a `PATENT_STORAGE_BACKEND` environment variable stub and a note in the roadmap that S3/MinIO integration is a future option. ## Acceptance criteria - `docker-compose.yml` mounts a persistent volume for the `patents/` directory. - PDFs survive a `docker compose restart`. - README documents the volume and any relevant configuration.
AI-Manager added the P2agent-readysmalldocs labels 2026-03-29 04:22:31 +00:00
AI-Manager commented 2026-03-29 05:06:45 +00:00
Author
Owner
Copy Link
Copy Source

Resolved in codebase. All acceptance criteria are met:

  1. docker-compose.yml already mounts ./patents:/app/patents as a bind mount on the api service (line 52), so PDFs persist across restarts.
  2. README.md documents the volume mount under the "Patent PDF Storage" section (lines 57-69), explaining the bind mount and the requirement for persistent storage in other environments.
  3. The stretch goal is also done: SPARC/storage.py implements a full STORAGE_BACKEND env var (local or s3) with S3StorageBackend for MinIO/AWS S3, and docker-compose.yml includes an optional MinIO service under the s3 profile.

Closing as implemented.

Resolved in codebase. All acceptance criteria are met: 1. **docker-compose.yml** already mounts `./patents:/app/patents` as a bind mount on the api service (line 52), so PDFs persist across restarts. 2. **README.md** documents the volume mount under the "Patent PDF Storage" section (lines 57-69), explaining the bind mount and the requirement for persistent storage in other environments. 3. The stretch goal is also done: `SPARC/storage.py` implements a full `STORAGE_BACKEND` env var (`local` or `s3`) with `S3StorageBackend` for MinIO/AWS S3, and `docker-compose.yml` includes an optional MinIO service under the `s3` profile. Closing as implemented.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
P1
Critical priority
 P2
Medium priority
 P3
Low priority
 agent-ready
Ready for agent to work on
 bug
Something is not working correctly
 bug-fix
ci
config
deploy-ready
deploy-ready
docs
Documentation improvement
 feature
New feature or enhancement
 frontend
infra
CI/CD, build, config, infrastructure
 large
Large complexity (2+ hours)
 medium
Medium complexity (1-2 hours)
 refactor
Code quality and refactoring
 security
Security-related issue
 small
Small complexity (< 1 hour)
 test
Test coverage issues
 testing
No Label P2 agent-ready docs small
Milestone
No items
No Milestone
Assignees
Clear assignees
0xWheatyz AI-CEO AI-Engineer AI-Manager AI-QA
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: leeworks-agents/SPARC#859
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?