Code Review Checklist
shareAI-lab/learn-claude-code
Reviews code against a five-part checklist covering security, correctness, performance, maintainability and testing, and reports findings in a fixed format.
Agent skill
by davidYichengWei in davidYichengWei/agentic-engineering-framework
提供性能优化方法论、设计原则和具体优化规则。在编写代码、优化性能瓶颈、进行 code review 时使用,涵盖 CPU、内存、I/O、并发等维度。
$ npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a claude-codeProject install by default; add -g for ~/.claude/skills/.
$ gh skill install davidYichengWei/agentic-engineering-framework bp-performance-optimization --agent claude-codeProject scope by default; add --scope user for a personal install. Needs GitHub CLI 2.90.0 or later (public preview).
$ git clone --depth 1 https://github.com/davidYichengWei/agentic-engineering-framework.git skills-src && mkdir -p .claude/skills && cp -r skills-src/skills/bp-performance-optimization .claude/skills/bp-performance-optimization && rm -rf skills-srcUse ~/.claude/skills/ instead of .claude/skills for a personal install. The folder must contain SKILL.md.
Claude Code skills documentation · loads skills from .claude/skills/
Install the "bp-performance-optimization" agent skill from https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimization into .claude/skills/bp-performance-optimization/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "bp-performance-optimization", then confirm the skill loads.Claude Code copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$skill-installer install https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimizationType this inside Codex. $skill-installer <name> installs a curated skill from openai/skills. The installer writes to $CODEX_HOME/skills (default ~/.codex/skills). Restart Codex if the skill does not show up.
$ npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a codexProject install goes to .agents/skills/; add -g for ~/.codex/skills/.
$ gh skill install davidYichengWei/agentic-engineering-framework bp-performance-optimization --agent codexProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/davidYichengWei/agentic-engineering-framework.git skills-src && mkdir -p .agents/skills && cp -r skills-src/skills/bp-performance-optimization .agents/skills/bp-performance-optimization && rm -rf skills-srcUse ~/.agents/skills/ instead of .agents/skills for a personal install.
Codex skills documentation · loads skills from .agents/skills/
Install the "bp-performance-optimization" agent skill from https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimization into .agents/skills/bp-performance-optimization/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "bp-performance-optimization", then confirm the skill loads.Codex copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a cursorProject install goes to .agents/skills/; add -g for ~/.cursor/skills/.
$ gh skill install davidYichengWei/agentic-engineering-framework bp-performance-optimization --agent cursorProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/davidYichengWei/agentic-engineering-framework.git skills-src && mkdir -p .cursor/skills && cp -r skills-src/skills/bp-performance-optimization .cursor/skills/bp-performance-optimization && rm -rf skills-srcUse ~/.cursor/skills/ instead of .cursor/skills for a personal install.
Cursor skills documentation · loads skills from .cursor/skills/, .agents/skills/, .claude/skills/, .codex/skills/
Install the "bp-performance-optimization" agent skill from https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimization into .cursor/skills/bp-performance-optimization/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "bp-performance-optimization", then confirm the skill loads.Cursor copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ gemini skills install https://github.com/davidYichengWei/agentic-engineering-framework.git --path skills/bp-performance-optimization--scope user (default) or --scope workspace; --path is the subfolder of the repo that holds the skill; --consent skips the security confirmation prompt.
$ npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a gemini-cliProject install goes to .agents/skills/; add -g for ~/.gemini/skills/.
$ gh skill install davidYichengWei/agentic-engineering-framework bp-performance-optimization --agent gemini-cliProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/davidYichengWei/agentic-engineering-framework.git skills-src && mkdir -p .gemini/skills && cp -r skills-src/skills/bp-performance-optimization .gemini/skills/bp-performance-optimization && rm -rf skills-srcUse ~/.gemini/skills/ instead of .gemini/skills for a personal install, then run /skills reload.
Gemini CLI skills documentation · loads skills from .gemini/skills/, .agents/skills/
Install the "bp-performance-optimization" agent skill from https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimization into .gemini/skills/bp-performance-optimization/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "bp-performance-optimization", then confirm the skill loads.Gemini CLI copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ gh skill install davidYichengWei/agentic-engineering-framework bp-performance-optimizationInstalls for Copilot at project scope by default; add --scope user for a personal install. Preview a skill first with gh skill preview. Needs GitHub CLI 2.90.0 or later (public preview).
$ npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a github-copilotProject install goes to .agents/skills/; add -g for ~/.copilot/skills/.
$ git clone --depth 1 https://github.com/davidYichengWei/agentic-engineering-framework.git skills-src && mkdir -p .github/skills && cp -r skills-src/skills/bp-performance-optimization .github/skills/bp-performance-optimization && rm -rf skills-srcUse ~/.copilot/skills/ instead of .github/skills for a personal install. Commit .github/skills so cloud agent and code review can use it.
GitHub Copilot skills documentation · loads skills from .github/skills/, .claude/skills/, .agents/skills/
Install the "bp-performance-optimization" agent skill from https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimization into .github/skills/bp-performance-optimization/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "bp-performance-optimization", then confirm the skill loads.GitHub Copilot copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a opencodeOpenCode documents no install command of its own. Project install goes to .agents/skills/; add -g for ~/.config/opencode/skills/.
$ gh skill install davidYichengWei/agentic-engineering-framework bp-performance-optimization --agent opencodeProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/davidYichengWei/agentic-engineering-framework.git skills-src && mkdir -p .opencode/skills && cp -r skills-src/skills/bp-performance-optimization .opencode/skills/bp-performance-optimization && rm -rf skills-srcUse ~/.config/opencode/skills/ instead of .opencode/skills for a personal install.
OpenCode skills documentation · loads skills from .opencode/skills/, .claude/skills/, .agents/skills/
Install the "bp-performance-optimization" agent skill from https://github.com/davidYichengWei/agentic-engineering-framework/tree/main/skills/bp-performance-optimization into .opencode/skills/bp-performance-optimization/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "bp-performance-optimization", then confirm the skill loads.OpenCode copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
bp-performance-optimization提供性能优化方法论、设计原则和具体优化规则。在编写代码、优化性能瓶颈、进行 code review 时使用,涵盖 CPU、内存、I/O、并发等维度。
Bp Performance Optimization is an agent skill from davidYichengWei/agentic-engineering-framework. 提供性能优化方法论、设计原则和具体优化规则。在编写代码、优化性能瓶颈、进行 code review 时使用,涵盖 CPU、内存、I/O、并发等维度。
Its SKILL.md is about 820 tokens, which your agent loads only when the skill is triggered. The skill folder holds 9 other files (for example `reference/cache-and-memory.md`, `reference/expression-rules.md` and `reference/logic-rules.md`).
It sits in Development, covering Performance optimization and Code review. The repository describes itself as: 开箱即用、可按项目定制的 AI Coding Agent Skills 框架:提供通用 Workflow、可扩展的项目私有知识、自我学习闭环和问题排查能力,适配任意技术栈和工程场景。兼容主流 Coding Agent。 The licence is MIT.
2 steps, taken from the first numbered list in SKILL.md.
Read from SKILL.md and the folder at commit 1f7ac0f. It shows what the files ask for, not the result of running them.
Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.
From allowed-tools in the SKILL.md frontmatter.
No scripts in the folder and no shell commands in SKILL.md.
From the folder's file list and the shell code blocks in SKILL.md.
No URLs in SKILL.md.
From URLs in SKILL.md, links to its own repository left out.
Names no API keys, tokens, secrets or passwords.
From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.
Bp Performance Optimization loads about 824 tokens when it runs. Until then it costs about 26 tokens; SKILL.md has 223 words of instructions outside code blocks.
Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.
The automated check found no risky patterns in SKILL.md.
Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.
The full file from davidYichengWei/agentic-engineering-framework at commit 1f7ac0f, republished under its MIT licence (© davidYichengWei). 223 words, ~824 tokens.
.claude/skills/bp-performance-optimization/SKILL.md (or your agent's skills folder). This skill also uses 8 other files; get the full folder from GitHub.本 skill 是参考资料,不是工作流入口。
在应用本 skill 的优化规则前,必须确认:
优化目标是否明确?
是否已经定位瓶颈?
如果以上问题不清楚:
workflow-code-generation Skill(它会引导需求澄清)优化的本质是简化。 复杂的代码通常更大、更慢、更耗能。
| 阶段 | 要点 |
|---|---|
| 1. 定义目标 | 明确成功标准:延迟?吞吐?内存?功耗? |
| 2. 设计评估 | 糟糕的设计无法靠优化弥补 |
| 3. Profile | 关注消耗指标(指令数、cache miss),不只是时间 |
| 4. 算法优先 | 先优化数据结构和算法 |
| 5. 应用规则 | 对热点应用具体优化规则 |
| 原则 | 说明 |
|---|---|
| 简单直接 | 避免过度抽象;能删代码比加代码好 |
| 连续优于分散 | vector/数组优于链表(cache 友好) |
| 直接优于间接 | 减少指针追逐,每次间接访问都是潜在 cache miss |
| 独占优于共享 | unique_ptr 无开销;shared_ptr 有原子操作开销 |
| 扁平优于深层 | 避免复杂继承层次和虚函数派发 |
| Early Binding | 把工作提前做一次,避免重复做多次 |
| 模块化 | 清晰契约,允许局部替换而不改架构 |
| 规则 | 说明 |
|---|---|
| Data Structure Augmentation | 添加冗余信息加速操作 |
| Precomputation | 预计算并存储结果 |
| Caching | 缓存频繁访问的数据 |
| Lazy Evaluation | 延迟计算直到真正需要 |
详见 reference/space-for-time.md
| 规则 | 说明 |
|---|---|
| Packing | 紧凑存储减少内存占用 |
| Overlaying | 复用内存空间 |
| Interpreters | 用解释器压缩程序表示 |
详见 reference/time-for-space.md
| 规则 | 说明 |
|---|---|
| Code Motion | 将循环不变量移出循环 |
| Combining Tests | 合并测试条件,使用哨兵 |
| Loop Unrolling | 展开循环减少迭代开销 |
| Loop Fusion | 合并相同范围的循环 |
| 规则 | 说明 |
|---|---|
| Algebraic Identities | 用等价的更廉价表达式替换 |
| Short-Circuiting | 提前终止求值 |
| Reordering Tests | 廉价/常成功的测试放前面 |
| Precompute Logical Functions | 查表替代逻辑计算 |
| 规则 | 说明 |
|---|---|
| Inlining | 内联展开减少调用开销 |
| Exploit Common Cases | 快速路径处理常见情况 |
| Tail Recursion Removal | 尾递归转循环 |
| Parallelism | 利用硬件并行能力 |
详见 reference/procedure-rules.md
| 规则 | 说明 |
|---|---|
| Compile-Time Initialization | 编译期初始化 |
| Strength Reduction | 用廉价操作替代昂贵操作 |
| Common Subexpression Elimination | 消除公共子表达式 |
| Word Parallelism | 利用字宽并行(位操作) |
详见 reference/expression-rules.md
| 规则 | 说明 |
|---|---|
| AoS → SoA | 单字段扫描时结构体数组转数组结构体 |
| Loop Tiling | 分块处理,使工作集 fit 进 cache |
| False Sharing | 多线程变量对齐到不同 cache line |
| Arena/Pool | 批量分配,避免频繁 malloc |
详见 reference/cache-and-memory.md
| 规则 | 说明 |
|---|---|
| string_view/span | 避免拷贝,零开销抽象 |
| Branchless | 减少分支预测失败惩罚 |
| CRTP | 静态多态替代虚函数 |
| Sharded Lock | 分片锁减少竞争 |
详见 reference/modern-cpp-tricks.md
设计层面:
实现层面:
© davidYichengWei, MIT. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file
SKILL.md and 8 other files in skills/bp-performance-optimization of davidYichengWei/agentic-engineering-framework.
Open the folder on GitHubat commit 1f7ac0f
Bp Performance Optimization next to the 5 skills that share the most tags, products or categories with it. Stars are the repository's; “used in” counts other GitHub owners with a copy.
| Skill | Stars | Used in | Tokens | Auto-check | Licence | Repo updated |
|---|---|---|---|---|---|---|
| Bp Performance Optimization this skilldavidYichengWei/agentic-engineering-framework | 158 | — | ~824 | Automated safety check: Pass | MIT | |
| Code Review ChecklistshareAI-lab/learn-claude-code | 78k | 5 repos | ~1.1k | Automated safety check: Pass | MIT | |
| Analyzing .NET Performancedotnet/skills | 5.6k | 3 repos | ~3.1k | Automated safety check: Pass | MIT | |
| MAUI PR Performance Analysisdotnet/maui | 23k | — | ~2.4k | Automated safety check: Pass | MIT | |
| Performance CheckZeroDeng01/sublinkPro | 1.7k | — | ~1.8k | Automated safety check: Pass | MIT | |
| Code Review Specialistluongnv89/claude-howto | 42k | — | ~764 | Automated safety check: Pass | MIT |
shareAI-lab/learn-claude-code
Reviews code against a five-part checklist covering security, correctness, performance, maintainability and testing, and reports findings in a fixed format.
dotnet/skills
Scans C# and .NET code for about 50 performance anti-patterns and reports prioritized findings with concrete fixes, at a scan depth you choose.
dotnet/maui
Interprets pinned managed benchmark evidence for a dotnet/maui pull request and writes a narrative for the performance review workflow, without running or publishing anything.
ZeroDeng01/sublinkPro
Checklist for reviewing code changes that touch queries, APIs, rendering, caching or algorithms for performance, scalability and resource-usage problems.
luongnv89/claude-howto
Reviews code for security, performance, quality and maintainability, using a checklist, a finding template and two metrics scripts.
pretend1111/claude-desktop-app
Reviews code or recent changes for bugs, security issues, performance problems and maintainability, reporting findings by severity with the reason and a fix.
davidYichengWei/agentic-engineering-framework
Gives architecture design principles for system design discussions and code review: module boundaries, dependency direction, data ownership and interface rules, plus a checklist.
davidYichengWei/agentic-engineering-framework
A checklist of language-neutral rules for writing and reviewing code: naming, function design, control flow, resource safety, comments and logging.
davidYichengWei/agentic-engineering-framework
Chinese-language checklists for component-level design: class and module structure, public interfaces, data models, concurrency and error handling.
davidYichengWei/agentic-engineering-framework
Chinese-language guide to writing and improving SKILL.md files: frontmatter rules, concise writing, progressive disclosure, common patterns and a pre-release checklist.
davidYichengWei/agentic-engineering-framework
Turns mistakes you correct into proposed updates to persistent Rules and Skills so the same error does not recur in later sessions, triggered automatically or with /reflect.
davidYichengWei/agentic-engineering-framework
Diagnoses compile errors, runtime exceptions, failing tests, pipeline failures and production alerts from code and logs, giving a root cause before any fix.
Categories
提供性能优化方法论、设计原则和具体优化规则。在编写代码、优化性能瓶颈、进行 code review 时使用,涵盖 CPU、内存、I/O、并发等维度。. Bp Performance Optimization is an agent skill from davidYichengWei/agentic-engineering-framework.
Bp Performance Optimization fits situations like: tasks that involve Performance optimization; tasks that involve Code review.
Run `npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a claude-code`. Or copy the skill folder (skills/bp-performance-optimization in davidYichengWei/agentic-engineering-framework) into .claude/skills/bp-performance-optimization in your project. Claude Code loads it when a task matches its description.
Run `npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a codex`. Or copy the skill folder (skills/bp-performance-optimization in davidYichengWei/agentic-engineering-framework) into .agents/skills/bp-performance-optimization in your project. Codex loads it when a task matches its description.
Cursor, Gemini CLI, GitHub Copilot and OpenCode also load SKILL.md folders. With the skills CLI, run `npx skills add davidYichengWei/agentic-engineering-framework --skill bp-performance-optimization -a cursor` (or -a gemini-cli, github-copilot or opencode for the others). To copy it by hand, put the folder in .cursor/skills/bp-performance-optimization, .gemini/skills/bp-performance-optimization, .github/skills/bp-performance-optimization and .opencode/skills/bp-performance-optimization in your project.
SKILL.md names no scripts, command-line tools or credentials: Bp Performance Optimization is instructions for the agent only.
SKILL.md contains no URLs. Any network use would come from the scripts or tools the agent runs. This is read from the text; nothing was executed.
Our automated static check of SKILL.md found no risky patterns, such as piping downloads into a shell, reading credential files or hidden Unicode. It is not a guarantee. Review the folder before installing.
Bp Performance Optimization is published under the MIT licence (the repository's licence). It allows redistribution, so the full SKILL.md is shown on this page.
About 824 tokens (SKILL.md is roughly 3.3k characters). Agents keep only the skill's name and description in context until a task matches; then they load SKILL.md in full.
Skills that share tags, products or a category with Bp Performance Optimization: Code Review Checklist (shareAI-lab/learn-claude-code, 78k stars), Analyzing .NET Performance (dotnet/skills, 5.6k stars), MAUI PR Performance Analysis (dotnet/maui, 23k stars) and Performance Check (ZeroDeng01/sublinkPro, 1.7k stars). The comparison table on this page puts their stars, adoption, token cost, safety result and licence side by side.
davidYichengWei (a GitHub user) maintains it in davidYichengWei/agentic-engineering-framework, which has 158 GitHub stars. The repository holds 14 skills in this directory. The repository was last updated on March 25, 2026.
Source: davidYichengWei/agentic-engineering-framework on GitHub. Facts on this page come from the repository at the commit we read; the author's words are quoted as theirs.