feat(cli): `cedar build` now also builds all packages #915

Tobbe · 2026-01-02T22:12:18Z

yarn cedar build used to only build the "web" and "api" ~~sides~~workspaces. Now it builds web, api and all packages

Output is a bit rough, but it works!

netlify · 2026-01-02T22:12:23Z

✅ Deploy Preview for cedarjs canceled.

Name	Link
🔨 Latest commit	`14fc1d9`
🔍 Latest deploy log	https://app.netlify.com/projects/cedarjs/deploys/6958a0eb127af60008f7c9ba

nx-cloud · 2026-01-02T22:15:08Z

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci

View your CI Pipeline Execution ↗ for commit 14fc1d9

Command	Status	Duration	Result
`nx run-many -t test --minWorkers=1 --maxWorkers=4`	✅ Succeeded	1m 50s	View ↗
`nx run-many -t build:pack --exclude create-ceda...`	✅ Succeeded	3s	View ↗
`nx run-many -t build`	✅ Succeeded	3m 25s	View ↗
`nx run-many -t test:types`	✅ Succeeded	12s	View ↗

☁️ Nx Cloud last updated this comment at 2026-01-03 05:10:24 UTC

nx-cloud · 2026-01-02T22:15:08Z

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci

View your CI Pipeline Execution ↗ for commit 54b0f1a

Command	Status	Duration	Result
`nx run-many -t test --minWorkers=1 --maxWorkers=4`	❌ Failed	1m 17s	View ↗
`nx run-many -t build`	✅ Succeeded	10s	View ↗
`nx run-many -t test:types`	✅ Succeeded	6s	View ↗
`nx run-many -t build:pack --exclude create-ceda...`	✅ Succeeded	7s	View ↗

☁️ Nx Cloud last updated this comment at 2026-01-02 22:21:21 UTC

nx-cloud · 2026-01-02T22:15:08Z

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci

View your CI Pipeline Execution ↗ for commit 54b0f1a

Command	Status	Duration	Result
`nx run-many -t test --minWorkers=1 --maxWorkers=4`	❌ Failed	1m 17s	View ↗
`nx run-many -t build`	✅ Succeeded	7s	View ↗
`nx run-many -t test:types`	✅ Succeeded	6s	View ↗
`nx run-many -t build:pack --exclude create-ceda...`	✅ Succeeded	7s	View ↗

☁️ Nx Cloud last updated this comment at 2026-01-02 22:21:25 UTC

nx-cloud · 2026-01-02T22:15:09Z

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci

View your CI Pipeline Execution ↗ for commit 54b0f1a

Command	Status	Duration	Result
`nx run-many -t test --minWorkers=1 --maxWorkers=4`	❌ Failed	1m 17s	View ↗
`nx run-many -t build:pack --exclude create-ceda...`	✅ Succeeded	7s	View ↗
`nx run-many -t build`	✅ Succeeded	7s	View ↗
`nx run-many -t test:types`	✅ Succeeded	6s	View ↗

☁️ Nx Cloud last updated this comment at 2026-01-02 22:22:23 UTC

greptile-apps · 2026-01-02T22:15:42Z

Greptile Summary

Extended cedar build to build all workspace packages in addition to web and api. The command now defaults to ['web', 'api', 'packages/*'] and builds packages concurrently using the concurrently library. The workspaces() utility was enhanced to return package names in multiple formats (directory name, relative path, and package.json name) to support various ways users might specify packages. Optimization added to skip expensive filesystem operations when only building default workspaces.

Key changes:

Default workspace parameter changed from ['web', 'api'] to ['web', 'api', 'packages/*']
Added package building step in buildHandler that runs before api/web builds
Packages are built concurrently with yarn build in each package directory
Workspace name validation enhanced to support package names in different formats
Performance optimization: skips workspaces() call when only default workspaces are specified

Confidence Score: 4/5

This PR is safe to merge with minor concerns about edge cases in package name resolution
The implementation is solid with proper error handling, concurrent building, and comprehensive test coverage. Score of 4 (not 5) due to the workspace path mapping logic that assumes directory names match the last segment of scoped package names - while this works for the current codebase structure and has existence checks as a safety net, it could fail for non-standard package directory naming
Pay attention to packages/cli/src/commands/buildHandler.js - verify the path mapping logic works with your package naming conventions

Important Files Changed

Filename	Overview
packages/cli/src/commands/build.js	updated default workspaces to include `packages/*` and added validation logic to filter known workspaces before expensive filesystem checks
packages/cli/src/commands/buildHandler.js	added package building logic with concurrency support, maps workspace names to filesystem paths with basic existence check

Sequence Diagram

