Skip to content

feat(ui-debug): hide the view for study without filesystem storage#3278

Merged
skamril merged 3 commits into
devfrom
feature/5111-hide-debug
Jun 30, 2026
Merged

feat(ui-debug): hide the view for study without filesystem storage#3278
skamril merged 3 commits into
devfrom
feature/5111-hide-debug

Conversation

@skamril

@skamril skamril commented Jun 24, 2026

Copy link
Copy Markdown
Member

ANT-5111

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR (ANT-5111) adds study storage-mode awareness to the webapp so the Debug explore view is hidden (and the route is blocked) for studies that are not stored on the filesystem.

Changes:

  • Extend StudyMetadata with a storageMode field and populate it from the DTO (storage_mode).
  • Hide the Debug tab in the explore layout unless storageMode is "filesystem".
  • Add a runtime guard in the debug route component to prevent access when storage mode is not filesystem.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
webapp/src/types/types.ts Adds storageMode: StorageMode to StudyMetadata so UI can branch on storage backend.
webapp/src/services/utils/index.ts Maps storage_mode from StudyMetadataDTO into storageMode in StudyMetadata.
webapp/src/routes/_authenticated/studies/$studyId/explore/route.tsx Conditionally includes the Debug tab only for filesystem-backed studies.
webapp/src/routes/_authenticated/studies/$studyId/explore/debug/index.tsx Throws an error if the debug route is accessed for non-filesystem studies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread webapp/src/routes/_authenticated/studies/$studyId/explore/debug/index.tsx Outdated
@skamril skamril force-pushed the feature/5111-hide-debug branch from b0716e3 to 5b90ad3 Compare June 29, 2026 15:21
@skamril skamril force-pushed the feature/5111-hide-debug branch from 5b90ad3 to 423cc46 Compare June 29, 2026 18:50
@skamril skamril merged commit 9dce164 into dev Jun 30, 2026
14 of 15 checks passed
@skamril skamril deleted the feature/5111-hide-debug branch June 30, 2026 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants