@typespec/emitter-framework 0.11.0-dev.5 → 0.11.0-dev.7

package/dist/src/csharp/components/class/declaration.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"declaration.d.ts","sourceRoot":"","sources":["../../../../../src/csharp/components/class/declaration.tsx"],"names":[],"mappings":"AAAA,OAAO,EAAO,KAAK,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AACpD,OAAO,KAAK,EAAE,MAAM,kBAAkB,CAAC;AAEvC,OAAO,EAAc,KAAK,SAAS,EAAE,KAAK,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAO5E,MAAM,WAAW,qBAAsB,SAAQ,IAAI,CAAC,EAAE,CAAC,qBAAqB,EAAE,MAAM,CAAC;IACnF,iFAAiF;IACjF,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,wCAAwC;IACxC,IAAI,EAAE,KAAK,GAAG,SAAS,CAAC;IACxB,8FAA8F;IAC9F,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAYD,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,qBAAqB,GAAG,QAAQ,CAuBvE"}
+{"version":3,"file":"declaration.d.ts","sourceRoot":"","sources":["../../../../../src/csharp/components/class/declaration.tsx"],"names":[],"mappings":"AAAA,OAAO,EAAO,KAAK,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AACpD,OAAO,KAAK,EAAE,MAAM,kBAAkB,CAAC;AAEvC,OAAO,EAAc,KAAK,SAAS,EAAE,KAAK,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAO5E,MAAM,WAAW,qBAAsB,SAAQ,IAAI,CAAC,EAAE,CAAC,qBAAqB,EAAE,MAAM,CAAC;IACnF,iFAAiF;IACjF,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,wCAAwC;IACxC,IAAI,EAAE,KAAK,GAAG,SAAS,CAAC;IACxB,8FAA8F;IAC9F,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAYD,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,qBAAqB,GAAG,QAAQ,CA6BvE"}

package/dist/src/csharp/components/class/declaration.js

