Skip to content

fix: chat share #7789

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wangdan-fit2cloud merged 1 commit into from
Nov 10, 2026
Merged

fix: chat share #7789

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ui/src/conversation-panel/components/header/chat/index.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const props = defineProps<{
const list = useConversationListStore()

const msgs = useMessageListStore()
// 标题栏分享不针对某一轮，进入后不预先勾选
// 标题栏分享默认全选当前可分享记录
const shareLink = useShareLinkStoreOptional()
// 回答生成中或没有消息时不可分享
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value)
Expand Down
2 changes: 1 addition & 1 deletion ui/src/conversation-panel/components/header/portal/index.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ const emit = defineEmits<{ share: [] }>()
const list = useConversationListStore()
const msgs = useMessageListStore()
// 回答生成中或没有消息时不可分享
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value)
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value && !msgs.loading.value)

const handleNewConversation = () => (props.newConversation ? props.newConversation() : list.newConversation())
</script>
Expand Down
4 changes: 2 additions & 2 deletions ui/src/conversation-panel/components/message-list/share/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,8 @@ export function createShareLinkStore(deps: ShareLinkDeps) {
selectedRecordIds.value = checked ? [...availableRecordIds.value] : []
}

// 进入分享选择：以传入记录为初始勾选，仅保留当前可分享记录，空值与重复项忽略
const enterShare = (recordIds: (string | undefined)[] = []) => {
// 会话分享默认全选；内容入口传入记录时仅勾选对应问答，空值与重复项忽略
const enterShare = (recordIds: (string | undefined)[] = availableRecordIds.value) => {
selectedRecordIds.value = availableRecordIds.value.filter((recordId) => recordIds.includes(recordId))
deps.setShared(true)
}
Expand Down
15 changes: 6 additions & 9 deletions ui/src/conversation-panel/left-sidebar/account-menu/ApiKeyDialog.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,18 +1,15 @@
<script setup lang="ts">
import { computed, ref } from 'vue'
import { useRouter } from 'vue-router'
import { ref } from 'vue'
import ChatUserApi from '@/api/chat/chat-user'
import { copyText } from '@/utils/clipboard'

defineOptions({ name: 'ConversationApiKeyDialog' })

const router = useRouter()
const dialogVisible = ref(false)
const apiKey = ref('')
const loading = ref(false)
const mcpServiceType = ref<'agent' | 'knowledge'>('agent')
const apiUrl = computed(() => new URL(router.resolve({ name: 'portal-home' }).href, window.location.origin).href)
const maskedApiKey = computed(() => (apiKey.value ? `******${apiKey.value.slice(-4)}` : ''))
const mcpConfig = ref('')

// 每次打开重新读取；重置直接替换当前用户的唯一密钥。
function loadApiKey(regenerate = false) {
Expand Down Expand Up @@ -46,10 +43,10 @@ defineExpose({ open })
<template #subtitle>每个用户仅一个 API Key，绑定当前对话用户、与其同权。重置后旧 Key 立即失效</template>

<div class="mb-2">我的 API Key</div>
<el-input :model-value="maskedApiKey" readonly class="mb-4">
<el-input :model-value="`******${apiKey.slice(-4)}`" readonly class="mb-4">
<template #suffix>
<!-- 复制 API URL -->
<el-button text @click="copyText(maskedApiKey)">
<el-button text @click="copyText(apiKey)">
<MkIcon name="icon_copy_outlined" />
</el-button>
<MkTooltip content="重新生成" placement="top">
Expand All @@ -69,9 +66,9 @@ defineExpose({ open })
</el-radio-group>

<div class="group relative">
<el-input :model-value="maskedApiKey" readonly :autosize="{ minRows: 8, maxRows: 24 }" type="textarea" />
<el-input :model-value="mcpConfig" readonly :autosize="{ minRows: 8, maxRows: 24 }" type="textarea" />
<!-- 复制 MCP 配置 -->
<el-button class="group-hover-visible absolute right-3 top-3 z-10 shadow-md" circle @click="copyText(maskedApiKey)">
<el-button class="group-hover-visible absolute right-3 top-3 z-10 shadow-md" circle @click="copyText(mcpConfig)">
<MkIcon name="icon_copy_outlined" />
</el-button>
</div>
Expand Down
6 changes: 3 additions & 3 deletions ui/src/conversation-panel/left-sidebar/conversation-list/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,8 @@ export function createConversationListStore(deps: ConversationListDeps) {
const composerResetSignal = ref(0)

// View 在两个 Store 创建后绑定加载方法，避免与 message-list 循环初始化。
let loadMessages: () => void = () => {}
const bindLoader = (loader: () => void) => (loadMessages = loader)
let loadMessages: () => void | Promise<void> = () => {}
const bindLoader = (loader: () => void | Promise<void>) => (loadMessages = loader)

const resetComposer = () => {
composerResetSignal.value++
Expand All @@ -78,7 +78,7 @@ export function createConversationListStore(deps: ConversationListDeps) {
const selectConversation = (id: string) => {
resetComposer()
currentChatId.value = id
loadMessages()
return loadMessages()
}

const newConversation = () => {
Expand Down
21 changes: 14 additions & 7 deletions ui/src/conversation-panel/left-sidebar/conversation-list/index.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import AccountMenu from '../account-menu/index.vue'

import { useMessageListStore } from '../../components/message-list/index'
import { useShareLinkStoreOptional } from '../../components/message-list/share/index'
import { useViewType, VIEW_TYPE ,type ViewType} from '../../view/index'
import { useViewType, VIEW_TYPE, type ViewType } from '../../view/index'

defineOptions({ name: 'ConversationList' })

Expand All @@ -20,13 +20,20 @@ const authProfile = computed(() => (viewType === VIEW_TYPE.CHAT ? auth.authProfi

const SHOW_PROFILE: readonly ViewType[] = [VIEW_TYPE.CHAT, VIEW_TYPE.PORTAL]
const msgs = useMessageListStore()
// 会话菜单分享不针对某一轮，进入后不预先勾选
// 会话菜单分享默认全选，内容分享由内容入口指定记录。
const shareLink = useShareLinkStoreOptional()
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value)
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value && !msgs.loading.value)

const { appInfo, conversations, currentChatId, selectConversation, newConversation, deleteChat, toggleLeftSide, hasMore, loadMore } =
useConversationListStore()

// 非当前会话先激活并加载消息，避免分享到其他会话。
async function handleShareConversation(conversation: Conversation) {
if (msgs.loading.value) return
if (currentChatId.value !== conversation.id) await selectConversation(conversation.id)
if (currentChatId.value === conversation.id && canShare.value) shareLink?.enterShare()
}

// 历史时间优先使用更新时间，缺少时间的记录不显示占位文案。
const conversationTime = (conversation: Conversation) => relativeTimeFormat(conversation.update_time || conversation.create_time)
// 会话维护沿用调试历史记录的确认与请求流程。
Expand Down Expand Up @@ -114,11 +121,11 @@ function handleClearConversations() {
<el-button text><MkIcon name="icon_more_outlined" /></el-button>
<template #dropdown>
<MkDropdownMenu>
<!-- 分享当前激活对话 -->
<!-- 激活所选对话并全选分享 -->
<MkDropdownItem
v-if="viewType === VIEW_TYPE.CHAT && currentChatId === conversation.id"
:disabled="!canShare"
@click="shareLink?.enterShare()"
v-if="viewType === VIEW_TYPE.CHAT"
:disabled="msgs.loading.value || (currentChatId === conversation.id && !canShare)"
@click="handleShareConversation(conversation)"
>
<template #icon><MkIcon name="icon_share_outlined" /></template>
<span>分享</span>

@shaohuzhang1 shaohuzhang1 Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Here is a review of the provided code patch, focusing on potential issues, logic flaws, and optimization suggestions.

1. Potential Logic Issues & Race Conditions

Issue: Asynchronous Race Condition in handleShareConversation
The function handleShareConversation introduces an await inside an event handler (implied by @click). If the user rapidly clicks the "Share" menu item for a conversation while it is loading or switching, multiple asynchronous operations might queue up. While the initial check if (msgs.loading.value) return helps, there is no guard against concurrent calls to selectConversation.

  • Risk: Two click events might both pass the if (currentChatId.value !== conversation.id) check before the first selectConversation completes, leading to redundant API calls or state inconsistencies.
  • Suggestion: Add a local loading/guard flag or ensure selectConversation itself handles concurrent requests idempotently. Alternatively, use nextTick or explicit waiting if the store method returns a promise that resolves only after full state sync.

Refactored Suggestion:

// Add a simple guard or ensure selectConversation is robust
async function handleShareConversation(conversation: Conversation) {
  // Check global loading first
  if (msgs.loading.value) return;

  // Optional: Prevent re-entry while this specific action is processing
  // You might need a component-level ref like: const isSharing = ref(false);
  
  if (currentChatId.value !== conversation.id) {
    await selectConversation(conversation.id);
  }
  
  // Re-check conditions after async operation
  if (currentChatId.value === conversation.id && canShare.value) {
    shareLink?.enterShare();
  }
}

Issue: Stale State Check After Async Operation
After await selectConversation(conversation.id), the code checks canShare.value. However, canShare depends on msgs.messageList.value.length and msgs.loading.value. Depending on how useMessageListStore and useViewType reactively update, there might be a brief moment where currentChatId has updated but the message list hasn't finished populating or updating the loading flag immediately. Ensure that canShare accurately reflects the newly loaded state immediately upon resolution of selectConversation.

2. Code Quality & Style

Positive Change: Formatting Fixes

  • The formatting change in the import statement (type ViewType} → , type ViewType }) and spacing improvements are good hygiene practices.

Comment Ambiguity

  • The comment // 非当前会话先激活并加载消息，避免分享到其他会话。 (Activate non-current sessions first and load messages to avoid sharing others.) is slightly misleading. The new logic allows sharing from non-current sessions by activating them first. A better phrasing might be: "Activate the target session if it's not current, ensuring its messages are loaded before entering share mode."

3. UI/UX & Disable Logic

Complexity in :disabled Prop
The template logic for disabling the button is somewhat convoluted:

:disabled="msgs.loading.value || (currentChatId === conversation.id && !canShare)"

This means:

  1. If any loading is happening globally (msgs.loading), disable it.
  2. If it is the current chat AND canShare is false (e.g., empty messages/streaming), disable it.

However, if you click a non-current conversation:

  1. currentChatId !== conversation.id.
  2. msgs.loading.value is checked. If false, the button is enabled.
  3. But what if msgs.loading refers to the current view's messages, not the target view? If msgs.loading is a global store reference indicating "main view is loading", then clicking a background conversation share button might be allowed even if the main view is busy, potentially causing conflicts when switching contexts. Ensure msgs.loading is context-aware or intended to be global.

Optimization: Consider extracting this logic into a computed property for readability.

const isShareDisabledForConv = (conv: Conversation) => {
  if (msgs.loading.value) return true;
  if (currentChatId.value === conv.id) return !canShare.value;
  return false; // Assuming non-current chats are always shareable until loaded? 
                // Or should it also check if the target chat is known/valid?
};

4. Missing Definitions / Dependencies

Undefined Types and Functions
The snippet uses Conversation, relativeTimeFormat, MkIcon, MkDropdownMenu, MkDropdownItem, el-button, and computed. These must be properly imported or defined elsewhere. Specifically:

  • Conversation: Ensure it is imported.
  • relativeTimeFormat: Ensure it is imported.
  • useConversationListStore: Ensure it is imported.

Missing Error Handling
selectConversation and shareLink?.enterShare() do not have .catch() blocks. If selectConversation fails (network error, permission denied), the UI will remain disabled or stuck, and no feedback is given to the user.

Suggestion:

async function handleShareConversation(conversation: Conversation) {
  if (msgs.loading.value) return;
  
  try {
    if (currentChatId.value !== conversation.id) {
      await selectConversation(conversation.id);
    }
    
    if (currentChatId.value === conversation.id && canShare.value) {
      shareLink?.enterShare();
    } else {
      console.warn("Share condition not met after activation.");
       // Optionally show a toast message
    }
  } catch (error) {
    console.error("Failed to activate conversation for sharing:", error);
    // Show user-friendly error message using useMessage() or similar
  }
}

Summary of Recommendations

  1. Add Try/Catch: Handle async errors in handleShareConversation.
  2. Race Condition Guard: Consider preventing repeated clicks during the async switching process.
  3. Refine Disable Logic: Verify if msgs.loading correctly prevents interaction with background conversations if intended. Extract complex disable logic to a helper/computed for readability if used in multiple places.
  4. Clarify Comments: Update comments to reflect that sharing is now possible for non-current sessions by activating them first.

Expand Down
19 changes: 9 additions & 10 deletions ...rc/conversation-panel/left-sidebar/portal-conversation-list/HistoryConversationDrawer.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,22 +8,18 @@ import { usePortalConversationListStore } from './index'
import RenameConversation from '../../components/rename/index.vue'

import { useMessageListStore } from '../../components/message-list/index'
import { useShareLinkStoreOptional } from '../../components/message-list/share/index'

defineOptions({ name: 'HistoryConversationDrawer' })
const props = defineProps<{ applicationId: string }>()
const visible = defineModel<boolean>({ required: true })
const portalList = usePortalConversationListStore()
const conversationList = useConversationListStore()
const msgs = useMessageListStore()
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value)
const shareLink = useShareLinkStoreOptional()
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value && !msgs.loading.value)

// 开启当前对话的分享选择（不预先勾选），并关闭历史抽屉。
const handleShareConversation = () => {
if (!canShare.value) return
shareLink?.enterShare()
visible.value = false
// 激活所选会话并全选分享，成功后关闭历史抽屉。
const handleShareConversation = async (conversation: PortalConversationSummary) => {
if (await portalList.shareConversation(selectedApplicationId.value, conversation.id)) visible.value = false
}
const selectedApplicationId = ref(props.applicationId)
const keyword = ref('')
Expand Down Expand Up @@ -102,8 +98,11 @@ const handleDeleteConversation = (conversation: PortalConversationSummary) => {
<el-button text><MkIcon name="icon_more_outlined" /></el-button>
<template #dropdown>
<MkDropdownMenu>
<!-- 分享当前激活对话 -->
<MkDropdownItem v-if="isActiveConversation(conversation)" :disabled="!canShare" @click="handleShareConversation">
<!-- 激活所选对话并全选分享 -->
<MkDropdownItem
:disabled="msgs.loading.value || (isActiveConversation(conversation) && !canShare)"
@click="handleShareConversation(conversation)"
>
<template #icon><MkIcon name="icon_share_outlined" /></template>
<span>分享</span>
</MkDropdownItem>

@shaohuzhang1 shaohuzhang1 Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Here is a review of the provided code diff, highlighting irregularities, potential issues, and optimization suggestions.

1. Logical & Functional Issues

A. Race Condition & State Management in handleShareConversation

  • Issue: The function calls portalList.shareConversation(...). If this method internally sets state (e.g., loading states) or if there’s a delay, closing the drawer (visible.value = false) immediately upon resolution might conflict with UI feedback expected by the user or other components.
  • Suggestion: Ensure shareConversation returns a promise that resolves only after the core action is successful. If it triggers a navigation or overlay change, closing the drawer before that might cause visual flickering. Consider adding a loading indicator specific to that dropdown item if the operation is slow.

B. Dependency Removal (useShareLinkStoreOptional)

  • Issue: You removed useShareLinkStoreOptional. The old logic called shareLink?.enterShare(), which likely handled complex pre-checks, permission validations, or state preparation for sharing. The new logic directly calls portalList.shareConversation.
  • Risk: If portalList.shareConversation does not encapsulate all the validation/preparation logic previously handled by the share store, you might lose features (e.g., checking if sharing is enabled for the app, generating link IDs, etc.).
  • Verification Required: Confirm that portalList.shareConversation handles all necessary pre-validation. If it doesn’t, moving this logic to the store might violate single-responsibility principles or introduce bugs where shares fail silently or crash.

C. canShare Computed Property Change

  • Change: Added && !msgs.loading.value.
  • Assessment: This is good. Preventing shares during initial message list load prevents sharing incomplete data.
  • Optimization: Ensure msgs.loading.value reflects the entirety of the message loading process, including any sub-fetches (like metadata or attachments), otherwise partial data might still be shared.

2. Code Quality & Best Practices

A. Direct Store Method Call from Component

  • Current: await portalList.shareConversation(selectedApplicationId.value, conversation.id)
  • Suggestion: It’s generally better practice to keep "action orchestration" within the store if it involves multiple steps or side effects. However, calling a clear, single-purpose store action from a component is acceptable in Pinia/Vuex if the action itself is well-defined.
  • Improvement: If shareConversation is an async API call, consider handling errors gracefully here. Currently, if shareConversation rejects, the drawer stays open, but there’s no explicit try/catch or error toast feedback in this snippet. Add error handling: 
    const handleShareConversation = async (conversation: PortalConversationSummary) => {
  try {
    const success = await portalList.shareConversation(selectedApplicationId.value, conversation.id);
    if (success) {
      visible.value = false;
      // Optionally show success toast
    }
  } catch (error) {
    // Handle error (e.g., show toast 'Share failed')
    console.error('Failed to share conversation', error);
  }
};

B. Dropdown Item Logic Complexity

  • Current: 
    :disabled="msgs.loading.value || (isActiveConversation(conversation) && !canShare)"
  • Analysis:
    • !canShare already includes !msgs.streamLoading.value and !msgs.loading.value (from the updated computed prop).
    • So msgs.loading.value is redundant in the disabled condition because canShare would be false if msgs.loading.value is true, but only if isActiveConversation(conversation) is true.
    • Bug/Inconsistency: If the conversation is NOT active, canShare is irrelevant to this item's logic in the second part of the OR condition. However, canShare is computed based on the active conversation's messages (msgs.messages.value).
    • Crucial Check: Does shareConversation require the conversation to be loaded/active?
      • If you are trying to share a non-active conversation, msgs.messages.value refers to the currently active conversation, not the conversation from the loop.
      • Therefore, canShare is technically misleading when applied to non-active conversations. You cannot determine if a non-active conversation has messages or is currently streaming without fetching its data.
      • If portalList.shareConversation(id) works regardless of active status, then the disabled check relying on canShare (which checks active conversation state) is logically flawed for non-active conversations.

Refinement for Dropdown Disabled Logic:

You should clarify why non-active conversations can be shared. If portalList.shareConversation can share any conversation by ID, then msgs.loading (of the active convo) should not block it unless the API requires background loading.

If the intention is: "Only allow sharing if the conversation is ready," but we don't know the readiness of non-active convos, you might need to remove the canShare check entirely for non-active items, or fetch metadata.

Suggested Simpler Logic:
Since shareConversation takes the id directly, it likely handles its own state validation. The disabled prop should probably only block if the global system is too busy or the specific item is pending.

  • Remove dependence on canShare (which is active-convo-specific) from the general dropdown unless shareConversation requires local message cache.
  • If it requires cache, you must load the conversation before clicking, or disable non-active items until they've been viewed.

3. Optimization Suggestions

  1. Remove Redundant Loading Checks:
    Since canShare now includes !msgs.loading.value, and messages.value.length > 0 implies loads finished for active convo, the disabled logic in the template is messy.

    If shareConversation is backend-driven and doesn't need local messages:

    <MkDropdownItem
  :disabled="msgs.loading.value" <!-- Only block if global loading -->
  @click="handleShareConversation(conversation)"
>

    If it does need local messages (only possible for active convo):

    <MkDropdownItem
  v-if="isActiveConversation(conversation)"
  :disabled="!canShare"
  @click="handleShareConversation(conversation)"
>

    And hide/omit share option for non-active conversations entirely if they can't be shared without activation. This is cleaner than trying to share non-active via a potentially unreliable shareConversation that might trigger heavy lazy-loading.

  2. Type Safety:
    The diff uses conversation: PortalConversationSummary in arguments but the original removed usage of PortalConversationSummary import? No, it seems present. Ensure PortalConversationSummary is imported in the script setup section if used in TS types (it seems to be part of the existing structure).

  3. UX Improvement:
    Adding an icon or text change for "Sharing..." state while waiting for await portalList.shareConversation would improve UX, as closing the drawer might feel abrupt if the network request is slow.

Summary of Recommendations

  1. Fix Potential Logic Error: Verify if canShare (based on active conversation) should apply to non-active conversations in the dropdown. It likely shouldn't. If non-active convos can be shared via ID, remove canShare from their disabled logic. If they can't, restrict the dropdown item to active conversations only (v-if="isActiveConversation(...)").
  2. Add Error Handling: Wrap handleShareConversation in try/catch to notify users if sharing fails.
  3. Verify Store Responsibilities: Ensure portalList.shareConversation replicates necessary logic from the removed useShareLinkStoreOptional (e.g., permissions, cleanup).
  4. Simplify Template Logic: Decouple msgs.loading (global/active state) from individual item state where inappropriate.

Expand Down
3 changes: 3 additions & 0 deletions ui/src/conversation-panel/left-sidebar/portal-conversation-list/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ export interface PortalConversationListDeps {
openApplication: (applicationId: string) => void
/** 打开其他智能体的会话。 */
openConversation: (applicationId: string, chatId: string) => void
/** 激活所选会话，加载后全选分享；成功进入返回 true。 */
shareConversation: (applicationId: string, chatId: string) => Promise<boolean>
/** 打开全部智能体页。 */
openApplicationList: () => void
/** 打开选择智能体抽屉。 */
Expand Down Expand Up @@ -155,6 +157,7 @@ export function createPortalConversationListStore(deps: PortalConversationListDe
isApplicationListActive: deps.isApplicationListActive,
openApplication: deps.openApplication,
openConversation: deps.openConversation,
shareConversation: deps.shareConversation,
openApplicationList: deps.openApplicationList,
newConversation,
}
Expand Down
11 changes: 11 additions & 0 deletions ui/src/conversation-panel/left-sidebar/portal-conversation-list/index.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,12 @@ import HistoryConversationDrawer from './HistoryConversationDrawer.vue'
import AccountMenu from '../account-menu/index.vue'
import { usePortalConversationListStore, type PortalApplicationGroup } from './index'
import RenameConversation from '../../components/rename/index.vue'
import { useMessageListStore } from '../../components/message-list/index'

defineOptions({ name: 'PortalConversationList' })

const msgs = useMessageListStore()
const canShare = computed(() => msgs.messages.value.length > 0 && !msgs.streamLoading.value && !msgs.loading.value)
const portalList = usePortalConversationListStore()
const { applicationGroups, currentApplicationId, isApplicationListActive, toggleGroup } = portalList
const { conversations, currentChatId, selectConversation, deleteChat, newConversation, switchShared, toggleLeftSide } = useConversationListStore()
Expand Down Expand Up @@ -193,6 +196,14 @@ const handleClearConversations = () => {
<el-button text><MkIcon name="icon_more_outlined" /></el-button>
<template #dropdown>
<MkDropdownMenu>
<!-- 激活所选对话并全选分享 -->
<MkDropdownItem
:disabled="msgs.loading.value || (isActiveConversation(group, conversation) && !canShare)"
@click="portalList.shareConversation(group.id, conversation.id)"
>
<template #icon><MkIcon name="icon_share_outlined" /></template>
<span>分享</span>
</MkDropdownItem>
<RenameConversation :conversation="conversation" :application-id="group.id" @renamed="conversation.abstract = $event" />
<!-- 删除历史对话 -->
<MkDropdownItem divided @click="handleDeleteConversation(group, conversation)">
Expand Down
6 changes: 4 additions & 2 deletions ui/src/conversation-panel/main/portal-chat-panel/index.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
<script setup lang="ts">
import { useShareLinkStoreOptional } from '../../components/message-list/share/index'
import PortalHeader from '../../components/header/portal/index.vue'
import MessageList from '../../components/message-list/index.vue'
import MessageInput from '../../components/message-input/index.vue'
Expand All @@ -8,6 +9,7 @@ import { usePortalConversationListStore } from '../../left-sidebar/portal-conver

defineOptions({ name: 'PortalChatPanel' })

const shareLink = useShareLinkStoreOptional()
const list = useConversationListStore()
const input = useMessageInputStore()
const portalList = usePortalConversationListStore()
Expand All @@ -19,12 +21,12 @@ const handleDrop = (e: DragEvent) => input.addFiles(e.dataTransfer?.files)
<!-- 门户对话主面板:header + message-list + message-input -->
<main class="mk-conversation-main" @drop.prevent="handleDrop" @dragover.prevent>
<!-- 收起侧栏时标题栏的新建对话与侧栏一致：没有当前智能体时选择智能体 -->
<PortalHeader :new-conversation="portalList.newConversation" />
<PortalHeader :new-conversation="portalList.newConversation" shareable @share="shareLink?.enterShare()" />
<MessageList :key="list.currentChatId.value" v-if="portalList.currentApplicationId.value" :app-name="list.appInfo.value?.name || 'AI 助手'" />
<!-- 未确定智能体时不显示欢迎语：分组加载完成且没有可用智能体时显示空状态 -->
<div v-else class="flex-center min-h-0 flex-1">
<MkEmpty v-if="portalList.isApplicationGroupsLoaded.value" description="暂无可用智能体" />
</div>
<footer class="flex-center shrink-0 p-4"><MessageInput /></footer>
<footer v-if="!list.isShared.value" class="mk-conversation-content flex-center shrink-0 p-4"><MessageInput /></footer>
</main>
</template>
32 changes: 31 additions & 1 deletion ui/src/conversation-panel/view/portal/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { computed, watch } from 'vue'
import { computed, nextTick, watch } from 'vue'
import { useRoute, useRouter } from 'vue-router'
import ConversationApi from '@/api/chat/conversation'
import PortalApi from '@/api/chat/portal'
Expand Down Expand Up @@ -62,6 +62,36 @@ export function createPortalConversation(options: PortalConversationOptions): Po
currentApplicationId,
isApplicationListActive,
openApplication,
// 跨智能体通过导航激活，等待路由驱动的消息加载结束后再全选分享。
shareConversation: async (applicationId, chatId) => {
if (bundle.msgs.loading.value) return false
if (applicationId !== currentApplicationId.value) {
const failure = await router.push({ name: 'portal-application-chat', params: { applicationId, chatId }, query: route.query })
if (failure) return false
await nextTick()
if (bundle.msgs.loading.value) {
await new Promise<void>((resolve) => {
const stop = watch(bundle.msgs.loading, (loading) => {
if (!loading) {
stop()
resolve()
}
})
})
}
} else if (bundle.list.currentChatId.value !== chatId) {
await bundle.list.selectConversation(chatId)
}
if (
currentApplicationId.value !== applicationId ||
bundle.list.currentChatId.value !== chatId ||
!bundle.msgs.messages.value.length ||
bundle.msgs.streamLoading.value
)
return false
bundle.share.enterShare()
return true
},
openConversation: (applicationId, chatId) =>
void router.push({ name: 'portal-application-chat', params: { applicationId, chatId }, query: route.query }),
openApplicationList: () => void router.push({ name: 'portal-application-list', query: route.query }),

@shaohuzhang1 shaohuzhang1 Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Review

1. Correctness / likely bugs

a) Strict type comparison on route params (high)
currentApplicationId almost certainly comes from route.params.applicationId, which is always a string in Vue Router. If callers pass a numeric id, both
applicationId !== currentApplicationId.value and the final guard
currentApplicationId.value !== applicationId will always be true → you'll always navigate, and then always return false. Normalize with String(...) or type the params as string.

b) router.push return value
push resolves with a NavigationFailure for benign cases too — notably NavigationFailureType.duplicated (same route). Treating any truthy return as failure means a no-op navigation that is already on the target route makes sharing silently fail. Use isNavigationFailure(failure, NavigationFailureType.duplicated) to distinguish. Also push can reject (guard throws in some setups) — wrap in try/catch.

c) The enterShare() return value is ignored, yet shareConversation returns true. If enterShare() can fail / be a no-op (already in share mode, no share permission, empty selection), the caller gets a false positive. Return bundle.share.enterShare() ?? true or check its result; also ensure it's awaited if async.

d) Behavior asymmetry vs. openConversation
The else if branch calls bundle.list.selectConversation(chatId) without touching the route, whereas openConversation does router.push. Result: the URL no longer reflects the current chat — reload / deep-link / share-link diverge, and the next route-driven watcher may fight this state. If that's intentional, document it; otherwise push here too.

e) The initial if (bundle.msgs.loading.value) return false contradicts the later "wait for load" logic
You bail if a load is in flight at call time, but you happily wait out a load you triggered yourself. If the in-flight load happens to be for the very conversation being shared, you return false for no good reason. Decide on one policy.

2. Race conditions / robustness

f) The busy-wait on bundle.msgs.loading is fragile and can hang forever (high)

  • If the flag never flips to false (request fails without resetting it, request cancelled, navigation aborted), the promise never resolves — the function hangs, and multiple rapid clicks accumulate a new watcher per call.
  • The watcher is never stopped on early exit / component unmount / rejection → leak, plus it can fire later and call enterShare() for a conversation that is no longer current (the final guards help, but they run only once the watcher fires).
  • A brief true → false → true transition (two consecutive loads) can resolve on the first false[…]. Add a timeout, a cancellation flag, and re-verify currentChatId inside the watcher.

g) await nextTick() (~1 tick) is not a reliable sync point
It assumes the route watcher sets loading = true within one tick. If the trigger uses flush: 'post', an async guard, or an onMounted fetch, the flag may not be set yet → you fall through to messages.length === 0 and return false. Best fix: have the loader expose a promise (e.g. bundle.msgs.loadConversation(chatId): Promise<void>) and await it directly; keep the reactive-flag wait only as a fallback (VueUse until/watchOnce with immediate: true + timeout).

