From 103c6bc8a0d723a44f68c8f5846c6717d82e8d7c Mon Sep 17 00:00:00 2001 From: Shirone Date: Wed, 21 Jan 2026 22:26:01 +0100 Subject: [PATCH] docs: improve comment clarity for resolvePhaseModel usage MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Updated the comment to better explain why resolveModelString is not needed after resolvePhaseModel - the latter already handles model alias resolution internally. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- apps/server/src/services/ideation-service.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/server/src/services/ideation-service.ts b/apps/server/src/services/ideation-service.ts index 1035ce76..aa6790c6 100644 --- a/apps/server/src/services/ideation-service.ts +++ b/apps/server/src/services/ideation-service.ts @@ -696,7 +696,7 @@ export class IdeationService { '[IdeationService]' ); const resolved = resolvePhaseModel(phaseResult.phaseModel); - // resolvePhaseModel returns the canonical model identifier (e.g., 'sonnet' → 'claude-sonnet-4-5-20250929') + // resolvePhaseModel already resolves model aliases internally - no need to call resolveModelString again const modelId = resolved.model; const claudeCompatibleProvider = phaseResult.provider; const credentials = phaseResult.credentials;