dan/memory
1
0
Fork 0
mirror of https://github.com/mruwnik/memory.git synced 2026-01-02 17:22:58 +01:00
Code Issues 1 Packages Projects Releases Wiki Activity

Continue bug verification in INVESTIGATION.md

Browse Source 
Additional bugs verified as fixed:
- BUG-017: collection_name index exists ✅
- BUG-020: server_id index exists ✅
- BUG-039: Email folder errors handled gracefully ✅
- BUG-041: N/A - reasonable behavior (backup disabled without key)

Updated bug count to 30+ confirmed fixed

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
mruwnik 2025-12-19 20:45:43 +00:00
parent c2be97aad5
commit 74e26e1892
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
INVESTIGATION.md
View File

@ -5,7 +5,7 @@
- **Last Updated:** 2025-12-19 (Third Pass - Verification) - **Last Updated:** 2025-12-19 (Third Pass - Verification)
- **Status:** Ongoing - **Status:** Ongoing
- **Total Issues Found:** 100+ (original) + 10 new critical issues - **Total Issues Found:** 100+ (original) + 10 new critical issues
- **Bugs Fixed:** 25+ confirmed fixed - **Bugs Fixed:** 30+ confirmed fixed
--- ---
@ -307,10 +307,10 @@ Based on git history analysis, the following bugs have been FIXED:
## Medium Severity Bugs ## Medium Severity Bugs
### Data Layer ### Data Layer
- BUG-017: Missing `collection_name` index on Chunk table (`source_item.py:165-168`) - BUG-017: ✅ Missing `collection_name` index - FIXED (Index exists at source_item.py:168)
- BUG-018: AgentObservation dead code for future embedding types (`source_items.py:1005-1028`) - BUG-018: AgentObservation dead code for future embedding types (`source_items.py:1005-1028`)
- BUG-019: ✅ Embed status never set to STORED after push - FIXED (properly sets STORED at lines 169, 245) - BUG-019: ✅ Embed status never set to STORED after push - FIXED (properly sets STORED at lines 169, 245)
- BUG-020: Missing server_id index on DiscordMessage (`source_items.py:426-435`) - BUG-020: ✅ Missing server_id index on DiscordMessage - FIXED (Index exists at source_items.py:428-432)
### Content Processing ### Content Processing
- BUG-021: No chunk validation after break_chunk (`embedding.py:49-58`) - BUG-021: No chunk validation after break_chunk (`embedding.py:49-58`)
@ -337,11 +337,11 @@ Based on git history analysis, the following bugs have been FIXED:
- BUG-036: Database integrity errors not properly handled (`discord.py:310-321`) - BUG-036: Database integrity errors not properly handled (`discord.py:310-321`)
- BUG-037: ✅ Timezone bug in scheduled calls - FIXED (properly converts to UTC and strips tzinfo for DB comparison) - BUG-037: ✅ Timezone bug in scheduled calls - FIXED (properly converts to UTC and strips tzinfo for DB comparison)
- BUG-038: Beat schedule not thread-safe for distributed deployment (`ingest.py:19-56`) - BUG-038: Beat schedule not thread-safe for distributed deployment (`ingest.py:19-56`)
- BUG-039: Email sync fails entire account on single folder error (`email.py:84-152`) - BUG-039: ✅ Email sync fails entire account on single folder error - FIXED (process_folder has own try-except, continues to next folder)
### Infrastructure ### Infrastructure
- BUG-040: ✅ Missing resource limits for postgres, redis, qdrant, api - FIXED in BUG-067 - BUG-040: ✅ Missing resource limits for postgres, redis, qdrant, api - FIXED in BUG-067
- BUG-041: Backup encryption silently disabled if key missing (`settings.py:215-216`) - BUG-041: N/A Backup encryption silently disabled - actually reasonable (S3_BACKUP_ENABLED=False when no key)
- BUG-042: Restore scripts don't validate database integrity (`restore_databases.sh:79`) - BUG-042: Restore scripts don't validate database integrity (`restore_databases.sh:79`)
- BUG-043: ✅ Health check doesn't check dependencies - FIXED (now checks database and Qdrant connections) - BUG-043: ✅ Health check doesn't check dependencies - FIXED (now checks database and Qdrant connections)
- BUG-044: Uvicorn trusts all proxy headers (`docker/api/Dockerfile:63`) - BUG-044: Uvicorn trusts all proxy headers (`docker/api/Dockerfile:63`)