Questions on optional schema parameter

[mihaigalos]

Bug Report

1. Minimal reproduce step (Required)

Tried to run the example from the docs.

schema Person:
    gender: "Male" | "Female"
    name: Name

schema Name:
    first: str
    middle?: str  # Optional, but must be non-empty when specified
    last: str

    check:
        first != ""
        last != ""
        middle != "" if middle

alice = Person {
    gender: "Female"
    name.first: "Alice"
    name.last: "White"
    name.middle: ""
}

Ran with kcl person.k.

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

Error, since middle name is empty.

3. What did you see instead (Required)

alice:
  gender: Female
  name:
    first: Alice
    last: White
    middle: ''     <-- Expected an error on this line.

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

I don't have the folder required for this step on my machine (Kusion/kclvm-go/KCLVM), since built the binaries from source (commit 82c2be0 from March 8th).

Playground link.

[Peefy]

@mihaigalos. Hello, thank you for your question. In KCL, if expressions will filter all expressions that are not true (False, empty string, empty list, etc.), just like Python and Javascript. Ref KCL FAQ: https://kcl-lang.io/docs/user_docs/support/faq-kcl#11-how-to-judge-whether-a-variable-is-noneundefined-and-whether-a-stringdictlist-is-empty

[Peefy]

@mihaigalos. Hello, thank you for your question. In KCL, if expressions will filter all expressions that are not true (False, empty string, empty list, etc.), just like Python and Javascript. Ref KCL FAQ: https://kcl-lang.io/docs/user_docs/support/faq-kcl#11-how-to-judge-whether-a-variable-is-noneundefined-and-whether-a-stringdictlist-is-empty

http://39.106.40.108/-/play/index.html?id=41a726771bb5ba45a5a947e9c85dff4f5694e210 I wrote another example to show the difference.

[mihaigalos]

Awesome, thank you. The docs are a bit misleading with this example.

[Peefy]

Thank you too. I have revised the example.

[Peefy]

I will transfer it to discussion