impacted 0.0.5 → 0.0.8

package/README.md

@@ -9,18 +9,21 @@ A userland implementation of [predictive test selection for Node.js test runner]
 ## Usage
 
 ```bash
-# Run only impacted tests
+# Run only impacted tests (--since gets changed files from git diff)
+node --test $(npx impacted --since main)
+
+# Pipe changed files from stdin
 node --test $(git diff --name-only main | npx impacted)
 
 # Works with any test runner
-vitest $(git diff --name-only main | npx impacted)
-jest $(git diff --name-only main | npx impacted)
+vitest $(npx impacted --since main)
+jest $(npx impacted --since main)
 
 # Custom test pattern
-git diff --name-only main | npx impacted -p "src/**/*.spec.js"
+npx impacted --since main -p "src/**/*.spec.js"
 
 # Multiple patterns
-git diff --name-only main | npx impacted -p "test/**/*.test.js" -p "test/**/*.spec.js"
+npx impacted --since main -p "test/**/*.test.js" -p "test/**/*.spec.js"
 ```
 
 ## GitHub Action
@@ -33,7 +36,7 @@ git diff --name-only main | npx impacted -p "test/**/*.test.js" -p "test/**/*.sp
 - uses: sozua/impacted@v1
   id: impacted
   with:
-    pattern: '**/*.{test,spec}.{js,mjs,cjs,jsx}'  # default
+    pattern: '**/*.{test,spec}.{js,mjs,cjs,jsx,ts,mts,cts,tsx}'  # default
 
 - name: Run impacted tests
   if: steps.impacted.outputs.has-impacted == 'true'
