Skip to content

[Backport 5.x] chore: npm ignore remaining agent files - #7003

Merged
mcollina merged 1 commit into
5.xfrom
backport-6996-to-5.x
Sep 1, 2026
Merged

mcollina merged 1 commit into
5.xfrom
backport-6996-to-5.x

Conversation

@github-actions

Copy link
Copy Markdown

Backport 4cdb0c5 from #6996.

The published tarball for fastify 5.12.1 ships a
`.pi/self-learning-memory/` directory of local agent notes. `.gitignore`
already lists the agent files under "Agents files", but `.npmignore`
only covers `.claude/` and `CLAUDE.md`, and npm stops reading
`.gitignore` once an `.npmignore` exists.

This adds the remaining entries so `AGENTS.md`, `.agent`, `.agents/` and
`.pi/` stay out of the package. None of them are tracked in the repo, so
`npm pack --dry-run` on a clean checkout shows no change. With the files
present in the working tree, which is how they ended up in the 5.12.1
tarball, they are listed before the change and gone after it, and
nothing else drops out of the file list.

`5.x` has the same gap, so this probably wants a `backport 5.x` label on
merge.

Closes #6995

#### Checklist

- [x] commit message and PR title follow [conventional
commits](https://www.conventionalcommits.org/en/v1.0.0/)

(cherry picked from commit 4cdb0c5)

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Tony133
Tony133 requested a review from a team September 1, 2026 08:18
@mcollina
mcollina merged commit dd2ba36 into 5.x Sep 1, 2026
35 checks passed
@mcollina
mcollina deleted the backport-6996-to-5.x branch September 1, 2026 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants