Alert fix 3471#314
Conversation
* Doc Updates: Yet another round (EVM / VM / Other) (ethereumjs#3999) * EVM README updates * VM README updates * Minor update * More README doc updates * Update packages/evm/README.md * Update packages/vm/README.md --------- Co-authored-by: Gabriel Rocheleau <contact@rockwaterweb.com> * monorepo: clean up more test data (ethereumjs#4001) * monorepo: clean up more test data * chore: linting * vm: fix example * util: prefixed hex string type improvements (ethereumjs#3995) * chore: revert startsWith0x * util: fix hexToBytes in usage * util: remoe redundant byte checking * chore: more type adjustments * format: linting * monorepo: more type issues * util: undo remove undefined * chore: more type fixes * client: remove typecasting * client: simplify typecasting * common: remove typecasting * chore: remove unused import * chore: address review comments * chore: remove unused var * chore: strictEqual * monorepo: npm audit fix (ethereumjs#4003) * EVM: cleanup error messages and fix styling (ethereumjs#3994) * evm/vm: use constant as string to ref for `EVMError` * Remove unused EOFError key-values * Remove unused SimpleErrors * Remove unused EVMErrorMessages elements * Rename EvmErrorResult to EVMErrorResult * Rename Evm to EVM in comments and strings * Make EVMErrorMessages a static field in EVMError --------- Co-authored-by: Amir <indigophi@protonmail.com> * fix: ethash test script (ethereumjs#4007) * util: remove undefined handling from bytesToHex (ethereumjs#4004) * util: document and remove undefined handling from bytesToHex * util: remove typecasting * util: refactor account handling * chore: simplify handling * util: deprecate account constructor and update docs * vm: fix event test * client: fix client tests * lint: remove console olgs * client: fix more client tests * lint: remove empty block * client: remove unnecessary optional chaining * client: remove it.only * feat(block): Add CLRequests test, example, and documentation (ethereumjs#4008) * feat(block): add CLRequests test, example, and documentation * fix(block): update CLRequests examples to use bytesToHex * chore: specify Node.js 20 in .nvmrc * util: replace unnecessary toBytes usage (ethereumjs#4014) * util: replace some toBytes usage * client: more hexToBytes * chore: remove more toBytes * chore: remove toBytes usage * lint: remove unused imports * chore: remove unused import * chore: remove unused import * Docs Cleanup / Cautious Restructuring / README ToCs (ethereumjs#4010) * A somewhat more useful EVM EIP activation example (now with 7702 being active by default) * Same for VM * Add prominent v10 README header additions * Add README ToC, eventually restructure (binarytree) * Add README ToC, eventually restructure (block) * Add README ToC, eventually restructure (blockchain) * Add README ToC, eventually restructure (common) * Add README ToC, eventually restructure (devp2p) * Add README ToC, eventually restructure (era) * Add README ToC, eventually restructure (ethash) * Some EVM README section reordering * Add README ToC, eventually restructure (EVM) * Add README ToC, eventually restructure (genesis) * Add README ToC, eventually restructure (mpt) * Add README ToC, eventually restructure (rlp) * Add README ToC, eventually restructure (statemanager) * Add README ToC, eventually restructure (tx) * Add sub-ToC for tx types * Add README ToC, eventually restructure (util) * Add README ToC, eventually restructure (verkle) * Add README ToC, eventually restructure (vm) * Add README ToC, eventually restructure (wallet) * Undo robot nonsense * evm: upgrade noble curves to 1.9.0 (ethereumjs#4018) --------- Co-authored-by: Holger Drewes <Holger.Drewes@gmail.com> Co-authored-by: Gabriel Rocheleau <contact@rockwaterweb.com> Co-authored-by: Jochem Brouwer <jochembrouwer96@gmail.com> Co-authored-by: Amir <indigophi@protonmail.com> Co-authored-by: avdhesh.eth <49278246+avdheshcharjan@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
This reverts commit 20cae2b.
Bumps the npm_and_yarn group with 1 update in the / directory: [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite). Updates `vite` from 6.2.4 to 6.3.3 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.3.3/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 6.3.3 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>
Add .circleci/config.yml
…_yarn-9e12ea7c2e build(deps-dev): bump vite from 6.2.4 to 6.3.3 in the npm_and_yarn group across 1 directory
pull config bug build
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Update docker-image.yml
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Create mdbook.yml
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Create nextjs.yml
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Signed-off-by: AU_gdev_19 <64915515+Dargon789@users.noreply.github.com>
Create devcontainer.json
…ry (#120) Bumps the npm_and_yarn group with 1 update in the / directory: [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite). Updates `vite` from 6.3.3 to 6.3.4 - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v6.3.4/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 6.3.4 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com> Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
|
|
Reviewer's GuideRefactors CI/workflow configuration for the VM jobs, adjusts Docker image build config, standardizes GitHub issue templates, and introduces several new sample CI/Pages deployment workflows and devcontainer scaffolding. Flow diagram for simplified vm-pr GitHub Actions workflowflowchart TD
A["pull_request opened_reopened_synchronize"] --> B["Start_VM_workflow"]
subgraph "Job_vm-api"
B --> C["actions_checkout_submodules_recursive"]
C --> D["actions_setup-node_version_18_cache_npm"]
D --> E["npm_ci_root"]
E --> F["npm_run_lint"]
F --> G["npm_run_coverage"]
G --> H["codecov_action_v3_upload_lcov"]
end
subgraph Job_vm-state_matrix
B --> I["checkout_with_submodules"]
I --> J["setup-node_18_cache_npm"]
J --> K["npm_ci_root"]
K --> L["npm_run_test_state_with_fork_matrix"]
end
subgraph "Job_vm-state-extended_matrix"
B --> M["checkout_with_submodules"]
M --> N["setup-node_18_cache_npm"]
N --> O["npm_ci_root"]
O --> P["npm_run_test_state_with_extended_fork_matrix"]
end
subgraph "Job_vm-blockchain_matrix"
B --> Q["checkout_with_submodules"]
Q --> R["setup-node_18_cache_npm"]
R --> S["npm_ci_root"]
S --> T["npm_run_test_blockchain_with_args_matrix"]
end
subgraph "Job_vm-benchmarks"
B --> U["checkout"]
U --> V["setup-node_18_cache_npm"]
V --> W["npm_ci_root"]
W --> X["npm_run_build_benchmarks_in_vm_package"]
end
H --> Y["VM_workflow_complete"]
L --> Y
P --> Y
T --> Y
X --> Y
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Summary of ChangesHello @Dargon789, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly upgrades the project's development workflow and community interaction mechanisms. It establishes a foundation for continuous integration using CircleCI and ensures a consistent development environment across contributors through a new DevContainer setup. Furthermore, the PR refines the project's issue management by introducing new, comprehensive issue templates and simplifying existing package-specific templates, thereby improving the clarity and flexibility of issue reporting. Minor dependency updates are also included to maintain toolchain currency. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Hey - I've found 2 issues, and left some high level feedback:
- The
docker-image.ymlchange to thetagsfield appears syntactically broken (ghcr.io/${{ github.repository_owner/ethereumjs-monorepo:latestis missing a closing}}and slash), which will likely cause the Docker build step to fail. - The VM workflows now use Node 18 instead of 20 while the newly added Next.js workflow explicitly uses Node 20; consider aligning Node versions across workflows (or documenting the reason to diverge) to avoid subtle runtime/tooling inconsistencies.
- Several newly added CI/workflow files (Next.js, mdBook, Jekyll, CircleCI) look like generic templates and may not be wired correctly for this repo (e.g.,
next buildwriting to./out, thecurl ... -yin mdBook, multiple Pages workflows); it would be good to either remove unused templates or adapt and verify them for this project before merging.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `docker-image.yml` change to the `tags` field appears syntactically broken (`ghcr.io/${{ github.repository_owner/ethereumjs-monorepo:latest` is missing a closing `}}` and slash), which will likely cause the Docker build step to fail.
- The VM workflows now use Node 18 instead of 20 while the newly added Next.js workflow explicitly uses Node 20; consider aligning Node versions across workflows (or documenting the reason to diverge) to avoid subtle runtime/tooling inconsistencies.
- Several newly added CI/workflow files (Next.js, mdBook, Jekyll, CircleCI) look like generic templates and may not be wired correctly for this repo (e.g., `next build` writing to `./out`, the `curl ... -y` in mdBook, multiple Pages workflows); it would be good to either remove unused templates or adapt and verify them for this project before merging.
## Individual Comments
### Comment 1
<location> `.github/workflows/mdbook.yml:35-39` </location>
<code_context>
+ MDBOOK_VERSION: 0.4.36
+ steps:
+ - uses: actions/checkout@v4
+ - name: Install mdBook
+ run: |
+ curl --proto '=https' --tlsv1.2 https://sh.rustup.rs -sSf -y | sh
+ rustup update
+ cargo install --version ${MDBOOK_VERSION} mdbook
+ - name: Setup Pages
+ id: pages
</code_context>
<issue_to_address>
**issue (bug_risk):** The mdBook install script likely fails due to incorrect rustup invocation and missing cargo PATH setup.
This sequence is likely to fail:
```sh
curl --proto '=https' --tlsv1.2 https://sh.rustup.rs -sSf -y | sh
rustup update
cargo install --version ${MDBOOK_VERSION} mdbook
```
Two issues:
- `-y` should be passed to the installer via `sh`, not to `curl` (use `| sh -s -- -y`). As written, the installer script may not run as expected.
- After installing rustup, `cargo` isn’t on `PATH` until you source `$HOME/.cargo/env`, so `cargo install` in the same step will likely fail.
You can fix this with:
```sh
curl --proto '=https' --tlsv1.2 https://sh.rustup.rs -sSf | sh -s -- -y
source "$HOME/.cargo/env"
rustup update
cargo install --version "${MDBOOK_VERSION}" mdbook
```
</issue_to_address>
### Comment 2
<location> `.github/ISSUE_TEMPLATE/bug_report.md:32` </location>
<code_context>
+ - Version [e.g. 22]
+
+**Smartphone (please complete the following information):**
+ - Device: [e.g. iPhone6]
+ - OS: [e.g. iOS8.1]
+ - Browser [e.g. stock browser, safari]
</code_context>
<issue_to_address>
**nitpick (typo):** Add a space in the device example (`iPhone6` → `iPhone 6`)
Please update the example device name to `iPhone 6` for correctness and readability.
```suggestion
- Device: [e.g. iPhone 6]
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Code Review
This pull request introduces a significant number of configuration updates for CI/CD, the development environment, and GitHub issue templates. The changes are well-structured and improve the project's tooling. I've provided a couple of suggestions to pin base image versions in the CircleCI and dev container configurations to ensure reproducible environments, which is a crucial best practice. The rest of the changes, including dependency updates and issue template modifications, look solid.
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com> Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> Signed-off-by: Dargon789 <64915515+Dargon789@users.noreply.github.com>
Summary by Sourcery
Update CI and GitHub configuration for VM workflows, container builds, issue templates, and site deployment tooling.
New Features:
Enhancements:
CI:
Documentation: