Skip to content

chore(pr): add PR readiness workflow#131

Merged
BunsDev merged 1 commit into
mainfrom
finish/cli-only-auth-pr-ready
Jul 5, 2026
Merged

chore(pr): add PR readiness workflow#131
BunsDev merged 1 commit into
mainfrom
finish/cli-only-auth-pr-ready

Conversation

@BunsDev

@BunsDev BunsDev commented Jul 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Add a repository PR template with explicit context, validation, and readiness sections.
  • Add AGENTS.md guidance for PR-ready branch preparation and parallel-agent PR handoffs.
  • Add a bundled pr-ready skill and tighten the batch skill so worker branches produce review-ready PRs.

Context

  • Related issue/user request: consolidate remaining branch/worktree work back toward main.
  • Scope: PR readiness workflow docs/template plus bundled skill guidance.
  • Non-goals: no TUI behavior changes; the older accessibility/auth UI branch content already landed separately or was intentionally left out.

Changes

  • Added .github/PULL_REQUEST_TEMPLATE.md.
  • Documented PR readiness expectations in AGENTS.md.
  • Added pr-ready bundled skill aliases and tests.
  • Updated the batch bundled skill prompt to require PR template awareness, body-file usage, verification evidence, and parseable PR output.

Validation

Record exact commands and outcomes. Mark items N/A with a reason when they do not apply.

  • git diff --check - PASS
  • cargo fmt --all - PASS
  • cargo check --workspace - PASS
  • cargo clippy --workspace --all-targets -- -D warnings - PASS
  • cargo test --workspace - PASS
  • Targeted/manual checks:
    • cargo test --package claurst-tools pr_ready_skill_enforces_template_and_body_file_contract - PASS
    • cargo test --package claurst-tools batch_skill_requires_pr_readiness_contract_for_workers - PASS
  • Not run: N/A

PR Readiness

  • Diff is limited to the intended files.
  • Generated files were not edited by hand.
  • User-facing docs/help were updated or are not needed.
  • No secrets, credentials, local paths, or unrelated logs are included.
  • Remaining risks or follow-up work are listed above.

Copilot AI review requested due to automatic review settings July 5, 2026 00:01
@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Jul 5, 2026 12:01am

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a lightweight “PR readiness” workflow to standardize how contributors (and parallel agents) prepare, verify, and author review-ready pull requests, aligning bundled skills with the new expectations.

Changes:

  • Introduces a repository PR template that captures context, scope, validation evidence, and readiness checks.
  • Documents PR readiness and parallel-agent handoff expectations in AGENTS.md.
  • Adds a new pr-ready bundled skill (with aliases) and updates the batch skill prompt to enforce PR-template-aware, evidence-backed, parseable PR outputs; includes tests to lock in the contract.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src-rust/crates/tools/src/bundled_skills.rs Adds pr-ready bundled skill + strengthens batch worker PR readiness contract and tests.
AGENTS.md Documents PR readiness expectations and parallel-agent PR handoff requirements.
.github/PULL_REQUEST_TEMPLATE.md Adds a standardized PR body template with validation and readiness checklists.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BunsDev BunsDev merged commit 073f7b4 into main Jul 5, 2026
3 checks passed
@BunsDev BunsDev deleted the finish/cli-only-auth-pr-ready branch July 5, 2026 00:03
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