Skip to content

feat: mention @mtrunkat in the review summary on findings#12

Merged
mtrunkat merged 1 commit into
mainfrom
claude/mongodb-query-index-check-mention-mtrunkat
May 18, 2026
Merged

feat: mention @mtrunkat in the review summary on findings#12
mtrunkat merged 1 commit into
mainfrom
claude/mongodb-query-index-check-mention-mtrunkat

Conversation

@mtrunkat
Copy link
Copy Markdown
Member

Summary

One-line change to prompts/review.md: tell Claude to end the review summary body with cc @mtrunkat so they get a GitHub notification for every PR where the action finds something.

Behaviour

  • No findings → no review posted → no mention. The prompt's silence rule still applies; nothing changes for clean PRs.
  • One mention per review, not per finding. Goes in the summary body that wraps the inline comments, so a 5-finding review still produces a single notification.

Test plan

  • pnpm run lint / type-check / test — clean, 35/35
  • No new envsubst placeholders (the string is literal in the prompt)
  • Real run: trigger on a PR with a known finding and confirm the review summary contains cc @mtrunkat.

Generated by Claude Code

Instruct Claude to end the review summary body with `cc @mtrunkat` when
a review is posted, so the reviewer gets a notification for every PR
where the action finds something. Only fires when there's at least one
finding (no review = no mention), keeping the noise floor to a single
ping per review (not per inline comment).
Copilot AI review requested due to automatic review settings May 18, 2026 09:11
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an instruction to the MongoDB query index check review prompt so that Claude appends cc @mtrunkat to the summary body of any posted review, ensuring a GitHub notification for every PR where findings are reported.

Changes:

  • Append guidance to end the review summary body with cc @mtrunkat.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mtrunkat mtrunkat merged commit 2f0becd into main May 18, 2026
6 checks passed
@mtrunkat mtrunkat deleted the claude/mongodb-query-index-check-mention-mtrunkat branch May 18, 2026 09:24
fnesveda pushed a commit that referenced this pull request May 18, 2026
🤖 I have created a release *beep* *boop*
---


## [1.1.0](v1.0.0...v1.1.0)
(2026-05-18)


### Features

* add mongodb-query-index-check action
([#3](#3))
([e288951](e288951))
* add python-package-check composite action
([#11](#11))
([cafe9c0](cafe9c0))
* bump max-turns default to 100 and stream full Claude output
([#7](#7))
([812c5cb](812c5cb))
* expand allowed-tools list for mongodb-query-index-check
([#6](#6))
([42e0fe2](42e0fe2))
* make the review prompt directive instead of descriptive
([#8](#8))
([910af2a](910af2a))
* mention [@mtrunkat](https://github.com/mtrunkat) in the review summary
on findings ([#12](#12))
([2f0becd](2f0becd))


### Bug Fixes

* move state files into workspace and address bash sandbox denials
([#9](#9))
([6e2aa05](6e2aa05))
* Stop using `@octokit/rest` in scripts
([#10](#10))
([232b613](232b613))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.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.

4 participants