Skip to content

fix(ce-code-review): cover adversarial after quota or auth no-review - #1380

Merged
tmchow merged 6 commits into
mainfrom
tmchow/debug-issue-1343
Aug 15, 2026
Merged

fix(ce-code-review): cover adversarial after quota or auth no-review#1380
tmchow merged 6 commits into
mainfrom
tmchow/debug-issue-1343

Conversation

@tmchow

@tmchow tmchow commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

When a cross-model peer starts and then dies on session quota or execution-context auth, the adversarial lens is no longer treated as covered. Fold-in announces the next eligible different-family peer, or runs the local reviewer if none remains.

An explicit user-named recipient still fails clearly instead of switching providers. A plain 429 rate limit stays a same-route retry.

The worker now emits peer skip class: so fold-in does not have to guess from free text.

Fixes #1343

Related: #1344

Validation

Focused suites tests/skills/ce-code-review-cross-model-routes.test.ts and tests/review-skill-contract.test.ts: 127 passed.

Security Disclosure

No security-relevant changes. Skip-class state is a log token only. No new persistence, credentials, or payload handling.

Agent Disclosure

  • Model: Grok Build · grok-4.6

Compound Engineering

A started peer that dies on session quota or execution-context auth
no longer keeps exclusive ownership of the adversarial lens. Fold-in
announces the next eligible peer or restores the local reviewer.
@cursor

cursor Bot commented Aug 14, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Changes adversarial review orchestration and fold-in routing when peers fail, but rules are explicit and capped (one replacement, no silent provider hopping on explicit user choice).

Overview
When a cross-model adversarial peer starts but exits without a fold-in artifact because of session quota or execution-context auth, fold-in no longer treats the adversarial lens as satisfied.

The worker now logs a structured peer skip class: token (session_quota, execution_context_auth, transient_rate_limit, other) so orchestration does not infer failure mode from free text. Skill and reference docs define a did-not-run fallback: retry one alternate attested-different peer (unless the user explicitly named the failed recipient), then run in-process adversarial-reviewer; plain 429 rate limits stay on the existing same-route retry path.

Contract and route tests lock in the new log token, fallback rules, and classification behavior.

Reviewed by Cursor Bugbot for commit 9773f83. Bugbot is set up for automated code reviews on this repo. Configure here.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f32b82a9f0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/ce-code-review/scripts/cross-model-adversarial-review.sh Outdated
Comment thread skills/ce-code-review/references/cross-model-review.md
Comment thread skills/ce-code-review/references/cross-model-review.md
Comment thread skills/ce-code-review/scripts/cross-model-adversarial-review.sh Outdated
- Classify bare HTTP 401/Unauthorized as execution-context auth
- Treat weekly/Opus/hit-your-* limits as session quota
- Wait and fold the one replacement peer; if it also dies, go local

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9773f83a67

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/ce-code-review/scripts/cross-model-adversarial-review.sh Outdated
Comment thread skills/ce-code-review/scripts/cross-model-adversarial-review.sh Outdated
Comment thread skills/ce-code-review/scripts/cross-model-adversarial-review.sh Outdated
- Keep "hit your rate limit" as transient_rate_limit
- Match 401 Unauthorized, not every "unauthorized" mention

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 876e22b12c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread skills/ce-code-review/references/cross-model-review.md Outdated
tmchow added 2 commits August 14, 2026 17:09
Drop the keyword skip-class matcher. The worker still prints bounded
skip evidence; fold-in judges quota vs auth vs rate limit from that
text and keeps the did-not-run fallback.
A throttle no longer leaves the adversarial lens uncovered: retry the
already disclosed route once, then restore the local reviewer.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 71e9614. Configure here.

