Skip to content

Conversation

@tiljrd
Copy link

@tiljrd tiljrd commented Sep 10, 2025

User description

Org rename: LZeroAnalytics → 0xBloctopus

Summary

Updates GitHub organization references from "LZeroAnalytics" to "0xBloctopus" in the ethereum-package repository as part of the broader organizational rename. Changes include:

  • kurtosis.yml: Updated package name from github.com/LZeroAnalytics/ethereum-package to github.com/0xBloctopus/ethereum-package
  • main.star: Updated blockscout-package import from github.com/LZeroAnalytics/blockscout-package to github.com/0xBloctopus/blockscout-package
  • README.md: Updated all Kurtosis run commands and GitHub links to use new org name (ethereum-faucet, uniswap-package references)

Review & Testing Checklist for Human

  • Verify GitHub org rename is complete - Confirm that github.com/0xBloctopus/ethereum-package actually exists and is accessible
  • Test package functionality - Run kurtosis run --enclave test github.com/0xBloctopus/ethereum-package to verify the package loads correctly with new org name
  • Check dependent package renames - Verify that blockscout-package, ethereum-faucet, and uniswap-package have also been renamed to 0xBloctopus org
  • Search for missed references - Double-check for any remaining "LZeroAnalytics" references in files that might have been missed

Notes

This is part of a coordinated organization rename across multiple repositories. The functionality should remain identical - only the GitHub organization references are changing.

Link to Devin run: https://app.devin.ai/sessions/c902258a44f043638d91d9f06ccef179
Requested by: Til Jordan (@tiljrd)


PR Type

Other


Description

  • Update GitHub organization references from LZeroAnalytics to 0xBloctopus

  • Change package name in kurtosis.yml configuration

  • Update blockscout-package import path in main.star

  • Modify README documentation links and commands


Diagram Walkthrough

flowchart LR
  A["LZeroAnalytics org"] -- "rename to" --> B["0xBloctopus org"]
  B --> C["kurtosis.yml package name"]
  B --> D["main.star import path"]
  B --> E["README.md links & commands"]
Loading

File Walkthrough

Relevant files
Documentation
README.md
Update documentation links and commands                                   

README.md

  • Updated GitHub organization references in documentation links
  • Changed kurtosis run commands to use new org name
  • Modified faucet and uniswap package documentation URLs
+5/-5     
Configuration changes
kurtosis.yml
Update package name configuration                                               

kurtosis.yml

  • Changed package name from github.com/LZeroAnalytics/ethereum-package
    to github.com/0xBloctopus/ethereum-package
+1/-1     
Dependencies
main.star
Update blockscout package import path                                       

main.star

  • Updated blockscout-package import path from LZeroAnalytics to
    0xBloctopus organization
+1/-1     

@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@qodo-code-review
Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 Security concerns

Sensitive information exposure:
The README includes a concrete-looking private key in the faucet example. Even if it’s for documentation only, ensure it’s a clearly fake placeholder (e.g., "<YOUR_FAUCET_PRIVATE_KEY>") and not present in any example that tooling might accidentally consume. Also verify no automation ingests README snippets into runtime configs.

⚡ Recommended focus areas for review

Sensitive Example Key

The README includes a hardcoded private key in the faucet params example. Even if illustrative, ensure it's clearly marked as an example and not used anywhere in code or CI. Consider replacing with a placeholder and adding a warning.

faucet_params:
  # Private key for faucet account - needs to be prefunded
  private_key: bcdf20249abf0ed6d944c0288fad489e33f66b3960d9e6229c1cd214ed3bbe31
Import Path Validity

The updated import path for the blockscout package must exist under the new org and support the expected version string interpolation. Validate that the module path and tags/branches under 0xBloctopus mirror the old structure.

blockscout = import_module("github.com/0xBloctopus/blockscout-package@{}/main.star".format(env))
"""Launches an arbitrarily complex ethereum testnet based on the arguments provided

@qodo-code-review
Copy link

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
Make import path configurable

Avoid hard-coding the organization in the import path; make it configurable via
args with a sensible default. This preserves compatibility during the org
migration and prevents hard failures if the new repository isn't available in
some environments.

main.star [75]

-blockscout = import_module("github.com/0xBloctopus/blockscout-package@{}/main.star".format(env))
+blockscout_org = args.get("blockscout_org", "0xBloctopus")
+blockscout_path = "github.com/{}/blockscout-package@{}/main.star".format(blockscout_org, env)
+blockscout = import_module(blockscout_path)
  • Apply / Chat
Suggestion importance[1-10]: 6

__

Why: The suggestion correctly proposes making the hardcoded organization name configurable via arguments, which improves the package's flexibility and maintainability.

Low
  • More

@tiljrd tiljrd merged commit f587bba into main Sep 11, 2025
1 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants