Skip to content

fix(ui): avoid resetting instance page on same route#53

Merged
DonPrus merged 1 commit into
nullclaw:mainfrom
vernonstinebaker:fix/instance-route-guard
May 7, 2026
Merged

fix(ui): avoid resetting instance page on same route#53
DonPrus merged 1 commit into
nullclaw:mainfrom
vernonstinebaker:fix/instance-route-guard

Conversation

@vernonstinebaker

Copy link
Copy Markdown
Contributor

Summary

  • track the initialized instance route key on the instance detail page
  • skip the destructive reset/reload path when reactive updates fire for the same route

Why

The instance page effect that clears local state and re-runs refresh(true, true) was firing even when the route had not actually changed. That caused unnecessary churn on the detail page and regressed the local chat experience by repeatedly clearing instance/config/provider/module state while staying on the same instance.

Validation

  • npm --prefix ui ci --no-audit --no-fund
  • npm --prefix ui run build
  • zig build -Dbuild-ui=false
  • local smoke: chat on the instance page remained working after the rebuild instead of regressing under same-route refreshes

@DonPrus DonPrus merged commit 09b691c into nullclaw:main May 7, 2026
3 checks passed
@vernonstinebaker vernonstinebaker deleted the fix/instance-route-guard branch May 7, 2026 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants