From 7fc6355208a46b826deb18b765423bb87240d431 Mon Sep 17 00:00:00 2001
From: Michael Cousins <michael@cousins.io>
Date: Thu, 3 Oct 2024 01:33:44 -0400
Subject: [PATCH] Do not treat trailing `#` in gitignore as comment (#797)

---
 packages/knip/src/util/glob-core.ts                      | 2 +-
 .../util/parse-and-convert-gitignore-patterns.test.ts    | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/packages/knip/src/util/glob-core.ts b/packages/knip/src/util/glob-core.ts
index a13db7c23..babce2ad1 100644
--- a/packages/knip/src/util/glob-core.ts
+++ b/packages/knip/src/util/glob-core.ts
@@ -54,7 +54,7 @@ export const parseAndConvertGitignorePatterns = (patterns: string, ancestor?: st
     .split(/\r?\n/)
     .filter(line => line.trim() && !line.startsWith('#'))
     .flatMap(line => {
-      const pattern = line.replace(/(?<!\\)#.*/, '').trim();
+      const pattern = line.replace(/^\\(?=#)/, '').trim();
       if (ancestor && matchFrom) {
         if (pattern.match(matchFrom)) return [pattern.replace(matchFrom, '$1')];
         if (pattern.startsWith('/**/')) return [pattern.slice(1)];
diff --git a/packages/knip/test/util/parse-and-convert-gitignore-patterns.test.ts b/packages/knip/test/util/parse-and-convert-gitignore-patterns.test.ts
index 8d40e4ef4..2bf73ab89 100644
--- a/packages/knip/test/util/parse-and-convert-gitignore-patterns.test.ts
+++ b/packages/knip/test/util/parse-and-convert-gitignore-patterns.test.ts
@@ -24,3 +24,12 @@ test('parseAndConvertGitignorePatterns (ancestor)', async () => {
     { negated: false, patterns: ['c', 'c/**'] },
   ]);
 });
+
+test('parseAndConvertGitignorePatterns (hashes)', async () => {
+  const gitignorePatterns = ['#comment', 'ends-with-hash#', String.raw`\#starts-with-hash`];
+  const globPatterns = parse(gitignorePatterns.join(EOL));
+  assert.deepEqual(globPatterns, [
+    { negated: false, patterns: ['**/ends-with-hash#', '**/ends-with-hash#/**'] },
+    { negated: false, patterns: ['**/#starts-with-hash', '**/#starts-with-hash/**'] },
+  ]);
+});