feat: add configurable dependency trace root - #7
Draft
yimingjfe wants to merge 6 commits into
Draft
Conversation
Keep tracing, path restoration, and dependency classification on one canonical root while preserving non-base nft option overrides.
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.
Summary
traceRootoption tonodeDepEmitappDirwhile preserving the existing/defaultbaseowned by nde while forwarding other nft options and filesystem hooksMotivation
A configurable trace boundary lets callers narrow nft dependency analysis instead of always tracing from the filesystem root. Keeping tracing and path restoration on the same base prevents nft's relative results from being resolved against a different root.
Scope
This change configures nft's base and keeps nde path restoration and dependency classification aligned with it. Entry discovery, package lookup, output paths, and existing hook semantics remain unchanged.
Validation
pnpm testpnpm buildgit diff --check