docs: reflect SafeMini source integration - #54
Merged
JustinJLeopard merged 1 commit intoAug 10, 2026
Conversation
Replace stale scaffold-only language with the merged opt-in SafeMini adapter boundary. Keep installation, productive runtime, and release claims explicitly out of scope. Co-Authored-By: Codex <noreply@openai.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Updates public-facing documentation to accurately reflect that the SafeMini substrate now exists as a separate repository and that JustAi integrates with it via an explicit, opt-in source adapter (without implying installation, runtime activation, or a release).
Changes:
- Refreshes top-level repo status wording to describe SafeMini as an existing separate repo consumed via an opt-in adapter.
- Updates the demo README to describe the Runner Protocol as a compatibility layer and to document the SafeMini adapter mapping.
- Revises
docs/ARCHITECTURE.mdto reflect current “source integration” state (separate repo + pinned Git revision) while explicitly avoiding product/runtime/release claims.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| README.md | Rewords status bullets to reflect SafeMini as an existing separate repo used via an explicit opt-in adapter. |
| justai-demo-readme/README.md | Updates component and phase descriptions to document the SafeMini adapter and clarify “source integration only”. |
| docs/ARCHITECTURE.md | Updates architecture and sequencing language to reflect the separate SafeMini repo and explicit adapter, with clear non-claims. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
JustinJLeopard
deleted the
desktop-codex/docs/justai-safe-mini-current-state
branch
August 10, 2026 13:40
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.
Summary
Correct public architecture wording that still said SafeMini did not exist or was only a future scaffold.
Validation
git diff --checkPYTHONPATH=/home/justai/projects/safe-mini-worktrees/justai-executor-factory PYTHONDONTWRITEBYTECODE=1 /usr/bin/python3 -B -m pytest -q -p no:cacheprovider(626 passed, 14 subtests passed)Non-goals
No runtime routing, package installation, service activation, release/tag, or product claim changes.