h) No concurrency guard
Two rapid invocations create two watchers and two enterShare() calls. Add an inFlight boolean (or an AbortController/token) and bail/return the existing promise.

i) bundle.list.selectConversation(chatId) isn't guarded
If it rejects, shareConversation rejects instead of returning false, unlike every other failure path. Wrap the whole body in try/catch/finally (finally → stop()).

j) messages.value.length === 0 check
Fine as a guard, but note it also fails for: lazily-paginated/virtualized conversations, or a conversation whose first page hasn't arrived yet — the same items (f) leaves you with. Also use messages.value.length === 0 rather than !…length for clarity.

3. Style / typing / minor

  • Types: the object-literal property has no contextual typing for applicationId/chatId; with noImplicitAny this is an error. Annotate (applicationId: string, chatId: string): Promise<boolean>.
  • Missing braces on the multi-line if (...) + bare return false — brittle on future edits and usually flagged by curly. Add braces.
  • Comment mismatch: the comment says "全选分享" (select-all & share) but no selection happens — only enterShare(). Either fix the code or the comment. Also consider matching the file's language for comments.
  • enterShare is called even when the user is already in share mode — verify it is idempotent.
  • Duplicated currentApplicationId / currentChatId checks across functions — consider extracting a small isCurrentConversation(appId, chatId) helper.
  • watch/nextTick imports are fine, but if you switch to awaiting the loader promise, watch (or nextTick) may become unused — tidy the import.

