Skip to content

Fixed dockerfile - #527

Merged
chiragjn merged 2 commits into
mainfrom
fix_build
Aug 31, 2026
Merged

Fixed dockerfile#527
chiragjn merged 2 commits into
mainfrom
fix_build

Conversation

@innoavator

@innoavator innoavator commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #

Changes

How was this tested?

Checklist

  • I have read the contributing guidelines
  • pnpm build, pnpm test, pnpm typecheck, pnpm lint:ci, and pnpm format:check pass locally
  • Tests added/updated where it makes sense
  • No hand-edits to generated code (packages/trueforge-sdk, .github/fern/openapi/openapi.json, docs/openapi.json) — fork PRs omit SDK regen; maintainers regenerate after merge
  • Docs / .env.example updated if configuration or behavior changed

Note

Low Risk
Docker dev build ordering and artifact copy only; no application logic or runtime behavior changes beyond fixing a broken image build.

Overview
Fixes dev Docker image builds when @truefoundry/trueforge depends on the workspace @truefoundry/trueforge-sdk (e.g. schedule dispatch).

The builder stage now copies packages/trueforge-sdk, runs pnpm --filter @truefoundry/trueforge-sdk build before compiling trueforge-core and the server, with a comment documenting that ordering requirement.

The runner stage now includes the built SDK (package.json + dist) alongside trueforge-core so production node_modules symlinks resolve at runtime.

Reviewed by Cursor Bugbot for commit 7f1bf52. Bugbot is set up for automated code reviews on this repo. Configure here.

@changeset-bot

changeset-bot Bot commented Aug 31, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 7f1bf52

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chiragjn
chiragjn merged commit b3f541b into main Aug 31, 2026
6 checks passed
@chiragjn
chiragjn deleted the fix_build branch August 31, 2026 19:56
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