alibaba / alibaba/open-code-review

feat(allowlist): add Solidity / Web3 (.sol, .vy) support

Open
#1,032 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
24.4k
Forks
1.8k
Avg merge
2d 6h
Merged PRs (30d)
105

Description

Tracking parent issue: #470

## Scope

Add the **Solidity / Web3** language group to the review allowlist:

- Extensions: `.sol` (Solidity), `.vy` (Vyper)
- Test exclude patterns: `**/test/**/*.sol` (Hardhat), `**/*.t.sol` (Foundry)
- Rule docs: `internal/config/rules/rule_docs/solidity.md` and `vyper.md`
- Register `**/*.sol` -> `solidity.md` and `**/*.vy` -> `vyper.md` in `system_rules.json`
- Tests: `allowed_ext_test.go` and `system_rules_test.go`

## Acceptance

- [x] `make test` passes
- [x] `make check` passes

Contributor guide

Open the contributing guide

Research direction

Start with allowed_ext_test.go and system_rules_test.go, then inspect system_rules.json and the rule documentation paths under internal/config/rules/rule_docs/. Add the Solidity and Vyper extensions, test exclude patterns, rule registrations, and docs described in the issue; done means make test and make check pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, solidity
Domain
tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
75/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.