Release release-2025-02-13

.changeset/chilly-yaks-listen.md

@@ -0,0 +1,13 @@
+---
+'@evidence-dev/tailwind': patch
+---
+
+[!] BREAKING CHANGE:
+Custom theme colors are converted implicitly to kebab-case by tailwindcss
+
+This means that some color defined in `evidence.config.yaml` as `myCustomColor`
+should be used as (for example) `text-my-custom-color`. (`text-myCustomColor` will not work)
+
+When referencing custom theme colors in Javascript the original case should still be used
+(for example) `myCustomColor` would still be `<BarChart ... fillColor=myCustomColor />`
+_NOT_ `<BarChart ... fillColor="my-custom-color" />`

.changeset/four-mugs-taste.md

@@ -0,0 +1,11 @@
+---
+'@evidence-dev/source-javascript': patch
+'@evidence-dev/component-utilities': patch
+'@evidence-dev/preprocess': patch
+'@evidence-dev/sdk': patch
+'@evidence-dev/universal-sql': patch
+'@evidence-dev/core-components': patch
+'@evidence-dev/tailwind': patch
+---
+
+bump vitest

.changeset/hungry-donkeys-run.md

@@ -0,0 +1,6 @@
+---
+'@evidence-dev/sdk': patch
+'@evidence-dev/universal-sql': patch
+---
+
+Fix type errors

.changeset/kind-fishes-mate.md

@@ -0,0 +1,7 @@
+---
+'@evidence-dev/core-components': patch
+'@evidence-dev/tailwind': patch
+'@evidence-dev/evidence': patch
+---
+
+Use theme class instead of data-

.changeset/seven-books-press.md

@@ -0,0 +1,6 @@
+---
+'@evidence-dev/evidence': minor
+'@evidence-dev/core-components': minor
+---
+
+Improved settings page UI

.changeset/strong-wombats-admire.md

@@ -0,0 +1,5 @@
+---
+'@evidence-dev/evidence': patch
+---
+
+Add theme.css export for svelte component <style/> tags that need to access tailwind theme

.eslintrc.cjs

@@ -10,5 +10,15 @@ module.exports = {
 		browser: true,
 		es2017: true,
 		node: true
+	},
+	rules: {
+		'no-unused-vars': [
+			'error',
+			{
+				argsIgnorePattern: '^_',
+				varsIgnorePattern: '^_',
+				caughtErrorsIgnorePattern: '^_'
+			}
+		]
 	}
 };

.gitignore

@@ -39,5 +39,6 @@ packages/ui/core-components/coverage
 sites/docs/.evidence/meta/query-cache/hashes.json
 
 priority_support_users.csv
+.aider*
 
 evidence-*.tgz

e2e/base-path/tests/tests.spec.js

@@ -191,6 +191,24 @@ test.describe('Page', () => {
 	});
 });
 
