sqlite-hub 0.17.0 → 1.0.0

package/tests/settings-view.test.js

@@ -29,4 +29,51 @@ test("settings view shows SQLite runtime version and custom port command", async
   assert.match(rendered.main, /SQLite_Runtime/);
   assert.match(rendered.main, /3\.53\.1/);
   assert.match(rendered.main, /sqlite-hub --port:PORT/);
+  assert.match(rendered.main, /data-section="information"/);
+  assert.match(rendered.main, /data-section="api-tokens"/);
+  assert.doesNotMatch(rendered.main, /data-form="create-api-token"/);
+});
+
+test("settings view scopes API token controls to the active database", async () => {
+  const { renderSettingsView } = await loadSettingsViewModule();
+  const rendered = renderSettingsView({
+    settings: {
+      loading: false,
+      error: null,
+      appVersion: "1.2.3",
+      sqliteVersion: "3.53.1",
+      section: "api-tokens",
+      tokenDatabase: { id: "db-one", label: "Database One" },
+      apiTokens: [
+        {
+          id: "token-one",
+          name: "Automation",
+          tokenPrefix: "shub_example",
+          createdAt: "2026-06-15T10:00:00.000Z",
+        },
+      ],
+      createdApiToken: null,
+      tokenSaving: false,
+    },
+  });
+
+  assert.match(rendered.main, /API Tokens/);
+  assert.match(rendered.main, /Security \/\/ Database API access/);
+  assert.match(rendered.main, /Database One/);
+  assert.match(rendered.main, /Database ID/);
+  assert.match(rendered.main, /data-database-id/);
+  assert.match(rendered.main, /data-action="copy-database-id"/);
+  assert.match(rendered.main, /Copy ID/);
+  assert.match(rendered.main, /sm:grid-cols-\[minmax\(0,1fr\)_9rem\]/);
+  assert.equal((rendered.main.match(/w-full self-center justify-center/g) ?? []).length, 2);
+  assert.match(rendered.main, /shub_example\.\.\./);
+  assert.match(rendered.main, /data-api-token-name/);
+  assert.match(rendered.main, /data-form="create-api-token"/);
+  assert.match(rendered.main, /name="name"/);
+  assert.match(rendered.main, /type="submit"/);
+  assert.match(rendered.main, /data-action="open-delete-api-token-modal"/);
+  assert.match(rendered.main, /bg-surface-container-lowest/);
+  assert.doesNotMatch(rendered.main, /sqlite-hub --port:PORT/);
+  assert.doesNotMatch(rendered.main, /Open Github/);
+  assert.doesNotMatch(rendered.main, /token_hash/);
 });

package/tests/sql-identifier-safety.test.js

