Files
AILang/skills/brainstorm/SKILL.md
T
Brummel 51da9fab53 iter disc.1: boss-only commits + main-as-quarantine (no branches in implement)
Two project-wide rules are now explicit across every skill:

1. Only the Boss commits. No skill agent (implementer,
   brainstormer, planner, debugger, fieldtester, docwriter,
   architect, bencher) runs `git commit`. Agents write their
   artefacts to the working tree as unstaged changes; the Boss
   inspects, decides commit shape, and commits.
2. main HEAD is sacrosanct. No actor runs `git reset` or
   `git revert` on main. Bad work stays in the working tree
   where it is still discardable via `git checkout -- <paths>`.

Implement loses the `iter/<iter_id>` branch mechanic entirely;
Phase 0 of the orchestrator-agent now does a clean-tree check
and refuses to start on a dirty tree. Per-task agent commits
are removed everywhere; reviewers operate against
`git diff HEAD` instead of `pre_task_sha..head_sha`.

Motivation: 2026-05-11 iter 23.4 stranded prep2/prep3 commits on
an iter-branch that never integrated to main, then a corrected
spec falsely claimed those commits had shipped. Branch-per-iter
+ manual-Boss-merge + iter-stacking made the strand structurally
possible. See docs/journals/2026-05-11-iter-disc.1.md for the
full per-task notes and motivation.
2026-05-11 22:44:43 +02:00

11 KiB

name, description
name description
brainstorm Use when a new milestone is starting — gathers requirements, explores 2-3 approaches with trade-offs, presents a sectioned design with user approval, writes the spec to docs/specs/. Hard-gate before any plan or code work; applies even when the user says they have a clear vision.

brainstorm — milestone spec generator

Violating the letter of these rules is violating the spirit.

Overview

The spec is where the orchestrator does the design work. It is the artefact that future iterations, the architect agent, and any future LLM read to understand WHAT the milestone delivers and WHY. Skipping the spec means shipping the user's intuition unfiltered, which is exactly the reactive-deference failure mode AILang's orchestrator discipline exists to prevent.

It produces specs at docs/specs/<milestone>.md.

When to Use / Skipping

Triggers:

  • A new milestone is starting (e.g. user proposes a new feature area, a new subsystem, or a major refactor of an existing one).
  • A milestone-scope spec is missing for an existing line of work and one is needed retroactively.

Skipping is permitted only for:

  • A bug-fix iteration (use debug directly).
  • A tidy iteration (use audit directly).
  • A trivial mechanical edit (≤1 file, ≤30 LOC, no design judgement).

Skipping is not permitted for:

  • A user request framed as "I have a clear vision, just plan it".
  • A milestone described as "small, only two iterations".
  • A milestone where you are confident you know what to build.

The first two cases above are the most common pressure to skip; both are explicitly resisted by this skill (see Common Rationalisations).

The Hard-Gate

NO IMPLEMENTATION, SCAFFOLDING, OR DOWNSTREAM SKILL INVOCATION
UNTIL A SPEC HAS BEEN PRESENTED AND THE USER HAS APPROVED IT.

The hard-gate applies regardless of perceived simplicity. The spec can be short for a small milestone (a few sections), but it MUST exist and be approved before any plan or code work begins.

The Process

Step 1 — Explore project context

Before asking any clarifying questions:

  • Read the latest entries linked from docs/journals/INDEX.md (most recent milestones, current state).
  • Skim docs/DESIGN.md for the invariants the new milestone might touch.
  • git log --oneline -20 to see what just shipped.
  • Identify scope: is this one milestone, or does it need to be decomposed into sub-milestones first? If multi-subsystem, decompose.

Step 2 — Ask clarifying questions, one at a time

Multiple-choice when possible (easier to answer). One question per message — no batching. Focus on:

  • Purpose: what problem does this solve?
  • Constraints: what semantic / performance / compatibility invariants must hold?
  • Success criteria: how do we know it shipped?
  • Scope: what is OUT of scope?

