From c979ac37c361da564dcce100a4f2623d94ef54c8 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" <41898282+github-actions[bot]@users.noreply.github.com> Date: Mon, 29 Jun 2026 07:31:56 +0000 Subject: [PATCH] Sync generated provider output --- .../skills/impeccable/reference/critique.md | 28 ++++++++++++++----- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .kiro/skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .pi/skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- .trae/skills/impeccable/reference/critique.md | 19 +++++++++++-- .../skills/impeccable/reference/critique.md | 19 +++++++++++-- 13 files changed, 213 insertions(+), 43 deletions(-) diff --git a/.agents/skills/impeccable/reference/critique.md b/.agents/skills/impeccable/reference/critique.md index 6a13c48f5..331ea9972 100644 --- a/.agents/skills/impeccable/reference/critique.md +++ b/.agents/skills/impeccable/reference/critique.md @@ -5,8 +5,9 @@ Resolve one stable target, run two independent assessments, synthesize a design ### Hard Invariants - Assessment A (design review) and Assessment B (detector/browser evidence) are both required. +- Assessment A and B MUST run as two isolated sub-agents whenever a sub-agent/Task tool is exposed. Running them inline in this context is "possible" but is NOT permitted; it is a degraded run. Inline is allowed ONLY when no sub-agent tool exists (or the user declined, on harnesses that ask). +- If you degrade for any reason, the report's first line MUST be a banner: `⚠️ DEGRADED: single-context ()`. A silent degraded critique is a failed critique. - Assessment A must finish before detector findings enter the parent synthesis context. Detector output is deterministic, but it still anchors judgment. -- If sub-agents are unavailable, fall back sequentially: finish and record Assessment A first, then run Assessment B, then synthesize. - A skipped detector is a failed critique run unless `detect.mjs` is missing or crashes after a real attempt. - Viewable targets require browser inspection when available. - Any local server started only for critique visualization must run in the background, have a recorded stop method, and be stopped before final reporting unless the user asks to keep it. @@ -27,14 +28,21 @@ Resolve one stable target, run two independent assessments, synthesize a design ### Assessment Orchestration -Delegate Assessment A and Assessment B to separate sub-agents when possible. They must not see each other's output. Do not show findings to the user until synthesis. +Delegate Assessment A and Assessment B to separate sub-agents. They must not see each other's output. Do not show findings to the user until synthesis. -Codex sub-agent gate: +Sub-agent gate (all harnesses): +- Unless a harness-specific gate below overrides this, spawn A and B as two isolated, parallel sub-agents whenever a sub-agent/Task tool is exposed. This is the default and is mandatory; do not run them inline because it is faster. +- "Unavailable" means exactly one thing: no sub-agent/Task tool is exposed in this session (or, on harnesses that ask, the user declined). It does not mean inconvenient. +- If and only if sub-agents are unavailable, fall back sequentially: finish and record Assessment A, then run Assessment B, then synthesize, and emit the degraded banner. +- Whichever path you take, declare it in the report header (see Report header provenance). Skipping sub-agents without the banner is the most common failure of this command. + +Codex sub-agent gate (overrides the default above; Codex's permission model requires asking before spawning): +- Asking is the normal path, not a degradation. Approving and spawning is the dual-agent path; do not emit the degraded banner just for asking. - If `spawn_agent` is exposed and the user explicitly allowed sub-agents, delegation, or parallel agent work, spawn A and B immediately. - If `spawn_agent` is exposed but the user did not explicitly allow sub-agents, ask exactly once: "Impeccable critique is designed to run two independent sub-agents for an unanchored assessment. May I use sub-agents for this critique?" Then stop until the user answers. -- If allowed, spawn A and B. If declined, run sequentially and report `Assessment independence: degraded (sub-agents declined by user)`. -- If `spawn_agent` is not exposed, do not ask; run sequentially and report `Assessment independence: degraded (spawn_agent unavailable in this session)`. -- If spawning fails after permission, run sequentially and report `Assessment independence: degraded (sub-agent spawn failed: )`. +- If allowed, spawn A and B. If declined, run sequentially and lead the report with `⚠️ DEGRADED: single-context (sub-agents declined by user)`. +- If `spawn_agent` is not exposed, do not ask; run sequentially and lead with `⚠️ DEGRADED: single-context (spawn_agent unavailable in this session)`. +- If spawning fails after permission, run sequentially and lead with `⚠️ DEGRADED: single-context (sub-agent spawn failed: )`. Prefer `fork_context: false` with self-contained prompts containing cwd, target, live URL, references, product context, and output contract. If using `fork_context: true`, omit `agent_type`, `model`, and `reasoning_effort`. If browser automation is available, each assessment creates its own new tab. Never reuse an existing tab, even if it is already at the right URL. @@ -69,7 +77,7 @@ node .agents/skills/impeccable/scripts/detect.mjs --json [target] Browser visualization is required for a viewable target when browser automation is available. Use a localhost dev/static URL for local files; avoid `file://` unless the available browser explicitly supports this workflow. Overlay flow: -1. Create a fresh tab and navigate. +1. Create a fresh tab and navigate. Prefer the harness's native/browser-canvas screenshot path before hand-rolling a Playwright/Puppeteer script; only fall back to a custom script when no native browser tool is exposed. 2. Preflight mutable injection by setting `document.title` and appending a `