chore: Remove revert strings from tests profile

[fusmanii]

No description provided.

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	eslint@​7.32.0
	zksync-web3@​0.14.3
	eslint-config-prettier@​8.4.0
	eslint-plugin-node@​11.1.0
	prettier-plugin-rust@​0.1.9
	buffer-layout@​1.2.2
	ts-mocha@​10.1.0
	husky@​4.3.8
	codama@​1.3.0
	eslint-plugin-promise@​5.2.0
	ts-node@​10.6.0
	eslint-plugin-import@​2.25.4
	ethers@​5.7.2
	bs58@​6.0.0
	solhint@​3.6.2
	chai@​4.3.7 ⏵ 4.5.0
	multiformats@​9.9.0
	eslint-config-standard@​16.0.3
	yargs@​17.7.2
	pretty-quick@​4.2.2
	prettier-plugin-solidity@​2.2.1
	typescript@​5.6.2
	prettier@​3.8.1
	eslint-plugin-prettier@​3.4.1
	dotenv@​10.0.0
	mocha@​8.4.0 ⏵ 10.8.2	+1		+1

View full report

[socket-security]

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action	Severity	Alert  (click "▶" to expand/collapse)
Warn		Critical CVE: Elliptic's private key extraction in ECDSA upon signing a malformed input (e.g. a string) CVE: GHSA-vjh7-7g9h-fjfh Elliptic's private key extraction in ECDSA upon signing a malformed input (e.g. a string) (CRITICAL) Affected versions: < 6.6.1 Patched version: 6.6.1 From: ? → npm/elliptic@6.5.7 ℹ Read more on: This package | This alert | What is a critical CVE? Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev. Suggestion: Remove or replace dependencies that include known critical CVEs. Consumers can use dependency overrides or npm audit fix --force to remove vulnerable dependencies. Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/elliptic@6.5.7. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 427a79f390

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Keep fork tests in the default compile surface

Changing profile.default.test to test/evm/foundry/local means the only Foundry commands we run in CI now ignore test/evm/foundry/fork/*: upload-evm-artifacts builds with the default profile in .github/workflows/pr.yml:57-62, and test-evm runs yarn test-evm-foundry in .github/workflows/pr.yml:216-217, which now resolves to plain forge test. As a result, syntax/import/type errors in fork tests can merge unnoticed even though this commit edits files in that suite.

Useful? React with 👍 / 👎.