Skip to content

Commit 3415a3f

Browse files
committed
docs: clarify workflow PR ownership
1 parent de689a2 commit 3415a3f

2 files changed

Lines changed: 21 additions & 2 deletions

File tree

prpm.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "agent-workforce-skills",
3-
"version": "1.0.0",
3+
"version": "1.0.1",
44
"description": "Skills for multi-agent coordination - swarm patterns, workflow building, relay usage, and headless orchestration",
55
"author": "khaliqgant",
66
"organization": "agent-relay",
@@ -28,7 +28,7 @@
2828
},
2929
{
3030
"name": "writing-agent-relay-workflows",
31-
"version": "1.6.1",
31+
"version": "1.6.2",
3232
"description": "Use when building multi-agent workflows with the relay broker-sdk - covers conversation-shape vs pipeline-shape coordination, WorkflowBuilder API, DAG step dependencies, agent definitions, output chaining via {{steps.X.output}}, verification gates, evidence-based completion, channels, swarm patterns, chat-native coordination recipes (Q/A, broadcast-ack, peer review, standup, hand-off), error handling, event listeners, step sizing, lead+workers team pattern, and parallel wave planning",
3333
"format": "claude",
3434
"subtype": "skill",

skills/writing-agent-relay-workflows/SKILL.md

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -280,6 +280,25 @@ runWorkflow().catch((error) => {
280280

281281
Do not end workflow files with bare top-level `await workflow(...).run(...)`.
282282

283+
### 1b. Make commit and PR boundaries explicit
284+
285+
Workflows do **not** get a PR for free just because they pass validation. If the intended deliverable is a branch, commit, push, or GitHub PR, the workflow itself must own that boundary explicitly and document the expected file scope.
286+
287+
Use this pattern only when the workflow is supposed to own repository delivery:
288+
289+
1. Preflight the git state and fail on unexpected staged changes.
290+
2. Create or verify the intended branch.
291+
3. Run implementation, review, soft validation, fix, and hard validation gates.
292+
4. Stage only the declared target files and review/signoff artifacts.
293+
5. Commit with a deterministic message.
294+
6. Push the branch.
295+
7. Use the GitHub primitive or `gh pr create` to open the PR.
296+
8. Verify the PR URL/state deterministically and write it into the final signoff artifact.
297+
298+
Do not hide commit/PR work in agent prose. Model it as deterministic steps whenever possible. If using an agent or GitHub primitive for PR creation, the downstream hard gate must still verify the PR exists before signoff.
299+
300+
If commit or PR creation is intentionally outside the workflow, say that directly in the workflow description and signoff so the operator knows to do it after completion.
301+
283302
### 2. Avoid raw fenced code blocks inside workflow task template literals
284303

285304
Raw triple-backtick code fences inside large inline `task: \`...\`` template strings are fragile and can break outer TypeScript parsing, especially when they contain language tags like `swift` or `diff`.

0 commit comments

Comments
 (0)