You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ci.yml (Validate repository) runs only npm run test:unit and npm run build:production. No workflow — ci.yml, deploy-gh-pages.yml, or any other — runs check:spelling, check:markdown, or check:format on PRs or on main. Verified by reading .github/workflows/ @ 60ddab0.
Current main @ 60ddab0, clean checkout: check:markdown exits 123 with 12 errors in 4 files (6× LAUNCH.md MD060, 3× adr/README.md MD060, 2× README.md MD009, 1× stakeholder-outreach MD040); check:spelling fails on 1 word (lightboxes, CONTRIBUTING.md:39 — re-broken after fix: add missing cspell words and drop phantom analyses path #292's cspell-word merge).
Nothing in CI flagged any of this: the lint signal exists only for contributors who run npm run check locally.
Recommendation
Add a lint job to ci.yml running npm run check:spelling && npm run check:markdown && npm run check:format (or npm run check minus link-checking), and make it required alongside Validate repository (#357). Until then, every docs PR can silently re-break the checks #315/#292 just fixed — #264 tracks the current red state.
Filed by scanner agent (ACMM L4 — issues-only mode)
Finding
ci.yml(Validate repository) runs onlynpm run test:unitandnpm run build:production. No workflow — ci.yml, deploy-gh-pages.yml, or any other — runscheck:spelling,check:markdown, orcheck:formaton PRs or on main. Verified by reading .github/workflows/ @ 60ddab0.The consequence played out live today:
fix: make check:markdown pass on a clean checkout) merged at ~14:07Z and made markdownlint green.check:markdownon main.check:markdownexits 123 with 12 errors in 4 files (6× LAUNCH.md MD060, 3× adr/README.md MD060, 2× README.md MD009, 1× stakeholder-outreach MD040);check:spellingfails on 1 word (lightboxes, CONTRIBUTING.md:39 — re-broken after fix: add missing cspell words and drop phantom analyses path #292's cspell-word merge).Nothing in CI flagged any of this: the lint signal exists only for contributors who run
npm run checklocally.Recommendation
Add a lint job to ci.yml running
npm run check:spelling && npm run check:markdown && npm run check:format(ornpm run checkminus link-checking), and make it required alongside Validate repository (#357). Until then, every docs PR can silently re-break the checks #315/#292 just fixed — #264 tracks the current red state.Filed by scanner agent (ACMM L4 — issues-only mode)
🐝 Hive Agent:
scanner| Instance:hosted-available-lke648397-260827-5n31| SHA:60ddab0— hive: agent=scanner backend=copilot model=kimi-k3