MOB-51822: Add Slack ready-for-review notification to prisma-taurus skill#2004
Merged
Merged
Conversation
…rus skill Mirrors Blazemeter/taurus-cloud#789. Adds a best-effort final Slack chat.postMessage summary (new step 17) after Copilot triage, registers optional SLACK_BOT_TOKEN / SLACK_SPARTA_CHANNEL_ID env vars, and renumbers the trailing steps (Report 17->18, Final summary 18->19). The post is best-effort: it never fails, blocks, or errors the run, skips cleanly if unconfigured, and only fires when a PR was created. Doc-only change. Jira: MOB-51822 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Jira: MOB-51822
Skill-documentation change to
.claude/skills/prisma-taurus/SKILL.md. No product code, Dockerfile, or requirements changes — only theSKILL.mdfile. Mirrors the sibling change made to the taurus-cloud CVE skills (MOB-51821, Blazemeter/taurus-cloud#789).What this adds
A new step 17 that posts a single Slack
chat.postMessage"ready for review" summary once the PR exists and Copilot's one triage round is done (or timed out) — signalling the PR is basically ready for human review.ok:falseare all caught.SLACK_BOT_TOKEN/SLACK_SPARTA_CHANNEL_ID; skips cleanly if either is unset. The three required creds still hard-stop.Y), grouped by package with CVEs listed vertically; the count is computed from the fixed list.🛠️ Prisma CVE fixes ready for review — taurus #<PR> · N CVEs fixed · <MOB-XXXXX>. Byte-identical to the taurus-cloud version except the repo/link wording (taurus #<PR>) and thePrismascan-type tag.&<>) in dynamic text so version constraints (e.g.<13.0.0) render literally; drops the Jira segment entirely when no ticket exists (headless/cron).Placement
Inserted after the Copilot-triage step (step 16). Fires on every exit path, including the Copilot-timeout path (step 16a's "go to step 17" now lands on it). Trailing steps renumbered (Report 17→18, Final summary 18→19) and a
Slack:outcome line added to the final summary.Verification
superpowers:requesting-code-review— no Critical/Important findings; step numbering integrity and best-effort error handling confirmed.json.dumpsround-trip).🤖 Generated with Claude Code