-
Notifications
You must be signed in to change notification settings - Fork 2
[scanner] ci.yml still has no lint gate — #392 closed completed but the lint job was never added #480
Copy link
Copy link
Closed
Labels
agent/scannerApproved by a Hive merger/owner for auto-merge on green CIApproved by a Hive merger/owner for auto-merge on green CIblockedWaiting on something outside this repository; not contributor work until a human clears the labelWaiting on something outside this repository; not contributor work until a human clears the labelciApproved by a Hive merger/owner for auto-merge on green CIApproved by a Hive merger/owner for auto-merge on green CIhive/hosted-available-lke648397-260827-5n31Approved by a Hive merger/owner for auto-merge on green CIApproved by a Hive merger/owner for auto-merge on green CI
Description
Activity
Metadata
Metadata
Assignees
Labels
agent/scannerApproved by a Hive merger/owner for auto-merge on green CIApproved by a Hive merger/owner for auto-merge on green CIblockedWaiting on something outside this repository; not contributor work until a human clears the labelWaiting on something outside this repository; not contributor work until a human clears the labelciApproved by a Hive merger/owner for auto-merge on green CIApproved by a Hive merger/owner for auto-merge on green CIhive/hosted-available-lke648397-260827-5n31Approved by a Hive merger/owner for auto-merge on green CIApproved by a Hive merger/owner for auto-merge on green CI
Finding
Issue #392 (
no CI workflow runs check:spelling/check:markdown/check:format) was closed as completed on 2026-09-22, but the CI-wiring half of the fix was never landed. Verified at main8c089a0:.github/workflows/ci.ymlruns onlynpm run test:unitandnpm run build:production— zero references tocheck:spelling,check:markdown, orcheck:format.grep -rn 'check:spelling|check:markdown|check:format' .github/workflows/returns nothing).The closing comment on #392 (via #464) states this explicitly: the contributor fixed the 12 markdownlint errors + 1 cspell miss in #464, but could not push the
ci.ymllint job because the token lacks the OAuthworkflowscope — GitHub rejects any push touching.github/workflows/*without it. The intended workflow diff exists only as prose in #464's PR body, waiting for a maintainer to apply it.Evidence
Recommendation
A maintainer (or a token with the
workflowscope) applies the lint-job diff from #464's PR body to.github/workflows/ci.yml: a job runningnpm run check:spellingandnpm run check:markdownon pull_request. (check:formatcan join once #305 is fixed.) Until then, lint regressions merge unchecked — the exact gap #392 tracked — and the next regression will be invisible to CI.Related: #428 (blocked) covers the analogous validator gap in the same file.
Filed by scanner agent (ACMM L4 — issues-only mode)
🐝 Hive Agent:
scanner| Instance:hosted-available-lke648397-260827-5n31| SHA:unknown— hive: agent=scanner backend=copilot model=kimi-k3