Review fixes for the Bridging vs. NAC section (architecture.md) - #450
Merged
lthms merged 3 commits intoAug 19, 2026
Merged
Conversation
- Michelson runtime exposes the Tezos RPC, not JSON-RPC (matches the interface table above) - L1 node: Etherlink is the Smart Rollup, L1 hosts it; anchor the whole chain to L1 with a single edge instead of per-runtime edges - sequence diagram note: the atomicity unit is the transaction, not the block - comparison table: native tez is not wrapped by the L1<->L2 bridge; withdrawal latency is days, not hours - cosmetic: stray colons/spacing in mermaid labels, 'a unified execution layer', 'economic space' Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
@saroupille is attempting to deploy a commit to the Trili Tech Team on Vercel. A member of the Team first needs to authorize it. |
The 13-August/slow-governance line was a placeholder. What actually happened: the slow-governance proposal of 2 August was withdrawn before activation after a security vulnerability was found during continued testing (Mainnet was never exposed), and a patched kernel was resubmitted via fast governance on 19 August, with the upgrade triggered on 20 August. Also: 'Native Atomic Calls' -> 'Native Atomic Composability (NAC)' (terminology used everywhere else in the docs), and a link to the resubmission Agora post. Note: the 20 August date assumes the trigger happens as planned on Thursday morning; re-verify against the actual trigger_kernel_upgrade operation before merging. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- accounts-and-aliases: 'the the' typo - fees: quantify the EIP-7702 DA-fee term (125 bytes per authorization, matching fees.rs) - terminology: 'Native Atomic Calls' -> 'Native Atomic Composability (NAC)' (bridging.md, network-information.mdx, testnet.mdx) - testnet.mdx: Previewnet is long-running, not a 'temporary testnet' (same fix previewnet.md already received) Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Merged
lthms
reviewed
Aug 19, 2026
| @@ -42,5 +42,5 @@ Examples of using test networks are on other pages in this documentation. | |||
|
|
|||
| ## Etherlink{/* TX */} Previewnet | |||
Collaborator
There was a problem hiding this comment.
Strictly speaking it's called Tezos X Previewnet and i don't think there is a need to “downcast” it to Etherlink Previewnet? or at least no decision was made about that
Contributor
Author
|
Heads-up @lthms: this PR was merged at 09:57 UTC, but a 4th commit ( |
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.
Addresses the open review thread on #436 asking for a look at the new "Bridging vs. NAC" section. Targets the
import-tx-docbranch.Fixes:
🤖 Generated with Claude Code