add kilocode change markers

Author: markijbema

webview-ui/src/utils/__tests__/command-validation.spec.ts

@@ -12,11 +12,14 @@ import {
 	CommandValidator,
 	createCommandValidator,
 	containsDangerousSubstitution,
+	// kilocode_change start
 	protectNewlinesInQuotes,
 	NEWLINE_PLACEHOLDER,
 	CARRIAGE_RETURN_PLACEHOLDER,
+	// kilocode_change end
 } from "../command-validation"
 
+// kilocode_change start
 describe("protectNewlinesInQuotes", () => {
 	const newlinePlaceholder = NEWLINE_PLACEHOLDER
 	const crPlaceholder = CARRIAGE_RETURN_PLACEHOLDER
@@ -135,6 +138,7 @@ describe("protectNewlinesInQuotes", () => {
 		})
 	})
 })
+// kilocode_change end
 
 describe("Command Validation", () => {
 	describe("parseCommand", () => {
@@ -232,6 +236,7 @@ describe("Command Validation", () => {
 				])
 			})
 
+			// kilocode_change start allowed newlines within quotes
 			it("preserves newlines within quotes", () => {
 				// Newlines inside quoted strings should be preserved as part of the command
 				// Using template literal to create actual newline
@@ -241,6 +246,7 @@ git status`
 				// The newlines inside quotes are preserved, so we get two commands
 				expect(parseCommand(commandWithNewlineInQuotes)).toEqual(['echo "Hello\nWorld"', "git status"])
 			})
+			// kilocode_change end
 
 			it("handles quoted strings on single line", () => {
 				// When quotes are on the same line, they are preserved
@@ -1204,6 +1210,7 @@ describe("Unified Command Decision Functions", () => {
 			expect(getCommandDecision("dangerous", allowed, denied)).toBe("ask_user")
 			expect(getCommandDecision("npm install && dangerous", allowed, denied)).toBe("ask_user")
 		})
+		// kilocode_change start
 		// Real-world regression: multi-line git commit message in quotes should be treated as a single command
 		describe("real-world: multi-line git commit message", () => {
 			it("auto-approves when commit message is single-line", () => {
@@ -1240,6 +1247,7 @@ describe("Unified Command Decision Functions", () => {
 				expect(parsed[0]).toContain("- point b")
 			})
 		})
+		// kilocode_change end
 	})
 
 	describe("CommandValidator Integration Tests", () => {

webview-ui/src/utils/command-validation.ts

@@ -2,6 +2,7 @@ import { parse } from "shell-quote"
 
 type ShellToken = string | { op: string } | { command: string }
 
+// kilocode_change start
 /**
  * Placeholders used to protect newlines within quoted strings during command parsing.
  * These constants are used by the protectNewlinesInQuotes function to temporarily replace
@@ -10,6 +11,7 @@ type ShellToken = string | { op: string } | { command: string }
  */
 export const NEWLINE_PLACEHOLDER = "___NEWLINE___"
 export const CARRIAGE_RETURN_PLACEHOLDER = "___CARRIAGE_RETURN___"
+// kilocode_change end
 
 /**
  * # Command Denylist Feature - Longest Prefix Match Strategy
@@ -131,6 +133,7 @@ export function containsDangerousSubstitution(source: string): boolean {
 	)
 }
 
+// kilocode_change start
 /**
  * Protect newlines inside quoted strings by replacing them with placeholders.
  * This handles proper shell quoting rules where quotes can be concatenated.
@@ -223,7 +226,9 @@ export function protectNewlinesInQuotes(
 
 	return result
 }
+// kilocode_change end
 
+// kilocode_change start added exception for quoted newlines
 /**
  * Split a command string into individual sub-commands by
  * chaining operators (&&, ||, ;, |, or &) and newlines.
@@ -235,16 +240,19 @@ export function protectNewlinesInQuotes(
  * - Chain operators (&&, ||, ;, |, &)
  * - Newlines as command separators (but not within quotes)
  */
+//kilocode_change end
 export function parseCommand(command: string): string[] {
 	if (!command?.trim()) return []
 
+	// kilocode_change start
 	// First, protect newlines inside quoted strings by replacing them with placeholders
 	// This prevents splitting multi-line quoted strings (e.g., git commit -m "multi\nline")
 	const protectedCommand = protectNewlinesInQuotes(command, NEWLINE_PLACEHOLDER, CARRIAGE_RETURN_PLACEHOLDER)
 
 	// Split by newlines (handle different line ending formats)
 	// This regex splits on \r\n (Windows), \n (Unix), or \r (old Mac)
 	const lines = protectedCommand.split(/\r\n|\r|\n/)
+	// kilocode_change end
 	const allCommands: string[] = []
 
 	for (const line of lines) {
@@ -256,12 +264,14 @@ export function parseCommand(command: string): string[] {
 		allCommands.push(...lineCommands)
 	}
 
+	// kilocode_change start
 	// Restore newlines and carriage returns in quoted strings
 	return allCommands.map((cmd) =>
 		cmd
 			.replace(new RegExp(NEWLINE_PLACEHOLDER, "g"), "\n")
 			.replace(new RegExp(CARRIAGE_RETURN_PLACEHOLDER, "g"), "\r"),
 	)
+	// kilocode_change end
 }
 
 /**