Skip to content

Fix runtime and consolidate Document Manager - #13

Merged
JayRHa merged 1 commit into
mainfrom
agent/consolidate-document-manager
Jul 27, 2026
Merged

JayRHa merged 1 commit into
mainfrom
agent/consolidate-document-manager

Conversation

@JayRHa

@JayRHa JayRHa commented Jul 27, 2026

Copy link
Copy Markdown
Owner

What changed

  • declare DocumentManager as the canonical repository
  • add a safe .env.example and Docker context exclusions
  • replace shell evaluation of .env with Docker --env-file
  • remove the unused second ChromaDB/supervisor process
  • align backup paths and fix typed settings access
  • harden tar backup extraction and local bind defaults
  • remove the unused sentence-transformers/PyTorch dependency chain
  • update security-sensitive dependency pins
  • add regression tests for configuration, backup extraction, and runtime layout

Why

DocumentManagement and DocumentManager had diverged. The maintained repository also contained duplicated Chroma storage, inconsistent backup paths, unsafe environment loading, and obsolete runtime dependencies. This change makes DocumentManager the single maintained implementation before the legacy repository is archived.

Verification

  • 14 passed
  • bash -n setup.sh docker-entrypoint.sh
  • production Docker image built successfully
  • isolated container returned HTTP 200 from /api/health
  • runtime contained one Uvicorn process and /app/data/backups

@JayRHa
JayRHa force-pushed the agent/consolidate-document-manager branch from 44d54df to 84fee60 Compare July 27, 2026 05:30
@JayRHa
JayRHa marked this pull request as ready for review July 27, 2026 05:30
@JayRHa
JayRHa merged commit e18091c into main Jul 27, 2026
1 check passed
@JayRHa
JayRHa deleted the agent/consolidate-document-manager branch July 27, 2026 05:31
@JayRHa JayRHa mentioned this pull request Jul 27, 2026
Closed
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.

2 participants