docs(spec): sessie-usage per taak en schatting vs werkelijk (IDEA-235) #301
No reviewers
Labels
No labels
severity/s3
severity/s4
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
janpeter/Scrum4Me!301
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "claude/idea-235-session-usage-spec"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Spec voor IDEA-235: usage per taak en per sprint in interactieve Claude Code-sessies, uitgesplitst per agent × model, met een schatting vooraf (ceremonie) en een vergelijking met de werkelijke usage.
docs/superpowers/specs/2026-10-04-session-usage-estimates-design.md(revisie 4)Eerste increment is een praktijkproef met alleen de mod (in
janpeter/my-mods), vóór enig Scrum4Me-schema.🤖 Generated with Claude Code
REQUEST_CHANGES
Findings:
ERROR (blokkerend) — docs/superpowers/plans/2026-10-05-session-usage-estimates.md:112: De voorloopbuffer en pendingOverhead bewaren geen opgebouwde activeMs. Na een afgeronde hoofdbeurt vóór de eerste in_progress is turnStartedAt weer null, zonder plek voor de gemeten actieve tijd. Bij latere materialisatie van overhead is die tijd niet meer te reconstrueren; startedAt geeft alleen wandkloktijd en omvat ook wachttijd. Voeg een actieve-tijdaccumulator toe voor buffer en lui geopende overhead, inclusief AskUserQuestion-aftrek. Test meerdere afgeronde beurten met een pauze vóór de eerste taak en een afgeronde beurt vóór de eerste overhead-step.
ERROR (blokkerend) — docs/superpowers/plans/2026-10-05-session-usage-estimates.md:336: record_usage_segment heeft geen anchor_task_id, terwijl overhead task_id = null vereist (spec §3.2). Een nieuw overheadbericht kan daarmee geen taak/anker aanleveren om sprint, product en toegang te bepalen. task_id invullen met het anker zou overhead als taakusage opslaan. Voeg anchor_task_id en de null-semantiek van task_id expliciet aan het toolcontract toe en test een nieuw overheadsegment, ook als het sluitbericht als eerste aankomt.
Plan gekoppeld via pr_refs. Referentie:
docs/superpowers/specs/2026-10-04-session-usage-estimates-design.md. De gekoppelde content is afgekapt op 20.000 tekens (truncated: true); de volledige specificatie staat in de aangeleverde diff en is meegenomen.omittedis leeg.De PR bevat uitsluitend twee nieuwe Markdown-documenten en claimt geen afgeronde implementatie. Ontbrekende runtime-code, migraties en uitgevoerde praktijktests zijn daarom geen findings. De bovenstaande punten betreffen tegenstrijdigheden tussen de voorgeschreven interfaces en de meet-/overheadvereisten van de specificatie. De DB-contracts en statusmapping sluiten verder aan op de geraadpleegde productrichtlijnen (db-access-policy en ADR-0004).
Verificatie: volledige diff statisch beoordeeld; Forgejo-PR is open en head SHA komt overeen met
baa245c911afe64ae238645071c2a76c42190a77. Geen build of tests uitgevoerd: deze diff bevat geen uitvoerbare wijziging.REQUEST_CHANGES
Plan gekoppeld via pr_refs (product Scrum4Me). Referentie:
docs/superpowers/specs/2026-10-04-session-usage-estimates-design.md. Geen omitted onderdelen gemeld.Findings:
UNTRACKED-pad zet een voorloopbuffer met requests inpendingOverhead(regel 161), maar alleen een volgendeaddStepmaterialiseert die.endSessiongooit deze toestand vervolgens weg; een directein_progressop Y sluit eveneens alleen open overhead (regel 158). Reproductie:/clear→ requests →doneX zonder handdruk → sessie-einde, zonder nieuwe request. De gebufferde tokens, actieve tijd en kosten verdwijnen. Dit spreekt spec §4.3/§9 en de eigen test op regel 184 tegen. Materialiseer gevulde overhead bij de bufferomzetting, of sluit deze expliciet bij sessie-einde én taakwissel. Alleen overhead zonder requests mag vervallen.Beoordeling: beide toegevoegde Markdown-bestanden gelezen; PR-head
de8b992ec4838462667f4cbe1ea000686c3b7eedbevestigd. Getoetst aan product-docsrunbooks/db-access-policyen ADR-0004. De eerdere punten over actieve tijd enanchor_task_idzijn verwerkt. Deze PR claimt alleen documentatie; nog niet uitgevoerde increments zijn geen finding. Geen runtimecode gewijzigd, daarom geen build of tests uitgevoerd. De beschreven regressietest kan niet slagen wanneer de huidige afsluitregels letterlijk worden gevolgd.APPROVED
Findings
Plan gekoppeld via pr_refs. Referentie:
docs/superpowers/specs/2026-10-04-session-usage-estimates-design.md. Geen omitted-verwijzingen; het gekoppelde document was afgekapt in de payload, maar de volledige specificatie is uit de diff en de exacte PR-head gelezen.De PR rondt uitsluitend documentatie af. Het implementatieplan sluit aan op de specificatie, inclusief de vastgelegde afwijkingen: afgeleide wandkloktijd en kostenstanden in het MCP-bericht. Buffertijd, overheadankers, UNTRACKED, hoofdbeurtherkenning, eigendomscontrole en monotone segmentafsluiting zijn beschreven met gerichte verificaties. TaskEstimate en additieve tabelcontracten passen bij de bestaande adoptie- en databasepolicy; statusmapping en rate-cardgebruik volgen de productstandaarden.
Implementatie en praktijkbewijs behoren tot de latere increments en zijn voor deze PR geen ontbrekende oplevering. Externe plugin-API-aannames worden expliciet getoetst vóór increment 2.
Verificatie op commit
9afb400d3103dc893003ab7f39e9e6502c128df5: payloadbestanden identiek aan de checkout;git diff --check origin/main HEADgeslaagd;npm run docs:check-linksgeslaagd (310 bestanden). Runtime-tests, typecheck en build zijn niet uitgevoerd voor deze uitsluitend Markdown-wijziging.