From 239e20d2cf2242d636f9d42684f76f5777d9d1ee Mon Sep 17 00:00:00 2001 From: brother7 <7brother7@gmail.com> Date: Wed, 26 Aug 2026 23:53:38 +0800 Subject: [PATCH] chore(coding): record ML-08 verification --- ...6-makelore-data-service-client-b72e4c19.md | 46 ++++++++++++++++++- 1 file changed, 44 insertions(+), 2 deletions(-) diff --git a/.project-docs/30-worklog/tasks/20260826-makelore-data-service-client-b72e4c19.md b/.project-docs/30-worklog/tasks/20260826-makelore-data-service-client-b72e4c19.md index fa8b54ee..b19bc340 100644 --- a/.project-docs/30-worklog/tasks/20260826-makelore-data-service-client-b72e4c19.md +++ b/.project-docs/30-worklog/tasks/20260826-makelore-data-service-client-b72e4c19.md @@ -8,7 +8,7 @@ - Worktree: D:\Datas\OthersProjects\makelore-data-service-client-b72e4c19 - Base commit: f2456039230f68e3ecb2c8653fc3379c23bbe5ce - Owner: codex-root -- Status: In Progress (ML-01 through ML-03 integrated) +- Status: In Progress (ML-01 through ML-08 integrated; ML-09/X-01 pending) ## Scope @@ -155,6 +155,13 @@ this coordinator after cherry-pick; the source record remains in `D:\Datas\OthersProjects\makelore-ml07-data-service-sdk-skill-6b4e91c2`. The ML-08 frontier is `549069d` plus this coordinator cleanup/record commit. +- Completed ML-08 merge-and-verify on the exact clean product frontier + `a981b07faaa64b20bf9aef6353218fef26695ca2`, with fixed client base + `f2456039230f68e3ecb2c8653fc3379c23bbe5ce`. ML-01 through ML-07 are present + in dependency order through that product HEAD; this step made no product + changes. The five required verification commands and their one known E2E + failure are recorded below. The product HEAD immediately before this + documentation update was `a981b07faaa64b20bf9aef6353218fef26695ca2`. ## Verification @@ -348,6 +355,33 @@ the Data Service skill assets, registry/display registration, focused tests, and the task record. `git diff --check` passed after foreign task-record cleanup. +- ML-08 precondition: coordinator branch was clean at product HEAD + `a981b07faaa64b20bf9aef6353218fef26695ca2`, based on + `f2456039230f68e3ecb2c8653fc3379c23bbe5ce`; no product file was changed by + this verification step. +- ML-08 command ledger (run in this coordinator worktree): + `pnpm typecheck` — exit 0, passed, approximately 5.32s. +- `pnpm lint:check` — exit 0, passed with 0 errors and the same 5 existing + warnings in `src/pages/Home/index.tsx` and `src/pages/Makelore/index.tsx`, + approximately 10.01s. +- `pnpm test` — exit 0, passed; main suite 188 files / 1610 tests passed / 2 + skipped (Vitest-reported 42.86s), followed by the pressure suite 1 file / 1 + test passed (1.77s). No assertion failure or worker exit occurred. +- `pnpm test:electron:windows` — exit 0, passed; 2 files / 4 tests passed + (Vitest-reported 1.02s; command approximately 2.34s). +- `pnpm test:e2e` — exit 1; its required `build:vite` phases completed + successfully, then Playwright ran 26 tests with 25 passed and 1 failed. + The failure is the known `tests/e2e/pi-coding-first-chat.spec.ts:575` + `locator.selectOption` 30s timeout because combobox `当前对话模型` remained + disabled; this is not an E2E pass and remains a release/acceptance deviation. + Playwright reported approximately 1.0m for the run. The build emitted only + existing dynamic-import and chunk-size warnings. +- ML-08 post-command hygiene: `git diff --check` passed; the first drift + invocation was unavailable because `python` is not on the PowerShell PATH, + then the same `check_doc_drift.py --task-id + 20260826-makelore-data-service-client-b72e4c19` check passed using the + configured bundled Python runtime. Test/build output remained ignored and + the worktree was clean before this task-record update. ## Follow-ups @@ -361,7 +395,15 @@ and ML-07 consumes only that local capability while keeping ordinary arbitrary URL opens data-free. - The broader ML-00 through ML-09 coordinator task remains in progress; this - record does not claim full client-suite, build, review, or release proof. + record now has ML-08 command verification but does not claim ML-09 review, + release proof, or live acceptance. The client/server draft PR promotion is + externally blocked in this environment because GitLab HTTPS credentials are + unavailable non-interactively and `gh`/`glab` are not installed; no draft PR + is claimed from this worktree. +- ML-09 remains pending for repository-local review and one remediation + implementer if findings require it. X-01 remains a TODO after both reviews: + exercise real PostgreSQL with a signed-in MakeLore preview and record live + acceptance separately; repository E2E results cannot substitute for it. ## Promotion Candidates