Repository navigation
docs(interactive): Trusted Host V2 security boundary and native Grok acceptance design - #14
Merged
Merged
Conversation
…submit tool event ordering
…g independent gates
…ox and reconciliation
… principal isolation
… principal separation
…nd handle inheritance
…al contract traceability
…event proof before loop
…optional transport profile
…rate PoC from NAT-14
…n deployed profile
…hecklist ambiguity
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.
Why
Phase 1 of the approved Interactive full-delivery roadmap is merged into main at
9bac8f86347a405efe068f96423ae3e501005061. The real Grok Build acceptance report showed live-model participation but no Windows Host isolation or C12/S13 native source attestation. We must freeze the trust model before touching Core.Design artifacts
Proposed architecture
Review and safety gates
Design PR only. No production Core/SDK/Program ABI/schema changes; no executable Host implementation; no release. This PR cannot pass Windows/Grok native security by CI alone. P0/P1 threats are enumerated with practical negative tests. Marking the PR green does not close C12/S13, C08 or M6.
Required review
Merge policy: Keep Draft until independent security review and exact-HEAD documentation CI. Do not automatically begin Phase 3 production security implementation on a merely green docs PR.
Exact-HEAD CI and review evidence
d1c7c76d03780f04a5b761699cdbc9ac82f0b86c.Important limitation: these checks confirm existing implementation regression and document compatibility, NOT Windows secure Host ACL tests, Grok Build native source provenance or Phase 3/4 implementation.
External independent security review is still required to approve the architecture before Phase 3.
External Grok F-01–F-07 review reconciliation (2026-11-11)
FlowKit-PR14-Trusted-Host-V2-Independent-Security-Review-Grok.md, SHA-2563d4f14298b5d347d5deb7f34ce9256e79bade70d28965cfa85a94453108293fb; verdict CONDITIONAL PASS against frozen commitd1c7c76d03780f04a5b761699cdbc9ac82f0b86c.--toolsregistering arbitrary JS.toolInputDigestandreceiptHashnow have distinct meanings. Nativetool_calltiming/large-input binding not yet demonstrated.receiptHashbound to proofId for safe expiry handling.Changes are documentation-only, not security implementation. PR stays Draft; do not merge until focused independent re-review of revisions and recovery of formal contract originals. Exact-HEAD CI is not Windows ACL/native Grok security evidence.
Second external Grok re-review — 2026-11-11
Reviewed frozen commit
51a759975abcaee928faf50aec99ebed324fdfccwithFlowKit-PR14-Trusted-Host-V2-F01-F07-Targeted-Rereview-Grok.md(uploaded SHA-2564528278eddb1ade36c8a388c4ad4181f93f8d364a323ca54b0a4c19ea4ee3062). Verdict: CONDITIONAL PASS.H3-05depended indirectly onH3-02and Phase 4 NAT-14; corrected toH3-05 ← H3-01only for OS-only tests. Separate minimal pre-implementation Grok tool-call capability probe from formal Phase 4 NAT-14. H3-04 completes active-ingress/signer tests before full Phase 3 exit.SEC-05still tests the actual candidate ingress;SEC-07may beN/A—NO_LISTENERonly if no TLS endpoint exists.Updated docs-only head records the independent second review and remediation. PR remains Draft until targeted diff recheck and formal requirements provenance resolution. No FlowKit Core, SDK, JS Program ABI, SQLite schema or M5 receipt V1 changes; no main merge or release. Existing scripted CI cannot pass Windows native OS isolation or C12/S13.
Latest exact-HEAD Phase 2 review patch
316e426aa8d96e4a6b6bee8afd35c0cd313fd244: Actions run 43045141302 7/7 SUCCESS.SEC-05still tests actual model-result ingress;SEC-07becomesN/A—NO_LISTENERonly when that TLS endpoint genuinely does not exist.Third-round final review — F-01 and F-R2-01 CLOSED
FlowKit-PR14-F01-FR2-01-Final-Targeted-Review-Grok.md; SHA-2567936d6cef9d3c26f5870eb2303fe52c6f215487e740c1fa57787e06716c67933. It independently reviewed frozen HEAD316e426aa8d96e4a6b6bee8afd35c0cd313fd244and marked F-01 and F-R2-01 CLOSED at design-document level.b9d0c1efe17c386f798e30b4331f7bd90a1936e3. Exact-HEAD CI: run 43096045021 (pending at time of update).Note
Low Risk
Documentation and traceability updates only; no runtime, authentication, or data-handling code paths are modified.
Overview
Documentation-only Phase 2 design record for Trusted Host V2 and native Grok Build bridging—no Core, SDK, or test code changes.
The delivery roadmap is revised for post–Phase 1 main (
9bac8f8), merged PR #13 / closed PR #7, and Phase 2 CONDITIONAL status after three rounds of Grok review (F-01/F-R2-01 closed at doc level; F-02/F-04 partial; F-07 formal V0.3/V0.6-R5 conformance still BLOCKED). Phase 3/6 text now spells out optional TLS/mTLS/listeners (NO_LISTENER), H3-05 OS-only tests depending only on H3-01, and that Grok tool evidence is not required before OS isolation work.Four new draft artifacts freeze the intended security posture without implementing it: deployment design (MODEL_USER vs HOST_USER TCB, Host-only tool observation, Ed25519 receipt v1 unchanged), 30-threat matrix, SEC-01–19 / NAT-01–15 acceptance tiers (P/M/T/N/V), and a review ledger reconciling DR-01–15 and F-01–F-07. Cross-cutting rules added in docs include rejecting default Node Named Pipe as secure IPC, mandatory validation of actual model ingress paths, sign-before-
tool_call_updatetiming (NAT-14), and Host-only OBSERVED/SIGNED/ACKED outbox recovery.Merge is framed as an explicitly limited design record; it does not authorize Phase 3 implementation or M6/C12/S13 pass.
Reviewed by Cursor Bugbot for commit 42961d8. Bugbot is set up for automated code reviews on this repo. Configure here.