Skip to content

Commit e4c2893

Browse files
codepuncherCopilot
andcommitted
docs: clarify PR comment workflow sequence
Make it explicit that PR comment management is a two-step process: 1. Get comments first (to retrieve thread IDs) 2. Use those thread IDs to reply/resolve Changes: - Add 'Workflow Overview' section in tools.md - Add step numbers and explanatory notes - Clarify that thread-id comes from step 1 output - Add workflow summary to quick-reference.md This prevents confusion about where thread IDs come from and makes the command sequence clear. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 1f03a2e commit e4c2893

4 files changed

Lines changed: 20 additions & 9 deletions

File tree

shell/copilot-instructions/quick-reference.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,9 +112,11 @@ wp @production cache flush # Write operation: be careful
112112

113113
## PR Comment Management
114114

115+
**Workflow:** Get comments first (to find thread IDs), then reply/resolve using those IDs.
116+
115117
### Get Comments
116118
```bash
117-
# All comments in pretty format
119+
# Step 1: Get all comments (includes thread IDs in output)
118120
~/Code/misc/itineris-bin/gh-pr-get-comments <pr-number>
119121

120122
# Filter by author
@@ -130,9 +132,9 @@ wp @production cache flush # Write operation: be careful
130132

131133
### Reply to Comments
132134
```bash
133-
# Reply to thread
135+
# Step 2: Reply using thread-id from step 1
134136
~/Code/misc/itineris-bin/gh-pr-reply-to-thread <pr-number> \
135-
--thread-id='PRRT_kwDOABCDEF' \
137+
--thread-id='PRRT_xxx' \
136138
--message='Fixed in commit abc1234'
137139

138140
# Reply and resolve

shell/copilot-instructions/tools.md

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,17 @@ Scripts, utilities, and tools available for Itineris development workflows.
44

55
## GitHub PR Comment Management
66

7-
### Get PR Comments
7+
### Workflow Overview
88

9-
Use `gh-pr-get-comments` script to retrieve all comments on a PR:
9+
1. **First:** Get PR comments to see thread IDs
10+
2. **Then:** Reply/resolve specific threads using the thread IDs from step 1
11+
12+
### Step 1: Get PR Comments
13+
14+
Use `gh-pr-get-comments` to retrieve all comments and their thread IDs:
1015

1116
```bash
12-
# Get all comments in pretty format
17+
# Get all comments in pretty format (includes thread IDs)
1318
~/Code/misc/itineris-bin/gh-pr-get-comments <pr-number>
1419

1520
# Filter by author (e.g., Copilot bot)
@@ -22,9 +27,11 @@ Use `gh-pr-get-comments` script to retrieve all comments on a PR:
2227
~/Code/misc/itineris-bin/gh-pr-get-comments <pr-number> --format=json
2328
```
2429

25-
### Reply to PR Comments
30+
**Note:** The output includes `thread-id` values (e.g., `PRRT_kwDOABCDEF`) which you need for the next step.
31+
32+
### Step 2: Reply to PR Comments
2633

27-
Use `gh-pr-reply-to-thread` script to reply and resolve review threads:
34+
Use `gh-pr-reply-to-thread` with the thread-id from step 1:
2835

2936
```bash
3037
# Reply to a review thread (use thread-id from gh-pr-get-comments output)

shell/functions

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,8 @@ install_wsl_packages() {
203203
install_git_delta
204204
# ensure fd is available
205205
ln -sf "$(which fdfind)" ~/.local/bin/fd
206+
# ensure bat is available
207+
ln -sf "$(which batcat)" ~/.local/bin/bat
206208
}
207209

208210
setup_arch_services() {

terminals/alacritty.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ decorations = "None"
5050
decorations_theme_variant = "Dark"
5151
dynamic_padding = true
5252
dynamic_title = true
53-
opacity = 0.75
53+
opacity = 1
5454
startup_mode = "Maximized"
5555

5656
[window.padding]

0 commit comments

Comments
 (0)