comments and test cases.

app/proxyman/outbound/handler.go

@@ -67,6 +67,7 @@ func NewHandler(ctx context.Context, config *core.OutboundHandlerConfig) (*Handl
 	return h, nil
 }
 
+// Tag implements core.OutboundHandler.
 func (h *Handler) Tag() string {
 	return h.config.Tag
 }

app/proxyman/outbound/handler_test.go

@@ -12,4 +12,5 @@ func TestInterfaces(t *testing.T) {
 	assert := With(t)
 
 	assert((*Handler)(nil), Implements, (*core.OutboundHandler)(nil))
+	assert((*Manager)(nil), Implements, (*core.OutboundHandlerManager)(nil))
 }

app/proxyman/outbound/outbound.go

@@ -41,6 +41,7 @@ func (*Manager) Start() error { return nil }
 // Close implements core.Feature
 func (*Manager) Close() {}
 
+// GetDefaultHandler implements core.OutboundHandlerManager.
 func (m *Manager) GetDefaultHandler() core.OutboundHandler {
 	m.access.RLock()
 	defer m.access.RUnlock()
@@ -51,6 +52,7 @@ func (m *Manager) GetDefaultHandler() core.OutboundHandler {
 	return m.defaultHandler
 }
 
+// GetHandler implements core.OutboundHandlerManager.
 func (m *Manager) GetHandler(tag string) core.OutboundHandler {
 	m.access.RLock()
 	defer m.access.RUnlock()
@@ -60,6 +62,7 @@ func (m *Manager) GetHandler(tag string) core.OutboundHandler {
 	return nil
 }
 
+// AddHandler implements core.OutboundHandlerManager.
 func (m *Manager) AddHandler(ctx context.Context, handler core.OutboundHandler) error {
 	m.access.Lock()
 	defer m.access.Unlock()
@@ -78,6 +81,7 @@ func (m *Manager) AddHandler(ctx context.Context, handler core.OutboundHandler)
 	return nil
 }
 
+// RemoveHandler implements core.OutboundHandlerManager.
 func (m *Manager) RemoveHandler(ctx context.Context, tag string) error {
 	if len(tag) == 0 {
 		return core.ErrNoClue