Bug: Auto-refresh does not trigger in query views

[abdotop]

refactor: Refactor columnsMap to a plain object and adjust its usage, alongside various UI layout and overflow fixes.

[Copilot]

Pull request overview

This PR refactors the columnsMap data structure from a Map<string, ColumnInfo> to a plain JavaScript object (Record<string, ColumnInfo>), and includes several UI layout fixes to address overflow issues.

Changes:

• Refactored columnsMap from Map to plain object in schema refresh logic
• Added overflow and sizing fixes to UI components for better layout control
• Reconstructed Map from plain object when needed for table data fetching

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
api/sql.ts	Changed columnsMap construction to use plain object instead of Map
api/routes.ts	Added logic to reconstruct Map from plain object when calling fetchTablesData
web/pages/DeploymentPage.tsx	Changed table width class from w-full to min-w-full
web/components/SideBar.tsx	Added overflow-hidden class to drawer-side container
web/components/BackgroundPattern.tsx	Added explicit sizing constraints to fixed background container

Comments suppressed due to low confidence (2)

api/sql.ts:92

• The TableInfo type definition still declares columnsMap as Map<string, ColumnInfo>, but the implementation now stores it as Record<string, ColumnInfo>. Update the type definition to match: columnsMap: Record<string, ColumnInfo>

  columnsMap: Map<string, ColumnInfo>

api/sql.ts:110

• The initialization still uses new Map() but should be changed to {} to match the new plain object implementation.

        tableMap.set(key, { schema, table, columns: [], columnsMap: new Map() })

---

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