Skip to content

chore: remove accidentally committed .paper-nms/ directory and add to .gitignore#2942

Merged
tastybento merged 2 commits into
developfrom
copilot/fix-remove-paper-nms-directory
Apr 14, 2026
Merged

chore: remove accidentally committed .paper-nms/ directory and add to .gitignore#2942
tastybento merged 2 commits into
developfrom
copilot/fix-remove-paper-nms-directory

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 14, 2026

.paper-nms/ is a local build artifact generated by the paper-nms-maven-plugin that was accidentally committed (mappings_1.21.11.tiny.missing). It should never be tracked.

Changes

  • .gitignore — adds .paper-nms/ under a new # Paper NMS section
  • .paper-nms/1.21.11/mappings_1.21.11.tiny.missing — removed from git tracking (git rm --cached); local file preserved
Original prompt

Problem

The .paper-nms/ directory has been accidentally committed to the repository. Specifically, .paper-nms/1.21.11/mappings_1.21.11.tiny.missing (an empty marker file) is tracked by git. This directory is generated by the paper-nms-maven-plugin and is a local build artifact — it should never be committed.

The root cause is that .paper-nms/ is not listed in the .gitignore.

Required Changes

  1. Add .paper-nms/ to .gitignore — add it under an appropriate section (e.g., Maven or a new "Paper NMS" section).
  2. Remove the committed .paper-nms/ directory from git tracking — delete the files from the repository so they are no longer tracked. The files to remove are:
    • .paper-nms/1.21.11/mappings_1.21.11.tiny.missing

The local files do not need to be deleted from disk (i.e., this is effectively a git rm --cached operation), but since this is a file-based PR, simply removing the file from the repository tree is sufficient.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: This @BentoBoxWorld/BentoBox/files/.paper-nms/1.21.11 should not be in the repo afaik

Assistant: You're right — this file should not be committed to the repo. Here's what's going on:

What it is

The folder .paper-nms/1.21.11/ contains a single file:

  • mappings_1.21.11.tiny.missing (empty, 0 bytes)

This is a cache/marker file generated by the paper-nms-maven-plugin, which is used to remap Minecraft NMS classes for Paper plugins. These are local build artifacts and should not be committed.

The problem

.paper-nms/ is not listed in the .gitignore, which is why it got accidentally committed.

Fix

Two things need to happen:

  1. Add .paper-nms/ to .gitignore
  2. Remove the committed file from the repo's history (or at minimum, untrack it)

Would you like me to open a PR that:

  • Adds .paper-nms/ to .gitignore
  • Removes the committed .paper-nms/ directory from tracking (via git rm --cached)?

User: yes

Assistant: We're waiting for your response to the tool call dialog to continue creating the PR.

This pull request was created from Copilot chat.

Copilot AI changed the title [WIP] Remove .paper-nms/ directory from tracking and update .gitignore chore: remove accidentally committed .paper-nms/ directory and add to .gitignore Apr 14, 2026
Copilot AI requested a review from tastybento April 14, 2026 11:22
@tastybento tastybento marked this pull request as ready for review April 14, 2026 11:41
@tastybento tastybento merged commit 9080549 into develop Apr 14, 2026
@tastybento tastybento deleted the copilot/fix-remove-paper-nms-directory branch April 19, 2026 12:49
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