Conversation
Add the ability to run `kilo run --auto` on a KiloClaw instance from the dashboard, with real-time output polling and DB persistence. Stack: controller spawn route -> CF Worker DO proxy -> platform routes -> tRPC procedures -> React polling page. Includes admin visibility, changelog entry, and 12 controller route tests.
Sync the Kilo CLI's model config with the user's selected KiloClaw default model (KILOCODE_DEFAULT_MODEL). Previously the CLI ignored this env var and fell back to kilo-auto/small. Now the model is written to opencode.json on both fresh installs and every boot, converting the kilocode/ provider prefix to kilo/ for the CLI's naming convention.
Wrap the user's prompt with system context (key paths, architecture, diagnostic commands, and fix instructions) so the agent knows where to look and how to repair broken OpenClaw instances. The original user prompt is still stored for UI display; only the expanded prompt is passed to `kilo run --auto`.
…CLI run UI Export path constants from config-writer and kilo-cli-config so the prompt template references them instead of duplicating string literals. Add openclaw doctor to diagnostics. Improve the CLI run detail page with SetPageTitle, remove max-height on output, and clean up layout.
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Contributor
Code Review SummaryStatus: 5 Issues Found | Recommendation: Address before merge Overview
Fix these issues in Kilo Cloud Issue Details (click to expand)WARNING
Other Observations (not in diff)Issues found in unchanged code that cannot receive inline comments:
Resolved since previous review:
Files Reviewed (5 files)
Reviewed by gpt-5.4-20260305 · 3,615,017 tokens |
RSO
commented
Mar 27, 2026
Comment on lines
+193
to
+195
| if (env.KILOCODE_API_BASE_URL) { | ||
| env.KILO_API_URL = new URL(env.KILOCODE_API_BASE_URL).origin; | ||
| } |
Contributor
Author
There was a problem hiding this comment.
This replaces the providers.kilo.options.baseURL setting in the opencode.json config, which doesn't work as expected.
| config.provider = config.provider || {}; | ||
| config.provider.kilo = config.provider.kilo || {}; | ||
| config.provider.kilo.options = config.provider.kilo.options || {}; | ||
| delete config.provider.kilo.options.baseURL; |
Contributor
Author
There was a problem hiding this comment.
As far as I could test baseURL just never works as you want it to because either the models-fetch or the chat-completion fails because it uses the wrong path.
St0rmz1
reviewed
Mar 27, 2026
St0rmz1
reviewed
Mar 27, 2026
St0rmz1
reviewed
Mar 27, 2026
Exclude the output column from listAllCliRuns to avoid sending up to ~25MB per page. Add a dedicated getCliRunOutput procedure that fetches output for a single run on demand when selected in the admin panel.
jeanduplessis
approved these changes
Mar 30, 2026
# Conflicts: # kiloclaw/controller/src/config-writer.ts # packages/trpc/dist/index.d.ts
| .input(z.object({ runId: z.string().uuid() })) | ||
| .mutation(async ({ ctx, input }) => { | ||
| const client = new KiloClawInternalClient(); | ||
| const result = await client.cancelKiloCliRun(ctx.user.id); |
Contributor
There was a problem hiding this comment.
WARNING: runId still is not sent to the controller cancel API
cancelKiloCliRun only narrows the database update. The controller request here still cancels the user's current run regardless of input.runId, so canceling an older row after a newer recovery starts can terminate the newer job and leave the wrong row marked as cancelled.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
kilo run --autoon a KiloClaw instance from the dashboard as a recovery tool for stuck/failing instances, with real-time output polling and DB persistenceVerification
pnpm typecheck— passes (all packages)pnpm format:check— passespnpm lint— passes (0 warnings, 0 errors)kilo-cli-run.test.ts)Visual Changes
hooks-enabled-recovery.mp4
Reviewer Notes
kiloclaw_cli_runstable with FK tokilocode_usersand indexes onuser_idandstarted_at.kilo-cli-config.tsgives the agent context about the OpenClaw environment.@tanstack/react-querywith a 2s refetch interval while the run is active.