docs(adr): shader auto-apply — keep-last-good + always-on apply contract#101
Merged
Conversation
New ADR docs/adr/2026-07-14-shader-auto-apply.md recording, against the shipped PR #92 diff: the keep-last-good node contract (old program renders while a newer revision compiles or fails; per-revision failure latch; swap-on-success; accepted compile-stall and 2x-JIT-buffer costs), the always-auto apply UX (500 ms debounce, no toggle or Apply button, silent apply-now keymap, verdict chase, gentle two-half bar with always-available Revert), the manual-save crash firewall, and the fuel posture (watchdog recovery interim; lpvm-native fuel is the raised-priority fix). The 2026-07-04 editing-model ADR's 'Explicit Apply, no auto-apply' bullet gets a 'revised by' annotation. Fuel/metering and budgeted/async compile join the ADR README's Deferred Decisions index. ADR: docs/adr/2026-07-14-shader-auto-apply.md Plan: Planning/lp2025/2026-07-14-shader-auto-apply/03-adr-docs-cleanup.md Co-Authored-By: Claude Fable 5 <[email protected]>
Yona-Appletree
added a commit
that referenced
this pull request
Jul 16, 2026
…gate Brings in PR #92 (shader auto-apply + split-bar editor chrome), PR #102 (popover chrome as one SVG merged-outline path), PR #101 (auto-apply ADR), PR #100 (path-gated wgpu CI job), PR #99 (interp.f32 in DEFAULT_TARGETS). Conflicts were story-baseline PNGs only (both sides re-captured) — taken from main and re-captured from the merged tree in the follow-up commit. The auto-merged generated tailwind.css was discarded and regenerated with pinned [email protected] (braces verified balanced). Co-Authored-By: Claude Opus 4.8 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
P3 close-out of the shader auto-apply plan (follows #92):
docs/adr/2026-07-14-shader-auto-apply.md: keep-last-good node contract, always-auto apply UX (no toggle/Apply button — U6), manual-save crash firewall, fuel posture, alternatives considered.Sweep found the P2/P2b commits had already updated the README and module docs —
rg 'Explicit Apply'now hits only the annotated historical bullet and the new ADR's Revises line. Docs-only diff; fulljust check && build-ci && testgreen (153 suites).🤖 Generated with Claude Code