Skip to content

fix: revert canvas-builder from Dockerfile, restore publish-hostless pipeline#140

Open
jameswillis99 wants to merge 2 commits intomasterfrom
fix-canvas-builder-dockerfile
Open

fix: revert canvas-builder from Dockerfile, restore publish-hostless pipeline#140
jameswillis99 wants to merge 2 commits intomasterfrom
fix-canvas-builder-dockerfile

Conversation

@jameswillis99
Copy link
Collaborator

Summary

  • Removes the canvas-builder stage from the WAB Dockerfile (was causing makePkgTypes.js build failures)
  • Restores Dockerfile.publish-hostless which layers canvas-packages on top of the deployed WAB image
  • Restores the publish-hostless workflow to build canvas-packages in CI and run the ECS task
  • Replaces terraform network config lookups with GitHub environment variables (PRIVATE_SUBNET_IDS, ECS_SECURITY_GROUP_ID)

Test plan

  • Merge to master and verify deploy-integration workflow builds and pushes WAB image successfully
  • Manually trigger publish-hostless workflow and verify it builds canvas-packages, builds the publish-hostless image, and runs the ECS task to completion
  • Confirm PRIVATE_SUBNET_IDS and ECS_SECURITY_GROUP_ID are set in the integration GitHub environment

🤖 Generated with Claude Code

jwillis24 and others added 2 commits February 18, 2026 16:02
…es yarn install

The canvas-packages prepare script runs makePkgTypes.js during yarn install.
Both makePkgTypes.js and typedPkgsList.json must be present before yarn install
runs, otherwise the postinstall step fails with MODULE_NOT_FOUND.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…pipeline

Remove the canvas-builder stage from the WAB Dockerfile and restore the
Dockerfile.publish-hostless approach. The publish-hostless workflow now
builds canvas-packages in CI, layers them on top of the deployed WAB image
via Dockerfile.publish-hostless, and runs the ECS task.

Network config now comes from GitHub environment variables (PRIVATE_SUBNET_IDS,
ECS_SECURITY_GROUP_ID) rather than terraform outputs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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

Comments