fix(guards): quote kebab-case object property identifiers

Signed-off-by: Ryan Bower <rbower@qti.qualcomm.com>

Author: R-Bower

packages/common/typedoc/src/guards.ts

@@ -38,6 +38,12 @@ export function isUndefinedType(
   return type?.type === "intrinsic" && type?.name === "undefined"
 }
 
+const identifierName = /^[$A-Z_a-z][$\w]*$/
+
+export function formatObjectPropertyName(name: string) {
+  return identifierName.test(name) ? name : JSON.stringify(name)
+}
+
 export function getTypeFromProperties(
   properties: SerializedType["properties"],
 ) {
@@ -47,7 +53,9 @@ export function getTypeFromProperties(
   return properties
     .filter((property) => property.name && property.type)
     .map((property) => {
-      return `${property.name}${property.required ? "" : "?"}: ${
+      return `${formatObjectPropertyName(property.name)}${
+        property.required ? "" : "?"
+      }: ${
         property.inheritDoc
           ? property.type
           : (property.referencedType ?? property.type)

packages/common/typedoc/tests/src/package-1/types.ts

@@ -17,3 +17,20 @@ export type TestTuple1 = [string, boolean, string, number]
  * Template literal
  */
 export type TestTemplateLiteral1 = `${string}.${number}`
+
+export interface DashCasedBindingProps {
+  "data-disabled": ""
+  "data-stepper-part": "prev-trigger"
+  dir: "ltr" | "rtl"
+  disabled: boolean
+}
+
+export type DashCasedRenderProp<Props> = string | ((props: Props) => string)
+
+/**
+ * @public
+ * @interface
+ */
+export interface DashCasedRenderPropHost {
+  children: DashCasedRenderProp<DashCasedBindingProps>
+}

packages/common/typedoc/tests/types.spec.ts

@@ -62,6 +62,23 @@ function getTestCases(parseResult: ParseResult): TestCase[] {
         expected: "`${string}.${number}`",
       },
     ],
+    [
+      "DashCasedBindingProps",
+      {
+        actual: interfaces.DashCasedBindingProps.resolvedType.type,
+        expected:
+          "{\"data-disabled\": ''; \"data-stepper-part\": 'prev-trigger'; dir: 'ltr' | 'rtl'; disabled: boolean;}",
+      },
+    ],
+    [
+      "DashCasedRenderPropHost.children",
+      {
+        actual: findResolvedType(props.DashCasedRenderPropHost, "children")
+          .type,
+        expected:
+          "string | ((props: {\"data-disabled\": ''; \"data-stepper-part\": 'prev-trigger'; dir: 'ltr' | 'rtl'; disabled: boolean;}) => string)",
+      },
+    ],
     [
       "TestTuple1",
       {