Skip to content

Add merge-pr safety preflight skill#220

Open
andrei-hasna wants to merge 3 commits into
mainfrom
feat/merge-pr-skill-mainline-20260712
Open

Add merge-pr safety preflight skill#220
andrei-hasna wants to merge 3 commits into
mainfrom
feat/merge-pr-skill-mainline-20260712

Conversation

@andrei-hasna

Copy link
Copy Markdown
Contributor

Why

PR #215 contains the desired merge-pr skill work, but it targets fleet/spark02-preserve-20260625-codewith. Retargeting that PR to main would include unrelated preservation-branch changes, which makes it unsuitable as a releaseable mainline PR.

This PR supersedes #215 for releaseability by starting from current main and carrying only .codewith/skills/merge-pr/** from de0db42c005d80b5d1ec900023311af53a645730.

What changed

  • Adds the Codewith merge-pr skill instructions and merge safety reference.
  • Adds a read-only merge_pr_preflight.py helper for PR mergeability snapshots.
  • Supports current and legacy gh pr checks status fields such as bucket, state, status, and conclusion.
  • Adds fixture-backed tests for successful, failed, pending, and actual-merge artifact-blocking preflight cases.

Validation

  • PYTHONDONTWRITEBYTECODE=1 python3 .codewith/skills/merge-pr/tests/test_merge_pr_preflight.py
  • python3 -m py_compile .codewith/skills/merge-pr/scripts/merge_pr_preflight.py
  • git diff --check
  • git diff --cached --check
  • shield review
  • todos redaction scan --file over each staged file
  • python3 .codewith/skills/merge-pr/scripts/merge_pr_preflight.py 215 --repo hasna/codewith --mode preflight

The live PR #215 preflight was read-only and confirmed the current gh pr checks JSON shape is parsed. It reported #215 as not_mergeable because that preservation-branch PR currently has failing and pending checks.

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@andrei-hasna
andrei-hasna force-pushed the feat/merge-pr-skill-mainline-20260712 branch from 92ab649 to cc4d33b Compare July 14, 2026 23:30
@andrei-hasna
andrei-hasna force-pushed the feat/merge-pr-skill-mainline-20260712 branch from cc4d33b to cbae048 Compare July 15, 2026 08:11
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.

1 participant