Repository navigation
feat(gateway): warn that forest-mcp-server and forest-bff are deprecated - #1990
Merged
nbouliol merged 1 commit intoOct 8, 2026
Conversation
1 new issue
|
|
Coverage Impact This PR will not change total coverage. Modified Files with Diff Coverage (1)
🛟 Help
|
Tonours
approved these changes
Oct 8, 2026
Tonours
left a comment
Member
There was a problem hiding this comment.
Approved after /validator run 1990-20261008132500-Tonours: nothing at or above should-fix posted, 0 suppressed.
Base automatically changed from
feature/prd-1338-unify-the-standalone-gateway-configuration-on-one-set-of-env
to
feature/prd-1337-create-the-forestadmingateway-package-that-serves-both
October 8, 2026 13:38
nbouliol
marked this pull request as ready for review
October 8, 2026 13:41
Both bins log one line at boot pointing to forest-gateway; READMEs flag both packages as internal building blocks of @forestadmin/gateway. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
nbouliol
force-pushed
the
feature/prd-1342-deprecate-the-standalone-bins
branch
from
October 8, 2026 13:42
3e8e284 to
a2ad56c
Compare
nbouliol
merged commit Oct 8, 2026
f65b49f
into
feature/prd-1337-create-the-forestadmingateway-package-that-serves-both
43 checks passed
Tonours
pushed a commit
that referenced
this pull request
Oct 12, 2026
* feat(gateway): serve the MCP and the API on one port Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ci: lint and test the gateway package Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(gateway): build against the workspace agent-bff and mcp-server Exact pins fall behind the workspace on each release, since a private package is skipped by multi-semantic-release, and Yarn then pulls the published versions instead of linking the local ones. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(mcp-server): export the listener variables parseMcpEnv reads Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(gateway): follow the listener variable names agent-bff and mcp-server own A rename on their side now fails the gateway's typecheck, or carries over through the exported name, instead of silently moving the API's public url and port. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(agent-bff): export the openapi --output handling for other CLIs Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * refactor(gateway): reuse agent-bff's openapi --output handling Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * feat(gateway): read one set of env vars, legacy names as aliases (#1980) * feat(gateway): warn that forest-mcp-server and forest-bff are deprecated (#1990) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
forest-mcp-serverandforest-bfflog one warning at boot namingforest-gateway; both READMEs flag the package as an internal building block of@forestadmin/gateway.Both bins keep serving exactly as before. The link points to the standalone Gateway page added by the docs PR (ForestAdmin/docs, stacked on #53).
Evidence
npx forest-bff/npx forest-mcp-serverboot silently.After:
openapi-cli.test.ts:dispatchCli([])logs['Warn', DEPRECATION_WARNING]once;dispatchCli(['openapi'])does not.mcp-server/test/cli.test.ts: starting the bin logs oneconsole.warncontainingforest-mcp-server is deprecatedandforest-gateway.Merge Danger
Door: two-way
Revert the commit. Must ship with or after the standalone Gateway docs, so the link resolves.
Blast Radius: logs
One extra warning line at boot, including on the API service Forest hosts until it moves to the Gateway image.
Part of PRD-1342
🤖 Generated with Claude Code
Note
Warn that
forest-mcp-serverandforest-bffare deprecated, add gateway Docker pipelineforest-mcp-serverandforest-bffCLIs that direct users to theforest-gatewaypackage, with matching README notices in mcp-server/README.md and agent-bff/README.mdFOREST_GATEWAY_*variable names, with oldBFF_*andAGENT_URLnames kept as warned aliases; conflicting alias values now fail startupdocker-publish-gateway.ymlworkflow that builds amd64/arm64 images and publishes multi-arch tags to GHCR/api, agent URLs must be absolute HTTP(S) without query or fragment, and conflicting legacy environment aliases now cause startup errors instead of being ignored; existingforest-bffdeployments relying on legacy names still work but see warningsMacroscope summarized a2ad56c.