HttpRouter.mountApp prefix matching fixed

[mlegenhausen]

Type

• Refactor
• Feature
• Bug Fix
• Optimization
• Documentation Update

Description

Example router

HttpRouter.empty.pipe(
  HttpRouter.mountApp('/a', HttpServerResponse.text('OK')),
  HttpRouter.get('/ab', HttpServerResponse.text('Unreachable'))
)

the first mounted app matches everything that starts with /a which is unexpected as you would expect that it only matches on exactly /a or /a/*. It even get's more confusion if you explicitly add trailing slash to the mount.

My PR tries to fix that issue by doing the bare minimum. It might make sense to also add support for something like /a* if you want to match prefixes.

[changeset-bot]

🦋 Changeset detected

Latest commit: 9cf7767

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 28 packages

Name	Type
@effect/platform	Patch
@effect/cli	Patch
@effect/cluster	Patch
@effect/experimental	Patch
@effect/opentelemetry	Patch
@effect/platform-browser	Patch
@effect/platform-bun	Patch
@effect/platform-node-shared	Patch
@effect/platform-node	Patch
@effect/rpc	Patch
@effect/sql-clickhouse	Patch
@effect/sql-d1	Patch
@effect/sql-drizzle	Patch
@effect/sql-libsql	Patch
@effect/sql-mssql	Patch
@effect/sql-mysql2	Patch
@effect/sql-pg	Patch
@effect/sql-sqlite-bun	Patch
@effect/sql-sqlite-node	Patch
@effect/sql	Patch
@effect/workflow	Patch
@effect/ai	Patch
@effect/ai-anthropic	Patch
@effect/ai-openai	Patch
@effect/sql-sqlite-do	Patch
@effect/sql-sqlite-react-native	Patch
@effect/sql-sqlite-wasm	Patch
@effect/sql-kysely	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR