Skip to content

fix: restrict app cloning for read-only users - #3526

Open
tlokeshrao wants to merge 3 commits into
devfrom
fix/project-clone-permissions
Open

fix: restrict app cloning for read-only users#3526
tlokeshrao wants to merge 3 commits into
devfrom
fix/project-clone-permissions

Conversation

@tlokeshrao

@tlokeshrao tlokeshrao commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Description

Prevent read-only users from cloning regular apps and projects while preserving cloning for agents and skills. Keep catalog action controls visually aligned when no overflow actions are available.

Changes Made

  • Hide Clone for read-only users on regular project types (CODE, BLOCKS, and INSIGHT).
  • Continue showing Clone for agents (WORKSPACE) and skills.
  • Require owner/editor permission for regular-project clone actions in the catalog, including bookmarked, personal, and discoverable views.
  • Reserve the overflow-action slot on catalog cards and show a muted, disabled vertical-ellipsis button when no actions are available.

How to Test

  1. Open the App catalog as a read-only user and verify regular apps do not offer Clone.
  2. Verify owner/editor users can still clone regular apps.
  3. Verify read-only users can still clone agents and skills.
  4. Compare catalog cards with and without overflow actions and verify the action icons remain aligned; unavailable overflow actions should appear disabled.

Validation

  • Biome checks pass for the touched frontend files.

Notes

The backend authorization guard is implemented in the companion Semoss repository change on the matching fix/project-clone-permissions branch.

@snyk-io

snyk-io Bot commented Aug 6, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@tlokeshrao tlokeshrao changed the title fix: viewers can't clone apps fix: restrict app cloning for read-only users Aug 6, 2026
@tlokeshrao
tlokeshrao marked this pull request as ready for review August 7, 2026 17:30
@tlokeshrao
tlokeshrao requested a review from a team as a code owner August 7, 2026 17:30
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