Skip to content

Remove integrations route ownership - #813

Merged
jeevanpillay merged 1 commit into
feat/www-start-core-infrafrom
feat/drop-integrations
Jun 5, 2026
Merged

Remove integrations route ownership#813
jeevanpillay merged 1 commit into
feat/www-start-core-infrafrom
feat/drop-integrations

Conversation

@jeevanpillay

Copy link
Copy Markdown
Member

Summary

  • Remove stale /integrations and /integrations/:path* ownership from apps/app/microfrontends.json.
  • Add negative assertions so the old www MFE table no longer claims integrations routes.

Stack

Test Plan

  • pnpm --filter @lightfast/app exec vitest run src/__tests__/microfrontends-config.test.ts
  • pnpm --filter @lightfast/app typecheck

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@vercel

vercel Bot commented Jun 4, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
lightfast-app Error Error Jun 4, 2026 8:05am
lightfast-mcp Ready Ready Preview, Comment Jun 4, 2026 8:05am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
lightfast-platform Skipped Skipped Jun 4, 2026 8:05am
lightfast-www Skipped Skipped Jun 4, 2026 8:05am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (2)
  • main
  • develop

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 7d1a97b2-7da5-4308-a297-8aa35b358274

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/drop-integrations
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch feat/drop-integrations

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

Base automatically changed from feat/drop-pitch-deck to feat/www-start-core-infra June 5, 2026 03:04
@jeevanpillay
jeevanpillay merged commit dfc5abc into feat/www-start-core-infra Jun 5, 2026
10 of 11 checks passed
@jeevanpillay
jeevanpillay deleted the feat/drop-integrations branch June 5, 2026 03:04

Copy link
Copy Markdown
Member Author

Folded this route-ownership cleanup into #811 so the migration lands as one PR.

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