Skip to content

fix: Make canvas paths docroot-aware in host commands - #72

Merged
penyaskito merged 3 commits into
drupal-canvas:mainfrom
goba:fix/docroot-aware-canvas-paths
Aug 31, 2026
Merged

fix: Make canvas paths docroot-aware in host commands#72
penyaskito merged 3 commits into
drupal-canvas:mainfrom
goba:fix/docroot-aware-canvas-paths

Conversation

@goba

@goba goba commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Derive the canvas module location from composer installer-paths so xb-setup, xb-cypress, and xb-playwright work with both docroot=web and docroot="" projects (including Drupal core-style checkouts with drupal-dev). Also update README setup/usage wording to reflect docroot-agnostic behavior.

LLM assistance was used to build this PR

Derive the canvas module location from composer installer-paths so xb-setup, xb-cypress, and xb-playwright work with both docroot=web and docroot="" projects (including Drupal core-style checkouts with drupal-dev). Also update README setup/usage wording to reflect docroot-agnostic behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Comment thread commands/host/xb-cypress Outdated
Comment thread commands/host/xb-cypress Outdated
Move docroot-aware canvas path resolution into a shared host helper script and reuse it from xb-setup, xb-cypress, and xb-playwright. This removes duplicated logic and keeps path derivation consistent across commands.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@amateescu amateescu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks much better!

Keeps the shared helper in sync on add-on upgrades, matching the marker
convention DDEV's upgrade path and post_install_actions cleanup key off.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@penyaskito
penyaskito merged commit 20d72cd into drupal-canvas:main Aug 31, 2026
2 checks passed
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.

3 participants