Deep Dive rewrite: align every page to the current architecture - #43
Draft
haimbj1 wants to merge 51 commits into
Draft
Deep Dive rewrite: align every page to the current architecture#43haimbj1 wants to merge 51 commits into
haimbj1 wants to merge 51 commits into
Conversation
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
haimbj1
force-pushed
the
docs/deep-dive-rewrite
branch
from
August 23, 2026 14:19
255d316 to
b6c72b9
Compare
haimbj1
force-pushed
the
docs/deep-dive-rewrite
branch
2 times, most recently
from
August 24, 2026 10:42
a751e61 to
b8c7afb
Compare
… replaces threshold-network content
… and current SDK signatures
…tecture.svg with Mermaid
…and commitment leg, replace Transactions.svg with Mermaid
…s and the ZK Verifier, replace Encrypt-a-value.svg with Mermaid
…heck and auditability, horizontal state diagram
…able guards) and readable diagram sizing
…th contract-bound proof (ACP upgrade)
… batch proof (ACP upgrade)
…batch signature (ACP upgrade)
… revocation (ACP upgrade)
…es one (ACP upgrade)
…st use (ACP upgrade)
…digest (ACP upgrade)
…er custody and attested release
…nt links from teecryptor and zk-verifier
…-coded flows, CT Server as the database gate
… diagram, partners beside the TEEs
…ves, horizontal service chain at the bottom
haimbj1
force-pushed
the
docs/deep-dive-rewrite
branch
from
August 27, 2026 09:13
b32efdf to
0137fe1
Compare
guya-fhenix
reviewed
Aug 27, 2026
|
|
||
| ## The offchain services | ||
|
|
||
| - **[Client SDK](/client-sdk/introduction/overview)** (`@cofhe/sdk`): the TypeScript library applications use to encrypt inputs, manage [permits](/client-sdk/guides/permits), and decrypt outputs. |
There was a problem hiding this comment.
I don't think all the services, even with a deep dive, are relevant to users.
When we give the users the option to run cofhe by themselves, it will be relevant, but not now.
We do need to keep the things they interact with, or big blocks for the overview.
The list IMO needs to match the diagram suggestion in the comment above.
Contributor
Author
There was a problem hiding this comment.
I wrote a response on the diagram suggestion. Let's focus the discussion there on the scope of the overview and the deep-dive docs. If we drop the services from the overview, how do we elaborate on them in the other sections? That would mean removing the other pages as well, right?
…t-state ZK Verifier bullet, drop flow word
…lues before signing the batch approval
… is before why, remove private-repo links
… stages into one pipeline page
… redirect both to fhe-engine
…E Engine as one component
…replace per-service steps and queues
…e, not a named service
…red result processor and threshold network rows
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
2 participants
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.
Rewrites the whole Deep Dive tab to match the deployed system. One commit per page so pages can be cherry-picked individually.
Do not merge before the ACP cutover (cofhe #898) is deployed — the last seven commits document the ACP upgrade (contracts v0.2.0: batch input verification, contract-bound proofs, permits as ACP), verified against the exact submodule pin 2412a67b.
What changed:
Every page passes Vale and lint-docs with zero errors and zero warnings.
Draft on purpose: review per commit, cherry-pick what you like. The Mintlify preview shows the rendered result.
🤖 Generated with Claude Code
https://claude.ai/code/session_018LwPCqvYddjyX527UiVQSo