Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 28 additions & 0 deletions docs/plans/2026-10-29-green-harness-bundle.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
---
canonical: github:fredericrous/decisions/docs/plans/2026-10-29-green-harness-bundle.md
phases: [7]
status: done
branch: feat/grh-skills-from-repo
---
# Green Harness bundle

Pointer. The plan and its review panel live in decisions, with ADR-0050
(harness.skills-source). This repository carries phase 7: the published
skills and agents leave `dot_claude/` (grh places them from
greenharness/skills), and the private overlay keeps the Forgejo recipes.

## Verification

| Check | Input | Expected | Actual |
|---|---|---|---|
| backup first | tar of `~/.claude/{skills,agents}` | tarball before any change | `~/.local/state/grh-migration-backup-20261030-004629.tgz`, 463 entries |
| source check | `make check` (render, lint, private-refs) | rc 0 | rc 0 |
| laptop install | `grh install full workflow --replace --yes` (grh 0.1.0 from the tap) | exit 0; existing binaries adopted; published skills replace the old copies | exit 0; aval, amont, amont-agent, relais, adr-lint adopted; decisions and guardrails machine integrations already present, recorded adopted; router a reported gap (relais doctor has no integration state yet); 13 skills and 11 agents placed, each overwritten file backed up under `~/.local/state/grh/backup/`; relais-architecture-conflict and relais-verified-push left to relais (its marker) |
| status | `grh status` | parts installed | decisions, gate, guardrails, router, conform, workflow installed; router machine unknown |
| integrations | `amont-agent doctor`, `claude mcp get aval` | green, registered | green, registered |
| session | a fresh Claude Code session | workflow skills and the overlay listed | listed (forgejo-recipes, merge-when-green, plan-review, tag-release…) |
| no drift | `chezmoi diff --source <this branch> ~/.zshenv ~/.claude/skills/forgejo-recipes ~/.claude/CLAUDE.md` | empty | 0 lines (the laptop's newer cd-rule wording folded into CLAUDE.md.tmpl) |

## Implementation review

approve-with-changes after two deltas (44k + 47k + 49k). Fixed: the overlay names merge-when-green instead of "above", says which sweep.py reads the variables, and the zshenv guard needs both host and slug.
6 changes: 3 additions & 3 deletions dot_claude/CLAUDE.md.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ the named skill, rule (`aval rule <id>`), amont-agent rule, or memory file.

## Git hygiene

- Name the repository in each mutating git command (`git -C <repo>` or `cd <repo> &&`
in the same call): parallel calls share one cwd.
- Name the repository in each mutating git command (`git -C <repo>` or a subshell
`(cd <repo> && …)`): a bare `cd` carries into the next call, batched siblings included.
- Run `git status` before every commit and stage explicit paths (amont-agent `git-add-broad`).
- Release: commit, check HEAD advanced, then tag, then push the tag — the
`tag-release` skill. A tag is never force-pushed; a fix gets the next version.
Expand Down Expand Up @@ -56,7 +56,7 @@ the named skill, rule (`aval rule <id>`), amont-agent rule, or memory file.
## Forgejo API ({{ .forge.host }})

- Both credentials are required: the mTLS cert `~/.config/mtls/{{ .forge.slug }}-client.{crt,key}`
(missing → curl HTTP 000) and the token in `~/.netrc` — the curl recipe is in `merge-when-green`.
(missing → curl HTTP 000) and the token in `~/.netrc` — the curl recipe is in `forgejo-recipes`.
- Probe `/repos/{owner}/{repo}` to test access; `/api/v1/user` 403s on this token's
scope. Never echo the token into output, a commit or a PR body; pass it in `$TOK`.
More: the `reference_forgejo_*` memories.
Expand Down
103 changes: 0 additions & 103 deletions dot_claude/agents/implementation-review.md

This file was deleted.

35 changes: 0 additions & 35 deletions dot_claude/agents/interface-review.md

This file was deleted.

63 changes: 0 additions & 63 deletions dot_claude/agents/plan-review-architect.md

This file was deleted.

72 changes: 0 additions & 72 deletions dot_claude/agents/plan-review-backend.md

This file was deleted.

Loading
Loading