Repository navigation
fix(settings): preserve saved apply plans during pending runtime changes - #7263
Merged
Lightheartdevs merged 1 commit intoOct 8, 2026
Merged
Lightheartdevs merged 1 commit into
Lightheartdevs merged 1 commit into
Conversation
This was referenced Oct 4, 2026
tang-vu
force-pushed
the
fix/dashboard-runtime-settings-20261004
branch
from
October 4, 2026 09:54
45c372c to
3262be3
Compare
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
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.
Summary
Exclude edits, saves, reloads and refreshes while apply or reads are pending; preserve the saved plan after failures and unlock for retry.
Reconstructed only the reviewed #7263 scope on current upstream
mainb4f015a921c59eeb20317c5f48adf2dfb43b1430. Local candidate3262be3c7979a525ecf7e00771d978af2b3eec8f; previous published head45c372c975558ab29526a4ef9a93e353ef9c80f2is retained locally underrefs/archive/ods-20261004/pr-7263. No behavioral changes were added during reconstruction; current-main changes outside this scope are preserved.Issue association: Closes #7297.
AI Assistance
The human contributor and Codex collaborated on this PR. The contributor provided project direction, scope decisions and environment setup; Codex assisted with analysis, implementation or reconstruction, tests, review and PR preparation. The validation commands were executed by Codex, with results and limitations documented in this PR. The human contributor remains responsible for the final changes and validation claims. Maintainer review and applicable release gates remain required before merge. Earlier independent AI review receipts apply only to their recorded heads; current-candidate validation is identified separately in this PR.
Release Lane
mainChanged Surface
Risk And Validation
Risk level: Medium
git diff --check
Focused tests
Commit identity gate against exact current-main base/candidate
23 rendered Settings tests passed. Production Vite build passed; ESLint on three changed files reported 0 errors and 52 warnings.
The focused results were run on the exact candidate file tree before committing; the commit only recorded that tested tree. Original-head/full native and combined integration receipts remain archived, and are inherited evidence rather than fresh install or new-head hosted CI results.
CI execution: dashboard.yml runs npm test for the dashboard.
Operational Change Check
Notes For Reviewers
Rendered jsdom tests exercise promises and controls; live browser/host recreation and cross-tab/router navigation are unverified.
Rollback: revert this scope; preserve existing recovery receipts and configured data. Local native tests and green original-head CI do not replace human review or deployed-runtime gates.