fix: The use_sticky_commentoption does not reference the input bot_id.
#651
+1
−3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ad-m-ss
reviewed
Oct 30, 2025
| }); | ||
| const existingComment = comments.data.find((comment) => { | ||
| const idMatch = comment.user?.id === CLAUDE_APP_BOT_ID; | ||
| const idMatch = comment.user?.id === CLAUDE_APP_BOT_ID || comment.user?.id === parseInt(context.inputs.botId); |
There was a problem hiding this comment.
Suggested change
| const idMatch = comment.user?.id === CLAUDE_APP_BOT_ID || comment.user?.id === parseInt(context.inputs.botId); | |
| const idMatch = comment.user?.id === parseInt(context.inputs.botId); |
Why is there a need for a hardcoded constant if that value is also defined in https://github.com/huuyafwww/claude-code-action/blob/87cf21a4f276c5c71a16d388a981a6f16ba7af13/action.yml#L83 ? If there are no hardcoded constants, there won't be such mistakes, and there will only be one way for it to work.
There was a problem hiding this comment.
BTW. That constant is duplicated:
,There was a problem hiding this comment.
There are two different numbers, so never mind.
Author
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello, thank you for developing such an excellent library.
I am currently using this action in my project.
I'm utilizing the
use_sticky_commentoption, buta new comment is created with every commit.
Upon investigating the cause, I found that the bot user commenting in my project was not Claude.
Therefore, the
use_sticky_commentoption is not functioning properly.When Claude runs, it first comments “Claude Code is working…”, right?
It reflects the execution results by updating this comment.
However, when a new commit is added, it cannot find an
existingComment, so it creates a new comment again.In my project, the GitHub Actions official user comments,
but the current code only considers comments made by the official Claude App user.
Therefore, I modified it so that if the botId in the actions input matches, it is also recognized as an
existingComment.This allows
existingCommentsto be found even when users other than the official Claude App comment.Once this change is reflected, my project will specify
botIdas41898282.Below is the information for the GitHub Actions bot.
https://api.github.com/users/github-actions[bot]