session-report

Session Report

Audit your own session against the task file(s) you were given. Surface assumptions you made and context gaps that caused struggles. These are observations for the human — not things you can fix after the fact.

Review Process

Work through each dimension in order. For every finding, cite specific evidence.

Dimension 1: Assumption Audit

Identify every place you made a decision that was not dictated by the task file or spec. For each assumption:

• Ambiguity — what the spec didn't say
• Choice — what you decided and why (the reasoning, not just the outcome)
• Risk — what could go wrong if this choice is wrong. Be specific: name the scenario, the affected code path, and what would break. "Could surprise future workflows" is not specific enough — say which workflow shape triggers it and what the symptom would be.
• Impact classification:
  • benign — reasonable default, no real consequence (do not report)
  • notable — meaningful choice that could have gone differently
  • risky — could cause problems, warrants review
• Recommendation — what the reviewer should do: verify behavior, add a test, clarify the spec, or approve as-is with rationale.

Report only risky and notable assumptions. Drop benign entirely — they dilute the signal. Sort risky first, then notable.

Dimension 2: Context Gaps

A context gap is a major, avoidable hole in the task file or project context that sent you down a fundamentally wrong path, caused you to build the wrong thing, or wasted significant effort that better upfront information would have prevented entirely.

The bar is high. Most sessions have zero context gaps. Fixing a linter warning is not a context gap. Searching for a convention is not a context gap. Iterating on validator feedback is not a context gap — it is the normal development loop.

Examples of real context gaps:

• The task said to extend module X, but module X was deleted last week and replaced by module Y — you built against a nonexistent target
• The task required integrating with service A but didn't mention that service A requires an auth token from a separate provisioning step — you couldn't have known this without tribal knowledge
• The task file pointed you to the wrong directory or package, wasting significant effort before you found the right one

Things that are NOT context gaps (do not report these):

• Anything the validator caught (lint, security, permissions, test failures) — the validator's entire purpose is to catch what the task doesn't enumerate
• Quick searches for conventions, patterns, or existing code — that is normal development
• Iterative fix cycles of any kind — fixing things the first pass missed is expected, not a gap
• Learning something new about the codebase during implementation

For each genuine context gap:

• What happened — the concrete wasted effort or wrong direction
• What was missing — the specific information that, if present, would have prevented it entirely

Output Format

## Assumption Audit

### Risky

1. **[What the spec didn't say]**
   - **Choice:** [What you decided and why]
   - **Risk:** [Specific scenario that breaks — name the code path, workflow shape, and symptom]
   - **Recommendation:** [What the reviewer should do]

### Notable

1. **[What the spec didn't say]**
   - **Choice:** [What you decided and why]
   - **Risk:** [Specific scenario that breaks — name the code path, workflow shape, and symptom]
   - **Recommendation:** [What the reviewer should do]

## Context Gaps

- [What happened] — Missing context: [what was needed]

Omit any section (Risky, Notable) that has zero items. Do not include a Benign section.

Guardrails

• Do not suggest fixes for context gaps. Report what needs attention; leave solutions to the human.
• Do not conflate code quality issues with assumptions. An assumption is a decision you made where the spec was silent — not a code style choice.