Engifoundry review

EngiFoundry Review

Reviewer Selection

Read the complete project-owned Executor configuration and the complete schema referenced by its schemaRef. Use the configured reviewer; never replace it with a host-native subagent or another available CLI merely because that mechanism has lower overhead.

The Agent arranging Review must invoke that Reviewer in a genuinely fresh context through its verified CLI usage. It is eligible only when it can inspect the complete primary subject and evidence, perform every required check with verified available tools, start without implementation discussion or an intended conclusion, and did not perform the reviewed work. The same model as the producer is acceptable when context is genuinely fresh.

Objective unavailability before a conclusion makes Review unavailable; record the factual blocker and do not silently substitute another Reviewer. pass and rework-required are conclusions, not selection failures; never repeat Review to seek a preferred result. A new Review requires a changed subject, an invalid prior attempt, or an explicit multiple-Review requirement.

When no eligible fresh-context mechanism is actually available after inspecting host capabilities, Review is unavailable. Record that objective fact in the subject state; do not fabricate a Review attempt or reuse the producing context.

Read ./engifoundry.config.json, the project-owned workspace guide, contracts.md, the complete subject, its applicable requirements and contracts, and the evidence needed to assess it. Do not rely on handback summaries when primary evidence is available.

Review

Inspect every applicable concern:

• requirement and contract compliance;
• scope, dependencies, allowed and forbidden areas;
• correctness and completeness;
• tests and verification evidence supporting the claimed result;
• record and reference consistency;
• material delivery risk.

Order findings by impact and tie each to evidence. After rework, review the complete current subject again; never perform a delta-only approval.

The Reviewer Agent records exactly one conclusion:

• pass: no actionable factual defect remains.
• rework-required: at least one correctable subject or contract finding exists.
• blocked: an objective condition prevents Review itself from reaching a conclusion and no valid alternative evidence exists.

A missing output, failed check, incorrect content, contract conflict, or insufficient evidence is rework-required, not blocked. Never use blocked for uncertainty, difficulty, preference, or work the producing Agent can correct.

Record And State

For every actual Review attempt, the Reviewer Agent writes one new immutable Review record and updates the subject's reviewRef when it has one. Never overwrite an earlier attempt.

• Planning ready means the current planning subject has a clean-context pass and no newer invalidating fact. rework-required means a current finding remains. blocked means Review itself is factually unavailable.
• Job pending-review with a pass reviewRef means Review is complete but Exec completion or approval facts are not yet recorded. rework-required and blocked carry the matching current Review evidence.
• Other durable outputs use the immutable Review record without an invented control state.

The Reviewer Agent records findings and applicable state facts. It does not repair the reviewed subject, change its contract, accept delivery, or continue unrelated work.