Comment thread skills/ce-code-review/references/cross-model-review.md
- Allow late in-process adversarial restore after a failed same-route rate-limit retry
- Keep 429 out of the quota/auth did-not-run fallback so it does not switch recipients
@tmchow
tmchow merged commit 9154a7c into main Aug 15, 2026
5 checks passed
@github-actions github-actions Bot mentioned this pull request Aug 15, 2026
ethras added a commit to ethras/compound-engineering-orca that referenced this pull request Aug 16, 2026
* fix(ce-commit-push-pr): root PR stacks on the parent PR the user named (EveryInc#1365)

* fix(ce-babysit-pr): decode gh output as UTF-8 on Windows (EveryInc#1368)

* fix(ce-prototype): cover decisions settled by seeing, not just driving (EveryInc#1369)

* perf(tests): cut suite wall time by splitting the largest test file (EveryInc#1370)

* fix(tests): stop the cross-model routes test reading the working tree (EveryInc#1371)

* fix(ce-doc-review): ask only where a real choice exists, batch the rest (EveryInc#1373)

* chore(orca): re-pin upstream provenance baseline to 421a337

Co-authored-by: Cursor <cursoragent@cursor.com>

* fix(ce-doc-review): align Orca ownership wording with Apply routing

Co-authored-by: Cursor <cursoragent@cursor.com>

* feat(ce-prototype): add a seeing-mode craft floor and durable storage (EveryInc#1374)

* fix(ce-pov): stop the panel guessing the cross-model host argument (EveryInc#1375)

* chore(cross-model): pin the Grok peer to 4.6 (EveryInc#1376)

* docs(skills): rewrite user skill pages for accuracy and clearer use (EveryInc#1377)

* fix(commit): append known plan unit ids to commit subjects (EveryInc#1379)

* fix(ce-work): stop sandboxed workers committing in linked worktrees (EveryInc#1382)

* fix(ce-doc-review): edit HTML plans in native format (EveryInc#1381)

* fix(ce-code-review): cover adversarial after quota or auth no-review (EveryInc#1380)

* fix(skills): correct a rejected dispatch instead of spending the fallback (EveryInc#1383)

* fix(ce-compound): find Claude sessions started outside the repo root (EveryInc#1378)

* ci(windows-native): retry peer-job-runner smoke on ctypes flake (EveryInc#1384)

* fix(ce-debug): stop asking at the handoff, stop shipping unoffered work (EveryInc#1385)

* docs(solutions): record why skill gates state conditions, not git commands (EveryInc#1386)

* fix(skills): drop the residual-findings record file for real sinks (EveryInc#1387)

* fix(ce-doc-review): run the cross-model pass when CROSS_MODEL_PEERS is unset (EveryInc#1389)

* fix(ce-proof): sync with current Proof v3 contract (EveryInc#1390)

* fix(skill-authoring): make goal-first the default when authoring and reviewing skills (EveryInc#1391)

* fix(cross-model): let reviews run on Fable and pin model/effort from CE config (EveryInc#1392)

* docs(cross-model): point superseded peer benchmarks at the luna/xhigh decision (EveryInc#1393)

* fix(cross-model): discover the Codex.app-bundled codex CLI and name the peer-CLI requirement (EveryInc#1395)

* feat(cross-model): add cross_model_review_mode checkout egress gate (EveryInc#1396)

* fix(ce-compound-refresh): compare knowledge-track learnings against guidance they name (EveryInc#1399)

* docs(solutions): capture the named-guidance contradiction-check learning (EveryInc#1400)

* fix(ce-compound): prefer the repo's own frontmatter vocabulary over the Rails-era enums (EveryInc#1394)

* fix(ce-work): stop asking about branches before starting work (EveryInc#1397)

* fix(review): answer covered cases on skill prose with the condition, not a patch (EveryInc#1401)

* fix(scratch): fall back to $TMPDIR when /tmp cannot host the scratch root (EveryInc#1398)

* feat(ce-skill-work): repo-local skill for authoring, editing, reviewing, and responding to review on skills (EveryInc#1402)

* fix(ce-pov): reject non-final peer positions instead of folding them in (EveryInc#1403)

* feat(manifest): add Agent Plugins v1.0.0 manifest support (EveryInc#1345)

* chore: release main (EveryInc#1354)

* fix(ce-work): run cross-model verification on warm checkouts (EveryInc#1404)

* fix(orca): reconcile upstream skill contracts

* fix(orca): preserve additive Codex session roots

---------

Co-authored-by: Trevin Chow <trevin@trevinchow.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
ethras added a commit to ethras/compound-engineering-orca that referenced this pull request Aug 21, 2026
* fix(ce-commit-push-pr): root PR stacks on the parent PR the user named (EveryInc#1365)

* fix(ce-babysit-pr): decode gh output as UTF-8 on Windows (EveryInc#1368)

* fix(ce-prototype): cover decisions settled by seeing, not just driving (EveryInc#1369)

* perf(tests): cut suite wall time by splitting the largest test file (EveryInc#1370)

* fix(tests): stop the cross-model routes test reading the working tree (EveryInc#1371)

* fix(ce-doc-review): ask only where a real choice exists, batch the rest (EveryInc#1373)

* feat(ce-prototype): add a seeing-mode craft floor and durable storage (EveryInc#1374)

* fix(ce-pov): stop the panel guessing the cross-model host argument (EveryInc#1375)

* chore(cross-model): pin the Grok peer to 4.6 (EveryInc#1376)

* docs(skills): rewrite user skill pages for accuracy and clearer use (EveryInc#1377)

* fix(commit): append known plan unit ids to commit subjects (EveryInc#1379)

* fix(ce-work): stop sandboxed workers committing in linked worktrees (EveryInc#1382)

* fix(ce-doc-review): edit HTML plans in native format (EveryInc#1381)

* fix(ce-code-review): cover adversarial after quota or auth no-review (EveryInc#1380)

* fix(skills): correct a rejected dispatch instead of spending the fallback (EveryInc#1383)

* fix(ce-compound): find Claude sessions started outside the repo root (EveryInc#1378)

* ci(windows-native): retry peer-job-runner smoke on ctypes flake (EveryInc#1384)

* fix(ce-debug): stop asking at the handoff, stop shipping unoffered work (EveryInc#1385)

* docs(solutions): record why skill gates state conditions, not git commands (EveryInc#1386)

* fix(skills): drop the residual-findings record file for real sinks (EveryInc#1387)

* fix(ce-doc-review): run the cross-model pass when CROSS_MODEL_PEERS is unset (EveryInc#1389)

* fix(ce-proof): sync with current Proof v3 contract (EveryInc#1390)

* fix(skill-authoring): make goal-first the default when authoring and reviewing skills (EveryInc#1391)

* fix(cross-model): let reviews run on Fable and pin model/effort from CE config (EveryInc#1392)

* docs(cross-model): point superseded peer benchmarks at the luna/xhigh decision (EveryInc#1393)

* fix(cross-model): discover the Codex.app-bundled codex CLI and name the peer-CLI requirement (EveryInc#1395)

* feat(cross-model): add cross_model_review_mode checkout egress gate (EveryInc#1396)

* fix(ce-compound-refresh): compare knowledge-track learnings against guidance they name (EveryInc#1399)

* docs(solutions): capture the named-guidance contradiction-check learning (EveryInc#1400)

* fix(ce-compound): prefer the repo's own frontmatter vocabulary over the Rails-era enums (EveryInc#1394)

* fix(ce-work): stop asking about branches before starting work (EveryInc#1397)

* fix(review): answer covered cases on skill prose with the condition, not a patch (EveryInc#1401)

* fix(scratch): fall back to $TMPDIR when /tmp cannot host the scratch root (EveryInc#1398)

* feat(ce-skill-work): repo-local skill for authoring, editing, reviewing, and responding to review on skills (EveryInc#1402)

* fix(ce-pov): reject non-final peer positions instead of folding them in (EveryInc#1403)

* feat(manifest): add Agent Plugins v1.0.0 manifest support (EveryInc#1345)

* chore: release main (EveryInc#1354)

* fix(ce-work): run cross-model verification on warm checkouts (EveryInc#1404)

* fix(ce-skill-work): author for Sol/Fable, not Opus-era procedure (EveryInc#1408)

* docs(skill-design): retarget stale learning citations (EveryInc#1409)

* fix(ce-setup): support read-only sandboxes (EveryInc#1407)

* fix(ce-skill-work): require pointer descriptions (EveryInc#1410)

* chore: release main (EveryInc#1405)

* fix(ce-work): let a project-defined shipping process override ce-commit-push-pr (EveryInc#1416)

* fix(ce-doc-review): state the CROSS_MODEL_PEERS gate as a condition at the gate (EveryInc#1421)

* fix(ce-strategy): ground the interview in the repo and share the file safely (EveryInc#1419)

* fix(ce-babysit-pr): fall back for private ref 404 (EveryInc#1418)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-commit-push-pr): make medium and large PR descriptions scannable (EveryInc#1422)

* chore: release main (EveryInc#1420)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix(ce-plan): make Goal Capsule Objective outcome-shaped with a Means slot (EveryInc#1424)

* fix(manifest): drop Agent Plugins $schema so Codex stops truncating skills at 8KB (EveryInc#1426)

* fix(manifest): also block Agent Plugins $schema while skill frontmatter is non-conformant (EveryInc#1427)

* chore: release main (EveryInc#1425)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* docs(solutions): capture why the Agent Plugins $schema is a host routing switch (EveryInc#1428)

* fix(skills): Make CE portable on no-checkout / shared-workspace hosts (EveryInc#1429)

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Trevin Chow <tmchow@users.noreply.github.com>

* chore: release main (EveryInc#1430)

* fix(orca): reconcile fork identity and guards with upstream 3.22.4

Re-pin the fork release identity to 3.22.4-orca.5 (release-as, version
literals in the Orca suites), resolve the upstream-currency check
against the latest compound-engineering-v* release tag instead of the
upstream/main tip (main carries unreleased commits), regenerate the
skill-local Orca role-registry bundles, re-anchor the lfg step-8
ordering test to upstream's rewritten Ship step, and compact the
ce-simplify-code Orca hook to a conditional pointer so the skill fits
Codex's 8000-byte prompt bound (mechanism stays in
references/orca-review-dispatch.md, which already owns it).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019PxdaWqiHDDsPPBHdDevQC

* ci(orca): fetch upstream release tags for the provenance currency check

The check now resolves the latest compound-engineering-v* tag; without
tags the CI fallback compared the pin to the upstream/main tip, which
fails whenever upstream carries unreleased commits.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019PxdaWqiHDDsPPBHdDevQC

* ci: pin bun to 1.3.14

bun 1.4.0 (picked up by bun-version: latest on 2026-08-21) hangs the
subprocess-heavy suites on Linux runners: ce-pov and cross-model tests
hit their 5s/20s caps and the profile-lock contention test times out,
inflating the test job from ~2m to 13m. The same suites pass on 1.3.14
locally and in the last green run (2026-08-19).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019PxdaWqiHDDsPPBHdDevQC

---------

Co-authored-by: Trevin Chow <trevin@trevinchow.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Ruslan Kurkebayev <kurkebayev.ruslan@gmail.com>
Co-authored-by: cmbish <carter.m.bish@gmail.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Trevin Chow <tmchow@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
ethras added a commit to ethras/compound-engineering-orca that referenced this pull request Sep 1, 2026
* fix(ce-commit-push-pr): root PR stacks on the parent PR the user named (EveryInc#1365)

* fix(ce-babysit-pr): decode gh output as UTF-8 on Windows (EveryInc#1368)

* fix(ce-prototype): cover decisions settled by seeing, not just driving (EveryInc#1369)

* perf(tests): cut suite wall time by splitting the largest test file (EveryInc#1370)

* fix(tests): stop the cross-model routes test reading the working tree (EveryInc#1371)

* fix(ce-doc-review): ask only where a real choice exists, batch the rest (EveryInc#1373)

* feat(ce-prototype): add a seeing-mode craft floor and durable storage (EveryInc#1374)

* fix(ce-pov): stop the panel guessing the cross-model host argument (EveryInc#1375)

* chore(cross-model): pin the Grok peer to 4.6 (EveryInc#1376)

* docs(skills): rewrite user skill pages for accuracy and clearer use (EveryInc#1377)

* fix(commit): append known plan unit ids to commit subjects (EveryInc#1379)

* fix(ce-work): stop sandboxed workers committing in linked worktrees (EveryInc#1382)

* fix(ce-doc-review): edit HTML plans in native format (EveryInc#1381)

* fix(ce-code-review): cover adversarial after quota or auth no-review (EveryInc#1380)

* fix(skills): correct a rejected dispatch instead of spending the fallback (EveryInc#1383)

* fix(ce-compound): find Claude sessions started outside the repo root (EveryInc#1378)

* ci(windows-native): retry peer-job-runner smoke on ctypes flake (EveryInc#1384)

* fix(ce-debug): stop asking at the handoff, stop shipping unoffered work (EveryInc#1385)

* docs(solutions): record why skill gates state conditions, not git commands (EveryInc#1386)

* fix(skills): drop the residual-findings record file for real sinks (EveryInc#1387)

* fix(ce-doc-review): run the cross-model pass when CROSS_MODEL_PEERS is unset (EveryInc#1389)

* fix(ce-proof): sync with current Proof v3 contract (EveryInc#1390)

* fix(skill-authoring): make goal-first the default when authoring and reviewing skills (EveryInc#1391)

* fix(cross-model): let reviews run on Fable and pin model/effort from CE config (EveryInc#1392)

* docs(cross-model): point superseded peer benchmarks at the luna/xhigh decision (EveryInc#1393)

* fix(cross-model): discover the Codex.app-bundled codex CLI and name the peer-CLI requirement (EveryInc#1395)

* feat(cross-model): add cross_model_review_mode checkout egress gate (EveryInc#1396)

* fix(ce-compound-refresh): compare knowledge-track learnings against guidance they name (EveryInc#1399)

* docs(solutions): capture the named-guidance contradiction-check learning (EveryInc#1400)

* fix(ce-compound): prefer the repo's own frontmatter vocabulary over the Rails-era enums (EveryInc#1394)

* fix(ce-work): stop asking about branches before starting work (EveryInc#1397)

* fix(review): answer covered cases on skill prose with the condition, not a patch (EveryInc#1401)

* fix(scratch): fall back to $TMPDIR when /tmp cannot host the scratch root (EveryInc#1398)

* feat(ce-skill-work): repo-local skill for authoring, editing, reviewing, and responding to review on skills (EveryInc#1402)

* fix(ce-pov): reject non-final peer positions instead of folding them in (EveryInc#1403)

* feat(manifest): add Agent Plugins v1.0.0 manifest support (EveryInc#1345)

* chore: release main (EveryInc#1354)

* fix(ce-work): run cross-model verification on warm checkouts (EveryInc#1404)

* fix(ce-skill-work): author for Sol/Fable, not Opus-era procedure (EveryInc#1408)

* docs(skill-design): retarget stale learning citations (EveryInc#1409)

* fix(ce-setup): support read-only sandboxes (EveryInc#1407)

* fix(ce-skill-work): require pointer descriptions (EveryInc#1410)

* chore: release main (EveryInc#1405)

* fix(ce-work): let a project-defined shipping process override ce-commit-push-pr (EveryInc#1416)

* fix(ce-doc-review): state the CROSS_MODEL_PEERS gate as a condition at the gate (EveryInc#1421)

* fix(ce-strategy): ground the interview in the repo and share the file safely (EveryInc#1419)

* fix(ce-babysit-pr): fall back for private ref 404 (EveryInc#1418)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-commit-push-pr): make medium and large PR descriptions scannable (EveryInc#1422)

* chore: release main (EveryInc#1420)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix(ce-plan): make Goal Capsule Objective outcome-shaped with a Means slot (EveryInc#1424)

* fix(manifest): drop Agent Plugins $schema so Codex stops truncating skills at 8KB (EveryInc#1426)

* fix(manifest): also block Agent Plugins $schema while skill frontmatter is non-conformant (EveryInc#1427)

* chore: release main (EveryInc#1425)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* docs(solutions): capture why the Agent Plugins $schema is a host routing switch (EveryInc#1428)

* fix(skills): Make CE portable on no-checkout / shared-workspace hosts (EveryInc#1429)

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Trevin Chow <tmchow@users.noreply.github.com>

* chore: release main (EveryInc#1430)

* docs(readme): fix Grok Bot install instructions (EveryInc#1431)

* docs(readme): fix Grok Bot install path (EveryInc#1432)

* fix(ce-babysit-pr): no unasked base merges, 91% smaller, pipelined stacks (EveryInc#1433)

* docs(skill-design): 8KB is a ceiling, not the target, in the size-restructure playbook (EveryInc#1437)

* fix(ce-skill-work): savings come from structure, not squeezed sentences (EveryInc#1460)

* fix(ce-strategy): 53% smaller SKILL.md, references carry the interview and update run (EveryInc#1436)

* fix(ce-strategy): STRATEGY.md is the agreed shared project doc (EveryInc#1459)

* docs(skill-design): delegating skills need a live eval, not a fake-boundary one (EveryInc#1462)

* fix(ce-commit-push-pr): write the scope map first and audit the opening against it (EveryInc#1457)

* fix(ce-proof): 62% smaller SKILL.md, references carry the API and workflows (EveryInc#1443)

* docs(skill-design): record the three relocation failures from the 8KB sweep (EveryInc#1453)

* fix(ce-handoff): 60% smaller SKILL.md, references carry create and resume (EveryInc#1446)

* fix(ce-handoff): separate the writing agent's voice from the user's in handoffs (EveryInc#1464)

* fix(ce-explain): 28% smaller SKILL.md, orchestration moves to a reference (EveryInc#1451)

* fix(ce-debug): 50% smaller SKILL.md, references carry the phase procedures (EveryInc#1449)

* fix(ce-prototype): 43% smaller SKILL.md, references carry scoping and build (EveryInc#1444)

* fix(ce-sweep): 57% smaller SKILL.md, references/run.md carries the phases (EveryInc#1439)

* fix(ce-debug): name the fixed-but-unpushed pipeline outcome at both ends (EveryInc#1463)

* fix(ce-retune): 39% smaller SKILL.md, references carry the rest (EveryInc#1438)

* fix(ce-commit-push-pr): 69% smaller SKILL.md, references carry the rest (EveryInc#1454)

* fix(ce-compound-refresh): 70% smaller SKILL.md, references carry the rest (EveryInc#1442)

* fix(ce-doc-review): 65% smaller SKILL.md, references carry the rest (EveryInc#1450)

* fix(ce-optimize): 82% smaller SKILL.md, references carry the phase procedures (EveryInc#1456)

* fix(ce-setup): 44% smaller SKILL.md, references carry the rest (EveryInc#1445)

* fix(ce-dogfood): 67% smaller SKILL.md, references/phases.md carries the procedure (EveryInc#1447)

* fix(ce-product-pulse): 51% smaller SKILL.md, references carry config and run (EveryInc#1448)

* fix(ce-ideate): 84% smaller SKILL.md, references carry the phases (EveryInc#1455)

* fix(ce-test-browser): 43% smaller SKILL.md, references carry the rest (EveryInc#1441)

* fix(ce-debug): body enum names fixed-not-pushed like the producer and consumer (EveryInc#1466)

* fix(ce-test-browser): resolve the dev-server port with a bundled script instead of a prose-emitted line (EveryInc#1468)

* fix(ce-resolve-pr-feedback): 31% smaller SKILL.md, references carry the rest (EveryInc#1435)

* fix(ce-resolve-pr-feedback): judge top-level PR comments, not only review threads (EveryInc#1467)

* fix(ce-pov): 66% smaller SKILL.md, references carry the rest (EveryInc#1440)

* fix(ce-explain): 32% smaller SKILL.md, the Phase 6 close moves to a required-read reference (EveryInc#1469)

* fix(ce-debug): audit body pins by provenance, relocate what only wording held in the window (EveryInc#1472)

* fix(ce-code-review): 86% smaller SKILL.md, four step references carry the rest (EveryInc#1471)

* fix(ce-compound): 90% smaller SKILL.md, references carry the phases (EveryInc#1477)

* fix(ce-work): 48% smaller SKILL.md, references carry the execution manual (EveryInc#1478)

* fix(ce-plan): 71% smaller SKILL.md, five phase references carry the rest (EveryInc#1470)

* fix(ce-plan): audit body pins by provenance, relocate what only wording held in the window (EveryInc#1475)

* fix(ce-brainstorm): 86% smaller SKILL.md, six references carry the phases (EveryInc#1476)

* fix(lfg): 72% smaller SKILL.md, six required-read references carry the rest (EveryInc#1479)

* docs(skill-design): third-sweep learnings — place blocks by executing step, audit pins, size evals to reach (EveryInc#1483)

* feat(skill-eval): host-CLI catalog for skill-behavior A/B (EveryInc#1484)

* fix(skills): make model-invoked descriptions context pointers (EveryInc#1486)

* fix(ce-brainstorm): ask only decisions the environment cannot settle (EveryInc#1487)

* fix(skills): complete below-cap refactor sweep (EveryInc#1490)

* fix(ce-resolve-pr-feedback): keep replies out of pending reviews (EveryInc#1491)

* fix(skills): unify needs-human decision handoff (EveryInc#1492)

* fix(cross-model): elevate peer start when host sandbox blocks provider network (EveryInc#1496)

* fix: honor active model-config keys in skill runs (surface + salience) (EveryInc#1489)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(skills): remove obsolete dispatch context hook (EveryInc#1499)

* fix(ce-optimize): keep multi-objective wins and screen expensive runs (EveryInc#1506)

* fix(skills): load ce-plan and ce-work phases from references under the 8KB cap (EveryInc#1508)

* test(cross-model): give the subprocess-heavy route suites a 30s per-test ceiling (EveryInc#1510)

* fix(cross-model): accept provider-qualified codex model ids in cross_model_model (EveryInc#1501)

* docs(agent-plugins): attribute the 8000-byte skill cap to its real owner (EveryInc#1511)

* chore: release main (EveryInc#1434)

* fix(ce-code-review): stop the reviewer template from embedding the diff twice (EveryInc#1512)

* chore: release main (EveryInc#1513)

* fix(skills): right-size ceremony for small work in ce-plan, ce-brainstorm, and ce-work (EveryInc#1514)

* fix(cross-model): attest Grok Build as a host and bind its native CLI (EveryInc#1516)

* fix(ce-doc-review): activate product-lens only on a product position (EveryInc#1517)

* docs(ce-skill-work): size a behavioral eval to the decision, not the workflow (EveryInc#1520)

* fix(cross-model): bound peer retries and raise review headroom (EveryInc#1519)

* chore: release main (EveryInc#1515)

* docs: refresh cross-model peer learnings (EveryInc#1521)

* docs(solutions): refresh stale learnings against current code (EveryInc#1524)

* docs(readme): make the README a front door, split detail into linked docs (EveryInc#1525)

* fix(skills): match blocking questions from the current tool list (EveryInc#1526)

* fix(lfg): invoke ce-code-review from the host catalog path (EveryInc#1527)

* fix(orca): reconcile upstream 3.23.2 overlay

* fix(ce-babysit-pr): defer to GitHub for trunk drift on managed stacks (EveryInc#1529)

* fix(ce-code-review): collect async reviewer returns (EveryInc#1530)

* chore: release main (EveryInc#1528)

* fix(cross-model): drop empty Codex usage files and unblock heartbeat teardown (EveryInc#1533)

* fix(skills): anchor the plan Objective above the component being changed (EveryInc#1535)

* docs(solutions): record what a condition costs a literal host (EveryInc#1536)

* fix(ce-skill-work): treat a removed concrete shape as a behavior change (EveryInc#1537)

* fix(ce-code-review): keep missing peer config from skipping the pass (EveryInc#1538)

* docs(strategy): add STRATEGY.md as the project's direction anchor (EveryInc#1539)

* fix(concepts): give CONCEPTS.md a retention lifecycle (EveryInc#1540)

* fix(peer-job-runner): keep Windows reap from killing a recycled pid (EveryInc#1541)

* chore: release main (EveryInc#1534)

* fix(ce-compound-refresh): preserve supported guidance across regressions (EveryInc#1542)

* docs(guides): move skill catalog from docs/skills to skills/guides (EveryInc#1551)

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

* perf(tests): reuse seed git fixtures to cut suite wall time (EveryInc#1548)

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

* fix: preserve release automation ownership after upstream sync

* fix(ce-work): parallelize independent work by default; fix(lfg): narrate progress and gate DONE (EveryInc#1558)

Co-authored-by: Claude <noreply@anthropic.com>

* fix(ce-compound): check in-repo absolute path citations (EveryInc#1561)

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

* fix(guides): keep catalog out of the plugin skills tree (EveryInc#1571)

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

* fix(ce-code-review): take review criteria from a repo-owned standards file (EveryInc#1572)

* docs(guides): document the CODING_STANDARDS.md criteria file (EveryInc#1573)

* fix(ce-work): scrub GIT_INTERNAL_SUPER_PREFIX from inherited git env (EveryInc#1564)

* fix(ce-setup): detect the retired Codex tool map (EveryInc#1577)

* fix(lfg): record unapplied review findings in the PR body, not tickets (EveryInc#1580)

* fix(ce-plan): require a holdable Goal Capsule Objective (EveryInc#1592)

* fix(ce-commit-push-pr): let a PR opening carry what motivates it (EveryInc#1576)

* fix(ce-code-review): anchor reviewer checks to named canonical frameworks (EveryInc#1594)

* fix(ce-commit-push-pr): catch a PR opening that names the mechanism (EveryInc#1595)

* fix(ce-babysit-pr): keep stale-computation degrade off the settle clock (EveryInc#1568)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* docs(ce-compound): capture at the completion checkpoint, not at merge (EveryInc#1596)

* fix(ce-work): allow parallel subagent waves in a shared workspace (EveryInc#1598)

* fix(codex): reconcile config-only plugin installs (EveryInc#1599)

* fix(ce-debug): gate hypotheses on a runnable reproduction check (EveryInc#1600)

* docs(lfg): align leftover-findings FAQ with PR-body checklist (EveryInc#1582)

Co-authored-by: chouti <chouti@upai.com>

* fix(ce-commit): make the stage-and-commit example PowerShell-safe (EveryInc#1587)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-work): require evidence before unavailable review fallback (EveryInc#1578)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-babysit-pr): persist review invariant rounds across heads (EveryInc#1585)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-code-review): isolate testing reviewer mutation work (EveryInc#1584)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-work): out-of-repo units have no git-derived completion (EveryInc#1583)

Co-authored-by: Trevin Chow <trevin@trevinchow.com>

* fix(ce-work): verify worktree snapshot fidelity before isolated dispatch (EveryInc#1602)

* feat(harness): add opencode as a named peer and work engine (EveryInc#1604)

Co-authored-by: Hally Maschine <hally@rocketable.com>

* fix(ci): absorb the windows _ctypes flake without hiding real failures (EveryInc#1605)

* chore: release main (EveryInc#1543)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix(orca): re-pin upstream baseline to 3.24.0

Record compound-engineering-v3.24.0 as the fork baseline, retarget the
pending release identity to 3.24.0-orca.7, and regenerate skill-local
role registries.

---------

Co-authored-by: Trevin Chow <trevin@trevinchow.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Ruslan Kurkebayev <kurkebayev.ruslan@gmail.com>
Co-authored-by: cmbish <carter.m.bish@gmail.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Trevin Chow <tmchow@users.noreply.github.com>
Co-authored-by: Kevin Old <kevin@kevinold.com>
Co-authored-by: Dennis Traub <dennis.traub@gmail.com>
Co-authored-by: Kieran Klaassen <kieranklaassen@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: chouti <chouti@gmail.com>
Co-authored-by: Saurabh <saurabhkagent@gmail.com>
Co-authored-by: chouti <chouti@upai.com>
Co-authored-by: Morgan Touverey Quilling <morgan@mtq.io>
Co-authored-by: Thomas Steibl <309042+Rowdy@users.noreply.github.com>
Co-authored-by: Hally Maschine <hally@rocketable.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ce-code-review: detect exhausted peer routes before review payload egress

1 participant