@carecard/auth-util 3.17.0 → 3.19.0

package/lib/jwtUtilAuth.js

@@ -57,7 +57,11 @@ const isNonEmptyString = value => {
 };
 
 const audienceIsNonEmptyStringArray = audience => {
-  return Array.isArray(audience) && audience.length > 0 && audience.every(audienceValue => isNonEmptyString(audienceValue));
+  return (
+    Array.isArray(audience) &&
+    audience.length > 0 &&
+    audience.every(audienceValue => isNonEmptyString(audienceValue))
+  );
 };
 
 const isNonEmptyAudience = audience => {
@@ -65,7 +69,9 @@ const isNonEmptyAudience = audience => {
 };
 
 const normalizeSeconds = value => {
-  if (!Number.isFinite(value)) return null;
+  if (!Number.isFinite(value)) {
+    return null;
+  }
   return value > 1000000000000 ? Math.floor(value / 1000) : Math.floor(value);
 };
 
@@ -77,7 +83,9 @@ const normalizeSeconds = value => {
  * @return {string|null}
  */
 const createSignedJwtFromObject = (headerObject, payloadObject, privateKey) => {
-  if (!privateKey) return null;
+  if (!privateKey) {
+    return null;
+  }
 
   const header = { ...headerObject };
   header.alg = header.alg || 'EdDSA';
@@ -116,7 +124,9 @@ function createServiceJwt({
   }
 
   const iat = normalizeSeconds(issuedAt);
-  if (!Number.isInteger(iat)) return null;
+  if (!Number.isInteger(iat)) {
+    return null;
+  }
 
   // Service-to-service tokens use registered JWT claims so receivers can
   // validate sender, intended audience, subject, and lifetime consistently.
@@ -147,9 +157,13 @@ function createServiceAuthorizationHeader(options = {}) {
  */
 const verifyJwtSignature = (jwt, publicKey) => {
   try {
-    if (!jwt || !publicKey) return false;
+    if (!jwt || !publicKey) {
+      return false;
+    }
     const splitJWT = jwt.split('.');
-    if (splitJWT.length !== 3) return false;
+    if (splitJWT.length !== 3) {
+      return false;
+    }
 
     const [header, payload, signature] = splitJWT;
     const token = header + '.' + payload;
@@ -157,7 +171,9 @@ const verifyJwtSignature = (jwt, publicKey) => {
 
     return _verify(token, signature, headerObject.alg, publicKey);
   } catch (error) {
-    if (!(error instanceof SyntaxError)) throw error;
+    if (!(error instanceof SyntaxError)) {
+      throw error;
+    }
     return false;
   }
 };
@@ -169,16 +185,22 @@ const verifyJwtSignature = (jwt, publicKey) => {
  */
 const getHeaderPayloadFromJwt = jwt => {
   try {
-    if (typeof jwt !== 'string') return null;
+    if (typeof jwt !== 'string') {
+      return null;
+    }
     const splitJWT = jwt.split('.');
-    if (splitJWT.length !== 3) return null;
+    if (splitJWT.length !== 3) {
+      return null;
+    }
 
     const headerObject = _decode(splitJWT[0]);
     const payloadObject = _decode(splitJWT[1]);
 
     return { header: headerObject, payload: payloadObject };
   } catch (error) {
-    if (!(error instanceof SyntaxError)) throw error;
+    if (!(error instanceof SyntaxError)) {
+      throw error;
+    }
     return null;
   }
 };

package/lib/pwdUtilAuth.js

@@ -22,10 +22,14 @@ const createPasswordHashWithRandomSalt = (password, secret, algorithm) => {
  * @return {string}
  */
 const createPasswordHashBasedOnSavedAlgorithmSalt = (password, savedPasswordHash, secret) => {
-  if (typeof savedPasswordHash !== 'string') return null;
+  if (typeof savedPasswordHash !== 'string') {
+    return null;
+  }
 
   const splitStringArray = savedPasswordHash.split('$');
-  if (splitStringArray.length !== 6) return null;
+  if (splitStringArray.length !== 6) {
+    return null;
+  }
 
   const algBase64 = splitStringArray[2];
   const salt = splitStringArray[4];

package/lib/stringUtilAuth.js

@@ -60,7 +60,9 @@ const base64ToAscii = codedString => {
  */
 const dollarSignConnectedStringToAlgorithmHashSalt = passwordHash => {
   const splitStringArray = passwordHash.split('$');
-  if (splitStringArray.length !== 6) return null;
+  if (splitStringArray.length !== 6) {
+    return null;
+  }
   return {
     version: splitStringArray[1],
     alg: splitStringArray[2],
@@ -76,9 +78,13 @@ const dollarSignConnectedStringToAlgorithmHashSalt = passwordHash => {
  * @deprecated Use native Buffer methods or other modern alternatives.
  */
 const dotConnectedStringToHeaderPayloadSignature = jwt => {
-  if (typeof jwt !== 'string') return null;
+  if (typeof jwt !== 'string') {
+    return null;
+  }
   const splitJWT = jwt.split('.');
-  if (splitJWT.length !== 3) return null;
+  if (splitJWT.length !== 3) {
+    return null;
+  }
   return {
     header: splitJWT[0],
     payload: splitJWT[1],

package/lint-staged.config.mjs

@@ -0,0 +1,36 @@
+import { ESLint } from 'eslint';
+
+const eslint = new ESLint();
+
+// Pattern: Pure Function - builds one deterministic command without shell interpolation.
+const createCommand = (command, filePaths) =>
+  `${command} ${filePaths.map(filePath => JSON.stringify(filePath)).join(' ')}`;
+
+// Pattern: Adapter - derives lint-staged input from ESLint's authoritative ignore rules.
+const removeEslintIgnoredFiles = async filePaths => {
+  const ignoredFileStates = await Promise.all(
+    filePaths.map(filePath => eslint.isPathIgnored(filePath)),
+  );
+
+  return filePaths.flatMap((filePath, index) => (ignoredFileStates[index] ? [] : [filePath]));
+};
+
+// Pattern: Pipeline - preserves ESLint-before-Prettier ordering for staged code.
+const createJavaScriptTasks = async filePaths => {
+  const lintableFilePaths = await removeEslintIgnoredFiles(filePaths);
+  const tasks = [];
+
+  if (lintableFilePaths.length > 0) {
+    tasks.push(createCommand('eslint --fix --max-warnings 0', lintableFilePaths));
+  }
+
+  tasks.push(createCommand('prettier --write', filePaths));
+  return tasks;
+};
+
+const lintStagedConfig = {
+  '*.{js,jsx,mjs,cjs,ts,tsx,mts,cts}': createJavaScriptTasks,
+  '*.{json,jsonc,md,mdx,css,scss,yaml,yml}': ['prettier --write'],
+};
+
+export default lintStagedConfig;

package/package.json

@@ -1,21 +1,23 @@
 {
   "name": "@carecard/auth-util",
-  "version": "3.17.0",
+  "version": "3.19.0",
   "repository": "https://github.com/CareCard-ca/pkg-auth-util.git",
   "description": "Auth utility functions",
   "main": "index.js",
   "types": "index.d.ts",
   "scripts": {
-    "test": "npm run test:order && node test/index.test.js",
-    "test:order": "node --test scripts/testOrder/randomizeTestOrder.test.mjs scripts/testOrder/testOrderPolicy.test.mjs scripts/testParallel/runIndexedMochaTests.test.mjs scripts/testParallel/parallelTestPolicy.test.mjs",
-    "test:types": "npm run test:order && tsc --noEmit && mocha --require ./scripts/testOrder/randomizeTestOrder.cjs -r ts-node/register test/types.test.ts",
-    "test:coverage": "npm run test:order && tsc --noEmit && nyc node test/index.test.js",
-    "test:All": "npm run test && npm run test:types",
-    "format": "prettier --write .",
-    "format:check": "prettier --check .",
+    "test": "node scripts/runPackageTask.mjs test",
+    "test:order": "node --test scripts/testOrder/randomizeTestOrder.test.mjs scripts/testParallel/runIndexedMochaTests.test.mjs scripts/packageTaskRunner.test.mjs scripts/canonicalTestCommand.test.mjs",
+    "test:types": "node scripts/runPackageTask.mjs test:types",
+    "test:coverage": "node scripts/runPackageTask.mjs test:coverage",
+    "test:All": "node scripts/runPackageTask.mjs test:All",
+    "validate:audits": "node scripts/runPackageTask.mjs validate:audits",
+    "format": "prettier --write \"**/*.{js,jsx,mjs,cjs,ts,tsx,mts,cts}\"",
+    "format:check": "prettier --check \"**/*.{js,jsx,mjs,cjs,ts,tsx,mts,cts}\"",
+    "lint-staged": "lint-staged",
     "prepare": "husky",
-    "lint:fix": "eslint --fix",
-    "lint": "eslint"
+    "lint:fix": "eslint . --fix --max-warnings 0",
+    "lint": "eslint . --max-warnings 0"
   },
   "keywords": [
     "auth",
@@ -26,31 +28,35 @@
   "author": "CareCard team",
   "license": "ISC",
   "devDependencies": {
+    "@eslint/js": "9.39.5",
     "@istanbuljs/nyc-config-typescript": "1.0.2",
     "@types/mocha": "10.0.10",
     "@types/node": "25.9.3",
-    "eslint": "9.39.4",
+    "@typescript-eslint/parser": "8.67.0",
+    "eslint": "9.39.5",
     "express": "5.2.1",
+    "globals": "17.7.0",
     "husky": "9.1.7",
-    "lint-staged": "17.0.7",
+    "lint-staged": "17.2.0",
     "mocha": "11.7.6",
     "nyc": "18.0.0",
-    "prettier": "3.8.4",
+    "prettier": "3.9.6",
     "source-map-support": "0.5.21",
     "supertest": "7.2.2",
     "ts-node": "10.9.2",
     "typescript": "6.0.3"
   },
   "dependencies": {
-    "@carecard/common-util": "3.17.0",
-    "@carecard/validate": "3.17.0",
+    "@carecard/common-util": "3.18.0",
+    "@carecard/validate": "3.18.0",
     "@types/express": "5.0.6"
   },
   "overrides": {
     "diff": "8.0.4",
     "glob": "13.0.6",
-    "minimatch": "10.2.5",
+    "brace-expansion": "5.0.9",
+    "minimatch": "10.2.6",
     "serialize-javascript": "7.0.5",
-    "js-yaml": "4.3.0"
+    "js-yaml": "4.3.1"
   }
 }

package/readme.md

@@ -11,7 +11,9 @@ Utility package for authentication and authorization in the CareCard ecosystem.
 
 ## Development Rule
 
-Non-negotiable TDD rule: Always write the failing test first, run it to confirm it fails for the intended reason, then implement the code and rerun the test until it passes. Test Driven Development is required for all coding work and must not be skipped. For documentation- or skill-only edits, add or update the relevant validation check before changing the prose.
+Non-negotiable TDD rule: Always write the failing test first, run it to confirm it fails for the intended reason, then implement the code and rerun the test until it passes. Test Driven Development is required for all coding work and must not be skipped. For documentation- or skill-only edits, run the relevant focused non-test
+validation before changing the prose; do not add automated tests that inspect
+prose, files, or repository structure.
 
 Non-negotiable repository isolation rule: Every repository must run its Husky hooks and tests using only files, code, fixtures, dependencies, and services contained within that repository. Tests and Husky scripts must not import, require, read, execute, or otherwise depend on sibling repositories or paths outside the repository root. app-e2e-tests is the only exception because cross-repository end-to-end testing is its explicit responsibility.
 
@@ -172,3 +174,22 @@ immediately when no helper remains, allow only a bounded 250 ms settlement
 window for already-stopping helpers, fail persistent descendants, preserve
 failures and output, use exit code `124` only for a real outer deadline, and
 remain a final guard rather than a substitute for explicit cleanup.
+
+## TDD And Validation
+
+Test Driven Development is a non-negotiable requirement.
+
+The sole purpose of automated tests is to verify observable functionality and externally visible behavior.
+Tests must validate what the system does through its public interfaces and expected outcomes.
+
+Tests must not assert, inspect, or depend on implementation details, including but not limited to:
+
+- The existence of specific lines of code, statements, functions, classes, files, or modules.
+- Specific algorithms, control flow, variable names, method calls, code snippets, or internal implementation choices.
+- Any internal structure that can change without changing externally observable behavior.
+
+A correct implementation may be completely rewritten or refactored without requiring changes to functional tests, provided its externally observable behavior remains unchanged.
+
+Any test that fails solely because the implementation changed while the externally observable behavior remained correct is incorrectly designed and must be rewritten or removed.
+
+This requirement is mandatory for all new tests and must be applied whenever existing tests are modified.

package/scripts/canonicalTestCommand.test.mjs

@@ -0,0 +1,33 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+
+import { runPackageTask } from './runPackageTask.mjs';
+
+test('the complete test command runs each validation category exactly once', () => {
+  const executedSteps = [];
+
+  const exitCode = runPackageTask('test', taskStep => {
+    executedSteps.push([taskStep.command, ...(taskStep.arguments ?? [])].join(' '));
+    return 0;
+  });
+
+  assert.equal(exitCode, 0);
+  assert.deepEqual(executedSteps, [
+    'npm run validate:audits',
+    'npm run test:order',
+    'tsc --noEmit',
+    'mocha --require ./scripts/testOrder/randomizeTestOrder.cjs -r ts-node/register test/types.test.ts',
+    'nyc node test/index.test.js',
+  ]);
+});
+
+test('the legacy aggregate command delegates to the complete test command once', () => {
+  const executedSteps = [];
+
+  runPackageTask('test:All', taskStep => {
+    executedSteps.push([taskStep.command, ...(taskStep.arguments ?? [])].join(' '));
+    return 0;
+  });
+
+  assert.deepEqual(executedSteps, ['npm test']);
+});

package/scripts/packageTaskRunner.audit.mjs

@@ -0,0 +1,37 @@
+import assert from 'node:assert/strict';
+import { readFileSync } from 'node:fs';
+import test from 'node:test';
+
+const packageJson = JSON.parse(readFileSync(new URL('../package.json', import.meta.url), 'utf8'));
+const composedCommandPattern = /&&|\|\||\bsh -c\b|\bnode -e\b/u;
+const runnerCommandPattern = /^node scripts\/runPackageTask\.mjs ([A-Za-z0-9:_-]+)$/u;
+
+async function loadPackageTaskRunner() {
+  return import(new URL('./runPackageTask.mjs', import.meta.url));
+}
+
+test('delegates composed package tasks to the repository runner', async () => {
+  const runnerCommands = Object.entries(packageJson.scripts ?? {}).filter(([, command]) =>
+    runnerCommandPattern.test(command),
+  );
+
+  for (const [scriptName, command] of Object.entries(packageJson.scripts ?? {})) {
+    assert.doesNotMatch(
+      command,
+      composedCommandPattern,
+      `${scriptName} must delegate composition to runPackageTask.mjs`,
+    );
+  }
+
+  assert.ok(runnerCommands.length > 0, 'at least one package task must use the runner');
+  const { packageTasks } = await loadPackageTaskRunner();
+  for (const [scriptName] of runnerCommands) {
+    assert.ok(packageTasks[scriptName], `${scriptName} must have a runner task`);
+  }
+});
+
+test('does not suppress command execution errors at the CLI boundary', () => {
+  const runnerSource = readFileSync(new URL('./runPackageTask.mjs', import.meta.url), 'utf8');
+
+  assert.doesNotMatch(runnerSource, /catch\s*\{/u);
+});

package/scripts/packageTaskRunner.test.mjs

@@ -0,0 +1,64 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+
+async function loadPackageTaskRunner() {
+  return import(new URL('./runPackageTask.mjs', import.meta.url));
+}
+
+test('runs task steps in order and stops at the first failure', async () => {
+  const { runPackageTask } = await loadPackageTaskRunner();
+  const executedCommands = [];
+  const fixtureTasks = {
+    fixture: [{ command: 'first' }, { command: 'second' }, { command: 'third' }],
+  };
+
+  const exitCode = runPackageTask(
+    'fixture',
+    step => {
+      executedCommands.push(step.command);
+      return step.command === 'second' ? 7 : 0;
+    },
+    fixtureTasks,
+  );
+
+  assert.equal(exitCode, 7);
+  assert.deepEqual(executedCommands, ['first', 'second']);
+});
+
+test('runs conditional steps only when their required output is missing', async () => {
+  const { runPackageTask } = await loadPackageTaskRunner();
+  const executedCommands = [];
+  const fixtureTasks = {
+    fixture: [{ command: 'build', whenMissing: 'dist/runtime.js' }, { command: 'execute' }],
+  };
+
+  const exitCode = runPackageTask(
+    'fixture',
+    step => {
+      executedCommands.push(step.command);
+      return 0;
+    },
+    fixtureTasks,
+    () => true,
+  );
+
+  assert.equal(exitCode, 0);
+  assert.deepEqual(executedCommands, ['execute']);
+});
+
+test('merges task environment overrides without mutating inherited values', async () => {
+  const { createTaskEnvironment } = await loadPackageTaskRunner();
+  const inheritedEnvironment = { NODE_ENV: 'test', PATH: '/bin' };
+
+  const environment = createTaskEnvironment(
+    { NODE_ENV: 'production', DB_ENV: 'privileged' },
+    inheritedEnvironment,
+  );
+
+  assert.deepEqual(environment, {
+    NODE_ENV: 'production',
+    PATH: '/bin',
+    DB_ENV: 'privileged',
+  });
+  assert.deepEqual(inheritedEnvironment, { NODE_ENV: 'test', PATH: '/bin' });
+});

package/scripts/runPackageTask.mjs

@@ -0,0 +1,135 @@
+import { spawnSync } from 'node:child_process';
+import { existsSync, rmSync } from 'node:fs';
+import { resolve } from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+export const packageTasks = Object.freeze({
+  'validate:audits': [
+    { command: 'node', arguments: ['scripts/packageTaskRunner.audit.mjs'] },
+    { command: 'node', arguments: ['scripts/testOrder/testOrderPolicy.audit.mjs'] },
+    { command: 'node', arguments: ['scripts/testParallel/parallelTestPolicy.audit.mjs'] },
+    { command: 'mocha', arguments: ['test/config/repositoryIsolation.audit.js'] },
+    { command: 'mocha', arguments: ['test/config/tddGuidanceDocs.audit.js'] },
+    { command: 'mocha', arguments: ['test/dependencyOverrides.audit.js'] },
+  ],
+  test: [
+    { command: 'npm', arguments: ['run', 'validate:audits'] },
+    { command: 'npm', arguments: ['run', 'test:order'] },
+    { command: 'tsc', arguments: ['--noEmit'] },
+    {
+      command: 'mocha',
+      arguments: [
+        '--require',
+        './scripts/testOrder/randomizeTestOrder.cjs',
+        '-r',
+        'ts-node/register',
+        'test/types.test.ts',
+      ],
+    },
+    { command: 'nyc', arguments: ['node', 'test/index.test.js'] },
+  ],
+  'test:types': [
+    { command: 'npm', arguments: ['run', 'test:order'] },
+    { command: 'tsc', arguments: ['--noEmit'] },
+    {
+      command: 'mocha',
+      arguments: [
+        '--require',
+        './scripts/testOrder/randomizeTestOrder.cjs',
+        '-r',
+        'ts-node/register',
+        'test/types.test.ts',
+      ],
+    },
+  ],
+  'test:coverage': [
+    { command: 'npm', arguments: ['run', 'validate:audits'] },
+    { command: 'npm', arguments: ['run', 'test:order'] },
+    { command: 'tsc', arguments: ['--noEmit'] },
+    { command: 'nyc', arguments: ['node', 'test/index.test.js'] },
+  ],
+  'test:All': [{ command: 'npm', arguments: ['test'] }],
+});
+
+export function createTaskEnvironment(overrides = {}, inheritedEnvironment = process.env) {
+  return { ...inheritedEnvironment, ...overrides };
+}
+
+function getTaskExitCode(result) {
+  if (result.error) {
+    throw result.error;
+  }
+  if (typeof result.status === 'number') {
+    return result.status;
+  }
+  if (result.signal === 'SIGINT') {
+    return 130;
+  }
+  if (result.signal === 'SIGTERM') {
+    return 143;
+  }
+  return 1;
+}
+
+export function executeTaskStep(taskStep) {
+  if (taskStep.removePath) {
+    rmSync(taskStep.removePath, { recursive: true, force: true });
+    return 0;
+  }
+
+  const result = spawnSync(taskStep.command, taskStep.arguments ?? [], {
+    env: createTaskEnvironment(taskStep.environment),
+    shell: false,
+    stdio: 'inherit',
+  });
+  return getTaskExitCode(result);
+}
+
+function shouldRunTaskStep(taskStep, pathExists) {
+  return !taskStep.whenMissing || !pathExists(taskStep.whenMissing);
+}
+
+export function runPackageTask(
+  taskName,
+  executeTask = executeTaskStep,
+  taskDefinitions = packageTasks,
+  pathExists = existsSync,
+) {
+  const taskSteps = taskDefinitions[taskName];
+  if (!Array.isArray(taskSteps)) {
+    throw new Error('Unknown package task.');
+  }
+
+  for (const taskStep of taskSteps) {
+    if (!shouldRunTaskStep(taskStep, pathExists)) {
+      continue;
+    }
+    const exitCode = executeTask(taskStep);
+    if (exitCode !== 0) {
+      return exitCode;
+    }
+  }
+  return 0;
+}
+
+function isDirectExecution() {
+  if (!process.argv[1]) {
+    return false;
+  }
+  return resolve(process.argv[1]) === fileURLToPath(import.meta.url);
+}
+
+function runCommandLineTask() {
+  const taskName = process.argv[2];
+  if (!taskName || !packageTasks[taskName]) {
+    process.stderr.write('[PACKAGE_TASK_CONFIG] Unknown package task.\n');
+    process.exitCode = 2;
+    return;
+  }
+
+  process.exitCode = runPackageTask(taskName);
+}
+
+if (isDirectExecution()) {
+  runCommandLineTask();
+}

package/scripts/testOrder/randomizeTestOrder.cjs

@@ -3,10 +3,16 @@
 const MAX_TEST_ORDER_SEED = 2_147_483_647;
 
 function resolveTestOrderSeed(configuredSeed) {
-  if (configuredSeed === undefined) return undefined;
-  if (!/^[1-9]\d*$/.test(configuredSeed)) throw new Error('TEST_ORDER_SEED must be a positive 32-bit integer.');
+  if (configuredSeed === undefined) {
+    return undefined;
+  }
+  if (!/^[1-9]\d*$/.test(configuredSeed)) {
+    throw new Error('TEST_ORDER_SEED must be a positive 32-bit integer.');
+  }
   const seed = Number(configuredSeed);
-  if (!Number.isSafeInteger(seed) || seed > MAX_TEST_ORDER_SEED) throw new Error('TEST_ORDER_SEED must be a positive 32-bit integer.');
+  if (!Number.isSafeInteger(seed) || seed > MAX_TEST_ORDER_SEED) {
+    throw new Error('TEST_ORDER_SEED must be a positive 32-bit integer.');
+  }
   return seed;
 }
 function createSeededRandom(seed) {
@@ -25,14 +31,18 @@ function shuffleValues(values, random) {
   }
 }
 function shuffleSuiteTree(suite, random) {
-  for (const childSuite of suite.suites) shuffleSuiteTree(childSuite, random);
+  for (const childSuite of suite.suites) {
+    shuffleSuiteTree(childSuite, random);
+  }
   shuffleValues(suite.tests, random);
   shuffleValues(suite.suites, random);
 }
 const mochaHooks = {
   beforeAll() {
     const seed = resolveTestOrderSeed(process.env.TEST_ORDER_SEED);
-    if (seed === undefined) return;
+    if (seed === undefined) {
+      return;
+    }
     console.log(`Test order seed: ${seed} (reproduce with TEST_ORDER_SEED=${seed})`);
     shuffleSuiteTree(this.test.parent, createSeededRandom(seed));
   },

package/scripts/testOrder/{testOrderPolicy.test.mjs → testOrderPolicy.audit.mjs}

@@ -7,7 +7,9 @@ const TEST_ORDER_INVARIANCE_RULE =
   "Non-negotiable test order invariance rule: Every test must pass independently of which tests run before or after it, and the suite must pass in every execution order. Each test must establish the state it needs, isolate mutable state, and clean up state it owns; it must never rely on another test's setup, mutations, or cleanup. Default test, CI, and Husky commands must use the test framework's ordinary ordering and must not force randomized ordering. Random-order execution is an explicit diagnostic only, and every failure it exposes must be fixed at the root cause.";
 
 function listRepositoryFiles() {
-  return execFileSync('git', ['ls-files', '--cached', '--others', '--exclude-standard'], { encoding: 'utf8' })
+  return execFileSync('git', ['ls-files', '--cached', '--others', '--exclude-standard'], {
+    encoding: 'utf8',
+  })
     .trim()
     .split('\n')
     .filter(Boolean);
@@ -43,6 +45,10 @@ test('keeps default package scripts on the test framework ordinary ordering', ()
 
   for (const [scriptName, command] of Object.entries(packageJson.scripts ?? {})) {
     assert.equal(typeof command, 'string', `${scriptName} must be a string command.`);
-    assert.doesNotMatch(command, /--test-randomize|--test-random-seed/, `${scriptName} must not force randomized test ordering.`);
+    assert.doesNotMatch(
+      command,
+      /--test-randomize|--test-random-seed/,
+      `${scriptName} must not force randomized test ordering.`,
+    );
   }
 });