mirror of
https://github.com/Crosstalk-Solutions/project-nomad.git
synced 2026-10-17 05:14:19 +02:00
32e2c3d10e
ZimService.registerLocalUpload() dispatched EmbedFileJob unconditionally, gated only on whether Ollama was reachable. It never read rag.defaultIngestPolicy, so a user who deliberately chose Manual still got sideloaded ZIMs embedded into the knowledge base behind their back. PR #919 fixed exactly this for the post-download dispatch path. The local-upload path was missed. Rather than inline a third copy of the Always/Manual conditional, this reuses decideScanAction, the same helper the scanner uses. That also means an existing browse_only or pending_decision row is now honored instead of being overridden by the act of re-uploading the file, which the inline version in run_download_job.ts does not do. Unset policy is still treated as Always, so existing installs keep their current behavior. Reported by @just-jbc on #1119, which also proposed disabling ZIM auto-discovery entirely. That larger change is not included here: turning discovery off by default would mean a user who downloads Wikipedia through the curated flow gets no AI answers from it and no explanation why. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| app | ||
| bin | ||
| commands | ||
| config | ||
| constants | ||
| database | ||
| docs | ||
| inertia | ||
| providers | ||
| public | ||
| resources | ||
| scripts | ||
| start | ||
| tests | ||
| types | ||
| util | ||
| views | ||
| .editorconfig | ||
| .env.example | ||
| ace.js | ||
| adonisrc.ts | ||
| eslint.config.js | ||
| package-lock.json | ||
| package.json | ||
| tailwind.config.ts | ||
| tsconfig.json | ||
| vite.config.ts | ||