Skip to content

docs(process): slices, project gate rules, acceptance tests and the declared integration path - #40

Open
pasilastbot wants to merge 11 commits into
mainfrom
feat/slices-gates-uat
Open

pasilastbot wants to merge 11 commits into
mainfrom
feat/slices-gates-uat

Conversation

@pasilastbot

Copy link
Copy Markdown
Contributor

Canonical process text for slice SLICE-RDD-FLOW. Entry approved: USER:2026-10-14:slice-entry-approve. Design and decisions: design/2026-10-13-slices-gates-uat.md, which stays out of the packaged tree.

What changes

  • Slices (REQ-CROSS-562):
    • Work is planned and delivered in slices. "Sprint" is gone, and the code a batch lands is an increment (D4).
    • The new "### Slices" section and file-state/SLICE.md.
    • The build is red-first per connected batch (D1), with one full verification, one code review and one completion per slice.
    • The WORK-SELECTION.md shape is unchanged.
  • Stop only at human decisions (REQ-CROSS-586, process side):
    • Agents stop at an open human gate, a question only a human can answer, or a requested stop. An applied entry approval lets the build start.
    • Merging, deploying and releasing wait for a human.
    • Briefs use plain language, with jargon at most a trailing breadcrumb.
  • Project gate rules (REQ-CROSS-573):
    • The project's process/rdd-human-gates.md, with default and autopilot presets in presets/ and the shape in file-state/HUMAN-GATES.md.
    • Sign-in, payments, existing data and external contracts stay human in both presets (D2).
    • Light paths are off in the kit default and on in autopilot. An incident counts only when a person declares it.
    • The ten remaining risks are listed.
  • Acceptance test step (REQ-CROSS-581, process side):
    • A new step between the delivered-revision check and the completion trace.
    • The new skill skills/rdd-acceptance and the shape file-state/ACCEPTANCE-GUIDE.md.
    • The guide is required per slice. Sign-off follows the project's rules (D3).
  • Declared integration path (EPIC-CLI-DELIVERY-PATH):
    • Completion is checked on the project's declared path, from process/integration-path.md with the shape in file-state/INTEGRATION-PATH.md.
    • On merge, the remote default branch must contain each tested commit. On deploy, the deployed commit is named and a deployment run is recorded.
    • The as-built (reverse-engineer) exact-tip check is unchanged, because the CLI still enforces it.

How it ships

The modernpath CLI embeds this text at a pinned commit (rdd-source.txt). The CLI change in ModernPath/modernpath-v1 (branch feat/delivery-path-completion) pins this branch's head for its tests. After this PR merges, it re-pins to the merge commit before it merges itself.

🤖 Generated with Claude Code

@mattias-modernpath

Copy link
Copy Markdown
Contributor

Reviewed with Astra, xhigh effort, base 25d85d660fe8e8d500c220e36bfc721502d608ad → head efe87476c4342623d3eff96fc28774903c023da4.

Two process-contract findings:

  1. [P2] Define entry and RED exceptions for fix-first changes — PROCESS.md:471–476. The incident path permits implementing the fix before its reproduction test and entry records; the defect path also records entry afterwards. But PROCESS.md:985–1000 still prohibits implementation before applied entry except diagnostic tests or uncommitted reversible build-ahead, PROCESS.md:822–825 still requires RED before implementing every clause, and completion review rejects missing required RED. An agent following the enabled path cannot consistently satisfy build and completion. The acknowledged uncertainty about server ordering does not resolve this canonical-process conflict. Add explicit fix-first exceptions to the entry/evidence rules and affected skills, including the required retrospective proof. Grounding: REQ-CROSS-573-AC2 and canonical entry/evidence consistency.

  2. [P2] Align autopilot closeout with one full-verification point — PROCESS.md:623. This requires full verification once at the delivered revision. However, skills/rdd-autopilot/SKILL.md:136–145 requires it before independent review of the branch head; that review precedes delivery, after which skills/rdd-completion-review/SKILL.md:39–41 requires full verification again. A normal merge with a new merge commit therefore requires two full runs, or violates one instruction. Identify one full-run point and explicitly state when equivalent evidence may carry across delivery. Grounding: REQ-CROSS-562-AC2.

The PR groups epics/requirements into slices, adds gate presets and acceptance guides, and documents merge/deploy completion. Updated tracked files and skills coordinate the work while retaining each member's records and gates. The intended outcome is supported, but the execution instructions need the corrections above.

Coverage: all 23 changed files plus relevant surrounding rules; REQ-CROSS-562/573/581/586, the delivery-path epic and checked-in design. Citation-auditor tests: 15 passed. Checked 12 relative Markdown paths: none missing. These checks do not establish execution of the companion CLI workflows. REQ-ENG-001–008 are PROPOSED candidates, not governed constraints; their frontend/backend/runtime concerns do not apply to this documentation-only change.

pasilastbot added a commit that referenced this pull request Oct 15, 2026
…istent

Review on #40:
- The defect and incident light paths let the fix precede entry and RED,
  but the entry rule, the autopilot RED rule, rdd-build and the
  completion-review stop list had no matching exception. They now name
  fix-first as an exception, with the reproduction proof standing as
  its RED. For an incident the proof is the test failing at the pre-fix
  revision and passing at the fix, cited in the gap record.
- Full verification was required both at closeout (before review) and
  again after delivery. The one run is now the closeout run, carried to
  the delivered revision by the existing containment rule; it is re-run
  there only when containment cannot be shown (squash or rebase).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pasilastbot