4. Suggested rewrite

import { computed, nextTick, watch } from 'vue'
import { isNavigationFailure, NavigationFailureType } from 'vue-router'
import { until } from '@vueuse/core' // optional helper

const SHARE_LOAD_TIMEOUT = 10_000

shareConversation: async (applicationId: string, chatId: string): Promise<boolean> => {
  // Guard against concurrent invocations
  if (bundle.share.enterShare.pending) return false

  try {
    if (String(applicationId) !== String(currentApplicationId.value)) {
      const failure = await router.push({
        name: 'portal-application-chat',
        params: { applicationId, chatId },
        query: route.query,
      })
      if (failure && !isNavigationFailure(failure, NavigationFailureType.duplicated)) {
        return false
      }

      await nextTick()

      // Prefer: await bundle.msgs.loaded(chatId)
      if (bundle.msgs.loading.value) {
        const settled = await until(() => !bundle.msgs.loading.value)
          .toPromise()
          .then(() => true)
          .catch(() => false)
        // add a timeout wrapper around this in practice
        if (!settled) return false
      }
    } else if (bundle.list.currentChatId.value !== chatId) {
      await bundle.list.selectConversation(chatId)
    }

    if (
      String(currentApplicationId.value) !== String(applicationId) ||
      bundle.list.currentChatId.value !== chatId ||
      bundle.msgs.loading.value ||
      bundle.msgs.streamLoading.value ||
      bundle.msgs.messages.value.length === 0
    ) {
      return false
    }

    bundle.share.enterShare()
    return true
  } catch {
    return false
  }
}

