Conversation
code-crusher
commented
Jan 16, 2026
- ui changes + IDE theme fixes
- move model selector
There was a problem hiding this comment.
🧪 PR Review is completed: Review of release/v5.2.2. Identified a critical configuration issue with the authentication URL for production, a potential build inefficiency on Windows due to path separators, and some dead code cleanup.
Skipped files
CHANGELOG.md: Skipped file patternwebview-ui/src/i18n/locales/ar/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/ca/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/cs/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/de/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/en/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/es/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/fr/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/hi/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/id/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/it/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/ja/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/ko/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/nl/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/pl/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/pt-BR/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/ru/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/th/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/tr/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/uk/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/vi/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/zh-CN/kilocode.json: Skipped file patternwebview-ui/src/i18n/locales/zh-TW/kilocode.json: Skipped file pattern
⬇️ Low Priority Suggestions (3)
cli/src/utils/browserAuth.ts (1 suggestion)
Location:
cli/src/utils/browserAuth.ts(Lines 132-132)🔴 Configuration
Issue: The authentication URL is hardcoded to
localhost:3000. This is likely a development value and will fail for users in production who are not running the web application locally.Fix: Use an environment variable or the production URL for the release build.
Impact: Authentication will be broken for all users in the release version.
- const authUrl = `http://localhost:3000/authentication/sign-in?loginType=extension&source=${encodeURIComponent( + const authUrl = `${process.env.AXON_SITE_URL ?? "https://app.axoncode.ai"}/authentication/sign-in?loginType=extension&source=${encodeURIComponent(
webview-ui/src/components/ui/select-dropdown.tsx (1 suggestion)
Location:
webview-ui/src/components/ui/select-dropdown.tsx(Lines 262-262)🔵 Code Quality
Issue:
w-min-contentis not a standard Tailwind CSS class. The intended class is likelyw-min(min-content),w-max(max-content), orw-fit(fit-content).Fix: Use
w-fitto ensure the container adapts to its content width.Impact: Prevents potential layout issues.
- <div className="flex flex-col w-min-content"> + <div className="flex flex-col w-fit">
webview-ui/src/components/kilocode/KiloModeSelector.tsx (1 suggestion)
Location:
webview-ui/src/components/kilocode/KiloModeSelector.tsx(Lines 52-87)🔵 Code Quality
Issue: Extensive commented-out logic for organization modes is left in the codebase.
Fix: Remove the dead code.
Impact: Improves code readability.
- const opts: DropdownOption[] = [ - // { - // value: "shortcut", - // label: modeShortcutText, - // disabled: true, - // type: DropdownOptionType.SHORTCUT, - // }, - ] - - // Add organization modes section if any exist - // if (organizationModes.length > 0) { - // // Add header as a disabled item - // opts.push({ - // value: "org-header", - // label: t("chat:modeSelector.organizationModes"), - // disabled: true, - // type: DropdownOptionType.SHORTCUT, - // }) - // opts.push( - // ...organizationModes.map((mode) => ({ - // value: mode.slug, - // label: mode.name, - // codicon: mode.iconName || "codicon-organization", - // // description: mode.description, - // type: DropdownOptionType.ITEM, - // })), - // ) - // opts.push({ - // value: "sep-org", - // label: t("chat:separator"), - // type: DropdownOptionType.SEPARATOR, - // }) - // } - - // Add other modes - opts.push( + const opts: DropdownOption[] = [] + + // Add other modes + opts.push(
| const relativePath = src.replace(/.*\/bin-unpacked\/extension\//, "") | ||
| return !relativePath.startsWith("webview-ui") && !relativePath.startsWith("assets") |
There was a problem hiding this comment.
🟡 Cross-Platform Compatibility
Issue: The regex /.*\/bin-unpacked\/extension\// expects forward slashes. On Windows, src paths from cpSync typically contain backslashes, causing this regex to fail. This results in the filter returning true for all files, copying unnecessary directories (like webview-ui) which are only removed later by removeUnneededFiles, making the build inefficient.
Fix: Normalize path separators to forward slashes before applying the regex.
Impact: Improves build performance on Windows by correctly filtering files.
| const relativePath = src.replace(/.*\/bin-unpacked\/extension\//, "") | |
| return !relativePath.startsWith("webview-ui") && !relativePath.startsWith("assets") | |
| // Skip webview-ui and assets to reduce size | |
| const relativePath = src.replace(/\\/g, "/").replace(/.*\/bin-unpacked\/extension\//, "") | |
| return !relativePath.startsWith("webview-ui") && !relativePath.startsWith("assets") |