Fix review findings: document implements clause behavior, remove unused imports

- Add explanatory comment on getComposition/iface.name in object-type.tsx
  clarifying that pre-mutation names match post-mutation names (mutation
  engine doesn't rename models)
- Remove unused `type Model` import from all G2 test files

Author: FionaBronwen

packages/graphql/src/components/types/input-type.tsx

@@ -1,4 +1,4 @@
-import { type Model, getDoc } from "@typespec/compiler";
+import type { Model } from "@typespec/compiler";
 import * as gql from "@alloy-js/graphql";
 import { useTsp } from "@typespec/emitter-framework";
 import { useGraphQLSchema } from "../../context/index.js";
@@ -17,9 +17,9 @@ export interface InputTypeProps {
  * - Otherwise, uses the name as-is
  */
 export function InputType(props: InputTypeProps) {
-  const { program } = useTsp();
+  const { $ } = useTsp();
   const { modelVariants } = useGraphQLSchema();
-  const doc = getDoc(program, props.type);
+  const doc = $.type.getDoc(props.type);
   const properties = Array.from(props.type.properties.values());
 
   // If there's an output variant with the same name, add Input suffix

packages/graphql/src/components/types/interface-type.tsx

@@ -1,4 +1,4 @@
-import { type Model, getDoc } from "@typespec/compiler";
+import type { Model } from "@typespec/compiler";
 import * as gql from "@alloy-js/graphql";
 import { useTsp } from "@typespec/emitter-framework";
 import { Field } from "../fields/index.js";
@@ -14,8 +14,8 @@ export interface InterfaceTypeProps {
  * Interfaces are marked with @Interface decorator in TypeSpec
  */
 export function InterfaceType(props: InterfaceTypeProps) {
-  const { program } = useTsp();
-  const doc = getDoc(program, props.type);
+  const { $ } = useTsp();
+  const doc = $.type.getDoc(props.type);
   const properties = Array.from(props.type.properties.values());
 
   return (

packages/graphql/src/components/types/object-type.tsx

@@ -1,4 +1,4 @@
-import { type Model, getDoc } from "@typespec/compiler";
+import type { Model } from "@typespec/compiler";
 import * as gql from "@alloy-js/graphql";
 import { useTsp } from "@typespec/emitter-framework";
 import { Field, OperationField } from "../fields/index.js";
@@ -19,13 +19,16 @@ export interface ObjectTypeProps {
  * - Operation fields via @operationFields
  */
 export function ObjectType(props: ObjectTypeProps) {
-  const { program } = useTsp();
-  const doc = getDoc(program, props.type);
+  const { $, program } = useTsp();
+  const doc = $.type.getDoc(props.type);
   const properties = Array.from(props.type.properties.values());
   const implementations = getComposition(program, props.type);
   const operationFields = getOperationFields(program, props.type);
 
-  // Convert interface implementations to string references
+  // Convert interface implementations to string references.
+  // Note: getComposition returns original (pre-mutation) models from decorator state.
+  // This works because the mutation engine doesn't rename models, so iface.name
+  // matches the name used by InterfaceType for the same model.
   const implementsRefs = implementations?.map((iface) => iface.name) || [];
 
   return (

packages/graphql/test/components/input-type.test.tsx

@@ -1,4 +1,3 @@
-import { type Model } from "@typespec/compiler";
 import { t } from "@typespec/compiler/testing";
 import { describe, expect, it, beforeEach } from "vitest";
 import { InputType } from "../../src/components/types/index.js";

packages/graphql/test/components/interface-type.test.tsx

@@ -1,4 +1,3 @@
-import { type Model } from "@typespec/compiler";
 import { t } from "@typespec/compiler/testing";
 import { describe, expect, it, beforeEach } from "vitest";
 import { InterfaceType } from "../../src/components/types/index.js";

packages/graphql/test/components/object-type.test.tsx

@@ -1,4 +1,3 @@
-import { type Model } from "@typespec/compiler";
 import { t } from "@typespec/compiler/testing";
 import * as gql from "@alloy-js/graphql";
 import { describe, expect, it, beforeEach } from "vitest";