Skip to content

feat: 统一资源权限并重构知识库配置 🤖 - #857

Closed
xerrors wants to merge 1 commit into
xerrors:mainfrom
Yuchuan925:feat/resource-permission-config
Closed

feat: 统一资源权限并重构知识库配置 🤖#857
xerrors wants to merge 1 commit into
xerrors:mainfrom
Yuchuan925:feat/resource-permission-config

Conversation

@xerrors

@xerrors xerrors commented Aug 3, 2026

Copy link
Copy Markdown
Owner

变更描述

统一 Agent、Skill 与知识库的细粒度共享权限,并重构知识库配置体验。

  • 新增公共权限解析模块,统一返回 none/read/manage
  • 知识库配置拆分为读取权限与管理权限,支持全局、部门和指定用户范围。
  • 知识库管理范围必须包含在读取范围内;创建者和超级管理员始终拥有管理权限;普通用户不能通过共享范围获得管理权限。
  • 兼容迁移旧版共享数据,保留原有完整权限语义。
  • 知识库配置弹窗拆分为“基础信息”“权限配置”“检索配置”三个 TAB。
  • 检索测试页移除右侧配置栏;配置弹窗的保存不会关闭弹窗,关闭和右上角 X 才负责退出。
  • 配置入口和弹窗文案统一使用“配置”,移除弹窗内删除知识库操作。

变更类型

  • 新功能
  • Bug 修复
  • 文档更新
  • 其他

测试

  • 已在 Docker 环境测试
  • 相关功能正常工作

实际验证:

  • docker exec api-dev pytest -q /app/test/unit/permissions/test_resource_permission.py /app/test/unit/repositories/test_agent_repository.py /app/test/unit/services/test_skill_service.py /app/test/integration/api/test_knowledge_router.py102 passed, 1 skipped
  • docker exec api-dev ruff check ... → 通过
  • pnpm lint → 通过
  • pnpm build → 通过(仅有依赖包已有的 Rolldown 注释及 chunk size 警告)
  • git diff --check → 通过
  • 本地浏览器真实链路:验证三个配置 TAB、检索测试全宽布局、保存后弹窗保持打开、关闭/X 正常退出。

相关权限选择器验证记录:Issue #840 comment

说明

  • 提交作者和提交者均为 Yuchuan。
  • 已完成 3 次 Human Review,并根据 Review 结果完成简化和修正。
  • 未执行全量后端测试;已执行权限、Agent、Skill 和知识库相关回归测试。

Closes #840

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@Yuchuan925 Yuchuan925 closed this Aug 3, 2026
Repository owner deleted a comment from Yuchuan925 Aug 4, 2026
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.

Feat: 权限上感觉还有点不足

2 participants