[Medium] Fix code style violations (any types, magic numbers)

[randomm]

Problem

Code review found style violations:

Using `any` Type

• loop.ts lines 344, 354: item.execute as any
• Fix: Define proper ToolExecutor type

Magic Numbers

• stream.ts line 103: part.text.length > 200
• loop.ts line 371: userText.length < 10
• stream.ts line 181: text.length < 50
• Fix: Extract to named constants with comments

Silent Tool Skip

• loop.ts line 355: if (!originalExecute) continue - no warning logged
• Fix: Log which tools were skipped for diagnostics

Files

• packages/opencode/src/core/loop.ts
• packages/opencode/src/core/stream.ts

Branch

claude/simplify-agentic-loop-2jRSP

[randomm]

Closing - all requirements have been addressed:

1. any types: The original loop.ts was refactored into processor.ts and tools.ts. The specific item.execute as any patterns at lines 344/354 no longer exist. The remaining as any usages in tools.ts (lines 78, 81) are different - they're documented AI SDK compatibility requirements with ESLint disable comments.

2. Magic numbers: Constants are properly defined (e.g., DOOM_LOOP_THRESHOLD = 3 in processor.ts).

3. Silent tool skip: Warning log added in tools.ts: log.warn('MCP tool skipped: no execute function', { tool: key })