Skip to content

chore: triage and close 77 stale conflicting draft PRs#486

Open
thisrohangupta wants to merge 2 commits into
mainfrom
cursor/close-stale-draft-prs-1b48
Open

chore: triage and close 77 stale conflicting draft PRs#486
thisrohangupta wants to merge 2 commits into
mainfrom
cursor/close-stale-draft-prs-1b48

Conversation

@thisrohangupta

Copy link
Copy Markdown
Collaborator

Summary

Audited 92 open draft PRs on harness/mcp-server. The cloud agent token cannot close PRs directly (Resource not accessible by integration), so this adds a maintainer script plus the triage rationale.

Current draft landscape

Status Count Meaning
DIRTY 69 Merge conflicts with main
BLOCKED 21 No conflicts, CI/review blocked
CLEAN 2 Mergeable

Root cause

Most drafts are duplicate Cursor-agent runs on two themes:

  • cursor/mcp-server-coding-standards-* (33 branches)
  • cursor/missing-test-coverage-* (35 branches)

Much of this work is already merged on main via #447, #446, #449, #469, #440, #398, #391, #430, etc.


Recommended closures (77 PRs)

Close all 69 DIRTY drafts (merge conflicts)

These cannot merge without substantial rebases and largely duplicate landed work.

Highest-conflict drafts (10+ files) — prioritize closing first:

PR Files Title
#443 24 fix: harden pipeline lint and template catalog routing
#441 24 test: cover pipeline lint gate (PR #440)
#453 23 fix: strengthen coding-standards enforcement
#442 23 test: strengthen coding standards guardrails for pipeline-lint
#387 22 fix: enforce Zod describe ordering
#452 21 test: cover semantic search routing
#412 21 test: cover CCM extractors, schema cache
#411 21 test: extend coding standards enforcement
#388 20 feat: add automated coding standards enforcement
#384 20 test: cover PR #377 preflight refactors
#382 19 test: add coverage for PR #375
#383 18 chore: strengthen coding standards enforcement
#381 17 Add automated architecture standards enforcement
#380 17 feat: add automated coding standards enforcement
#415 15 fix: enforce coding standards — Zod describe-last
#409 14 test: cover PR #398 Zod descriptions
#400 14 chore: strengthen coding standards enforcement
#410 13 Enforce registry-driven MCP architecture standards
#414 12 test: extend automated coding standards enforcement
#401 12 fix: enforce coding standards for Zod schemas
#389 12 test: add regression coverage for CCM, STO, schema
#473 11 test: cover search routing eval edge cases
#386 11 chore: enforce MCP server coding standards in CI
#466 10 fix: enforce registry dispatch pattern
#419 10 test: cover dbops authoring flow
#385 10 test: extend identity injection coverage

Oldest stale drafts (pre-June, still conflicting): #104, #125, #167, #168, #319, #323, #330, #332, #338, #340, #343, #349, #356, #357, #364, #365, #367, #369, #371, #394

Full list is encoded in scripts/close-stale-draft-prs.sh.

Close 8 BLOCKED duplicates (no conflicts, but redundant)

PR Files Superseded by
#474 12 #447 (merged)
#478 9 #447 (merged)
#462 13 #447 / #398
#471 11 #447 / #398
#468 7 #447 / #398
#408 17 #398 (merged)
#421 10 #391 / #430
#424 3 #391 / #430

Keep open for review (15 PRs)

CLEAN — mergeable, target mcp_search branch

BLOCKED — no conflicts, potentially unique recent work


How to run

# Preview PR numbers
./scripts/close-stale-draft-prs.sh --dry-run

# Close conflicting drafts only
./scripts/close-stale-draft-prs.sh --dirty-only

# Close blocked duplicates only
./scripts/close-stale-draft-prs.sh --blocked-dupes-only

# Close all 77 recommended closures
./scripts/close-stale-draft-prs.sh

Requires gh authenticated with pull-request write access.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other — maintainer triage tooling

Checklist

  • Script is self-contained; no build/test impact
  • Maintainer runs script with write-capable gh token
Open in Web Open in Cursor 

Adds a maintainer script that closes 69 DIRTY (merge-conflicting) draft PRs
and 8 BLOCKED duplicates of already-merged coding-standards work.

Triage keeps 13 unique BLOCKED drafts and 2 CLEAN drafts targeting mcp_search.

Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
@thisrohangupta thisrohangupta marked this pull request as ready for review June 30, 2026 00:57

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Risk Assessment: Very Low

Decision: Approved — no reviewer assignment required.

Evidence (from diff only)

Factor Assessment
Files changed 1 (scripts/close-stale-draft-prs.sh)
Production code None — no src/, tests, or config changes
Blast radius Zero runtime impact; maintainer-only gh pr close helper
Complexity Low — self-contained bash with --dry-run and scoped flags
Infra / security No deployment, auth, or shared-system changes

Rationale

This PR adds an internal maintainer script to batch-close stale conflicting draft PRs. It does not modify MCP server behavior, CI pipelines, or user-facing surfaces. The script requires explicit maintainer invocation with a write-capable gh token and includes a dry-run mode.

Per policy: Very Low risk → approve directly.

Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers

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.

3 participants