mirror of
https://github.com/mattermost/mattermost.git
synced 2026-08-27 05:37:15 -05:00
Eva Sarafianou
fee649d063
* Run docs-impact-review as regular CI instead of slash command Switch the Documentation Impact Review workflow from a `/docs-review` slash command trigger to running automatically on pull_request events. The analysis posts a sticky PR comment and inline comments, and adds a `docs/needed` label when documentation changes are needed. Made-with: Cursor * Address review feedback: harden docs-impact-review workflow - Remove invalid `direct_prompt` input not recognized by the pinned claude-code-action version (auto-detects mode from event type). - Guard Claude step with secret availability check so fork and Dependabot PRs skip gracefully instead of failing. - Move label management out of the LLM into a deterministic github-script step that parses the sticky comment, eliminating prompt-injection and excessive-agency risks. Made-with: Cursor * Fix secret check: use job-level env var instead of secrets context in step if The secrets context is not available in step-level if expressions. Evaluate it as a job-level env var and reference that instead. Made-with: Cursor * Use paginated API call to fetch all PR comments Prevents missing the bot comment on busy PRs with more than 30 comments. Made-with: Cursor * Address review: add issues:write permission, remove auto-label-removal - Upgrade issues permission from read to write (required for addLabels API) - Remove automatic docs/needed label removal to avoid stripping the label prematurely when the analysis comment hasn't been posted yet Made-with: Cursor