ai-engineering-review
Codex compatibility note:
- Invoke
$skill-name: load the registered.agents/skills/<name>/SKILL.mdand execute it with Codex tools; no separateSkilltool or Claude mode switch is needed. Author.claude/**; mirrors are generated.- Track steps before execution and keep progress/evidence current. Explicit skill requests execute their protocol. Use the native question tool for required questions.
- Map foreign tool names to available host capabilities. A name mismatch needs no approval; an actually unavailable required capability stops with its step/evidence and a user question before adaptation.
- Subagent authorization: when a skill is user-invoked or AI-detected and its protocol requires subagents, that skill activation authorizes use of the required
spawn_agentsubagent(s) for that task.- Preserve protocol order unless the user approves a deviation. Workflows use
$start-workflow: gates stay fixed; other steps may flex with logged reasons and evidence.
Quick Summary
Review modes: For review/audit work, standalone defaults to
--review-only(--report-onlyalias);--fix-loopenables review → validate → authorized fix → fresh re-review, default cap 3.--fix-loop --loop-owner=callerreturns a read-only pass to the caller that owns fixes/re-review. Create triage, review, validation and re-review todo tasks first; apply the carriedreview-policycontract for LOW deferral and user-approved bounded extensions. Non-review modes and terminal findings validation keep their own dispatch.
Goal: Review a plan or a change that calls a model — LLM calls, prompts, agents, tool use, RAG, MCP, evals, guardrails, fine-tuned or classical ML — against the AI-engineering protocol, and report evidence-backed findings ranked by real consequence. Skip when nothing in scope touches a model.
Summary:
- Load protocols and project policy → detect AI surfaces and skip when none → map kind, autonomy, data, sinks and trifecta legs.
- Check framing (
AF-1–AF-6) → nine focused engineering passes (AE-1.1–AE-9.4) → conditional RAG / agent / ML sweeps → verify provider facts from current docs. - Write the AI Gate Report → validate findings with
$why-review→ report, or repair and freshly re-review under standalone--fix-loop. Code and plan modes share this authority rule.