fix: architecture_linter_v2 — repo-aware, test-backed, CI-enforced (#437) #453

Merged
Rockachopa merged 2 commits from burn/20260409-1926-linter-v2 into main 2026-04-10 09:36:23 +00:00
Owner

Closes #437

Changes

  • Fix broken regex: API_KEY_REGEX was invalid (caused runtime crash in check_api_keys)
  • Fix v1 syntax error: malformed character class in architecture_linter.py
  • Add --repo flag and --json output for CI/scripted use
  • Port v1 sovereignty rules (cloud API endpoint/provider checks) into v2
  • Add LinterResult class for structured programmatic access
  • Skip dirs: .git, node_modules, __pycache__; skip .env.example files
  • 19 test cases in tests/test_linter.py covering all checks
  • CI workflow in .gitea/workflows/architecture-lint.yml

Verification

  • python3 -m py_compile scripts/architecture_linter_v2.py passes
  • python3 -m py_compile scripts/architecture_linter.py passes
  • python3 -m pytest tests/test_linter.py -v — 19/19 passed
Closes #437 ## Changes - **Fix broken regex**: `API_KEY_REGEX` was invalid (caused runtime crash in `check_api_keys`) - **Fix v1 syntax error**: malformed character class in `architecture_linter.py` - **Add `--repo` flag** and `--json` output for CI/scripted use - **Port v1 sovereignty rules** (cloud API endpoint/provider checks) into v2 - **Add `LinterResult` class** for structured programmatic access - **Skip dirs**: `.git`, `node_modules`, `__pycache__`; skip `.env.example` files - **19 test cases** in `tests/test_linter.py` covering all checks - **CI workflow** in `.gitea/workflows/architecture-lint.yml` ## Verification - `python3 -m py_compile scripts/architecture_linter_v2.py` passes - `python3 -m py_compile scripts/architecture_linter.py` passes - `python3 -m pytest tests/test_linter.py -v` — 19/19 passed
Timmy added 1 commit 2026-04-09 23:30:10 +00:00
fix: architecture_linter_v2 — repo-aware, test-backed, CI-enforced (#437)
Some checks failed
Architecture Lint / Linter Tests (pull_request) Successful in 28s
PR Checklist / pr-checklist (pull_request) Successful in 4m25s
Architecture Lint / Lint Repository (pull_request) Failing after 21s
ee025957d9
- Fix broken API_KEY_REGEX in linter_v2.py (was invalid regex causing runtime crash)
- Fix syntax error in architecture_linter.py (malformed character class)
- Add --repo flag and --json output to linter_v2
- Add LinterResult class for structured programmatic access
- Port v1 sovereignty rules (cloud API endpoint/provider checks) into v2
- Skip .git, node_modules, __pycache__ dirs; skip .env.example files
- Add tests/test_linter.py (19 tests covering all checks)
- Add .gitea/workflows/architecture-lint.yml for CI enforcement
- All files pass python3 -m py_compile

Refs: #437
Rockachopa reviewed 2026-04-10 03:41:13 +00:00
Rockachopa left a comment
Owner

Auto-approved: clean diff, no conflicts, mergeable.

Auto-approved: clean diff, no conflicts, mergeable.
Rockachopa scheduled this pull request to auto merge when all checks succeed 2026-04-10 03:41:14 +00:00
Timmy added 1 commit 2026-04-10 09:36:20 +00:00
Merge branch 'main' into burn/20260409-1926-linter-v2
Some checks failed
Architecture Lint / Linter Tests (pull_request) Successful in 10s
PR Checklist / pr-checklist (pull_request) Failing after 1m13s
Architecture Lint / Lint Repository (pull_request) Failing after 6s
836849ffeb
Rockachopa merged commit 2677e1c796 into main 2026-04-10 09:36:23 +00:00
Rockachopa referenced this issue from a commit 2026-04-10 09:36:24 +00:00
Sign in to join this conversation.