Step 3 — Propose 2-3 approaches with trade-offs

Lead with your recommended option and explain why. Each approach gets:

  • core idea (1-2 sentences)
  • substantive trade-offs (semantic fit, schema impact, performance, composability)
  • effort observation (NOT effort as rationale — see CLAUDE.md "Design rationale ≠ implementation effort")

Wait for user choice before proceeding.

Step 4 — Apply the feature-acceptance criterion

A feature ships only if:

  • An LLM author (= future me) naturally produces code that uses it, AND
  • The feature measurably improves correctness or removes redundancy.

Aesthetic appeal does not count. Human ergonomics do not count. If the answer to either criterion is "I'm not sure", the feature is probably not ready. The full criterion lives in docs/DESIGN.md ("Feature-acceptance criterion"); this skill is the gate that applies it during spec writing.

Step 5 — Present design in sections

Scale each section to its complexity (a few sentences if straightforward, up to 200-300 words if nuanced). Cover:

  • Architecture
  • Components
  • Data flow
  • Error handling
  • Testing strategy

Ask after each section whether it looks right. Be ready to revisit earlier sections if a later one surfaces a contradiction.

Step 6 — Write the spec

Path: docs/specs/YYYY-MM-DD-<milestone>.md.

Structure:

# <Title> — Design Spec

**Date:** YYYY-MM-DD
**Status:** Draft — awaiting user spec review
**Authors:** Brummel (orchestrator) + Claude

## Goal
## Architecture
## Components
## Data flow
## Error handling
## Testing strategy
## Acceptance criteria

Step 7 — Self-review

Inline checklist (not a subagent dispatch):

  1. Placeholder scan: any "TBD", "TODO", incomplete sections, vague requirements? Fix.
  2. Internal consistency: sections must not contradict. Architecture must match component descriptions.
  3. Scope check: focused enough for one plan, or does it need sub-milestone decomposition?
  4. Ambiguity check: could any requirement be interpreted two ways? If so, pick one and make it explicit.

Fix issues inline. No need to re-review — fix and commit.

Step 7.5 — Grounding-Check Agent (hard-gate)

Dispatch the read-only ailang-grounding-check agent. The agent reads the just-written spec with fresh context, extracts its load-bearing assumptions about current compiler / checker / codegen / schema behaviour, and for each one searches the workspace for a currently-green test that ratifies it.

Dispatch carrier:

Field Content
spec_path path to the spec just written in Step 6
iteration_scope which sections cover the imminent first iteration (verbatim from the brainstorm session)

The agent returns a PASS / BLOCK / INFRA_ERROR report. Boss interpretation:

Agent status Boss action
PASS Proceed to Step 8. The report goes into the brainstorm chat for the record.
BLOCK Present the report to the user. User either: (a) accepts discard → execute the failure-mode procedure below; or (b) overrides in free chat ("Test <path>::<name> ratifies that") → log the override note in the brainstorm chat and proceed to Step 8.
INFRA_ERROR Abort the brainstorm. The spec stays as-is on disk; debug the workspace out-of-band and re-dispatch the agent.

Failure-mode procedure (no-override BLOCK):

  1. Delete the just-written spec file from the working tree (rm docs/specs/<spec_filename>.md — a Bash-level delete on the working tree, NOT git rm; the file was never committed).

  2. Append a roadmap entry to docs/roadmap.md pointing forward to a future re-brainstorm, under the appropriate priority bucket:

    - [ ] **[<kind>]** <idea title> — <one-line description>
      - context: <why this is deferred>
      - depends on: <name of the unratified mechanism>
    
  3. Tell the user the spec was retired and which roadmap entry was added. The user / Boss commits the roadmap edit (suggested commit subject: spec: retire <spec topic> — depends on <unratified mechanism>).

After the failure-mode procedure runs, the brainstorm session ENDS. There is no Step 8, no Step 9. The idea is parked until the dependency ships, at which point a fresh brainstorm session re-derives the spec from scratch.

