GeezerVison

Summary

This PR implements the feature requested in #3457, allowing hooks to run on all service methods while providing fine-grained control over which methods are exposed externally.

Changes

@feathersjs/feathers

Type Declarations (declarations.ts)

• Updated methods description to clarify it controls which methods all hooks apply to
• Added new externalMethods option to control HTTP transport exposure

Service (service.ts)

• Added getExternalMethods() function (defaults to methods for backwards compatibility)
• Updated normalizeServiceOptions() to include externalMethods

Hooks (hooks.ts)

• Added allMethods to hook store to track which methods receive all hooks
• Modified collectHooks() to only include all hooks for methods in allMethods
• Updated hookMixin() to allow lazy creation of hook managers for any service method

HTTP Transport (http/index.ts)

• Updated to use getExternalMethods() for HTTP access control

Tests

• Added tests for lazy hook registration on unconfigured methods
• Added tests for all hooks scoping
• Added tests for externalMethods HTTP behavior

Features

1. Individual method hooks work without configuration

// Can register hooks on ANY method, even if not in `methods`
service.hooks({
  helperMethod: [myHook]  // works!
})

methods controls which methods receive all hooks

app.use('myService', new MyService(), {
  methods: ['find', 'get', 'create']
})

service.hooks({
  around: {
    all: [loggingHook]  // only runs on find, get, create
  }
})

3. externalMethods controls HTTP exposure

app.use('myService', new MyService(), {
  methods: ['find', 'get', 'create', 'internalAction'],  // all get hooks
  externalMethods: ['find', 'get', 'create']  // only these exposed via HTTP
})

Backwards Compatibility

• If externalMethods is not specified, it defaults to methods
• Existing code works without modification
• All 354 existing tests pass

Related

• Fixes Hooks should run on non-enabled methods #3457
• Companion PR for dove (v5.x): feat: Add externalMethods option for transport exposure (#3457) #3637