feat(assets): remove the assets UI (fetch_assets tool, resource, and view) - #30
Merged
Conversation
…view) The media asset gallery served here is superseded by Studio's native per-site Assets tab, which reads/uploads/deletes objects directly against the site's storage bucket. Remove the whole assets feature: - api/tools/assets.ts (fetch_assets / upload_asset / delete_asset) - api/resources/assets.ts (the "Assets UI" app resource) - web/tools/assets/ (the React gallery view) - their registrations in api/tools/index.ts, api/app.ts, web/router.tsx No other code references these symbols. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
admin-mcp | fc4bfca | Commit Preview URL Branch Preview URL |
Aug 19 2026, 06:00 PM |
guitavano
added a commit
to decocms/studio
that referenced
this pull request
Aug 19, 2026
…sites admin-MCP no longer serves fetch_assets (decocms/admin-mcp#30), so sites imported before this change carry a dead "Assets" pinned view. Skip it at render time — the native always-on Assets tab replaces it — so no data migration is needed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
guitavano
added a commit
to decocms/studio
that referenced
this pull request
Aug 19, 2026
…flag (#6262) * feat(assets): make the native Assets tab always-on, drop the feature flag The external admin-MCP assets UI is being removed (decocms/admin-mcp#30), so the native per-site Assets tab is now the assets experience — no longer gated. - use-main-panel-tabs: show the Assets tab whenever a bucket is associated to the site's slug, dropping the `native_assets_tab` gate. - Remove `native_assets_tab` from OrgFlagsSchema + its Navigation settings toggle and i18n; regenerate tool contracts. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * feat(assets): stop creating the fetch_assets pinned view on deco.cx import The native always-on Assets tab replaces the admin-MCP assets view, so the deco.cx import no longer pins `fetch_assets`. The Monitor pinned view (get_monitor_data) stays — admin-MCP still serves it. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * fix(assets): drop stale fetch_assets pinned view on already-imported sites admin-MCP no longer serves fetch_assets (decocms/admin-mcp#30), so sites imported before this change carry a dead "Assets" pinned view. Skip it at render time — the native always-on Assets tab replaces it — so no data migration is needed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removes the media assets gallery feature from admin-mcp. It's superseded by Studio's new native per-site Assets tab (decocms/studio#6251), which lists/uploads/deletes objects directly against the site's associated storage bucket instead of proxying through this MCP's UI.
Pure deletion — 6 files, ~1033 lines, no additions:
api/tools/assets.ts— thefetch_assets/upload_asset/delete_assettools + schemasapi/resources/assets.ts— theui://mcp-app/assets"Assets UI" app resourceweb/tools/assets/— the React gallery viewapi/tools/index.ts,api/app.ts,web/router.tsxVerified no other code references
assetsTool/uploadAssetTool/deleteAssetTool/ASSETS_RESOURCE_URI/createAssetsAppResource/AssetsPage/fetch_assets.Migration note
Studio creates the "Assets" pinned view (
fetch_assets) on deco.cx import. Once this ships, that pinned view has no tool to call — the native Assets tab (behind thenative_assets_taborg flag) replaces it. Coordinate enabling the flag / dropping the pinned view on the Studio side.Testing notes
tscor Biome issues introduced. The repo'scheck(node_modules zod version conflict) andci:check(a pre-existing format nit inweb/tools/monitor/index.tsx) already fail onmainfor reasons unrelated to this change; none of the touched files are flagged.🤖 Generated with Claude Code
Summary by cubic
Removes the legacy Assets gallery (tools and UI) in favor of Studio’s native per‑site Assets tab. Previously
fetch_assetspowered an in-app gallery with upload/delete; now those tools and theui://mcp-app/assetsresource are gone, and asset management happens in Studio.What changed
api/tools/assets.ts(fetch_assets,upload_asset,delete_asset) and their schemas.api/resources/assets.ts(removedui://mcp-app/assets) andweb/tools/assets/React gallery.api/tools/index.ts,api/app.ts, andweb/router.tsx. No remaining references tofetch_assets/upload_asset/delete_asset/ASSETS_RESOURCE_URI/AssetsPage.Migration
native_assets_taborg flag and remove any pinned “Assets” view that callsfetch_assets(it will have no tool).fetch_assets,upload_asset, ordelete_asset, migrate those flows to Studio’s Assets tab or direct bucket operations; this repo no longer provides replacements.Written for commit fc4bfca. Summary will update on new commits.