Step 8 — User-review gate

The spec sits in the working tree as an unstaged file. Tell the user:

Spec written to <path> (uncommitted). Please review it and let me know if you want changes before we start the implementation plan. The Boss will commit it once you approve.

Wait for the user's response. If they request changes, edit the file in place (still uncommitted) and re-run Step 7. Only proceed to the next step after approval.

The Boss commits the approved spec after the user signs off (suggested commit subject: spec: <milestone> <topic>). The brainstorm skill does not perform the commit itself.

Step 9 — Hand off to planner

The terminal state of brainstorm is invoking planner. NO other skill is invoked from brainstorm. NO direct jump to implement.

Hand off carries:

  • path to docs/specs/<milestone>.md
  • iteration scope ("the first iteration covers section X+Y of the spec")

Handoff Contract

Direction Carrier
user → brainstorm new milestone request
brainstormailang-grounding-check (Step 7.5) spec_path + iteration_scope
brainstormplanner (Step 9, on PASS or overridden BLOCK) path to spec + iteration scope
brainstorm → roadmap (Step 7.5, on no-override BLOCK) depends on: entry added to docs/roadmap.md in the working tree, spec file deleted from the working tree; Boss commits the roadmap edit

Common Rationalisations

Excuse Reality
"User says 'clear vision, just plan it' — skip the spec" "Clear vision" is a starting point, not a spec. The user's intuition hasn't priced in semantic commitments (RC, uniqueness, codegen, schema). Write the spec; frame it as "load-bearing decisions this idea forces on us".
"Milestone is small, two iterations, no spec needed" Iteration count isn't the metric — feature surface is. A two-iter feature touching DESIGN.md invariants needs a spec; a CLI flag does not. Assess what the feature changes in invariants.
"Three approaches in, none feel right, ship the least bad" Three unsatisfying approaches usually means the problem is mis-framed. Stop, write a JOURNAL entry capturing the three approaches and what fails about each, sleep on it. End-of-day pressure is the worst signal to resolve a design fork.
"User is busy, present my own design without Q&A" Reactive deference disguised as decisiveness. The Q&A surfaces constraints the user hasn't articulated; skipping it means shipping the user's defaults, not their intent.
"Spec exists from previous milestone, append to it" New milestone = new spec file. The architect agent reads spec files per milestone; mixing scopes makes drift review unreadable.
"Approaches A, B, C are all bad — proceed with A" This is exactly the moment to surface "the problem is mis-framed" rather than ratify a known-bad shape into DESIGN.md. JOURNAL the impasse, escalate to user.

Red Flags — STOP

  • "Skip the spec" thoughts (any flavour)
  • "One paragraph is enough"
  • "User said it's clear, who am I to push back"
  • "The 2-3 approaches all suck, just pick one"
  • Skipping Step 7 self-review because "I wrote it carefully"
  • Jumping straight from spec to implement (must go via planner)
  • Inventing a design rationale that's actually about effort, not semantics

Cross-references

  • Project source: docs/DESIGN.md "Feature-acceptance criterion" — gate this skill applies during spec writing.
  • Output target: skills/planner/SKILL.md — only valid next skill.
  • Private agent: skills/brainstorm/agents/ailang-grounding-check.md — dispatched in Step 7.5 as a hard-gate. The agent reads the spec with fresh context and reports PASS / BLOCK / INFRA_ERROR. See docs/specs/2026-05-11-brainstorm-grounding-check.md for the design rationale.
  • Ad-hoc dispatch. The Boss MAY also ad-hoc dispatch skills/planner/agents/ailang-plan-recon.md during Step 1 when the milestone enters code territory the Boss has not recently read; this is opt-in and not part of the standard process. Rationale: brainstorm's recon phase is interwoven with the user Q&A — see docs/specs/2026-05-11-plan-recon-subagent.md for the full reasoning.