Skip to content

Implement useLoadMoreSentinel hook and refactor navigation components#449

Merged
chenyme merged 3 commits into
devfrom
loading
Jul 8, 2026
Merged

Implement useLoadMoreSentinel hook and refactor navigation components#449
chenyme merged 3 commits into
devfrom
loading

Conversation

@chenyme

@chenyme chenyme commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix sidebar conversation navigation occasionally behaving like a full page reload when selecting historical conversations.

The sidebar conversation click handling now uses an explicit client-side navigation path for normal left-clicks while preserving native browser behavior for modified clicks, middle-clicks, and new-tab targets. This keeps conversation switches inside the SPA and avoids reload-like navigation behavior.

Change type

  • Bug fix
  • Feature
  • Documentation
  • Refactor
  • Configuration / deployment
  • Security hardening
  • Other

Affected areas

  • Frontend / UI
  • Backend / API
  • Authentication / authorization
  • Conversations / streaming
  • Files / RAG / extraction
  • Model routing / providers
  • MCP / tools
  • Billing / payments
  • Admin console
  • Deployment / Docker / configuration
  • Documentation

Verification

  • cd frontend && pnpm lint
  • git diff --check

Screenshots, API examples, or logs

No screenshots. This is a navigation behavior fix for sidebar conversation links.

Configuration, migration, and compatibility notes

No configuration or migration changes.

Normal left-clicks use client-side navigation. Modified clicks such as Cmd/Ctrl-click, Shift/Alt-click, middle-click, and explicit new-tab targets keep native browser behavior.

Documentation

  • Documentation is not needed for this change.
  • Documentation was updated.
  • Documentation still needs to be updated.

Security and privacy

  • No secrets, tokens, credentials, local config, or personal data are included.
  • User data access remains scoped by authenticated user context unless an admin-only path explicitly requires broader access.
  • Security-sensitive behavior was reviewed, including authentication, authorization, provider routing, file processing, billing, and admin APIs where relevant.

Checklist

  • I searched existing issues and pull requests.
  • Changes are focused and do not include unrelated refactors.
  • Tests or static verification were run where practical.
  • User-facing behavior, deployment steps, API contracts, or configuration changes are documented.
  • Generated artifacts are included only when this project explicitly requires them.
  • Caches, build output, .pyc files, .env files, and local storage data are not committed.

@chenyme chenyme self-assigned this Jul 8, 2026
@chenyme chenyme linked an issue Jul 8, 2026 that may be closed by this pull request
3 tasks
@chenyme
chenyme merged commit aed7635 into dev Jul 8, 2026
10 checks passed
@chenyme
chenyme deleted the loading branch July 8, 2026 12:22
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.

[Bug]: 侧边栏向下滚动无法加载更多对话

1 participant