@vit-foundation/ui 0.24.0 → 0.24.2

package/dist/components/ui/Logo.svelte

@@ -9,16 +9,32 @@
 	let { size = 32, withWordmark = false }: Props = $props();
 </script>
 
+<!--
+	The mark is ALWAYS decorative. It carries no accessible name of its own, in
+	either branch, because at every call site something else already supplies
+	one — and when it tried, it doubled.
+
+	`withWordmark` used to flip three accessibility facts together:
+	`aria-hidden`, `role="img"` and the presence of `<title>`. With the wordmark
+	on, the mark was named "Brain VIT" AND the wordmark rendered the same string
+	beside it, so a screen reader announced the name twice on all three of
+	Brain's sign-in pages. Inverting the branch does not help: with the wordmark
+	off, the rail already names it through `<a aria-label="Home">` and the print
+	sheet sits it beside an `<h1>`, so naming the mark there doubles at the other
+	end.
+
+	Which leaves one honest rule — never name it — and the naming stays where
+	the context is: the visible wordmark below, the anchor around it, or the
+	heading beside it. `Logo.svelte.test.ts` pins both branches.
+-->
 <span class="logo">
 	<svg
 		viewBox="0 0 48 56"
 		width={size * (48 / 56)}
 		height={size}
-		aria-hidden={withWordmark ? undefined : true}
-		role={withWordmark ? 'img' : undefined}
+		aria-hidden="true"
 		focusable="false"
 	>
