v2.4.0
Notable Changes
- preserve existing signature (no second param) to matching any existing introspection schemes - maybe cost a few characters, but it's a bit more future-proof/robust to potential future options we add.
- Router
options
are now:base?: string
- optional base pathroutes?: any[]
- optional array of routes (for introspection or preloading with advanced regex)- unrelated: adds a bit of added interface for the internal
Request
definition to address #47 - unrelated: export TS interfaces to address #38
- thanks to the @taralx refactor, we can now support preloading routes, which means those rare edge cases of crazy regex (previously not possible) requirements, can now be force-fed into the router (at config stage, or even after... also thanks to the more exposed
router.routes
array. Pretty sick!
Example of manual loading routes:
// EXAMPLE 1 mmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm
it('allows preloading advanced routes', async () => {
const basicHandler = jest.fn(req => req.params)
const customHandler = jest.fn(req => req.params)
const router = Router({
routes: [
[ 'GET', /^\/test\.(?<x>[^/]+)\/*$/, [basicHandler] ],
[ 'GET', /^\/custom-(?<custom>\d{2,4})$/, [customHandler] ], // match 2 to 4 digits only
]
})
await router.handle(buildRequest({ path: '/test.a.b' }))
expect(basicHandler).toHaveReturnedWith({ x: 'a.b' })
await router.handle(buildRequest({ path: '/custom-12345' }))
expect(customHandler).not.toHaveBeenCalled() // custom route mismatch
await router.handle(buildRequest({ path: '/custom-123' }))
expect(customHandler).toHaveReturnedWith({ custom: '123' }) // custom route hit
})
// EXAMPLE 2 mmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm
it('allows loading advanced routes after config', async () => {
const handler = jest.fn(req => req.params)
const router = Router()
// allows manual loading (after config)
router.routes.push([ 'GET', /^\/custom2-(?<custom>\w\d{3})$/, [handler] ])
await router.handle(buildRequest({ path: '/custom2-a456' }))
expect(handler).toHaveReturnedWith({ custom: 'a456' }) // custom route hit
})
DISCUSSION: If we change r
to routes
, and prevent the prebuild.js from mangling it, the filesize does grow (405b --> 409b), but so does the legibility around this weird introspection. Thoughts? I'm all for absolute minimum filesize, but... legibility. Torn. Don't think we would have to count the decision on this one as a breaking change (if we opt for "routes"), because the existing pattern isn't documented, and is hopefully only used in debugging patterns at most. Regardless of the path chosen here, documentation will be added to disclose this functionality.
UPDATE: unless there's strong objection, I'm opting for readability over ultimate filesize... router.routes
it is
Filesize Comparison (pretty sure this includes the type declaration, so appears artificially inflated a bit)
- current: https://bundlephobia.com/package/[email protected]
- @next ("routes"): https://bundlephobia.com/package/[email protected]
- @next ("r"): https://bundlephobia.com/package/[email protected]