Skip to content

npm pkg fix changes#18

Closed
WGriffing wants to merge 5 commits into
kmtusher97:mainfrom
WGriffing:npm-pkg-fix
Closed

npm pkg fix changes#18
WGriffing wants to merge 5 commits into
kmtusher97:mainfrom
WGriffing:npm-pkg-fix

Conversation

@WGriffing

Copy link
Copy Markdown

No description provided.

WGriffing and others added 5 commits March 31, 2026 14:44
…ero dependencies

- Rename package to ya-repo-toc
- Add git-native file discovery via git ls-files with filesystem fallback
- Add --recurse-gitignore / -r flag for nested .gitignore support
- Add --prune-empty / -p flag to hide directories with no matching files
- Replace yargs with Node built-in util.parseArgs (zero runtime dependencies)
- Fix # in folder names breaking markdown links (upstream#8)
- Fix gitignore pattern matching (anchored, dirOnly, negation, ** globs)
- Fix test parallelization race conditions
- Add comprehensive gitignore pattern test coverage
- Add LICENSE with original author attribution
- Update README, CI workflows, and add CHANGELOG
- Require Node >= 18.3.0
- CI tests against Node 18, 20, 22
- Use OIDC trusted publishing for npm (no long-lived tokens)
Switch list markers from * to - and drop bold ** on directory names.
Eliminates the need for markdownlint carve-outs in consuming READMEs.
@WGriffing WGriffing closed this Apr 1, 2026
@kmtusher97 kmtusher97 reopened this Jul 16, 2026
Comment thread package.json
"name": "repo-toc",
"version": "1.3.0",
"description": "https://github.com/kmtusher97/repo-toc",
"name": "ya-repo-toc",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@WGriffing can we keep the name?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I had an urgent need to move forward, so I forked and published. I forgot I had opened this PR, sorry.

I can re-open another PR with just the relevant changes, but I substantially changed the behavior in the fork to suit my project's needs.

@kmtusher97

Copy link
Copy Markdown
Owner

@WGriffing thanks for the PR, sorry I missed the notification. Will review and merge it ASAP.

@WGriffing

Copy link
Copy Markdown
Author

Closing this as I forgot I had opened it. I can do a different, targeted PR instead. Or you can grab whatever you want from the other project too, if you prefer.

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