fix(file): Stop appending /** to ignore rules that end in a wildcard - #1877

Fixes #1876

Five built-in default ignore rules end in a bare wildcard (**/npm-debug.log*, **/yarn-debug.log*, **/yarn-error.log*, **/repomix-output.*, **/repopack-output.*) and none of them ignores anything today, because normalizeGlobPattern appends /** to every **/-prefixed pattern that has no inner /**. **/npm-debug.log* becomes **/npm-debug.log*/**, which in fast-glob matches only paths below a match — so the file the rule names can never be matched by it.

Visible effect on a pristine build of 9f01703a, packing a fixture that contains the files those rules name:

$ node bin/repomix.cjs . -o /tmp/out.xml && grep -o 'path="[^"]*"' /tmp/out.xml
path="docs/repomix-output.xml"     # **/repomix-output.*  → packed
path="src/a.ts"
path=".gitignore"
path="npm-debug.log.1"             # **/npm-debug.log*    → packed
path="probe-xyz.old1"
path="README.md"
path="repomix-output.1.xml"        # **/repomix-output.*  → packed
path="repomix-output.xml"          # **/repomix-output.*  → packed
path="yarn-error.log.old"          # **/yarn-error.log*   → packed

The repomix-output pair is the one that bites hardest: a repository with a committed previous pack anywhere in it embeds that pack into the new one, which is what those two lines exist to prevent.

It is not limited to the default list — any user pattern in that shape is inert, while the two neighbouring shapes work:

The change

The rewrite is scoped to directories by its own JSDoc ("Makes **/folder, **/folder/, and **/folder/**/* behave identically") and by the comment above it, but the guard only tests the prefix. This adds the missing shape test, so it fires only for rules that currently match nothing:

- if (pattern.startsWith('**/') && !pattern.includes('/**')) {
+ if (pattern.startsWith('**/') && !pattern.includes('/**') && !pattern.endsWith('*')) {

Verified against globby's own matcher rather than inferred: of the 86 default patterns 32 take this branch, and exactly the 5 above lose every match under it — **/node_modules/**-style directory rules and **/folder-style rules are untouched, which is why the failure is easy to miss.

After the change the same fixture packs src/a.ts, .gitignore, probe-xyz.old1, README.md only, and -i '**/probe-xyz*' now excludes the probe file. The new test pins the three built-in shapes; the existing **/bin / **/nested/folder / **/folder/** cases still assert the directory rewrite unchanged.

Deliberately out of scope: --split-output writes out.1.xml… beside out.xml, and getIgnorePatterns only adds the base output path, so a second split run re-packs the first run's parts. Different rule, different line — worth its own issue.

Checklist

• Run npm run test
• Run npm run lint