Clean up package hygiene and remove dead code

- Remove unimplemented emitter options (strict, new-line)
- Remove dead GraphQLSchemaRecord type from types.d.ts
- Fix capitalization: GraphqlTestLibrary -> GraphQLTestLibrary
- Consolidate duplicated propertiesEqual into single location
- Remove commented-out import in operation-fields.ts
- Add standard files: api-extractor.json, CHANGELOG.md
- Add tspMain to package.json, fix node version, update lint scripts
- Add peer deps to devDependencies
- Delete empty tspconfig.yaml and unused test/main.tsp

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: FionaBronwen

packages/graphql/CHANGELOG.md

@@ -0,0 +1 @@
+# Change Log - @typespec/graphql

packages/graphql/api-extractor.json

@@ -0,0 +1,4 @@
+{
+  "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
+  "extends": "../../api-extractor.base.json"
+}

packages/graphql/package.json

@@ -17,6 +17,7 @@
     "typespec"
   ],
   "type": "module",
+  "tspMain": "lib/main.tsp",
   "main": "dist/src/index.js",
   "exports": {
     ".": {
@@ -30,10 +31,7 @@
     }
   },
   "engines": {
-    "node": ">=18.0.0"
-  },
-  "graphql": {
-    "documents": "test/**/*.{js,ts}"
+    "node": ">=20.0.0"
   },
   "dependencies": {
     "@alloy-js/core": "^0.11.0",
@@ -45,9 +43,10 @@
     "build": "tsc -p .",
     "watch": "tsc --watch",
     "test": "vitest run",
+    "test:ci": "vitest run --coverage --reporter=junit --reporter=default",
     "test:watch": "vitest -w",
-    "lint": "eslint src/ test/ --report-unused-disable-directives --max-warnings=0",
-    "lint:fix": "eslint . --report-unused-disable-directives --fix"
+    "lint": "eslint . --max-warnings=0",
+    "lint:fix": "eslint . --fix"
   },
   "files": [
     "lib/*.tsp",
@@ -56,11 +55,14 @@
   ],
   "peerDependencies": {
     "@typespec/compiler": "workspace:~",
-    "@typespec/http": "workspace:~",
-    "@typespec/emitter-framework": "^0.5.0"
+    "@typespec/emitter-framework": "workspace:~",
+    "@typespec/http": "workspace:~"
   },
   "devDependencies": {
     "@types/node": "~22.13.13",
+    "@typespec/compiler": "workspace:~",
+    "@typespec/emitter-framework": "workspace:~",
+    "@typespec/http": "workspace:~",
     "rimraf": "~6.0.1",
     "source-map-support": "~0.5.21",
     "typescript": "~5.8.2",

packages/graphql/src/emitter.ts

@@ -1,12 +1,10 @@
-import type { EmitContext, NewLine } from "@typespec/compiler";
+import type { EmitContext } from "@typespec/compiler";
 import { resolvePath } from "@typespec/compiler";
 import { createGraphQLEmitter } from "./graphql-emitter.js";
 import type { GraphQLEmitterOptions } from "./lib.js";
 
 const defaultOptions = {
-  "new-line": "lf",
   "omit-unreachable-types": false,
-  strict: false,
 } as const;
 
 export async function $onEmit(context: EmitContext<GraphQLEmitterOptions>) {
@@ -17,9 +15,7 @@ export async function $onEmit(context: EmitContext<GraphQLEmitterOptions>) {
 
 export interface ResolvedGraphQLEmitterOptions {
   outputFile: string;
-  newLine: NewLine;
   omitUnreachableTypes: boolean;
-  strict: boolean;
 }
 
 export function resolveOptions(
@@ -30,8 +26,6 @@ export function resolveOptions(
 
   return {
     outputFile: resolvePath(context.emitterOutputDir, outputFile),
-    newLine: resolvedOptions["new-line"],
     omitUnreachableTypes: resolvedOptions["omit-unreachable-types"],
-    strict: resolvedOptions["strict"],
   };
 }

packages/graphql/src/graphql-emitter.ts

@@ -1,10 +1,16 @@
-import { emitFile, interpolatePath, type EmitContext } from "@typespec/compiler";
-import { printSchema } from "graphql";
+import { emitFile, interpolatePath, type Diagnostic, type EmitContext } from "@typespec/compiler";
+import { type GraphQLSchema, printSchema } from "graphql";
 import type { ResolvedGraphQLEmitterOptions } from "./emitter.js";
 import type { GraphQLEmitterOptions } from "./lib.js";
+import type { Schema } from "./lib/schema.js";
 import { listSchemas } from "./lib/schema.js";
 import { createSchemaEmitter } from "./schema-emitter.js";
-import type { GraphQLSchemaRecord } from "./types.js";
+
+interface GraphQLSchemaRecord {
+  readonly schema: Schema;
+  readonly graphQLSchema: GraphQLSchema;
+  readonly diagnostics: readonly Diagnostic[];
+}
 
 export function createGraphQLEmitter(
   context: EmitContext<GraphQLEmitterOptions>,
@@ -34,7 +40,6 @@ export function createGraphQLEmitter(
         await emitFile(program, {
           path: filePath,
           content: printSchema(schemaRecord.graphQLSchema),
-          newLine: options.newLine,
         });
       }
     }
@@ -47,7 +52,7 @@ export function createGraphQLEmitter(
       schemas.push({ type: program.getGlobalNamespaceType() });
     }
     for (const schema of schemas) {
-      const schemaEmitter = createSchemaEmitter(schema, context, options);
+      const schemaEmitter = createSchemaEmitter(schema, context, context.options);
       const document = await schemaEmitter.emitSchema();
       if (document === undefined) {
         continue;

packages/graphql/src/lib.ts

@@ -19,25 +19,13 @@ export interface GraphQLEmitterOptions {
    */
   "output-file"?: string;
 
-  /**
-   * Set the newline character for emitting files.
-   * @default lf
-   */
-  "new-line"?: "crlf" | "lf";
-
   /**
    * Omit unreachable types.
    * By default all types declared under the schema namespace will be included. With this flag on only types references in an operation will be emitted.
    * @default false
    */
   "omit-unreachable-types"?: boolean;
 
-  /**
-   * Only emit types if a correct GraphQL translation type is found. Don't emit Any types and operations that don't have the GraphQL decorators.
-   * By default a best effort is made to emit all types.
-   * @default false
-   */
-  strict?: boolean;
 }
 
 const EmitterOptionsSchema: JSONSchemaType<GraphQLEmitterOptions> = {
@@ -62,13 +50,6 @@ const EmitterOptionsSchema: JSONSchemaType<GraphQLEmitterOptions> = {
         " - `Org1.Schema2.graphql`",
       ].join("\n"),
     },
-    "new-line": {
-      type: "string",
-      enum: ["crlf", "lf"],
-      default: "lf",
-      nullable: true,
-      description: "Set the newLine character for emitting files.",
-    },
     "omit-unreachable-types": {
       type: "boolean",
       nullable: true,
@@ -78,15 +59,6 @@ const EmitterOptionsSchema: JSONSchemaType<GraphQLEmitterOptions> = {
         "With this flag on only types references in an operation will be emitted.",
       ].join("\n"),
     },
-    strict: {
-      type: "boolean",
-      nullable: true,
-      description: [
-        "Only emit types if a correct GraphQL translation type is found.",
-        "Don't emit Any types and operations that don't have the GraphQL decorators.",
-        "By default a best effort is made to emit all types.",
-      ].join("\n"),
-    },
   },
   required: [],
 };

packages/graphql/src/lib/interface.ts

@@ -11,6 +11,7 @@ import {
 import { useStateMap, useStateSet } from "@typespec/compiler/utils";
 import { GraphQLKeys, NAMESPACE, reportDiagnostic } from "../lib.js";
 import type { Tagged } from "../types.d.ts";
+import { propertiesEqual } from "./utils.js";
 
 // This will set the namespace for decorators implemented in this file
 export const namespace = NAMESPACE;
@@ -75,13 +76,6 @@ function validateNoCircularImplementation(
   return valid;
 }
 
-function propertiesEqual(prop1: ModelProperty, prop2: ModelProperty): boolean {
-  // TODO is there some canonical way to do this?
-  return (
-    prop1.name === prop2.name && prop1.type === prop2.type && prop1.optional === prop2.optional
-  );
-}
-
 function validateImplementsInterfaceProperties(
   context: DecoratorContext,
   modelProperties: Map<string, ModelProperty>,

packages/graphql/src/lib/operation-fields.ts

@@ -7,9 +7,6 @@ import {
   type Operation,
   type Program,
 } from "@typespec/compiler";
-
-// import { createTypeRelationChecker } from "../../../compiler/dist/src/core/type-relation-checker.js";
-
 import { useStateMap } from "@typespec/compiler/utils";
 import { GraphQLKeys, NAMESPACE, reportDiagnostic } from "../lib.js";
 import { operationsEqual } from "./utils.js";

packages/graphql/src/testing/index.ts

@@ -1,7 +1,7 @@
 import type { TypeSpecTestLibrary } from "@typespec/compiler/testing";
 import { createTestLibrary, findTestPackageRoot } from "@typespec/compiler/testing";
 
-export const GraphqlTestLibrary: TypeSpecTestLibrary = createTestLibrary({
+export const GraphQLTestLibrary: TypeSpecTestLibrary = createTestLibrary({
   name: "@typespec/graphql",
   packageRoot: await findTestPackageRoot(import.meta.url),
 });

packages/graphql/src/types.d.ts

@@ -1,22 +1,3 @@
-import type { Diagnostic } from "@typespec/compiler";
-import type { GraphQLSchema } from "graphql";
-import type { Schema } from "./lib/schema.ts";
-
-/**
- * A record containing the GraphQL schema corresponding to
- * a particular schema definition.
- */
-export interface GraphQLSchemaRecord {
-  /** The declared schema that generated this GraphQL schema */
-  readonly schema: Schema;
-
-  /** The GraphQLSchema */
-  readonly graphQLSchema: GraphQLSchema;
-
-  /** The diagnostics created for this schema */
-  readonly diagnostics: readonly Diagnostic[];
-}
-
 declare const tags: unique symbol;
 
-type Tagged<BaseType, Tag extends PropertyKey> = BaseType & { [tags]: { [K in Tag]: void } };
+export type Tagged<BaseType, Tag extends PropertyKey> = BaseType & { [tags]: { [K in Tag]: void } };