Skip to content

feat: add intelligent PR failure handling workflow#52

Merged
4eckd merged 2 commits intomainfrom
claude/fix-pr-creation-01YKBgUAPTZkmtdPRf8vcwt1
Nov 22, 2025
Merged

feat: add intelligent PR failure handling workflow#52
4eckd merged 2 commits intomainfrom
claude/fix-pr-creation-01YKBgUAPTZkmtdPRf8vcwt1

Conversation

@4eckd
Copy link
Contributor

@4eckd 4eckd commented Nov 21, 2025

Changes:

  • Modified auto-draft-pr.yml to gracefully handle PR creation failures
  • Created pr-failure-handler.yml to monitor CI/CD results
  • Automatically closes PRs with critical failures (security, build issues)
  • Creates issues for documented bugs with detailed troubleshooting info
  • Detects patterns like:
    • GitHub Actions permission errors
    • npm security vulnerabilities
    • Exposed secrets
    • Build failures
    • Test failures

This replaces the previous automatic PR creation approach with a more intelligent system that protects the repository from risky merges.

Resolves issue with GitHub Actions not being permitted to create PRs.

Changes:
- Modified auto-draft-pr.yml to gracefully handle PR creation failures
- Created pr-failure-handler.yml to monitor CI/CD results
- Automatically closes PRs with critical failures (security, build issues)
- Creates issues for documented bugs with detailed troubleshooting info
- Detects patterns like:
  * GitHub Actions permission errors
  * npm security vulnerabilities
  * Exposed secrets
  * Build failures
  * Test failures

This replaces the previous automatic PR creation approach with a more
intelligent system that protects the repository from risky merges.

Resolves issue with GitHub Actions not being permitted to create PRs.
@vercel
Copy link

vercel bot commented Nov 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
attorney-finder-bot Ready Ready Preview Comment Nov 21, 2025 2:52pm
design Error Error Nov 21, 2025 2:52pm
devops-panel Ready Ready Preview Comment Nov 21, 2025 2:52pm

@github-actions
Copy link
Contributor

⚪ Workflow Summary: SEO & Marketing Automation

Status: Pending | Progress: 0% Complete

█░░░░░░░░░░░░░░░░░░░░ 0%

📊 Check Results

Status Count
✅ Passed 0
❌ Failed 0
⏳ Pending 0
⏭️ Skipped 0
Total 0

📝 Detailed Results

🔗 Links


Report generated at 2025-11-21 14:46:24 UTC by Workflow Summary Reporter

@github-actions
Copy link
Contributor

⚪ Workflow Summary: Commit Message Linting

Status: Pending | Progress: 0% Complete

█░░░░░░░░░░░░░░░░░░░░ 0%

📊 Check Results

Status Count
✅ Passed 0
❌ Failed 0
⏳ Pending 0
⏭️ Skipped 0
Total 0

📝 Detailed Results

🔗 Links


Report generated at 2025-11-21 14:46:26 UTC by Workflow Summary Reporter

@github-actions
Copy link
Contributor

⚪ Workflow Summary: Enhanced CI/CD Pipeline

Status: Pending | Progress: 0% Complete

█░░░░░░░░░░░░░░░░░░░░ 0%

📊 Check Results

Status Count
✅ Passed 0
❌ Failed 0
⏳ Pending 0
⏭️ Skipped 0
Total 0

📝 Detailed Results

🔗 Links


Report generated at 2025-11-21 14:46:47 UTC by Workflow Summary Reporter

BREAKING CHANGE: Automatic PR creation completely disabled

Changes:
- Renamed auto-draft-pr.yml to CI/CD Failure Monitor
- Removed all automatic PR creation logic
- Workflow now ONLY monitors CI/CD failures and creates issues
- Disabled auto-pr-merge.yml workflow (changed trigger to workflow_dispatch only)
- Added delayed check: waits up to 10 minutes for CI/CD pipeline completion
- Creates detailed issues with troubleshooting steps on failure
- Automatically assigns issues to the commit author
- Categorizes failures: security, build, test, other
- Marks critical failures (security/build) as high priority
- Updates existing issues if branch already has a failure issue

Why:
GitHub Actions does not have permission to create PRs by default,
causing workflow failures. Manual PR creation is now required.

Issues created will include:
- Detailed failure information with logs
- Step-by-step troubleshooting guides
- Links to workflow runs and commits
- Automatic assignment for tracking
- Proper labels and priority
@vercel
Copy link

vercel bot commented Nov 21, 2025

Deployment failed with the following error:

The `vercel.json` schema validation failed with the following message: `framework` should be equal to one of the allowed values "blitzjs, nextjs, gatsby, remix, react-router, astro, hexo, eleventy, docusaurus-2, docusaurus, preact, solidstart-1, solidstart, dojo, ember, vue, scully, ionic-angular, angular, polymer, svelte, sveltekit, sveltekit-1, ionic-react, create-react-app, gridsome, umijs, sapper, saber, stencil, nuxtjs, redwoodjs, hugo, jekyll, brunch, middleman, zola, hydrogen, vite, tanstack-start, vitepress, vuepress, parcel, fastapi, flask, fasthtml, sanity-v3, sanity, storybook, nitro, hono, express, h3, nestjs, elysia, fastify, xmcp, "

Learn More: https://vercel.com/docs/concepts/projects/project-configuration

@4eckd 4eckd added this pull request to the merge queue Nov 22, 2025
Merged via the queue into main with commit b34c3a3 Nov 22, 2025
17 of 19 checks passed
@4eckd 4eckd deleted the claude/fix-pr-creation-01YKBgUAPTZkmtdPRf8vcwt1 branch November 22, 2025 01:13
4eckd pushed a commit that referenced this pull request Nov 25, 2025
This audit report identifies all unmerged commits across branches:
- 2 CRITICAL security fixes (SSRF vulnerability, unused imports)
- 6 feature branches with unmerged work (email handling, UI updates, health monitoring, etc.)
- 2 workflow/config fixes (CodeQL setup, merge error handling)

Key findings:
- 10 branches with unmerged commits (~30+ commits total)
- Most recent successful merges: PR #54 and #52
- Several high-value features not in production

Recommendations include immediate merge of security fixes and
establishing better branch lifecycle management.
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.

2 participants