[Bug]: bytes data type isn't recognized properly

[AnReZa]

Describe the bug

We're using TypeSpec to auto-generate c# classes. However, there is a problem with the auto-generated .Equals method. In the default .mustache file of the openapi generator, there's a check for #isContainer which determins, if a regular == comparison or .SequenceEqual should be used:

/// <summary>
/// Returns true if {{classname}} instances are equal
/// </summary>
/// <param name="other">Instance of {{classname}} to be compared</param>
/// <returns>Boolean</returns>
public bool Equals({{classname}}? other)
{
    if (other is null) return false;
    if (ReferenceEquals(this, other)) return true;

    return {{#vars}}{{^isContainer}}
        (
            {{name}} == other.{{name}} ||
            {{^vendorExtensions.x-is-value-type}}{{name}} != null &&{{/vendorExtensions.x-is-value-type}}
            {{name}}.Equals(other.{{name}})
        ){{^-last}} && {{/-last}}{{/isContainer}}{{#isContainer}}
        (
            {{name}} == other.{{name}} ||
            {{^vendorExtensions.x-is-value-type}}{{name}} != null &&
            other.{{name}} != null &&
            {{/vendorExtensions.x-is-value-type}}{{name}}.SequenceEqual(other.{{name}})
        ){{^-last}} && {{/-last}}{{/isContainer}}{{/vars}}{{^vars}}false{{/vars}};
}

However, contrary to my expectation for the bytes data type, it doesn't use the SequenceEqual comparison, but the regular == one, which doesn't work for byte arrays in c#. Why is that?

bytes gets converted into a type: string, format: byte data type, which seems odd to me. Is that right?

Right now, it correctly generates a property of type byte[] in c#, but the equals method doesn't work, as it uses .Equals instead of the correct SequenceEqual.

Reproduction

model Test {
  test: bytes
}

openapi: 3.0.0
info:
  title: (title)
  version: 0.0.0
tags: []
paths: {}
components:
  schemas:
    Test:
      type: object
      required:
        - test
      properties:
        test:
          type: string
          format: byte

Checklist

• Follow our Code of Conduct
• Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion.
• The provided reproduction is a minimal reproducible example of the bug.

[markcowl]

@AnReZa Is this about the c-sharp client generator, or something else?

[AnReZa]

@markcowl Yes and no. I can't tell, if it is an issue of the conversion step between TypeSpec and OpenAPI, or between OpenAPI and c#.