Skip to content

orchestrate/teamwork: conditional decomposition guidance (tracer bullets, expand-contract) v4.9.0 - #85

Merged
sumitake merged 1 commit into
mainfrom
dev/claude/decomposition-guidance
Aug 3, 2026
Merged

orchestrate/teamwork: conditional decomposition guidance (tracer bullets, expand-contract) v4.9.0#85
sumitake merged 1 commit into
mainfrom
dev/claude/decomposition-guidance

Conversation

@sumitake

@sumitake sumitake commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

Deferred follow-up B from #83 (companion to merged #84): orchestrate and teamwork gain conditional decomposition guidance for implementation work:

  • Tracer-bullet vertical slices preferred when (and only when) a graph/milestone plan decomposes product-feature implementation: complete narrow path through every affected layer, demoable or independently verifiable, sized to one bounded worker invocation/session with explicit acceptance criteria (durable phrasing, not context-window-relative). Prefactoring first.
  • Wide refactors are the exception, narrowly triggered: atomically codemoddable mechanical changes run as checkpointed, fully verified batches; expand–contract is reserved for migrations where old and new forms must genuinely coexist to keep callers and CI working — never merely because blast radius is large.
  • Named inapplicability: research, operations, configuration, and decision-only graphs/milestones are explicitly out of scope for the guidance.
  • Licensing: adapted portions attributed to mattpocock/skills to-tickets @2ab95809 (blob 96deac51, verified locally); full MIT notice in both specs; both members join MIXED_LICENSE_SKILL_MEMBERS (LicenseRef-PolyForm-Strict-1.0.0 AND MIT); provenance rows added. NOTICE unchanged (generalized in code-review: spec-fidelity + smell-baseline lenses (backward-compatible JSONL extension) v4.8.0 #84 by design).
  • Version 4.8.0 → 4.9.0 across all seven surfaces; changelog fragment; marketplace regenerated.

Validation

  • build_skills.py --check clean; check_release_consistency.py RESULT: OK; build-changelog.py --dry-run OK
  • unittest discover -s scripts 319/319 OK; unittest discover -s tests 658/658 OK

Cross-check (verbatim)

Same consultation as #84 — Codex (gpt-5.6-sol, effort medium, direct codex exec under operator broker-bypass authorization) reviewed both follow-up plans in one round:

VERDICT: PROCEED-WITH-MODIFICATIONS

PR-B-relevant concerns integrated here: (6) expand–contract trigger narrowed to coexistence-required migrations, with checkpointed-batch alternative for codemoddable refactors; (7) "one context window" replaced with a durable bounded-invocation + acceptance-criteria bound; (5) full MIT notice carried in both touched specs with SPDX mixed-member classification.

author: claude
author_model: claude-fable-5 (effort: default)
standing_directives: skill-change workflow; #1 trigger phrases (descriptions unchanged — guidance is body-level); #3 release withheld per operator instruction 2026-08-03; #4 N/A (plugin change); #5 plugin README updated in this PR; #7 model+effort stated; #8 no fan-out
tier: 2
cross_check: PROCEED-WITH-MODIFICATIONS (codex gpt-5.6-sol, effort medium, direct codex exec under operator broker-bypass authorization 2026-08-03; raw verdict quoted in PR body; concerns 6+7 integrated here, 1-5 in #84)
post_condition: 319/319 + 658/658 tests OK; build --check clean; release-consistency OK; changelog dry-run OK; no coordinator/route change
mcp_coverage_gap: none — prose guidance only; gemini_mcp_path: retired / not applicable
contributor_rights: OPERATOR-CONFIRMED
operator_reserved: no
plugin_affected: this PR is the plugin change
readme_refresh_status: README updated (plugin repo); workspace companion at release (release currently withheld by operator)

🤖 Generated with Claude Code

…omposition guidance v4.9.0

Second deferred follow-up from #83. Feature-implementation-only scope
with named inapplicability; expand-contract reserved for genuine
coexistence-required migrations (codemoddable changes run as
checkpointed verified batches); bounded-worker-invocation sizing.
Adapted portions attributed (to-tickets @2ab95809); both members mixed
PolyForm AND MIT in SPDX.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@sumitake
sumitake merged commit 3523412 into main Aug 3, 2026
16 checks passed
@sumitake
sumitake deleted the dev/claude/decomposition-guidance branch August 3, 2026 20:50

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cd8fa13893

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +52 to +55
- **Wide refactors are the exception.** When one mechanical change fans across
the codebase so no vertical slice can land green: if the change is
atomically codemoddable, run it as checkpointed batches with full
verification after each batch. Reserve **expand–contract** sequencing —

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep atomically codemoddable changes atomic

When a mechanical interface or schema rename cannot leave any partial subset green—the exact case described here—splitting it into batches makes the first batch fail full verification, so the workflow cannot satisfy its acceptance check before proceeding. Apply an atomically codemoddable change as one atomic node and verify afterward; reserve independently verified batches for changes whose intermediate states remain valid. The same contradictory instruction should be corrected in teamwork and the generated skills regenerated.

AGENTS.md reference: AGENTS.md:L9-L10

Useful? React with 👍 / 👎.

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