Skip to content

fix(session-import): show discovery errors as notifications - #3571

Merged
kev1n77 merged 1 commit into
GCWing:mainfrom
kev1n77:fmy/bugfix
Oct 28, 2026
Merged

kev1n77 merged 1 commit into
GCWing:mainfrom
kev1n77:fmy/bugfix

Conversation

@kev1n77

@kev1n77 kev1n77 commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

Cursor and other external session discovery failures now appear as separate transient notifications instead of a persistent warning occupying the session list.

Type and Areas

Type: Bug fix / UI/UX
Areas: Rust core, Web UI, i18n

Motivation / Impact

External records that cannot be read and records without an unambiguous workspace match are different conditions, but the session list previously combined them in one long-lived alert. The host now sends structured diagnostic categories, and the Web UI localizes and displays each category as a deduplicated warning notification. Existing warning fields remain available for older hosts.

Verification

  • cargo test --locked -p openbitfun-core --no-default-features --features external-sources,git,product-search --lib external_session_import:: — passed
  • cargo test --locked -p openbitfun-services-core --no-default-features --features local-storage --lib session::page::tests — 11 passed
  • pnpm --dir src/web-ui run test:run src/app/components/NavPanel/sections/sessions/useWorkspaceSessionDiscovery.test.tsx src/app/components/NavPanel/sections/sessions/SessionsSectionLayout.test.ts src/infrastructure/api/service-api/SessionAPI.test.ts — 37 passed
  • pnpm run type-check:web — passed
  • pnpm run i18n:audit — passed
  • git diff --check — passed

Reviewer Notes

  • Discovery diagnostics are keyed by ecosystem and category and deduplicated per active surface/workspace.
  • Existing native sessions remain available when external discovery is partial or unavailable.

@kev1n77
kev1n77 force-pushed the fmy/bugfix branch 4 times, most recently from 059d8e8 to fb1c35e Compare October 28, 2026 03:57
@kev1n77
kev1n77 merged commit 96d0833 into GCWing:main Oct 28, 2026
13 checks passed
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