fix: add @calcom/api-v2 to dev:api script

[Vitalini]

Problem

The dev:api script in the root package.json starts the web app, API proxy, and API v1 via Turborepo filters, but does not include @calcom/api-v2. Developers working on API v2 need to start it separately.

Solution

Add --filter="@calcom/api-v2" to the dev:api turbo command:

- "dev:api": "turbo run dev --filter=\"@calcom/web\" --filter=\"@calcom/api-proxy\" --filter=\"@calcom/api\"",
+ "dev:api": "turbo run dev --filter=\"@calcom/web\" --filter=\"@calcom/api-proxy\" --filter=\"@calcom/api\" --filter=\"@calcom/api-v2\"",

This ensures both API v1 and v2 start together during local development.

Fixes #28736

[cubic-dev-ai]

No issues found across 1 file

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a90decd3-c5da-4dc4-88fe-59b916da5dc3

📥 Commits

Reviewing files that changed from the base of the PR and between 3734ea6 and e4116e1.

📒 Files selected for processing (1)

• apps/api/index.js

---

📝 Walkthrough

Walkthrough

Updated monorepo dev scripts to include the @calcom/api-v2 workspace in turbo run filters. In the API app, hard-coded ports were replaced with environment-driven constants (PROXY_PORT, API_V1_PORT, API_V2_PORT), proxy targets now use those ports, the v2 proxy is mounted before the root proxy, and the server listens on PROXY_PORT.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	❓ Inconclusive	The changes to apps/api/index.js (replacing hard-coded ports with environment variables) appear to be scope-related infrastructure improvements supporting the v2 server integration, though this goes slightly beyond the simple filter addition described in the PR title.	Clarify whether the port configuration changes in apps/api/index.js are necessary for the v2 integration or represent scope creep; consider splitting into separate PRs if the port changes are unrelated improvements.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title clearly describes the main change: adding @calcom/api-v2 to the dev:api script, which matches the primary objective of the changeset.
Description check	✅ Passed	The description explains the problem and solution, directly relating to the changes made in package.json and apps/api/index.js to include @calcom/api-v2 in development scripts.
Linked Issues check	✅ Passed	The PR addresses issue #28736 by adding @calcom/api-v2 filter to dev:api and dev:api:console scripts in package.json, and updates apps/api/index.js to use environment-driven configuration for ports to support the v2 server startup.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

[Vitalini]

Thanks for the ping. This PR is ready from my side and still relevant. Please let me know if you’d like any changes, and I can update it promptly.

[romitg2]

@Vitalini please resolve merge conflicts

[Vitalini]

@romitg2 rebased onto latest main and resolved conflicts in package.json. The final diff now only adds @calcom/api-v2 filter to dev:api and dev:api:console scripts (upstream had already removed the @calcom/api filter). Ready for another look 🙏

[CLAassistant]

All committers have signed the CLA.

[Vitalini]

Thanks @romitg2 — really nice catch on the middleware ordering. With apiProxyV1 mounted at / ahead of /v2, every /v2/* request was being shadowed by the v1 proxy and never reaching the v2 mount, so dev:api adding @calcom/api-v2 would have started the process but no traffic would have flowed to it. The env-driven ports are a nice cleanup on top of that — makes the multi-API local setup much easier to override per machine.

Glad to keep this moving — let me know if there's anything you'd like me to do on my side.

[Vitalini]

Quick status update: the CLA is now signed (see the CLAassistant comment above — signed badge). The remaining required check is the external-PR CI gate that needs the run-ci label from a maintainer to trigger. Once that's applied the rest of the workflow should run end-to-end. Thanks for the patience!

[github-actions]

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.