Skip to content

Commit c72366a

Browse files
committed
List required env vars for fetch_reviews.sh in SKILL.md
The `Get the reviews` section previously said only "Replace `$VARIABLES` with the actual values or variables in the command", which left readers to open `fetch_reviews.sh` to discover which variables the script actually expects. Replace that single line with two complementary blocks: - A one-line invocation example pre-filling all five required variables (`PR_NUMBER`, `NUMBER_OF_PR_COMMENTS`, `NUMBER_OF_REVIEWS`, `NUMBER_OF_THREADS`, `NUMBER_OF_COMMENTS_PER_THREAD`) plus the optional `LAST_CURSOR`. - A bullet list explaining what each variable controls. The script itself is unchanged — its fail-fast guards already cover missing values. Addresses: - #765 (comment) Assisted-by: Claude Code:claude-opus-4-7
1 parent c479e9c commit c72366a

1 file changed

Lines changed: 19 additions & 1 deletion

File tree

.agents/skills/get-reviews/SKILL.md

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,28 @@ Use the GraphQL API to fetch the reviews for a specific pull request.
3232
Check [fetch\_reviews.sh](./fetch_reviews.sh) to fetch the reviews
3333
and save them in a JSON file:
3434

35-
- Replace `$VARIABLES` with the actual values or variables in the command.
35+
- Fill the variables in the command and run the command to fetch the reviews:
36+
37+
~~~~ bash
38+
# Omit the new lines
39+
PR_NUMBER= NUMBER_OF_PR_COMMENTS= NUMBER_OF_REVIEWS= \
40+
NUMBER_OF_THREADS= NUMBER_OF_COMMENTS_PER_THREAD= LAST_CURSOR= \
41+
bash .agents/skills/get-reviews/fetch_reviews.sh
42+
~~~~
43+
44+
45+
- `PR_NUMBER`: The number of the pull request to fetch reviews from.
46+
- `NUMBER_OF_PR_COMMENTS`: The number of PR comments to fetch.
47+
- `NUMBER_OF_REVIEWS`: The number of reviews to fetch.
48+
- `NUMBER_OF_THREADS`: The number of review threads to fetch.
49+
- `NUMBER_OF_COMMENTS_PER_THREAD`:
50+
The number of comments per review thread to fetch.
51+
- `LAST_CURSOR`: The cursor for incremental fetches. Optional.
52+
3653
- For incremental fetches, save `pageInfo.endCursor` from the previous
3754
fetch and pass it as `after: $LAST_CURSOR` (a base64 cursor, not a
3855
review thread node ID) so the query returns only new review threads.
56+
3957
- Use `jq` to filter the reviews and information if necessary.
4058

4159
The fetched JSON files in *plans/{PR\_NUMBER}/fetched/* contain the raw data

0 commit comments

Comments
 (0)