Skip to content

feat: Chainable @hooks().params() decorator with hookMixin support #3650

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
marshallswain wants to merge 7 commits into from
Closed

feat: Chainable @hooks().params() decorator with hookMixin support #3650

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6044134
feat(hooks): Add chainable @hooks().params() decorator syntax
marshallswain Jan 31, 2026
ed6d650
feat: hookMixin respects custom params from @hooks().params() decorator
marshallswain Jan 31, 2026
1f684b4
docs: add documentation for chainable @hooks().params() decorator
marshallswain Jan 31, 2026
6f3e99d
refactor: DRY up chainable decorator and hookMixin implementation
marshallswain Jan 31, 2026
acf79c2
refactor: simplify Feathers context prop helpers
marshallswain Jan 31, 2026
4f878f6
refactor: use public HookManager API in chainable decorator
marshallswain Jan 31, 2026
be6dde7
fix: prefix unused test params with underscore to satisfy eslint
marshallswain Jan 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: use public HookManager API in chainable decorator 
Use clone().params/props/defaults() instead of directly accessing
internal _params/_props/_defaults properties.
  • Loading branch information
@marshallswain
marshallswain committed Jan 31, 2026
commit 4f878f6951a96130146076387bca96c9469797ce
21 changes: 6 additions & 15 deletions packages/feathers/src/hooks/hooks.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -147,23 +147,14 @@ function createChainableDecorator(manager: HookManager): ChainableHookDecorator
throw new Error('Can not apply hooks.')
}

decorator.params = (...params: string[]): ChainableHookDecorator => {
const clone = manager.clone()
clone._params = params
return createChainableDecorator(clone)
}
decorator.params = (...params: string[]): ChainableHookDecorator =>
createChainableDecorator(manager.clone().params(...params))

decorator.props = (props: HookContextData): ChainableHookDecorator => {
const clone = manager.clone()
clone._props = clone._props ? { ...clone._props, ...props } : { ...props }
return createChainableDecorator(clone)
}
decorator.props = (props: HookContextData): ChainableHookDecorator =>
createChainableDecorator(manager.clone().props(props))

decorator.defaults = (defaults: HookDefaultsInitializer): ChainableHookDecorator => {
const clone = manager.clone()
clone._defaults = defaults
return createChainableDecorator(clone)
}
decorator.defaults = (defaults: HookDefaultsInitializer): ChainableHookDecorator =>
createChainableDecorator(manager.clone().defaults(defaults))

return decorator as ChainableHookDecorator
}
Expand Down
Loading