mpfaffenberger / mpfaffenberger/code_puppy
Repo hygiene: coverage.json (1.6MB) tracked again; .gitignore ends with broken '.json' entry; node_modules/ unignored
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:
-
coverage.json(1.6 MB) is tracked by git. It was removed once ina51f1cd5("chore: remove coverage.json artifact") but came back in5720fd84("test: 97.5% coverage for command_line modules"). It is a regenerated test artifact and bloats every clone. -
The last line of
.gitignoreis literally.json, which only ignores a file named.json— it does not ignorecoverage.json(which is why it got re-committed) and was almost certainly a typo forcoverage.json. Introduced inf1715e25.
# last lines of .gitignore
.claude/hooks/ts-hooks/dist/
.json # <-- matches nothing useful
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 accidentalgit add .from committing it. There's nopackage.jsonat 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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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