Should typespec error if a model used for merge patch includes a nullable property

[m-nash]

In the below scenario since the model is used for merge patch it should include null in the possible range of values for any of its properties.

model Foo {
  name?: string
  description?: string | null
}

op Update(@header content-type: "application/merge-patch+json", ...Foo) : void

[bterlson]

I believe you mean should not include null? E.g. that you have an example of an error case, here? If so, I agree, although I could imagine ignoring/suppressing the error if you don't particularly care that you can't update a field's value to null for whatever reason.

[markcowl]

• add a warning diagnostic in http lib when this occurs (linting rule)

[markcowl]

est: 3