Skip to content

Resolve standalone inline-manifest roots in file commands#571

Merged
akhilles merged 1 commit intomainfrom
fix/standalone-layout-path
Feb 24, 2026
Merged

Resolve standalone inline-manifest roots in file commands#571
akhilles merged 1 commit intomainfrom
fix/standalone-layout-path

Conversation

@akhilles
Copy link
Contributor

@akhilles akhilles commented Feb 23, 2026

Note

Low Risk
Small, mechanical change to how the resolve root is computed for several CLI/MCP entrypoints; main risk is unexpected workspace discovery differences when running commands from unusual directories.

Overview
Fixes dependency resolution for file-based commands when invoked on standalone .zen files (including inline manifests) by passing the actual .zen path into crate::resolve::resolve(...) instead of only the parent directory, across bom, layout, open, route, sim, and MCP run_layout.

Updates the changelog to note the resulting behavior change: Board(..., layout_path=...) now resolves to package://workspace/... rather than leaking absolute paths for standalone inline-manifest .zen files.

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


Open with Devin

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@akhilles akhilles merged commit 8e4f1fe into main Feb 24, 2026
12 checks passed
@akhilles akhilles deleted the fix/standalone-layout-path branch February 24, 2026 21:59
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