@@ -114,7 +114,8 @@ test("data browser mutations preserve quoted dynamic identifiers", () => {
         "UPDATE",
         quoteIdentifier(tableName),
         "SET",
-        quoteIdentifier(valueColumn) + " = ?",
+        quoteIdentifier(valueColumn) + " = ?,",
+        quoteIdentifier(noteColumn) + " = ?",
         "WHERE",
         "rowid IS ?",
       ].join(" ")
@@ -125,6 +126,11 @@ test("data browser mutations preserve quoted dynamic identifiers", () => {
         oldValue: "before",
         newValue: "after",
       },
+      {
+        column: noteColumn,
+        oldValue: "NULL",
+        newValue: "",
+      },
     ]);
 
     const updated = service.updateTableRow(tableName, {
@@ -136,7 +142,7 @@ test("data browser mutations preserve quoted dynamic identifiers", () => {
     });
 
     assert.equal(updated.row[valueColumn], "after");
-    assert.equal(updated.row[noteColumn], null);
+    assert.equal(updated.row[noteColumn], "");
     const persistedRow = db.prepare(
       [
         "SELECT",
@@ -148,7 +154,7 @@ test("data browser mutations preserve quoted dynamic identifiers", () => {
     ).get();
 
     assert.equal(persistedRow[valueColumn], "after");
-    assert.equal(persistedRow[noteColumn], null);
+    assert.equal(persistedRow[noteColumn], "");
 
     const deleted = service.deleteTableRow(tableName, {
       identity: updated.row.__identity,

package/tests/sql-result-limit.test.js

@@ -0,0 +1,66 @@
+const Database = require("better-sqlite3");
+const assert = require("node:assert/strict");
+const test = require("node:test");
+const {
+  DEFAULT_RESULT_ROW_LIMIT,
+  SqlExecutor,
+} = require("../server/services/sqlite/sqlExecutor");
+
+function createExecutor(db) {
+  return new SqlExecutor({
+    connectionManager: {
+      getActiveDatabase: () => db,
+      getActiveConnection: () => ({ id: "result-limit-test" }),
+    },
+    appStateStore: {
+      recordQueryExecution: () => 1,
+    },
+  });
+}
+
+test("interactive SQL results stop at the configured row limit", () => {
+  const db = new Database(":memory:");
+
+  try {
+    db.exec(`
+      CREATE TABLE items (id INTEGER PRIMARY KEY, label TEXT);
+      INSERT INTO items (label) VALUES ('one'), ('two'), ('three'), ('four'), ('five');
+    `);
+
+    const result = createExecutor(db).execute("SELECT id, label FROM items ORDER BY id", {
+      maxRows: 3,
+    });
+
+    assert.equal(DEFAULT_RESULT_ROW_LIMIT, 5000);
+    assert.equal(result.rows.length, 3);
+    assert.deepEqual(result.rows.map((row) => row.id), [1, 2, 3]);
+    assert.equal(result.truncated, true);
+    assert.equal(result.rowLimit, 3);
+    assert.equal(result.statements[0].truncated, true);
+    assert.equal(result.statements[0].rowCount, 3);
+  } finally {
+    db.close();
+  }
+});
+
+test("unlimited SQL execution remains available for export services", () => {
+  const db = new Database(":memory:");
+
+  try {
+    db.exec(`
+      CREATE TABLE items (id INTEGER PRIMARY KEY);
+      INSERT INTO items VALUES (1), (2), (3), (4), (5);
+    `);
+
+    const result = createExecutor(db).execute("SELECT id FROM items ORDER BY id", {
+      maxRows: null,
+      persistHistory: false,
+    });
+
+    assert.equal(result.rows.length, 5);
+    assert.equal(result.truncated, false);
+    assert.equal(result.rowLimit, null);
+  } finally {
+    db.close();
+  }
+});

package/tests/table-scroll-state.test.js

@@ -0,0 +1,70 @@
+const assert = require("node:assert/strict");
+const { readFileSync } = require("node:fs");
+const path = require("node:path");
+const test = require("node:test");
+
+let tableScrollStateModulePromise = null;
+
+function loadTableScrollStateModule() {
+  if (!tableScrollStateModulePromise) {
+    const source = readFileSync(
+      path.resolve(__dirname, "../frontend/js/utils/tableScrollState.js"),
+      "utf8"
+    );
+    const url = `data:text/javascript;base64,${Buffer.from(source).toString("base64")}`;
+
+    tableScrollStateModulePromise = import(url);
+  }
+
+  return tableScrollStateModulePromise;
+}
+
+test("table scroll state restores horizontal position for the matching grid", async () => {
+  const { captureTableHorizontalScrollState, restoreTableHorizontalScrollState } =
+    await loadTableScrollStateModule();
+  const key = 'data:events "archive"';
+  const previousGrid = { dataset: { tableScrollKey: key }, scrollLeft: 640 };
+  const nextGrid = { dataset: { tableScrollKey: key }, scrollLeft: 0 };
+  const snapshot = captureTableHorizontalScrollState({
+    routeName: "data",
+    scrollNodes: [previousGrid],
+  });
+
+  assert.equal(
+    restoreTableHorizontalScrollState({
+      snapshot,
+      routeName: "data",
+      scrollNodes: [nextGrid],
+    }),
+    true
+  );
+  assert.equal(nextGrid.scrollLeft, 640);
+});
+
+test("table scroll state is not restored across routes or different grids", async () => {
+  const { captureTableHorizontalScrollState, restoreTableHorizontalScrollState } =
+    await loadTableScrollStateModule();
+  const snapshot = captureTableHorizontalScrollState({
+    routeName: "editorResults",
+    scrollNodes: [{ dataset: { tableScrollKey: "editor:12" }, scrollLeft: 320 }],
+  });
+  const nextGrid = { dataset: { tableScrollKey: "editor:13" }, scrollLeft: 0 };
+
+  assert.equal(
+    restoreTableHorizontalScrollState({
+      snapshot,
+      routeName: "data",
+      scrollNodes: [nextGrid],
+    }),
+    false
+  );
+  assert.equal(
+    restoreTableHorizontalScrollState({
+      snapshot,
+      routeName: "editorResults",
+      scrollNodes: [nextGrid],
+    }),
+    false
+  );
+  assert.equal(nextGrid.scrollLeft, 0);
+});