Skip to content

[$6 BOUNTY] Enforce uppercase LEGACY comment markers (#2)#72

Open
zhuyuhao030612 wants to merge 1 commit into
cuentaprueba244w-dotcom:mainfrom
zhuyuhao030612:bounty/legacy-caps
Open

[$6 BOUNTY] Enforce uppercase LEGACY comment markers (#2)#72
zhuyuhao030612 wants to merge 1 commit into
cuentaprueba244w-dotcom:mainfrom
zhuyuhao030612:bounty/legacy-caps

Conversation

@zhuyuhao030612

Copy link
Copy Markdown

Summary

Audits and enforces uppercase LEGACY comment markers across the repository.

Changes

  • tools/legacy_caps_audit.py: New audit script with --check mode for CI
  • Scans all files for "legacy" (case-insensitive)
  • Verifies uppercase LEGACY marker per language comment syntax
  • Fixed 48 files that were missing the required LEGACY marker
  • Supports: Rust (//), Python (#), C/C++/Java/Go/JS/TS (//), Lua/Haskell (--), Ruby (#)

Verification

python tools/legacy_caps_audit.py --check
# LEGACY audit passed. Checked 51 legacy file(s). Exit 0

Closes #2

Samourai Wallet: Do4v7foHJvRJLpRRoGaVPWX6DDEjX3yTK7J91gpwUQpE

?? Generated with Claude Code

@
feat: enforce uppercase LEGACY comment markers (cuentaprueba244w-dotcom#2)

- Add tools/legacy_caps_audit.py — audit and auto-fix script
- Scan all files for "legacy" (case-insensitive)
- Verify uppercase LEGACY marker exists for each file
- Fix 48 files missing the required marker
- Supports --check mode for CI (exit 1 on violations)

Closes cuentaprueba244w-dotcom#2
@
@zhuyuhao030612

zhuyuhao030612 commented Jul 5, 2026

Copy link
Copy Markdown
Author

✅ Acceptance Criteria Verification

Re-checked against the bounty requirements (issue #2):

Criteria Status
Audit script with --check mode for CI tools/legacy_caps_audit.py
Case-insensitive scan for "legacy" ✅ All file types covered
Uppercase LEGACY marker enforcement per language ✅ Language-specific comment syntax
Fix all existing violations ✅ 48 violations fixed
Exit code 0 on clean scan
python3 build.py passes

All legacy comment markers now use uppercase LEGACY format. The audit script can be added to CI to prevent regressions.

Ready for review.

@zhuyuhao030612

Copy link
Copy Markdown
Author

Hi @maintainer, this PR is ready for review — no conflicts, clean merge. Could you please approve the workflow and take a look? Thanks! 🙏

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.

[$6 BOUNTY] [Python] Enforce uppercase LEGACY comments

1 participant