Skip to content

Feature: 知识库界面优化 #3909

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

Open
hzwllm wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Feature: 知识库界面优化 #3909

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2b31f61
Feature: 知识库界面优化
hzw110204 Sep 11, 2026
cb0baf2
Fix: 修复知识库门禁问题
hzw110204 Sep 14, 2026
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
478 changes: 236 additions & 242 deletions frontend/app/[locale]/knowledges/KnowledgeBaseConfiguration.tsx
View file
Open in desktop

Large diffs are not rendered by default.

62 changes: 22 additions & 40 deletions frontend/app/[locale]/knowledges/components/document/DocumentChunk.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,6 @@ interface DocumentChunkProps {
knowledgeBaseId: string; // Internal knowledge base ID / Elasticsearch index name
documents: Document[];
getFileIcon: (type: string) => string;
currentEmbeddingModel?: string | null;
knowledgeBaseEmbeddingModel?: string;
onChunkCountChange?: () => void; // Callback when chunk count changes (for updating KnowledgeBaseList)
permission?: string; // User's permission for this knowledge base (READ_ONLY, EDIT, etc.)
}
Expand All @@ -84,8 +82,6 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({
knowledgeBaseId,
documents,
getFileIcon,
currentEmbeddingModel,
knowledgeBaseEmbeddingModel,
onChunkCountChange,
permission,
}) => {
Expand Down Expand Up @@ -140,33 +136,10 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({

const effectiveIndexName = knowledgeBaseId || knowledgeBaseName;

const hasKnowledgeBaseModel =
Boolean(knowledgeBaseEmbeddingModel) &&
knowledgeBaseEmbeddingModel !== "unknown";
const hasCurrentModel = Boolean(currentEmbeddingModel);

// Determine if embedding models mismatch (specific condition for tooltip)
const isEmbeddingModelMismatch = React.useMemo(() => {
if (!hasKnowledgeBaseModel) {
return false;
}
return (
!hasCurrentModel || currentEmbeddingModel !== knowledgeBaseEmbeddingModel
);
}, [
currentEmbeddingModel,
hasCurrentModel,
hasKnowledgeBaseModel,
knowledgeBaseEmbeddingModel,
]);

// Determine if in read-only mode (embedding model mismatch OR user has READ_ONLY permission)
// Note: isReadOnlyMode is broader, includes model mismatch and other conditions
const isReadOnlyMode = React.useMemo(() => {
return permission === "READ_ONLY" || isEmbeddingModelMismatch;
}, [permission, isEmbeddingModelMismatch]);

const isSearchDisabled = isEmbeddingModelMismatch;
// The knowledge base owns its embedding model, so the global model must not
// disable chunk search or editing in this knowledge base.
const isReadOnlyMode = permission === "READ_ONLY";
const isSearchDisabled = false;

useEffect(() => {
if (isSearchDisabled) {
Expand Down Expand Up @@ -644,11 +617,12 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({
}, {});
}, [chunkSearchResult]);

const visibleDocuments = isChunkSearchActive && !chunkSearchLoading
? documents.filter(
(doc) => (chunkSearchResultMap?.[doc.id]?.length ?? 0) > 0
)
: documents;
const visibleDocuments =
isChunkSearchActive && !chunkSearchLoading
? documents.filter(
(doc) => (chunkSearchResultMap?.[doc.id]?.length ?? 0) > 0
)
: documents;

const tabItems = visibleDocuments.map((doc) => {
const chunkCount = isChunkSearchActive
Expand Down Expand Up @@ -762,7 +736,9 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({
className="self-center"
/>
</TooltipTrigger>
<TooltipContent>{t("document.chunk.tooltip.edit")}</TooltipContent>
<TooltipContent>
{t("document.chunk.tooltip.edit")}
</TooltipContent>
</Tooltip>
)}
<Tooltip>
Expand All @@ -775,7 +751,9 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({
className="self-center"
/>
</TooltipTrigger>
<TooltipContent>{t("document.chunk.tooltip.download")}</TooltipContent>
<TooltipContent>
{t("document.chunk.tooltip.download")}
</TooltipContent>
</Tooltip>
{!isReadOnlyMode && (
<Tooltip>
Expand All @@ -789,7 +767,9 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({
className="self-center"
/>
</TooltipTrigger>
<TooltipContent>{t("document.chunk.tooltip.delete")}</TooltipContent>
<TooltipContent>
{t("document.chunk.tooltip.delete")}
</TooltipContent>
</Tooltip>
)}
</div>
Expand Down Expand Up @@ -913,7 +893,9 @@ const DocumentChunk: React.FC<DocumentChunkProps> = ({
onClick={openCreateChunkModal}
/>
</TooltipTrigger>
<TooltipContent>{t("document.chunk.tooltip.create")}</TooltipContent>
<TooltipContent>
{t("document.chunk.tooltip.create")}
</TooltipContent>
</Tooltip>
)}
</div>
Expand Down
Loading
Loading