sequenceDiagram
    participant User
    participant CLI as build.js
    participant Handler as buildHandler.js
    participant Project as project.js
    participant FS as File System
    participant Concurrently

    User->>CLI: cedar build [workspace..]
    CLI->>CLI: Parse workspace args
    
    alt workspace filtering needed
        CLI->>Project: workspaces({includePackages: true})
        Project->>FS: glob packages/*/package.json
        FS-->>Project: package paths
        Project->>FS: read package.json files
        FS-->>Project: package names
        Project-->>CLI: all valid workspace names
        CLI->>CLI: validate workspace args
    end
    
    CLI->>Handler: handler(options)
    Handler->>FS: read package.json workspaces
    FS-->>Handler: workspace config
    
    alt packages to build
        Handler->>FS: glob packages/*
        FS-->>Handler: all package paths
        
        alt specific packages requested
            Handler->>Handler: map workspace names to paths
            Handler->>FS: check path existence
            FS-->>Handler: path exists
        end
        
        Handler->>Concurrently: build packages in parallel
        Concurrently->>FS: yarn build (multiple packages)
        FS-->>Concurrently: build results
        Concurrently-->>Handler: completion status
    end
    
    alt api workspace included
        Handler->>Handler: verify schema & build API
    end
    
    alt web workspace included
        Handler->>Handler: build Web
    end
    
    Handler-->>User: build complete

greptile-apps

_{5 files reviewed, 2 comments}

_{Edit Code Review Agent Settings | Greptile}

packages/cli/src/commands/build.js

greptile-apps · 2026-01-02T22:15:41Z

packages/cli/src/commands/buildHandler.js

+          : restWorkspaces.map((w) => {
+              return path.join(cedarPaths.packages, w.split('/').at(-1))
+            })


logic: path mapping assumes directory name matches last segment of scoped package name. for @my-org/my-package, this creates path packages/my-package, but directory could actually be named differently (e.g., just the org name or a different structure)

check if the mapped path exists before using it, or verify the package.json name matches. are package directory names guaranteed to match the last segment of the package name in your workspace setup?

Prompt To Fix With AI

This is a comment left during a code review. Path: packages/cli/src/commands/buildHandler.js Line: 92:94 Comment: **logic:** path mapping assumes directory name matches last segment of scoped package name. for `@my-org/my-package`, this creates path `packages/my-package`, but directory could actually be named differently (e.g., just the org name or a different structure) check if the mapped path exists before using it, or verify the package.json name matches. are package directory names guaranteed to match the last segment of the package name in your workspace setup? How can I resolve this? If you propose a fix, please make it concise.

Tobbe · 2026-01-03T04:37:32Z

@greptileai Please do a full review again, and update your review summary post

greptile-apps

_{4 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

packages/cli/src/commands/buildHandler.js

feat(cli): cedar build now builds all packages

9c6fdca

github-actions bot added this to the next-release milestone Jan 2, 2026

explain filtering

54b0f1a

greptile-apps bot reviewed Jan 2, 2026

View reviewed changes

Tobbe changed the title ~~feat(cli): cedar build now builds all packages~~ feat(cli): cedar build now builds all workspaces Jan 2, 2026

Tobbe changed the title ~~feat(cli): cedar build now builds all workspaces~~ feat(cli): cedar build now also builds all packages Jan 2, 2026

Tobbe added 2 commits January 3, 2026 04:58

Merge branch 'main' into tobbe-feat-packages-workspace-cedar-build

c79b049

fix tests

9ab4383

Spead up build tests

2b2e326

greptile-apps bot reviewed Jan 3, 2026

View reviewed changes

packages/cli/src/commands/buildHandler.js Outdated Show resolved Hide resolved

Simplify mocks

14fc1d9

Tobbe merged commit f1894bc into main Jan 3, 2026
41 checks passed

Tobbe deleted the tobbe-feat-packages-workspace-cedar-build branch January 3, 2026 05:12

Tobbe mentioned this pull request Jan 3, 2026

chore(cli): Direct import of handler in tests for speedup #918

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(cli): `cedar build` now also builds all packages #915

feat(cli): `cedar build` now also builds all packages #915

Tobbe commented Jan 2, 2026 •

edited

Loading

Uh oh!

netlify bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

greptile-apps bot commented Jan 2, 2026 •

edited

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

Uh oh!

greptile-apps bot Jan 2, 2026

Uh oh!

Tobbe commented Jan 3, 2026

Uh oh!

greptile-apps bot left a comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

feat(cli): cedar build now also builds all packages #915

feat(cli): cedar build now also builds all packages #915

Conversation

Tobbe commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

netlify bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for cedarjs canceled.

Uh oh!

nx-cloud bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 Nx Cloud AI Fix

Uh oh!

nx-cloud bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 Nx Cloud AI Fix

Uh oh!

nx-cloud bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 Nx Cloud AI Fix

Uh oh!

nx-cloud bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🤖 Nx Cloud AI Fix

Uh oh!

greptile-apps bot commented Jan 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 4/5

Important Files Changed

Sequence Diagram

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

greptile-apps bot Jan 2, 2026

Choose a reason for hiding this comment

Uh oh!

Tobbe commented Jan 3, 2026

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

feat(cli): `cedar build` now also builds all packages #915

feat(cli): `cedar build` now also builds all packages #915

Tobbe commented Jan 2, 2026 •

edited

Loading

netlify bot commented Jan 2, 2026 •

edited

Loading

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

nx-cloud bot commented Jan 2, 2026 •

edited

Loading

greptile-apps bot commented Jan 2, 2026 •

edited

Loading