Skip to content

docs: document include_comments_by_actor and exclude_comments_by_actor#990

Open
yoheidemachi wants to merge 3 commits intoanthropics:mainfrom
yoheidemachi:docs/document-comment-filtering
Open

docs: document include_comments_by_actor and exclude_comments_by_actor#990
yoheidemachi wants to merge 3 commits intoanthropics:mainfrom
yoheidemachi:docs/document-comment-filtering

Conversation

@yoheidemachi
Copy link

Summary

Closes #972

  • Add include_comments_by_actor and exclude_comments_by_actor to the inputs table in docs/usage.md with descriptions, defaults, and wildcard support
  • Add a new "Comment Filtering" section in docs/usage.md with precedence rules and three practical examples (exclude bots, allowlist users, exclude specific bot)
  • Add comment filtering guidance to the "Prompt Injection Risks" section in docs/security.md, referencing include_comments_by_actor as a mitigation for public repos

Test plan

  • Verify links between security.md and usage.md#comment-filtering resolve correctly
  • Confirm parameter descriptions match action.yml definitions
  • Verify precedence rule (exclusion wins) matches src/github/utils/actor-filter.ts implementation

🤖 Generated with Claude Code

Yohei Demachi and others added 3 commits March 1, 2026 05:22
Add both parameters to the Inputs table in usage.md with descriptions,
defaults, and wildcard support. Add a Comment Filtering section with
examples for common use cases. Mention comment filtering in the Prompt
Injection Risks section of security.md as an additional mitigation for
public repositories.

Closes anthropics#972

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Rename "Include all bots except a specific one" to "Exclude a specific
bot" since the example only excludes renovate[bot] from the default
(include-all) behavior.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document include_comments_by_actor and exclude_comments_by_actor in usage.md and security.md

1 participant