huang
409a68d60b
All checks were successful
构建并部署到测试环境（无 SSH） / build-and-deploy (push) Successful in 5m45s
主要变更： 1. OpenAPI 文档契约对齐 - 统一错误响应字段名为 msg（非 message） - 规范 envelope 响应结构（code, msg, data, timestamp） - 个人客户路由纳入文档体系（使用 Register 机制） - 新增 BuildDocHandlers() 统一管理 handler 构造 - 确保文档生成的幂等性 2. Service 层错误处理统一 - 全面替换 fmt.Errorf 为 errors.New/Wrap - 统一错误码使用规范 - Handler 层参数校验不泄露底层细节 - 新增错误码验证集成测试 3. 代码质量提升 - 删除未使用的 Task handler 和路由 - 新增代码规范检查脚本（check-service-errors.sh） - 新增注释路径一致性检查（check-comment-paths.sh） - 更新 API 文档生成指南 4. OpenSpec 归档 - 归档 openapi-contract-alignment 变更（63 tasks） - 归档 service-error-unify-core 变更 - 归档 service-error-unify-support 变更 - 归档 code-cleanup-docs-update 变更 - 归档 handler-validation-security 变更 - 同步 delta specs 到主规范文件 影响范围： - pkg/openapi: 新增 handlers.go，优化 generator.go - internal/service/*: 48 个 service 文件错误处理统一 - internal/handler/admin: 优化参数校验错误提示 - internal/routes: 个人客户路由改造，删除 task 路由 - scripts: 新增 3 个代码检查脚本 - docs: 更新 OpenAPI 文档（15750+ 行） - openspec/specs: 同步 3 个主规范文件 破坏性变更：无 向后兼容：是
28 lines
809 B
Bash
Executable File
28 lines
809 B
Bash
Executable File
#!/bin/bash
|
||
# 检查 Service 层是否使用 fmt.Errorf 对外返回
|
||
|
||
echo "🔍 检查 Service 层错误处理规范..."
|
||
|
||
FILES=$(find internal/service -name "*.go" -type f 2>/dev/null)
|
||
if [ -z "$FILES" ]; then
|
||
echo "⚠️ 未找到 Service 层文件"
|
||
exit 0
|
||
fi
|
||
|
||
VIOLATIONS=$(grep -n "fmt\.Errorf" $FILES | grep -v "// whitelist:")
|
||
|
||
if [ -n "$VIOLATIONS" ]; then
|
||
echo ""
|
||
echo "❌ 发现 Service 层使用 fmt.Errorf："
|
||
echo "$VIOLATIONS"
|
||
echo ""
|
||
echo "请使用以下方式替代："
|
||
echo " - 业务错误：errors.New(code, msg)"
|
||
echo " - 系统错误：errors.Wrap(code, err, msg)"
|
||
echo ""
|
||
echo "如果某处确实需要使用 fmt.Errorf（如内部调试），请添加注释：// whitelist:"
|
||
exit 1
|
||
fi
|
||
|
||
echo "✅ Service 层错误处理检查通过"
|