mpfaffenberger / mpfaffenberger/code_puppy

Repo hygiene: coverage.json (1.6MB) tracked again; .gitignore ends with broken '.json' entry; node_modules/ unignored

Open Beginner friendly
#402 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
814
Forks
278
Avg merge
2d 5h
Merged PRs (30d)
76

Description

Problem

Three repo-hygiene problems verified via git ls-files / git check-ignore:

  1. coverage.json (1.6 MB) is tracked by git. It was removed once in a51f1cd5 ("chore: remove coverage.json artifact") but came back in 5720fd84 ("test: 97.5% coverage for command_line modules"). It is a regenerated test artifact and bloats every clone.

  2. The last line of .gitignore is literally .json, which only ignores a file named .json — it does not ignore coverage.json (which is why it got re-committed) and was almost certainly a typo for coverage.json. Introduced in f1715e25.

# last lines of .gitignore
.claude/hooks/ts-hooks/dist/

.json        # <-- matches nothing useful
  1. node_modules/ (4.6 MB, containing docx/jszip/pako/etc.) sits at the repo root with no ignore entry. It's currently untracked, but nothing prevents an accidental git add . from committing it. There's no package.json at the root either, so it appears to be leftover tooling debris.

Untracked-but-ignored junk also present locally (.coverage at root and code_puppy/.coverage, .DS_Store files) — those are correctly covered by .gitignore lines 12 and 24, no action needed. code_puppy/models_dev_api.json (535 KB) is tracked deliberately as wheel shared-data per pyproject.toml, so it is not flagged here.

Suggested fix

git rm --cached coverage.json

And in .gitignore, replace the trailing .json line:

-.json
+coverage.json
+node_modules/

Filed by Zen Reviewer C (code-puppy-60635a)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting the repository root .gitignore and verify the reported state with git ls-files and git check-ignore. Remove the tracked coverage.json entry and ensure coverage.json and node_modules/ are ignored; done means coverage.json is no longer tracked and both artifact paths are covered by .gitignore.

Written by the indexing model from the issue text.

Assessment

Tech stack
git
Domain
tooling
Issue type
Refactor
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
88/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.