Priority order: (a) type/string comparison, (f) watcher leak/hang + timeout, (b) duplicated-navigation handling, (c) unawaited enterShare, (d) route/state divergence. The rest are polish.

Expand Down
4 changes: 2 additions & 2 deletions ui/src/layout/sidebar/WorkspaceSidebar.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ function navigateToWorkspaceMenu(workspaceMenuItem: LayoutMenuItem) {
<template v-for="workspaceMenuItem in workspaceMenuItems" :key="workspaceMenuItem.name">
<div
v-if="workspaceMenuItem.permission ? workspaceMenuItem.permission(route) : true"
class="flex-col-center w-full cursor-pointer rounded-md py-2 text-xs font-semibold text-N600 hover:bg-N900/10"
:class="isWorkspaceMenuActive(workspaceMenuItem) && 'bg-white text-N900'"
class="flex-col-center w-full cursor-pointer rounded-md py-2 text-xs font-semibold text-N600"
:class="isWorkspaceMenuActive(workspaceMenuItem) ? 'bg-white text-N900' : 'hover:bg-N900/10'"
@click="navigateToWorkspaceMenu(workspaceMenuItem)"
>
<MkIcon
Expand Down
4 changes: 3 additions & 1 deletion ui/src/views/VIEW_README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -514,7 +514,9 @@ System 资源管理的修正内容与添加知识库选项仅包含当前智能
全部智能体与选择抽屉复用卡片，抽屉每次打开重新查询，搜索状态独立。
门户历史侧栏仅展示有历史的智能体及最近 5 条会话，剩余数量取分组总数；“查看更多”打开历史对话抽屉，
支持切换智能体、摘要搜索、滚动分页及打开／删除会话，删除后同步刷新侧栏。
单应用历史列表与门户历史抽屉仅在当前激活会话的菜单显示分享入口；无消息或回答生成中不可分享。
单应用历史列表、门户分组历史列表与历史抽屉支持先激活所选会话、等待消息加载后进入分享；
门户支持跨智能体激活，标题栏也可分享当前对话。
无消息或回答生成中不可分享；会话分享默认全选当前已加载的可分享记录，内容入口仅选中对应问答记录。
门户复用 Chat 分享 Store，抽屉点击分享后关闭并进入消息选择模式。
Chat／Portal 视图将左栏布局模式绑定到会话列表 Store；开启分享时，抽屉模式自动收起左栏。

Expand Down
17 changes: 6 additions & 11 deletions ui/src/views/chat/login/index.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,10 +134,10 @@ onBeforeMount(async () => {
}
})