@@ -19,6 +19,13 @@ export function ClassDeclaration(props) {
   return [_$createComponent(cs.ClassDeclaration, _$mergeProps(props, {
     name: className,
     refkey: refkeys,
+    get baseType() {
+      return props.baseType ?? (props.type.kind === "Model" && props.type.baseModel ? _$createComponent(TypeExpression, {
+        get type() {
+          return props.type.baseModel;
+        }
+      }) : undefined);
+    },
     get doc() {
       return getDocComments($, props.type);
     },

package/dist/src/csharp/components/property/property.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"property.d.ts","sourceRoot":"","sources":["../../../../../src/csharp/components/property/property.tsx"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAG/C,OAAO,EAAE,KAAK,aAAa,EAAiC,MAAM,oBAAoB,CAAC;AAKvF,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,aAAa,CAAC;IACpB,8FAA8F;IAC9F,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAED;;GAEG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,aAAa,GAAG,QAAQ,CAiBvD;AAED,MAAM,WAAW,sBAAsB;IACrC,IAAI,EAAE,aAAa,CAAC;CACrB"}
+{"version":3,"file":"property.d.ts","sourceRoot":"","sources":["../../../../../src/csharp/components/property/property.tsx"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAG/C,OAAO,EAAe,KAAK,aAAa,EAAiC,MAAM,oBAAoB,CAAC;AAMpG,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,aAAa,CAAC;IACpB,8FAA8F;IAC9F,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AAED;;GAEG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,aAAa,GAAG,QAAQ,CAkDvD;AAED,MAAM,WAAW,sBAAsB;IACrC,IAAI,EAAE,aAAa,CAAC;CACrB"}

package/dist/src/csharp/components/property/property.js

@@ -1,10 +1,11 @@
 import { memo as _$memo, createComponent as _$createComponent } from "@alloy-js/core/jsx-runtime";
 import * as cs from "@alloy-js/csharp";
 import { Attribute } from "@alloy-js/csharp";
-import { resolveEncodedName } from "@typespec/compiler";
+import { getProperty, resolveEncodedName } from "@typespec/compiler";
 import { useTsp } from "../../../core/index.js";
 import { TypeExpression } from "../type-expression.js";
 import { getDocComments } from "../utils/doc-comments.js";
+import { getNullableUnionInnerType } from "../utils/nullable-util.js";
 /**
  * Create a C# property declaration from a TypeSpec property type.
  */
@@ -13,6 +14,31 @@ export function Property(props) {
   const {
     $
   } = useTsp();
+  let overrideType = "";
+  let isVirtual = false;
+  if (props.type.model) {
+    if (props.type.model.baseModel) {
+      const base = props.type.model.baseModel;
+      const baseProperty = getProperty(base, props.type.name);
+      if (baseProperty) {
+        const baseResult = preprocessPropertyType(baseProperty);
+        if (baseResult.nullable === result.nullable && baseResult.type === result.type) {
+          overrideType = "override";
+        } else {
+          overrideType = "new";
+        }
+      }
+    }
+    if (overrideType === "" && props.type.model.derivedModels && props.type.model.derivedModels.length > 0) {
+      isVirtual = props.type.model.derivedModels.some(derived => {
+        const derivedProperty = derived.properties.get(props.type.name);
+        if (derivedProperty) {
+          const derivedResult = preprocessPropertyType(derivedProperty);
+          return derivedResult.nullable === result.nullable && derivedResult.type === result.type;
+        }
+      });
+    }
+  }
   return _$createComponent(cs.Property, {
     get name() {
       return props.type.name;
@@ -24,7 +50,10 @@ export function Property(props) {
         }
       });
     },
+    override: overrideType === "override",
+    "new": overrideType === "new",
     "public": true,
+    virtual: isVirtual,
     get required() {
       return !props.type.optional;
     },
@@ -59,34 +88,23 @@ function JsonNameAttribute(props) {
 }
 function preprocessPropertyType(prop) {
   const type = prop.type;
-  if (prop.optional) {
-    return {
-      type,
-      nullable: true
-    };
-  }
-  const {
-    $
-  } = useTsp();
   if (type.kind === "Union") {
-    const variants = type.variants;
-    const nonNullVariant = [...variants.values()].find(v => v.type !== $.intrinsic.null);
-    const nullVariant = [...variants.values()].find(v => v.type !== $.intrinsic.null);
-    if (nonNullVariant && nullVariant && variants.size === 2) {
+    const innerType = getNullableUnionInnerType(type);
+    if (innerType) {
       return {
-        type: nonNullVariant.type,
+        type: innerType,
         nullable: true
       };
     } else {
       return {
         type,
-        nullable: false
+        nullable: prop.optional
       };
     }
   } else {
     return {
       type,
-      nullable: false
+      nullable: prop.optional
     };
   }
 }

package/dist/src/csharp/components/property/property.test.js

@@ -1,5 +1,6 @@
 import { createComponent as _$createComponent } from "@alloy-js/core/jsx-runtime";
 import { Tester } from "#test/test-host.js";
+import { List } from "@alloy-js/core";
 import { ClassDeclaration, createCSharpNamePolicy, Namespace, SourceFile } from "@alloy-js/csharp";
 import { t } from "@typespec/compiler/testing";
 import { beforeEach, describe, expect, it } from "vitest";
@@ -84,6 +85,30 @@ it("maps optional properties to nullable properties", async () => {
       }
   `);
 });
+it("maps optional and nullable properties to nullable properties", async () => {
+  const {
+    prop1
+  } = await tester.compile(t.code`
+    model TestModel {
+      ${t.modelProperty("prop1")}?: string | null;
+    }
+  `);
+  expect(_$createComponent(Wrapper, {
+    get children() {
+      return _$createComponent(Property, {
+        type: prop1
+      });
+    }
+  })).toRenderTo(`
+      namespace TestNamespace
+      {
+          class Test
+          {
+              public string? Prop1 { get; set; }
+          }
+      }
+  `);
+});
 describe("jsonAttributes", () => {
   it("adds [JsonNameAttribute]", async () => {
     const {
@@ -138,4 +163,80 @@ describe("jsonAttributes", () => {
       }
   `);
   });
+  it("inherit prop: override, new", async () => {
+    const r = await tester.compile(t.code`
+    model TestModel extends BaseModel {
+      ${t.modelProperty("prop1")}: string;
+      ${t.modelProperty("prop2")}: string | null;
+    }
+    model BaseModel {
+      prop1: string | null;
+      prop2: string | null;
+    }
+  `);
+    expect(_$createComponent(Wrapper, {
+      get children() {
+        return _$createComponent(List, {
+          get children() {
+            return [_$createComponent(Property, {
+              get type() {
+                return r.prop1;
+              }
+            }), _$createComponent(Property, {
+              get type() {
+                return r.prop2;
+              }
+            })];
+          }
+        });
+      }
+    })).toRenderTo(`
+      namespace TestNamespace
+      {
+          class Test
+          {
+              public new required string Prop1 { get; set; }
+              public override required string? Prop2 { get; set; }
+          }
+      }
+  `);
+  });
+  it("inherit prop: virtual", async () => {
+    const r = await tester.compile(t.code`
+    model TestModel extends BaseModel {
+      prop1: string;
+      prop2: string | null;
+    }
+    model BaseModel {
+      ${t.modelProperty("prop1")}: string | null;
+      ${t.modelProperty("prop2")}: string | null;
+    }
+  `);
+    expect(_$createComponent(Wrapper, {
+      get children() {
+        return _$createComponent(List, {
+          get children() {
+            return [_$createComponent(Property, {
+              get type() {
+                return r.prop1;
+              }
+            }), _$createComponent(Property, {
+              get type() {
+                return r.prop2;
+              }
+            })];
+          }
+        });
+      }
+    })).toRenderTo(`
+      namespace TestNamespace
+      {
+          class Test
+          {
+              public required string? Prop1 { get; set; }
+              public virtual required string? Prop2 { get; set; }
+          }
+      }
+  `);
+  });
 });

package/dist/src/csharp/components/type-expression.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"type-expression.d.ts","sourceRoot":"","sources":["../../../../src/csharp/components/type-expression.tsx"],"names":[],"mappings":"AACA,OAAO,EAAQ,KAAK,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAErD,OAAO,EAIL,KAAK,aAAa,EAClB,KAAK,MAAM,EACX,KAAK,IAAI,EAEV,MAAM,oBAAoB,CAAC;AAC5B,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,4BAA4B,CAAC;AAK1D,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,IAAI,CAAC;CACZ;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,mBAAmB,GAAG,QAAQ,CAmCnE;AAwBD,QAAA,MAAM,yBAAyB,4BAqC7B,CAAC;AAEH,wBAAgB,4BAA4B,CAC1C,CAAC,EAAE,OAAO,EACV,IAAI,EAAE,MAAM,GAAG,aAAa,GAC3B,MAAM,GAAG,IAAI,CAoBf;AAyBD,OAAO,EAAE,yBAAyB,EAAE,CAAC"}
+{"version":3,"file":"type-expression.d.ts","sourceRoot":"","sources":["../../../../src/csharp/components/type-expression.tsx"],"names":[],"mappings":"AACA,OAAO,EAAQ,KAAK,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAErD,OAAO,EAGL,KAAK,aAAa,EAClB,KAAK,MAAM,EACX,KAAK,IAAI,EACV,MAAM,oBAAoB,CAAC;AAC5B,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,4BAA4B,CAAC;AAM1D,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,IAAI,CAAC;CACZ;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,mBAAmB,GAAG,QAAQ,CAmCnE;AAED,QAAA,MAAM,yBAAyB,4BAqC7B,CAAC;AAEH,wBAAgB,4BAA4B,CAC1C,CAAC,EAAE,OAAO,EACV,IAAI,EAAE,MAAM,GAAG,aAAa,GAC3B,MAAM,GAAG,IAAI,CAoBf;AAyBD,OAAO,EAAE,yBAAyB,EAAE,CAAC"}

package/dist/src/csharp/components/type-expression.js

@@ -2,9 +2,10 @@ import { createComponent as _$createComponent } from "@alloy-js/core/jsx-runtime
 import { Experimental_OverridableComponent } from "#core/index.js";
 import { code } from "@alloy-js/core";
 import { Reference } from "@alloy-js/csharp";
-import { getTypeName, isNullType, isVoidType } from "@typespec/compiler";
+import { getTypeName, isVoidType } from "@typespec/compiler";
 import { useTsp } from "../../core/index.js";
 import { reportTypescriptDiagnostic } from "../../typescript/lib.js";
+import { getNullableUnionInnerType } from "./utils/nullable-util.js";
 import { efRefkey } from "./utils/refkey.js";
 export function TypeExpression(props) {
   if (props.type.kind === "Union") {
@@ -59,29 +60,6 @@ export function TypeExpression(props) {
   }
   throw new Error(`Unsupported type for TypeExpression: ${props.type.kind} (${getTypeName(props.type)})`);
 }
-
-/** Get the inner type if the union is a nullable, otherwise return undefined */
-function getNullableUnionInnerType(u) {
-  const isNull = type => isNullType(type) || isVoidType(type);
-  if (Array.from(u.variants.values()).some(v => isNull(v.type))) {
-    const {
-      $
-    } = useTsp();
-    const left = Array.from(u.variants.values()).filter(v => !isNull(v.type));
-    if (left.length === 0) {
-      // a union only has null or void?
-      return $.intrinsic.void;
-    } else if (left.length === 1) {
-      return left[0].type;
-    } else {
-      return $.union.create({
-        name: u.name,
-        variants: left
-      });
-    }
-  }
-  return undefined;
-}
 const intrinsicNameToCSharpType = new Map([
 // Core types
 ["unknown", "object"],

package/dist/src/csharp/components/utils/nullable-util.d.ts

@@ -0,0 +1,4 @@
+import type { Type, Union } from "@typespec/compiler";
+/** Get the inner type if the union is a nullable, otherwise return undefined */
+export declare function getNullableUnionInnerType(u: Union): Type | undefined;
+//# sourceMappingURL=nullable-util.d.ts.map

package/dist/src/csharp/components/utils/nullable-util.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"nullable-util.d.ts","sourceRoot":"","sources":["../../../../../src/csharp/components/utils/nullable-util.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAEtD,gFAAgF;AAChF,wBAAgB,yBAAyB,CAAC,CAAC,EAAE,KAAK,GAAG,IAAI,GAAG,SAAS,CAoBpE"}

package/dist/src/csharp/components/utils/nullable-util.js

@@ -0,0 +1,26 @@
+import { useTsp } from "#core/index.js";
+/** Get the inner type if the union is a nullable, otherwise return undefined */
+export function getNullableUnionInnerType(u) {
+  const {
+    $
+  } = useTsp();
+  const isNull = type => type === $.intrinsic.null || type === $.intrinsic.void;
+  if (Array.from(u.variants.values()).some(v => isNull(v.type))) {
+    const {
+      $
+    } = useTsp();
+    const left = Array.from(u.variants.values()).filter(v => !isNull(v.type));
+    if (left.length === 0) {
+      // a union only has null or void?
+      return $.intrinsic.void;
+    } else if (left.length === 1) {
+      return left[0].type;
+    } else {
+      return $.union.create({
+        name: u.name,
+        variants: left
+      });
+    }
+  }
+  return undefined;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typespec/emitter-framework",
-  "version": "0.11.0-dev.5",
+  "version": "0.11.0-dev.7",
   "type": "module",
   "main": "dist/index.js",
   "repository": {

package/src/csharp/components/class/declaration.tsx

@@ -41,6 +41,12 @@ export function ClassDeclaration(props: ClassDeclarationProps): Children {
         {...props}
         name={className}
         refkey={refkeys}
+        baseType={
+          props.baseType ??
+          (props.type.kind === "Model" && props.type.baseModel ? (
+            <TypeExpression type={props.type.baseModel} />
+          ) : undefined)
+        }
         doc={getDocComments($, props.type)}
       >
         {props.type.kind === "Model" && (

package/src/csharp/components/property/property.test.tsx

@@ -1,5 +1,5 @@
 import { Tester } from "#test/test-host.js";
-import { type Children } from "@alloy-js/core";
+import { List, type Children } from "@alloy-js/core";
 import { ClassDeclaration, createCSharpNamePolicy, Namespace, SourceFile } from "@alloy-js/csharp";
 import { t, type TesterInstance } from "@typespec/compiler/testing";
 import { beforeEach, describe, expect, it } from "vitest";
@@ -69,6 +69,28 @@ it("maps optional properties to nullable properties", async () => {
   `);
 });
 
+it("maps optional and nullable properties to nullable properties", async () => {
+  const { prop1 } = await tester.compile(t.code`
+    model TestModel {
+      ${t.modelProperty("prop1")}?: string | null;
+    }
+  `);
+
+  expect(
+    <Wrapper>
+      <Property type={prop1} />
+    </Wrapper>,
+  ).toRenderTo(`
+      namespace TestNamespace
+      {
+          class Test
+          {
+              public string? Prop1 { get; set; }
+          }
+      }
+  `);
+});
+
 describe("jsonAttributes", () => {
   it("adds [JsonNameAttribute]", async () => {
     const { prop1 } = await tester.compile(t.code`
@@ -116,4 +138,66 @@ describe("jsonAttributes", () => {
       }
   `);
   });
+
+  it("inherit prop: override, new", async () => {
+    const r = await tester.compile(t.code`
+    model TestModel extends BaseModel {
+      ${t.modelProperty("prop1")}: string;
+      ${t.modelProperty("prop2")}: string | null;
+    }
+    model BaseModel {
+      prop1: string | null;
+      prop2: string | null;
+    }
+  `);
+
+    expect(
+      <Wrapper>
+        <List>
+          <Property type={r.prop1} />
+          <Property type={r.prop2} />
+        </List>
+      </Wrapper>,
+    ).toRenderTo(`
+      namespace TestNamespace
+      {
+          class Test
+          {
+              public new required string Prop1 { get; set; }
+              public override required string? Prop2 { get; set; }
+          }
+      }
+  `);
+  });
+
+  it("inherit prop: virtual", async () => {
+    const r = await tester.compile(t.code`
+    model TestModel extends BaseModel {
+      prop1: string;
+      prop2: string | null;
+    }
+    model BaseModel {
+      ${t.modelProperty("prop1")}: string | null;
+      ${t.modelProperty("prop2")}: string | null;
+    }
+  `);
+
+    expect(
+      <Wrapper>
+        <List>
+          <Property type={r.prop1} />
+          <Property type={r.prop2} />
+        </List>
+      </Wrapper>,
+    ).toRenderTo(`
+      namespace TestNamespace
+      {
+          class Test
+          {
+              public required string? Prop1 { get; set; }
+              public virtual required string? Prop2 { get; set; }
+          }
+      }
+  `);
+  });
 });

package/src/csharp/components/property/property.tsx

@@ -1,10 +1,11 @@
 import { type Children } from "@alloy-js/core";
 import * as cs from "@alloy-js/csharp";
 import { Attribute } from "@alloy-js/csharp";
-import { type ModelProperty, resolveEncodedName, type Type } from "@typespec/compiler";
+import { getProperty, type ModelProperty, resolveEncodedName, type Type } from "@typespec/compiler";
 import { useTsp } from "../../../core/index.js";
 import { TypeExpression } from "../type-expression.jsx";
 import { getDocComments } from "../utils/doc-comments.jsx";
+import { getNullableUnionInnerType } from "../utils/nullable-util.js";
 
 export interface PropertyProps {
   type: ModelProperty;
@@ -19,11 +20,44 @@ export function Property(props: PropertyProps): Children {
   const result = preprocessPropertyType(props.type);
   const { $ } = useTsp();
 
+  let overrideType: "" | "override" | "new" = "";
+  let isVirtual = false;
+  if (props.type.model) {
+    if (props.type.model.baseModel) {
+      const base = props.type.model.baseModel;
+      const baseProperty = getProperty(base, props.type.name);
+      if (baseProperty) {
+        const baseResult = preprocessPropertyType(baseProperty);
+        if (baseResult.nullable === result.nullable && baseResult.type === result.type) {
+          overrideType = "override";
+        } else {
+          overrideType = "new";
+        }
+      }
+    }
+    if (
+      overrideType === "" &&
+      props.type.model.derivedModels &&
+      props.type.model.derivedModels.length > 0
+    ) {
+      isVirtual = props.type.model.derivedModels.some((derived) => {
+        const derivedProperty = derived.properties.get(props.type.name);
+        if (derivedProperty) {
+          const derivedResult = preprocessPropertyType(derivedProperty);
+          return derivedResult.nullable === result.nullable && derivedResult.type === result.type;
+        }
+      });
+    }
+  }
+
   return (
     <cs.Property
       name={props.type.name}
       type={<TypeExpression type={result.type} />}
+      override={overrideType === "override"}
+      new={overrideType === "new"}
       public
+      virtual={isVirtual}
       required={!props.type.optional}
       nullable={result.nullable}
       doc={getDocComments($, props.type)}
@@ -47,22 +81,14 @@ function JsonNameAttribute(props: JsonNameAttributeProps): Children {
 function preprocessPropertyType(prop: ModelProperty): { type: Type; nullable: boolean } {
   const type = prop.type;
 
-  if (prop.optional) {
-    return { type, nullable: true };
-  }
-
-  const { $ } = useTsp();
-
   if (type.kind === "Union") {
-    const variants = type.variants;
-    const nonNullVariant = [...variants.values()].find((v) => v.type !== $.intrinsic.null);
-    const nullVariant = [...variants.values()].find((v) => v.type !== $.intrinsic.null);
-    if (nonNullVariant && nullVariant && variants.size === 2) {
-      return { type: nonNullVariant.type, nullable: true };
+    const innerType = getNullableUnionInnerType(type);
+    if (innerType) {
+      return { type: innerType, nullable: true };
     } else {
-      return { type, nullable: false };
+      return { type, nullable: prop.optional };
     }
   } else {
-    return { type, nullable: false };
+    return { type, nullable: prop.optional };
   }
 }

package/src/csharp/components/type-expression.tsx

@@ -3,16 +3,15 @@ import { code, type Children } from "@alloy-js/core";
 import { Reference } from "@alloy-js/csharp";
 import {
   getTypeName,
-  isNullType,
   isVoidType,
   type IntrinsicType,
   type Scalar,
   type Type,
-  type Union,
 } from "@typespec/compiler";
 import type { Typekit } from "@typespec/compiler/typekit";
 import { useTsp } from "../../core/index.js";
 import { reportTypescriptDiagnostic } from "../../typescript/lib.js";
+import { getNullableUnionInnerType } from "./utils/nullable-util.js";
 import { efRefkey } from "./utils/refkey.js";
 
 export interface TypeExpressionProps {
@@ -56,28 +55,6 @@ export function TypeExpression(props: TypeExpressionProps): Children {
   );
 }
 
-/** Get the inner type if the union is a nullable, otherwise return undefined */
-function getNullableUnionInnerType(u: Union): Type | undefined {
-  const isNull = (type: Type) => isNullType(type) || isVoidType(type);
-
-  if (Array.from(u.variants.values()).some((v) => isNull(v.type))) {
-    const { $ } = useTsp();
-    const left = Array.from(u.variants.values()).filter((v) => !isNull(v.type));
-    if (left.length === 0) {
-      // a union only has null or void?
-      return $.intrinsic.void;
-    } else if (left.length === 1) {
-      return left[0].type;
-    } else {
-      return $.union.create({
-        name: u.name,
-        variants: left,
-      });
-    }
-  }
-  return undefined;
-}
-
 const intrinsicNameToCSharpType = new Map<string, string | null>([
   // Core types
   ["unknown", "object"], // Matches C#'s `object`

package/src/csharp/components/utils/nullable-util.ts

@@ -0,0 +1,25 @@
+import { useTsp } from "#core/index.js";
+import type { Type, Union } from "@typespec/compiler";
+
+/** Get the inner type if the union is a nullable, otherwise return undefined */
+export function getNullableUnionInnerType(u: Union): Type | undefined {
+  const { $ } = useTsp();
+  const isNull = (type: Type) => type === $.intrinsic.null || type === $.intrinsic.void;
+
+  if (Array.from(u.variants.values()).some((v) => isNull(v.type))) {
+    const { $ } = useTsp();
+    const left = Array.from(u.variants.values()).filter((v) => !isNull(v.type));
+    if (left.length === 0) {
+      // a union only has null or void?
+      return $.intrinsic.void;
+    } else if (left.length === 1) {
+      return left[0].type;
+    } else {
+      return $.union.create({
+        name: u.name,
+        variants: left,
+      });
+    }
+  }
+  return undefined;
+}