GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — V6 pluggable router by marshallswain · Pull Request #3606 · feathersjs/feathers

Conversation

@marshallswain

@marshallswain marshallswain commented Sep 4, 2025

Copy link
Copy Markdown
Member

Summary

Introduces a pluggable routing system to assist users to upgrade:

  • Add base router interface with framework-agnostic routing patterns
  • Implement Express router adapter for gradual migration
  • Implement Koa router adapter as Express alternative
  • Update core application to support pluggable routers
  • Include comprehensive test coverage for router implementations

This enables users to incrementally reduce their Express dependency by switching to lighter alternatives while maintaining compatibility.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@marshallswain
marshallswain changed the base branch from dove to v6 September 4, 2025 01:42
marshallswain added 2 commits December 19, 2025 16:38
@marshallswain
feat: Add pluggable router to reduce Express dependency
d4307a3 
Introduces a pluggable routing system to help users migrate away from Express:
- Add base router interface with framework-agnostic routing patterns
- Implement Express router adapter for gradual migration
- Implement Koa router adapter as Express alternative
- Update core application to support pluggable routers
- Include comprehensive test coverage for router implementations

This enables users to incrementally reduce their Express dependency by switching to lighter alternatives while maintaining compatibility.
@marshallswain
fix(routing): optimize lookup performance and expose router options
1a7b51e 
- Remove redundant RegExp creation on each lookup
- Add Set for O(1) duplicate path detection
- Expose caseSensitive and trailing options in router constructors
- Update README with correct examples and options documentation
- Update tests to use constructor options
@marshallswain
marshallswain force-pushed the v6-pluggable-router branch from c9f1acb to 1a7b51e Compare December 19, 2025 23:45
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Dec 19, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Deploying feathers-eagle with  Cloudflare Pages  Cloudflare Pages

Latest commit: a8d3619
Status: ✅  Deploy successful!
Preview URL: https://9427ccc3.feathers-a8l.pages.dev
Branch Preview URL: https://v6-pluggable-router.feathers-a8l.pages.dev

View logs
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Dec 19, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Deploying feathers-dove with  Cloudflare Pages  Cloudflare Pages

Latest commit: a8d3619
Status: ✅  Deploy successful!
Preview URL: https://6a87d51f.feathers.pages.dev
Branch Preview URL: https://v6-pluggable-router.feathers.pages.dev

View logs
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@marshallswain
fix(routing): add type check for non-string paths in lookup
a8d3619

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

@marshallswain
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.