test: migrate TypeScript tests to @typescript/server-harness

- Add new test directory test/ts/ for TypeScript plugin tests
- Tests use @typescript/server-harness to test TypeScript features
- Migrate hover, definitions, completions, rename, diagnostics, code-action tests
- Update language server tests to remove TypeScript-dependent assertions
- Keep basic language server functionality tests
- Add test-ts script to run TypeScript plugin tests separately

Author: johnsoncodehk

packages/language-server/package.json

@@ -28,7 +28,8 @@
   ],
   "scripts": {
     "test-api": "node --test 'test/*.test.js'",
-    "test": "npm run test-api"
+    "test-ts": "node --test 'test/ts/*.test.js'",
+    "test": "npm run test-api && npm run test-ts"
   },
   "dependencies": {
     "@mdx-js/language-service": "0.7.2",

packages/language-server/test/code-action.test.js

@@ -26,7 +26,6 @@ afterEach(() => {
 
 test('return empty code actions for non-existent file', async () => {
   const uri = fixtureUri('node16/non-existent.mdx')
-
   const codeActions = await serverHandle.sendCodeActionsRequest(
     uri,
     {
@@ -39,6 +38,5 @@ test('return empty code actions for non-existent file', async () => {
       triggerKind: CodeActionTriggerKind.Invoked
     }
   )
-
   assert.deepEqual(codeActions, [])
 })

packages/language-server/test/definitions.test.js

@@ -42,7 +42,6 @@ test('does not resolve shadow content', async () => {
     line: 0,
     character: 37
   })
-
   assert.deepEqual(result, [])
 })
 
@@ -52,6 +51,5 @@ test('ignore non-existent mdx files', async () => {
     line: 7,
     character: 15
   })
-
   assert.deepEqual(result, [])
 })

packages/language-server/test/diagnostics.test.js

@@ -30,7 +30,6 @@ test('parse errors', async () => {
     'mdx'
   )
   const diagnostics = await serverHandle.sendDocumentDiagnosticRequest(uri)
-
   assert.deepEqual(diagnostics, {
     kind: 'full',
     items: [
@@ -78,7 +77,6 @@ test('does not resolve shadow content', async () => {
     'mdx'
   )
   const diagnostics = await serverHandle.sendDocumentDiagnosticRequest(uri)
-
   assert.deepEqual(diagnostics, {
     items: [],
     kind: 'full'

packages/language-server/test/ts/code-action.test.js

@@ -0,0 +1,46 @@
+/**
+ * @fileoverview TypeScript code action tests for MDX files
+ *
+ * These tests verify that code actions (like organize imports)
+ * work correctly in MDX files through the TypeScript plugin.
+ */
+import assert from 'node:assert/strict'
+import {after, before, test} from 'node:test'
+import {fixturePath, getTsServer} from './server.js'
+
+/** @type {Awaited<ReturnType<typeof getTsServer>>} */
+let server
+
+before(async () => {
+  server = await getTsServer()
+})
+
+after(() => {
+  server.shutdown()
+})
+
+test('organize imports', async () => {
+  const filePath = fixturePath('node16/organize-imports.mdx')
+  await server.openFile(filePath)
+
+  try {
+    const res = await server.tsserver.message({
+      seq: server.nextSeq(),
+      type: 'request',
+      command: 'organizeImports',
+      arguments: {
+        scope: {
+          type: 'file',
+          args: {file: filePath}
+        }
+      }
+    })
+
+    assert.ok(res.success, 'Request should succeed')
+    assert.ok(res.body, 'Response should have body')
+    // Organize imports should return file changes
+    assert.ok(Array.isArray(res.body), 'Response body should be an array')
+  } finally {
+    await server.closeFile(filePath)
+  }
+})

packages/language-server/test/ts/completions.test.js

@@ -0,0 +1,80 @@
+/**
+ * @fileoverview TypeScript completion tests for MDX files
+ *
+ * These tests verify that code completion works correctly
+ * in MDX files through the TypeScript plugin.
+ */
+import assert from 'node:assert/strict'
+import {after, before, test} from 'node:test'
+import {fixturePath, getTsServer} from './server.js'
+
+/** @type {Awaited<ReturnType<typeof getTsServer>>} */
+let server
+
+before(async () => {
+  server = await getTsServer()
+})
+
+after(() => {
+  server.shutdown()
+})
+
+test('support completion in ESM', async () => {
+  const filePath = fixturePath('node16/completion.mdx')
+  await server.openFile(filePath)
+
+  try {
+    const res = await server.tsserver.message({
+      seq: server.nextSeq(),
+      type: 'request',
+      command: 'completions',
+      arguments: {
+        file: filePath,
+        line: 2,
+        offset: 1,
+        includeExternalModuleExports: true
+      }
+    })
+
+    assert.ok(res.success, 'Request should succeed')
+    assert.ok(res.body, 'Response should have body')
+    assert.ok(res.body.length > 0, 'Should have completions')
+
+    const booleanCompletion = res.body.find(
+      (/** @type {{name: string}} */ c) => c.name === 'Boolean'
+    )
+    assert.ok(booleanCompletion, 'Should have Boolean completion')
+  } finally {
+    await server.closeFile(filePath)
+  }
+})
+
+test('support completion in JSX', async () => {
+  const filePath = fixturePath('node16/completion.mdx')
+  await server.openFile(filePath)
+
+  try {
+    const res = await server.tsserver.message({
+      seq: server.nextSeq(),
+      type: 'request',
+      command: 'completions',
+      arguments: {
+        file: filePath,
+        line: 6,
+        offset: 3,
+        includeExternalModuleExports: true
+      }
+    })
+
+    assert.ok(res.success, 'Request should succeed')
+    assert.ok(res.body, 'Response should have body')
+    assert.ok(res.body.length > 0, 'Should have completions')
+
+    const booleanCompletion = res.body.find(
+      (/** @type {{name: string}} */ c) => c.name === 'Boolean'
+    )
+    assert.ok(booleanCompletion, 'Should have Boolean completion')
+  } finally {
+    await server.closeFile(filePath)
+  }
+})

packages/language-server/test/ts/definitions.test.js

@@ -0,0 +1,48 @@
+/**
+ * @fileoverview TypeScript definition tests for MDX files
+ *
+ * These tests verify that go-to-definition works correctly
+ * in MDX files through the TypeScript plugin.
+ */
+import assert from 'node:assert/strict'
+import {after, before, test} from 'node:test'
+import {fixturePath, getTsServer} from './server.js'
+
+/** @type {Awaited<ReturnType<typeof getTsServer>>} */
+let server
+
+before(async () => {
+  server = await getTsServer()
+})
+
+after(() => {
+  server.shutdown()
+})
+
+test('resolve file-local definitions in ESM', async () => {
+  const filePath = fixturePath('node16/a.mdx')
+  await server.openFile(filePath)
+
+  try {
+    const res = await server.tsserver.message({
+      seq: server.nextSeq(),
+      type: 'request',
+      command: 'definition',
+      arguments: {
+        file: filePath,
+        line: 5,
+        offset: 3
+      }
+    })
+
+    assert.ok(res.success, 'Request should succeed')
+    assert.ok(res.body, 'Response should have body')
+    assert.ok(res.body.length > 0, 'Should have at least one definition')
+
+    const def = res.body[0]
+    assert.ok(def.file.endsWith('a.mdx'), 'Definition should be in a.mdx')
+    assert.equal(def.start.line, 2, 'Definition should be on line 2')
+  } finally {
+    await server.closeFile(filePath)
+  }
+})

packages/language-server/test/ts/diagnostics.test.js

@@ -0,0 +1,43 @@
+/**
+ * @fileoverview TypeScript diagnostics tests for MDX files
+ *
+ * These tests verify that TypeScript type errors are correctly
+ * reported in MDX files through the TypeScript plugin.
+ */
+import assert from 'node:assert/strict'
+import {after, before, test} from 'node:test'
+import {fixturePath, getTsServer} from './server.js'
+
+/** @type {Awaited<ReturnType<typeof getTsServer>>} */
+let server
+
+before(async () => {
+  server = await getTsServer()
+})
+
+after(() => {
+  server.shutdown()
+})
+
+test('type errors', async () => {
+  const filePath = fixturePath('node16/type-errors.mdx')
+  await server.openFile(filePath)
+
+  try {
+    const res = await server.tsserver.message({
+      seq: server.nextSeq(),
+      type: 'request',
+      command: 'semanticDiagnosticsSync',
+      arguments: {
+        file: filePath
+      }
+    })
+
+    assert.ok(res.success, 'Request should succeed')
+    assert.ok(res.body, 'Response should have body')
+    // Type errors should be reported
+    assert.ok(res.body.length > 0, 'Should have type errors')
+  } finally {
+    await server.closeFile(filePath)
+  }
+})