Skip to content

Adopt Loadable lifecycle improvements (abort handling, handleLoadException)#848

Open
lbwexler wants to merge 11 commits into
developfrom
staleAndAutoRefreshHandle
Open

Adopt Loadable lifecycle improvements (abort handling, handleLoadException)#848
lbwexler wants to merge 11 commits into
developfrom
staleAndAutoRefreshHandle

Conversation

@lbwexler

@lbwexler lbwexler commented May 4, 2026

Copy link
Copy Markdown
Member

Builds on spanItAll. Picks up the Hoist Loadable lifecycle improvements from xh/hoist-react#4361:

  • Adopts abortMode / LoadAbortedException in example models — superseded loads now abort cleanly via FetchService instead of being manually guarded with isStale / isObsolete checks.
  • Replaces ad-hoc try/catch error handling with overrides of Loadable.handleLoadException() for app-specific cleanup, dropping boilerplate around quiet/auto-refresh exceptions.

@lbwexler lbwexler requested a review from amcclain May 4, 2026 22:12
lbwexler and others added 6 commits May 8, 2026 16:31
Tracks API tightening in hoist-react PR #4360:
- `spanPrefix` → `telemetryPrefix` on all model/service overrides.
- `Runner.track` → `withTrack` on builder chains.
- `Runner.fetchJson` → `runFetchJson` on builder chains.

Folded post-fetch work that mutates or caches results (ContactService,
DocService) into the `.run()` lambda so it's covered by the runner's span.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…eshHandle

# Conflicts:
#	client-app/src/examples/contact/svc/ContactService.ts
Base automatically changed from spanItAll to develop June 2, 2026 21:54
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.

1 participant