Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update L2 Testnet Tutorial to Use Latest Releases #1304

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

krofax
Copy link
Collaborator

@krofax krofax commented Jan 30, 2025

Description

Summary

This PR updates the L2 Testnet Tutorial to ensure that users download and use the latest released versions of op-node, op-batcher, and op-proposer from the Optimism releases page, instead of relying on the outdated tutorials/chain branch.

Changes Made

  • Added instructions on how to find and download the latest releases of required components.
  • Updated build steps to guide users on how to extract and install these components properly.
  • Included a warning callout to inform users that the tutorials/chain branch contains outdated components.
  • Provided additional clarity on ensuring compatibility with the latest OP Stack versions.

Tests

Additional context

Metadata

Copy link

netlify bot commented Jan 30, 2025

Deploy Preview for docs-optimism ready!

Name Link
🔨 Latest commit 08d864e
🔍 Latest deploy log https://app.netlify.com/sites/docs-optimism/deploys/67a4d8d009bf640008c33796
😎 Deploy Preview https://deploy-preview-1304--docs-optimism.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@krofax krofax marked this pull request as ready for review January 31, 2025 14:54
@krofax krofax requested a review from a team as a code owner January 31, 2025 14:54
@krofax krofax changed the title Update l2 testnet release instructions Update L2 Testnet Tutorial to Use Latest Releases Jan 31, 2025
Copy link
Contributor

coderabbitai bot commented Jan 31, 2025

📝 Walkthrough

Walkthrough

This update revises the "Creating Your Own L2 Rollup Testnet" tutorial. The document now shifts the approach from building the OP Stack chain from source code to using pre-built binaries from the latest Optimism release. Detailed instructions for downloading, verifying (including checksum verification), and extracting the required binaries for op-node, op-batcher, and op-proposer have been added. The formatting has been improved with additional headings and clearer step-by-step guidance, ensuring users are informed about managing dependencies, verifying file integrity, and avoiding issues with outdated components. No changes were made to exported or public entities.

Sequence Diagram(s)

sequenceDiagram
    participant U as User
    participant T as Tutorial Page
    participant R as Releases Page

    U->>T: Open L2 Rollup Tutorial
    T->>U: Display updated instructions for pre-built binaries
    U->>R: Download latest op-node, op-batcher, op-proposer binaries
    U->>U: Verify binaries using checksum instructions
    U->>U: Extract binaries and prepare environment for OP Stack testnet setup
Loading

Possibly related issues

Possibly related PRs

Suggested labels

tutorial

Suggested reviewers

  • sbvegan

Tip

🌐 Web search-backed reviews and chat
  • We have enabled web search-based reviews and chat for all users. This feature allows CodeRabbit to access the latest documentation and information on the web.
  • You can disable this feature by setting web_search: false in the knowledge_base settings.
  • Please share any feedback in the Discord discussion.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🧹 Nitpick comments (2)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (2)

89-90: Improve clarity of binary usage explanation.

The explanation of using pre-built binaries could be more specific about the benefits.

-You're going to be spinning up your OP Stack chain using pre-built binaries from the latest Optimism release instead of building directly from source. 
-This simplifies the setup process and ensures you're using the latest stable version.
+You'll be using pre-built binaries from the latest Optimism release instead of building from source. 
+This approach simplifies setup, ensures stability, and reduces compilation time while providing the latest stable version.

170-172: Add error handling guidance for version checks.

The version verification commands should include expected output and troubleshooting steps.

-op-node --version
-op-batcher --version
-op-proposer --version
+# Verify each component's version matches the downloaded release
+op-node --version     # Should match the downloaded release version
+op-batcher --version  # Should match the downloaded release version
+op-proposer --version # Should match the downloaded release version
+
+# If any command fails, ensure:
+# 1. The binary is correctly extracted
+# 2. The binary has execute permissions
+# 3. The binary is in your system PATH
📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between db5b6a8 and a2e4533.

