fix: add exclude paths to pre-commit date update; update test input dates;

Author: SteinGabriel

tools/copyright/sync-header-years.mjs

@@ -20,7 +20,7 @@ function run() {
   const changedFiles = [];
 
   for (const file of stagedFiles) {
-    if (!isFile(file)) {
+    if (!isFile(file) || isExcluded(file)) {
       continue;
     }
     const absolutePath = resolve(process.cwd(), file);
@@ -77,6 +77,18 @@ function getStagedFiles() {
   return output.split('\n').filter(Boolean);
 }
 
+export function isExcluded(filePath) {
+  return EXCLUDE_PATTERNS.some((pattern) => pattern.test(filePath));
+}
+
+const EXCLUDE_PATTERNS = [
+  /\.test\.[cm]?[jt]sx?$/i,
+  /\.spec\.[cm]?[jt]sx?$/i,
+  /(^|[/\\])dist[/\\]/,
+  /(^|[/\\])vendor[/\\]/,
+  /(^|[/\\])node_modules[/\\]/,
+];
+
 function isFile(filePath) {
   try {
     return statSync(filePath).isFile();

tools/copyright/sync-header-years.test.mjs

@@ -1,22 +1,26 @@
 import assert from 'node:assert/strict';
 import test from 'node:test';
 
-import { hasInvalidPingCopyrightHeader, updateCopyrightYears } from './sync-header-years.mjs';
+import {
+  hasInvalidPingCopyrightHeader,
+  isExcluded,
+  updateCopyrightYears,
+} from './sync-header-years.mjs';
 
 test('updates stale range end year and keeps start year', () => {
-  const input = '/* Copyright 2020 - 2026 Ping Identity. All Rights Reserved */';
+  const input = '/* Copyright 2020-2024 Ping Identity. All Rights Reserved */';
   const actual = updateCopyrightYears(input, 2026);
   assert.equal(actual, '/* Copyright 2020 - 2026 Ping Identity. All Rights Reserved */');
 });
 
 test('normalizes separator on an already-current range', () => {
-  const input = '/* Copyright 2020 - 2026 Ping Identity. All Rights Reserved */';
+  const input = '/* Copyright 2020-2026 Ping Identity. All Rights Reserved */';
   const actual = updateCopyrightYears(input, 2026);
   assert.equal(actual, '/* Copyright 2020 - 2026 Ping Identity. All Rights Reserved */');
 });
 
 test('expands stale single year to a range preserving start year', () => {
-  const input = '/* Copyright 2020 - 2026 Ping Identity. All Rights Reserved */';
+  const input = '/* Copyright 2020 Ping Identity. All Rights Reserved */';
   const actual = updateCopyrightYears(input, 2026);
   assert.equal(actual, '/* Copyright 2020 - 2026 Ping Identity. All Rights Reserved */');
 });
@@ -29,8 +33,8 @@ test('does not change an already-current spaced range', () => {
 
 test('supports © and © variants', () => {
   const input = [
-    '/* © Copyright 2020 - 2026 Ping Identity. */',
-    '<!-- &copy; Copyright 2020 - 2026 Ping Identity. -->',
+    '/* © Copyright 2020-2024 Ping Identity. */',
+    '<!-- &copy; Copyright 2020-2024 Ping Identity. -->',
   ].join('\n');
   const actual = updateCopyrightYears(input, 2026);
   assert.equal(
@@ -49,7 +53,7 @@ test('does not update non-Ping headers', () => {
 });
 
 test('updates Ping Identity Corporation ranges with spaces and (c)', () => {
-  const input = '/* Copyright (c) 2023 - 2026 Ping Identity Corporation. All right reserved. */';
+  const input = '/* Copyright (c) 2023 - 2024 Ping Identity Corporation. All right reserved. */';
   const actual = updateCopyrightYears(input, 2026);
   assert.equal(
     actual,
@@ -58,7 +62,7 @@ test('updates Ping Identity Corporation ranges with spaces and (c)', () => {
 });
 
 test('expands stale single year with (c) to a range for Ping Identity Corporation', () => {
-  const input = '/* Copyright (c) 2023 - 2026 Ping Identity Corporation. All right reserved. */';
+  const input = '/* Copyright (c) 2023 Ping Identity Corporation. All right reserved. */';
   const actual = updateCopyrightYears(input, 2026);
   assert.equal(
     actual,
@@ -80,3 +84,19 @@ test('does not flag non-header Ping copyright text', () => {
   const input = 'This document is Copyright Ping Identity Corporation.';
   assert.equal(hasInvalidPingCopyrightHeader(input), false);
 });
+
+test('excludes test files from processing', () => {
+  assert.equal(isExcluded('src/foo.test.ts'), true);
+  assert.equal(isExcluded('src/foo.test.mjs'), true);
+  assert.equal(isExcluded('src/foo.spec.js'), true);
+});
+
+test('excludes dist and vendor paths from processing', () => {
+  assert.equal(isExcluded('dist/foo.js'), true);
+  assert.equal(isExcluded('vendor/lib.js'), true);
+});
+
+test('does not exclude regular source files', () => {
+  assert.equal(isExcluded('src/foo.ts'), false);
+  assert.equal(isExcluded('packages/sdk/src/index.ts'), false);
+});