docs: clarify object parameter pattern exception for CLI entry points

[EdwardIrby]

Summary

• Adds clarification to the Object Parameter Pattern rule explaining the CLI entry point exception
• CLI functions take args: string[] because that's what the shell provides—parsing happens inside
• The object parameter rule applies to internal APIs where callers pass typed values directly

Test plan

• Verify markdown renders correctly
• No code changes, documentation only

🤖 Generated with Claude Code

[claude]

Code Review: PR #19

Summary: Documentation improvement clarifying the Object Parameter Pattern exception for CLI entry points.

✅ Strengths

1. Clear and Accurate: The exception accurately describes the CLI pattern used throughout the codebase (verified in src/lsp-*.ts, src/validate-skill.ts, src/scaffold-rules.ts)

2. Well-Positioned: Placed immediately after the Object Parameter Pattern rule, making it easy to find when applying the guideline

3. Concise Rationale: Explains both the technical reason (args: string[] is what the shell provides) and the scope (rule applies to internal APIs with typed values)

4. Consistent Formatting: Follows the existing documentation style with bold Exception prefix

📝 Observations

• No code changes, documentation only ✓
• Markdown syntax is correct ✓
• Aligns with existing CLI implementations ✓
• No test changes needed (documentation-only change) ✓

🎯 Recommendations

None - This is a straightforward documentation improvement that adds valuable clarification without introducing any issues.

Final Assessment

Approved ✅

This change improves the code review guidelines by documenting an existing, legitimate pattern used throughout the CLI codebase. The exception is well-justified and clearly scoped.

---

Review conducted following code-review.md standards

[claude]

This PR adds valuable clarification to the Object Parameter Pattern rule. The documentation change is well-written and accurately reflects the codebase patterns.

LGTM - I verified that all CLI functions in the codebase (lspHover, lspFind, lspRefs, lspAnalyze, validateSkill, scaffoldRules, lspSymbols) follow the args: string[] pattern with internal parsing via parseArgs.

The clarification correctly distinguishes CLI entry points from internal APIs, preventing future confusion about when to apply the object parameter pattern.

No changes requested.