[Test] Add Step Boundary Range Assertion to Weekly Review Test Suite

• Labels: good first issue, testing, difficulty:beginner, area:core
• Target File: tests/weekly-review.test.ts
• Estimated Time: 15–20 minutes
• Difficulty: Level 3 (Beginner)

Why This Matters

The Weekly Review ritual progresses through 5 discrete steps (1: Retrospective, 2: Tasks Audit, 3: Habit Scorecard, 4: Goal Alignment, 5: Strategic Commitments). Boundary tests ensure invalid step indices (< 1 or > 5) are rejected cleanly.

Current Behavior

tests/weekly-review.test.ts validates standard step advancement, but does not explicitly assert that step indices 0 or 6 throw validation errors.

Requested Change

Add a dedicated test block in tests/weekly-review.test.ts asserting that step transitions clamp to valid range [1, 5].

Acceptance Criteria

• Boundary check test case added to tests/weekly-review.test.ts.
• Test executes and passes cleanly.

Verification

npm run test:file -- tests/weekly-review.test.ts

Contributor Notes

Self-contained unit test addition.