All checks were successful
构建并部署到测试环境(无 SSH) / build-and-deploy (push) Successful in 6m33s
**变更说明**: - 删除所有 *_test.go 文件(单元测试、集成测试、验收测试、流程测试) - 删除整个 tests/ 目录 - 更新 CLAUDE.md:用"测试禁令"章节替换所有测试要求 - 删除测试生成 Skill (openspec-generate-acceptance-tests) - 删除测试生成命令 (opsx:gen-tests) - 更新 tasks.md:删除所有测试相关任务 **新规范**: - ❌ 禁止编写任何形式的自动化测试 - ❌ 禁止创建 *_test.go 文件 - ❌ 禁止在任务中包含测试相关工作 - ✅ 仅当用户明确要求时才编写测试 **原因**: 业务系统的正确性通过人工验证和生产环境监控保证,测试代码维护成本高于价值。 Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
11 KiB
11 KiB
junhong_cmp_fiber 项目开发规范
重要: 本文件包含核心规范。详细规范已提取为 Skills,在特定任务时按需加载。
专项规范 Skills(按需加载)
以下规范在相关任务时自动触发,无需手动加载:
| 任务类型 | 触发 Skill | 说明 |
|---|---|---|
| 创建/修改 DTO 文件 | dto-standards |
description 标签、枚举字段、验证标签规范 |
| 创建/修改 Model 模型 | model-standards |
GORM 模型结构、字段标签、TableName 规范 |
| 注册 API 路由 / 新增 Handler | api-routing |
Register() 函数、RouteSpec、文档生成器更新 |
| 测试接口/验证数据 | db-validation |
PostgreSQL MCP 使用方法和验证示例 |
| 数据库迁移 | db-migration |
迁移命令、文件规范、执行流程、失败处理 |
| 维护规范文档 | doc-management |
规范文档流程和维护规则 |
⚠️ 新增 Handler 时必须同步更新文档生成器
新增 Handler 后,接口不会自动出现在 OpenAPI 文档中。必须手动更新以下两个文件:
// cmd/api/docs.go 和 cmd/gendocs/main.go
handlers := &bootstrap.Handlers{
// ... 添加新 Handler
NewHandler: admin.NewXxxHandler(nil),
}
完整检查清单: 参见 docs/api-documentation-guide.md
语言要求
必须遵守:
- 永远用中文交互
- 注释必须使用中文
- 文档必须使用中文
- 日志消息必须使用中文
- 用户可见的错误消息必须使用中文
- 变量名、函数名、类型名必须使用英文(遵循 Go 命名规范)
- GIT提交的commit必须使用中文
技术栈
必须严格遵守,禁止替代方案:
| 类型 | 技术 |
|---|---|
| HTTP 框架 | Fiber v2.x |
| ORM | GORM v1.25.x |
| 配置管理 | Viper |
| 日志 | Zap + Lumberjack.v2 |
| JSON 序列化 | sonic(优先),encoding/json(必要时) |
| 验证 | Validator |
| 任务队列 | Asynq v0.24.x |
| 数据库 | PostgreSQL 14+ |
| 缓存 | Redis 6.0+ |
禁止:
- 直接使用
database/sql(必须通过 GORM) - 使用
net/http替代 Fiber - 使用
encoding/json替代 sonic(除非必要)
架构分层
必须遵循以下分层架构:
Handler → Service → Store → Model
- Handler: 只处理 HTTP 请求/响应,不包含业务逻辑
- Service: 包含所有业务逻辑,支持跨模块调用
- Store: 统一管理所有数据访问,支持事务处理
- Model: 定义数据结构和 DTO
核心原则
错误处理
- 所有错误必须在
pkg/errors/中定义 - 使用统一错误码系统
- Handler 层通过返回
error传递给全局 ErrorHandler
错误报错规范(必须遵守)
- Handler 层禁止直接返回/拼接底层错误信息给客户端(例如
"参数验证失败: "+err.Error()、err.Error()) - 参数校验失败:对外统一返回
errors.New(errors.CodeInvalidParam)(详细校验错误写日志) - Service 层禁止对外返回
fmt.Errorf(...),必须返回errors.New(...)或errors.Wrap(...) - 约定用法:
errors.New(code[, msg])、errors.Wrap(code, err[, msg])
响应格式
- 所有 API 响应使用
pkg/response/的统一格式 - 格式:
{code, msg, data, timestamp}
常量管理
- 所有常量定义在
pkg/constants/ - Redis key 使用函数生成:
Redis{Module}{Purpose}Key(params...) - 禁止硬编码字符串和 magic numbers
- 必须为所有常量添加中文注释
Go 代码风格
- 使用
gofmt格式化 - 遵循 Effective Go
- 包名: 简短、小写、单数、无下划线
- 接口命名: 使用
-er后缀(Reader、Writer、Logger)
数据库设计
核心规则:
- ❌ 禁止建立外键约束
- ❌ 禁止使用 GORM 关联关系标签(foreignKey、hasMany、belongsTo)
- ✅ 关联通过存储 ID 字段手动维护
- ✅ 关联数据在代码层面显式查询
Go 惯用法 vs Java 风格
✅ Go 风格(推荐)
- 扁平化包结构(最多 2-3 层)
- 小而专注的接口(1-3 个方法)
- 直接访问导出字段(不用 getter/setter)
- 组合优于继承
- 显式错误返回和检查
❌ Java 风格(禁止)
- 过度抽象(不必要的接口、工厂)
- Getter/Setter 方法
- 深层继承层次
- 异常处理(panic/recover)
- 类型前缀(IService、AbstractBase、ServiceImpl)
⚠️ 测试禁令(强制执行)
本项目不使用任何形式的自动化测试代码。
绝对禁止:
- ❌ 禁止编写单元测试 - 无论任何场景
- ❌ 禁止编写集成测试 - 无论任何场景
- ❌ 禁止编写验收测试 - 无论任何场景
- ❌ 禁止编写流程测试 - 无论任何场景
- ❌ 禁止编写 E2E 测试 - 无论任何场景
- ❌ 禁止创建
*_test.go文件 - 除非用户明确要求 - ❌ 禁止在任务中包含测试相关工作 - 规划和实现均不涉及测试
- ❌ 禁止在文档中提及测试要求 - 规范、设计文档均不讨论测试
唯一例外:
- ✅ 仅当用户明确要求时才编写测试代码
- ✅ 用户必须主动说明"请写测试"或"需要测试"
原因说明:
- 业务系统的正确性通过人工验证和生产环境监控保证
- 测试代码的维护成本高于价值
- 快速迭代优先于测试覆盖率
替代方案:
- 使用 PostgreSQL MCP 工具手动验证数据
- 使用 Postman/curl 手动测试 API
- 依赖生产环境日志和监控发现问题
性能要求
- API P95 响应时间 < 200ms
- API P99 响应时间 < 500ms
- 数据库查询 < 50ms
- 列表查询必须分页(默认 20,最大 100)
- 避免 N+1 查询,使用批量操作
文档要求
- 每个功能在
docs/{feature-id}/创建总结文档 - 文档文件名和内容使用中文
- 同步更新 README.md
- 为导出的函数、类型编写文档注释
函数复杂度
- 函数长度 ≤ 100 行(核心逻辑建议 ≤ 50 行)
main()函数只做编排,不含具体实现- 遵循单一职责原则
访问日志
- 所有 HTTP 请求记录到
access.log - 记录完整的请求/响应(限制 50KB)
- 包含: method, path, query, status, duration, request_id, ip, user_agent, user_id, bodies
- 使用 JSON 格式,配置自动轮转
OpenSpec 工作流
创建提案前的检查清单:
- ✅ 技术栈合规
- ✅ 架构分层正确
- ✅ 使用统一错误处理
- ✅ 常量定义在 pkg/constants/
- ✅ Go 惯用法(非 Java 风格)
- ✅ 性能考虑
- ✅ 文档更新计划
- ✅ 中文优先
Code Review 检查清单
错误处理
- Service 层无
fmt.Errorf对外返回 - Handler 层参数校验不泄露细节
- 错误码使用正确(4xx vs 5xx)
- 错误日志完整(包含上下文)
代码质量
- 遵循 Handler → Service → Store → Model 分层
- 函数长度 ≤ 100 行(核心逻辑 ≤ 50 行)
- 常量定义在
pkg/constants/ - 使用 Go 惯用法(非 Java 风格)
文档和注释
- 所有注释使用中文
- 导出函数/类型有文档注释
- API 路径注释与真实路由一致
越权防护规范
适用场景:任何涉及跨用户、跨店铺、跨企业的资源访问
三层防护机制:
-
路由层中间件(粗粒度拦截)
- 用于明显的权限限制(如企业账号禁止访问账号管理)
- 示例:
group.Use(func(c *fiber.Ctx) error { userType := middleware.GetUserTypeFromContext(c.UserContext()) if userType == constants.UserTypeEnterprise { return errors.New(errors.CodeForbidden, "无权限访问账号管理功能") } return c.Next() })
-
Service 层业务检查(细粒度验证)
- 使用
middleware.CanManageShop(ctx, targetShopID, shopStore)验证店铺权限 - 使用
middleware.CanManageEnterprise(ctx, targetEnterpriseID, enterpriseStore, shopStore)验证企业权限 - 类型级权限检查(如代理不能创建平台账号)
- 示例见
internal/service/account/service.go
- 使用
-
GORM Callback 自动过滤(兜底)
- 已有实现,自动应用到所有查询
- 代理用户:
WHERE shop_id IN (自己店铺+下级店铺) - 企业用户:
WHERE enterprise_id = 当前企业ID - 无需手动调用
统一错误返回:
- 越权访问统一返回:
errors.New(errors.CodeForbidden, "无权限操作该资源或资源不存在") - 不区分"不存在"和"无权限",防止信息泄露
审计日志规范
适用场景:任何敏感操作(账号管理、权限变更、数据删除等)
使用方式:
-
Service 层集成审计日志:
type Service struct { store *Store auditService AuditServiceInterface } func (s *Service) SensitiveOperation(ctx context.Context, ...) error { // 1. 执行业务操作 err := s.store.DoSomething(ctx, ...) if err != nil { return err } // 2. 记录审计日志(异步) s.auditService.LogOperation(ctx, &model.OperationLog{ OperatorID: middleware.GetUserIDFromContext(ctx), OperationType: "operation_type", OperationDesc: "操作描述", BeforeData: beforeData, // 变更前数据 AfterData: afterData, // 变更后数据 RequestID: middleware.GetRequestIDFromContext(ctx), IPAddress: middleware.GetIPFromContext(ctx), UserAgent: middleware.GetUserAgentFromContext(ctx), }) return nil } -
审计日志字段说明:
operator_id,operator_type,operator_name: 操作人信息(必填)target_*: 目标资源信息(可选)operation_type: 操作类型(create/update/delete/assign_roles等)operation_desc: 操作描述(中文,便于查看)before_data,after_data: 变更数据(JSON 格式)request_id,ip_address,user_agent: 请求上下文
-
异步写入:
- 审计日志使用 Goroutine 异步写入
- 写入失败不影响业务操作
- 失败时记录 Error 日志,包含完整审计信息
示例参考:internal/service/account/service.go
⚠️ 任务执行规范(必须遵守)
提案中的 tasks.md 是契约,不可擅自变更:
| 规则 | 说明 |
|---|---|
| ❌ 禁止跳过任务 | 每个任务都是经过规划的,不能因为"简单"或"显而易见"而跳过 |
| ❌ 禁止简化任务 | 不能将多个任务合并或简化执行,除非获得明确许可 |
| ❌ 禁止自作主张优化 | 发现可以优化的地方,必须先询问是否可以调整 |
| ✅ 必须逐项完成 | 按照 tasks.md 中的顺序逐一执行并标记完成 |
| ✅ 必须询问后变更 | 如需调整任务(简化/跳过/合并/优化),先询问用户确认 |
询问示例:
"我注意到任务 2.1 和 2.2 可以合并为一步完成,是否可以这样优化?" "任务 3.1 在当前实现中可能不需要,是否可以跳过?"
详细规范和 OpenSpec 工作流请查看: @/openspec/AGENTS.md