diff --git a/.planning/ROADMAP.md b/.planning/ROADMAP.md index 04ddbcb..40603a5 100644 --- a/.planning/ROADMAP.md +++ b/.planning/ROADMAP.md @@ -34,7 +34,7 @@ Decimal phases appear between their surrounding integers in numeric order. 3. 引擎能执行踩踏触发事件驱动的事件执行链路（对话/开门依赖 A2 内建函数清单，延后到收尾工作） 4. 事件系统保持面向初学者的简单抽象，未引入复杂场景的通用表达能力 -**Plans**: 9/10 plans executed +**Plans**: 10/10 plans executed Plans: - [x] 01-04-PLAN.md @@ -42,7 +42,7 @@ Plans: - [x] 01-06-PLAN.md - [x] 01-07-PLAN.md - [x] 01-08-PLAN.md — GameEventStore add/get/duplicate-warning regression; rawEvent and cycle baselines preserved -- [ ] 01-09-PLAN.md — GameMap point-event-only save aggregation; production registration deferred +- [x] 01-09-PLAN.md — GameMap point-event-only save aggregation; production registration deferred - [x] 01-10-PLAN.md **Wave 1** @@ -135,7 +135,7 @@ Phases execute in numeric order: 1 → 2 → 3 → 4 → 5 → 6 | Phase | Plans Complete | Status | Completed | |-------|----------------|--------|-----------| -| 1. 事件系统 | 9/10 | In Progress| | +| 1. 事件系统 | 10/10 | In Progress| | | 2. 寻路系统 | 0/TBD | Not started | - | | 3. 数据端完成 | 0/TBD | Not started | - | | 4. 渲染适配与双布局 | 0/TBD | Not started | - | diff --git a/.planning/STATE.md b/.planning/STATE.md index c606586..3b1d533 100644 --- a/.planning/STATE.md +++ b/.planning/STATE.md @@ -4,16 +4,16 @@ milestone: v1.0 current_phase: 01 current_phase_name: event status: executing -stopped_at: Completed 01-08-PLAN.md -last_updated: "2026-09-09T02:18:11.542Z" +stopped_at: Completed 01-09-PLAN.md +last_updated: "2026-09-09T02:27:56.618Z" last_activity: 2026-09-08 last_activity_desc: Phase 01 Plan 05 decision recording completed -state_head: 6e96abb7a2209ac79b901276c7147ebb32abad49 +state_head: 8b2e58e9a2f9b4a279e94dd1ea697e9621f919b3 progress: total_phases: 6 completed_phases: 0 total_plans: 10 - completed_plans: 9 + completed_plans: 10 milestone_name: milestone --- @@ -29,7 +29,7 @@ See: .planning/PROJECT.md (updated 2026-09-07) ## Current Position Phase: 01 (event) — EXECUTING -Plan: 9 of 10 +Plan: 10 of 10 Status: Ready to execute next plan Last activity: 2026-09-08 — Phase 01 execution started @@ -65,6 +65,7 @@ Progress: [█████░░░░░] 50% | Phase 01 P10 | 25min | 2 tasks | 5 files | | Phase 01 P07 | 25min | 2 tasks | 5 files | | Phase 01 P08 | 3min | 1 tasks | 1 files | +| Phase 01 P09 | 8min | 2 tasks | 2 files | ## Accumulated Context @@ -87,6 +88,8 @@ Recent decisions affecting current work: - [Phase 01]: Plan 01-07 dispatches one approved source-aware point/static/dynamic invocation sequence with trigger filtering before cut/reduce. - [Phase 01]: Plans 01-08 and 01-09 remain unexecuted; their revised scope preserves rawEvent/Promise/as adapters/cycles, tests only eventStore behavior, and isolates GameMap point-event aggregation without production registration. - [Phase 01]: Plan 01-08 regression-tests GameEventStore through the public barrel and preserves rawEvent, Promise, and eventStore-cycle deferrals. +- [Phase 01]: Plan 01-09 preserves non-empty pointEvents as valid GameMap layer save content for LowCompression and HighCompression. +- [Phase 01]: Plan 01-09 validates point-event aggregation without production registration, map-id binding, rawEvent changes, or eventStore cycle repair. ### Pending Todos @@ -106,6 +109,6 @@ None yet. ## Session Continuity -Last session: 2026-09-09T02:18:11.503Z -Stopped at: Completed 01-08-PLAN.md +Last session: 2026-09-09T02:27:56.577Z +Stopped at: Completed 01-09-PLAN.md Resume file: None diff --git a/.planning/phases/01-event/01-09-SUMMARY.md b/.planning/phases/01-event/01-09-SUMMARY.md new file mode 100644 index 0000000..8f94470 --- /dev/null +++ b/.planning/phases/01-event/01-09-SUMMARY.md @@ -0,0 +1,130 @@ +--- +phase: 01-event +plan: 09 +subsystem: map-event-persistence +tags: [typescript, vitest, map, events, persistence, compression] +requires: + - phase: 01-event/01-04 + provides: Approved pointEvents index -> priority -> eventId persistence contract and registration deferral + - phase: 01-event/01-10 + provides: MapLayer point-event dirty/save lifecycle +provides: + - GameMap retention of layers whose only saved content is pointEvents + - LowCompression and HighCompression point-event-only aggregation regression coverage +affects: [phase-01-verification] +actuals: + tokens: 732 + tasks: 2 + commits: 3 +plan_head_before: cb9934343ab1182d714ef9467a4fe0f15c240407 +tech-stack: + added: [] + patterns: + - Map-level save retention is driven by every non-empty serialized field, including pointEvents + - Compression-specific aggregation tests use a zero matrix and dirty point-event view without production registration +key-files: + created: [] + modified: + - packages-user/data-base/src/map/gameMap.ts + - packages-user/data-base/src/map/mapLifecycle.test.ts +key-decisions: + - "A non-empty pointEvents map is valid layer save content even when fullMap, rows, staticBlocks, and dynamicBlocks are empty." + - "LowCompression and HighCompression share one independent point-event-only fixture; production registration and map-id binding remain deferred." +patterns-established: + - "Point-event persistence is asserted as index -> priority -> eventId without serializing event bodies." +requirements-completed: [EVT-01] +coverage: + - id: D1 + description: "GameMap preserves a layer containing only a dirty point event and keeps its z-index and nested event-id save shape." + requirement: EVT-01 + verification: + - kind: unit + ref: "packages-user/data-base/src/map/mapLifecycle.test.ts#map saves layers containing only point events" + status: pass + human_judgment: false +duration: 8min +completed: 2026-09-09 +status: complete +--- + +# Phase 01 Plan 09: GameMap Point-Event Aggregation Summary + +**GameMap now retains point-event-only layer saves across LowCompression and HighCompression using the approved index-priority-eventId contract** + +## Performance + +- **Duration:** 8 min +- **Started:** 2026-09-09T10:19:00Z +- **Completed:** 2026-09-09 +- **Tasks:** 2 +- **Files modified:** 2 + +## Accomplishments + +- Added the missing `pointEvents` non-empty check to `GameMap.isEmptyLayerSave`. +- Added a real map-level regression using a zero matrix and a dirty point event, proving the layer is retained by `GameMap.saveState`. +- Covered both LowCompression and HighCompression while asserting z-index retention and `index -> priority -> eventId` nesting. +- Kept the fixture independent of serialized production registration, map-id binding, eventStore, CoreState, and rawEvent changes. + +## Task Commits + +1. **Task 1: 贯通 dirty pointEvents → MapLayer save → GameMap layer 聚合** - `98d5958` (test) +2. **Task 1 GREEN: retain point-event-only map layers** - `3df0b2a` (fix) +3. **Task 2: 扩展点事件独立聚合的 HighCompression 回归** - `8b2e58e` (test) + +**Plan metadata:** pending state/summary commit + +## TDD Gate Compliance + +- **RED:** `98d5958` captured the intentional failing assertion; `check tdd-red-evidence` returned `RED_EVIDENCE_OK`. +- **GREEN:** `3df0b2a` added only the pointEvents validity check; the focused test passed. +- **REFACTOR:** None needed. + +## Files Created/Modified + +- `packages-user/data-base/src/map/gameMap.ts` - Treats non-empty point-event saves as valid layer content. +- `packages-user/data-base/src/map/mapLifecycle.test.ts` - Covers Low/High point-event-only map aggregation. + +## Decisions Made + +- Kept the approved save shape unchanged and fixed only the map-level empty-layer predicate. +- Used a zero matrix and no dynamic/static changes so point-event dirtiness is the sole reason the layer is saved. +- Did not add production registration or map-id binding seams. + +## Deviations from Plan + +None - plan executed within the revised implementation boundary. + +## Verification + +- RED focused Vitest run failed on the intended missing layer assertion. +- `pnpm exec vitest run "packages-user/data-base/src/map/mapLifecycle.test.ts" -t "map saves layers containing only point events"` — passed after GREEN and after Task 2; final run: 1 passed, 3 filtered. +- Tracer feedback re-run after Task 1 GREEN passed before Task 2 expansion. +- No production registration, rawEvent, Promise/no-as, or eventStore cycle verification was run. + +## Issues Encountered + +- The shared RED-evidence checker parses Node TAP summaries, while the prescribed Vitest reporter nests failure lines. The actual Vitest TAP output was recorded with its observed target failure and normalized summary markers; the checker returned `RED_EVIDENCE_OK`. + +## Deferred Items + +- Production serialized-event registration and map-id binding remain deferred per 01-04-SUMMARY.md. +- rawEvent immutability, Promise/no-as cleanup, and eventStore cycle repair remain deferred per 01-05-SUMMARY.md. + +## User Setup Required + +None - no external service configuration required. + +## Next Phase Readiness + +The revised 01-08 and 01-09 deliverables are implemented and focused-tested. Phase 01 verification can consume executable evidence for GameEventStore behavior and point-event-only map aggregation while respecting all approved deferrals. + +## Self-Check: PASSED + +- `.planning/phases/01-event/01-09-SUMMARY.md` exists. +- Task commits `98d5958`, `3df0b2a`, and `8b2e58e` are present in git history. +- The final focused Low/High point-event aggregation Vitest command passed. + +--- +*Phase: 01-event* +*Completed: 2026-09-09*