Skip to content

feat(cli): remove framework config and simplify build command#2895

Merged
matthewvolk merged 1 commit intoalphafrom
CATALYST-1780-rm-framework
Feb 20, 2026
Merged

feat(cli): remove framework config and simplify build command#2895
matthewvolk merged 1 commit intoalphafrom
CATALYST-1780-rm-framework

Conversation

@matthewvolk
Copy link
Copy Markdown
Contributor

What/Why?

The framework configuration concept (catalyst | nextjs) is no longer needed. The CLI will only be installed when using native hosting, so the nextjs framework path is a dead code path. This PR removes it entirely and simplifies the build command to always call buildCatalystProject() directly.

Changes

  • project-config.ts — Remove framework from ProjectConfigSchema and its Conf schema definition
  • build.ts — Remove .allowUnknownOption(), [next-build-options...] argument, --framework option, and the nextjs branch in the action handler; remove unused existsSync import
  • project.ts — Remove config.set('framework', 'catalyst') from create and link actions
  • Tests — Remove all framework-related assertions, mocks, and test cases across build.spec.ts, project.spec.ts, and project-config.spec.ts

Testing

  • vitest run — all tests pass (2 pre-existing failures in index.spec.ts and version.spec.ts are unrelated)
  • eslint . — clean
  • tsc --noEmit — clean

Migration

The framework key in .bigcommerce/project.json is no longer read or written. Existing config files with this key will be silently ignored by conf.

@matthewvolk matthewvolk requested a review from a team February 20, 2026 16:31
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 20, 2026

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

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Feb 20, 2026 5:22pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 20, 2026

🦋 Changeset detected

Latest commit: 3970f55

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst Minor

Not sure what this means? Click here to learn what changesets are.

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

Comment thread packages/catalyst/src/cli/lib/project-config.ts
@matthewvolk matthewvolk force-pushed the CATALYST-1780-rm-framework branch from 3cd91f7 to 3970f55 Compare February 20, 2026 17:20
@matthewvolk matthewvolk merged commit 8794db1 into alpha Feb 20, 2026
8 checks passed
@matthewvolk matthewvolk deleted the CATALYST-1780-rm-framework branch February 20, 2026 17:29
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.

4 participants