Skip to content

create-pr: drop the "Next step" PR-body hint - #86

Merged
mattpocock merged 1 commit into
mainfrom
remove-create-pr-next-step-hint
Oct 9, 2026
Merged

mattpocock merged 1 commit into
mainfrom
remove-create-pr-next-step-hint

Conversation

@mattpocock

Copy link
Copy Markdown
Owner

Summary

create-pr stops telling the student what to do with their PR. That's the teacher's job, not the CLI's. Patch changeset included. Size: 1 file, -3/+1 lines.

 ✓ Draft PR for <lesson-id>: https://github.com/<owner>/<repo>/pull/32
-
-Next step: ask your agent to rewrite the PR body, e.g.
-  "Read https://github.com/<owner>/<repo>/pull/32 and rewrite its description to explain the change."

The PR URL line and the drafts-unavailable note are unchanged. Nothing else referenced the hint: no tests asserted on it, and it had no helpers to remove.

Evidence

  • Before: the output after the PR URL line included the "Next step" hint.
    After: the output ends at the PR URL line. pnpm check, pnpm lint and pnpm test all pass (18 files, 188 tests, including create-pr-e2e, which asserts on result.prUrl).

Merge Danger

Door: two-way

Blast Radius: cosmetic

The only change is CLI output text. Nothing parses it.

🤖 Generated with Claude Code

What to do with the PR is the teacher's call, not the CLI's. Output now
ends with the PR URL line.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pkg-pr-new

pkg-pr-new Bot commented Oct 9, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/ai-hero-cli@86

commit: 55dcf83

@mattpocock
mattpocock merged commit cf5f51c into main Oct 9, 2026
3 checks passed
@mattpocock
mattpocock deleted the remove-create-pr-next-step-hint branch October 9, 2026 10:55
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