rill-close-handbook-html: aggregated workspace handbook at /close - #87
Merged
Merged
Conversation
Wire the "workspace completion" read moment from rill-html-output.md
into the close skill, and absorb the two HTML-canonical handoffs from
the doctrine task's review.
- .claude/commands/_close/handbook-agent.md (new): third /close
sub-agent template (ADR-073 structure). Fresh context reads
_summary.md + all deliverables (MD + HTML-canonical) and authors an
aggregated handbook into workspace/{id}/.view/ following the shared
design language: start page re-expresses _summary.md with navigation
to every artifact; single page (handbook.html) vs multi-page
(handbook/ hub + detail pages) is a qualitative size judgment;
layer-not-delete; self-contained; provenance comment; stable name
(close-time content is final, so the dated-snapshot rule for
volatile-phase digests does not apply); output_language injection.
- skills/close/SKILL.md: spawn the handbook agent in the Phase 4
background queue alongside distillation; result excluded from the
Phase 5 coverage equation; generation failure non-fatal; user skip
line; rill book build role distinction; completion-report line.
- .claude/commands/_close/distillation-agent.md: cross-deliverable
verification now covers HTML-canonical artifacts (glob *.{md,html},
derived .view/ and same-basename twins excluded) instead of *.md
only (doctrine review round 13 handoff).
- skills/promote/SKILL.md: Phase 2 artifact-reference extraction
handles HTML-canonical artifacts (<title> / first <h1> with filename
fallback; HTML class label in place of frontmatter type) (doctrine
review round 13 handoff).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
tarr1124
added a commit
that referenced
this pull request
Aug 13, 2026
…oments (#88) Wire the "project review" read moment of rill-html-output.md into skills/project/SKILL.md, completing the three-layer read-moment model (ADR-082 Phase 2 / G5) after /focus (#86) and /close (#87): - New section "Project Digest (read-moment HTML)": one generator, two section weightings (run-end leads with stop reason + pending decisions + next actions; status leads with current position + next actions) - Ordering: refresh-project (already run) -> refresh-decisions (synchronous) -> read refreshed sources -> generate - Pending decisions re-express the contract-v1.1 fields (ADR-084); consequence-framed, zero-context readable (D82-8 / D84-7) - Dated snapshots at projects/{slug}/.view/digest-{YYYY-MM-DD}.html, never overwritten; self-contained; provenance comment; non-fatal with partial-file cleanup; skip line - Run mode Step 3 persists the run summary as a project artifact (NNN-run-summary-{date}.md) so the run-end digest stays a pure derived view (principle 4), and the final rill push covers it Co-authored-by: tarr1124 <tarr1124@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Wires the "workspace completion" read moment from
rill-html-output.mdinto the close skill, and absorbs the two HTML-canonical handoffs from the doctrine task's review (#85 round 13)..claude/commands/_close/handbook-agent.md(new) — third /close sub-agent template (ADR-073 structure). Fresh context reads_summary.md+ all deliverables (MD + HTML-canonical) and authors an aggregated handbook intoworkspace/{id}/.view/following the shared design language (_view/design-language.md): start page re-expresses_summary.mdwith navigation to every artifact; single page (handbook.html) vs multi-page (handbook/hub + detail pages) is a qualitative size judgment; layer-not-delete; self-contained; provenance comment;output_languageinjection.skills/close/SKILL.md— spawns the handbook agent in the Phase 4 background queue alongside distillation. Parent-owned lifecycle: prior outputs deleted at step start (before the skip decision), retraction on any interruption beforestatus: completedlands, partial output deleted on agent error. Result excluded from the Phase 5 coverage equation; generation failure non-fatal; user skip line;rill book buildrole distinction; completion report shows the returned path. Stable name rationale documented (close-time content is final — the dated-snapshot rule for volatile-phase digests does not apply)..claude/commands/_close/distillation-agent.md— cross-deliverable verification now covers HTML-canonical artifacts (glob="*.{md,html}", derived.view/and same-basename twins excluded) instead of*.mdonly.skills/promote/SKILL.md— Phase 2 artifact-reference extraction handles HTML-canonical artifacts (<title>/ first<h1>with filename fallback; HTML class label in place of frontmattertype).Codex review
codex review --base main, 5 rounds: r1 2 [P2] (report path / stale outputs) → r2 1 [P2] (retract on Phase 5 STOP) → r3 2 [P2] (skip-path staleness / Phase 6-7 interruptions) → restructured to a parent-owned lifecycle → r4 1 [P2] (error-branch partial output) → r5 0 P-findings. No [P1] at any round.Guards
cjk-guard exit 0, pii-regex-guard exit 0; diff scans: non-ASCII = em dash only (main convention), PII mapping 0, email/phone 0.
🤖 Generated with Claude Code