IW-636: Two-line PURPOSE headers for command scripts and PhaseMergeHarnessTest, and remove research/ #655

Merged

mph merged 14 commits from IW-636 into main 2026-09-29 12:36:16 +00:00

Conversation 0 Commits 14 Files changed 81 +1095 -2891

claude commented 2026-09-29 11:39:31 +00:00

Member

Copy link

Summary

Every command script and PhaseMergeHarnessTest.scala now starts with exactly two // PURPOSE: lines, and the unused research/ prototypes are deleted. This unblocks the repository-wide header check in #630.

Changes

• 27 commands/*.scala shims: second // PURPOSE: line added. docs/testing.md no longer describes shims as an exception to the two-line rule.
• .iw/commands/test.scala: second // PURPOSE: line added above // USAGE:.
• core/test/PhaseMergeHarnessTest.scala: lines 2-3 replaced by one line that describes what the tests cover.
• research/ deleted (41 files; no build file or doc refers to it).
• Phase PR: #653.

Testing

Comment-only change plus deletion. Header counts verified with rg -c '^// PURPOSE:'; command scripts compile; PhaseMergeHarnessTest passes (see implementation log). Review packet: project-management/issues/IW-636/review-packet.md.

Follow-up

• #654: line 1 of .iw/commands/test.scala does not name the lint and contract suites.

Release notes

Datum: 2026-09-29

Tato změna se netýká chování nástroje iw-cli. Příkazy fungují stejně jako dosud a uživatel nic nového nastavovat nemusí. Přínos mají vývojáři, kteří s projektem pracují: popis účelu na začátku každého souboru je nyní úplný a jednotný, takže rychlé vyhledání těchto popisů dává o všech příkazech celistvý přehled.

Každý příkazový skript nyní na začátku nese dva řádky popisu. První řádek říká, co příkaz dělá, druhý pak přesně popisuje, jak skript předává práci dál a s jakým výsledkem končí. Stejně je doplněn i skript, který spouští testy projektu, a popis jedné testové sady byl zkrácen na dva řádky, které odpovídají tomu, co sada skutečně ověřuje. Vývojářská příručka o testování už pro tyto skripty neuvádí žádnou výjimku, protože pravidlo dvou řádků nyní platí pro všechny soubory bez rozdílu.

Z repozitáře zmizel také adresář se starými prototypy, na které se už nic neodkazovalo. Projekt je tak přehlednější a připravený na automatickou kontrolu, která bude jednotnou podobu těchto popisů hlídat u všech souborů.

🤖 Generated with Claude Code

Closes #636

## Summary Every command script and `PhaseMergeHarnessTest.scala` now starts with exactly two `// PURPOSE:` lines, and the unused `research/` prototypes are deleted. This unblocks the repository-wide header check in #630. ## Changes - 27 `commands/*.scala` shims: second `// PURPOSE:` line added. `docs/testing.md` no longer describes shims as an exception to the two-line rule. - `.iw/commands/test.scala`: second `// PURPOSE:` line added above `// USAGE:`. - `core/test/PhaseMergeHarnessTest.scala`: lines 2-3 replaced by one line that describes what the tests cover. - `research/` deleted (41 files; no build file or doc refers to it). - Phase PR: #653. ## Testing Comment-only change plus deletion. Header counts verified with `rg -c '^// PURPOSE:'`; command scripts compile; `PhaseMergeHarnessTest` passes (see implementation log). Review packet: `project-management/issues/IW-636/review-packet.md`. ## Follow-up - #654: line 1 of `.iw/commands/test.scala` does not name the lint and contract suites. ## Release notes **Datum:** 2026-09-29 Tato změna se netýká chování nástroje iw-cli. Příkazy fungují stejně jako dosud a uživatel nic nového nastavovat nemusí. Přínos mají vývojáři, kteří s projektem pracují: popis účelu na začátku každého souboru je nyní úplný a jednotný, takže rychlé vyhledání těchto popisů dává o všech příkazech celistvý přehled. Každý příkazový skript nyní na začátku nese dva řádky popisu. První řádek říká, co příkaz dělá, druhý pak přesně popisuje, jak skript předává práci dál a s jakým výsledkem končí. Stejně je doplněn i skript, který spouští testy projektu, a popis jedné testové sady byl zkrácen na dva řádky, které odpovídají tomu, co sada skutečně ověřuje. Vývojářská příručka o testování už pro tyto skripty neuvádí žádnou výjimku, protože pravidlo dvou řádků nyní platí pro všechny soubory bez rozdílu. Z repozitáře zmizel také adresář se starými prototypy, na které se už nic neodkazovalo. Projekt je tak přehlednější a připravený na automatickou kontrolu, která bude jednotnou podobu těchto popisů hlídat u všech souborů. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Closes #636

claude added 13 commits 2026-09-29 11:39:31 +00:00

workflow(wf): Create analysis for IW-636 ... a9c4355d5c

Command scripts and PhaseMergeHarnessTest.scala do not have exactly two PURPOSE lines

Artifacts:
- analysis.md
- issue.md

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

workflow(wf): Create task index for IW-636 ... 7d36b9c1cc

Command scripts and PhaseMergeHarnessTest.scala do not have exactly two PURPOSE lines

Artifacts:
- tasks.md
- review-state.json (tasks_ready)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

docs(IW-636): intake verdict — agent-ready ... b96474c900

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Phase 1: context ready ... aa13e8083b

Issue: IW-636
Phase: 1 - Two-line PURPOSE headers for command scripts and PhaseMergeHarnessTest, and remove research/

This commit marks the checkpoint for phase 1 context generation.
Use wf-implement IW-636 to return to this point.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

chore(IW-636): update review-state to context_ready bab852457a

workflow(wf): Generate Phase 1 tasks for IW-636 ... 5765df42d5

Two-line PURPOSE headers for command scripts and PhaseMergeHarnessTest, and remove research/

Artifacts:
- phase-01-tasks.md

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

chore(IW-636): update review-state to tasks_ready aa4b8d7e3e

Phase 1: Two-line PURPOSE headers for command scripts and PhaseMergeHarnessTest, and remove research/ (#653) 925d738988

chore(IW-636): update review-state for phase 01 8cbeaa11cd

chore(IW-636): mark phase 01 complete in tasks.md 6914cb0aca

chore(IW-636): file follow-up as IW-654, remove FOLLOW-UPS.md ... 33bcfd016c

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

docs(IW-636): add release notes ... a9e7511d25

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

docs(IW-636): add review packet ... 7deac01a30

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

mph added 1 commit 2026-09-29 11:40:33 +00:00

chore(IW-636): update review-state to all_complete 1352443553

mph merged commit 33a770b874 into main 2026-09-29 12:36:16 +00:00

mph referenced this pull request from a commit 2026-09-29 12:36:17 +00:00

Merge pull request 'IW-636: Two-line PURPOSE headers for command scripts and PhaseMergeHarnessTest, and remove research/' (#655) from IW-636 into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

agent-ready

  

bug

Something isn't working

  

contract

Run the tool-contract suite on this PR

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

needs-human

  

needs-input

Intake asked the human a question on the issue; waiting for the answer

  

question

Further information is requested

  

wontfix

This will not be worked on

No labels

agent-ready

bug

contract

documentation

duplicate

enhancement

good first issue

help wanted

invalid

needs-human

needs-input

question

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

iterative-works/iw-cli!655

No description provided.