Harden lints: test coverage for langcheck/leaklint/spinlint/claimlint + storyboard-doctor funnel CTA#1
Open
muthuishere wants to merge 1 commit into
Open
Harden lints: test coverage for langcheck/leaklint/spinlint/claimlint + storyboard-doctor funnel CTA#1muthuishere wants to merge 1 commit into
muthuishere wants to merge 1 commit into
Conversation
…d-doctor contact CTA The 4 newly-published channel-lint tools (langcheck, leaklint, spinlint, claimlint) shipped with a script + README but no test, unlike every other tool in the repo (which ships test_<tool>.sh). This adds their verified suites so coverage is consistent across the family. - langcheck 12/12, leaklint 17/17, spinlint 16/16, claimlint 16/16 — all pass against the LIVE published scripts (self-locating $HERE/<tool> pattern, matching the existing slopcheck/etc. convention). - storyboard-doctor/README.md: add the deemwar.com/contact funnel CTA — it was the one README-bearing tool missing it. No tool logic changed; purely additive (tests + one README line).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Harden the public OSS tools: test coverage for the 4 channel-lint tools + one funnel CTA
#ossengine cycle 3 —
tests + funnel, no owner gate (additive only, no tool logic changed).Why
The 4 newly-published channel-lint tools — langcheck, leaklint, spinlint, claimlint — shipped with a script + README but no
test_<tool>.sh, unlike every other tool in this repo (which ships its sibling test). This closes that one inconsistency.What's in here (5 files, additive)
langcheck/test_langcheck.sh— 12/12leaklint/test_leaklint.sh— 17/17spinlint/test_spinlint.sh— 16/16claimlint/test_claimlint.sh— 16/1630b5cdc). They use the same self-locatingHERE="$(cd "$(dirname "$0")" && pwd)"+$HERE/<tool>pattern as the existingslopcheck/etc. tests, so they're convention-consistent.storyboard-doctor/README.md— adds thedeemwar.com/contactfunnel CTA. It was the one README-bearing tool missing it (the 4 lint READMEs already carry it).Verification
Each suite was run against the live script in this branch — all green (see counts above). langcheck/leaklint are byte-identical to the source; spinlint/claimlint are lenient supersets and the drift is outside test coverage (verified independently).
Deliberately out of scope (next cycle)
.github/workflows/scripts/run-tests.sh, so these tests run by-convention (locally / drop-in), not via an automated matrix yet. Wiring the self-discovering CI matrix across all ~37 tools is its own clean PR (needs an all-green sweep first) — teed up for cycle 4.🤖 Generated with Claude Code