Skip to content

Add parallel-team preset and cross-session delivery contract - #69

Merged
olehsvyrydov merged 2 commits into
mainfrom
feat/parallel-team-preset
Aug 9, 2026
Merged

Add parallel-team preset and cross-session delivery contract#69
olehsvyrydov merged 2 commits into
mainfrom
feat/parallel-team-preset

Conversation

@olehsvyrydov

Copy link
Copy Markdown
Owner

Summary

  • Adds a parallel-delivery skill covering the claim protocol (assignee is the lock), workspace isolation (one clone/worktree per ticket), the ticket-is-sole-context rule, and the report-back contract (a PR + ticket comment + tracker transition — no side-channel status docs).
  • Adds the parallel-team workflow preset (always_required: [CODE_REVIEWED], reviewer must be a different session than the implementer) to workflow.yaml, and its enum entry in workflow.schema.json; wires --preset=parallel-team into install.sh and its --help text.
  • Extends both ticket templates (templates/user-story-template.md and the Scrum Master's references/templates.md) with Branch, Claimed by, Negative acceptance criteria, and Verification steps fields.
  • Replaces the "conductor of the orchestra — every agent reports to you" language in commands/sm.md and commands/luda.md — walked back as unfalsifiable in the 5.1.0 changelog but never fixed at the command layer — with the board-integrity framing the Scrum Master skill itself already uses.
  • Updates the skill/doc counts (42 → 43) and the README skill list to keep the framework's own validator green.

Dry run — one sample ticket filled from the new template fields alone

**Branch**: feat/login-attempt-throttle
**Claimed by**: worker-3 (tracker assignee field; this is the claim/lock)

## Negative Acceptance Criteria
- [ ] A 6th failed login within 60s for the same account is rejected, not just logged — enforced by `LoginThrottle.checkAttempt()`
- [ ] The throttle does not reset on a successful login from a different account sharing the same IP — enforced by `LoginThrottle` keying on `(account_id)`, not `(ip)`

## Verification Steps
1. `npm test -- login-throttle.spec.ts` → expected: `7 passed, 0 failed`
2. `curl -s -o /dev/null -w '%{http_code}' -X POST localhost:3000/login -d 'user=x&pass=wrong' (x6)` → expected: 6th response is `429`

Every field above was fillable from the template + the parallel-delivery skill's instructions alone, with no external report to read.

Test plan

  • python3 scripts/validate-framework.py — all 7 gates pass (skills=43 commands=50 templates=6)
  • claude plugin validate ./claude — passes (pre-existing warnings only: no version key by design, plugin-root CLAUDE.md not auto-loaded by design)
  • workflow.yaml with preset: parallel-team validated against workflow.schema.json via jsonschema.validate — passes
  • ./install.sh --dry-run --editors=claude --scope=project --preset=parallel-team --yes — installs cleanly, writes preset 'parallel-team' to the override file
  • ./install.sh --dry-run --editors=all --scope=project --yes (default solo preset) — unchanged, still installs cleanly
  • Diff of workflow.yaml/workflow.schema.json is additions-only: solo, small-team, regulated blocks are byte-for-byte unchanged
  • No new persona/command added; no references to private projects, internal plans, or model names in the diff

New skill `parallel-delivery` covers the claim protocol (assignee is
the lock), workspace isolation (one clone/worktree per ticket), the
ticket-is-sole-context rule for a worker with no one to ask, and the
report-back contract (a PR + ticket comment + tracker transition, no
side-channel status docs).

Adds the `parallel-team` workflow preset (peer review always required,
by a different session than the implementer) and its schema enum
entry, wires up `--preset=parallel-team` in install.sh, and extends
both ticket templates with Branch / Claimed by / Negative acceptance
criteria / Verification steps fields.

Replaces the stale "conductor of the orchestra — every agent reports
to you" language in the Scrum Master commands (walked back as
unfalsifiable in 5.1.0, but never fixed at the command layer) with the
same board-integrity framing the skill itself already uses.
Copilot AI lite review requested due to automatic review settings August 9, 2026 21:14

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.

Pull request overview

Adds a new workflow preset (parallel-team) and a new cross-session process skill (parallel-delivery) to support parallel ticket execution with explicit claim/lock semantics, workspace isolation, and a standardized report-back contract, plus template and documentation updates to keep the framework consistent.

Changes:

  • Added parallel-delivery skill documenting the cross-session claim/isolation/report-back protocol.
  • Introduced parallel-team workflow preset (and schema/docs) and wired it into install.sh/README preset lists.
  • Extended ticket templates and updated related command/docs copy and skill-count references (42 → 43).

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
README.md Updates skill count and documents the new parallel-delivery process skill and preset option.
install.sh Accepts --preset=parallel-team and validates it in argument parsing.
claude/workflow/workflow.yaml Adds the parallel-team preset with CODE_REVIEWED always required.
claude/workflow/workflow.schema.json Extends preset enum to include parallel-team.
claude/workflow/README.md Documents parallel-team in the presets table.
claude/templates/user-story-template.md Adds Branch, Claimed by, Negative AC, and Verification Steps fields to the user story template.
claude/skills/parallel-delivery/SKILL.md New skill defining the parallel delivery contract and expectations.
claude/skills/management/scrum-master/references/templates.md Extends the Scrum Master ticket template with Branch/Claimed by, Negative AC, and Verification Steps.
claude/commands/sm.md Reframes Scrum Master “core principle” away from “conductor” language toward board-integrity framing.
claude/commands/luda.md Same core-principle reframing as sm.md.
claude/CLAUDE.md Updates skill count reference (42 → 43).
claude/AGENTS.md Updates skill count reference (42 → 43).
CLAUDE.md Updates skill count reference (42 → 43).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread claude/templates/user-story-template.md Outdated
Comment thread claude/skills/management/scrum-master/references/templates.md Outdated
Comment thread claude/workflow/README.md Outdated
Comment thread claude/workflow/workflow.yaml Outdated
@olehsvyrydov
olehsvyrydov merged commit 19d1164 into main Aug 9, 2026
3 checks passed
@olehsvyrydov
olehsvyrydov deleted the feat/parallel-team-preset branch August 9, 2026 21: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.

2 participants