-
Notifications
You must be signed in to change notification settings - Fork 0
Align desk docs with C++ Alexandria #131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,14 @@ | ||
| # Secure local document ingestion | ||
|
|
||
| This adapter extracts explicitly supplied local files, rejects likely credential | ||
| material, and sends one strict JSON document per file to the Go Memory Store. It | ||
| does not run inside the privileged kernel and never writes MongoDB directly. | ||
| material, and sends one strict JSON document per file to `memory-store.exe` | ||
| (desk C++ Release, else Go rollback). It does not run inside the privileged | ||
|
Owner
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [bug] The new parenthetical claims the adapter sends JSON to Suggestion: Either restore the Go-default wording (and keep |
||
| kernel and never writes MongoDB directly. | ||
|
|
||
| Build the Memory Store first, set `MONGODB_URI`, then ingest UTF-8 text: | ||
|
|
||
| ```powershell | ||
| Push-Location godbrain_core\memory_store | ||
| go build -o memory-store.exe .\cmd\memory-store | ||
| Pop-Location | ||
| .\scripts\build_pipeline.ps1 | ||
| python -m godbrain_core.local_ingestion --source-label research .\notes.md | ||
|
|
||
| ``` | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[suggestion] Setup now lists mongo-c-driver at
C:\Tools\mongo-c-driveras a hard prerequisite, which matchesCMakeLists.txt's defaultMONGOC_ROOT. Without that driver,build_pipeline.ps1still configures, butrag-service/rag-rebuildare omitted (protocol-only), so the later.\build\cpp_memory_store\Release\rag-rebuild.exestep fails. The repo already hasscripts\Fetch-MongoCDriver.ps1for that prefix; this setup page never names it.Suggestion: Point the prerequisite at
.\scripts\Fetch-MongoCDriver.ps1(or a short "if missing, run …" under section 3). Keep theC:\Tools\mongo-c-driverpath; that is the CMake default.