feat: exec output limits, maintain health check, store provenance#5
Merged
Conversation
Mirrors Dependabot PR #4 so the dependency bump and feature work share one CI run.
timorunge
force-pushed
the
feat/maintenance-and-exec-hardening
branch
from
86101ef to
87732ba
Compare
- exec: add `max_output_bytes` (default 10 MiB); stream stdout/stderr through bounded reads and kill the child process on timeout - cli: `lore maintain health` reports config validity, compiled features, LLM client init, and store reachability - store: stamp chunk-config fingerprint + version in lore.meta and warn at ingest when an existing store was built with different settings - store: federated multi-store search with BM25 score normalization - llm: surface missing-credential warnings via the ingest observer - test: pure chunker boundary cases (empty, short, min-chunk, indices) - chore(lints): enforce missing_errors_doc/missing_panics_doc and document the public API surface
timorunge
force-pushed
the
feat/maintenance-and-exec-hardening
branch
from
87732ba to
673cf7f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Combines the pending dependency bump with feature/hardening work so they share a single CI run. Supersedes #4 (the dependency updates are included here).
Dependencies
allgroup across 1 directory with 8 updates (same set as build(deps): bump the all group across 1 directory with 8 updates #4).Features & hardening
max_output_bytes(default 10 MiB); stdout/stderr are streamed through bounded reads and the child process is killed on timeout — preventing unbounded memory use and orphanedshprocesses.lore maintain health: one-screen report covering config validity, compiled features, LLM client initialization, and store reachability.lore.meta; warn at ingest when an existing store was built with different settings.Tests & lints
missing_errors_doc/missing_panics_doc; documented the public API surface.Test plan
cargo fmt --checkcargo clippy --all-targets -- -D warnings(all-features, no-default-features, default)cargo doc -D warningscargo test --all-features— 172 passing