Skip to content

feat(local-explorer-ui): Add data studio table explorer tab#12599

Open
NuroDev wants to merge 30 commits intomainfrom
NuroDev/local-explorer-studio-table-explorer
Open

feat(local-explorer-ui): Add data studio table explorer tab#12599
NuroDev wants to merge 30 commits intomainfrom
NuroDev/local-explorer-studio-table-explorer

Conversation

@NuroDev
Copy link
Member

@NuroDev NuroDev commented Feb 18, 2026

Fixes DEVX-2415.

Changes

  • Added the initial tab definition for the table explorer with placeholder for later PR's
  • Moved isEqual to a shared utility
  • Moved tokenizeSQL to a dedicated module
  • Adds a number of dependencies (required for the above) to the local explorer package
Screenshot 2026-02-18 at 5 14 39 pm
  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows: The WIP local explorer does not contain a test suite yet. UI tests are planned to be added in future.
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: WIP feature

A picture of a cute animal (not mandatory, but encouraged)


Open with Devin

@NuroDev NuroDev self-assigned this Feb 18, 2026
@changeset-bot
Copy link

changeset-bot bot commented Feb 18, 2026

🦋 Changeset detected

Latest commit: a806a1a

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@claude
Copy link
Contributor

claude bot commented Feb 18, 2026

Claude finished @NuroDev's task —— View job


Changeset Review

Reviewing .changeset/light-clocks-enter.md...

  • Read changeset guidelines
  • Review changeset file
  • Validate version type, changelog quality, and formatting
  • Post findings

⚠️ Issues found

The changeset needs improvement:

  1. Missing experimental feature opt-in details (per criterion 6): The changeset mentions "This is a WIP experimental feature" but doesn't explain how users can access or opt into this feature. According to the guidelines, changesets for experimental features should include a note on how users can opt in.

  2. Description could be more detailed: The title "Adds the tab definition for the table explorer" is clear, but the body is quite brief. While the changeset mentions this is a stepping stone and experimental, it would be helpful to include:

    • What the table explorer is/does
    • How users will interact with it (once complete)
    • Since this adds a "tab definition" but is WIP, clarify what's actually functional in this release vs. what's coming later

Suggested revision:

---
"@cloudflare/local-explorer-ui": minor
---

Add table explorer tab definition to data studio

This adds the initial tab definition and infrastructure for the table explorer in the local explorer's data studio. The table explorer will allow users to browse and edit D1 database tables with a visual interface.

**Note:** This is an experimental WIP feature. The tab is currently a placeholder and full functionality will be added in upcoming releases. To use the local explorer, run `wrangler dev --x-local-explorer`.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 18, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12599

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12599

miniflare

npm i https://pkg.pr.new/miniflare@12599

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12599

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12599

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12599

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12599

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12599

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@12599

wrangler

npm i https://pkg.pr.new/wrangler@12599

commit: a806a1a

@NuroDev NuroDev marked this pull request as ready for review February 18, 2026 17:09
@NuroDev NuroDev requested a review from a team as a code owner February 18, 2026 17:09
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@NuroDev
Copy link
Member Author

NuroDev commented Feb 18, 2026

Ignoring / resolving all Devin suggestions for now to avoid any logic changes from what we know works. We can revisit the suggestions in the future after the data studio has landed.

Comment on lines +1 to +3
@source "../node_modules/@cloudflare/kumo/dist/**/*.{js,jsx,ts,tsx}";

@import "@cloudflare/kumo/styles/tailwind";
Copy link
Member Author

Choose a reason for hiding this comment

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

Kumo styles were actually not being loaded before. This ensures we actually load the the CSS needed for them.

devin-ai-integration[bot]

This comment was marked as resolved.

@petebacondarwin
Copy link
Contributor

Just trying this out locally...

  • The "New" (table?) button is disabled. Expected?
  • There is a sqlite_sequence table. Should that be hidden?
  • Initially I have no rows in my table but there is no button to add one
  • At the bottom it says "Rows Read: 1" even though there are no rows in my table
  • I added some rows, programmatically but these are not showing in the table
  • But now it is showing the add and delete row buttons
  • I tried adding a new row - I was not able to set any of the fields but then I got this confirmation where the layout looks a bit broken:
    Screenshot 2026-02-19 at 16 34 13
  • Clicking "Confirm & Execute" resulted in this being displayed ([object Object]):
    Screenshot 2026-02-19 at 16 35 25
  • It is not clear what the 50 refers to in the pager bottom-right:
    image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Untriaged

Development

Successfully merging this pull request may close these issues.

2 participants

Comments