
This is a structured approach to letting AI agents do the grunt work while you stay in control of quality and costs. You define evals before writing code, break work into 15-minute chunks that can be verified independently, and route tasks to Haiku for simple edits or Opus for architectural decisions. The eval-first loop is smart: run tests, capture failures, implement, then compare deltas. What's refreshing is the cost discipline section that actually tells you to track tokens and retries per task, and the review guidance that says skip style nitpicks if you already have automated formatting. If you're shipping features with AI agents instead of just using them for autocomplete, this gives you guard rails without turning into process theater.
$ npx -y skills add affaan-m/everything-claude-code --skill agentic-engineering --agent claude-codeInstalls into .claude/skills of the current project.
Use this skill for engineering workflows where AI agents perform most implementation work and humans enforce quality and risk controls.
Apply the 15-minute unit rule:
Prioritize:
Do not waste review cycles on style-only disagreements when automated format/lint already enforce style.
For document reads and processing loops, check three boundaries in order:
result.ok is not a universal MCP
field. Missing or unrecognized status stays unknown.Keep the operation name, request/correlation ID when available, relevant status and error details, and the original response needed for diagnosis. Redact secrets and private document data before logging or sharing. Do not reconstruct missing responses from a later response's shape.
The adapter below uses an illustrative read-tool contract, not an MCP schema:
isError is a required boolean; when false, result.ok is a required operation
status. A successful read returns result.document with a string id; a failed
read may carry result.error. Replace this adapter with the actual tool's
contract, including its documented decoding and document validation rules.
function decodeRead(response) {
if (!response || typeof response.isError !== 'boolean') {
return { kind: 'unknown_response' };
}
if (response.isError) return { kind: 'tool_error' };
const result = response.result;
if (result?.ok === false) {
return { kind: 'operation_error', error: result.error };
}
if (result?.ok !== true) return { kind: 'unknown_response' };
const document = result.document;
if (!document || typeof document.id !== 'string') {
return { kind: 'invalid_document' };
}
return { kind: 'document', document };
}
function inspectRead(response, expectedId, decode = decodeRead) {
const outcome = decode(response);
if (outcome.kind !== 'document') return outcome;
return outcome.document.id === expectedId
? { kind: 'match', document: outcome.document }
: { kind: 'field_mismatch', actualId: outcome.document.id, expectedId };
}
Synthetic evaluation cases (not a reproduction of any historical incident):
| Input under this illustrative contract | Expected classification |
|---|---|
isError: true | tool_error; no field comparison |
isError: false, result.ok: false, maintenance error, no document | operation_error; preserve maintenance details, no mismatch |
isError: false, missing operation status | unknown_response; no field comparison |
| Confirmed success, missing document or non-string ID | invalid_document; no field comparison |
| Confirmed success, valid document with expected ID | match |
| Confirmed success, valid document with a different ID | field_mismatch |
Classification does not authorize a write or retry. Temporary maintenance remains an operation failure with its own evidence; follow the tool's documented recovery policy. A documented service-wide maintenance failure should stop a batch rather than trigger the same request for every row; a lone authorization error does not establish service-wide maintenance. If a write times out or its outcome is unknown, preserve that uncertainty and reconcile using a documented status/read-back mechanism before considering a retry. Never infer that a write failed, repeat it, or overwrite a document solely because a read failed or a returned field differs.
Track per task:
Escalate model tier only when lower tier fails with a clear reasoning gap.
Sponsors