Make typescript a peerDependency (#228)

* chore: make typescript a peerDependency

Co-authored-by: James Cleveland <james.cleveland@fresha.com>

* Add TypeScript version matrix to CI

* Fix pnpm workspace root flag for TypeScript install step

* Only run TypeScript version matrix on test job

The build job's "assert no files changed" step checks generated
output against what's committed, which is version-specific. The TS
matrix belongs only on the test (unit test) job.

* Add TypeScript 6.0 support and pin CI versions

- Remove deprecated `downlevelIteration`, add explicit `types: ["node"]`
  for TS 6's stricter type resolution
- Fix catch clauses and implicit any for TS 6 compatibility
- Add TS 6.0.2 to CI matrix and pin all TS versions to exact patches
- Update changelog

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Update generated llm-docs for changelog entry

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: James Cleveland <james.cleveland@fresha.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 3 people

.github/workflows/ci.yaml

@@ -56,6 +56,11 @@ jobs:
     strategy:
       matrix:
         node-version: *node-versions
+        typescript-version:
+          - "5.5.4"
+          - "5.7.3"
+          - "5.9.3"
+          - "6.0.2"
     steps:
       - uses: actions/checkout@v3
       - uses: pnpm/action-setup@v2
@@ -68,6 +73,8 @@ jobs:
           cache: "pnpm"
       - name: Install dependencies
         run: pnpm install
+      - name: Install TypeScript ${{ matrix.typescript-version }}
+        run: pnpm add -wD typescript@${{ matrix.typescript-version }}
       - name: Build # Integration tests depend upon having .js versions created by tsc
         run: pnpm run build
       - name: Unit Tests

llm-docs/changelog.md

@@ -6,6 +6,10 @@ Changes in this section are not yet released. If you need access to these change
 
 -   **Features**
     -   Added support for deriving `@gqlEnum` from const arrays (`(typeof X)[number]`) and const objects (`(typeof X)[keyof typeof X]`). This allows defining enums with runtime-accessible values without using TypeScript's `enum` syntax. The const declaration must immediately precede the type alias. See [enum docs](./docblock-tags/enums.md#runtime-accessible-enums) for details.
+-   **Improvements**
+    -   `typescript` is now a peer dependency (`>=5.5`) instead of a direct dependency, allowing you to use your own TypeScript version. ([PR](https://github.com/captbaritone/grats/pull/228))
+    -   Added support for TypeScript 6.0. ([PR](https://github.com/captbaritone/grats/pull/228))
+    -   CI now tests against TypeScript 5.5, 5.7, 5.9, and 6.0.
 
 ## 0.0.36
 

package.json

@@ -22,8 +22,10 @@
   "dependencies": {
     "commander": "^14.0.1",
     "graphql": "^16.11.0",
-    "semver": "^7.7.2",
-    "typescript": "5.9.2"
+    "semver": "^7.7.2"
+  },
+  "peerDependencies": {
+    "typescript": ">=5.5"
   },
   "devDependencies": {
     "@eslint/eslintrc": "^3.3.1",
@@ -39,7 +41,8 @@
     "path-browserify": "^1.0.1",
     "prettier": "^3.6.2",
     "process": "^0.11.10",
-    "tsx": "^4.19.0"
+    "tsx": "^4.19.0",
+    "typescript": "5.9.2"
   },
   "prettier": {
     "trailingComma": "all"

pnpm-lock.yaml

@@ -3046,7 +3046,7 @@ function graphQLNameValidationMessage(name: string): string | null {
   try {
     assertName(name);
     return null;
-  } catch (e) {
+  } catch (e: any) {
     return e.message;
   }
 }

src/Extractor.ts

@@ -189,7 +189,7 @@ export default class TestRunner {
   async transform(code: string, filename: string): Promise<TransformerResult> {
     try {
       return await this._transformer(code, filename);
-    } catch (e) {
+    } catch (e: any) {
       console.error(e);
       return { kind: "ERROR", err: e.message };
     }

src/tests/TestRunner.ts

@@ -126,7 +126,7 @@ const testDirs: TestDir[] = [
           );
         }
         parsed = parsedResult.value;
-      } catch (e) {
+      } catch (e: any) {
         return err(e.message);
       }
       console.warn = consoleWarn;
@@ -191,7 +191,7 @@ const testDirs: TestDir[] = [
           );
         }
         parsedOptions = parsedOptionsResult.value;
-      } catch (e) {
+      } catch (e: any) {
         return err(e.message);
       }
 

src/tests/test.ts

@@ -59,7 +59,7 @@ export function visitDefinitions(doc: DocumentNode, visitors: Visitor) {
   });
 }
 
-const emptyArray = [];
+const emptyArray: never[] = [];
 
 /**
  * Simplified, more performant, version of graphql-js's visit function that only

src/utils/visitor.ts

@@ -4,10 +4,10 @@
   },
   "compilerOptions": {
     "skipLibCheck": true,
+    "types": ["node"],
     "lib": ["es2020"],
     "experimentalDecorators": true,
     "outDir": "dist",
-    "downlevelIteration": true,
     "strictNullChecks": true,
     "declaration": true,
     "resolveJsonModule": true,

tsconfig.json

@@ -6,6 +6,10 @@ Changes in this section are not yet released. If you need access to these change
 
 - **Features**
   - Added support for deriving `@gqlEnum` from const arrays (`(typeof X)[number]`) and const objects (`(typeof X)[keyof typeof X]`). This allows defining enums with runtime-accessible values without using TypeScript's `enum` syntax. The const declaration must immediately precede the type alias. See [enum docs](../04-docblock-tags/07-enums.mdx#runtime-accessible-enums) for details.
+- **Improvements**
+  - `typescript` is now a peer dependency (`>=5.5`) instead of a direct dependency, allowing you to use your own TypeScript version. ([PR](https://github.com/captbaritone/grats/pull/228))
+  - Added support for TypeScript 6.0. ([PR](https://github.com/captbaritone/grats/pull/228))
+  - CI now tests against TypeScript 5.5, 5.7, 5.9, and 6.0.
 
 ## 0.0.36