refactor: move new changes to kilo folders

This commit is contained in:
Catriel Müller
2026-06-01 23:33:44 -03:00
parent 643810c5cb
commit 859af5033b
3 changed files with 8 additions and 10 deletions
+1 -1
View File
@@ -4,7 +4,7 @@ import { pathToFileURL } from "url"
import { UI } from "../ui"
import { cmd } from "./cmd"
import { Flag } from "@opencode-ai/core/flag/flag"
import { buildRunMessage } from "./run-message" // kilocode_change
import { buildRunMessage } from "@/kilocode/cli/cmd/run-message" // kilocode_change
import { bootstrap } from "../bootstrap"
import { EOL } from "os"
import { text as streamText } from "node:stream/consumers"
@@ -1,4 +1,3 @@
// kilocode_change - new file
// Atoms before `--` are positional shell arguments where re-quoting around
// embedded spaces preserves the user's word-binding intent (PR #4979).
// Atoms in `args["--"]` are raw passthrough per yargs `populate--` semantics:
@@ -6,7 +5,7 @@
// not synthesize quote bytes around them. Re-quoting raw atoms breaks
// leading-dash inputs like `kilo run -- "- Who are you?"` (#9622) by
// emitting `"- Who are you?"` (literal quotes) into the model prompt.
export function buildRunMessage(positionals: string[], dashDash?: string[]): string {
export function buildRunMessage(positionals: string[], dash?: string[]): string {
const quoted = positionals.map((arg) => (arg.includes(" ") ? `"${arg.replace(/"/g, '\\"')}"` : arg))
return [...quoted, ...(dashDash ?? [])].join(" ")
return [...quoted, ...(dash ?? [])].join(" ")
}
@@ -1,6 +1,5 @@
// kilocode_change - new file
import { describe, expect, test } from "bun:test"
import { buildRunMessage } from "../../../src/cli/cmd/run-message"
import { buildRunMessage } from "../../../../src/kilocode/cli/cmd/run-message"
describe("buildRunMessage", () => {
test("preserves shell-bound multi-word positionals via wrap-quote (PR #4979)", () => {
@@ -16,21 +15,21 @@ describe("buildRunMessage", () => {
})
test("passes args['--'] through verbatim without wrap-quote (#9622)", () => {
// `kilo run -- "- Who are you?"` yargs+populate-- captures the leading-dash
// `kilo run -- "- Who are you?"` - yargs+populate-- captures the leading-dash
// phrase as a single atom in args["--"]. The assembler must NOT wrap it,
// because the user typed `--` precisely to opt out of further parsing.
expect(buildRunMessage([], ["- Who are you?"])).toBe("- Who are you?")
})
test("does not synthesize quote bytes around dashDash atoms even when they contain spaces", () => {
test("does not synthesize quote bytes around dash atoms even when they contain spaces", () => {
expect(buildRunMessage([], ["one two", "three"])).toBe("one two three")
})
test("combines positionals and dashDash with appropriate quoting per source", () => {
test("combines positionals and dash args with appropriate quoting per source", () => {
expect(buildRunMessage(["pre", "fix arg"], ["raw arg", "tail"])).toBe('pre "fix arg" raw arg tail')
})
test("handles undefined and empty dashDash identically", () => {
test("handles undefined and empty dash args identically", () => {
expect(buildRunMessage(["x"], undefined)).toBe("x")
expect(buildRunMessage(["x"], [])).toBe("x")
})