Skip to content

Comments

chore: Add prompt templates for PR workflows#6543

Merged
christianklotz merged 1 commit intomainfrom
chore/pi-pr-workflow-prompt-templates
Feb 1, 2026
Merged

chore: Add prompt templates for PR workflows#6543
christianklotz merged 1 commit intomainfrom
chore/pi-pr-workflow-prompt-templates

Conversation

@christianklotz
Copy link
Contributor

@christianklotz christianklotz commented Feb 1, 2026

Add prompt template with Pi's argument syntax.

Changes

  • landpr.md: Use $1 for positional arg
  • reviewpr.md: Use $1 for positional arg
  • pr.md: Deleted (replaced by the more specific reviewpr.md and landpr.md)

Testing

Verified templates expand correctly:

  • /reviewpr 6527 → expands with PR number in the right places
  • /landpr https://github.com/openclaw/openclaw/pull/6527 → works with full URL

Ref: https://github.com/badlogic/pi-mono/blob/main/packages/coding-agent/docs/prompt-templates.md

Greptile Overview

Greptile Summary

This PR updates Pi prompt templates under .pi/prompts/ to use the correct positional argument syntax for Pi template expansion (switching to $1), and splits the previous generic PR review template into two more specific prompts: reviewpr.md (review-only) and landpr.md (review-only recommendation for whether a PR is ready to land). It also deletes the older pr.md template that used $@ for arguments.

Confidence Score: 5/5

  • This PR is safe to merge; it only adjusts prompt template text and removes an obsolete template file.
  • Changes are limited to markdown prompt templates, with straightforward argument syntax fixes and no impact on runtime code paths; the new templates are consistent and syntactically valid for Pi positional arguments.
  • No files require special attention

(2/5) Greptile learns from your feedback when you react with thumbs up/down!

Context used:

  • Context from dashboard - CLAUDE.md (source)
  • Context from dashboard - AGENTS.md (source)

- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
@christianklotz christianklotz changed the title chore: fix Pi prompt template argument syntax chore: Add prompt templates for PR workflows Feb 1, 2026
@christianklotz christianklotz merged commit 395810a into main Feb 1, 2026
37 of 43 checks passed
@christianklotz christianklotz deleted the chore/pi-pr-workflow-prompt-templates branch February 1, 2026 20:14
bennewton999 pushed a commit to bennewton999/openclaw that referenced this pull request Feb 2, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
buiilding pushed a commit to buiilding/openclaw that referenced this pull request Feb 2, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
HashWarlock pushed a commit to HashWarlock/openclaw that referenced this pull request Feb 4, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
uxcu pushed a commit to uxcu/kook-openclaw that referenced this pull request Feb 5, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
bestNiu pushed a commit to bestNiu/clawdbot that referenced this pull request Feb 5, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
batao9 pushed a commit to batao9/openclaw that referenced this pull request Feb 7, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
hughdidit pushed a commit to hughdidit/DAISy-Agency that referenced this pull request Feb 8, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)

(cherry picked from commit 395810a)

# Conflicts:
#	.pi/prompts/pr.md
battman21 pushed a commit to battman21/openclaw that referenced this pull request Feb 12, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
battman21 pushed a commit to battman21/openclaw that referenced this pull request Feb 12, 2026
- Fix @1 -> $1 in landpr.md
- Fix $@ -> $1 in reviewpr.md
- Remove stray /reviewpr line from reviewpr.md
- Delete old pr.md (replaced by reviewpr.md and landpr.md)
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