Polish populated workflow guidance
This commit is contained in:
@@ -3276,3 +3276,36 @@ Limitations:
|
||||
|
||||
Next recommended pass:
|
||||
- After deploy and visual validation, continue with any remaining end-to-end workflow handoff polish surfaced by the live audit.
|
||||
|
||||
## Sprint 91 populated workflow audit polish (2026-06-21)
|
||||
|
||||
Changed:
|
||||
- Audited the live populated demo state on `http://192.168.10.150:1202` across Overview, Data, Map, QA/QC, AI Labs and Exports.
|
||||
- Changed the Overview workflow guidance badge to show `Ready for handoff` when project, dataset, map, QA/AI and export state are all present.
|
||||
- Clarified the Map workflow step so it reports layer feature count separately from AOI context and shows `AOI loaded` for AOI-only map context.
|
||||
- Extended `backend/tests/test_sprint90_workflow_guidance.py` with complete-state and Map-copy regression coverage.
|
||||
- Updated `frontend/README.md`, `docs/TODO.md` and `CHANGELOG.md`.
|
||||
|
||||
Tested:
|
||||
- Live pre-change browser audit passed for the populated demo workflow: Data had 2 ready datasets, Map rendered `demo_predicted_buildings.geojson` with 2 layer features, QA/QC showed 2 checks, AI Labs showed model/result/QA surfaces, Exports showed 50 artifacts, and no checked workspace had console warnings/errors or horizontal overflow.
|
||||
- Red step: `python -m pytest backend/tests/test_sprint90_workflow_guidance.py -q` failed on missing `workflowGuidanceComplete` / `Ready for handoff` / precise Map copy contracts.
|
||||
- Post-deploy live check found the AOI-only reload state rendered `0 layer features + AOI`; a second red step failed on missing `AOI loaded` copy before the fix.
|
||||
- `python -m pytest backend/tests/test_sprint90_workflow_guidance.py -q` (`4 passed`)
|
||||
- `python -m pytest backend/tests/test_sprint90_workflow_guidance.py backend/tests/test_sprint83_workspace_panel_hierarchy.py backend/tests/test_sprint82_shell_density_polish.py -q` (`10 passed`)
|
||||
- `cd frontend && npm run typecheck`
|
||||
- `cd frontend && npm run build`
|
||||
- Local browser UI check against `http://127.0.0.1:5174` passed for the Overview guidance panel with no horizontal overflow. Local console showed expected Vite proxy 500s because only the frontend server was running.
|
||||
- `bash scripts/run_readiness_check.sh` (`307 passed`; frontend typecheck/build passed; Alembic head `202606120900`; live smoke syntax passed)
|
||||
- Tower deploy via `scripts/deploy_tower.ps1` rebuilt and restarted the all-in-one container on `http://192.168.10.150:1202` from commit `97b943d`.
|
||||
- Tower deploy live migration smoke passed with PostGIS `3.6 USE_GEOS=1 USE_PROJ=1 USE_STATS=1` and required runtime schema objects present.
|
||||
- Browser runtime verification passed for frontend, proxied API and icon.
|
||||
- Live post-change browser check passed: AOI-only Overview state showed `Map AOI loaded`; after opening `demo_predicted_buildings.geojson` on the map, Overview showed `Map 2 layer features + AOI`; both states showed `Ready for handoff`, no horizontal overflow and no console warnings/errors.
|
||||
|
||||
Open:
|
||||
- None known after full readiness, Tower deploy and live browser validation.
|
||||
|
||||
Limitations:
|
||||
- Frontend Overview copy/state polish only; no API contract, persistence, migration, provider fetching or AI/model behavior changes.
|
||||
|
||||
Next recommended pass:
|
||||
- After deploy and live validation, continue with any remaining populated-state visual issues found in the next audit.
|
||||
|
||||
@@ -362,3 +362,4 @@ This file now starts with the current implementation status. Older preparation/b
|
||||
- [x] Add mobile overflow hardening for workbench navigation, inspector and long QA identifiers.
|
||||
- [x] Add QA/QC result filtering and density controls for long-running demo projects.
|
||||
- [x] Add Overview workflow guidance for the V1 project -> data -> map -> QA/AI -> export path.
|
||||
- [x] Audit populated demo workflow and tighten complete-state Overview guidance copy.
|
||||
|
||||
Reference in New Issue
Block a user