
abczsl520/bug-audit-skill
Code reviewこのページは AI により自動翻訳されています。技術的な詳細は原文をご確認ください。 原文を見る
OpenClawスキル:Node.js Webプロジェクト(ゲーム、データツール、WeChat、API、ボット)の動的バグ監査。 200以上の現実的な落とし穴.
概要
チェックリストを実行しないでください。 プロジェクトを解散し、すべての組織を徹底的に検証します。 難しいレッスンから構築されたプロジェクトは、一般的なチェックリストを使用して172のバグを見つけました。 ポスト・モレテムは、プロジェクト固有のチェック・マトリクスを最初に作成すると、ほとんどのバグが発生したことを明らかにしました。 一般的なチェックリストは、「既知のパターン」バグ(CORS、XSS、タイムゾーン)をキャッチします。 しかし、最も重要なバグは : - API を購入すると、コスト = 0 → 無料の購入 (任意のチェックリストではなく) - 購入を呼び出しずに raid-result コール可能 → 無限のお金の悪用 - 検索完了は距離を検証しません → リモートローテーション これらのバグは、個々のコードパターンではなく、 最大のバグ(バイパス、レイドトークンの欠落)は、API間でリンクを隠します。 「对这个项目行行列行列」または「バグのこのプロジェクトを聴く」 Wikiのフルドキュメント: - 解剖学的分岐法 - 各テーブルをビルドする方法、何を尋ねる - 排結同法一 - 補足チェックのための9つの一般的なモジュール - 实战踩坑迅速な添付 — 高頻度下落 + リモートデバッグ
概要(AI 翻訳)
チェックリストを実行しないでください。 プロジェクトを解散し、すべての組織を徹底的に検証します。
難しいレッスンから構築されたプロジェクトは、一般的なチェックリストを使用して172のバグを見つけるために**21ラウンドを取った。 ポスト・モレテムは、プロジェクト固有のチェック・マトリクスを最初に作成すると、**3-4ラウンドでほとんどのバグが発生したことを明らかにした。
チェックリストの問題
一般的なチェックリストは、「既知のパターン」バグ(CORS、XSS、タイムゾーン)をキャッチします。 しかし、最も重要なバグはプロジェクト固有のロジック脆弱性です。
- [
buy] API はcost=0] → 無料の購入 (任意のチェックリストではない場合) - [[
raid-result]を呼び出しずにコール可能[buy] - 検索完了は距離→リモートローティングを検証しません
これらのバグは、個々のコードパターンではなく、APIs間の関連性に住んでいます。
ソリューション: 解散 → 検証 → 補足
##6テーブル
| 表 | 抽出物 | 重要事項 | お問い合わせ | APIエンドポイント | 各路線:方法・経路・auth・パラメータ | 迂回できますか? ゴミを送ったら? | | ステートマシン | あらゆる状態変数: 定数・読者・ライフサイクル | ライフサイクルを横断するリークは? | | タイマー | 各セットタイムアウト・セットインターバル | 清掃後の発火 |
このページは AI により自動翻訳されています。技術的な詳細は原文をご確認ください。
README
以下は原文 README の抜粋です。原文を見る。
🔍 Bug Audit Skill
Don’t run a checklist. Dissect the project, then exhaustively verify every entity.
Built from a hard lesson: a project took 21 rounds to find 172 bugs using generic checklists. Post-mortem revealed that building project-specific check matrices first would have caught most bugs in 3-4 rounds.
The Problem with Checklists
Generic checklists catch “known pattern” bugs (CORS, XSS, timezone). But most critical bugs are project-specific logic vulnerabilities:
buyAPI acceptscost=0→ free purchases (not in any checklist)raid-resultcallable without callingbuyfirst → infinite money exploit- Search completion doesn’t verify distance → remote looting
These bugs live in the relationships between APIs, not in individual code patterns.
The Solution: Dissect → Verify → Supplement
Phase 1: Dissect — Read code, build 6 project-specific tables (10-15 min)
Phase 2: Verify — Exhaustively check every row in every table
Phase 3: Supplement — Run generic modules as safety net
Phase 4: Regress — Check fixes didn't introduce new bugs
Phase 5: Archive — Record pitfalls for next audit
The 6 Tables
| Table | Extracts | Key Question |
|---|---|---|
| API Endpoints | Every route: method, path, auth, params | Can I bypass? What if I send garbage? |
| State Machines | Every state variable: setter, reader, lifecycle | Does it leak across lifecycles? |
| Timers | Every setTimeout/setInterval | Does it fire after cleanup? |
| Numeric Values | Every user-influenceable number | What if 0? Negative? Huge? |
| Data Flows | Every related API pair (buy→use) | Can I skip Step 1 and call Step 2 directly? |
| Resource Ledger | Every resource: all inflows, all outflows | Is there an infinite loop? |
Data Flows table is the most critical. The biggest bugs (buy bypass, missing raid tokens) hide in the links between APIs.
Install
clawhub install bug-audit
Or manually:
git clone https://github.com/abczsl520/bug-audit-skill.git ~/.openclaw/skills/bug-audit
Then say: “对这个项目执行bug排查” or “audit this project for bugs”
What’s Inside
| File | Content |
|---|---|
SKILL.md |
Core methodology: 6 tables + 5 phases |
references/modules.md |
9 generic audit modules for Phase 3 |
references/pitfalls.md |
200+ real-world pitfalls + debugging techniques |
Documentation
Full docs on the Wiki:
- 解剖流程详解 — How to build each table, what to ask
推奨ツール
別のキーワードを試すか、フィルタを外してください。
インストール
npx skillfish add abczsl520/bug-audit-skill