Skip to content

Commit 15166f3

Browse files
Copilotpelikhan
andcommitted
Fix code review feedback - remove tool references and update docs
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
1 parent 6e7a00c commit 15166f3

2 files changed

Lines changed: 14 additions & 20 deletions

File tree

docs/q.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,9 @@ To trigger the workflow on a specific issue or pull request, add a comment with
5050

5151
## Configuration
5252

53-
This workflow requires no configuration and works out of the box. You can customize optimization behavior and analysis scope. Local configuration can be done in `.github/workflows/agentics/q.config.md`.
53+
This workflow requires no configuration and works out of the box. You can customize optimization behavior and analysis scope if needed.
5454

55-
After editing run `gh aw compile` to update the workflow and commit all changes to the default branch.
55+
After editing the workflow file, recompile and commit all changes to the default branch.
5656

5757
## What it reads from GitHub
5858

workflows/q.md

Lines changed: 12 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -67,10 +67,9 @@ This workflow was triggered from a comment on issue #${{ github.event.issue.numb
6767

6868
**Important**: Before proceeding with your analysis, retrieve the full issue details to understand the context of the work to be done:
6969

70-
1. Use the `issue_read` tool with method `get` to fetch issue #${{ github.event.issue.number }}
71-
2. Review the issue title, body, and labels to understand what workflows or problems are being discussed
72-
3. Consider any linked issues or previous comments for additional context
73-
4. Use this issue context to inform your investigation and recommendations
70+
1. Read the issue title, body, and labels to understand what workflows or problems are being discussed
71+
2. Consider any linked issues or previous comments for additional context
72+
3. Use this issue context to inform your investigation and recommendations
7473
{{/if}}
7574

7675
{{#if ${{ github.event.pull_request.number }} }}
@@ -80,10 +79,9 @@ This workflow was triggered from a comment on pull request #${{ github.event.pul
8079

8180
**Important**: Before proceeding with your analysis, retrieve the full PR details to understand the context of the work to be done:
8281

83-
1. Use the `pull_request_read` tool with method `get` to fetch PR #${{ github.event.pull_request.number }}
84-
2. Review the PR title, description, and changed files to understand what changes are being proposed
85-
3. Consider the PR's relationship to workflow optimizations or issues
86-
4. Use this PR context to inform your investigation and recommendations
82+
1. Review the PR title, description, and changed files to understand what changes are being proposed
83+
2. Consider the PR's relationship to workflow optimizations or issues
84+
3. Use this PR context to inform your investigation and recommendations
8785
{{/if}}
8886

8987
{{#if ${{ github.event.discussion.number }} }}
@@ -93,10 +91,9 @@ This workflow was triggered from a comment on discussion #${{ github.event.discu
9391

9492
**Important**: Before proceeding with your analysis, retrieve the full discussion details to understand the context of the work to be done:
9593

96-
1. Use GitHub tools to fetch discussion #${{ github.event.discussion.number }}
97-
2. Review the discussion title and body to understand the topic being discussed
98-
3. Consider the discussion context when planning your workflow optimizations
99-
4. Use this discussion context to inform your investigation and recommendations
94+
1. Review the discussion title and body to understand the topic being discussed
95+
2. Consider the discussion context when planning your workflow optimizations
96+
3. Use this discussion context to inform your investigation and recommendations
10097
{{/if}}
10198
</current_context>
10299

@@ -164,17 +161,14 @@ Based on your analysis, make targeted improvements to workflow files:
164161

165162
If logs show missing tool reports:
166163
- Add the tools to the appropriate workflow frontmatter
167-
- Ensure proper MCP server configuration
168164
- Add shared imports if the tool has a standard configuration
169165

170166
Example:
171167
```yaml
172168
tools:
173-
github:
174-
allowed:
175-
- issue_read
176-
- list_commits
177-
- create_issue_comment
169+
web-search:
170+
bash:
171+
edit:
178172
```
179173
180174
#### 4.2 Fix Permission Issues

0 commit comments

Comments
 (0)