shell-review

Review CTP shell testcase diffs for compliance and correctness.

1|Updated Apr 2, 2026
One-click install
npx skills add https://github.com/tw-kang/skills --skill shell-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: shell-review
Source: https://github.com/tw-kang/skills/tree/main/shell-review
Command: npx skills add https://github.com/tw-kang/skills --skill shell-review

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve?

This Skill automates thorough reviews of CUBRID CTP shell testcase diffs to detect lifecycle violations, unstable patterns, portability issues, and missing CTP helper usage so reviewers can quickly identify actionable defects before merging.

Core Features & Use Cases

  • Structured Diff Review: Classifies changed files as entry vs helper and runs CTP-specific checks (lifecycle, path conventions, helpers, syntax, stability).
  • Stability & Safety Checks: Detects unbounded loops, long sleeps, orphaned background processes, and flaky output sources and recommends concrete fixes.
  • Guided Remediation: Produces severity-classified findings (blocker/major/minor/note) with evidence snippets and suggested fixes for PR reviews and CI gating.
  • Use Cases: Reviewing PRs that touch shell testcases, auditing new entry scripts, validating helper scripts for portability, and training reviewers with examples.

Quick Start

Review a shell testcase diff and produce a structured CTP compliance report with findings and suggested fixes.

Frequently Asked Questions about shell-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review shell testcase diffs for CTP compliance and correctness?

To review shell testcase diffs for CTP compliance, apply automated checks to changed entry and helper scripts to detect lifecycle violations, path convention errors, and missing CTP helper usage, producing a structured severity-classified report with evidence and suggested fixes.

What are common stability and safety risks in bash test scripts?

Common stability and safety risks in bash test scripts include unbounded loops, long sleeps, orphaned background processes, and flaky output sources, which can be detected and remediated with concrete fixes during a structured shell test script review.

How do I automate PR reviews for CUBRID shell testcases?

Automate PR reviews for CUBRID shell testcases by applying a structured diff review that classifies changed files and runs CTP-specific checks, producing severity-classified findings with evidence snippets and suggested fixes for quick actionable defect identification before merging.

Does shell code review check for orphaned processes and portability issues?

Yes, shell code review checks for orphaned background processes and portability issues by analyzing changed shell scripts for syntax compliance, unbounded loops, and path conventions, generating a structured report with severity classifications and suggested fixes.

What is the best way to validate CTP helper usage in shell scripts?

The best way to validate CTP helper usage in shell scripts is to perform a structured diff review that classifies changed files as entry or helper, running CTP-specific checks to detect missing helper usage and generate a severity-classified compliance report.