@@ -54,15 +57,37 @@ const tests = await findImpacted({
 });
 ```
 
+### `node:test` `run()` integration
+
+```javascript
+import { run } from 'node:test';
+import { findImpacted } from 'impacted';
+
+const files = await findImpacted({
+  changedFiles: ['src/utils.js'],
+  testFiles: 'test/**/*.test.js',
+});
+
+run({ files });
+```
+
+See [examples/05-node-test-run](./examples/05-node-test-run) for a full working example.
+
+## TypeScript
+
+TypeScript files (`.ts`, `.mts`, `.cts`, `.tsx`) are supported out of the box on Node.js >= 22.7. Type stripping is handled via `node:module.stripTypeScriptTypes()` — no additional dependencies required.
+
+Follows Node.js core's TypeScript philosophy: explicit extensions, no `tsconfig.json`, no path aliases.
+
 ## Limitations
 
-- JavaScript only (`.js`, `.mjs`, `.cjs`, `.jsx`) — no TypeScript yet
 - Static analysis only — dynamic `require(variable)` not supported
 - Local files only — `node_modules` changes won't trigger tests
+- TypeScript support requires Node.js >= 22.7 (JS analysis works on Node.js >= 18)
 
 ## Requirements
 
-- Node.js >= 18
+- Node.js >= 18 (TypeScript support requires >= 22.7)
 
 ## License
 

package/bin/impacted.js

@@ -1,35 +1,23 @@
 #!/usr/bin/env node
 
+import { execSync } from 'node:child_process';
 import { findImpacted } from '../src/index.js';
 
 const args = process.argv.slice(2);
 
 // Parse -p/--pattern flags (supports multiple: -p "*.test.js" -p "*.spec.js")
 const patterns = [];
+let since = null;
 for (let i = 0; i < args.length; i++) {
   if ((args[i] === '-p' || args[i] === '--pattern') && args[i + 1]) {
     patterns.push(args[++i]);
+  } else if (args[i] === '--since' && args[i + 1]) {
+    since = args[++i];
   }
 }
-const pattern = patterns.length > 0 ? patterns : '**/*.{test,spec}.{js,mjs,cjs,jsx}';
-
-// Read changed files from stdin
-let input = '';
-process.stdin.setEncoding('utf8');
-
-process.stdin.on('readable', () => {
-  let chunk;
-  while ((chunk = process.stdin.read()) !== null) {
-    input += chunk;
-  }
-});
-
-process.stdin.on('end', async () => {
-  const changedFiles = input
-    .split('\n')
-    .map((line) => line.trim())
-    .filter((line) => line.length > 0);
+const pattern = patterns.length > 0 ? patterns : '**/*.{test,spec}.{js,mjs,cjs,jsx,ts,mts,cts,tsx}';
 
+async function run(changedFiles) {
   if (changedFiles.length === 0) {
     process.exit(0);
   }
@@ -42,4 +30,41 @@ process.stdin.on('end', async () => {
   for (const file of impacted) {
     console.log(file);
   }
-});
+}
+
+// --since <ref>: get changed files from git diff
+if (since) {
+  let output;
+  try {
+    output = execSync(`git diff --name-only ${since}`, { encoding: 'utf8' });
+  } catch {
+    process.exit(1);
+  }
+
+  const changedFiles = output
+    .split('\n')
+    .map((line) => line.trim())
+    .filter((line) => line.length > 0);
+
+  await run(changedFiles);
+} else {
+  // Read changed files from stdin
+  let input = '';
+  process.stdin.setEncoding('utf8');
+
+  process.stdin.on('readable', () => {
+    let chunk;
+    while ((chunk = process.stdin.read()) !== null) {
+      input += chunk;
+    }
+  });
+
+  process.stdin.on('end', async () => {
+    const changedFiles = input
+      .split('\n')
+      .map((line) => line.trim())
+      .filter((line) => line.length > 0);
+
+    await run(changedFiles);
+  });
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "impacted",
-  "version": "0.0.5",
+  "version": "0.0.8",
   "description": "Find test files impacted by code changes using static dependency analysis",
   "type": "module",
   "main": "./src/index.js",

package/src/constants.js

@@ -0,0 +1,3 @@
+export const JS_EXTENSIONS = new Set(['.js', '.mjs', '.cjs', '.jsx']);
+export const TS_EXTENSIONS = new Set(['.ts', '.mts', '.cts', '.tsx']);
+export const SUPPORTED_EXTENSIONS = new Set([...JS_EXTENSIONS, ...TS_EXTENSIONS]);

package/src/graph.js

@@ -2,8 +2,7 @@ import { readFileSync } from 'node:fs';
 import { extname } from 'node:path';
 import { parseImports } from './parser.js';
 import { resolveSpecifier } from './resolver.js';
-
-const SUPPORTED_EXTENSIONS = ['.js', '.mjs', '.cjs', '.jsx'];
+import { SUPPORTED_EXTENSIONS, TS_EXTENSIONS } from './constants.js';
 
 /** @typedef {import('./cache.js').ImportCache} ImportCache */
 
@@ -13,7 +12,7 @@ const SUPPORTED_EXTENSIONS = ['.js', '.mjs', '.cjs', '.jsx'];
  * @returns {boolean}
  */
 function isSupportedFile(filePath) {
-  return SUPPORTED_EXTENSIONS.includes(extname(filePath));
+  return SUPPORTED_EXTENSIONS.has(extname(filePath));
 }
 
 /**
@@ -28,7 +27,8 @@ export function extractImports(filePath) {
   } catch {
     return [];
   }
-  return parseImports(source);
+  const typescript = TS_EXTENSIONS.has(extname(filePath));
+  return parseImports(source, { typescript });
 }
 
 const DEFAULT_EXCLUDE_PATHS = ['node_modules', 'dist'];

package/src/index.d.ts

@@ -52,8 +52,13 @@ export function invertGraph(
   graph: Map<string, Set<string>>,
 ): Map<string, Set<string>>;
 
+export interface ParseImportsOptions {
+  /** Strip TypeScript syntax before parsing (requires Node >= 22.7) */
+  typescript?: boolean;
+}
+
 /** Extract import/require specifiers from source code */
-export function parseImports(source: string): string[];
+export function parseImports(source: string, options?: ParseImportsOptions): string[];
 
 /** Resolve an import specifier to an absolute file path */
 export function resolveSpecifier(

package/src/parser.js

@@ -1,13 +1,34 @@
 import * as acorn from 'acorn';
 import * as walk from 'acorn-walk';
+import * as nodeModule from 'node:module';
 
 /**
  * Extract all import/require specifiers from source code
  * Handles ESM (import/export) and CJS (require) patterns
  * @param {string} source - The source code
+ * @param {Object} [options]
+ * @param {boolean} [options.typescript=false] - Strip TypeScript syntax before parsing (requires Node >= 22.7)
  * @returns {string[]} Array of import specifiers
  */
-export function parseImports(source) {
+export function parseImports(source, { typescript = false } = {}) {
+  if (typescript) {
+    if (!nodeModule.stripTypeScriptTypes) {
+      if (!parseImports._tsWarned) {
+        parseImports._tsWarned = true;
+        process.emitWarning(
+          'TypeScript support requires Node.js >= 22.7',
+          'UnsupportedWarning',
+        );
+      }
+      return [];
+    }
+    try {
+      source = nodeModule.stripTypeScriptTypes(source);
+    } catch {
+      return [];
+    }
+  }
+
   const imports = [];
 
   let ast;

package/src/resolver.js

@@ -1,7 +1,8 @@
 import { createRequire } from 'node:module';
 import { builtinModules } from 'node:module';
-import { readFileSync } from 'node:fs';
-import { dirname, join, resolve } from 'node:path';
+import { existsSync, readFileSync } from 'node:fs';
+import { dirname, extname, join, resolve } from 'node:path';
+import { TS_EXTENSIONS } from './constants.js';
 
 /**
  * Find the nearest package.json from a given path
@@ -111,6 +112,14 @@ export function resolveSpecifier(specifier, parentPath) {
     const require = createRequire(parentPath);
     return require.resolve(specifier);
   } catch {
+    // require.resolve doesn't handle TS extensions
+    // For relative specifiers with explicit TS extensions, try manual resolution
+    if (specifier.startsWith('.') && TS_EXTENSIONS.has(extname(specifier))) {
+      const resolved = resolve(dirname(parentPath), specifier);
+      if (existsSync(resolved)) {
+        return resolved;
+      }
+    }
     return null;
   }
 }