Skip to content

ci: enforce pull-request template and assigned-issue admission - #233

Draft
Nitjsefnie wants to merge 2 commits into
mainfrom
issue-55-pr-gate
Draft

ci: enforce pull-request template and assigned-issue admission#233
Nitjsefnie wants to merge 2 commits into
mainfrom
issue-55-pr-gate

Conversation

@Nitjsefnie

@Nitjsefnie Nitjsefnie commented Sep 8, 2026

Copy link
Copy Markdown
Owner

Summary

Add pull-request admission checks against Overflow's existing template, exact Footer and Bugs Discovered content, and the assignment of referenced and closing issues. Corrective edits update one gate comment and can recover interrupted automatic reopening of a pull request the gate closed.

Related Issues and Pull Requests

Fixes #55

Changes

  • Port the rendered-HTML parser, assignment checks, and guarded comment/close/reopen controller from Daedalus.
  • Recognize template instruction tags without case sensitivity while preserving Overflow's required Footer and conditional sections.
  • Require one final attribution line and verify filed-bug pointers against exact GitHub issue titles, sharing the 20-reference lookup budget.
  • Retain gate closure ownership until reopening succeeds, with replay coverage for interrupted reads/writes and maintainer closures.
  • Run the Python regressions through the existing Vitest command, reject empty execution, and pin repository context, workflow permissions, and trusted checkout.

Testing

The complete local sequence passed on commit d87266cdff1bcce0f6d60a75a289b6dfe166ac2b: pnpm db:migrate && pnpm test --run && pnpm lint && pnpm typecheck && pnpm build, using an isolated PostgreSQL 17 database. The test run passed 2,717 Vitest tests across 119 files; its Python bridge also passed all 163 cases across nine gate suites. The focused workflow/integration run passed all 14 cases. Real-file mutation checks caught loss of reopen ownership, empty execution/discovery, hard-coded repository context, invalid attribution acceptance, and mismatched bug-title acceptance; all mutants were restored. actionlint 1.7.12 and zizmor 1.29.0 passed; zizmor ran in offline mode. Each new path and representative ignored junk were checked with git. The successful build emitted three Edge Runtime warnings about node:crypto imports in untouched application files.

Footer

Generated by GPT-6 Astra (implementer)

Nitjsefnie and others added 2 commits September 8, 2026 09:02
Co-Authored-By: GPT-6 Astra <noreply@openai.com>
Co-Authored-By: GPT-6 Astra <noreply@openai.com>
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.

Nothing checks that a pull request follows the template or that an issue is assigned

1 participant