fix(tree): Expand API-Extractor workaround to fix rollups, and link t…

…o related github issue (microsoft#19465)

Adds additional workaround for existing issue, and links to
microsoft/rushstack#4507

packages/dds/tree/api-report/tree.api.md

@@ -1654,10 +1654,10 @@ export class test_RecursiveObject extends test_RecursiveObject_base {
 
 // @internal
 export const test_RecursiveObject_base: TreeNodeSchemaClass<"Test Recursive Domain.testObject", NodeKind.Object, object & TreeNode & ObjectFromSchemaRecord<    {
-readonly recursive: FieldSchema<import("./schemaTypes.js").FieldKind.Optional, readonly [() => typeof test_RecursiveObject]>;
+readonly recursive: FieldSchema<FieldKind.Optional, readonly [() => typeof test_RecursiveObject]>;
 readonly number: TreeNodeSchema<"com.fluidframework.leaf.number", NodeKind.Leaf, number, number>;
 }> & WithType<"Test Recursive Domain.testObject">, object & InsertableObjectFromSchemaRecord<    {
-readonly recursive: FieldSchema<import("./schemaTypes.js").FieldKind.Optional, readonly [() => typeof test_RecursiveObject]>;
+readonly recursive: FieldSchema<FieldKind.Optional, readonly [() => typeof test_RecursiveObject]>;
 readonly number: TreeNodeSchema<"com.fluidframework.leaf.number", NodeKind.Leaf, number, number>;
 }>, true>;
 

packages/dds/tree/src/domains/json/jsonDomainSchema.ts

@@ -6,7 +6,7 @@
 // Adding this unused import makes the generated d.ts file produced by TypeScript stop breaking API-Extractor's rollup generation.
 // Without this import, TypeScript generates inline `import("../..")` statements in the d.ts file,
 // which API-Extractor leaves as is when generating the rollup, leaving them pointing at the wrong directory.
-// TODO: Understand and/or remove the need for this workaround.
+// API-Extractor issue: https://github.com/microsoft/rushstack/issues/4507
 // eslint-disable-next-line @typescript-eslint/no-unused-vars, unused-imports/no-unused-imports
 import { ValueSchema } from "../../core/index.js";
 

packages/dds/tree/src/simple-tree/testRecursiveDomain.ts

@@ -14,12 +14,9 @@ import { SchemaFactoryRecursive } from "./schemaFactoryRecursive.js";
 // This type is referred to by the base type generated by builder.object.
 // Adding the import of it here prevents TypeScript from generating inline includes for it in the d.ts file.
 // If we let TypeScript generate these includes, they use relative paths which break API extractor's rollup.
-import { type NodeKind as _dummy } from "./schemaTypes.js";
-
-{
-	// Unused imports are an error, so make an unused type to use it.
-	type _X = _dummy;
-}
+// API-Extractor issue: https://github.com/microsoft/rushstack/issues/4507
+// eslint-disable-next-line @typescript-eslint/no-unused-vars, unused-imports/no-unused-imports
+import { type NodeKind, type FieldKind } from "./schemaTypes.js";
 
 const builder = new SchemaFactoryRecursive("Test Recursive Domain");