Skip to content

Simplify comment tool instructions in prompt - #1328

Merged
ashwin-ant merged 1 commit into
mainfrom
ashwin/update-comment-tool-instructions
May 19, 2026
Merged

Simplify comment tool instructions in prompt#1328
ashwin-ant merged 1 commit into
mainfrom
ashwin/update-comment-tool-instructions

Conversation

@ashwin-ant

Copy link
Copy Markdown
Collaborator

Summary

  • Replace the <comment_tool_info> block in the tag-mode prompt with a single concise instruction line for updating the comment.

Test plan

  • bun run typecheck
  • bun test test/create-prompt

@ashwin-ant
ashwin-ant requested a review from blois May 19, 2026 21:34

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — small prompt-text-only simplification with no logic or security implications.

Extended reasoning...

Overview

This PR replaces a multi-line <comment_tool_info> block in the tag-mode prompt (src/create-prompt/index.ts) with a single concise instruction line telling the agent to use mcp__github_comment__update_claude_comment and to load it with ToolSearch first. It is a 1-file, prompt-text-only change.

Security risks

None. The change does not touch authentication, permissions, tool allowlists, sanitization, or any code path that processes user input. The MCP tool name itself remains in the allowed-tools list elsewhere in the same file, so capability scope is unchanged.

Level of scrutiny

Low. Prompt wording changes can affect agent behavior, but the worst plausible outcome is a wasted ToolSearch call or a slightly less verbose instruction set — not a correctness or security regression. The new wording aligns with the SDK's deferred-tool/ToolSearch loading pattern. Type-check and the create-prompt test suite pass per the test plan.

Other factors

This continues the recent trend of trimming the tag-mode prompt (cf. the simplified-prompt work in earlier commits). The change is self-contained and intent is clear. No bugs were flagged by the bug hunting system.

@ashwin-ant
ashwin-ant merged commit fd1877d into main May 19, 2026
40 checks passed
@ashwin-ant
ashwin-ant deleted the ashwin/update-comment-tool-instructions branch May 19, 2026 21:50
mergify Bot added a commit to ArcadeData/arcadedb-usecases that referenced this pull request May 24, 2026
…33 in the github-actions group [skip ci]

Bumps the github-actions group with 1 update: [anthropics/claude-code-action](https://github.com/anthropics/claude-code-action).
Updates `anthropics/claude-code-action` from 1.0.123 to 1.0.133
Release notes

*Sourced from [anthropics/claude-code-action's releases](https://github.com/anthropics/claude-code-action/releases).*

> v1.0.133
> --------
>
> What's Changed
> --------------
>
> * Use workload identity federation for Claude auth in CI workflows by [`@​ashwin-ant`](https://github.com/ashwin-ant) in [anthropics/claude-code-action#1344](https://redirect.github.com/anthropics/claude-code-action/pull/1344)
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.133>
>
> v1.0.132
> --------
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.132>
>
> v1.0.131
> --------
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.131>
>
> v1.0.130
> --------
>
> What's Changed
> --------------
>
> * Add Workload Identity Federation (OIDC) authentication support by [`@​ashwin-ant`](https://github.com/ashwin-ant) in [anthropics/claude-code-action#1338](https://redirect.github.com/anthropics/claude-code-action/pull/1338)
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.130>
>
> v1.0.129
> --------
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.129>
>
> v1.0.128
> --------
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.128>
>
> v1.0.127
> --------
>
> What's Changed
> --------------
>
> * Refactor allowed\_bots actor resolution by [`@​ashwin-ant`](https://github.com/ashwin-ant) in [anthropics/claude-code-action#1330](https://redirect.github.com/anthropics/claude-code-action/pull/1330)
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.127>
>
> v1.0.126
> --------
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.126>
>
> v1.0.125
> --------
>
> What's Changed
> --------------
>
> * Simplify comment tool instructions in prompt by [`@​ashwin-ant`](https://github.com/ashwin-ant) in [anthropics/claude-code-action#1328](https://redirect.github.com/anthropics/claude-code-action/pull/1328)
>
> **Full Changelog**: <anthropics/claude-code-action@v1...v1.0.125>
>
> v1.0.124
> --------
>
> What's Changed
> --------------
>
> * fix: add parentheses to fix operator precedence in co-author check by [`@​FuturizeRush`](https://github.com/FuturizeRush) in [anthropics/claude-code-action#1199](https://redirect.github.com/anthropics/claude-code-action/pull/1199)
> * Strengthen simplified tag-mode prompt (USE\_SIMPLE\_PROMPT) by [`@​ashwin-ant`](https://github.com/ashwin-ant) in [anthropics/claude-code-action#1313](https://redirect.github.com/anthropics/claude-code-action/pull/1313)
> * Fix prettier formatting in create-prompt by [`@​ashwin-ant`](https://github.com/ashwin-ant) in [anthropics/claude-code-action#1325](https://redirect.github.com/anthropics/claude-code-action/pull/1325)
>
> New Contributors
> ----------------

... (truncated)


Commits

* [`787c5a0`](anthropics/claude-code-action@787c5a0) chore: bump Claude Code to 2.1.150 and Agent SDK to 0.3.150
* [`4257c8e`](anthropics/claude-code-action@4257c8e) Use workload identity federation for Claude auth in CI workflows ([#1344](https://redirect.github.com/anthropics/claude-code-action/issues/1344))
* [`bbfaf8e`](anthropics/claude-code-action@bbfaf8e) chore: bump Claude Code to 2.1.149 and Agent SDK to 0.3.149
* [`4481e6d`](anthropics/claude-code-action@4481e6d) chore: bump Claude Code to 2.1.148 and Agent SDK to 0.3.148
* [`661a6fe`](anthropics/claude-code-action@661a6fe) Add Workload Identity Federation (OIDC) authentication support ([#1338](https://redirect.github.com/anthropics/claude-code-action/issues/1338))
* [`c9d66af`](anthropics/claude-code-action@c9d66af) chore: bump Claude Code to 2.1.147 and Agent SDK to 0.3.147
* [`20c8abf`](anthropics/claude-code-action@20c8abf) chore: bump Claude Code to 2.1.146 and Agent SDK to 0.3.146
* [`1dc994e`](anthropics/claude-code-action@1dc994e) Resolve actor account type before applying allowed\_bots ([#1330](https://redirect.github.com/anthropics/claude-code-action/issues/1330))
* [`ca89df3`](anthropics/claude-code-action@ca89df3) chore: bump Claude Code to 2.1.145 and Agent SDK to 0.3.145
* [`fd1877d`](anthropics/claude-code-action@fd1877d) Simplify comment tool instructions in prompt ([#1328](https://redirect.github.com/anthropics/claude-code-action/issues/1328))
* Additional commits viewable in [compare view](anthropics/claude-code-action@51ea8ea...787c5a0)
  
[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=anthropics/claude-code-action&package-manager=github\_actions&previous-version=1.0.123&new-version=1.0.133)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
Dependabot commands and options
  
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot show  ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore  major version` will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
- `@dependabot ignore  minor version` will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
- `@dependabot ignore ` will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
- `@dependabot unignore ` will remove all of the ignore conditions of the specified dependency
- `@dependabot unignore  ` will remove the ignore condition of the specified dependency and ignore conditions
mergify Bot added a commit to ArcadeData/arcadedb that referenced this pull request May 24, 2026