removed ai comments

Author: zknpr

.github/dependabot.yml

@@ -0,0 +1,10 @@
+version: 2
+updates:
+  - package-ecosystem: "npm"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+  - package-ecosystem: "npm"
+    directory: "/website"
+    schedule:
+      interval: "weekly"

core/ui/modules/clipboard.js

@@ -145,8 +145,6 @@ export async function clearSelectedCellValues() {
         state.lastSelectedCell = null;
         state.selectedColumns.clear();
 
-        // Full reload or just local update?
-        // Perform a full reload to ensure UI consistency with backend state.
         await loadTableData();
         updateToolbarButtons();
         updateStatus(`${label} - Ctrl+S to save`);

core/ui/modules/crud.js

@@ -116,8 +116,6 @@ async function submitDeleteRows() {
     if (state.selectedRowIds.size === 0) return;
 
     const rowIds = Array.from(state.selectedRowIds);
-    // Validation happens in HostBridge/backend now but good to be type-safe here?
-    // They are stored as numbers in state mostly.
 
     try {
         updateStatus('Deleting rows...');

core/ui/modules/dnd.js

@@ -44,9 +44,6 @@ function onDragOver(e) {
 }
 
 function onDragLeave(e) {
-    // Only remove if leaving the cell (not entering a child)
-    // But dragleave fires when entering a child too.
-    // Simpler to rely on dragover to manage the class, or clean up if leaving grid.
     if (e.target === lastHighlightedCell) {
         // This flickers. Rely on dragover.
     }
@@ -78,8 +75,7 @@ async function onDrop(e) {
         const uris = uriList.split(/\r?\n/);
         if (uris.length > 0 && uris[0]) {
             let uri = uris[0];
-            // Decode URI if needed, but VS Code usually provides encoded URIs
-            // We need a name. Try to extract from URI.
+            // Extract name from URI
             let name = 'unknown_file';
             try {
                 // Simple parsing for name
@@ -109,25 +105,16 @@ async function handleFileUpload(cell, fileName, fileBlob) {
 async function handleUriUpload(cell, fileName, uri) {
     try {
         updateStatus(`Fetching ${fileName}...`);
-        // Use backend to read file from workspace
-        // Response should be the buffer/array
         const result = await backendApi.readWorkspaceFileUri(uri);
 
-        // Result comes back as the data structure from RPC.
-        // HostBridge returns Uint8Array.
-        // PostMessage serialization handles Uint8Array correctly usually.
-        // If it comes as { type: 'Buffer', data: [...] } (Node Buffer serialization), we need to handle it.
-
         let uint8Array;
         if (result instanceof Uint8Array) {
             uint8Array = result;
         } else if (result && result.type === 'Buffer' && Array.isArray(result.data)) {
             uint8Array = new Uint8Array(result.data);
         } else if (result && typeof result === 'object' && Object.keys(result).some(k => !isNaN(k))) {
-             // Sometimes obj-like {0: x, 1: y...}
              uint8Array = new Uint8Array(Object.values(result));
         } else {
-             // Fallback or error
              console.error('Unknown data format from backend:', result);
              throw new Error('Received invalid data format from backend');
         }

core/ui/modules/edit.js

@@ -202,10 +202,6 @@ export async function openCellInVsCode() {
     const { rowIdx, colIdx, rowId, columnName, originalValue } = state.cellPreviewInfo;
     const column = state.tableColumns[colIdx];
 
-    // We need to determine if it's text, json, blob, etc.
-    // For now passing value as is.
-    // We pass metadata to help extension determine extension/language.
-
     // We get the webview id from dataset if available or assume 'default'
     const webviewId = document.getElementById('vscode-env')?.dataset.webviewId || 'default';
 

core/ui/modules/grid.js

@@ -290,11 +290,6 @@ export async function loadTableData(showSpinner = true, saveScrollPosition = tru
 
         const dataResult = await backendApi.fetchTableData(state.selectedTable, queryOptions);
 
-        // Data result rows now include rowid at index 0 if we requested it.
-        // `grid.js` logic: `getRowId` uses `row[0]`. `getCellValue` uses `colIdx + getRowDataOffset()`.
-        // `getRowDataOffset` returns 1 if table (skipping rowid).
-        // So if `dataResult.rows` has `[rowid, col1, col2]`, it matches the expectation!
-
         state.gridData = dataResult.rows || [];
 
         // If not showing spinner (background refresh), capture the current scroll position

core/ui/modules/sidebar.js

@@ -262,9 +262,6 @@ export async function applyBatchUpdate() {
         const freshSelectedCells = [];
         for (const oldCell of state.selectedCells) {
             // Find corresponding row in new gridData
-            // If pagination/sort changed, indices might be wrong, but we didn't change those.
-            // However, we re-fetched, so rows might have moved if we sorted by the column we updated?
-            // Assuming stable order for now.
             const newValue = state.gridData[oldCell.rowIdx][oldCell.colIdx + getRowDataOffset()];
             freshSelectedCells.push({ ...oldCell, value: newValue });
         }

core/ui/modules/utils.js

@@ -89,9 +89,7 @@ function formatDate(value, format) {
     if (value instanceof Date) {
         date = value;
     } else if (typeof value === 'number') {
-        // Assume unix timestamp (seconds if small, millis if large?)
-        // SQLite often uses seconds (REAL or INTEGER)
-        // If it's small (e.g. < 10^11), assume seconds.
+        // Assume unix timestamp
         if (value < 100000000000) {
             date = new Date(value * 1000);
         } else {

natives/native-worker.js

@@ -342,7 +342,7 @@ async function handleRequest(request) {
              result = runResult;
           } else {
               // tjs sqlite might not expose totalChanges/changes on db object
-              // We need to query for it if missing
+              // Query for value if missing
               if (db.changes !== undefined) {
                  result = {
                    changes: db.changes,

package.json

@@ -10,10 +10,20 @@
     "type": "git",
     "url": "https://github.com/zknpr/sqlite-explorer"
   },
-  "funding": {
-    "type": "individual",
-    "url": "https://buymeacoffee.com/zknpr"
-  },
+  "funding": [
+    {
+      "type": "github",
+      "url": "https://github.com/sponsors/zknpr"
+    },
+    {
+      "type": "ko-fi",
+      "url": "https://ko-fi.com/zknpr"
+    },
+    {
+      "type": "BuyMeACoffee",
+      "url": "https://buymeacoffee.com/zknpr"
+    }
+  ],
   "engines": {
     "vscode": "^1.83.1"
   },
@@ -161,4 +171,4 @@
     "@vscode/extension-telemetry": "^1.2.0",
     "sql.js": "^1.13.0"
   }
-}
+}