Skip to content

feat(ai): add verified local model artifact store - #106

Merged
McanKul merged 2 commits into
McanKul:epic/local-ai-runtimefrom
nonamexishere:feat/83-model-store
Sep 11, 2026
Merged

McanKul merged 2 commits into
McanKul:epic/local-ai-runtimefrom
nonamexishere:feat/83-model-store

Conversation

@nonamexishere

Copy link
Copy Markdown
Contributor

Summary

Adds a content-addressed local model store next to the #81 Fake adapter. Install is path / reader only: stage → SHA-256 verify → commit. Partial or truncated files never appear ready. The same bytes imported twice share one blob. Remove deletes that model’s files. No HTTP, no catalog, no auto-download, no Tauri commands.

Fixes #83.

Why

Parent #76 needs verified on-disk artifacts before Settings (#84) can install or load a model. Checksum identity keeps disk use honest and interrupted installs out of list_ready.

Validation

  • cargo test --manifest-path src-tauri/Cargo.toml --lib store_ (9 passed)
  • cargo test --manifest-path src-tauri/Cargo.toml --lib ai_ (20 passed, Fake keep-green)
  • npm test / npm run typecheck / cargo test --lib

Privacy Checklist

  • This keeps OffPDF usable offline.
  • This does not upload, log, or transmit user files.
  • New dependencies or bundled binaries have compatible licenses. (sha2 0.10, MIT/Apache-2.0)

Content-addressed blobs under an injected root. SHA-256 must match
before a model is ready. Interrupted staging and truncated files stay
unlistable. Duplicate imports reuse one blob. No HTTP, no auto-download,
no Tauri commands.
Rename-over the dest manifest instead of deleting it first. The no-GGUF
walk ignores gitignored bundle trees so local prepare scripts do not fail cargo test.

@McanKul McanKul left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks — the artifact store is bounded, checksum-verified, atomic at the ready-manifest boundary, and remains fully offline.

@McanKul
McanKul merged commit 61b3dd0 into McanKul:epic/local-ai-runtime Sep 11, 2026
2 checks passed
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.

2 participants