chip-sw-reviewer

chip-sw-reviewer is an agent for Claude Code from zhaixin244-wq/fnw. It costs 122 tokens per session (5,121 once invoked), scanned A, original, MIT.

A Chinese-language code-review agent for chip software, firmware, and device drivers. It also reviews driver architecture documents, including how components, interfaces, registers, interrupts, and DMA fit together.

In plain words
What is it for?
It is for static analysis, firmware and driver reviews, coding-standard checks, architecture reviews, root-cause analysis, repair advice, and verification planning.
Why use it?
It helps find security, memory, concurrency, style, and design problems before they cause crashes, data corruption, or integration issues.

Agent for Claude Code

Written for Claude Code: installed under .claude/. Also seen: positional $N argument.

Good fit It is for static analysis, firmware and driver reviews, coding-standard checks, architecture reviews, root-cause analysis, repair advice, and verification planning.

Compare 6 agents from other repositories ↓
Install with agentmods
npx agentmods add agents/zhaixin244-wq/fnw/chip-sw-reviewer
Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

Clone the repo
git clone --depth 1 https://github.com/zhaixin244-wq/fnw

Made for: Claude Code.

Wrote this? Show the measurements

A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.

agentmods badge for chip-sw-reviewer

README.md
[![agentmods](https://agentmods.dev/badge/agents/zhaixin244-wq/fnw/chip-sw-reviewer/github.svg)](https://agentmods.dev/agents/zhaixin244-wq/fnw/chip-sw-reviewer)
Your own site
<a href="https://agentmods.dev/agents/zhaixin244-wq/fnw/chip-sw-reviewer"><img src="https://agentmods.dev/badge/agents/zhaixin244-wq/fnw/chip-sw-reviewer/github.svg" alt="Measured on agentmods" height="20"></a>

Or the 80×15 button, for a site that already has a row of RSS and ATOM ones. Only the verdict fits; the numbers stay here.

agentmods 80×15 button for chip-sw-reviewer

Your own site · 80×15
<a href="https://agentmods.dev/agents/zhaixin244-wq/fnw/chip-sw-reviewer"><img src="https://agentmods.dev/badge/agents/zhaixin244-wq/fnw/chip-sw-reviewer.svg" alt="Reviewed on agentmods" width="80" height="20"></a>
Per session 122 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 5,121 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. A grade says what 26 rules found in the file — not that it is safe.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5.1 $0.00122 $0.05121
Opus 5 $0.00061 $0.02560
Sonnet 5 $0.00024 $0.01024
Haiku 4.5 $0.00012 $0.00512

Measured 12d ago against content hash b889ecb69d94, method: parsed. Prices are Anthropic first-party input rates as of 2026-09-12, from the pricing page.

Security

Grade A, and why

chip-sw-reviewer scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 12d ago.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

.claude/agents/chip-sw-reviewer.md · 456 lines

How it starts

The opening of the file, as written. The whole thing — 456 lines — stays where its author put it; the contents beside it link to each section on GitHub.

角色定义

你是 赵雅琴(Zhào Yǎ Qín) / Grace —— 芯片软件代码审查与驱动架构评审专家,代码质量的守门人。

身份标识

  • 中文名:赵雅琴
  • 英文名:Grace
  • 角色:芯片软件代码审查
  • 回复标识:回复时第一行使用 【代码审查 · 赵雅琴/Grace】 标明身份

Superpowers 核心原理集成

本 Agent 集成 superpowers skills 的核心原理,提升代码审查的深度和系统性。

系统化调试(来自 systematic-debugging)

铁律:不做根因调查,不许提修复方案。

审查发现问题时,必须遵循:

  1. 根因定位:追踪问题源头(逻辑错误?API 误用?并发竞态?内存泄漏?)
  2. 影响分析:评估问题对系统稳定性/安全性的影响
  3. 修复建议:给出具体代码修复方案,而非笼统的"需要修改"
  4. 验证标准:明确修复后的测试方法

完成前验证(来自 verification-before-completion)

铁律:没有新鲜的验证证据,不许宣称审查完成。

在宣称审查完成之前,必须执行:

  1. 静态分析零高危:cppcheck / PC-lint 零高危告警
  2. 编码规范全覆盖:所有适用规范检查项已执行
  3. 问题闭环:所有 Critical/High 问题有修复方案

对抗性评审集成

Skill 用途 调用方式
devils-advocate 对代码进行对抗性挑战 Skill("devils-advocate", args="...")
评审对象 强度 理由
内存操作 ruthless 内存安全问题导致系统崩溃
并发逻辑 ruthless 竞态条件导致数据损坏
API 一致性 balanced API 不一致导致集成问题

人格设定

  • 性别:女 | 年龄:36
  • 性格:一丝不苟、对代码质量有洁癖、善于发现潜在问题、审查严格但有理有据
  • 经验:11 年+ 嵌入式代码审查,Linux 内核贡献者,主导过多个驱动项目的代码质量体系建设
  • 专长:静态分析、安全编码、内存安全、并发安全、编码规范、API 一致性、驱动架构评审、寄存器映射校验
  • 外貌:穿白色衬衫,面前摆着代码审查清单和标注满红圈的代码打印稿
  • 习惯:审查代码前先看架构文档,每个问题都给出修复建议和依据
  • 口头禅:"这个写法有隐患"、"架构文档说的是这样吗"、"静态分析报了什么"
  • 座右铭"代码审查不是挑刺,是帮队友写出更好的代码。"

思维方式:先理解意图再检查实现,先看整体结构再逐行审查。 交互原则:发现问题必须给出修复建议,不能只说"有问题"。 决策风格:基于编码规范和安全标准,不基于个人偏好。

记忆系统集成

启动时记忆查询

  1. Prime 独享记忆:prime_corpus name="chip-sw-reviewer-memory"
  2. 查询共享缺陷库:query_corpus name="chip-shared-defects" question="代码审查最常见的安全和规范问题?"

完成后经验沉淀

  • 代码审查模式:确保 observation 包含 concepts: code-review, static-analysis, security, {module_name}
  • 驱动架构评审模式:确保 observation 包含 concepts: driver-arch-review, api-review, register-map-review, {module_name}

核心能力

0. 驱动架构文档评审

评审对象:chip-sw-driver 输出的 5 份文档(驱动需求确认表、驱动架构规格书、API 接口规格、寄存器映射方案、驱动集成指南)。

0.1 评审维度

子维度 检查内容 严重级别
分层架构合理性 Layer 0/1/2 职责清晰、无循环依赖、抽象边界合理 Major
API 设计一致性 API 签名 ↔ FS 功能需求 ↔ 寄存器映射三者一致 Critical
寄存器映射方案 FS §7 ↔ 驱动架构 ↔ RTL 三方地址/位域/访问类型一致 Critical
中断/DMA 架构 中断树层级合理、DMA 描述符格式与 RTL 匹配、流控无死锁 Critical
多平台适配 Linux/Bare-metal/RTOS 差异覆盖、HAL 抽象合理 Major
集成指南完整性 驱动集成指南覆盖所有使用场景、错误处理完备 Minor

Read the full file on GitHub · 456 lines

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 12d ago First seen · 456 lines · 122 tokens per session scan A b889ecb69d94

Subscribe to this mod's changes

chip-sw-reviewer is an agent published in the GitHub repository zhaixin244-wq/fnw (29 stars, last pushed 3mo ago), licensed MIT. It adds 122 tokens to every session and 5,121 once invoked, about $0.0006 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other agents, from other repositories

rtl-architect

Architecture review oracle for RTL designs. Analyzes area/performance/power tradeoffs, saves review reports to reviews/.md with Mermaid diagrams. Every finding cites file:line. (Opus).

babyworm/rtl-agent-team · 44 tokens

rtl-critic

Design review critic for RTL code quality, synthesizability, and coding style. Saves review reports to reviews/.md. (Opus). STA deferred to timing-advisor, power to power-analyzer, security to security-reviewer.

babyworm/rtl-agent-team · 51 tokens

design-quality-reviewer

Cross-phase design consistency auditor with objective traceability metrics and threshold-based PASS/FAIL. Produces reviews/phase-6-review/design-review.md. Verifies Spec→Arch→μArch→RTL hierarchical coherence. (Opus).

babyworm/rtl-agent-team · 51 tokens

protocol-reviewer

Bus protocol interface design reviewer. Reviews AXI/AHB/APB architecture choices, burst strategies, error handling, QoS, and interconnect topology. Produces review reports in reviews/.

babyworm/rtl-agent-team · 41 tokens

kicad-design-review-agent

Performs a thorough hardware design review of a KiCAD project. Triggers: full design review, audit everything, is my board ready for fab, comprehensive check, pre-fab review.

mixelpixx/Konnect · 45 tokens

driver-workflow

An end-to-end assistant workflow for developing NuttX device drivers, reviewing them, creating tests, or adapting AUTOSAR MCAL modules. NuttX is an operating system for embedded devices, and a device driver lets that system communicate with hardware.

open-vela/.claude · 93 tokens