Repository navigation
rc/v1.2.0 - #1
Merged
Merged
Conversation
- Fix progress symlink counting - Add regression test - Update changelog - Update release documentation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added
--versionflag —tree2guide --versionnow prints the version insteadof crashing with a missing-argument error
.claude/,.cursor/,.windsurf/,.roo/,.specify/,.roomodes,.maestro/,.githooks/in additionto the previously-existing
CLAUDE.md,AGENTS.md,.cursorrulesPHP MVC structurestack signal —Controller/,Model/,View/are recognized as a weak, generic MVC-framework signal withoutbeing misattributed to CakePHP specifically, since these directory names
are shared by CodeIgniter, Yii, and other PHP MVC frameworks
Scanning... N files, N dirsand a finalScan complete.summary,printed to stderr only (never stdout, so
--stdoutpiping isunaffected). Gated on elapsed wall-clock time, not entry count, so fast
scans of small projects print nothing extra.
--no-progresssuppressesthis entirely.
build_node_tree()gained an optionalon_progresscallback parameter (
Noneby default, zero effect on existing callers)--llmoutput now usecomma separators for readability on large projects (
227886→227,886)Fixed
_STACK_SIGNALSpatterns containing a path separator (e.g.bin/cake,config/app.php,bootstrap/app.php,bin/console,config/routes.rb,src/main/java, and 9 others) could never match anything, because stackdetection only ever compared against bare file/directory names, never
relative paths. These are now matched against each entry's actual
relative path, restoring the CakePHP, Laravel, Symfony, Ruby on Rails,
Java/Kotlin, and Flutter signals that depend on them — several of which
are the highest-confidence (weight 5) signal for their framework
cli.pyusedstr | None/list[str] | Nonetype-hint syntax withoutfrom __future__ import annotations, which would raiseTypeErroronPython 3.9 the moment the module was imported. No test previously
imported
tree2guide.cli, so this went undetected by CI despite the 3.9job reporting green; new CLI tests now cover this module directly
the final completion summary and
--llm's SIZE section both excludedthem — the same long-standing convention used since v1.0.0. On a
project with many symlinks (e.g. Python venvs,
node_modules/.bin),the live progress numbers could run noticeably ahead of the final
count. Progress now excludes symlinks, matching every other counting
function in the codebase