Skip to content

Stop auto-populating create PR fields#2660

Merged
brennanb2025 merged 1 commit into
mainfrom
brennanb2025/Create-PR-auto-populate
May 23, 2026
Merged

Stop auto-populating create PR fields#2660
brennanb2025 merged 1 commit into
mainfrom
brennanb2025/Create-PR-auto-populate

Conversation

@brennanb2025

@brennanb2025 brennanb2025 commented May 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Stop seeding Create PR title and description from commit subjects
  • Keep the base branch default, but require explicit or generated reviewer-facing copy
  • Improve the draft control to a full-width Create as draft checkbox row

Tests

  • pnpm run typecheck
  • pnpm run lint
  • git diff --check origin/main...HEAD
  • pnpm vitest run --config config/vitest.config.ts src/main/source-control/hosted-review-creation.test.ts
  • pnpm test:e2e -- tests/e2e/source-control-create-pr.spec.ts
  • Electron/CDP validation from /Users/thebr/orca/workspaces/orca/Create-PR-auto-populate; verified window.api.app.getIdentity() matched brennanb2025/Create-PR-auto-populate.
  • Used demo-project for the live app workflow: verified the real eligibility path returns no auto-generated title/body, opened the Create PR composer with empty title/description and base main, filled title/body, toggled Create as draft, clicked Create draft PR, and captured the dry-run payload with draft: true and the demo-project worktree path.
  • Inspected console/app logs for relevant failures; only warning was expected from the fake dry-run PR number not existing after the captured submit.

Made with Orca 🐋

Co-authored-by: Orca <help@stably.ai>
@brennanb2025 brennanb2025 merged commit 8ad76f7 into main May 23, 2026
2 checks passed
@brennanb2025 brennanb2025 deleted the brennanb2025/Create-PR-auto-populate branch May 23, 2026 05:48
thomaszdxsn pushed a commit to thomaszdxsn/orca that referenced this pull request Jun 15, 2026
Co-authored-by: Orca <help@stably.ai>
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