Skip to content

chore: update dev script to run api, web, and desktop#434

Merged
Kitenite merged 1 commit into
mainfrom
chore/update-dev-script
Dec 19, 2025
Merged

chore: update dev script to run api, web, and desktop#434
Kitenite merged 1 commit into
mainfrom
chore/update-dev-script

Conversation

@Kitenite
Copy link
Copy Markdown
Collaborator

@Kitenite Kitenite commented Dec 19, 2025

Summary

  • Update bun dev to run desktop, api, and web apps instead of just desktop
  • Remove redundant dev:app script (was identical to the new dev script)

Test plan

  • Run bun dev and verify api, web, and desktop apps all start

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated development workflow scripts to streamline initialization of multiple project components (API, Web, Desktop) in a single command.
    • Added dedicated development scripts for CLI and website builds.
    • Removed legacy app development script.

✏️ Tip: You can customize this high-level summary in your review settings.

- Change `bun dev` to run desktop, api, and web apps instead of just desktop
- Remove redundant `dev:app` script

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 19, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request modifies the development scripts in package.json. The dev script now includes filters for multiple packages (@superset/api, @superset/web, @superset/desktop), two new scripts (dev:cli and dev:website) are added, and the dev:app script is removed.

Changes

Cohort / File(s) Summary
Development Scripts Configuration
package.json
Updated dev script to include --filter=@superset/api --filter=@superset/web --filter=@superset/desktop; removed dev:app script; added dev:cli and dev:website scripts

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Single file change with straightforward script configuration updates
  • No logic or behavioral modifications

Possibly related PRs

  • add bun dev:app to run dev #378: Directly related—PR #378 adds a dev:app script with the same turbo filters that this PR consolidates into the main dev script, creating a conflict in approach.

Poem

🐰 A rabbit hops through scripts with glee,
Dev commands now unified, you see!
dev:cli and dev:website spring to life,
While dev:app bids farewell to strife,
Turbo filters dance in harmony! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/update-dev-script

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1655b32 and 0d004ae.

📒 Files selected for processing (1)
  • package.json (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Kitenite Kitenite merged commit fb7d46e into main Dec 19, 2025
10 of 11 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 19, 2025

🚀 Preview Deployment

🔗 Preview Links

Service Status Link
Neon Database (Neon) View Branch
Vercel API (Vercel) Open Preview
Vercel Web (Vercel) Open Preview
Vercel Marketing (Vercel) Open Preview
Vercel Admin (Vercel) Open Preview
Vercel Docs (Vercel) Open Preview

Preview updates automatically with new commits

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