Skip to content

Add Milestone 4 document retrieval - #1

Merged
KanuTomer merged 2 commits into
mainfrom
codex/milestone-4-document-retrieval
Jul 13, 2026
Merged

Add Milestone 4 document retrieval#1
KanuTomer merged 2 commits into
mainfrom
codex/milestone-4-document-retrieval

Conversation

@KanuTomer

Copy link
Copy Markdown
Owner

Summary

  • records the completed, budget-bounded Milestone 3 acceptance
  • adds deterministic PPTX, PDF, Markdown, and UTF-8 text ingestion
  • adds transactional SQLite FTS5 indexing, top-five bounded retrieval, local search/inspection UI, and strict citation validation
  • adds the 50-case offline M4 evaluator, packaged-Electron FTS5 smoke, and Windows artifact upload
  • preserves stock NSIS behavior while using app-builder-lib's static uninstaller extractor so enforced Smart App Control is not weakened

Validation

  • npm ci (patch applies cleanly)
  • npm run verify: 99/99 Vitest and 7/7 .NET tests
  • npm audit --audit-level=high: 0 vulnerabilities
  • npm run eval:m4: 50/50 top-five recall
  • npm run test:helper-smoke: 16 kHz mono WASAPI capture, temp file deleted
  • npm run package:win: unsigned NSIS installer generated
  • staged diff, credential, local-path, binary, generated-artifact, and redaction scans passed

The packaged executable smoke remains a required Windows CI gate because this development host's Smart App Control blocks launching all unsigned executables. No OpenAI request or API credit was used for M4. No public release or code signing is included.

@KanuTomer
KanuTomer force-pushed the codex/milestone-4-document-retrieval branch from ddfb6c4 to f0b2125 Compare July 13, 2026 19:01
@KanuTomer
KanuTomer force-pushed the codex/milestone-4-document-retrieval branch from f0b2125 to 5994f0d Compare July 13, 2026 19:06
@KanuTomer
KanuTomer marked this pull request as ready for review July 13, 2026 19:10
@KanuTomer
KanuTomer merged commit 3628466 into main Jul 13, 2026
1 check passed
@KanuTomer
KanuTomer deleted the codex/milestone-4-document-retrieval branch July 13, 2026 19:10
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.

1 participant