Skip to content

fix(file): trust local workspace roots for fs routes#527

Open
kaizhou-lab wants to merge 1 commit into
mainfrom
fix/fs-list-custom-workspace-478
Open

fix(file): trust local workspace roots for fs routes#527
kaizhou-lab wants to merge 1 commit into
mainfrom
fix/fs-list-custom-workspace-478

Conversation

@kaizhou-lab

@kaizhou-lab kaizhou-lab commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep /api/fs/list on the root-based local contract and trust the requested workspace root for listing
  • allow request-scoped workspace/source roots for fs dir/write/copy/remove/rename/zip routes so custom workspaces and external disks work
  • add API, service, and E2E coverage for non-sandbox roots plus boundary checks when an explicit root is supplied

Test Plan

  • cargo fmt --all -- --check
  • cargo test -p aionui-api-types
  • cargo test -p aionui-file
  • cargo test -p aionui-app --test file_e2e
  • cargo clippy -p aionui-api-types -p aionui-file -p aionui-app -- -D warnings

Note: pushed directly with git push --force-with-lease; did not run just push per request.

@kaizhou-lab kaizhou-lab force-pushed the fix/fs-list-custom-workspace-478 branch from cd4a5f5 to 9e378bd Compare June 26, 2026 14:11
@kaizhou-lab kaizhou-lab changed the title fix(file): authorize custom workspace listing fix(file): trust local workspace roots for fs routes Jun 26, 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.

1 participant