@henryqw/pi-pr 3.1.1 → 3.1.3

package/README.md

@@ -21,10 +21,13 @@ keeps the complete JSON output. Read one item with
 `pr-feedback.mjs show --snapshot FILE --id ID`.
 
 `show` prints one JSON record with its exact stored body and fields. A thread
-record includes child IDs without child bodies. A nested comment includes only
-its parent thread's ID, state, and location. Missing, unknown, duplicate, or
-ambiguous IDs fail. `show` does not run Git, call GitHub, use the network, or
-write files.
+record includes child IDs without child bodies. Treat that record as a container.
+Inspect the `thread_comment` IDs directly. Do not call `show` on the parent only
+to find children. Show the parent only when it has no child, or when you need
+parent-level metadata. Issue independent `show` lookups in one tool-call round.
+A nested comment includes only its parent thread's ID, state, and location.
+Missing, unknown, duplicate, or ambiguous IDs fail. `show` does not run Git, call
+GitHub, use the network, or write files.
 
 ## Works with
 
@@ -34,11 +37,13 @@ write files.
 
 ## Use
 
-Run `/pr` without arguments in a GitHub checkout. It reads the current branch pull request and local state, then runs one route. The PR hostname selects its GitHub API host, and the extension works outside Herdr.
+Run `/pr` in a GitHub checkout. It reads the current branch pull request and local state, then runs one route. The PR hostname selects its GitHub API host, and the extension works outside Herdr.
+
+Add optional instructions to guide a creation, branch-update, CI-fix, or feedback workflow. For example, run `/pr keep the title under 50 characters`. Direct routes, such as linking or merging, reject instructions instead of ignoring them.
 
 | Surface | Type | Purpose |
 | --- | --- | --- |
-| `/pr` | command | Run the current pull request's next safe route. |
+| `/pr [instructions]` | command | Run the current pull request's next safe route. |
 | Footer | ui | Show a linked `PR #number` and one plain-language status. |
 | Widget | ui | Show one actionable icon-prefixed `Run /pr to …` hint. |
 
@@ -104,7 +109,7 @@ The comment sweep resolves its bundled helper and references from the installed
 
 The footer and widget load at session start. A directory outside a Git worktree stays silent and does not start polling. The UI shows `PR · status unavailable` for other discovery failures and reports only a generic error.
 
-They refresh after local commits, PR creation, pushes, and each dispatched workflow settles. They also refresh after any successful delegated task settles. Active Git worktrees poll every 30 seconds. Polling updates presentation only and may be stale.
+They refresh after local commits, PR creation, pushes, and each dispatched workflow settles. During creation, intermediate refreshes wait until the workflow settles. They also refresh after any successful delegated task settles. Active Git worktrees poll every 30 seconds. Polling updates presentation only and may be stale.
 
 The create widget stays hidden until the local branch has a commit beyond its creation point. Any displayed widget clears as soon as `/pr` starts. A dispatched workflow keeps it hidden until the agent settles. A direct merge, no-action route, or failed command refreshes the widget when the handler finishes.
 

package/extensions/pr-command.ts

@@ -34,7 +34,12 @@ type PrCommandDependencies = {
 	linkInferredPullRequest?: typeof linkInferredPullRequest;
 };
 