+test.describe('Parquet Files', () => {
+	test('Manifest links should use base path', async ({ page }) => {
+		// Firefox does not behave as expected in playwright.
+		// This isn't a browser-specific test, so we can get away with
+		// not running on firefox
+		if (test.info().project.name === 'firefox') return;
+		page.addListener('request', (req) => {
+			console.log(req.url());
+		});
+		const reqPromise = page.waitForRequest(`${basePath}/data/needful_things/orders/orders.parquet`);
+		page.goto(basePath);
+		const req = await reqPromise;
+
+		const res = await req.response();
+		expect(res?.ok()).toBe(true);
+	});
+});
+
 test.describe('Components', () => {
 	test('Table row links should use base path', async ({ page }) => {
 		await page.goto(`${basePath}/table-row-links`);

e2e/dev-server-startup/package.json

@@ -20,7 +20,7 @@
     "@evidence-dev/duckdb": "workspace:*",
     "@evidence-dev/evidence": "workspace:*",
     "vite": "5.4.14",
-    "vitest": "^2.1.8"
+    "vitest": "^2.1.9"
   },
   "overrides": {
     "jsonwebtoken": "9.0.0",

e2e/hmr/tests/tests.spec.js

@@ -1,6 +1,6 @@
 // @ts-check
 import { test, expect } from '@playwright/test';
-import { createFile, deleteFile, editFile, restoreChangedFiles } from './fs-utils';
+import { editFile, restoreChangedFiles } from './fs-utils';
 import { waitForPageToLoad } from '../../test-utils';
 
 /** @param {import("@playwright/test").Page} page */
@@ -26,34 +26,6 @@ test.describe('pages', () => {
 
 		await expect(page.getByText('This page has some different text on it')).toBeVisible();
 	});
-
-	test('creating should add to the sidebar and allow navigation', async ({ page }) => {
-		await page.goto('/');
-		await waitForPageToLoad(page);
-
-		await expect(page.getByText('Index')).toBeVisible();
-
-		createFile('pages/new-page.md', 'This is a new page');
-		// file deletions trigger full reload, so we don't waitForHMR() here
-
-		await expect(page.getByRole('link', { name: 'New Page' })).toBeVisible();
-		await page.goto('/new-page');
-		await expect(page.getByText('This is a new page')).toBeVisible();
-	});
-
-	test('deleting should remove from the sidebar and prevent navigation', async ({ page }) => {
-		await page.goto('/');
-		await waitForPageToLoad(page);
-
-		await expect(page.getByText('Index')).toBeVisible();
-
-		deleteFile('pages/page.md');
-		await waitForHMR(page);
-
-		await expect(page.getByRole('link', { name: 'Page' })).not.toBeVisible();
-		await page.goto('/page');
-		await expect(page.getByText('Page Not Found')).toBeVisible();
-	});
 });
 
 test.describe('sources', () => {

e2e/themes/pages/index.md

@@ -6,7 +6,7 @@
 	div-primary-var
 </div>
 
-<div data-testid="div-myCustomColor-class" class="bg-myCustomColor">
+<div data-testid="div-myCustomColor-class" class="bg-my-custom-color">
 	div-myCustomColor-class
 </div>
 

package.json

@@ -26,6 +26,8 @@
 		"@evidence-dev/telemetry": "link:packages/lib/telemetry",
 		"@evidence-dev/trino": "link:packages/datasources/trino",
 		"@evidence-dev/universal-sql": "link:packages/lib/universal-sql",
+		"@tailwindcss/vite": "^4.0.0",
+		"tailwindcss": "^4.0.0",
 		"@parcel/packager-ts": "2.12.0",
 		"@parcel/transformer-typescript-types": "2.12.0",
 		"@playwright/test": "^1.45.3",

packages/datasources/javascript/package.json

@@ -12,7 +12,7 @@
   "license": "MIT",
   "devDependencies": {
     "typescript": "^5.3.3",
-    "vitest": "^2.0.5"
+    "vitest": "^2.1.9"
   },
   "dependencies": {
     "@evidence-dev/db-commons": "^1.0.5"

packages/evidence/package.json

@@ -21,6 +21,9 @@
 	"bin": {
 		"evidence": "cli.js"
 	},
+	"exports": {
+		"./theme.css": "./template/src/app.css"
+	},
 	"scripts": {
 		"build": "node scripts/build-template",
 		"prepublishOnly": "npm run build"
@@ -54,12 +57,10 @@
 		"autoprefixer": "^10.4.7",
 		"debounce": "^1.2.1",
 		"git-remote-origin-url": "4.0.0",
-		"postcss": "^8.4.14",
-		"postcss-load-config": "^4.0.1",
 		"svelte": "4.2.19",
 		"svelte-preprocess": "5.1.3",
 		"svelte2tsx": "0.7.4",
-		"tailwindcss": "^3.3.1",
+		"tailwindcss": "^4.0.0",
 		"typescript": "5.4.2",
 		"unist-util-visit": "4.1.2",
 		"vite": "5.4.14"

packages/evidence/scripts/build-template.js

@@ -21,9 +21,7 @@ const templatePaths = [
 	'src/pages/settings/',
 	'src/pages/explore',
 	'src/pages/api/',
-	'src/pages/manifest.webmanifest/+server.js',
-	'tailwind.config.cjs',
-	'postcss.config.cjs'
+	'src/pages/manifest.webmanifest/+server.js'
 ];
 const ignorePaths = ['static/data'];
 
@@ -54,6 +52,7 @@ fsExtra.outputFileSync(
 	import { isDebug } from '@evidence-dev/sdk/utils';
 	import { log } from "@evidence-dev/sdk/logger";
 	import { evidenceThemes } from '@evidence-dev/tailwind/vite-plugin';
+	import tailwindcss from '@tailwindcss/vite';
 
 
 	process.removeAllListeners('warning');
@@ -73,7 +72,7 @@ fsExtra.outputFileSync(
     /** @type {import('vite').UserConfig} */
      const config = 
     {
-        plugins: [sveltekit(), configVirtual(), queryDirectoryHmr, sourceQueryHmr(), evidenceThemes()],
+        plugins: [tailwindcss(), sveltekit(), configVirtual(), queryDirectoryHmr, sourceQueryHmr(), evidenceThemes()],
         optimizeDeps: {
             include: ['echarts-stat', 'echarts', 'blueimp-md5', 'nanoid', '@uwdata/mosaic-sql',
 				// We need these to prevent HMR from doing a full page reload

packages/evidence/scripts/svelte.config.js

@@ -1,5 +1,5 @@
 import evidencePreprocess from '@evidence-dev/preprocess';
-import preprocess from 'svelte-preprocess';
+
 import adapter from '@sveltejs/adapter-static';
 import { addBasePathToHrefAndSrc, injectComponents } from '@evidence-dev/sdk/build/svelte';
 import fs from 'fs';
@@ -37,14 +37,7 @@ function errorHandler(warning) {
 /** @type {import('@sveltejs/kit').Config} */
 const config = {
 	extensions: ['.svelte', '.md'],
-	preprocess: [
-		...evidencePreprocess(true),
-		injectComponents(),
-		preprocess({
-			postcss: true
-		}),
-		addBasePathToHrefAndSrc
-	],
+	preprocess: [...evidencePreprocess(true), injectComponents(), addBasePathToHrefAndSrc],
 	onwarn: errorHandler,
 	kit: {
 		adapter: adapter({

packages/lib/component-utilities/package.json

@@ -12,7 +12,7 @@
 	"devDependencies": {
 		"@evidence-dev/tailwind": "workspace:^",
 		"@faker-js/faker": "^8.0.2",
-		"vitest": "^2.0.5"
+		"vitest": "^2.1.9"
 	},
 	"scripts": {
 		"test": "vitest --run",
@@ -32,4 +32,4 @@
 		"ssf": "^0.11.2",
 		"svelte": "4.2.19"
 	}
-}
+}

packages/lib/component-utilities/src/stores.js

@@ -1,6 +1,6 @@
 import { dev } from '$app/environment';
 import { browser } from '$app/environment';
-import { writable } from 'svelte/store';
+import { get, writable } from 'svelte/store';
 
 export const pageHasQueries = writable(true);
 export const routeHash = writable('');
@@ -60,17 +60,6 @@ function createToastsObject() {
 /** @type {import('svelte/store').Readable<Toast[]> & { add: (toast: Toast, timeout: number) => void }} */
 export const toasts = createToastsObject();
 
-/**
- * @template T
- * @param {import('svelte/store').Readable<T>} store
- * @returns {T}
- */
-const getStoreVal = (store) => {
-	let v;
-	store.subscribe((x) => (v = x))();
-	return v;
-};
-
 /** @template T @typedef {{ serialize: (value: T) => string; deserialize: (raw: string) => T }} SerializeAndDeserialize */
 
 /**
@@ -99,7 +88,7 @@ export const localStorageStore = (key, init, serializeAndDeserialize) => {
 		}
 	};
 
-	flush(getStoreVal(store));
+	flush(get(store));
 
 	/** @type {Writable<T>} */
 	return {
@@ -109,7 +98,7 @@ export const localStorageStore = (key, init, serializeAndDeserialize) => {
 			flush(v);
 		},
 		update: (cb) => {
-			const updatedStore = cb(getStoreVal(store));
+			const updatedStore = cb(get(store));
 			set(updatedStore);
 			flush(updatedStore);
 		}

packages/lib/preprocess/package.json

@@ -39,7 +39,7 @@
 		"mock-fs": "^5.2.0",
 		"parcel": "^2.8.3",
 		"typescript": "5.4.2",
-		"vitest": "^2.0.5"
+		"vitest": "^2.1.9"
 	},
 	"peerDependencies": {
 		"@types/hast": "^2.0.0",
@@ -68,4 +68,4 @@
 	"@parcel/transformer-js": {
 		"inlineFS": true
 	}
-}
+}

packages/lib/sdk/package.json

@@ -110,7 +110,7 @@
 		"@types/node": "^20.11.0",
 		"@typescript-eslint/typescript-estree": "^6.18.1",
 		"@uwdata/mosaic-sql": "^0.10.0",
-		"@vitest/coverage-v8": "^1.2.0",
+		"@vitest/coverage-v8": "3.0.5",
 		"chalk": "^5.3.0",
 		"chokidar": "^3.5.3",
 		"deep-object-diff": "^1.1.9",
@@ -134,7 +134,7 @@
 		"svelte-sequential-preprocessor": "^2.0.1",
 		"sveltekit-autoimport": "^1.7.1",
 		"vite": "5.4.14",
-		"vitest": "^1.1.3",
+		"vitest": "^2.1.9",
 		"yaml": "^2.3.4",
 		"zod": "^3.23.7"
 	},
@@ -161,4 +161,4 @@
 			"optional": true
 		}
 	}
-}
+}

packages/lib/sdk/src/build-dev/vite/virtuals/queries.svelte.js

@@ -1,3 +1,5 @@
+// @ts-nocheck Adding nocheck because type errors in this file were failing the build and @ItsMeBrianD told said this code is dead
+
 import { getContext } from 'svelte';
 import { ALL_QUERIES_CONTEXT_KEY, QUERIES_CONTEXT_KEY } from '$evidence/contextKeys';
 import { Query } from '@evidence-dev/sdk/usql';