ci(gate): enforce mechanical PR verification, PR templates, and openrig agent specs #91

Merged
yuris merged 3 commits from chore/enforce-pr-verification-and-skills into main 2026-10-14 23:42:27 +00:00
Owner

Summary

Enforces mechanical PR convention verification, installs PR templates, and declares OpenRig agent specs with explicit skill manifests.

Principles applied

  • principle-encode-lessons-in-structure: Converted advisory skill guidelines into deterministic code gates.
  • principle-boundary-discipline: Isolated review and issue verification at the PR integration boundary.
  • principle-prove-it-works: Verified linter with negative and positive test fixtures.

Impact

  • For the consumer: Guarantees all PRs maintain simplicity, verified test proof, and clear user impact.
  • For the maintainer: Replaces manual review policing with automated mechanical gates in CI.

Anti-Overengineering Audit (ponytail-review)

  • No bespoke tools or in-house crates added when standard upstream libraries exist.
  • No cosmetic compliance.
  • Code subtraction evaluated: zero dead weight added.

Verification proof

  • pnpm exec vp check: All 244 files are correctly formatted, 0 warnings.
  • pnpm test: All 43 tests pass.
  • node scripts/verify-pr.mjs: Validated locally against compliant and non-compliant fixtures.
## Summary Enforces mechanical PR convention verification, installs PR templates, and declares OpenRig agent specs with explicit skill manifests. ### Principles applied - `principle-encode-lessons-in-structure`: Converted advisory skill guidelines into deterministic code gates. - `principle-boundary-discipline`: Isolated review and issue verification at the PR integration boundary. - `principle-prove-it-works`: Verified linter with negative and positive test fixtures. ### Impact - **For the consumer:** Guarantees all PRs maintain simplicity, verified test proof, and clear user impact. - **For the maintainer:** Replaces manual review policing with automated mechanical gates in CI. ### Anti-Overengineering Audit (ponytail-review) - [x] No bespoke tools or in-house crates added when standard upstream libraries exist. - [x] No cosmetic compliance. - [x] Code subtraction evaluated: zero dead weight added. ### Verification proof - `pnpm exec vp check`: All 244 files are correctly formatted, 0 warnings. - `pnpm test`: All 43 tests pass. - `node scripts/verify-pr.mjs`: Validated locally against compliant and non-compliant fixtures.
ci: enforce mechanical PR verification, PR templates, and openrig agent specs
All checks were successful
CI / Verify Quality & Build (pull_request) Successful in 1m45s
6b3e276c01
feat(skills): add 25 Addy Osmani engineering lifecycle skills
Some checks failed
CI / Verify Quality & Build (pull_request) Has been cancelled
dd9c13483f
chore(skills): prune 17 unused skills, keeping curated pstack, addy, and ponytail suites
All checks were successful
CI / Verify Quality & Build (pull_request) Successful in 1m51s
96a8112caf
yuris-bot approved these changes 2026-10-14 23:40:44 +00:00
yuris-bot left a comment

Verified architectural compliance. All mechanical checks, test suites, and lint rules pass. Approved for squash merge.

Verified architectural compliance. All mechanical checks, test suites, and lint rules pass. Approved for squash merge.
yuris merged commit 465ac2da9b into main 2026-10-14 23:42:27 +00:00
yuris deleted branch chore/enforce-pr-verification-and-skills 2026-10-14 23:42:27 +00:00
Sign in to join this conversation.
No reviewers
No labels
backend
frontend
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
yuris/titik-gambut-next!91
No description provided.