@henryqw/pi-pr 3.1.2 → 3.1.3
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +8 -5
- package/extensions/pr.ts +1 -1
- package/package.json +1 -1
- package/skills/pi-pr-comment-sweep/SKILL.md +6 -2
package/README.md
CHANGED
|
@@ -21,10 +21,13 @@ keeps the complete JSON output. Read one item with
|
|
|
21
21
|
`pr-feedback.mjs show --snapshot FILE --id ID`.
|
|
22
22
|
|
|
23
23
|
`show` prints one JSON record with its exact stored body and fields. A thread
|
|
24
|
-
record includes child IDs without child bodies.
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
24
|
+
record includes child IDs without child bodies. Treat that record as a container.
|
|
25
|
+
Inspect the `thread_comment` IDs directly. Do not call `show` on the parent only
|
|
26
|
+
to find children. Show the parent only when it has no child, or when you need
|
|
27
|
+
parent-level metadata. Issue independent `show` lookups in one tool-call round.
|
|
28
|
+
A nested comment includes only its parent thread's ID, state, and location.
|
|
29
|
+
Missing, unknown, duplicate, or ambiguous IDs fail. `show` does not run Git, call
|
|
30
|
+
GitHub, use the network, or write files.
|
|
28
31
|
|
|
29
32
|
## Works with
|
|
30
33
|
|
|
@@ -106,7 +109,7 @@ The comment sweep resolves its bundled helper and references from the installed
|
|
|
106
109
|
|
|
107
110
|
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.
|
|
108
111
|
|
|
109
|
-
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.
|
|
112
|
+
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.
|
|
110
113
|
|
|
111
114
|
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.
|
|
112
115
|
|
package/extensions/pr.ts
CHANGED
|
@@ -203,7 +203,7 @@ export default function pullRequestExtension(
|
|
|
203
203
|
|
|
204
204
|
const refresh = async (): Promise<void> => {
|
|
205
205
|
const ctx = context;
|
|
206
|
-
if (!ctx) return;
|
|
206
|
+
if (!ctx || [...activeInvocations.values()].includes("create-workflow")) return;
|
|
207
207
|
const generation = sessionGeneration;
|
|
208
208
|
if (active) {
|
|
209
209
|
queued = true;
|
package/package.json
CHANGED
|
@@ -33,13 +33,17 @@ Run standalone; never invoke, defer to, or modify Shipyard.
|
|
|
33
33
|
|
|
34
34
|
2. Initial fetch prints a compact index of every feedback ID. Later fetches into
|
|
35
35
|
a copied snapshot print compact indexes of additions, edits, and state changes.
|
|
36
|
-
The snapshot remains complete.
|
|
36
|
+
The snapshot remains complete. A thread record with child IDs is a container:
|
|
37
|
+
inspect `thread_comment` IDs directly. Do not call `show` on the parent solely
|
|
38
|
+
to discover its children. Show the parent only if it has no child or parent-level
|
|
39
|
+
metadata is actually needed. For each non-obvious item, inspect only that item:
|
|
37
40
|
|
|
38
41
|
```bash
|
|
39
42
|
node "<skill>/scripts/pr-feedback.mjs" show --snapshot "$SNAPSHOT" --id "$ID"
|
|
40
43
|
```
|
|
41
44
|
|
|
42
|
-
|
|
45
|
+
Issue independent `show` lookups in one tool-call round. Always inspect every
|
|
46
|
+
unresolved feedback item before triage. Never
|
|
43
47
|
read or print the whole raw snapshot when `show` provides the bounded lookup.
|
|
44
48
|
Follow [Thread triage](<skill>/references/thread-triage.md>) and ledger every item as
|
|
45
49
|
`actionable`, `non-actionable`, or `blocked`, with terse evidence, smallest
|