Rohithgilla12/data-peek

What The Format SQL menu item (Shift+Alt+F, apps/desktop/src/main/menu.ts:211) sends menu:format-sql to the renderer, and the preload exposes window.api.menu.onFormatSql, but nothing in apps/desktop/src/renderer/src subscribes to it. Choosing it from the menu, or pressing Shift+Alt+F, does nothing. The working shortcut is Monaco's own Cmd/Ctrl+Shift+F command in sql-editor.tsx. This came up in #283, which dropped Shift+Alt+F from the docs for this reason. These preload menu listeners also have no renderer subscriber, so their menu items may be no-ops when clicked. Their keyboard shortcuts may still work through renderer hotkeys, so please verify each one: onNewTab (menu:new-tab) onCloseTab (menu:close-tab) onExecuteQuery (menu:execute-query) onClearResults (menu:clear-results) How Subscribe to onFormatSql where the active SQL editor lives, and run the same format action as the Monaco Cmd/Ctrl+Shift+F command. Return the cleanup function from useEffect, like the existing onToggleWatc

What Follow-up to #278 / #284. Seeded data-generator runs are now reproducible for nulls, enums, and FK picks, but random-date still isn't. When a column has no minValue/maxValue, generateValue in apps/desktop/src/main/data-generator.ts builds its window from Date.now(): case 'random-date': { const from = new Date(col.minValue ?? Date.now() - 365 * 24 * 60 * 60 * 1000) const to = new Date(col.maxValue ?? Date.now()) return faker.date.between({ from, to }).toISOString() } The renderer heuristic (apps/desktop/src/renderer/src/lib/data-gen-heuristics.ts) emits a bare { generatorType: 'random-date' } for most date/timestamp columns, so this is the common path. Two runs with the same seed produce slightly different dates. How When config.seed is set, anchor the default window to a fixed reference instead of Date.now(). For example, capture now once at the start of generateRows, and use a fixed epoch when seeded (or faker's refDate). Unseeded runs should keep today's behaviour: the la

What batchInsert in apps/desktop/src/main/batch-insert.ts binds every cell as a parameter. The default batch size is 500 rows (apps/desktop/src/main/ipc/import-handlers.ts, apps/desktop/src/renderer/src/stores/data-gen-store.ts) and the CSV import dialog allows up to 5,000. SQL Server caps a statement at 2,100 parameters, so a 500-row import of a 5-column table fails. SQL Server also caps a VALUES list at 1,000 rows. Postgres caps parameters at 65,535 (5,000 rows × 14 columns). How Add a pure effectiveBatchSize(dialect, columnCount, requested) in batch-insert.ts that clamps to the dialect's parameter and row limits, and use it to split batches. Export buildInsertSql and buildPlaceholders so they can be tested directly. Add apps/desktop/src/main/__tests__/batch-insert.test.ts covering the clamp for each dialect and the placeholder numbering across split batches. Out of scope MSSQL onConflict: 'skip' | 'update' is currently a plain INSERT; that needs MERGE and its own issue. Done wh

What Schema Intel is in the sidebar (apps/desktop/src/renderer/src/components/app-sidebar.tsx) and now also powers npx data-peek doctor, but there is no feature page under apps/docs/content/docs/features/ and no README entry. How Write apps/docs/content/docs/features/schema-intel.mdx from SCHEMA_INTEL_CHECKS in packages/shared/src/index.ts: what each check finds, its severity, and a per-database support matrix (Postgres, MySQL, SQL Server, SQLite) from supportedDbTypes. Explain the Run Fix flow and that suggested SQL is a starting point, not a blind apply. Link to the CLI reference page at apps/docs/content/docs/reference/cli.mdx for the terminal version. Register the page in apps/docs/content/docs/features/meta.json and add a bullet to the Features section in README.md. Match the tone of features/health-monitor.mdx: what it is, when you reach for it, then the details. Done when pnpm --filter docs build passes and the page is in the sidebar. Difficulty: easy. Docs only.

What Three places where the docs and the code disagree. README.md says "Export results to CSV, JSON, or Excel". There is no Excel export. The header comment in apps/desktop/src/renderer/src/lib/export.ts and a stray 'xlsx' member in ExportOptions in the same file are leftovers. Fix the README and remove the dead type member. apps/docs/content/docs/features/health-monitor.mdx says Health Monitor is PostgreSQL-only. The MySQL and SQL Server adapters implement getActiveQueries, getTableSizes, getCacheStats, and getLocks. Verify each panel against a live MySQL and SQL Server connection, then update the support note to say exactly what works where. apps/docs/content/docs/database-support/index.mdx marks SQL Server custom types as unsupported, but getTypes in apps/desktop/src/main/adapters/mssql-adapter.ts returns user-defined types. Update the matrix. Done when All three statements match the code, with a line in the PR saying how each was verified. Difficulty: easy. Docs plus one dead-c