fix: Add tr-worker files, sanitize IPs, update validation checklists
- Add realtime_qdrant_watcher.py and mem-qdrant-watcher.service to tr-worker/ - Sanitize private IPs (10.0.0.x → <QDRANT_IP>, <OLLAMA_IP>) - Replace absolute paths with placeholders - Add GIT_VALIDATION_CHECK.md for security validation - Update validation checklists to v2.4 - Remove session.md from git (local-only file)
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
# TrueRecall v2 - Master Audit Checklist (LOCAL)
|
||||
# TrueRecall v2 - Master Audit Checklist (GIT)
|
||||
|
||||
**For:** `.local_projects/true-recall-v2/` (Working/Development Directory)
|
||||
**For:** `.git_projects/true-recall-v2/` (Git Repository - Sanitized)
|
||||
**Version:** 2.2
|
||||
**Last Updated:** 2026-02-25 10:07 CST
|
||||
|
||||
@@ -8,7 +8,12 @@
|
||||
|
||||
## Overview
|
||||
|
||||
This checklist validates the **local working directory** with real IPs, paths, and credentials. Use this for development, debugging, and local testing.
|
||||
This checklist validates the **git repository** where all private IPs, absolute paths, and credentials have been sanitized. Use this before pushing to public repositories.
|
||||
|
||||
**Related Files:**
|
||||
- `GIT_VALIDATION_CHECK.md` - Comprehensive git validation checklist
|
||||
- `LOCAL_VALIDATION_CHECK.md` - Local dev validation (in `.local_projects/`)
|
||||
- `VALIDATION_NOTES.md` - Auto-generated validation findings
|
||||
|
||||
---
|
||||
|
||||
@@ -22,8 +27,22 @@ This checklist validates the **local working directory** with real IPs, paths, a
|
||||
| Watcher stuck on old session | ✅ **Fixed 12:22** | Restarted watcher service |
|
||||
| Plugin capture 0 exchanges | ✅ **Fixed 12:34** | Added `extractMessageText()` for array content |
|
||||
| Plugin exchanges working | ✅ **Verified 12:41** | 9 exchanges extracted per session |
|
||||
| **HTML comments in UI** | ✅ **Fixed 14:02** | Changed `formatRelevantMemoriesContext` to clean text format |
|
||||
| **prependContext vs systemPrompt** | ✅ **Fixed 14:02** | Changed hook return from `prependContext` to `systemPrompt` for hidden injection |
|
||||
| **TypeScript source not updated** | ✅ **Fixed 14:02** | Updated `.ts` file, not just compiled `.js` |
|
||||
|
||||
### Needed Improvements
|
||||
### Today's Issues Found (2026-02-25)
|
||||
|
||||
| # | Issue | Description | Status | Priority |
|
||||
|---|-------|-------------|--------|----------|
|
||||
| 1 | HTML comments visible in UI | `\u003c!-- relevant-memories-start --\u003e` blocks showing in chat | ✅ **FIXED** | High |
|
||||
| 2 | Memory injection format | Was using HTML comment format, now clean "Memory Injection:" text | ✅ **FIXED** | High |
|
||||
| 3 | prependContext vs systemPrompt | Plugin was using `prependContext` (visible in user message) instead of `systemPrompt` (hidden in system prompt) | ✅ **FIXED** | High |
|
||||
| 4 | TypeScript source not updated | OpenClaw compiles from `.ts`, was editing `.js` only | ✅ **FIXED** | High |
|
||||
| 5 | Gateway restart issues | kill/killall not working reliably | ✅ **FIXED** | Medium |
|
||||
| 6 | **README needs update** | TrueRecall v2 is standalone, not addon to Jarvis Memory | ✅ **FIXED** | Medium |
|
||||
|
||||
### Needed Improvements (Carryover)
|
||||
|
||||
| Issue | Description | Priority |
|
||||
|-------|-------------|----------|
|
||||
@@ -263,6 +282,47 @@ This checklist validates the **local working directory** with real IPs, paths, a
|
||||
|
||||
---
|
||||
|
||||
## 10. Recent Fixes to Verify (2026-02-25)
|
||||
|
||||
### Plugin Memory Format Fix
|
||||
**Status:** ✅ **FIXED**
|
||||
|
||||
**Summary:**
|
||||
- Changed `formatRelevantMemoriesContext` from HTML comment format to clean text
|
||||
- Changed hook return from `prependContext` to `systemPrompt` (hides from UI)
|
||||
- Updated both TypeScript source (`.ts`) and compiled JavaScript (`.js`)
|
||||
|
||||
**Files Modified:**
|
||||
- `<OPENCLAW_PATH>/extensions/memory-qdrant/index.ts`
|
||||
- `<OPENCLAW_PATH>/extensions/memory-qdrant/index.js`
|
||||
|
||||
**What Changed:**
|
||||
```typescript
|
||||
// Before:
|
||||
return `<!-- relevant-memories-start -->
|
||||
<relevant-memories>...`;
|
||||
return { prependContext: formatRelevantMemoriesContext(...) };
|
||||
|
||||
// After:
|
||||
return `Memory Injection: Historical context from previous conversations:
|
||||
1. [category] text`;
|
||||
return { systemPrompt: formatRelevantMemoriesContext(...) };
|
||||
```
|
||||
|
||||
**Verification Checklist:**
|
||||
- [ ] Send test message - memories appear as "Memory Injection:" not HTML
|
||||
- [ ] No `<!-- -->` tags visible in chat
|
||||
- [ ] Gateway restarted after changes
|
||||
|
||||
### Pending Updates
|
||||
|
||||
| # | Item | Description | Status |
|
||||
|---|------|-------------|--------|
|
||||
| 1 | README update | Clarify v2 is standalone, not addon | ✅ **FIXED** |
|
||||
| 2 | Comparison table | Update v2 vs Jarvis vs v1 | ✅ **FIXED** |
|
||||
|
||||
---
|
||||
|
||||
## Sign-Off
|
||||
|
||||
| Role | Name | Date | Signature |
|
||||
|
||||
Reference in New Issue
Block a user