Skip to content

dev#866

Merged
johnyeocx merged 1 commit intomainfrom
dev
Mar 3, 2026
Merged

dev#866
johnyeocx merged 1 commit intomainfrom
dev

Conversation

@johnyeocx
Copy link
Collaborator

@johnyeocx johnyeocx commented Mar 3, 2026

Summary by cubic

Include packages/ksuid/package.json in the Docker build so the ksuid workspace is installed and layer caching works. Fixes Docker builds failing when ksuid is required.

Written for commit 3ffa4eb. Summary will update on new commits.

Greptile Summary

This PR adds the new packages/ksuid workspace package to the Dockerfile so its package.json is copied during the dependency-install layer, consistent with how all other workspace packages are handled in the monorepo build.

Key Changes:

  • [Improvements] docker/Dockerfile: Adds COPY packages/ksuid/package.json ./packages/ksuid/package.json to the pre-install layer, ensuring the new zero-dependency @autumn/ksuid package is included in the bun install step and Docker layer caching works correctly for the full workspace.

Confidence Score: 5/5

  • This PR is safe to merge — it is a minimal, correct Dockerfile update with no functional risk.
  • The change is a single line that follows an established, consistent pattern already present in the Dockerfile. The referenced package (packages/ksuid) exists in the repo, has no external dependencies, and the copy order is correct relative to bun install.
  • No files require special attention.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["FROM oven/bun:1.3.2"] --> B["COPY root package.json + bun.lock"]
    B --> C["COPY all workspace package.json files\n(server, shared, vite, scripts, apps/checkout,\npackages/autumn-js, packages/openapi,\npackages/ksuid ✅ NEW,\npackages/sdk, apps/sdk-test, apps/docs)"]
    C --> D["RUN bun install --ignore-scripts\n(cached layer — invalidated only when a package.json changes)"]
    D --> E["COPY . .\n(full source)"]
    E --> F["WORKDIR /app/server\nCMD bun start"]
Loading

Last reviewed commit: 3ffa4eb

@railway-app railway-app bot temporarily deployed to autumn / development March 3, 2026 10:34 Inactive
@johnyeocx johnyeocx requested a review from ay-rod as a code owner March 3, 2026 10:34
@vercel
Copy link

vercel bot commented Mar 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
autumn-vite (dev) Ready Ready Preview, Comment Mar 3, 2026 10:35am
autumn-vite Building Building Mar 3, 2026 10:35am

Request Review

@johnyeocx johnyeocx merged commit 593a2f7 into main Mar 3, 2026
7 of 12 checks passed
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

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.

1 participant