Skip to content

Plumb resolution through to eval config and output#524

Merged
akhilles merged 5 commits intomainfrom
wrap-info
Feb 11, 2026
Merged

Plumb resolution through to eval config and output#524
akhilles merged 5 commits intomainfrom
wrap-info

Conversation

@akhilles
Copy link
Contributor

@akhilles akhilles commented Feb 11, 2026

Note

Medium Risk
Touches core module loading/path resolution and propagates a new resolution/config object through eval, LSP, CLI, and tests, so regressions would primarily show up as broken load() resolution or stale caching across workspaces.

Overview
Evaluation now owns load resolution: EvalContextConfig stores file_provider, ResolutionResult, and a shared path_to_spec map, and EvalOutput carries the full config instead of a LoadResolver (removing CoreLoadResolver/LoadResolver APIs and updating all resolve_path/resolve_spec call sites).

Dependency resolution is promoted into core via a new ResolutionResult (with key canonicalization and package_closure), and workspace discovery now records a cache_dir via FileProvider::cache_dir; CLI/LSP/WASM code is refit to pass ResolutionResult into eval, cache resolutions per-workspace, and use resolution.workspace_info for file walking/vendoring/printing (print_dep_tree) and layout discovery call sites are adjusted accordingly.

Written by Cursor Bugbot for commit 1d51068. This will update automatically on new commits. Configure here.

@akhilles akhilles requested a review from LK February 11, 2026 16:53
@akhilles akhilles marked this pull request as ready for review February 11, 2026 16:53
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@akhilles akhilles merged commit eb3b9f4 into main Feb 11, 2026
12 checks passed
@akhilles akhilles deleted the wrap-info branch February 11, 2026 19:23
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