fix: rewrite v1.4.0 changelog and add license scan allowlist#15
Merged
carlos-alm merged 2 commits intomainfrom Feb 22, 2026
Merged
fix: rewrite v1.4.0 changelog and add license scan allowlist#15carlos-alm merged 2 commits intomainfrom
carlos-alm merged 2 commits intomainfrom
Conversation
…, multi-repo MCP roadmap - Node types: Go struct→struct, Rust struct/enum/trait, Java enum, C# struct/record/enum, PHP trait/enum, Ruby module — in both WASM and native Rust extractors - Add SYMBOL_KINDS constant and update all kind IN filters across queries, builder, export, embedder, cycles, watcher, and MCP - Add GitNexus column to README comparison table - Add Phase 2.5 Multi-Repo MCP to ROADMAP - Add PreToolUse hooks for Read/Grep context enrichment via codegraph deps - Update CLAUDE.md and README with new node kind documentation BREAKING CHANGE: Node kinds changed for structs, enums, traits, records, and modules. Rebuild with `codegraph build --no-incremental`.
Rewrite CHANGELOG.md with correct categorization (features, testing, docs, CI/CD, bug fixes) instead of dumping everything under bug fixes. Add explicit package allowlist to license compliance workflow for @img/sharp-* (dual-licensed Apache-2.0 + LGPL-3.0 via libvips). New LGPL packages will still be flagged — only reviewed entries pass.
Contributor
Greptile SummaryThis PR recovers two sets of changes lost when the Major changes:
All changes are well-structured and backwards-compatible (database schema unchanged, just kind values). Confidence Score: 5/5
Important Files Changed
Last reviewed commit: 9fbb084 |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
3 tasks
carlos-alm
pushed a commit
that referenced
this pull request
Feb 24, 2026
Add readFileSafe helper that retries on transient OS errors (EBUSY/EACCES/EPERM) from editors performing non-atomic saves. Replace bare readFileSync calls in builder.js (3 sites) and watcher.js (1 site). Add symlink loop detection to collectFiles via realpathSync tracking of visited directories, preventing infinite recursion from circular symlinks. Update architecture.md sections #7, #15, and summary table to reflect these fixes. Impact: 5 functions changed, 5 affected
3 tasks
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
1 participant
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.
Summary
Re-applies 2 fixes that were lost when the
release/v1.4.0branch was deleted after PR #13 merged (the commits were pushed after the merge).LICENSE_ALLOWLISTvariable for@img/sharp-*(dual-licensedApache-2.0 AND LGPL-3.0-or-latervia libvips). Uses package-name matching instead of a broad regex, so new LGPL deps will still be caughtTest plan
npm run lint— clean@img/sharp-*excluded, with justification)