review-ticket

$npx mdskill add HoangNguyen0403/agent-skills-standard/review-ticket

> [!IMPORTANT] > Review a ticket or PR through focused specialist lenses: scope, architecture, security, tests, AC coverage, and PR metadata.

SKILL.md

.github/skills/review-ticketView on GitHub ↗
---
name: review-ticket
description: "Review a ticket or PR through focused specialist lenses: scope, architecture, security, tests, AC coverage, and PR metadata."
metadata:
  triggers:
    keywords:
    - review ticket
    - workflow
---
# Review Ticket Skill

> [!IMPORTANT]
> Review a ticket or PR through focused specialist lenses: scope, architecture, security, tests, AC coverage, and PR metadata.

## Instructions

When the user asks to perform this workflow, execute the following steps:


# Review Ticket Workflow

Goal: Produce a PR-ready review verdict using compact specialist fanout and evidence-linked findings.

## Steps

1. Load scope:
   - Ticket/story, PR URL/diff, changed files, ACs, test evidence, and loaded framework skills.
   - Jira/GitHub/GitLab/ADO/Zephyr/code-review-graph MCPs when configured; otherwise use exported ticket, diff, and local files.

2. Run specialist lenses:
   - `specialist-codebase-scout`: affected files, patterns, blast radius, tests.
   - `specialist-pr-reviewer`: PR/MR metadata, active threads, template gaps.
   - `specialist-ac-verifier`: AC coverage and scope creep.
   - `specialist-architecture-guard`: architecture and design risks.
   - `specialist-security-reviewer`: OWASP, Vibe Security, data provenance.
   - `specialist-test-gap-finder`: missing tests and weak assertions.

3. Merge findings:
   - Deduplicate by root cause.
   - Keep only actionable findings with evidence.
   - Calibrate severity: Blocker, Major, Minor, Suggestion.
   - Mark unverified items as assumptions or requests for evidence.

4. Decide verdict:
   - APPROVE: no Blocker/Major, required evidence present.
   - CHANGES REQUESTED: fixable Blocker/Major.
   - BLOCKED: missing diff, ticket, environment, or required tool/export.

5. Optional publish:
   - Use `specialist-pr-commenter-batch` only after user approves posting comments.
   - Otherwise produce local review report.

## Output Template

```md
# Review Ticket Report

## Verdict

## Findings
| Severity | Lens | Evidence | Fix |
| --- | --- | --- | --- |
| [severity] | [lens] | [file/AC/tool] | [fix] |

## Evidence Gaps

## Next Workflow

## Cost Report
```

More from HoangNguyen0403/agent-skills-standard

SkillDescription
android-agp-upgradeUpgrade an Android project to Android Gradle Plugin (AGP) 9. Use when migrating to AGP 9, updating Gradle build files, migrating to built-in Kotlin, or adopting the new AGP DSL.
android-architectureApply Clean Architecture layering, modularization, and Unidirectional Data Flow in Android projects. Use when setting up project structure, placing code in layers, configuring feature/core modules, or implementing UDF patterns.
android-background-workImplement WorkManager and background processing correctly on Android. Use when creating Worker classes, scheduling tasks, choosing between WorkManager and Foreground Services, or setting up Hilt in workers.
android-composeBuild high-performance declarative UI with Jetpack Compose. Use when writing Composable functions, optimizing recomposition, hoisting state, or working with LazyColumn and side effects.
android-compose-migrationMigrate an Android XML View to Jetpack Compose following a structured 10-step workflow. Use when converting XML layouts to Compose, setting up Compose in an existing View-based project, or incrementally adopting Compose.
android-concurrencyWrite correct coroutine scopes, Flow collection, and dispatcher injection in Android. Use when writing suspend functions, choosing between StateFlow and SharedFlow, or injecting Dispatchers for testability.
android-deploymentConfigure release signing, R8 obfuscation, and App Bundle publishing for Android. Use when setting up signing configs, enabling minification, adding ProGuard keep rules, or preparing for Play Store submission.
android-design-systemEnforce Material Design 3 theming and design token usage in Jetpack Compose. Use when implementing M3 components, color schemes, typography, or design tokens.
android-diConfigure Hilt dependency injection with proper scoping, modules, and constructor injection in Android. Use when setting up Hilt DI, defining modules, or configuring component scoping.
android-edge-to-edgeMigrate a Jetpack Compose app to edge-to-edge display and fix system bar inset issues. Use when UI components are obscured by navigation/status bars, fixing IME insets, or enabling edge-to-edge for SDK 35+.