-function dispatchWorkflow(pi: PrCommandPi, ctx: ExtensionCommandContext, commandName: string): void {
+function dispatchWorkflow(
+	pi: PrCommandPi,
+	ctx: ExtensionCommandContext,
+	commandName: string,
+	instructions: string,
+): void {
 	const command = pi.getCommands().find((candidate) =>
 		candidate.name === commandName &&
 		candidate.source === "skill" &&
@@ -45,7 +50,7 @@ function dispatchWorkflow(pi: PrCommandPi, ctx: ExtensionCommandContext, command
 	const options = ctx.isIdle()
 		? { expandPromptTemplates: true }
 		: { deliverAs: "followUp" as const, expandPromptTemplates: true };
-	pi.sendUserMessage(`/${command.name}`, options);
+	pi.sendUserMessage(`/${command.name}${instructions ? ` ${instructions}` : ""}`, options);
 }
 
 function noActionNotification(pullRequest: CurrentPullRequest): { message: string; type: "info" | "warning" } {
@@ -171,10 +176,12 @@ export function createPrCommandHandler(
 	const load = dependencies.loadCurrentPullRequest ?? loadCurrentPullRequest;
 	const link = dependencies.linkInferredPullRequest ?? linkInferredPullRequest;
 	return async (args, ctx) => {
-		if (args.trim()) throw new Error("/pr does not accept arguments");
-
+		const instructions = args.trim();
 		const discovery = await load(pi, ctx);
 		const nextStep = deriveNextStep(discovery);
+		if (instructions && !(nextStep in WORKFLOWS)) {
+			throw new Error("The current /pr route does not accept instructions");
+		}
 		if (discovery.kind === "inactive") return nextStep;
 		if (discovery.kind === "blocked") {
 			return nextStep;
@@ -197,7 +204,7 @@ export function createPrCommandHandler(
 		}
 
 		if (!(nextStep in WORKFLOWS)) throw new Error(`/pr cannot dispatch route ${nextStep}`);
-		dispatchWorkflow(pi, ctx, WORKFLOWS[nextStep as WorkflowNextStep]);
+		dispatchWorkflow(pi, ctx, WORKFLOWS[nextStep as WorkflowNextStep], instructions);
 		return nextStep;
 	};
 }

package/extensions/pr.ts

@@ -203,7 +203,7 @@ export default function pullRequestExtension(
 
 	const refresh = async (): Promise<void> => {
 		const ctx = context;
-		if (!ctx) return;
+		if (!ctx || [...activeInvocations.values()].includes("create-workflow")) return;
 		const generation = sessionGeneration;
 		if (active) {
 			queued = true;
@@ -320,7 +320,7 @@ export default function pullRequestExtension(
 
 	const commandHandler = createCommandHandler(pi);
 	pi.registerCommand("pr", {
-		description: "Run the current branch pull request next step",
+		description: "[instructions] — Run the current branch pull request next step",
 		handler: async (args, ctx) => {
 			if (!ctx.hasUI || !context) return;
 			const generation = sessionGeneration;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-pr",
-  "version": "3.1.1",
+  "version": "3.1.3",
   "description": "Run /pr to safely discover or link the current pull request, then create, update, address feedback, fix CI, or merge when ready.",
   "keywords": [
     "pi-package",

package/skills/pi-pr-comment-sweep/SKILL.md

@@ -33,13 +33,17 @@ Run standalone; never invoke, defer to, or modify Shipyard.
 
 2. Initial fetch prints a compact index of every feedback ID. Later fetches into
    a copied snapshot print compact indexes of additions, edits, and state changes.
-   The snapshot remains complete. For each non-obvious item, inspect only that item:
+   The snapshot remains complete. A thread record with child IDs is a container:
+   inspect `thread_comment` IDs directly. Do not call `show` on the parent solely
+   to discover its children. Show the parent only if it has no child or parent-level
+   metadata is actually needed. For each non-obvious item, inspect only that item:
 
    ```bash
    node "<skill>/scripts/pr-feedback.mjs" show --snapshot "$SNAPSHOT" --id "$ID"
    ```
 
-   Always inspect every unresolved feedback item before triage. Never
+   Issue independent `show` lookups in one tool-call round. Always inspect every
+   unresolved feedback item before triage. Never
    read or print the whole raw snapshot when `show` provides the bounded lookup.
    Follow [Thread triage](<skill>/references/thread-triage.md>) and ledger every item as
    `actionable`, `non-actionable`, or `blocked`, with terse evidence, smallest