refactor(planning): remove the original decision trace #272

Merged

nessar merged 1 commits from feature/remove-decision-trace into develop 2026-10-15 21:26:36 +02:00

Conversation 0 Commits 1 Files Changed 43

+729 -8880

nessar commented 2026-10-15 20:43:06 +02:00

Owner

Copy link

Copy source

Calculations now return the displayed plan, stable explanation and final-state snapshots without recording or delivering decision history. Removes trace contracts and streaming, worker acknowledgement queues, the advanced explorer and replay UI, and remaining-balance attempt lists, including their styles and translations.

Preserves cancellation and checkpoint recovery, captured-input date comparisons for displayed placements, explanation provenance and remaining-balance reasons. Updates the domain vocabulary, ADRs and benchmark documentation, and adapts mixed tests to retain assertions for supported behavior.

Closes #76. Integrated functional verification is tracked in #77; performance acceptance and any further optimization remain in #78.

Local verification:

• 630 functional tests across 33 files passed with coverage under the 2 GB Node heap limit; the late-explanation regression also passed after its review correction.
• Application and test typechecks, ESLint, formatting and the production build passed. The build reports initial-bundle and calendar-stylesheet size warnings.
• Complete core outcomes match the original baseline (ea42561) for all seven benchmark scenarios.
• Independent Standards and Spec reviews have no remaining findings.
• The unchanged benchmark protocol timed out at its 300-second limit (exit 124). Its scenarios and gates remain unchanged; #78 owns the performance result.

Manual browser and physical-phone checks were not run.

Calculations now return the displayed plan, stable explanation and final-state snapshots without recording or delivering decision history. Removes trace contracts and streaming, worker acknowledgement queues, the advanced explorer and replay UI, and remaining-balance attempt lists, including their styles and translations. Preserves cancellation and checkpoint recovery, captured-input date comparisons for displayed placements, explanation provenance and remaining-balance reasons. Updates the domain vocabulary, ADRs and benchmark documentation, and adapts mixed tests to retain assertions for supported behavior. Closes #76. Integrated functional verification is tracked in #77; performance acceptance and any further optimization remain in #78. Local verification: - 630 functional tests across 33 files passed with coverage under the 2 GB Node heap limit; the late-explanation regression also passed after its review correction. - Application and test typechecks, ESLint, formatting and the production build passed. The build reports initial-bundle and calendar-stylesheet size warnings. - Complete core outcomes match the original baseline (`ea42561`) for all seven benchmark scenarios. - Independent Standards and Spec reviews have no remaining findings. - The unchanged benchmark protocol timed out at its 300-second limit (exit 124). Its scenarios and gates remain unchanged; #78 owns the performance result. Manual browser and physical-phone checks were not run.

nessar added 1 commit 2026-10-15 20:43:06 +02:00

refactor(planning): remove the original decision trace d9d9befa12

nessar merged commit 2f4af14869 into develop 2026-10-15 21:26:36 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

dependencies

docker

documentation

duplicate

enhancement

github_actions

good first issue

help wanted

invalid

javascript

needs-info

needs-triage

question

ready-for-agent

wontfix

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

renovate (renovate) Bot nessar (nessar)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: nessar/vac-optimizer#272