Skip to content

ci: adopt linecheck to enforce per-file line limits - #104

Merged
tupe12334 merged 1 commit into
mainfrom
ci/adopt-linecheck
Aug 20, 2026
Merged

ci: adopt linecheck to enforce per-file line limits#104
tupe12334 merged 1 commit into
mainfrom
ci/adopt-linecheck

Conversation

@tupe12334

Copy link
Copy Markdown
Member

Opened by the moadim routine Scout & wire linecheck adoption → Slack.

Closes #103

Changes

  • linecheck.yml — per-glob thresholds: 200 warn / 400 error for source, 400 warn / 1000 error for tests/**/*.rs; excludes Cargo.lock, CHANGELOG.md, lockfiles, and assets/.
  • .github/workflows/linecheck.yml — runs linecheck . via the tupe12334/linecheck@main composite action (pinned to 1.2.0) on every PR and push to main.

Verified locally

linecheck . exits 0 with two non-blocking warnings:

./tests/cli.rs: 857 lines (warn threshold: 400)
./agent-runner/src/index.js: 217 lines (warn threshold: 200)

No existing file trips the error tier, so CI stays green on day one while new bloat gets caught.

🤖 Generated with Claude Code

Add linecheck.yml (200/400 for source, 400/1000 for integration
tests, generated files excluded) and a GitHub Actions workflow
running linecheck on every PR and push to main.

Closes #103

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@tupe12334
tupe12334 merged commit 143fa62 into main Aug 20, 2026
2 checks passed
@tupe12334
tupe12334 deleted the ci/adopt-linecheck branch August 20, 2026 00:52
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.

Adopt linecheck: enforce per-file line-count limits in CI

1 participant