Skip to content

ci: add manifest lint check#328

Merged
olearycrew merged 1 commit intomainfrom
ci/add-manifest-lint
Apr 14, 2026
Merged

ci: add manifest lint check#328
olearycrew merged 1 commit intomainfrom
ci/add-manifest-lint

Conversation

@ScuttleBot
Copy link
Copy Markdown

The manifest header claims:

A CI lint check verifies that every task_*.md file appears here and every entry here has a corresponding file.

But this wasn't actually wired up. Now it is.

Adds python scripts/lint_manifest.py to the lint workflow, which checks:

  • Every manifest entry has a corresponding .md file
  • Every task_*.md file is listed in the manifest
  • No duplicate entries
  • Frontmatter id matches filename

Also adds pyyaml to the lint dependencies.

The manifest header claims a CI check verifies task files match the manifest,
but this wasn't actually wired up. Now it is.

Runs scripts/lint_manifest.py which checks:
- Every manifest entry has a corresponding .md file
- Every task_*.md file is listed in the manifest
- No duplicate entries
- Frontmatter id matches filename
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot bot commented Apr 14, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Solid CI addition — wires up the manifest lint check that was previously only documented but not enforced. Clean and minimal change.

Files Reviewed (1 file)
  • .github/workflows/lint.yml

Reviewed by claude-4.6-sonnet-20260217 · 58,311 tokens

@olearycrew olearycrew merged commit eab0944 into main Apr 14, 2026
2 checks passed
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.

2 participants