本提交含两部分。第一部分是本轮工作;第二部分是此前一直留在工作区、
从未提交的对象化重构,与第一部分在文件上互相咬合(internal/repository
整个包都是未跟踪状态,且 api 层已有文件引用它),无法拆成两个可编译的提交。
一、仓库层收口 A1 批(本轮工作)
把 api 层手写的 store.DB 查询收进具名仓库方法,只给真正获益的对象做方法,
不机械包裹全量。本批迁移 22 处裸查询(courses.go 9 / media.go 12 / products.go 1),
新增方法:
- MediaFileRepo.ListByBind / ListForAudit / MarkExtracted
- KnowledgeChunkRepo.CountByMediaFile
- ProductRepo.GetVisibleByID
两条业务口径改由仓库单点持有,避免各处手写漂移:
「只有 approved 素材出现在课程详情」与「已停用产品不在课程详情露出」。
修掉两个真实缺陷:
- ProductRepo.GetByID 缺 Where 条件。此前 GET /api/products/{id} 对任意 id 都返回
第一条产品、对不存在的 id 返回 200,且 PUT /api/products/{id} 会覆盖第一条产品
—— 数据损坏级。全仓扫描确认这是唯一一处同型写法。
- ProductRepo.Delete 写 status="deleted",而 DELETE 处理器文档与回包都声称
"inactive",接口在说谎;管理员用 status=all 拉列表会看到前端不认识的状态。
已对齐为 inactive(与 CourseRepo.Delete 一致)。
删除 8 个零调用且列名不存在的死方法(一调即 SQL 报错):
- media_file 上的 file_path / file_type / approval_status 三列并不存在,
GetByPath / ListByType / UpdateStatus 全废
- knowledge_chunk 上的 space_id 列不存在(模型早已改为 knowledge_space_key),
List / Total / ListBySpaceIDs / DeleteBySpace / SearchByVector 全废
取舍边界:能对当前 schema 跑通的死方法保留,跑不通的删或修。
CourseRepo.List 补齐 status=all 档(此前传给它会当作 status='all' 过滤出空列表)。
该方法此前零调用,现与产品列表语义对齐。
验证:go build ./... 与 go test ./... 全绿;另用真实 HTTP 请求验证 34 项
(课程 17 / 产品 3 / 素材 14),跑在数据库副本与独立 KB_DATA_DIR 上,
含 multipart 真上传 → 审批 → pdftotext 提取 → 分片入库的完整链路。
二、此前未提交的对象化重构(非本轮工作)
- 新增 internal/repository 仓库层、connectors、skills、specialists、xapps、jsonutil,
model/task_record|task_run|task_artifact、api/task_runtime|action_definition|chat_message
- 删除 api/app_definition、connectors、my_app_center、notification、office_skill、
export_docx|pptx|xlsx、official_account_* 等,随 XApp/Skill/Specialist/Connector
可插拔打包方向(AR10/AR11)调整
- 资产目录归位:backend-go/knowledge_source → assets/knowledge/source、
training_materials → assets/training/materials;README 内相对路径同步加深两级;
deploy env 补 ASSET_ROOT_DIR 并改 KNOWLEDGE_SOURCE_DIR / TRAINING_MATERIALS_DIR
- 前端新增 skills/ specialists/ connectors/ xapps/ 目录与对应页面
验证:前端 npm run build 通过(7.26s)。
Co-Authored-By: Claude Code <noreply@anthropic.com>
149 lines
3.8 KiB
Go
149 lines
3.8 KiB
Go
package api
|
||
|
||
import (
|
||
"strconv"
|
||
|
||
"github.com/gin-gonic/gin"
|
||
|
||
"eai_agentplatform/backend/internal/model"
|
||
|
||
"eai_agentplatform/backend/internal/web"
|
||
)
|
||
|
||
// 本文件的仓库实例:courseRepo 声明在 position.go,mediaRepo 声明在 media.go。
|
||
|
||
// courseView 课程详情视图(含绑定产品)
|
||
func courseView(c *gin.Context, co model.Course) {
|
||
out := gin.H{
|
||
"id": co.ID,
|
||
"code": co.Code,
|
||
"name": co.Name,
|
||
"category": co.Category,
|
||
"target_customers": co.TargetCustomers,
|
||
"forbidden_customers": co.ForbiddenCustomers,
|
||
"scripts": co.Scripts,
|
||
"sales_process": co.SalesProcess,
|
||
"objection_handling": co.ObjectionHandling,
|
||
"delivery_pitfalls": co.DeliveryPitfalls,
|
||
"report_rules": co.ReportRules,
|
||
"related_product_id": co.RelatedProductID,
|
||
"status": co.Status,
|
||
"created_at": co.CreatedAt,
|
||
"updated_at": co.UpdatedAt,
|
||
}
|
||
if co.RelatedProductID != nil {
|
||
if p, ok := productRepo.GetVisibleByID(*co.RelatedProductID); ok {
|
||
out["product"] = gin.H{"id": p.ID, "code": p.Code, "name": p.Name, "category": p.Category}
|
||
}
|
||
}
|
||
|
||
medias := mediaRepo.ListByBind("course", co.ID)
|
||
if len(medias) > 0 {
|
||
items := make([]gin.H, 0, len(medias))
|
||
for _, m := range medias {
|
||
items = append(items, gin.H{
|
||
"id": m.ID,
|
||
"filename": m.Filename,
|
||
"file_ext": m.FileExt,
|
||
"preview_url": "/api/media/preview/" + strconv.FormatUint(uint64(m.ID), 10),
|
||
})
|
||
}
|
||
out["medias"] = items
|
||
}
|
||
web.OK(c, out)
|
||
}
|
||
|
||
// ListCourses GET /api/courses?category=&status=
|
||
func ListCourses(c *gin.Context) {
|
||
web.OK(c, courseRepo.List(c.Query("category"), c.Query("status")))
|
||
}
|
||
|
||
// GetCourse GET /api/courses/{id}
|
||
func GetCourse(c *gin.Context) {
|
||
id, ok := parseID(c, "id")
|
||
if !ok {
|
||
return
|
||
}
|
||
co, found := courseRepo.GetByID(id)
|
||
if !found {
|
||
web.Fail(c, web.NewNotFoundError("课程不存在"))
|
||
return
|
||
}
|
||
courseView(c, co)
|
||
}
|
||
|
||
// CreateCourse POST /api/courses (admin)
|
||
func CreateCourse(c *gin.Context) {
|
||
var co model.Course
|
||
if err := c.ShouldBindJSON(&co); err != nil {
|
||
web.Fail(c, web.NewBadRequest("请求参数错误"))
|
||
return
|
||
}
|
||
if co.Code == "" || co.Name == "" || co.Category == "" {
|
||
web.Fail(c, web.NewBadRequest("编号、名称、分类为必填"))
|
||
return
|
||
}
|
||
if courseRepo.CountByCode(co.Code, nil) > 0 {
|
||
web.Fail(c, web.NewConflictError("课程编号已存在"))
|
||
return
|
||
}
|
||
if co.Status == "" {
|
||
co.Status = "active"
|
||
}
|
||
if !courseRepo.Insert(&co) {
|
||
web.Fail(c, web.NewBadRequest("创建课程失败"))
|
||
return
|
||
}
|
||
web.OK(c, co)
|
||
}
|
||
|
||
// UpdateCourse PUT /api/courses/{id} (admin) —— 支持绑定/解绑产品
|
||
func UpdateCourse(c *gin.Context) {
|
||
id, ok := parseID(c, "id")
|
||
if !ok {
|
||
return
|
||
}
|
||
co, found := courseRepo.GetByID(id)
|
||
if !found {
|
||
web.Fail(c, web.NewNotFoundError("课程不存在"))
|
||
return
|
||
}
|
||
var req model.Course
|
||
if err := c.ShouldBindJSON(&req); err != nil {
|
||
web.Fail(c, web.NewBadRequest("请求参数错误"))
|
||
return
|
||
}
|
||
if req.Name == "" || req.Category == "" {
|
||
web.Fail(c, web.NewBadRequest("名称、分类为必填"))
|
||
return
|
||
}
|
||
if req.Status == "" {
|
||
req.Status = "active"
|
||
}
|
||
|
||
if req.Code != "" && req.Code != co.Code {
|
||
if courseRepo.CountByCode(req.Code, &id) > 0 {
|
||
web.Fail(c, web.NewConflictError("课程编号已存在"))
|
||
return
|
||
}
|
||
co.Code = req.Code
|
||
}
|
||
co.Name = req.Name
|
||
co.Category = req.Category
|
||
co.TargetCustomers = req.TargetCustomers
|
||
co.ForbiddenCustomers = req.ForbiddenCustomers
|
||
co.Scripts = req.Scripts
|
||
co.SalesProcess = req.SalesProcess
|
||
co.ObjectionHandling = req.ObjectionHandling
|
||
co.DeliveryPitfalls = req.DeliveryPitfalls
|
||
co.ReportRules = req.ReportRules
|
||
co.RelatedProductID = req.RelatedProductID
|
||
co.Status = req.Status
|
||
|
||
if !courseRepo.Update(&co) {
|
||
web.Fail(c, web.NewBadRequest("更新课程失败"))
|
||
return
|
||
}
|
||
courseView(c, co)
|
||
}
|