catlog22/Claude-Code-Workflow
1
0
Fork 0
mirror of https://github.com/catlog22/Claude-Code-Workflow.git synced 2026-03-12 17:21:19 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(configStore): add type guard for CLI tool type validation

Browse Source 
- Add isValidType type guard to ensure type safety
- Validate tool type against allowed values before assignment
- Fixes TypeScript compilation error in build
This commit is contained in:
catlog22
2026-03-08 21:43:43 +08:00
parent 334f82eaad
commit f3ae78f95e
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
ccw/frontend/src/stores/configStore.ts
View File

@@ -114,6 +114,10 @@ function extractCliToolsFromBackend(data: unknown): Record<string, CliToolConfig
return null; return null;
} }
const validTypes = ['builtin', 'cli-wrapper', 'api-endpoint'] as const;
const isValidType = (t: unknown): t is 'builtin' | 'cli-wrapper' | 'api-endpoint' =>
typeof t === 'string' && (validTypes as readonly string[]).includes(t);
const cliTools: Record<string, CliToolConfig> = {}; const cliTools: Record<string, CliToolConfig> = {};
for (const [key, tool] of Object.entries(backendTools)) { for (const [key, tool] of Object.entries(backendTools)) {
const typedTool = tool as Record<string, unknown>; const typedTool = tool as Record<string, unknown>;
@@ -122,7 +126,7 @@ function extractCliToolsFromBackend(data: unknown): Record<string, CliToolConfig
primaryModel: typeof typedTool.primaryModel === 'string' ? typedTool.primaryModel : '', primaryModel: typeof typedTool.primaryModel === 'string' ? typedTool.primaryModel : '',
secondaryModel: typeof typedTool.secondaryModel === 'string' ? typedTool.secondaryModel : '', secondaryModel: typeof typedTool.secondaryModel === 'string' ? typedTool.secondaryModel : '',
tags: Array.isArray(typedTool.tags) ? typedTool.tags.filter((tag): tag is string => typeof tag === 'string') : [], tags: Array.isArray(typedTool.tags) ? typedTool.tags.filter((tag): tag is string => typeof tag === 'string') : [],
type: typeof typedTool.type === 'string' ? typedTool.type : 'builtin', type: isValidType(typedTool.type) ? typedTool.type : 'builtin',
envFile: typeof typedTool.envFile === 'string' ? typedTool.envFile : undefined, envFile: typeof typedTool.envFile === 'string' ? typedTool.envFile : undefined,
settingsFile: typeof typedTool.settingsFile === 'string' ? typedTool.settingsFile : undefined, settingsFile: typeof typedTool.settingsFile === 'string' ? typedTool.settingsFile : undefined,
availableModels: Array.isArray(typedTool.availableModels) availableModels: Array.isArray(typedTool.availableModels)