acefrost511/task-review-workflow
skills/task-review-workflow/SKILL.md
Standard PR review and merge workflow for task-driven development. Use when reviewing a programmer agent PR linked to a task, deciding merge vs change request, handling post-merge actions (Trello + branch cleanup), and sending a clear outcome handoff.
development
Updated Apr 3, 2026
$ install --global
skillsauthnpx skillsauth add acefrost511/workmemory task-review-workflowInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
Security Scan Results
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
3
Scanners Passed
9
Scanners in report
Clean
TrivyContainer and dependency vulnerability scanner
95%
Clean
SemgrepStatic code analysis for vulnerabilities
95%
Clean
mcp-scan (Snyk)Model Context Protocol security validation
95%
Skipped
Snyk (dep)Open source security scanning
50%
Skipped
Socket.devSupply chain security analysis
50%
Skipped
VirusTotalMulti-engine malware detection
50%
Skipped
CrowdStrikeAdvanced threat intelligence
50%
Skipped
OSV-ScannerOpen Source Vulnerability database check
50%
Skipped
OWASP Dep-Check
50%
Last scanned: Apr 3, 2026, 9:49 PM180.4s3 files scanned
SKILL.md
- name:
- task-review-workflow
- description:
- Standard PR review and merge workflow for task-driven development. Use when reviewing a programmer agent PR linked to a task, deciding merge vs change request, handling post-merge actions (Trello + branch cleanup), and sending a clear outcome handoff.
Task Review Workflow
Follow this workflow in order for every task-linked PR.
1) Gather Context
- Read the PR description.
- Open and read the linked task before reviewing code.
- Confirm expected behavior and acceptance criteria from the task context.
2) Review Against Standard
- Use
REVIEW_CHECKLIST.mdas the mandatory review baseline. - Check correctness, edge cases, regressions, security, performance, and test adequacy.
3) Review the Diff Thoroughly
- Review file-by-file.
- Flag logic flaws, unsafe assumptions, missing validation, unclear naming, dead code, and side-effect risks.
4) Validate Locally When Possible
- Check out the PR branch.
- Run relevant test/lint/build commands.
- Exercise changed behavior directly where practical.
5) Write Clear Review Feedback
- Leave actionable, specific CR comments.
- Separate must-fix issues from optional suggestions.
6) Decide Outcome
- If issues remain: request changes with a concrete fix list.
- If quality is acceptable: approve/merge with a short merge note.
7) Execute Post-Merge Steps
- Move the related Trello card to Done.
- Delete the task branch after merge.
- Never delete the
mainbranch.
8) Complete Handoff
- Send the final outcome back to the programmer agent:
merged, orCR sent, orwaiting for fixes.
- Ensure the next task starts only after this outcome message.
Related Skills
acefrost511/xlsx
development
VerifiedTrustedCommunity
Create, edit, analyze spreadsheets (.xlsx, .xlsm, .csv). Formulas, formatting, data analysis.
SKILL.mdUpdated Apr 3, 2026
acefrost511/self-improving-agent
testing
VerifiedTrustedCommunity
Self-improving agent system that analyzes conversation quality, identifies improvement opportunities, and continuously optimizes response strategies.
SKILL.mdUpdated Apr 3, 2026
acefrost511/using-superpowers
tools
VerifiedTrustedCommunity
Use when starting any conversation - establishes how to find and use skills, requiring Skill tool invocation before ANY response including clarifying questions
SKILL.mdUpdated Apr 3, 2026
acefrost511/topic-to-article-kit
documentation
VerifiedTrustedCommunity
Turn a chosen AI topic into a public-account writing package: collect evidence from X/Twitter and other sources, extract high-value comments, and produce title options + structured outline written into Obsidian. Use when user asks for topic-based research and article prep.
SKILL.mdUpdated Apr 3, 2026