test: add issue #621 regression test for declared type dependencies

Author: mattapperson

e2e/issues/issue-621-declared-type-dependencies.test.ts

@@ -0,0 +1,91 @@
+/**
+ * Regression test for GitHub issue #621
+ * https://github.com/OpenRouterTeam/ai-sdk-provider/issues/621
+ *
+ * Reported error: TS2307: Cannot find module '@ai-sdk/provider' (pnpm
+ * isolated with hoist=false, skipLibCheck: false); under other layouts the
+ * provider types silently became `any` or bound to an arbitrary hoisted copy.
+ * SDK version: @openrouter/ai-sdk-provider 3.1.0
+ *
+ * This test verifies that every package imported by the library source (and
+ * by the built `dist/**\/*.d.ts`, when present) is declared in `dependencies`
+ * or `peerDependencies`. Layout-level coverage against a packed tarball lives
+ * in `scripts/check-consumer-types.mjs` (`pnpm check:consumer-types`).
+ */
+import { existsSync, readdirSync, readFileSync, statSync } from 'node:fs';
+import { join } from 'node:path';
+import { describe, expect, it } from 'vitest';
+
+const repoRoot = join(import.meta.dirname, '..', '..');
+const pkg: {
+  dependencies?: Record<string, string>;
+  peerDependencies?: Record<string, string>;
+} = JSON.parse(readFileSync(join(repoRoot, 'package.json'), 'utf8'));
+const declared = new Set([
+  ...Object.keys(pkg.dependencies ?? {}),
+  ...Object.keys(pkg.peerDependencies ?? {}),
+]);
+
+function listFiles(dir: string, accept: (file: string) => boolean): string[] {
+  return readdirSync(dir).flatMap((entry) => {
+    const full = join(dir, entry);
+    if (statSync(full).isDirectory()) {
+      return listFiles(full, accept);
+    }
+    return accept(full) ? [full] : [];
+  });
+}
+
+function importedPackages(files: string[]): Set<string> {
+  const specifier = /(?:from|import)\s*\(?\s*['"]([^'"]+)['"]/g;
+  const packages = new Set<string>();
+  for (const file of files) {
+    for (const [, spec] of readFileSync(file, 'utf8').matchAll(specifier)) {
+      if (
+        !spec ||
+        spec.startsWith('.') ||
+        spec.startsWith('@/') ||
+        spec.startsWith('node:')
+      ) {
+        continue;
+      }
+      const parts = spec.split('/');
+      packages.add(
+        spec.startsWith('@') ? parts.slice(0, 2).join('/') : parts[0]!,
+      );
+    }
+  }
+  return packages;
+}
+
+const distDir = join(repoRoot, 'dist');
+
+describe('Issue #621: type declarations import undeclared packages', () => {
+  it('declares @ai-sdk/provider and @ai-sdk/provider-utils as dependencies', () => {
+    expect(pkg.dependencies).toHaveProperty('@ai-sdk/provider');
+    expect(pkg.dependencies).toHaveProperty('@ai-sdk/provider-utils');
+  });
+
+  it('declares every package imported by library source', () => {
+    const files = listFiles(
+      join(repoRoot, 'src'),
+      (f) => f.endsWith('.ts') && !f.endsWith('.test.ts'),
+    );
+    const undeclared = [...importedPackages(files)].filter(
+      (name) => !declared.has(name),
+    );
+    expect(undeclared).toEqual([]);
+  });
+
+  it.skipIf(!existsSync(distDir))(
+    'declares every package imported by built type declarations',
+    () => {
+      const files = listFiles(distDir, (f) => f.endsWith('.d.ts'));
+      expect(files.length).toBeGreaterThan(0);
+      const undeclared = [...importedPackages(files)].filter(
+        (name) => !declared.has(name),
+      );
+      expect(undeclared).toEqual([]);
+    },
+  );
+});