Skip to content

chore: sync the harness for D26 (#160) - #161

Merged
MatheusCastro99 merged 1 commit into
masterfrom
chore/160-harness-d26
Nov 10, 2026
Merged

MatheusCastro99 merged 1 commit into
masterfrom
chore/160-harness-d26

Conversation

@MatheusCastro99

Copy link
Copy Markdown
Owner

Closes #160. Harness sync for ProdHarnessConfig D26 (from 9d6087f, ProdHarnessConfig#7), from the meta-analysis in Evidence/research/13. Harness files, .claude/CLAUDE.md and project skills only: no app code changes.

Changes

  • G2 fixes go to the forked builder (E1). After G2, review makes typo and one-line doc fixes itself; anything else becomes a G2 fixes sub-phase built by build <n> amend on Sonnet, followed by one narrow re-review.
  • Auto-fix stays off (E2). set_monitor is allowed in settings.json; review turns Auto-fix off after each push and before the ready-to-merge note.
  • An Epic's close is one sequence (E3): the audit, a docs pass with its findings, then the cleanup. close.cjs exits 4 when there's nothing to clean up, and takes the current branch only when it's the issue's.
  • The handoff checks each required CI job by name (E5) (handoff-check --verify-head), and the build checks what it posts for details the plan says never to quote.
  • A read-only ui-check agent (new, .claude/agents/ui-check.md) runs the UI check in review (E7), outside the builder's context.
  • The Stop gate reads .harness/work/<n>/known-failures.md (E9): a failure you send to an open issue at G2 is reported, not blocked. It fails closed on anything it can't recognise.
  • scope (E4, E6, E8, E10, E11, C6): drafting before /scope, deferred items and new rules planned, fewer questions, a warning when the checkout is stale, the surveyor only for unfamiliar code.
  • Former trials: T1, T4, T6 and T7 are standing rules; T5 (security-review) is dropped.
  • Refactor: the installed files point at ProdHarnessConfig's Evidence/ paths; stale-docs --fail; every hook logs.
  • ux-design §4: the ui-check agent runs the UI check in review, not the builder, and the screenshots go in its report for G2. The rest of the method (the in-memory database, the run commands, the pane's limits) is unchanged.
  • CLAUDE.md: frontend-design is a standing rule now, not trial T7.
  • CLAUDE.md takes the template's new workflow lines: the narrow re-review after an amend on Small, the Epic close, drafting before /scope, and where the UI check runs. Standing rules are untouched.
  • .claude/harness-manifest.json: refreshed from 9d6087f.

Verification

  • ProdHarnessConfig's tests: 135/135, and a fresh-context red-team review with a narrow re-review on ProdHarnessConfig#7.
  • The installer's dry run and --apply reported no conflicts and no local edits.
  • check fast passes through the new runner here (3 commands).
  • stale-docs --staged flagged only .claude/ directory mentions, all still accurate; a search for what D26 changes (the builder running the UI check, trial labels, security-review, reviewOnReady) found only the ux-design lines updated here.

🤖 Generated with Claude Code

Installs ProdHarnessConfig D26 (9d6087f, ProdHarnessConfig#7): G2 fixes
built by the forked builder, Auto-fix kept off, the Epic close as one
sequence, required CI jobs checked by name, a read-only ui-check agent
in review, a known-failure list for the Stop gate, and the scope rules
from research/13. CLAUDE.md takes the template's new workflow lines.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@MatheusCastro99
MatheusCastro99 marked this pull request as ready for review November 10, 2026 21:26
Copilot AI balanced review requested due to automatic review settings November 10, 2026 21:26

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@MatheusCastro99
MatheusCastro99 merged commit 66435c2 into master Nov 10, 2026
8 checks passed
@MatheusCastro99
MatheusCastro99 deleted the chore/160-harness-d26 branch November 10, 2026 21:27
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.

Sync the harness for D26

2 participants