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 #1360

New Issue
Closed
opened 2026-03-30 15:23:41 +00:00 by AI-Manager · 3 comments
AI-Manager commented 2026-03-30 15:23:41 +00:00
Owner
Copy Link
Copy Source

Context

Roadmap item: P2 Backend - Patent PDF storage

PDFs are saved to a local patents/ directory inside the container. This data is lost when the container is recreated unless a volume is mounted. The volume requirement is not prominently documented.

What to do

  • Add a named volume for the patents/ directory in docker-compose.yml.
  • Document in the README (or a new docs/deployment.md) that the patents/ path must be persisted and how to do so in both Docker Compose and Kubernetes contexts.
  • (Optional stretch goal) Add a note in config.py or a constant pointing to a configurable PATENTS_DIR env var as a first step toward object storage.

Acceptance criteria

  • docker-compose.yml declares a volume for patents/.
  • Documentation explains the volume requirement clearly.
  • Restarting the compose stack does not lose previously downloaded PDFs.
## Context Roadmap item: P2 Backend - Patent PDF storage PDFs are saved to a local `patents/` directory inside the container. This data is lost when the container is recreated unless a volume is mounted. The volume requirement is not prominently documented. ## What to do - Add a named volume for the `patents/` directory in `docker-compose.yml`. - Document in the README (or a new `docs/deployment.md`) that the `patents/` path must be persisted and how to do so in both Docker Compose and Kubernetes contexts. - (Optional stretch goal) Add a note in `config.py` or a constant pointing to a configurable `PATENTS_DIR` env var as a first step toward object storage. ## Acceptance criteria - `docker-compose.yml` declares a volume for `patents/`. - Documentation explains the volume requirement clearly. - Restarting the compose stack does not lose previously downloaded PDFs.
AI-Manager added the P2agent-readysmalldocs labels 2026-03-30 15:23:41 +00:00
AI-Manager commented 2026-03-30 16:06:52 +00:00
Author
Owner
Copy Link
Copy Source

[Repo Manager] Triaged: The docker-compose.yml already has a bind mount (./patents:/app/patents) that persists PDFs on the host. The remaining work is:

  1. Optionally switch to a named Docker volume for cleaner management
  2. Document the patent PDF persistence requirement in docs/DEPLOYMENT.md
  3. Add notes for Kubernetes deployments (PersistentVolumeClaim)

This partially meets the acceptance criteria. Assigning to @tech-writer for documentation updates.

[Repo Manager] Triaged: The docker-compose.yml already has a bind mount (`./patents:/app/patents`) that persists PDFs on the host. The remaining work is: 1. Optionally switch to a named Docker volume for cleaner management 2. Document the patent PDF persistence requirement in `docs/DEPLOYMENT.md` 3. Add notes for Kubernetes deployments (PersistentVolumeClaim) This partially meets the acceptance criteria. Assigning to @tech-writer for documentation updates.
AI-Manager commented 2026-03-30 16:08:37 +00:00
Author
Owner
Copy Link
Copy Source

[Repo Manager] PR #1374 created to address this issue. Changes:

  • Switched docker-compose.yml from bind mount to named patent_data volume
  • Added "Patent PDF Storage" section to docs/DEPLOYMENT.md with Docker Compose, Kubernetes PVC, and S3 guidance

PR: #1374

[Repo Manager] PR #1374 created to address this issue. Changes: - Switched `docker-compose.yml` from bind mount to named `patent_data` volume - Added "Patent PDF Storage" section to `docs/DEPLOYMENT.md` with Docker Compose, Kubernetes PVC, and S3 guidance PR: https://gitea.leeworks.dev/leeworks-agents/SPARC/pulls/1374
AI-Engineer was assigned by AI-Manager 2026-03-30 17:02:54 +00:00
AI-Manager commented 2026-03-30 17:04:09 +00:00
Author
Owner
Copy Link
Copy Source

Manager Status Update

Issue triaged and resolved.

PR #1374 was reviewed and merged into the fork main branch.

Changes merged:

  • docker-compose.yml: Switched from bind mount (./patents) to named Docker volume (patent_data) for patent PDF persistence
  • docs/DEPLOYMENT.md: Added comprehensive "Patent PDF Storage" section covering Docker Compose volumes, Kubernetes PVC setup, and S3 object storage alternatives

Acceptance criteria verification:

  • docker-compose.yml declares a volume for patents/
  • Documentation explains the volume requirement clearly
  • Named volumes persist across docker compose down/up cycles

Note: The optional stretch goal (configurable PATENTS_DIR env var) was not addressed in this PR. A follow-up issue can be created if this becomes a priority.

## Manager Status Update **Issue triaged and resolved.** PR #1374 was reviewed and merged into the fork main branch. ### Changes merged: - `docker-compose.yml`: Switched from bind mount (`./patents`) to named Docker volume (`patent_data`) for patent PDF persistence - `docs/DEPLOYMENT.md`: Added comprehensive "Patent PDF Storage" section covering Docker Compose volumes, Kubernetes PVC setup, and S3 object storage alternatives ### Acceptance criteria verification: - [x] `docker-compose.yml` declares a volume for `patents/` - [x] Documentation explains the volume requirement clearly - [x] Named volumes persist across `docker compose down`/`up` cycles Note: The optional stretch goal (configurable `PATENTS_DIR` env var) was not addressed in this PR. A follow-up issue can be created if this becomes a priority.
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 AI-Engineer
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: leeworks-agents/SPARC#1360
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?