Variable override does not work

[legal90]

Bug Report

1. Minimal reproduce step (Required)

Example from the official doc:
https://www.kcl-lang.io/docs/reference/lang/tour#kcl-cli-variable-override

schema Person:
    name: str
    age: int
    ids?: [int]

person = Person {
    name = "Alice"
    age = 10
}

2. What did you expect to see? (Required)

With a string and integer values

kcl main.k -O :person.name=\"Bob\" -O :person.age=10

Expected Output:

person:
  name: Bob
  age: 10

With a list value

kcl main.k -O :person.ids=\[1,2\]

Expected Output:

person:
  name: Alice
  age: 10
  ids:
    - 1
    - 2

3. What did you see instead (Required)

With a string value

kcl main.k -O :person.name=\"Bob\" -O :person.age=10

Received Output:

Error: invalid argument ":person.name=\"Bob\"" for "-O, --overrides" flag: parse error on line 1, column 14: bare " in non-quoted-field

With a list value

kcl main.k -O :person.ids=\[1,2\]

Received Output:

Error: Invalid spec format ':person.ids=[1', expected <field_path>=filed_value>, <field_path>:filed_value>, <field_path>+=filed_value> or <field_path>-

4. What is your KCL components version? (Required)

$ kcl --version
kcl version v0.11.1

[legal90]

Without leading : in the dentifier - it works for integer:

$ kcl main.k -O person.age=10
person:
  name: Alice
  age: 10

But still does not work for strings:

$ kcl main.k -O 'person.name=\"Bob\"'
Error: invalid argument "person.name=\\\"Bob\\\"" for "-O, --overrides" flag: parse error on line 1, column 14: bare " in non-quoted-field

$ kcl main.k -O person.name=\"Bob\"
Error: invalid argument "person.name=\"Bob\"" for "-O, --overrides" flag: parse error on line 1, column 13: bare " in non-quoted-field

$ kcl main.k -O person.name="Bob"
Error: 
error[E2L23]: CompileError
 --> /private/tmp/main.k:7:5
  |
7 |     name = "Alice"
  |     ^ name 'Bob' is not defined
  |

[Peefy]

Seems that the , in the CLI flag causes this issue, I will try to fix it.

[Peefy]

For the string value, we can use the form like person.name=\'Bob\'

[Peefy]

The document is also wrong, and we can remove the leading : in the override spec.

[legal90]

Thank you, @Peefy! I confirm that person.name=\'Bob\' works as expected:

$ kcl . -O person.name=\'Bob\'
person:
  name: Bob
  age: 10

Side question:
Is the argument -d still supposed to be used for the in-place replacement of the variable in the file, like the doc says?

Besides, when we use KCL CLI -d argument, the KCL file will be modified to the following content at the same time

I find it a bit confusing, because for all other cases (including kcl run) -d used as a toggle for a debug mode.