如何编写高效可维护的Skill：从架构设计到代码实现

13次阅读

共计 2151 个字符，预计需要花费 6 分钟才能阅读完成。

在开发自定义 Skill 时，许多开发者会遇到以下典型问题：

架构混乱：业务逻辑与基础设施代码混杂，导致修改时牵一发而动全身
可测试性差：由于高度耦合，难以编写隔离的单元测试
扩展困难：新增功能时需要修改多处核心代码
错误处理缺失：异常情况被忽略或简单打印日志，缺乏统一处理机制

单一职责原则(SRP)
每个类 / 函数只做一件事
例如：将语音识别、业务逻辑、响应生成分离
开闭原则(OCP)
通过抽象接口支持扩展
示例：定义 IntentHandler 接口，新增意图只需实现新 handler
依赖倒置原则(DIP)
高层模块不依赖低层细节
使用依赖注入管理服务依赖

from abc import ABC, abstractmethod
from typing import Dict, Any

# 抽象接口定义
class IntentHandler(ABC):
    @abstractmethod
    def can_handle(self, intent_name: str) -> bool:
        pass

    @abstractmethod
    def handle(self, request: Dict[str, Any]) -> Dict[str, Any]:
        pass

# 具体业务实现
class WeatherIntentHandler(IntentHandler):
    def can_handle(self, intent_name: str) -> bool:
        return intent_name == "WeatherIntent"

    def handle(self, request: Dict[str, Any]) -> Dict[str, Any]:
        location = request['slots']['Location']
        # 业务逻辑隔离
        weather = self._fetch_weather(location)
        return {'text': f"{location}的天气是{weather}"
        }

    def _fetch_weather(self, location: str) -> str:
        # 实际调用天气 API
        return "晴朗"

# 基础设施层
class SkillDispatcher:
    def __init__(self, handlers: list[IntentHandler]):
        self.handlers = handlers

    def dispatch(self, request: Dict[str, Any]) -> Dict[str, Any]:
        intent_name = request['intent']['name']
        for handler in self.handlers:
            if handler.can_handle(intent_name):
                try:
                    return handler.handle(request)
                except Exception as e:
                    # 统一错误处理
                    return {
                        'text': "抱歉，处理请求时出现问题",
                        'error': str(e)
                    }
        return {'text': "未找到匹配的处理器"}

import pytest
from unittest.mock import Mock

class TestWeatherIntentHandler:
    def test_can_handle(self):
        handler = WeatherIntentHandler()
        assert handler.can_handle("WeatherIntent") is True
        assert handler.can_handle("OtherIntent") is False

    def test_handle_success(self, mocker):
        handler = WeatherIntentHandler()
        mocker.patch.object(handler, '_fetch_weather', return_value="多云")

        response = handler.handle({'slots': {'Location': '北京'}
        })

        assert "北京的天气是多云" in response['text']