Record final scenario save-scroll review and build evidence
This commit is contained in:
+6
-4
@@ -9,11 +9,13 @@ release-scenario-editing.md. Existing scene/order semantics and typed drafts are
|
|||||||
retained; direct drawing, explicit insertion, same-parent reorder and correlated
|
retained; direct drawing, explicit insertion, same-parent reorder and correlated
|
||||||
activation rendering are implemented in d1d8f0b, with independent-review fixes
|
activation rendering are implemented in d1d8f0b, with independent-review fixes
|
||||||
in 7a66607 for stale scroll slots, actual Iced scroll coordinates, operand-specific
|
in 7a66607 for stale scroll slots, actual Iced scroll coordinates, operand-specific
|
||||||
activation segments and repeated picker cost. Final Windows desktop tests pass
|
activation segments and repeated picker cost, plus e7e37d4 for passive scroll
|
||||||
282/282, strict all-target desktop Clippy and optimized build pass. Exact artifact
|
observations during async save. Final Windows desktop tests pass 283/283,
|
||||||
|
strict all-target desktop Clippy and optimized build pass. Exact artifact
|
||||||
hash, before/after failures and timings are in release-scenario-editing.md.
|
hash, before/after failures and timings are in release-scenario-editing.md.
|
||||||
Independent recheck and native Windows/Linux trials remain pending and root-owned;
|
Focused independent recheck reports no remaining medium/high finding. Native
|
||||||
this branch claims no native acceptance and does not close process/chain work.
|
Windows/Linux trials remain pending and root-owned; this branch claims no native
|
||||||
|
acceptance and does not close process/chain work.
|
||||||
|
|
||||||
The first-release goal remains active and incomplete. Deliver usable Rust/Iced
|
The first-release goal remains active and incomplete. Deliver usable Rust/Iced
|
||||||
Operational Analysis and System Analysis/System Architecture, equivalent graphical,
|
Operational Analysis and System Analysis/System Architecture, equivalent graphical,
|
||||||
|
|||||||
@@ -2,9 +2,9 @@
|
|||||||
|
|
||||||
Status: implementation and automated Windows checks passed in the isolated
|
Status: implementation and automated Windows checks passed in the isolated
|
||||||
`codex/scenario-direct` checkout, based on ab661f2. Initial production is d1d8f0b;
|
`codex/scenario-direct` checkout, based on ab661f2. Initial production is d1d8f0b;
|
||||||
independent-review fixes are in 7a66607. The fixes are awaiting independent recheck
|
independent-review fixes are in 7a66607 and e7e37d4. The focused independent
|
||||||
and root-owned Windows/Linux native trials. This does not complete the release
|
recheck reports no remaining medium/high finding. Root-owned Windows/Linux native
|
||||||
or the separate process/chain work.
|
trials remain pending. This does not complete the release or process/chain work.
|
||||||
|
|
||||||
## Decisions
|
## Decisions
|
||||||
|
|
||||||
@@ -166,3 +166,35 @@ It supersedes the initial artifact above. Native acceptance is still pending.
|
|||||||
Root should repeat the three primary native tasks, including inner/outer wheel
|
Root should repeat the three primary native tasks, including inner/outer wheel
|
||||||
and scrollbar cancellation at a middle slot, resize/remount/clamp recovery, and
|
and scrollbar cancellation at a middle slot, resize/remount/clamp recovery, and
|
||||||
the three-operand ALT fixture. The broader release objective remains incomplete.
|
the three-operand ALT fixture. The broader release objective remains incomplete.
|
||||||
|
|
||||||
|
|
||||||
|
## Final passive-scroll save correction — 2026-09-06
|
||||||
|
|
||||||
|
Production `e7e37d485d842731d270781bd4e60a323fe4cdba` addresses the focused review's remaining App lifecycle
|
||||||
|
finding. During an async save, Iced can change its scroll position while App's busy
|
||||||
|
gate rejects ordinary messages. The observer had already recorded its report,
|
||||||
|
so dropping that notice left App's offset stale after save completion. The new
|
||||||
|
regression failed before the fix: App retained offset (0, 0) instead of (25, 140).
|
||||||
|
|
||||||
|
App now dispatches only the passive Scrolled event before interaction gates. Its
|
||||||
|
existing handler still validates document, scenario and revision. Mutation input
|
||||||
|
remains gated. The test checks current offsets and cancellation epoch while busy,
|
||||||
|
rejects an old-document observation, delivers a successful Saved receipt, then
|
||||||
|
uses production Canvas pointer events to create the intended sender/receiver/slot
|
||||||
|
draft without a model or source-history mutation.
|
||||||
|
|
||||||
|
The independent reviewer rechecked this exact production change and regression;
|
||||||
|
no remaining medium/high finding was reported in the focused review. Final full
|
||||||
|
desktop tests pass 283/283, strict all-target desktop Clippy passes, optimized
|
||||||
|
desktop build passes, and git diff --check passes. Logs under D:/CodexBuilds are
|
||||||
|
`Capella2-scenario-busy-scroll-before.log`,
|
||||||
|
`Capella2-scenario-busy-scroll-after.log`,
|
||||||
|
`Capella2-scenario-busy-scroll-desktop-tests.log`,
|
||||||
|
`Capella2-scenario-busy-scroll-clippy.log` and
|
||||||
|
`Capella2-scenario-busy-scroll-release.log`.
|
||||||
|
|
||||||
|
Latest Windows artifact, superseding both builds above:
|
||||||
|
`D:/CodexBuilds/Capella2-scenario-target/release/arcadia-studio.exe`,
|
||||||
|
20,349,952 bytes, SHA-256 `CEEB8E277DFC47C0D55DA43F903FCA45AFBF902AFA418B3924375910D699373A`.
|
||||||
|
Native acceptance is pending. Include scrolling while saving in the root's native
|
||||||
|
recovery task before accepting the direct scenario milestone.
|
||||||
|
|||||||
Reference in New Issue
Block a user