From 62d62413482549ef238ef3581f6c7d88be57f5f7 Mon Sep 17 00:00:00 2001 From: weichen Date: Thu, 17 Sep 2026 07:08:07 +0000 Subject: [PATCH] docs(test): system admin console acceptance report and closeout - 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 --- .doc/ops/CURRENT.md | 6 +- .../2026-09-17-system-admin-console.md | 35 ++++---- .doc/status/FEATURE-MATRIX.md | 2 +- ...-09-17-system-admin-console-test-report.md | 87 +++++++++++++++++++ 4 files changed, 109 insertions(+), 21 deletions(-) create mode 100644 .doc/test-reports/2026-09-17-system-admin-console-test-report.md diff --git a/.doc/ops/CURRENT.md b/.doc/ops/CURRENT.md index 1a614d8e..d3ecbbbe 100644 --- a/.doc/ops/CURRENT.md +++ b/.doc/ops/CURRENT.md @@ -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 对齐） | diff --git a/.doc/ops/short-term/2026-09-17-system-admin-console.md b/.doc/ops/short-term/2026-09-17-system-admin-console.md index 88261aeb..9a561f62 100644 --- a/.doc/ops/short-term/2026-09-17-system-admin-console.md +++ b/.doc/ops/short-term/2026-09-17-system-admin-console.md @@ -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 自动退回 diff --git a/.doc/status/FEATURE-MATRIX.md b/.doc/status/FEATURE-MATRIX.md index 3066144e..afde913a 100644 --- a/.doc/status/FEATURE-MATRIX.md +++ b/.doc/status/FEATURE-MATRIX.md @@ -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` | --- diff --git a/.doc/test-reports/2026-09-17-system-admin-console-test-report.md b/.doc/test-reports/2026-09-17-system-admin-console-test-report.md new file mode 100644 index 00000000..288ac621 --- /dev/null +++ b/.doc/test-reports/2026-09-17-system-admin-console-test-report.md @@ -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 自动退回工作台（当前为提示 + 手动返回，便于验收截图）。