@nuxodin/item 0.6.16 → 0.6.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nuxodin/item",
-  "version": "0.6.16",
+  "version": "0.6.18",
   "description": "Reactive state library",
   "type": "module",
   "license": "MIT",

package/tools/schema/db/mysql/from-db.js

@@ -5,7 +5,14 @@ import { schemaFromField } from './from-field.js'
 
 const dialect = {
     tables:    async (query) => (await queryRows(query, 'SHOW TABLES')).map(r => Object.values(r)[0]),
-    fields:    async (query, table) => queryRows(query, `SHOW FULL FIELDS FROM ${quoteId(table)}`),
+    fields:    async (query, table) => {
+        const rows = await queryRows(query, `SHOW FULL FIELDS FROM ${quoteId(table)}`)
+        // MariaDB's JSON is LONGTEXT with a json_valid() check: read back as text it would be altered on every run
+        const checks = await queryRows(query, `SELECT CHECK_CLAUSE FROM information_schema.CHECK_CONSTRAINTS
+            WHERE CONSTRAINT_SCHEMA = DATABASE() AND TABLE_NAME = '${String(table).replaceAll("'", "''")}'`).catch(() => [])
+        const json = new Set(checks.map(c => /^json_valid\(`(.+)`\)$/.exec(String(c.CHECK_CLAUSE))?.[1]))
+        return rows.map(r => json.has(r.Field) ? { ...r, Type: 'json' } : r)
+    },
     fromField: (row) => ({
         name:       row.Field,
         prop:       schemaFromField(row),

package/tools/schema/db/sqlite/to-db.js

@@ -102,6 +102,9 @@ export async function schemaToDb(schema, query, { force = false, patch = false }
         const primaries = fields.filter(([, f]) => f['x-index'] === 'primary').map(([n]) => n)
 
         if (!currTables.includes(table)) {
+            // Idea: WITHOUT ROWID when the primary key is not a single integer column and all other columns
+            // are narrow (numbers, booleans, short strings), e.g. measurement tables: about half the size.
+            // Requires NOT NULL key columns; only applies to newly created tables.
             stmts.push(`CREATE TABLE ${quoteId(table)} (\n${createBody(table, fields, primaries)}\n);`)
             stmts.push(...indexStatements(table, fields, primaries))
         } else {