Skip to content

feat(offline): add manual connectivity verification - #104

Open
rdlabo wants to merge 2 commits into
mainfrom
feat/offline-connectivity-verifier
Open

feat(offline): add manual connectivity verification#104
rdlabo wants to merge 2 commits into
mainfrom
feat/offline-connectivity-verifier

Conversation

@rdlabo

@rdlabo rdlabo commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add manual API reachability verification to the existing offline network/coordinator services
  • share concurrent checks, apply an 8-second timeout, and bypass local read fallback
  • order API reachability observations so stale probe failures cannot overwrite newer results
  • cover success, HTTP errors, transport failures, timeout, retry, and single-flight behavior

Verification

  • npm run lint
  • npx ng test kit --watch=false (54 files, 1037 tests)
  • npm run prebuild:kit
  • upper-management review: approved after the observation-order fix
  • third-party acceptance review: approved after the observation-order fix

Rollout

After merge, publish a patch release before updating the five consuming apps.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

Devin Review

Comment thread projects/kit/offline/src/lib/offline-network.service.ts
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