feat: Update default ports for E2E applications to avoid conflicts #210

frontegg-david · 2026-01-15T00:57:45Z

Summary by CodeRabbit

Chores
- Added configurable auto-review base branches for main and release branches.
- Migrated and consolidated documentation paths into a unified live docs hierarchy with updated guidance.
- Added SDK Plugins documentation guidance.
- Updated default HTTP ports across demo apps.
- Adjusted publish scripts to include a "next" release tag.

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

coderabbitai · 2026-01-15T00:57:50Z

📝 Walkthrough

Walkthrough

Configuration: adds auto_review base_branches and migrates many documentation path_instructions to a new docs/live/** structure (plus a new plugins/** entry). Code: default HTTP ports updated across many e2e demo apps; e2e publish script now adds --tag next.

Changes

Cohort / File(s)	Summary
Configuration & Documentation Migration `\.coderabbit\.yaml`	Adds `reviews.auto_review.base_branches`; migrates documentation path_instructions and guidance from `docs/docs/`, `docs/v/`, `docs/draft/`, `docs/blogs/`, `docs/docs.json` etc. into `docs/live/`; adds new `plugins/` path_instructions and updates guidance text to reference `docs/live/*`.
E2E Demo Port Updates `apps/e2e/*/src/main.ts` (multiple) — e.g. `apps/e2e/demo-e2e-public/src/main.ts`, `apps/e2e/demo-e2e-agents/src/main.ts`, `apps/e2e/demo-e2e-transport-recreation/src/main.ts`, `apps/e2e/demo-e2e-standalone/src/main.ts`, `apps/e2e/demo-e2e-multiapp/src/main.ts`, `apps/e2e/demo-e2e-codecall/src/main.ts`, `apps/e2e/demo-e2e-redis/src/main.ts`, `apps/e2e/demo-e2e-cache/src/main.ts`, `apps/e2e/demo-e2e-ui/src/main.ts`, `apps/e2e/demo-e2e-remote/...`, `apps/e2e/demo-e2e-serverless/src/main.ts`, `apps/e2e/demo-e2e-direct/src/main.ts`, `apps/e2e/demo-e2e-providers/src/main.ts`, `apps/e2e/demo-e2e-remember/src/main.ts`, `apps/e2e/demo-e2e-errors/src/main.ts`, `apps/e2e/demo-e2e-notifications/src/main.ts`, `apps/e2e/demo-e2e-openapi/src/main.ts`, `apps/e2e/demo-e2e-config/src/main.ts`, `apps/e2e/demo-e2e-hooks/src/main.ts`, `apps/e2e/demo-e2e-orchestrated/src/main.ts`	Uniform updates to default HTTP port fallbacks (environment `PORT` fallback values changed across ~22 demo apps). No control-flow or error-handling changes.
E2E Publish Script `libs/cli/e2e/run-e2e.sh`	Adds `--tag next` to npm publish invocations (publishing paths for `libs/`, `plugins/`, and `libs/plugins/dist`).

Sequence Diagram(s)

(omitted — changes are configuration and port defaults without new multi-component control flow)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

fix(ci): Reversed version 0.3 due to bug in publishing workflow and fixed it #33 — related changes to doc-path handling and CI/publishing workflows; overlaps in doc path updates and publish procedure.
chore: Add e2e tests for mcp ui and enhance ui lib #129 — modifies the same e2e publish script and doc path migrations; likely closely related at code level.

"I hopped through ports and pages bright,
Docs moved to live, the mappings right.
Reviews now auto-find their lanes,
Publishes tagged for next-day trains.
A rabbit cheers with tiny bites of light 🐇✨"

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the primary change: updating default ports across 21 E2E application files to avoid conflicts, which is the dominant change in the PR.
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 docstrings

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9f73515 and 44645d6.

📒 Files selected for processing (1)

libs/cli/e2e/run-e2e.sh

🧰 Additional context used

📓 Path-based instructions (1)

libs/**

⚙️ CodeRabbit configuration file

libs/**: Contains publishable SDK libraries. Review for API correctness, breaking changes, and consistency with docs. When public APIs change, ensure there is a matching docs/draft/docs/** update (not direct edits under docs/docs/**).

Files:

libs/cli/e2e/run-e2e.sh

🧠 Learnings (3)

📚 Learning: 2026-01-04T14:35:18.366Z

Learnt from: CR
Repo: agentfront/frontmcp PR: 0
File: libs/uipack/CLAUDE.md:0-0
Timestamp: 2026-01-04T14:35:18.366Z
Learning: Applies to libs/uipack/**/index.{ts,js} : Export all public APIs through appropriate entry points (frontmcp/uipack, frontmcp/uipack/adapters, frontmcp/uipack/theme, etc.)

Applied to files:

libs/cli/e2e/run-e2e.sh

📚 Learning: 2026-01-04T14:35:18.366Z

Learnt from: CR
Repo: agentfront/frontmcp PR: 0
File: libs/uipack/CLAUDE.md:0-0
Timestamp: 2026-01-04T14:35:18.366Z
Learning: Applies to libs/uipack/**/{theme,adapters,bundler}/**/*.{test,spec}.{ts,tsx,js,jsx} : Test behavior across all supported platform configurations (OpenAI, Claude, etc.)

Applied to files:

libs/cli/e2e/run-e2e.sh

📚 Learning: 2026-01-06T02:34:55.689Z

Learnt from: CR
Repo: agentfront/frontmcp PR: 0
File: libs/plugins/CLAUDE.md:0-0
Timestamp: 2026-01-06T02:34:55.689Z
Learning: Applies to libs/plugins/**/*.ts : Use proper ES module imports instead of `require()` for SDK imports; avoid dynamic require of `frontmcp/sdk` modules

Applied to files:

libs/cli/e2e/run-e2e.sh

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (17)

GitHub Check: E2E Tests (demo-e2e-multiapp)
GitHub Check: E2E Tests (demo-e2e-cache)
GitHub Check: E2E Tests (demo-e2e-serverless)
GitHub Check: E2E Tests (demo-e2e-notifications)
GitHub Check: E2E Tests (demo-e2e-errors)
GitHub Check: E2E Tests (demo-e2e-hooks)
GitHub Check: E2E Tests (demo-e2e-config)
GitHub Check: E2E Tests (demo-e2e-ui)
GitHub Check: E2E Tests (demo-e2e-transparent)
GitHub Check: E2E Tests (demo-e2e-providers)
GitHub Check: E2E Tests (demo-e2e-redis)
GitHub Check: E2E Tests (demo-e2e-public)
GitHub Check: E2E Tests (demo-e2e-codecall)
GitHub Check: E2E Tests (demo-e2e-openapi)
GitHub Check: Unit Tests
GitHub Check: Lint & Format Checks
GitHub Check: Package E2E (Verdaccio)

🔇 Additional comments (1)

libs/cli/e2e/run-e2e.sh (1)

112-112: LGTM! Consistent --tag next addition across all publish commands.

Adding the --tag next flag is a reasonable practice for E2E testing, ensuring pre-release semantics even for the local Verdaccio registry. All three publish commands (libs, plugins, and meta-package) are now consistent.

Also applies to: 128-128, 143-143

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

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

… conflicts Cherry-picked from #210 (merged to release/0.8.x) Original commit: 5061fc0 Co-Authored-By: frontegg-david <69419539+frontegg-david@users.noreply.github.com>

github-actions · 2026-01-15T02:06:40Z

Cherry-pick Created

A cherry-pick PR to main has been automatically created.

Please review and merge if this change should also be in main.

If the cherry-pick is not needed, close the PR.

… conflicts (#211) Cherry-picked from #210 (merged to release/0.8.x) Original commit: 5061fc0 Co-authored-by: agentfront[bot] <agentfront[bot]@users.noreply.github.com> Co-authored-by: frontegg-david <69419539+frontegg-david@users.noreply.github.com>

feat: Update default ports for E2E applications to avoid conflicts

807a0e3

frontegg-david added 2 commits January 15, 2026 03:42

feat: Update paths in configuration for documentation and SDK plugins

9f73515

feat: Add --tag next option to npm publish commands in E2E script

44645d6

frontegg-david merged commit 5061fc0 into release/0.8.x Jan 15, 2026
22 checks passed

frontegg-david deleted the adjust-e2e-ports branch January 15, 2026 01:55

frontegg-david had a problem deploying to release January 15, 2026 01:55 — with GitHub Actions Error

github-actions bot mentioned this pull request Jan 15, 2026

Cherry-pick: feat: Update default ports for E2E applications to avoid conflicts #211

Merged

This was referenced Jan 15, 2026

feat: Set LOCAL_MCP_PORT in environment for local MCP server connectivity tests #214

Merged

Fix remote mcp test #216

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Update default ports for E2E applications to avoid conflicts #210

feat: Update default ports for E2E applications to avoid conflicts #210

Uh oh!

frontegg-david commented Jan 15, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Jan 15, 2026 •

edited

Loading

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Uh oh!

Uh oh!

github-actions bot commented Jan 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

feat: Update default ports for E2E applications to avoid conflicts #210

feat: Update default ports for E2E applications to avoid conflicts #210

Uh oh!

Conversation

frontegg-david commented Jan 15, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

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

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Uh oh!

Uh oh!

github-actions bot commented Jan 15, 2026

Cherry-pick Created

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

frontegg-david commented Jan 15, 2026 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 15, 2026 •

edited

Loading