Repository navigation
sprint: dependency resolution - #21
Merged
Merged
Conversation
Both HIGH findings resolved: 1. addDependents now uses merge/dedup semantics with multi-parent test 2. _cascade flag fully defined with shallow-cascade depth rule All 12 checklist items pass. Plan approved for execution.
- Task 2.1: community-index.js confirmed from Sprint 2 — fetchCommunityIndex and getStaleCommunityIndexCache already exported, no change needed - Task 2.2: resolver.js — buildPackageMap (official + community, community wins), DFS with cycle/optional/unknown handling, getInstalled loaded once per resolve() - Task 2.3: resolver.test.js — 9 test cases covering all resolver scenarios; 273/273 tests pass
- Wire remove.js: check pkg.dependents before removal; show select prompt (Cancel / Cascade / Force) when dependents exist and !_cascade - Cascade: removes each dependent with _cascade:true, then removes target - Force: removes only target file + trackRemove + removeDependentEdge per dependent - _pruneOrphans: fires after any successful remove; prompts or auto-prunes with --yes - Extend tests/remove.test.js with 6 new cases covering all paths; 287/287 pass
dsiddharth2
force-pushed
the
sprint/dep-resolution
branch
from
April 17, 2026 06:54
7b6b013 to
da6ec5a
Compare
…package resolution This commit fixes the following issues: 1. Support for left/right arrow navigation in the TUI installation scope selector. 2. Resolution of community packages during installation by including community index in findPackage/findAllPackages. 3. Robust handling of download paths for community packages, supporting custom entry points even when meta.json is missing.
…l packages Allows packages to specify a 'files' array in meta.json or community index to download multiple assets (scripts, prompts, etc.) during installation.
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
installed_as/dependencies/dependents, DFS resolver, and community-index dep lookup.Test plan