Skip to content

Search skills.../

Agent Skill Search Engine

Search

Search
Categories
Occupations

About

About
Privacy
Terms

© 2026 Skills Pool. All rights reserved.

Pr Review | Skills Pool

Skill File

Pr Review

Review a pull request for Sirah project code quality, conventions compliance, and potential issues.

dev-mohammed-hamada0 starsMar 18, 2026

Occupation
Categories: Code Quality

Skill Content

Pull Request Review

Review PR #$ARGUMENTS against Sirah project conventions.

Get PR Context

Fetch the PR diff and details:

!gh pr diff $ARGUMENTS
!gh pr view $ARGUMENTS

Checklist

Code Conventions

Files use kebab-case.ts, components use PascalCase.tsx
TypeScript strict mode — no any types
Functional components + hooks only (no class components)
Backend: thin controllers, logic in services
DTOs validated with

Related Skills

Quick Install

Pr Review

npx skillvault add dev-mohammed-hamada/dev-mohammed-hamada-sirah-app-claude-skills-pr-review-skill-md

Download Skill Open repository

Author: dev-mohammed-hamada
stars: 0
Updated: Mar 18, 2026
Occupation

On this page

01Pull Request Review

class-validator

RTL Compliance

No left/right in styles — uses start/end
All strings in i18n/ar.ts, not hardcoded
No I18nManager hacks

Security

No plain-text passwords (bcrypt only)
No secrets in client code
No any casts that bypass validation
Correct answer never sent to client (server-side validation only)

Islamic Content

All human figures are faceless silhouettes
No face depictions in any asset references
Content historically accurate per Ibn Hisham/Ibn Kathir

State Management

Zustand for client state only
TanStack Query for server state
No server data duplicated in Zustand stores

Game Mechanics

Hearts, XP, stars calculations match spec
Streak logic handles edge cases (same day, yesterday, older)
Star thresholds: 1-50→1★, 51-80→2★, 81-100→3★

Git

Conventional Commits format: type(scope): summary
One logical change per commit
No AI attribution in commits

Output

Provide a structured review with:

Summary — what the PR does
Issues — bugs, convention violations, security concerns
Suggestions — improvements, not blockers
Verdict — Approve / Request Changes / Comment

02

Get PR Context

04Code Conventions

05RTL Compliance

07Islamic Content

08State Management

09Game Mechanics

Software Quality Assurance Analysts and Testers

Openclaw Release Maintainer

Maintainer workflow for OpenClaw releases, prereleases, changelog release notes, and publish validation. Use when Codex needs to prepare or verify stable or beta release steps, align version naming, assemble release notes, check release auth requirements, or validate publish-time commands and artifacts.

Verify

Use when you want to validate changes before committing, or when you need to check all React contribution requirements.

Flow

Use when you need to run Flow type checking, or when seeing Flow type errors in React code.

Automation Tools

Fix

Use when you have lint errors, formatting issues, or before committing code to ensure it passes CI.

Hygiene

Use when making code changes to ensure they pass VS Code's hygiene checks. Covers the pre-commit hook, unicode restrictions, string quoting rules, copyright headers, indentation, formatting, ESLint, and stylelint. Run the hygiene check before declaring work complete.

microsoft184.0k

Add Policy

Use when adding, modifying, or reviewing VS Code configuration policies. Covers the full policy lifecycle from registration to export to platform-specific artifacts. Run on ANY change that adds a `policy:` field to a configuration property.

microsoft184.0k

Software Quality Assurance Analysts and Testers