The KiloProvider session refresh tests failed intermittently because
two test files each called mock.module('vscode', ...) with different
mock shapes. Since Bun's mock.module is global and tests run
concurrently, the module resolution could non-deterministically pick
up the wrong mock depending on execution order.
Fix: move the vscode mock to a shared preload file loaded via
bunfig.toml [test].preload. This ensures a single comprehensive mock
is registered once before any test file loads, eliminating the race.
* feat: add apply-to-local button in Agent Manager
* fix: address PR review - path traversal guard, safer regex, stdin null check
- Reject selectedFiles with absolute paths or .. components
- Narrow conflict regex to known git error patterns (avoids Windows colon misparse)
- Fail early if child.stdin is null instead of silently dropping patch data
* fix: address PR review round 2
- Add ApplyDialog.tsx to CSS consistency test TSX_FILES
- Clear applyTarget when worktree is deleted (avoids stale dialog)
- Add integration tests for checkApplyPatch and applyPatch
* fix: add missing vscode.Uri mock in KiloProvider session refresh test
The mock was missing the Uri namespace, causing TypeError on CI where
Bun resolves the type annotation at runtime. Locally it worked because
the constructor param was cast via 'as never'.
* fix: harden KiloProvider test mock with full Uri class and Disposable
The previous fix only added static methods to Uri. CI Bun resolves
vscode.Uri as a runtime class reference in the constructor parameter
type position. Provide a proper MockUri class, Disposable, and
EventEmitter stubs so the mock satisfies all runtime lookups.
Screenshot baselines are captured on Linux CI. Font rendering and
anti-aliasing differ on macOS, producing false-positive diffs. Skip
the entire suite on darwin with a warning message and avoid the
unnecessary Storybook fetch.
Per review feedback:
- Run runtime checks from packages/kilo-ui/ (the actual import path kilo-vscode
uses) instead of packages/ui/
- Add source-level check for onOpenFile prop and OpenFileFn type in DataProvider
(TypeScript types erased at runtime, so source analysis needed)
- Add source-level check for ToolInfo type icon/title fields
- Document why getToolInfo() can't be invoked (needs SolidJS context)
- ToolProps is a TS interface caught by typecheck, not runtime-testable
Instead of reading source files and matching regexes, actually import the
upstream modules and verify exports at runtime. Uses Bun.spawnSync to run
from packages/ui/ where the SolidJS JSX context is available.
- ToolRegistry.render(name) returns a function for each depended-on tool
- getToolInfo is an exported function
- DataProvider and useData are exported functions
This is more robust than regex matching and catches real breakage.
- Add package.json to Storybook cache key (dep version changes invalidate cache)
- Add res.ok check before parsing Storybook index JSON
- Remove duplicate maxDiffPixelRatio (already in playwright.config.ts)
- Use parseInt for PLAYWRIGHT_WORKERS to handle non-numeric values
- Add @playwright/test, playwright.config.ts (Chromium, 1280x720)
- Test file auto-discovers all stories from Storybook's index.json
- Screenshots cropped to #storybook-root, grouped per component
- CI workflow on Blacksmith with Storybook/Playwright/Bun caching
- Paths filter: only runs when kilo-ui or its deps change
- Auto-commits missing baselines back to the branch via CI
- Skips head-only (Font, Favicon) and non-deterministic (Typewriter) stories
- Documented in CONTRIBUTING.md
* fix(docs): self-host favicon assets and fix broken head declarations
- Remove duplicate ICO favicon (shortcut icon + icon), keep single icon declaration
- Add self-hosted favicon assets in public/favicon/ (copied from packages/ui v3 assets)
- Fix SVG favicon path (was 404 at /favicon.svg)
- Fix apple-touch-icon path (was 404 at /apple-touch-icon.png)
- Add 96x96 desktop PNG favicon
- Add apple-mobile-web-app-title meta tag
- Add site.webmanifest with correct Kilo Code branding
- All declared paths now resolve to actual files under /docs/ basePath
* fix(docs): use correct favicon assets from kilo.ai landing site
Replace incorrect favicon files (sourced from packages/ui) with the exact
assets used on kilo.ai (from kilocode-landing repo):
- Replace all icon PNGs with correct landing site versions
- Rename web-app-manifest-*.png to android-chrome-*.png to match landing site
- Remove favicon-96x96.png (not used on landing site)
- Update favicon.ico and favicon.svg with correct landing site versions
- Update site.webmanifest to reference android-chrome-* files without maskable purpose
- Add shortcut icon declaration to match landing site head config
* Update packages/kilo-docs/public/favicon/favicon.svg
Co-authored-by: kilo-code-bot[bot] <240665456+kilo-code-bot[bot]@users.noreply.github.com>
---------
Co-authored-by: kiloconnect[bot] <240665456+kiloconnect[bot]@users.noreply.github.com>
Co-authored-by: Brendan O'Leary <github@olearycrew.com>
Co-authored-by: kilo-code-bot[bot] <240665456+kilo-code-bot[bot]@users.noreply.github.com>
kilo-maintainer[bot]