IW-628: Move second PURPOSE line above package in six core/test files #640

Merged
claude merged 13 commits from IW-628 into main 2026-09-26 11:34:31 +00:00
Member

Summary

Six core/test files had their second // PURPOSE: line at line 4, below package iw.tests. The project rule puts both PURPOSE lines at lines 1-2. This PR moves each second line up to line 2, above package.

Defects fixed

  • Phase 1 (#639): second PURPOSE line after the package line in ConfigFileTest.scala, ConfigRepositoryTest.scala, DeletionSafetyTest.scala, GitRemoteTest.scala, LogTest.scala, OutputTest.scala.

Testing

Comment-only change. The phase PR compiled the files and passed CI. No runtime behaviour changes.

Release notes

See project-management/issues/IW-628/release-notes.md.

🤖 Generated with Claude Code

Closes #628

## Summary Six `core/test` files had their second `// PURPOSE:` line at line 4, below `package iw.tests`. The project rule puts both PURPOSE lines at lines 1-2. This PR moves each second line up to line 2, above `package`. ## Defects fixed - Phase 1 (#639): second PURPOSE line after the package line in `ConfigFileTest.scala`, `ConfigRepositoryTest.scala`, `DeletionSafetyTest.scala`, `GitRemoteTest.scala`, `LogTest.scala`, `OutputTest.scala`. ## Testing Comment-only change. The phase PR compiled the files and passed CI. No runtime behaviour changes. ## Release notes See `project-management/issues/IW-628/release-notes.md`. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Closes #628
Six more core/test files have their second PURPOSE line after the package line

Artifacts:
- analysis.md
- issue.md

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Six more core/test files have their second PURPOSE line after the package line

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

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Issue: IW-628
Phase: 1 - Second PURPOSE line sits after the package line in six core/test files

This commit marks the checkpoint for phase 1 context generation.
Use dx-fix IW-628 to return to this point.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Second PURPOSE line sits after the package line in six core/test files

Artifacts:
- phase-01-tasks.md

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
docs(IW-628): add release notes
Some checks failed
CI / format (pull_request) Successful in 10s
CI / changes (pull_request) Successful in 4s
CI / contract (pull_request) Has been skipped
CI / lint (pull_request) Successful in 11s
CI / test (pull_request) Has been cancelled
CI / dashboard-build (pull_request) Has been cancelled
CI / compile (pull_request) Has been cancelled
36ba5de7b2
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
chore(IW-628): update review-state to all_complete
All checks were successful
CI / changes (pull_request) Successful in 4s
CI / contract (pull_request) Has been skipped
CI / format (pull_request) Successful in 9s
CI / lint (pull_request) Successful in 11s
CI / compile (pull_request) Successful in 1m35s
CI / dashboard-build (pull_request) Successful in 1m13s
CI / test (pull_request) Successful in 15m51s
d9eccdf79b
claude merged commit 9caddfdae7 into main 2026-09-26 11:34:31 +00:00
Sign in to join this conversation.
No description provided.