Skip to content

feat: add reviewed table structure editor - #140

Open
Bhavik-ag wants to merge 1 commit into
feat/production-connectionsfrom
feat/structure-editor
Open

Bhavik-ag wants to merge 1 commit into
feat/production-connectionsfrom
feat/structure-editor

Conversation

@Bhavik-ag

Copy link
Copy Markdown
Contributor

Summary

  • edit existing PostgreSQL table columns, constraints, and indexes
  • preview generated SQL before every schema change
  • require typed confirmation for production connections
  • preserve exact PostgreSQL type modifiers and support reviewed USING expressions

Validation

  • 73 tests
  • desktop and shared TypeScript checks
  • desktop lint
  • production build
  • git diff --check

@vercel

vercel Bot commented Sep 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated
dbdesk-web Ignored Ignored Sep 19, 2026 3:21am UTC
dbdesk-server Skipped Skipped Sep 19, 2026 3:21am UTC

@vercel
vercel Bot temporarily deployed to Preview – dbdesk-server September 18, 2026 18:14 Inactive
@greptile-apps

greptile-apps Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable new issue or outstanding previous finding remains.

Summary

Adds a reviewed PostgreSQL table-structure editor with SQL preview, production confirmation, exact type-modifier introspection, metadata refresh, and tests.

  • Supports column, constraint, and index operations.
  • Generates and previews one guarded PostgreSQL statement before execution.
  • Refreshes table introspection and workspace column metadata after successful changes.
  • Migrates the editor form to TanStack Form as required by repository guidance.
  • Changes since the previous review do not introduce an actionable issue.

Reviews (4) · Last reviewed commit: "feat: add reviewed table structure edito..."

@Bhavik-ag
Bhavik-ag force-pushed the feat/structure-editor branch from f9a42d1 to 7bd7309 Compare September 18, 2026 18:28
@vercel
vercel Bot temporarily deployed to Preview – dbdesk-server September 18, 2026 18:29 Inactive
@Bhavik-ag
Bhavik-ag changed the base branch from main to feat/production-connections September 18, 2026 18:29
@greptile-apps

This comment has been minimized.

@Bhavik-ag
Bhavik-ag force-pushed the feat/production-connections branch 3 times, most recently from cc03c5e to 0e4ea3a Compare September 19, 2026 03:00
@Bhavik-ag
Bhavik-ag force-pushed the feat/structure-editor branch from 7bd7309 to 5a661f6 Compare September 19, 2026 03:07
@vercel
vercel Bot temporarily deployed to Preview – dbdesk-server September 19, 2026 03:07 Inactive
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