const enableAuth=computed(()=>{
const enableAuth = computed(() => {
return auth.authProfile?.enable_auth
})

const goBack = () => {
if (enableAuth.value) {
return
Expand All @@ -148,21 +148,16 @@ const goBack = () => {

<template>
<div class="chat-login flex-center flex-col">

<div class="mb-6 flex-align-center gap-4 max-w-full">
<ApplicationIcon :icon="auth.authProfile?.logo" :size="32" class="shrink-0" />
<h2 class="max-w-full truncate" :title="loginTitle">{{ loginTitle }}</h2>
</div>
<el-card class="chat-login__card w-100">
<!-- 返回上一页 -->
<span
v-if="!enableAuth"
class="mb-4 inline-flex items-center gap-1 cursor-pointer text-sm text-N900 hover:text-primary"
@click="goBack"
>
<MkIcon :icon="ArrowLeft" :size="14" />
返回
</span>
<el-button text v-if="!enableAuth" @click="goBack" class="-ml-1 mb-4 text-N900!">
<MkIcon :icon="ArrowLeft" />
<span>返回</span>
</el-button>
<LoginModePanel v-if="!isClientLoggingIn" :login-config="loginConfig" :login-service="loginService" />
</el-card>
</div>
Expand Down
2 changes: 1 addition & 1 deletion ui/src/views/home/components/HomeQuickCreate.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ onMounted(() =>
<h6>创建智能体</h6>
<p class="text-sm text-N600">手动搭建、AI 生成或导入创建</p>
</div>
<MkIcon name="icon_down_outlined" />
<MkIcon name="icon_down_outlined" class="text-N600!" />
</div>
</el-card>
</template>
Expand Down
Loading