lucas/omp-deck
lucas/omp-deck
Watch
1
Fork
You've already forked omp-deck
 0
Code Issues 9 Pull requests Projects Releases Packages Wiki Activity

Instruct agents to open unfinished PRs as drafts #70

Merged
lucas merged 1 commit from agent/issue-68-draft-pr-guidance into main 2026-10-26 13:32:18 +01:00
Conversation 1 Commits 1 Files changed 2 +6 -3
omp-deck commented 2026-10-26 13:31:33 +01:00
Collaborator

Adds draft/WIP guidance, as planned in the issue. This only changes description text, so no behavior changes:

  • forgejo_create_pr: unfinished work MUST use draft: true. The PR is marked ready by removing the WIP:/[WIP] prefix with forgejo_update_pr. I updated the draft param description to match.
  • forgejo_update_pr: explains how to mark a PR ready and how to turn it back into a draft.
  • Forgejo system context: new first bullet under "Pull request lifecycle".

Verification: in this checkout, bun test src/forgejo/tools passed 30 tests and failed 6. All 6 failures came from the missing module @oh-my-pi/pi-coding-agent, so dependencies aren't installed here. bun run typecheck also failed, in @omp-deck/web, which I didn't touch. I didn't do the manual check from the issue.

Closes: #68

Adds draft/WIP guidance, as planned in the issue. This only changes description text, so no behavior changes: - `forgejo_create_pr`: unfinished work MUST use `draft: true`. The PR is marked ready by removing the `WIP:`/`[WIP]` prefix with `forgejo_update_pr`. I updated the `draft` param description to match. - `forgejo_update_pr`: explains how to mark a PR ready and how to turn it back into a draft. - Forgejo system context: new first bullet under "Pull request lifecycle". Verification: in this checkout, `bun test src/forgejo/tools` passed 30 tests and failed 6. All 6 failures came from the missing module `@oh-my-pi/pi-coding-agent`, so dependencies aren't installed here. `bun run typecheck` also failed, in `@omp-deck/web`, which I didn't touch. I didn't do the manual check from the issue. Closes: #68
Tool descriptions for forgejo_create_pr/forgejo_update_pr and the Forgejo
system context now say when to use draft: true and how to mark a draft
ready (drop the WIP prefix via forgejo_update_pr).

Closes: #68
Co-Authored-By: Claude Opus 5.5 via omp-deck
lucas approved these changes 2026-10-26 13:32:14 +01:00
lucas merged commit 411155d885 into main 2026-10-26 13:32:18 +01:00
lucas deleted branch agent/issue-68-draft-pr-guidance 2026-10-26 13:32:19 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lucas
Labels
Clear labels   
Agent
Ignore
 omp-deck skips this pull request

  
Agent
In Progress
 An omp-deck implementation is running

  
Agent
Planned
 An approved plan has been written to the issue

  
Agent
Planning
 An omp-deck planning session is running

  
Agent
Ready
 Starts an omp-deck implementation of the issue

  
Agent
Triage
 Starts an omp-deck triage session on the issue

  
Compat
Breaking
 Breaking change that won't be backward compatible

  
Kind
Bug
 Something is not working

  
Kind
Documentation
 Documentation changes

  
Kind
Enhancement
 Improve existing functionality

  
Kind
Feature
 New functionality

  
Kind
Security
 This is security issue

  
Kind
Testing
 Issue or pull request related to testing

  
Priority
Critical
 The priority is critical

  
Priority
High
 The priority is high

  
Priority
Low
 The priority is low

  
Priority
Medium
 The priority is medium

  
Reviewed
Confirmed
 Issue has been confirmed

  
Reviewed
Duplicate
 This issue or pull request already exists

  
Reviewed
Invalid
 Invalid issue

  
Reviewed
Won't Fix
 This issue won't be fixed

  
Status
Abandoned
 Somebody has started to work on this but abandoned work

  
Status
Blocked
 Something is blocking this issue or pull request

  
Status
Need More Info
 Feedback is required to reproduce issue or to continue work

No labels
Agent
Ignore
Agent
In Progress
Agent
Planned
Agent
Planning
Agent
Ready
Agent
Triage
Compat
Breaking
Kind
Bug
Kind
Documentation
Kind
Enhancement
Kind
Feature
Kind
Security
Kind
Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
lucas/omp-deck!70
No description provided.