fix(ci): keep elapsed review deadlines out of ordinary merge gates - #533
Merged
Merged
Conversation
|
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
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.



Problem and result
Elapsed documentation and dependency-exception review dates were stopping unrelated source changes and cascading into skipped build/test lanes. Ordinary CI now reports elapsed review dates as maintenance reminders; structural/date consistency, exception ownership/evidence, dependency audits, generated facts, tests, and analysis remain blocking.
A separate weekly/manual Maintenance review deadlines workflow enforces full documentation and exception deadlines without participating in the merge gate. This extends the intent of #531. The nanoid exception was reviewed and retained because installed PostCSS 8.5.23 still declares ^3.3.16, permitting versions below the fixed 3.3.18 boundary; its next review is October 15. No package or runtime behavior changes.
Verification
pnpm health:check,pnpm lint,pnpm format:check, fullpnpm build, andpnpm typecheckpass.pnpm audit:securitypasses with the two existing patched image-size advisory exclusions; no new exclusion.node scripts/repository-health.mjs --maintenance, dependency governance, andgit diff --checkpass.Dependency evidence
Release and operations
No publication, image, deployment, or live workload change. The operator authorized this maintenance policy and admin merge after complete validation. Larger overlay work and existing QA trackers are outside this change.