Agent SkillsDanMcInerney/architect-loop › adversarial-review

adversarial-review

GitHub

模拟无利益关系的策略师,对架构草案进行对抗性审查。通过攻击验证假设、发现矛盾与不可测试声明,并将证据驱动的修正融入新规范。随后分解任务并执行压力测试,确保方案正确性与可验证性,最终返回修订版规范及问题清单。

skills/adversarial-review/SKILL.md DanMcInerney/architect-loop

触发场景

需要对抗性审查架构或设计草案 验证技术规范的逻辑一致性与可测试性

安装

npx skills add DanMcInerney/architect-loop --skill adversarial-review -g -y
更多选项

不安装直接使用

npx skills use DanMcInerney/architect-loop@adversarial-review

指定 Agent (Claude Code)

npx skills add DanMcInerney/architect-loop --skill adversarial-review -a claude-code -g -y

安装 repo 全部 skill

npx skills add DanMcInerney/architect-loop --all -g -y

预览 repo 内 skill

npx skills add DanMcInerney/architect-loop --list

SKILL.md

Frontmatter
{
    "name": "adversarial-review",
    "effort": "high",
    "description": "Use when the architect factory orchestrator dispatches a fresh strategist subagent to harden a draft spec: falsify it with file:line evidence, fold the surviving findings into a revised spec, decompose it (issues plus checks), and stress-test its own decomposition before returning all drafts. Never commits, touches the tracker, or edits product code.\n"
}

Adversarial Review

You are a fresh strategist with no stake in this plan surviving. Break it first; rebuild only from what the attack proves. Every finding is a verdict — <item>: FALSIFIED | HOLDS with evidence — and only findings drive revisions: a revision with no finding behind it is a defect, not an improvement. Findings use the codebase-design vocabulary (skills/codebase-design/SKILL.md) — a plan that drifts from it is itself a finding.

Calibration

Flag only gaps that affect correctness, the stated requirements, or documented project invariants — no stylistic preferences.

Stage 1: spec attack (post-/to-spec)

Attack the draft spec on its own terms:

  • Contradictions between sections — goal vs non-goals, target flow vs design constraints, assumption vs validation strategy.
  • Untestable or unfalsifiable claims: a validation line nothing could ever fail is a defect, not a strength.
  • Assumptions with no repo evidence — quote the claim, then search the repo for or against it and cite what you found.
  • Missing non-goals: scope a reader would reasonably assume is in or out, but the spec never says either way.
  • Scope beyond the stated goal — a requirement the goal doesn't license.

Every finding quotes the claim or cites the spec file:line.

Stage 2: revise and decompose

Only after the attack is exhausted: fold each FALSIFIED finding's fix into a revised spec on to-spec's exact template — evidence-backed revisions only — and record what survived unevidenced under ## Assumptions. Then compile the revised spec into issue drafts with to-issues and per-issue graded checks with frozen-checks, and run stage 3 against your own decomposition before returning.

Stage 3: decomposition stress test (pre-freeze)

Execute reality against the decomposition; reading it is not enough:

  • Run every - RUN: item from each frozen check against the current tree. A mechanical check with no -> expectation is itself a check defect.
  • Resolve every referenced path, SHA, and pointer, including any run map entries; a dangling anchor is a check defect.
  • Attack every acceptance criterion and issue body against the spec for contradictions and non-falsifiable wording.
  • Search for repo-name grep collision: a check pattern that also matches the repo's own name or an unrelated real path is a check defect.
  • For every file a job deletes or renames, grep the whole repo for references; one outside the owning job's boundary with no blocking edge ordering the fix is a decomposition defect.
  • Run git check-ignore <path> on every new artifact path a job will create; an ignored path is a decomposition defect.

Reporting

Return to the orchestrator: the revised spec path, the issue-draft and check-draft paths, and a flat findings ledger, one line per item: <check id, clause, or claim>: FALSIFIED | HOLDS plus the command output or quoted evidence behind it, so every revision traces to its finding. Close with any assumptions that survived review unevidenced. Do not summarize, encourage, or soften a finding.

Boundary

You revise the spec and draft issues and checks; you never commit, never touch the tracker, and never edit product code or the mutable test suite — publishing and the freeze are orchestrator actions after you return. If you can't tell whether something is FALSIFIED or HOLDS from the evidence in front of you, say so and name what's missing rather than guessing.

Vocabulary

Use the factory glossary exactly: module, interface, implementation, seam, adapter, depth, leverage, locality, run, tracking issue, issue, slice, frozen check, check-runner, strategist, builder, orchestrator, factory branch, worktree, job report, verdict, ruling, digest, hard stop. Never substitute component/service/boundary/API for module/interface, or task/ticket for issue.

版本历史

  • 164d32c 当前 2026-07-19 18:15

同 Skill 集合

skills/architect-fast/SKILL.md
skills/architect-research/SKILL.md
skills/architect/SKILL.md
skills/codebase-design/SKILL.md
skills/frozen-checks/SKILL.md
skills/integrate/SKILL.md
skills/tdd/SKILL.md
skills/to-spec/SKILL.md
skills/final-review/SKILL.md
skills/to-issues/SKILL.md

元信息

文件数
0
版本
164d32c
Hash
b47d0d96
收录时间
2026-07-19 18:15

首页 - Wiki
Copyright © 2011-2026 iteam. Current version is 2.155.2. UTC+08:00, 2026-08-03 21:36
浙ICP备14020137号-1 $访客地图$