Copy link
Copy Markdown
Contributor Author

Thanks. Both findings are fixed in ecb97d6:

  1. Fix-first paths contradicted the entry and RED rules.
    • The entry rule now lists fix-first on a light path (defect_fix, incident_fix) as its third exception. The autopilot RED rule, rdd-build step 1 and the rdd-completion-review stop list each point to the reproduction proof.
    • New wording for a person to confirm: for an incident, the proof is now the reproduction test failing at the pre-fix revision and passing at the fix, with the gap record citing both runs. The old text only said the test "follows as verification".
  2. Full verification was required twice.
    • The one full run is now the slice's closeout run. It carries to the delivered revision through the existing containment rule (§Evidence and completion), and is re-run there only when containment can't be shown (squash or rebase).
    • The same wording is applied in PROCESS.md (both tables, the development loop, the slice completion paragraph and the acceptance step), rdd-autopilot step 2 and rdd-completion-review step 4.
    • As you noted, containment doesn't prove the merged tree passes if the default branch moved. That is the existing rule's choice.

The auditor tests pass (15/15). #890 embeds this revision (a6cd5385e).

pasilastbot and others added 11 commits October 18, 2026 11:54
…raft)

Draft for the process owner's decisions (USER:2026-10-13:slices-gates-uat).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nd acceptance tests

USER:2026-10-13:slices-gates-uat-as-recommended.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
USER:2026-10-13:one-slice-dogfood.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A slice is a named set of Epics and requirements a developer delivers as
one package, recorded in a tracked process/slices/<SLICE-id>.md file with
no dates. Plan review, entry brief, full verification, independent code
review and completion brief run once per slice; red-first runs per
connected batch with focused tests per batch. The entry packet's RED
strategy names the stages. "Sprint" becomes "slice" and today's "slice"
becomes "increment". Adds file-state/SLICE.md; WORK-SELECTION.md is
unchanged.

REQ-CROSS-562

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A human's answer is applied and the loop continues: the agent stops only
at a human gate, a question only a human can answer, or where the user
asked it to stop. An applied entry approval lets the build start.
Merging, deploying and releasing still need the human's explicit
instruction. The autopilot sentence for gate answers stays word for word.
Everything shown to the human is plain product language with store
jargon at most a trailing breadcrumb, and review findings are walked
through one at a time.

REQ-CROSS-586 (process-text side)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
PROCESS.md names the decision points and what each carries; the
project's process/rdd-human-gates.md says who decides. The kit ships a
default preset (today's rules, light paths off, marked "kit default, not
decided by this project") and an autopilot preset (light paths on, Scope
and Expires required); the autopilot grant becomes that preset. Sign-in,
payments, existing data and external contracts stay with a person in
both, and the incident path keeps sign-in, payments and existing data
with a person; an incident counts only when a person's words declaring
it are quoted as a USER: source. An agent answer under a rules row
carries a disclosure, and the ten residual risks are stated. Adds
file-state/HUMAN-GATES.md and presets/.

REQ-CROSS-573

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Completion gains an acceptance test step between the delivered-revision
check and the completion trace: the agent drafts and completes an
acceptance test guide (a tracked file beside the slice file) for a human
tester, after the last record edit of the covered scopes. Tester results
are acceptance test results naming the tester, environment and tested
revision. The project's gate rules decide whether the guide and the
tester's sign-off are required; with sign-off required, every criterion
not proven by automated tests needs a current tester pass. Adds the
rdd-acceptance skill and file-state/ACCEPTANCE-GUIDE.md;
rdd-completion-review runs the step.

REQ-CROSS-581 (process side)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
EPIC-CLI-DELIVERY-PATH replaced the exact default-branch-tip rule with
completion on the path a project declares in process/integration-path.md:
merge (the default) or deploy. On merge the fetched default branch must
contain every requirement's tested commit wherever the checkout sits; on
deploy the deployed commit is named and a deployment run records the
environment and smoke checks. Failing, stale, missing or uncontained
evidence is refused by name before any write; squash or rebase merges
re-run the tests at the merged commit.

PROCESS.md §Evidence and completion and rdd-completion-review describe the
paths; file-state/INTEGRATION-PATH.md gives the file shape the CLI parser
reads. As-built acceptance keeps its exact-tip integration observation,
and PROCESS.md and rdd-reverse-engineer now say the declared path does not
change it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
file-state/HUMAN-GATES.md used «…» placeholders while presets/default.md
and presets/autopilot.md use <…>. The shape now uses <…> like the presets;
the CLI parser refuses both as placeholders.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The slice design (REQ-CROSS-585) has later rounds read the change list the
review snapshot carries; the skill now says so, with the closures of earlier
findings re-verified and findings and dispositions each recorded in one batch.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…istent

Review on #40:
- The defect and incident light paths let the fix precede entry and RED,
  but the entry rule, the autopilot RED rule, rdd-build and the
  completion-review stop list had no matching exception. They now name
  fix-first as an exception, with the reproduction proof standing as
  its RED. For an incident the proof is the test failing at the pre-fix
  revision and passing at the fix, cited in the gap record.
- Full verification was required both at closeout (before review) and
  again after delivery. The one run is now the closeout run, carried to
  the delivered revision by the existing containment rule; it is re-run
  there only when containment cannot be shown (squash or rebase).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@pasilastbot
pasilastbot force-pushed the feat/slices-gates-uat branch from ecb97d6 to bf18e1e Compare October 18, 2026 08:54
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.

2 participants