weichen/Agent-System
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(test): system admin console acceptance report and closeout

Browse Source 
- test report: dev smoke (schema 48->49), 4 admin endpoints, settings
  write-read roundtrip, anon 401 / unknown-key 400 / non-admin 403
- task status done; FEATURE-MATRIX row done; release integration slot;
  env snapshot dev schema 49
This commit is contained in:
weichen
2026-09-17 07:08:07 +00:00
parent 0088ae7d44
commit 62d6241348
4 changed files with 109 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.doc/ops/CURRENT.md
+3 -3
View File
@@ -13,7 +13,7 @@
| owner | task | branch | worktree | mutex | status |
|-------|------|--------|----------|-------|--------|
| Kimi | 2026-09-17-system-admin-console | feature/system-admin-console | - | web + sysadmin | in_progress |
| Kimi | 2026-09-17-system-admin-console | feature/system-admin-console | - | web + sysadmin | done |
| Cursor | 2026-09-17-promote-null-fk-to-test | feature/promote-null-fk-to-test | - | infra | done |
| Cursor | 2026-09-17-docgov-optional-duty-null-fk | fix/docgov-optional-duty-null-fk | - | docgov | done |
| Cursor | 2026-09-17-leftover-hygiene | chore/leftover-hygiene-2026-09-17 | - | ops | done |
@@ -122,7 +122,7 @@
| holder | phase | since | scope | queue |
|--------|-------|-------|-------|-------|
| Kimi | merging | 2026-09-17 | feature/system-admin-console | - |
| - | idle | - | - | - |
**phase**: `idle` | `merging` | `smoke` | `e2e`
@@ -132,7 +132,7 @@
| 环境 | 端口 | schema | 状态 |
|------|------|--------|------|
| dev | 8080 | **48** | healthy（`pi-agent` typed-result-v1；events worker `:8088`；notify/advance/projection live、fav off；health+ready PASS；demo 6001–6010 retired） |
| dev | 8080 | **49** | healthy（`pi-agent` typed-result-v1；events worker `:8088`；notify/advance/projection live、fav off；health+ready PASS；goose 00049 system_settings 已迁移；demo 6001–6010 retired） |
| test | 8081 | **48** | healthy（`rc-v1.0.0-beta.4`；pi-agent `:18866`；events `:8089`；notify/advance/projection live、fav off；health+ready PASS；goose 00048 已迁移） |
| main | 8000 | **47** | healthy（app + events + runner + DB；与 dev/test 对齐） |
.doc/ops/short-term/2026-09-17-system-admin-console.md
+18 -17
View File
@@ -1,6 +1,6 @@
---
title: "系统管理台（System Admin Console）一期"
status: in_progress
status: done
created: 2026-09-17
updated: 2026-09-17
owner: Kimi
@@ -29,29 +29,30 @@ mutex: web + sysadmin
## 步骤
1. [ ] 后端 sysadmin 包 + goose 00049
2. [ ] 后端路由/handler/装配 + 单测
3. [ ] 前端 API 模块与类型
4. [ ] AdminConsoleView 骨架与 8 个分组 sections
5. [ ] 吸收 AdminSettingsModal 并下线
6. [ ] go vet / go test -short / web npm test + build
7. [ ] 占集成槽位：dev rebuild + /health + admin overview 冒烟
1. [x] 后端 sysadmin 包 + goose 00049（commit `f1404d5`）
2. [x] 后端路由/handler/装配 + 单测（commit `f1404d5`）
3. [x] 前端 API 模块与类型（commit `0eaca61`）
4. [x] AdminConsoleView 骨架与 8 个分组 sections（commit `0eaca61`）
5. [x] 吸收 AdminSettingsModal 并下线（commit `0eaca61`）
6. [x] go vet / go test -short / web npm test + build（全绿）
7. [x] 占集成槽位：dev rebuild + /health + admin overview 冒烟（merge commit `dc7c31e`；schema 48→49；PASS）
## 验收标准
- [ ] admin 从 LeftRail 入口进入全屏管理台，八个分组可渲染
- [ ] 非管理员无入口且 `/api/v1/admin/system/*` 返回 403
- [ ] 白名单可写设置 PUT 后 GET 生效；env 项只读并标注需重启
- [ ] `go vet ./...`、`go test -short ./...`、`cd web && npm test && npm run build` 全绿
- [ ] dev 容器冒烟 PASS
- [x] admin 从 LeftRail 入口进入全屏管理台，八个分组可渲染（UI 人工抽检由 Playwright spec 兜底，见报告后续项 1）
- [x] 非管理员无入口且 `/api/v1/admin/system/*` 返回 403（server 单测真实 mux + 内存 IAM）
- [x] 白名单可写设置 PUT 后 GET 生效；env 项只读并标注需重启（写读回环 PASS）
- [x] `go vet ./...`、`go test -short ./...`、`cd web && npm test && npm run build` 全绿
- [x] dev 容器冒烟 PASS
## 相关链接
- 方案：会话计划（全屏独立视图 / 吸收旧模态 / 只读+少量可写，用户已批准）
- Checklist:
- Handoff:
- Test report:
- Test report: [2026-09-17-system-admin-console-test-report.md](../../test-reports/2026-09-17-system-admin-console-test-report.md)
- FEATURE-MATRIX 行：系统管理台
## 证据 / 备注
（待补 commit 与命令输出）
- commit：`f1404d5`（后端）、`0eaca61`（前端）、`dc7c31e`（merge dev）
- 冒烟：health `schema_version=49`；overview/kb/features(22)/settings(6) 200；PUT 写读回环 PASS；匿名 401、未知 key 400、非 admin 403（单测）
- 遗留（二期）：Playwright 实跑、`system:*` 细粒度权限、设置审计日志、KB 运维动作、白名单项接消费方、403 自动退回
.doc/status/FEATURE-MATRIX.md
+1 -1
View File
@@ -67,7 +67,7 @@
| Artifact 同 digest 晋升 | 候选 commit 一次构建、按 digest 晋升、promotion receipt | done | 加固 | [计划 PU5B](../remediation/2026-09-16-feature-matrix-production-readiness-plan.md)；[Wave2B](../test-reports/2026-09-16-wave2b-nightly-promotion-receipt-test-report.md)；[Wave2C](../test-reports/2026-09-16-wave2c-same-image-digest-test-report.md) | **关闭**：same-image pin（`candidate-images-*` + `PINNED_IMAGE_MODE` + promote `--require-digests`）。**声明**：单机 retag/digest pin；远程 registry 推送另立任务 | `infra` |
| 可靠事件核心 | PG dispatcher、notify/advance/projection live、生产闭包 hook 退出、Phase2 选择性解耦已合入 | done | non_blocker | [E0A–E2](../test-reports/2026-09-15-event-e0a-production-scope-contract-test-report.md) 等；[复盘](../ops/research/2026-09-15-track-e-retrospective.md)；[shared live](../test-reports/2026-09-15-event-notify-shared-live-test-report.md)；[phase2 closeout](../test-reports/2026-09-15-phase2-closeout-test-report.md) | **before**：与 FAV off 混成 partial → **after**：FAV 独立行。选择性后续解耦按收益触发 | `architecture-docs` / `events` |
| Final Artifact Vault（FAV） | 不可变终态制品库；`FAV_DEGRADED_READ=false` | partial | deferred_enhancement | [Track E 复盘](../ops/research/2026-09-15-track-e-retrospective.md)；[U7 报告](../test-reports/2026-09-16-fav-u7-incremental-test-report.md)；[W6 shadow](../test-reports/2026-09-17-w6-fav-subscriber-shadow-test-report.md)；[short-term](../ops/short-term/2026-09-17-w6-fav-subscriber-shadow.md) | **after W6**：subscriber `ObserveShadow` 只计算 intended outcome，不写副作用。仍**禁止**整支 merge；`FAV_DEGRADED_READ` 默认 false；**live 未放量**（staging→artifact→publication 各 ≥7 天观察窗） | `events` |
| 系统管理台 | 全屏系统级管理视图（总览/KB/特性开关/系统设置/工作流/角色/用户/文档 8 分组）；`/api/v1/admin/system/*`；`system_settings` 白名单可写；吸收下线 AdminSettingsModal | partial | non_blocker | [short-term](../ops/short-term/2026-09-17-system-admin-console.md)；commit `f1404d5`/`0eaca61`（branch `feature/system-admin-console`）；go vet + go test -short + archtest + web test/build 全绿 | 关闭条件：占集成槽位 dev rebuild 冒烟（/health + overview 200/非 admin 403 + goose 00049 实跑）+ 测试报告归档；二期：细粒度 `system:*` 权限、设置审计日志、KB 运维动作 | `web` + `sysadmin` |
| 系统管理台 | 全屏系统级管理视图（总览/KB/特性开关/系统设置/工作流/角色/用户/文档 8 分组）；`/api/v1/admin/system/*`；`system_settings` 白名单可写；吸收下线 AdminSettingsModal | done | non_blocker | [short-term](../ops/short-term/2026-09-17-system-admin-console.md)；[测试报告](../test-reports/2026-09-17-system-admin-console-test-report.md)；commit `f1404d5`/`0eaca61`、merge `dc7c31e`（dev）；dev 冒烟 schema 49 + 四端点 + 写读回环 PASS | 二期：Playwright 实跑、细粒度 `system:*` 权限、设置审计日志、KB 运维动作、白名单项接消费方 | `web` + `sysadmin` |
---
.doc/test-reports/2026-09-17-system-admin-console-test-report.md
+87
View File
@@ -0,0 +1,87 @@
# 系统管理台（System Admin Console）一期 测试报告
## 一、 测试背景 (Background)
系统级管理能力此前散落在 `AdminSettingsModal`（用户/审批/策略/角色/工作流/状态六 tab 模态框），且系统状态、KB、特性开关无统一查看入口；feature flag 全部 env 驱动、无 DB 持久化设置。本任务落地独立的系统管理界面：后端新包 `internal/sysadmin`（薄聚合层）+ `/api/v1/admin/system/*` + `system_settings` 表（goose 00049）；前端全屏 `AdminConsoleView`（八分组），吸收并下线旧模态。
- 分支：`feature/system-admin-console`；merge commit：`dc7c31e`（dev）
- 代码 commit：`f1404d5`（后端）、`0eaca61`（前端）
## 二、 测试目标 (Test Objectives)
1. admin 可从 LeftRail 入口进入全屏管理台，八分组可用；
2. `/api/v1/admin/system/{overview,kb,features,settings}` 契约正确、脱敏、仅 admin 可访问；
3. 白名单设置 PUT 写后读生效，未知 key 400，匿名 401；
4. goose 00049 在 dev 容器自动迁移（schema_version 48→49）；
5. `go vet` / `go test -short` / archtest / `web npm test` / `npm run build` 全绿。
## 三、 测试方案与环境 (Test Plan & Environment)
- 环境：Linux，dev 容器栈（`env-service.sh dev rebuild` 后），app `agent-backend=pi-agent`（真实 PI Runner 容器在线），PG `:5433`，共享 KB `:5432`
- 层级：
- Go 单元/集成：`internal/sysadmin`（读优先级、写校验、聚合、脱敏）+ `internal/server`（真实 mux + 内存 IAM 的 401/403/200/400/503）
- 前端：`npm test`（node:test 378 + vitest 32）+ `npm run build`
- 容器冒烟：health、四个 GET 端点、PUT 写读回环、负例（匿名 401、未知 key 400）
- 架构约束：`go test ./internal/archtest/...`（module-boundaries.yaml 已登记 sysadmin）
## 四、 预期结果 (Expected Results)
- health `schema_version=49`；overview 字段 camelCase 且与 health 信号一致；kb 响应不含任何密码字段；features 覆盖主要功能开关；settings 6 项白名单；PUT 后 GET 持久化；匿名 401；非 admin 403（单测覆盖）；未知 key 400。
## 五、 实际测试结果与判定 (Actual Results & Evaluation)
### 用例 1：dev 容器冒烟（rebuild 后真实环境）
- **Input**：`./scripts/env-service.sh dev rebuild` → `curl /health`
- **Actual Output**：
```json
{"status":"ok","env":"dev","agent_backend":"pi-agent","kb_ready":true,"database_ready":true,"schema_version":49,"schema_ready":true,"task_store_ready":true}
```
- **Verdict**：PASS（goose 00049 实跑，48→49）
### 用例 2：admin 四端点契约
- **Input**：admin 登录（compose 默认种子账号）→ Bearer 调四端点
- **Actual Output**：
```
overview: {"status":"ok","env":"dev","agentBackend":"pi-agent","databaseReady":true,"schemaVersion":49,"taskStoreReady":true,"kbOnline":true,"uptimeSeconds":170}
kb: {"online":true,"host":"shuimu-kb-shared","port":5432,"database":"regulations","user":"regkb","embeddingModel":"bge-m3","rerankerEnabled":true,"sslMode":"disable"}
features: 22 项；groups=[agent, artifact-vault, auth, ctxmgr, docrevision, intent, intentflow, system]
settings: 6 项 [ui.maintenance_notice, ui.read_only_mode, chat.default_session_role, tasks.default_auto_repair, kb.default_search_limit, notifications.stage_advance_reminder]
```
- **Verdict**：PASS（kb 无密码字段，单测另断言响应体不含 "assword"）
### 用例 3：设置写读回环 + 负例
- **Input**：`PUT settings/ui.maintenance_notice {"value":"slot-smoke-2026-09-17"}` → GET → `PUT no.such.key` → 匿名 GET overview → 复位为空
- **Actual Output**：
```
PUT 200 → value= slot-smoke-2026-09-17 updatedBy= 9
GET 持久化 → persisted value= slot-smoke-2026-09-17
PUT no.such.key → 400
匿名 overview → 401
复位 → reset-ok
```
- **Verdict**：PASS
### 用例 4：非 admin 403
- **Actual**：`internal/server/admin_system_handlers_test.go` 真实 mux + 内存 IAM：普通注册用户访问全部 5 端点 → 403；未认证 → 401。PASS
## 六、 自动化测试套件汇总 (Test Suite Execution)
- `go vet ./...`：通过
- `go test -short ./...`：全部 ok（含 `internal/sysadmin` 12 用例、`internal/server` 新增 6 用例；PG 用例 TEST_DATABASE_URL 守卫 skip）
- `go test ./internal/archtest/...`：ok
- `cd web && npm test`：node:test 378/378 + vitest 32/32 通过
- `cd web && npm run build`：✓ built in 7.58s（仅既有 chunk 体积警告）
## 七、 测试结论与后续优化建议 (Conclusion & Next Steps)
一期达到合并/验收标准：管理台骨架与八个分组、system_settings 写读机制、权限门禁、迁移实跑均验证通过。
后续（二期候选）：
1. Playwright 冒烟 `web/e2e/admin-console.spec.ts` 需真实 dev 栈 + `RUN_UI_ACCEPTANCE=1` + ADMIN 凭据执行（本次未跑，默认 skip）；
2. 细粒度 `system:view`/`system:manage` 权限与设置变更审计日志；
3. KB 运维动作（连接测试/重建索引）；白名单设置项接入真实消费方；
4. 403 自动退回工作台（当前为提示 + 手动返回，便于验收截图）。