-		{#if withWordmark}<title>Brain VIT</title>{/if}
 		<path
 			d="M25 2c11.6 0 21 9.4 21 21 0 6.9-3.3 13-8.5 16.8V54H17v-9h-4.5A4.5 4.5 0 0 1 8 40.5V34H4.2c-1.7 0-2.7-1.9-1.7-3.3L8 22.4C9 11 16.2 2 25 2Z"
 			fill="currentColor"

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vit-foundation/ui",
-	"version": "0.24.0",
+	"version": "0.24.2",
 	"scripts": {
 		"dev": "vite dev",
 		"build": "vite build && npm run prepack",
@@ -25,7 +25,8 @@
 		"!dist/**/*.spec.*",
 		"!dist/fixtures.*",
 		"!dist/**/stories/**",
-		"!dist/**/tests/**"
+		"!dist/**/tests/**",
+		"!dist/testing/imports.*"
 	],
 	"sideEffects": [
 		"**/*.css"

package/dist/testing/imports.d.ts

@@ -1,51 +0,0 @@
-/**
- * What a source scan needs to see, and the anchors that keep it honest.
- *
- * Two guards in this package walk the source: `contract.test.ts` follows the
- * relative import graph beneath `./contract`, and `no-app-imports.test.ts`
- * scans every module for an app virtual specifier. They ask different
- * questions of the same rule — WHICH MODULES DOES THIS ONE NAME — and each
- * used to answer it with a pattern of its own. `contract.test.ts` found the
- * defect in its copy and fixed it there:
- *
- *     "THREE import forms, because two of them used to be invisible here. The
- *     pattern required the `from` keyword, so a side-effect import and a
- *     dynamic one both walked past a guard whose whole job is to see them."
- *
- * The other copy was not fixed and still required `from`, so a dynamic import
- * of an $app specifier walked past the gate that decides whether this package
- * is publishable. Copying the lesson a third time is how a guard
- * starts missing things again; the grammar lives here instead, and each guard
- * filters the specifiers it cares about.
- */
-/**
- * Every module specifier a source file names, in all three forms a bundler
- * follows: `from '…'` (import and re-export alike), a bare side-effect
- * `import '…'`, and a dynamic `import('…')`.
- *
- * A type-only import is reported like any other. That is deliberate, and it
- * is `contract.test.ts`'s standing argument: erasure is a build-time fact, and
- * a pattern that has to understand `import type` is a pattern with a second
- * way to be wrong.
- */
-export declare function importSpecifiers(source: string): readonly string[];
-/**
- * Every `.ts` and `.svelte` file beneath `root`, and the ones the walk MUST
- * find.
- *
- * `mustReach` is a required argument for the reason `contract.test.ts` gives
- * at length: an assertion over an empty list passes. `expect(offenders)
- * .toEqual([])` is a clean bill of health over a walk that enumerated nothing
- * — a directory renamed, a `readdirSync` that threw and was caught upstream,
- * an entry filter that stopped matching — and a safety net that can narrow in
- * silence reproduces the defect it exists to prevent. fndvit-website's
- * `src/lib/testing/source-scan.ts` settled the shape: a scan cannot walk the
- * tree without saying what it must find.
- *
- * Anchors are relative to `root` with `/` separators, so one representation
- * crosses this interface and an anchor cannot silently fail to match an
- * absolute path. A FLOOR was the alternative and is the thing being replaced:
- * a walk that quietly stopped seeing two thirds of the tree clears any floor
- * worth setting.
- */
-export declare function sourceFiles(root: string, mustReach: readonly string[]): readonly string[];

package/dist/testing/imports.js

@@ -1,70 +0,0 @@
-import { readdirSync } from 'node:fs';
-import { join, relative } from 'node:path';
-/**
- * What a source scan needs to see, and the anchors that keep it honest.
- *
- * Two guards in this package walk the source: `contract.test.ts` follows the
- * relative import graph beneath `./contract`, and `no-app-imports.test.ts`
- * scans every module for an app virtual specifier. They ask different
- * questions of the same rule — WHICH MODULES DOES THIS ONE NAME — and each
- * used to answer it with a pattern of its own. `contract.test.ts` found the
- * defect in its copy and fixed it there:
- *
- *     "THREE import forms, because two of them used to be invisible here. The
- *     pattern required the `from` keyword, so a side-effect import and a
- *     dynamic one both walked past a guard whose whole job is to see them."
- *
- * The other copy was not fixed and still required `from`, so a dynamic import
- * of an $app specifier walked past the gate that decides whether this package
- * is publishable. Copying the lesson a third time is how a guard
- * starts missing things again; the grammar lives here instead, and each guard
- * filters the specifiers it cares about.
- */
-/**
- * Every module specifier a source file names, in all three forms a bundler
- * follows: `from '…'` (import and re-export alike), a bare side-effect
- * `import '…'`, and a dynamic `import('…')`.
- *
- * A type-only import is reported like any other. That is deliberate, and it
- * is `contract.test.ts`'s standing argument: erasure is a build-time fact, and
- * a pattern that has to understand `import type` is a pattern with a second
- * way to be wrong.
- */
-export function importSpecifiers(source) {
-    return [...source.matchAll(/(?:\bfrom|\bimport)\s*\(?\s*['"]([^'"]*)['"]/g)].map(([, specifier]) => specifier);
-}
-/**
- * Every `.ts` and `.svelte` file beneath `root`, and the ones the walk MUST
- * find.
- *
- * `mustReach` is a required argument for the reason `contract.test.ts` gives
- * at length: an assertion over an empty list passes. `expect(offenders)
- * .toEqual([])` is a clean bill of health over a walk that enumerated nothing
- * — a directory renamed, a `readdirSync` that threw and was caught upstream,
- * an entry filter that stopped matching — and a safety net that can narrow in
- * silence reproduces the defect it exists to prevent. fndvit-website's
- * `src/lib/testing/source-scan.ts` settled the shape: a scan cannot walk the
- * tree without saying what it must find.
- *
- * Anchors are relative to `root` with `/` separators, so one representation
- * crosses this interface and an anchor cannot silently fail to match an
- * absolute path. A FLOOR was the alternative and is the thing being replaced:
- * a walk that quietly stopped seeing two thirds of the tree clears any floor
- * worth setting.
- */
-export function sourceFiles(root, mustReach) {
-    const walk = (dir) => readdirSync(dir, { withFileTypes: true }).flatMap((entry) => {
-        const path = join(dir, entry.name);
-        if (entry.isDirectory())
-            return walk(path);
-        return /\.(ts|svelte)$/.test(entry.name) ? [path] : [];
-    });
-    const files = walk(root);
-    const found = new Set(files.map((file) => relative(root, file).replaceAll('\\', '/')));
-    const missing = mustReach.filter((anchor) => !found.has(anchor));
-    if (missing.length > 0) {
-        throw new Error(`the source scan enumerated ${files.length} files but not ${missing.join(', ')} — ` +
-            'either the module moved or the walk stopped seeing it');
-    }
-    return files;
-}