Skip to content

chore(repo-harness): refresh repo-local harness config to 0.32.0 - #590

Merged
Ancienttwo merged 3 commits into
mainfrom
chore/repo-harness-0.32-config
Nov 13, 2026
Merged

Ancienttwo merged 3 commits into
mainfrom
chore/repo-harness-0.32-config

Conversation

@Ancienttwo

@Ancienttwo Ancienttwo commented Nov 13, 2026 •

Copy link
Copy Markdown
Owner

Refresh the repo-local repo-harness configuration to the repo-harness 0.32.0 layout.

Change

  • .ai/harness/policy.json is rewritten to the 0.32 key set. publication.acceptance_merge stays on. Seven of the removed keys are still read by .ai/hooks/lib/workflow-state.sh: harness.failure_log_file, harness.events_file, harness.checks_file, harness.handoff_file, handoff_resume.resume_packet_file, tasks.reviews_dir and tasks.notes_dir. The helper is kept by the recorded upgrade decision (.ai/harness/upgrade-decisions.json), nothing in this repository sources it, and each read falls back to a default equal to the removed value. Behavior is therefore unchanged. archcontext product source reads none of the removed keys.
  • .gitignore adopts the slim managed block. The runtime-state ignores (.ai/harness/{runs,state,evidence,planning,security,checks,...}, .claude/settings.local.json) now live only in the user-level git excludes file (~/.config/git/ignore, repo-harness runtime-state block). On a host without that file, those paths show as untracked. The owner accepted that trade-off.
  • Retired assets are removed: .ai/hooks/README.md, .claude/templates/*.template.md and docs/prds/prd.template.md. Dated verification and research records under docs/verification/ and docs/researches/ that cite .ai/hooks/README.md are left as they are: they record the state at their date.
  • repo-harness' generic docs/architecture/index.md stub is not added: docs/architecture/archimap.html stays the single architecture entry, per AGENTS.md.
  • archimap.html remark R-A4 is narrowed to what is still stale, the module links in .ai/context/capabilities.json. This refresh removed the docs/architecture/index.md requirement from policy.json; workflow-contract.json is unchanged.

Verification

  • repo-harness init (apply, then check-context-files: SAFE).
  • repo-harness state resolve: idle, no blockers.
  • git check-ignore -v on this host matches .ai/harness/runs/, .ai/harness/state/ and .claude/settings.local.json through the user-level excludes.

Risk

Config and docs only, with no source or model change. Because policy.json carries the merge policy, the owner merges this PR.

Rollback: git revert the squash commit.

- rewrite .ai/harness/policy.json to the 0.32 key set (drops ~200 keys with no
  reader in 0.32; archcontext source does not read them)
- .gitignore: adopt the slim managed block; runtime-state ignores now live in
  the user-level git excludes file
- remove retired assets: .ai/hooks/README.md, .claude/templates/*.template.md,
  docs/prds/prd.template.md
- skip repo-harness' generic docs/architecture/index.md stub: archimap.html
  stays the single architecture entry per AGENTS.md

Verification: repo-harness init (apply + check-context-files SAFE);
repo-harness state resolve -> idle, no blockers.
Risk: low; config-only, no source or model change.
Rollback: git revert this commit.
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-13T10:00:11.966743Z
We couldn’t complete this review. Comment @codex review to try again.
e3e0f37 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@Ancienttwo

Copy link
Copy Markdown
Owner Author

repo-harness acceptance claim
Role: acceptor
Round: 1
Subject: e3e0f37
Tier: medium
Agent: codex (gpt-6.1-terra), read-only

@Ancienttwo Ancienttwo left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

repo-harness acceptance review
Round: 1
Subject: e3e0f37
Tier: medium
Verdict: FAIL
Reviewer: codex (gpt-6.1-terra)
Model relation: different-family

Cross-family review (Codex CLI, read-only). publication.acceptance_merge stays on. The .gitignore change removes repository-level ignores for local runtime state and settings, so ignoring them depends on each contributor's global excludes; the claim that no repository code reads the removed policy keys is inaccurate. Repository files cannot show whether repo-harness 0.32.0 defaults replace the removed safety settings (enforcement.completion_requires_checks, verification_gate, worktree_guard, review circuit breakers).

NITs

  • F3: docs/verification/practice-assets-s6-release-gate.md:364 and other readbacks still name the deleted hooks README as the hook policy authority.
  • F4: docs/architecture/archimap.html:417 still says policy.json requires docs/architecture/index.md; refresh through the archimap workflow.

Findings outside the diff

  • P2 F2 at .ai/hooks/lib/workflow-state.sh:133: The retained helper reads removed keys (harness.failure_log_file, harness.events_file, handoff_resume.resume_packet_file, tasks.reviews_dir, tasks.notes_dir, harness.checks_file, harness.handoff_file); its defaults match the removed values, but it contradicts the PR's 'reads none' claim. -> Correct the PR description and establish whether the helper is active or legacy.

Comment thread .gitignore
.ai/harness/mcp/
.ai/harness/triage/*
!.ai/harness/triage/.gitkeep
# Host and tool directories

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

P1 F1

Removed ignores for .ai/harness/{runs,state,evidence,planning,checks,...}, .claude/settings.local.json and related state. With core.excludesfile=/dev/null, hook-events.jsonl, agent-notice, planning state, evidence including SQLite/WAL files and .claude/settings.local.json appear untracked and can be staged on another host.

Fix: Restore repository-level ignores for local runtime state and settings, or an equally reliable repository-level protection.

@Ancienttwo

Copy link
Copy Markdown
Owner Author

repo-harness acceptance record
Claim: #590 (comment)

Round Role Subject Verdict
1 codex (gpt-6.1-terra), read-only e3e0f37be9aacd3461b357d39a7b0356dec5e795 FAIL: 1 P1, 1 P2, 2 P3

Model relation: different-family

Findings:

  • P1 F1 at .gitignore:58: Removed ignores for .ai/harness/{runs,state,evidence,planning,checks,...}, .claude/settings.local.json and related state. With core.excludesfile=/dev/null, hook-events.jsonl, agent-notice, planning state, evidence including SQLite/WAL files and .claude/settings.local.json appear untracked and can be staged on another host. -> Restore repository-level ignores for local runtime state and settings, or an equally reliable repository-level protection.
  • P2 F2 at .ai/hooks/lib/workflow-state.sh:133: The retained helper reads removed keys (harness.failure_log_file, harness.events_file, handoff_resume.resume_packet_file, tasks.reviews_dir, tasks.notes_dir, harness.checks_file, harness.handoff_file); its defaults match the removed values, but it contradicts the PR's 'reads none' claim. -> Correct the PR description and establish whether the helper is active or legacy.
  • P3 F3: docs/verification/practice-assets-s6-release-gate.md:364 and other readbacks still name the deleted hooks README as the hook policy authority.
  • P3 F4: docs/architecture/archimap.html:417 still says policy.json requires docs/architecture/index.md; refresh through the archimap workflow.

@Ancienttwo

Copy link
Copy Markdown
Owner Author

Decision (owner, pane t12, 2026-11-13, relayed by advisor t12) on GPT F1 (P1): user-level ignores are accepted. The owner keeps the .gitignore slimming and accepts the risk on hosts that lack ~/.config/git/ignore. On the owner's Mac, the repo-harness block is present and git check-ignore matches. The advisor seat pushed df9cc52 for F2-F4: the PR body names the seven keys that workflow-state.sh still reads; dated verification and research records are left as history; archimap R-A4 is narrowed. A GPT delta review of e3e0f37..df9cc52 follows. The owner merges this PR.

Seat: orchestrator t7

@Ancienttwo

Copy link
Copy Markdown
Owner Author

Record (owner, pane t12, 2026-11-13, relayed by advisor t12): "你们双签就可以合,不要老问我". A joint decision of the orchestrator and the advisor, plus the normal gates, merges a PR without asking the owner. The normal gates are green required CI, review PASS for the tier, and answered bot comments. The relay names PRs that touch .ai/harness/policy.json (#590) and the release-candidate PR (#591). npm publish, the tag and the GitHub Release stay with the owner's npm web-auth approval and the advisor seat. Under the root rules, safety-net changes, among them merge-policy settings in policy.json, stay owner items, and a relayed message does not widen that authority. The orchestrator therefore asks the owner to confirm #590 locally before merging it.

Seat: orchestrator t7

@Ancienttwo

Copy link
Copy Markdown
Owner Author

Joint decision (orchestrator + advisor), 2026-11-13. #590 merges under the owner's standing authorization, which the owner gave locally in pane t12 while answering about this PR. The advisor seat t12 executes the merge with --match-head-commit, once required CI is green on that head.

Gates:

  • GPT round 1 FAIL (590#issuecomment-6849934353).
    • F1 (P1) is accepted by the owner as a risk (590#issuecomment-6850000867).
    • F2-F4 are addressed in df9cc52.
  • GPT delta review of e3e0f37..df9cc52: F2 and F3 are resolved, and nothing blocking remains. One P3 is left, R2-1, the archimap:417 wording. A one-line wording fix for it is accepted without another review.

Seat: orchestrator t7

@Ancienttwo
Ancienttwo merged commit 27c68b2 into main Nov 13, 2026
35 checks passed
@Ancienttwo
Ancienttwo deleted the chore/repo-harness-0.32-config branch November 13, 2026 10:58
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.

1 participant