📒 Files selected for processing (1)
  • pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (3 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (1)

Pattern **/*.mdx: "ALWAYS review Markdown content THOROUGHLY with the following criteria:

  • Use proper nouns in place of personal pronouns like 'We' and 'Our' to maintain consistency in communal documentation.
  • Avoid gender-specific language and use the imperative form.
  • Monitor capitalization for emphasis. Avoid using all caps, italics, or bold for emphasis.
  • Ensure proper nouns are capitalized in sentences.
  • Apply the Oxford comma.
  • Use proper title case for buttons, tab names, page names, and links. Sentence case should be used for body content and short phrases, even in links.
  • Use correct spelling and grammar at all times (IMPORTANT).
  • For H1, H2, and H3 headers:
    1. Use sentence case, capitalizing only the first word.
    2. Preserve the capitalization of proper nouns, technical terms, and acronyms as defined in the 'nouns.txt' file located in the root directory of the project.
    3. Do not automatically lowercase words that appear in the 'nouns.txt' file, regardless of their position in the header.
  • Flag any headers that seem to inconsistently apply these rules for manual review.
  • When reviewing capitalization, always refer to the 'nouns.txt' file for the correct capitalization of proper nouns and technical terms specific to the project.
    "
⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: Redirect rules - docs-optimism
  • GitHub Check: Header rules - docs-optimism
  • GitHub Check: Pages changed - docs-optimism
  • GitHub Check: pr-workflow

krofax and others added 3 commits January 31, 2025 16:00
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@krofax krofax requested review from mslipper and sbvegan January 31, 2025 15:07
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🧹 Nitpick comments (2)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (2)

12-12: Improve document structure and clarity.

The tutorial's organization could be enhanced:

  1. The title should maintain proper capitalization of "L2" and "OP Stack" as technical terms
  2. The build section's introduction (lines 89-90) should be a separate callout for better visibility

Apply these changes:

-# Creating your own L2 rollup testnet
+# Creating Your Own L2 Rollup Testnet

-You're going to be spinning up your OP Stack chain using pre-built binaries from the latest Optimism release instead of building directly from source.
-This simplifies the setup process and ensures you're using the latest stable version.
+<Callout type="info">
+  You're going to be spinning up your OP Stack chain using pre-built binaries from the latest Optimism release instead of building directly from source.
+  This simplifies the setup process and ensures you're using the latest stable version.
+</Callout>

Also applies to: 89-90


19-20: Improve writing style and grammar.

Several style improvements are needed for better readability and professionalism.

Apply these changes:

-This tutorial is **designed for developers** who want to learn
+This tutorial is **intended for developers** who want to learn

-teach you all of the components
+teach you all the components

-To learn about the different components please
+To learn about the different components, please

-configuration file inside of the
+configuration file inside the

-blocks being created inside of
+blocks being created inside

Also applies to: 23-23, 389-389, 644-644

🧰 Tools
🪛 LanguageTool

[style] ~19-~19: Consider using an alternative to strengthen your wording.
Context: ...rial is designed for developers who want to learn about the OP Stack by spinning...

(WANT_KEEN)


[style] ~20-~20: Consider removing “of” to be more concise
Context: ...e full deployment process and teach you all of the components that make up the OP Stack, a...

(ALL_OF_THE)


[style] ~20-~20: Try using a synonym here to elevate your writing.
Context: ...nd teach you all of the components that make up the OP Stack, and **you'll end up with ...

(CONSTITUTE_COMPRISE)

📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3cc152a and be17ee2.

📒 Files selected for processing (2)
  • pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (14 hunks)
  • words.txt (0 hunks)
💤 Files with no reviewable changes (1)
  • words.txt
🧰 Additional context used
📓 Path-based instructions (1)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (1)

Pattern **/*.mdx: "ALWAYS review Markdown content THOROUGHLY with the following criteria:

  • Use proper nouns in place of personal pronouns like 'We' and 'Our' to maintain consistency in communal documentation.
  • Avoid gender-specific language and use the imperative form.
  • Monitor capitalization for emphasis. Avoid using all caps, italics, or bold for emphasis.
  • Ensure proper nouns are capitalized in sentences.
  • Apply the Oxford comma.
  • Use proper title case for buttons, tab names, page names, and links. Sentence case should be used for body content and short phrases, even in links.
  • Use correct spelling and grammar at all times (IMPORTANT).
  • For H1, H2, and H3 headers:
    1. Use sentence case, capitalizing only the first word.
    2. Preserve the capitalization of proper nouns, technical terms, and acronyms as defined in the 'nouns.txt' file located in the root directory of the project.
    3. Do not automatically lowercase words that appear in the 'nouns.txt' file, regardless of their position in the header.
  • Flag any headers that seem to inconsistently apply these rules for manual review.
  • When reviewing capitalization, always refer to the 'nouns.txt' file for the correct capitalization of proper nouns and technical terms specific to the project.
    "
🪛 LanguageTool
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx

[style] ~19-~19: Consider using an alternative to strengthen your wording.
Context: ...rial is designed for developers who want to learn about the OP Stack by spinning...

(WANT_KEEN)


[style] ~20-~20: Consider removing “of” to be more concise
Context: ...e full deployment process and teach you all of the components that make up the OP Stack, a...

(ALL_OF_THE)


[style] ~20-~20: Try using a synonym here to elevate your writing.
Context: ...nd teach you all of the components that make up the OP Stack, and **you'll end up with ...

(CONSTITUTE_COMPRISE)


[typographical] ~23-~23: Consider adding a comma here.
Context: ... To learn about the different components please read the [deployment overview page](/bu...

(PLEASE_COMMA)


[uncategorized] ~27-~27: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...r own needs. **The OP Stack is free and open source software licensed entirely under the MI...

(EN_COMPOUND_ADJECTIVE_INTERNAL)


[style] ~389-~389: This phrase is redundant. Consider using “inside”.
Context: ...etting-started.jsonconfiguration file inside of thedeploy-config` directory. ```ba...

(OUTSIDE_OF)


[uncategorized] ~422-~422: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...69 then the contract has been deployed and you can safely skip this section. {<...

(COMMA_COMPOUND_SENTENCE)


[style] ~644-~644: This phrase is redundant. Consider using “inside”.
Context: ...ou'll start to see blocks being created inside of op-geth. ...

(OUTSIDE_OF)

🪛 Gitleaks (8.21.2)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx

287-287: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


291-291: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


295-295: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


299-299: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: Redirect rules - docs-optimism
  • GitHub Check: Header rules - docs-optimism
  • GitHub Check: Pages changed - docs-optimism
  • GitHub Check: pr-workflow

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (9)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (9)

15-16: Informative callout text.
The callout instructing users to “be prepared to set aside approximately one hour” is helpful. Consider verifying that the emphasis is necessary and the wording reads smoothly.


24-24: Link clarity suggestion.
The prompt “read the deployment overview page” is clear; however, adding a comma after “please” may improve readability.


70-71: Direnv configuration callout improvement.
The callout explaining how to modify your shell configuration (e.g. “modify your shell configuration file (like ~/.bashrc or ~/.zshrc)”) is useful. Consider adding a comma after “file” for enhanced readability.


94-96: Work directory callout.
The instructions regarding using the home directory (i.e. ~/) are clear. For improved readability, consider splitting the second sentence or adding a brief pause (e.g. a comma) between ideas.


351-386: Loading environment variables via direnv.
The steps to load the environment (including the direnv allow command and its accompanying callouts) are clear. Consider revising sentence structure in the callouts (for example, adding commas where needed) to improve readability per style recommendations.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~364-~364: Possible missing comma found.
Context: ...](#configuring-direnv). Next you'll need to allow direnv to read t...

(AI_HYDRA_LEO_MISSING_COMMA)


457-473: Deploying the factory contract.
The long cast publish command is clearly documented; however, consider splitting it over multiple lines in the rendered documentation to improve readability.


525-532: JWT authentication key generation.
The command using openssl rand -hex 32 > jwt.txt is simple and understandable. Consider including a brief note on why this JWT is critical for the secure communication between clients.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~527-~527: Possible missing comma found.
Context: ...3>Create an authentication key} Next you'll create a [JSON Web Token](https:...

(AI_HYDRA_LEO_MISSING_COMMA)


639-688: Running op-node explanation.
The instructions for launching op-node—including the configuration options and callouts regarding peer-to-peer synchronization—are comprehensive. You might emphasize that the --p2p.disable flag is used for initial testnet setup and can be reconfigured once the network scales.

🧰 Tools
🪛 LanguageTool

[style] ~669-~669: This phrase is redundant. Consider using “inside”.
Context: ...ou'll start to see blocks being created inside of op-geth. ...

(OUTSIDE_OF)


19-20: Overall style and punctuation enhancements.
Static analysis has flagged several style improvements across the document:
• Consider alternative phrasings in lines 19–20 to remove redundancy (e.g. “designed for developers” might be reworded slightly for clarity).
• Insert a comma after introductory phrases in line 23 and review punctuation in compound sentences (see lines 335 and 447).
• Replace phrases like “inside of” with “inside” (as noted around lines 414 and 669).
• Ensure consistency in punctuation (for example, adding a comma in line 527).
These refinements will improve the document’s overall readability.

Also applies to: 23-23, 335-335, 364-364, 414-414, 447-447, 527-527, 669-669

🧰 Tools
🪛 LanguageTool

[style] ~19-~19: Consider using an alternative to strengthen your wording.
Context: ...rial is designed for developers who want to learn about the OP Stack by spinning...

(WANT_KEEN)


[style] ~20-~20: Consider removing “of” to be more concise
Context: ...e full deployment process and teach you all of the components that make up the OP Stack, a...

(ALL_OF_THE)


[style] ~20-~20: Try using a synonym here to elevate your writing.
Context: ...nd teach you all of the components that make up the OP Stack, and **you'll end up with ...

(CONSTITUTE_COMPRISE)

📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between be17ee2 and 3ee141c.

📒 Files selected for processing (1)
  • pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (14 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
`**/*.mdx`: "ALWAYS review Markdown content THOROUGHLY with ...

**/*.mdx: "ALWAYS review Markdown content THOROUGHLY with the following criteria:

  • Use proper nouns in place of personal pronouns like 'We' and 'Our' to maintain consistency in communal documentation.
  • Avoid gender-specific language and use the imperative form.
  • Monitor capitalization for emphasis. Avoid using all caps, italics, or bold for emphasis.
  • Ensure proper nouns are capitalized in sentences.
  • Apply the Oxford comma.
  • Use proper title case for buttons, tab names, page names, and links. Sentence case should be used for body content and short phrases, even in links.
  • Use correct spelling and grammar at all times (IMPORTANT).
  • For H1, H2, and H3 headers:
    1. Use sentence case, capitalizing only the first word.
    2. Preserve the capitalization of proper nouns, technical terms, and acronyms as defined in the 'nouns.txt' file located in the root directory of the project.
    3. Do not automatically lowercase words that appear in the 'nouns.txt' file, regardless of their position in the header.
  • Flag any headers that seem to inconsistently apply these rules for manual review.
  • When reviewing capitalization, always refer to the 'nouns.txt' file for the correct capitalization of proper nouns and technical terms specific to the project.
    "
  • pages/builders/chain-operators/tutorials/create-l2-rollup.mdx
🪛 LanguageTool
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx

[style] ~19-~19: Consider using an alternative to strengthen your wording.
Context: ...rial is designed for developers who want to learn about the OP Stack by spinning...

(WANT_KEEN)


[style] ~20-~20: Consider removing “of” to be more concise
Context: ...e full deployment process and teach you all of the components that make up the OP Stack, a...

(ALL_OF_THE)


[style] ~20-~20: Try using a synonym here to elevate your writing.
Context: ...nd teach you all of the components that make up the OP Stack, and **you'll end up with ...

(CONSTITUTE_COMPRISE)


[typographical] ~23-~23: Consider adding a comma here.
Context: ... To learn about the different components please read the [deployment overview page](/bu...

(PLEASE_COMMA)


[uncategorized] ~27-~27: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...r own needs. **The OP Stack is free and open source software licensed entirely under the MI...

(EN_COMPOUND_ADJECTIVE_INTERNAL)


[uncategorized] ~335-~335: Possible missing comma found.
Context: ...need to send any ETH to the Sequencer address as it does not send transactions.** ...

(AI_HYDRA_LEO_MISSING_COMMA)


[uncategorized] ~364-~364: Possible missing comma found.
Context: ...](#configuring-direnv). Next you'll need to allow direnv to read t...

(AI_HYDRA_LEO_MISSING_COMMA)


[style] ~414-~414: This phrase is redundant. Consider using “inside”.
Context: ...etting-started.jsonconfiguration file inside of thedeploy-config` directory. ```ba...

(OUTSIDE_OF)


[uncategorized] ~447-~447: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...69 then the contract has been deployed and you can safely skip this section. {<...

(COMMA_COMPOUND_SENTENCE)


[uncategorized] ~527-~527: Possible missing comma found.
Context: ...3>Create an authentication key} Next you'll create a [JSON Web Token](https:...

(AI_HYDRA_LEO_MISSING_COMMA)


[style] ~669-~669: This phrase is redundant. Consider using “inside”.
Context: ...ou'll start to see blocks being created inside of op-geth. ...

(OUTSIDE_OF)

🪛 Gitleaks (8.21.2)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx

312-312: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


316-316: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


320-320: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


324-324: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: Redirect rules - docs-optimism
  • GitHub Check: Header rules - docs-optimism
  • GitHub Check: Pages changed - docs-optimism
  • GitHub Check: pr-workflow
🔇 Additional comments (17)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (17)

12-12: Header update is clear.
The new level‑1 header “# Creating your own L2 rollup testnet” is concise and matches the document’s intent.


59-59: Section header consistency.
The updated header “#### direnv” is clear and maintains consistency with the other dependency sections.


89-90: Explanation of pre‑built binaries.
The revised lines clarifying that users will be spinning up their OP Stack chain using pre‑built binaries are clear and benefit the reader by highlighting process simplification.


102-120: Repository cloning steps look solid.
The stepwise instructions covering the clone, branch creation, and package file setup are methodical and easy to follow.


121-209: Comprehensive package configuration.
The new package.json content is detailed and well formatted. Please verify that version specifications (for example, "nx-cloud": "latest") are intentional. Pinning versions where possible might improve production stability.


229-247: op-geth cloning and build instructions.
The steps to clone op-geth and to build it (make geth) are clear and correctly sequenced.


254-274: Environment variables setup is clear.
The instructions—and accompanying table—for filling out environment variables (e.g. L1_RPC_URL and L1_RPC_KIND) read clearly. Ensure that the table renders properly on all devices.


394-424: Network configuration instructions.
The new steps for entering the Optimism monorepo, moving into the contracts-bedrock package, installing Foundry dependencies, generating the configuration file, and reviewing it are detailed and instructive.

🧰 Tools
🪛 LanguageTool

[style] ~414-~414: This phrase is redundant. Consider using “inside”.
Context: ...etting-started.jsonconfiguration file inside of thedeploy-config` directory. ```ba...

(OUTSIDE_OF)


432-448: Create2 factory deployment (optional) improvement.
The optional section on deploying the Create2 factory is clear. It might be useful to emphasize when this step is necessary versus when it can be skipped.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~447-~447: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...69 then the contract has been deployed and you can safely skip this section. {<...

(COMMA_COMPOUND_SENTENCE)


481-493: Deploy L1 contracts instructions.
The deployment instructions for the L1 contracts, along with the helpful note about potential EvmError: Revert issues related to IMPL_SALT, are well explained.


506-523: Genesis and rollup files generation.
The steps to create the genesis.json and rollup.json files are clear and precise. Double-check that all relative paths work correctly in varied setups.


535-542: Copying genesis files.
The instructions to copy the generated genesis.json and jwt.txt into the op-geth directory are concise and clear.


553-575: op-geth initialization process.
The step-by-step guide to navigate to op-geth, create the data directory, build the binary, and initialize the node is well documented.


584-629: op-geth startup command.
The detailed geth run command with all its flags is thorough. It is advisable to verify that all file paths, such as to jwt.txt, and networking parameters (e.g. --networkid=42069) are suitable for the intended tutorial environment.


701-728: op-batcher execution steps.
The commands and explanations for running op-batcher are clear. The detailed description of the --max-channel-duration setting aids in understanding its implications for cost and synchronization.


743-764: op-proposer operational instructions.
The stepwise guide to start op-proposer is clear, and the command format is consistent with the rest of the tutorial. Ensure that any references to environment variables (like $GS_PROPOSER_PRIVATE_KEY) remain secure and are documented as test values if applicable.


778-795: Getting ETH on your chain steps.
The instructions for obtaining Sepolia ETH using the L1StandardBridge are concise and actionable. Consider adding a troubleshooting tip if users experience delays in ETH appearing on L2.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (6)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (6)

15-16: Clear and engaging information callout
The callout text asking users to set aside approximately one hour is concise and helpful. Consider, however, whether a slightly more active tone (e.g., "Allocate approximately one hour…") might improve clarity.


22-24: Punctuation for better readability
In the sentence spanning lines 22–24—specifically before the directive "read the deployment overview page"—adding a comma after "components" could help clarify the instruction. For example:

"To learn about the different components, please read the [deployment overview page]…"

🧰 Tools
🪛 LanguageTool

[typographical] ~23-~23: Consider adding a comma here.
Context: ... To learn about the different components please read the [deployment overview page](/bu...

(PLEASE_COMMA)


19-21: Improve sentence structure and voice
The description in lines 19–21 is informative but can be made more direct. For instance, consider rephrasing to:

"This tutorial targets developers who wish to explore the OP Stack by deploying a testnet chain. It guides you through the deployment process and culminates in the creation of your own OP Stack testnet."
This revision uses an active voice and streamlines the explanation.

🧰 Tools
🪛 LanguageTool

[style] ~19-~19: Consider using an alternative to strengthen your wording.
Context: ...rial is designed for developers who want to learn about the OP Stack by spinning...

(WANT_KEEN)


[style] ~20-~20: Consider removing “of” to be more concise
Context: ...e full deployment process and teach you all of the components that make up the OP Stack, a...

(ALL_OF_THE)


[style] ~20-~20: Try using a synonym here to elevate your writing.
Context: ...nd teach you all of the components that make up the OP Stack, and **you'll end up with ...

(CONSTITUTE_COMPRISE)


414-414: Refine phrasing for conciseness
In line 414, the phrase "inside of the deploy-config directory" can be streamlined to "inside the deploy-config directory" for brevity and clarity.

🧰 Tools
🪛 LanguageTool

[style] ~414-~414: This phrase is redundant. Consider using “inside”.
Context: ...etting-started.jsonconfiguration file inside of thedeploy-config` directory. ```ba...

(OUTSIDE_OF)


442-447: Punctuation for clarity in instructions
In the explanation regarding the Create2 factory (lines 442–447), consider adding a comma before "and" in the sentence "If the command returns 69 then the contract has been deployed and you can safely skip this section." This will help delineate the independent clauses for improved readability.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~447-~447: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...69 then the contract has been deployed and you can safely skip this section. {<...

(COMMA_COMPOUND_SENTENCE)


669-669: Concise language suggestion for directory reference
In the op-geth section, the phrase "inside of op-geth" (line 669) could be more concise if changed to "inside op-geth".

🧰 Tools
🪛 LanguageTool

[style] ~669-~669: This phrase is redundant. Consider using “inside”.
Context: ...ou'll start to see blocks being created inside of op-geth. ...

(OUTSIDE_OF)

📜 Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3ee141c and 08d864e.

📒 Files selected for processing (1)
  • pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (14 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
`**/*.mdx`: "ALWAYS review Markdown content THOROUGHLY with ...

**/*.mdx: "ALWAYS review Markdown content THOROUGHLY with the following criteria:

  • Use proper nouns in place of personal pronouns like 'We' and 'Our' to maintain consistency in communal documentation.
  • Avoid gender-specific language and use the imperative form.
  • Monitor capitalization for emphasis. Avoid using all caps, italics, or bold for emphasis.
  • Ensure proper nouns are capitalized in sentences.
  • Apply the Oxford comma.
  • Use proper title case for buttons, tab names, page names, and links. Sentence case should be used for body content and short phrases, even in links.
  • Use correct spelling and grammar at all times (IMPORTANT).
  • For H1, H2, and H3 headers:
    1. Use sentence case, capitalizing only the first word.
    2. Preserve the capitalization of proper nouns, technical terms, and acronyms as defined in the 'nouns.txt' file located in the root directory of the project.
    3. Do not automatically lowercase words that appear in the 'nouns.txt' file, regardless of their position in the header.
  • Flag any headers that seem to inconsistently apply these rules for manual review.
  • When reviewing capitalization, always refer to the 'nouns.txt' file for the correct capitalization of proper nouns and technical terms specific to the project.
    "
  • pages/builders/chain-operators/tutorials/create-l2-rollup.mdx
🪛 LanguageTool
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx

[style] ~19-~19: Consider using an alternative to strengthen your wording.
Context: ...rial is designed for developers who want to learn about the OP Stack by spinning...

(WANT_KEEN)


[style] ~20-~20: Consider removing “of” to be more concise
Context: ...e full deployment process and teach you all of the components that make up the OP Stack, a...

(ALL_OF_THE)


[style] ~20-~20: Try using a synonym here to elevate your writing.
Context: ...nd teach you all of the components that make up the OP Stack, and **you'll end up with ...

(CONSTITUTE_COMPRISE)


[typographical] ~23-~23: Consider adding a comma here.
Context: ... To learn about the different components please read the [deployment overview page](/bu...

(PLEASE_COMMA)


[uncategorized] ~27-~27: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...r own needs. **The OP Stack is free and open source software licensed entirely under the MI...

(EN_COMPOUND_ADJECTIVE_INTERNAL)


[style] ~414-~414: This phrase is redundant. Consider using “inside”.
Context: ...etting-started.jsonconfiguration file inside of thedeploy-config` directory. ```ba...

(OUTSIDE_OF)


[uncategorized] ~447-~447: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...69 then the contract has been deployed and you can safely skip this section. {<...

(COMMA_COMPOUND_SENTENCE)


[style] ~669-~669: This phrase is redundant. Consider using “inside”.
Context: ...ou'll start to see blocks being created inside of op-geth. ...

(OUTSIDE_OF)

🪛 Gitleaks (8.21.2)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx

312-312: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


316-316: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


320-320: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


324-324: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: Redirect rules - docs-optimism
  • GitHub Check: Header rules - docs-optimism
  • GitHub Check: Pages changed - docs-optimism
  • GitHub Check: pr-workflow
🔇 Additional comments (2)
pages/builders/chain-operators/tutorials/create-l2-rollup.mdx (2)

12-12: Redundant H1 header after front matter
The document’s front matter already defines the title. Double-check whether the repeated H1 ("Creating your own L2 rollup testnet") is intentional as part of the design or if it might be redundant.


89-90: Clear update on build process
The modified text in lines 89–90 effectively communicates the shift from building from source to using pre-built binaries from the latest Optimism release. The simplified setup process is clearly explained.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants