Skip to content

ci(knowledge): enforce document metadata and seven-day reviews - #285

Merged
laipz8200 merged 2 commits into
mainfrom
laipz8200/knowledge-review-metadata
Sep 9, 2026
Merged

laipz8200 merged 2 commits into
mainfrom
laipz8200/knowledge-review-metadata

Conversation

@laipz8200

@laipz8200 laipz8200 commented Sep 8, 2026 •

Copy link
Copy Markdown
Member

Important

  1. Make sure you have read our contribution guidelines
  2. Search existing issues and pull requests to confirm this change is not a duplicate
  3. Open or identify the issue this pull request resolves or advances
  4. Use a Conventional Commits title for this pull request, and mark breaking changes with !
  5. Remember that the pull request title will become the squash merge commit message
  6. If CLA Assistant prompts you, sign CLA.md in the pull request conversation

Related Issue

Closes #284

Depends on #283; based on laipz8200/execution-file-runtime.

Summary

Standardize all 16 maintained knowledge documents on a hidden YAML metadata block with a UTC last_checked timestamp string. Include root guides, nested documentation, component/example guides, and completed plans; exclude the root legal agreement, release ledger, and GitHub templates.

Fail CI when the oldest recorded review is more than seven days old, or when required metadata is missing, malformed, or future-dated. Report the oldest document and timestamp. Enforce this through the existing CI pipeline.

Document how to review pages and update their timestamps, preserve existing review-date evidence, and correct the guide inaccuracies found while initializing metadata.

Checklist

  • This pull request links the issue it resolves or advances
  • This pull request title follows Conventional Commits, and any breaking change is marked with !
  • If CLA Assistant prompted me, I signed CLA.md in the pull request conversation

@laipz8200
laipz8200 force-pushed the laipz8200/knowledge-review-metadata branch from 3b01649 to d9ad80e Compare September 9, 2026 16:12
@laipz8200
laipz8200 force-pushed the laipz8200/knowledge-review-metadata branch from d9ad80e to 9f1c904 Compare September 9, 2026 16:20
Base automatically changed from laipz8200/execution-file-runtime to main September 9, 2026 16:24
Define shared last_checked metadata for maintained knowledge documents and initialize timestamps from review evidence. Require valid UTC timestamp strings and reject documents whose oldest review is more than seven days old.

Run knowledge checks on all pull requests, main pushes, daily schedules, and manual dispatch. Document review scope and timestamp maintenance, and correct guide inaccuracies found during metadata initialization.

Refs #284
Use the existing pytest jobs to enforce knowledge metadata and review freshness. Remove the duplicate workflow and its additional triggers, and update contributor documentation to describe the existing PR-to-main and release coverage.

Refs #284
@laipz8200
laipz8200 force-pushed the laipz8200/knowledge-review-metadata branch from 9f1c904 to 71ada7a Compare September 9, 2026 16:24
@laipz8200
laipz8200 merged commit 151e289 into main Sep 9, 2026
8 checks passed
@laipz8200
laipz8200 deleted the laipz8200/knowledge-review-metadata branch September 9, 2026 16:27
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.

Standardize knowledge metadata and enforce seven-day review freshness

2 participants