Repository navigation
Improve incident-postmortem skill: Google SRE format, fishbone + 5 Whys, multiple output formats 🤖🤖🤖 - #5247
Improve incident-postmortem skill: Google SRE format, fishbone + 5 Whys, multiple output formats 🤖🤖🤖#5247ngpayyappilly wants to merge 2 commits into
Conversation
… RCA - Follow the Google SRE post-mortem template (Detection, Resolution, Trigger, Where we got lucky, typed action items) - Add a fishbone (Ishikawa) step that feeds per-branch 5 Whys chains - Gather all incident inputs from the user; never invent details - Offer Markdown (default), HTML, DOCX (via md-to-docx) and PDF output - Move the template and a worked example into references/ Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
🚦 Submission status: 💬 Review in progressRisk tier: Why this tier
Automated checks
Review
Commands
Updated for |
🔒 PR Risk Scan ResultsScanned 3 changed file(s).
✅ No matching risk patterns were detected in changed files.
|
🔍 Vally Lint Results✅ All checks passed
Summary
Full linter output |
aaronpowell
left a comment
There was a problem hiding this comment.
Thanks for improving the existing skill. The strongest parts are the no-fabrication rule, cause status labels, evidence-backed 5 Whys and typed action items. Before merge:
- DOCX path (Step 8):
node skills/md-to-docx/scripts/md-to-docx.mjsonly works inside an awesome-copilot checkout. Make it conditional on md-to-docx being installed, or drop DOCX from the defaults. - Trim: cut untested HTML/PDF rows, collapse Step 5 into the template, remove Common Mistakes rows that repeat the Process, and link the template rather than re-listing sections in Step 7.
- Validation: run the skill once in GitHub Copilot with it loaded on a realistic incident and note the result.
A short note on compatibility for existing users of the #2019 template would also help.
|
@aaronpowell Thanks for the review and I'll address the review comments soon and request for further reviews. |
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.mainbranch for this pull request.Description
Improves the existing
incident-postmortemskill rather than adding a duplicate:md-to-docxskill) or PDF on request.references/to keepSKILL.mdfocused.Type of Contribution
Testing and AI disclosure
This PR was authored by an AI agent (Claude Code) at a human contributor's request and direction, hence the
🤖🤖🤖title marker.What was verified, and how:
npm run skill:validateandnpm run buildpass; the repo's CI checks pass.references/postmortem-template.md: inputs, timeline, fishbone, two 5 Whys chains from different categories, and Detect/Mitigate/Prevent action items. I then converted it with the existingmd-to-docxskill. The .docx generated without errors and contains all five tables and the fishbone tree.What was NOT verified:
Additional Notes
The skill name and trigger phrases are unchanged, so existing users are unaffected.
docs/README.skills.mdwas regenerated withnpm run build.By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.
🤖 Generated with Claude Code