Skip to content

feat(app): support app resource permission snapshots and runtime authorization - #7559

Open
DigHuang wants to merge 3 commits into
labring:mainfrom
DigHuang:feat/app-permission
Open

feat(app): support app resource permission snapshots and runtime authorization#7559
DigHuang wants to merge 3 commits into
labring:mainfrom
DigHuang:feat/app-permission

Conversation

@DigHuang

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 32.23% 3165 / 9819
🔵 Statements 32.13% 3339 / 10389
🔵 Functions 26.96% 637 / 2362
🔵 Branches 26.6% 1542 / 5795
File CoverageNo changed files found.
Generated in workflow #2937 for commit e3b2065 by the Vitest Coverage Report Action

@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown

Admin Preview Image Ready!

ghcr.io/labring/fastgpt-pr:admin_e3b2065b937d5069ac3e90e36359153fef746f48

🕒 Time: 2026-09-03 23:23:21 (UTC+8)

@DigHuang
DigHuang force-pushed the feat/app-permission branch 4 times, most recently from bff8b71 to e7a5d50 Compare August 21, 2026 03:32
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

Build and publish successful - Preview fastgpt Image:

ghcr.io/labring/fastgpt-pr:fastgpt_de4cb6faa5693941f75733ad90bea057f7012933

🕒 Time: 2026-09-03 20:26:53 (UTC+8)

@DigHuang
DigHuang force-pushed the feat/app-permission branch 19 times, most recently from f2f0d5f to f9b9a95 Compare August 26, 2026 06:04
@DigHuang
DigHuang marked this pull request as ready for review August 26, 2026 06:04
@DigHuang
DigHuang force-pushed the feat/app-permission branch 2 times, most recently from d58da43 to b49afa8 Compare August 26, 2026 09:27
@DigHuang
DigHuang force-pushed the feat/app-permission branch 12 times, most recently from 072cb23 to a44363c Compare August 28, 2026 03:49
@github-actions

github-actions Bot commented Aug 28, 2026

Copy link
Copy Markdown

Docs preview deployed

🔗 👀 Click here to visit preview

ghcr.io/labring/fastgpt-docs-pr:de4cb6faa5693941f75733ad90bea057f7012933

🕒 Time: 2026-09-03 20:20:54 (UTC+8)

@DigHuang
DigHuang force-pushed the feat/app-permission branch 7 times, most recently from 3647e3b to 68a4408 Compare September 3, 2026 04:19
…orization

feat(app): complete resource permission checks and runtime authorization

feat(app): complete resource permission checks and runtime authorization

feat(app): optimize resource migration batching and support system skill authorization

feat(app): migrate workflow state to versions and enforce resource permissions

fix(app): complete version workflow resource migration

fix(app): align resource lookup id queries

fix(app): align resource lookup id queries
feat: resource ACL markers and draft concurrency

Introduce resource-level permission markers and optimistic draft checks.

- Add permissionDenied flag to resource/type schemas (skills, datasets, tools) and propagate through rewriteAppWorkflowToDetail.
- Perform viewer-side permission hints for snapshot-external resources; normalize child tool ids into parent tool snapshot entries.
- Treat invalid stored snapshots as empty to avoid bypassing publish-time ACL checks; resolveStoredAppResources now merges and normalizes legacy ids.
- Publish API accepts draftVersionId for optimistic concurrency and enforces matchedCount checks on updates (autoSave & publish).
- Load draft chatConfig for workflow debug when client omits it.
- Frontend: show permission-denied UI, workflow check supports resource_no_permission.
- Tests updated/added to cover behaviors.

feat(app): replace draftVersionId pointer with latest-version reads

fix: CAS update for published app version
@DigHuang
DigHuang force-pushed the feat/app-permission branch from de4cb6f to e3b2065 Compare September 3, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant