Repository navigation
feat(kc): add optional local Compose integration (SHD-1075) - #33
Merged
Merged
Conversation
janosmurai
marked this pull request as ready for review
October 15, 2026 13:59
janosmurai
force-pushed
the
SHD-1075/optional-kc-compose
branch
from
October 18, 2026 17:13
6820f75 to
db246c1
Compare
janosmurai
force-pushed
the
SHD-1073/upload-lifecycle
branch
from
October 18, 2026 17:13
e34bc01 to
a0257ec
Compare
Signed-off-by: janosmurai <janos.murai@axem.dev>
janosmurai
force-pushed
the
SHD-1075/optional-kc-compose
branch
from
October 19, 2026 09:51
db246c1 to
73844ce
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
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.
Type of change
Related issue
SHD-1075
Description
Adds an explicitly selected local Compose overlay for the optional Knowledge Center. The server remains the public ingress and owns authentication/metadata; KC exclusively owns RustFS operations and storage credentials. Ordinary Compose behavior is unchanged when the overlay is absent.
Adds protected, exclusive credential-file setup and explicit account/model/readiness helpers. Authenticated operator requests reject redirects, and readiness enforces supported registered-model dimensions. Persistent server, object and vector state uses named volumes; the notification relay accepts only its fixed route.
Validation
cargo +1.98.1 fmt --checkcargo +1.98.1 clippy --offline --locked --all-targets --all-features -- -D warningscargo +1.98.1 build --offline --locked --verbose --all-featurescargo +1.98.1 test --offline --locked --verbose --all-featuresIndependent helper/config verification passed 27 tests, including actual local HTTP redirect rejection, dimension validation, exclusive file creation and optional/no-KC configuration. Independent source review passed after the scoped corrections.
The corrected independent named-volume runtime suite passed 87 assertions: public PDF/DOCX upload through KC, signed caller-scoped MCP citations, exact object/vector persistence across down/up, deletion with dependency readback, and preservation of an unrelated uploaded file. Cleanup and unchanged protected services were verified.
Runtime verification used compatible cached images with source-matched native binary mounts and an explicitly synthetic embedding double. It does not establish packaged-image startup, operator-provider compatibility, backup/restore or migration. Standard local gates passed on unchanged source; the ordinary parallel Rust suite passed 180 tests with one inherited transport diagnostic ignored.
Deployment and compatibility
Opt-in only; requires Docker Compose 2.24.4+, locally available compatible images, protected credentials and a registered embedding provider.
Documentation
Adds operator instructions for setup, explicit account/model registration, trust boundaries, liveness versus readiness, private image prerequisites and volume-preserving operation. Links the optional overlay from the main README.
Checklist
Signed-off-bytrailer (git commit -s).AI assistance: