learningfoundry 0.28.0__tar.gz → 0.30.0__tar.gz

{learningfoundry-0.28.0 → learningfoundry-0.30.0}/CHANGELOG.md

@@ -7,6 +7,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.30.0] - 2026-04-27
+
+### Fixed
+
+- **`Cannot call fetch eagerly during server-side rendering with relative URL (/curriculum.json)`** — SvelteKit's prerender pass was subscribing to the `curriculum` readable store during SSR of `+layout.svelte`, triggering a relative-URL fetch on the server. The template is a pure CSR SPA (runtime curriculum fetch, IndexedDB, sql.js/WASM) and was never intended to render on the server.
+  - `src/learningfoundry/sveltekit_template/src/routes/+layout.ts` — new file exporting `ssr = false` and `prerender = false`. With `adapter-static` + `fallback: 'index.html'` already configured in `svelte.config.js`, the SPA fallback handles every route client-side; prerendering is not needed and was failing on dynamic `[module]/[lesson]` routes anyway.
+
+## [0.29.0] - 2026-04-27
+
+### Fixed
+
+- **Lesson content never rendered after clicking "Start module" / Next / Previous.** `navigateTo`, `navigateNext`, and `navigatePrev` in the SvelteKit template only updated the `currentPosition` Svelte store; they never changed the URL. Because lesson content is mounted by the dynamic route `/[module]/[lesson]/+page.svelte`, the route was never visited and `LessonView` (which renders the inlined markdown) never mounted — the left nav title updated, but the content area stayed on the home dashboard.
+  - `src/learningfoundry/sveltekit_template/src/lib/stores/curriculum.ts` — `navigateTo` now also calls `goto('/${moduleId}/${lessonId}')`; `navigateNext` and `navigatePrev` refactored to compute the target and delegate to `navigateTo` so URL navigation happens for them too.
+
+### Added
+
+- **Frontend unit-test infrastructure** for the SvelteKit template (the navigation regression went uncaught because the template had no test suite):
+  - `sveltekit_template/package.json` — added `jsdom` to devDependencies (`vitest` was already present)
+  - `sveltekit_template/vite.config.ts` — added vitest config block (`environment: 'jsdom'`, `include: src/**/*.{test,spec}.{js,ts}`)
+  - `sveltekit_template/src/lib/stores/curriculum.test.ts` — 9 cases covering `navigateTo` (URL + store update), `navigateNext` (within module / across modules / final-lesson no-op / null position), `navigatePrev` (within module / across modules / first-lesson no-op / null position); mocks `$app/navigation`'s `goto` via `vi.mock` and stubs global `fetch` to seed the curriculum readable
+- `tests/test_smoke_sveltekit.py::test_pnpm_test_passes` — runs `pnpm test` (vitest) inside the installed template so the smoke run catches future frontend regressions
+
+### Verified
+
+- `pyve test -m smoke` — 7/7 passed (Python build + vitest)
+- Full Python suite — 195/195 passed; ruff and mypy clean
+
 ## [0.28.0] - 2026-04-26
 
 ### Fixed

{learningfoundry-0.28.0 → learningfoundry-0.30.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: learningfoundry
-Version: 0.28.0
+Version: 0.30.0
 Summary: A curriculum engine that turns a YAML curriculum definition into a deployable SvelteKit learning application.
 Project-URL: Homepage, https://github.com/pointmatic/learningfoundry
 Project-URL: Repository, https://github.com/pointmatic/learningfoundry

{learningfoundry-0.28.0 → learningfoundry-0.30.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "learningfoundry"
-version = "0.28.0"
+version = "0.30.0"
 description = "A curriculum engine that turns a YAML curriculum definition into a deployable SvelteKit learning application."
 readme = "README.md"
 license = "Apache-2.0"

{learningfoundry-0.28.0 → learningfoundry-0.30.0}/src/learningfoundry/__init__.py

@@ -1,4 +1,4 @@
 # Copyright 2026 Pointmatic
 # SPDX-License-Identifier: Apache-2.0
 
-__version__ = "0.28.0"
+__version__ = "0.30.0"

{learningfoundry-0.28.0 → learningfoundry-0.30.0}/src/learningfoundry/sveltekit_template/package.json

@@ -23,6 +23,7 @@
     "@sveltejs/vite-plugin-svelte": "^7.0.0",
     "@tailwindcss/vite": "^4.0.0",
     "@types/sql.js": "^1.4.11",
+    "jsdom": "^25.0.0",
     "prettier": "^3.0.0",
     "prettier-plugin-svelte": "^3.0.0",
     "svelte-check": "^4.0.0",

learningfoundry-0.30.0/src/learningfoundry/sveltekit_template/src/lib/stores/curriculum.test.ts

@@ -0,0 +1,164 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+import { beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
+import { get } from 'svelte/store';
+import type { Curriculum } from '$lib/types/index.js';
+
+// Mock $app/navigation BEFORE importing the module under test so its
+// top-level `import { goto }` binds to the mock.
+const gotoMock = vi.fn();
+vi.mock('$app/navigation', () => ({
+	goto: gotoMock
+}));
+
+// Two-module fixture covering: same-module advance, cross-module advance,
+// final-lesson edge case, and reverse traversal across module boundaries.
+const FIXTURE: Curriculum = {
+	version: 1,
+	title: 'Test Curriculum',
+	description: 'For navigation tests',
+	modules: [
+		{
+			id: 'mod-01',
+			title: 'Module One',
+			description: '',
+			pre_assessment: null,
+			post_assessment: null,
+			lessons: [
+				{ id: 'lesson-01', title: 'L1', content_blocks: [] },
+				{ id: 'lesson-02', title: 'L2', content_blocks: [] }
+			]
+		},
+		{
+			id: 'mod-02',
+			title: 'Module Two',
+			description: '',
+			pre_assessment: null,
+			post_assessment: null,
+			lessons: [
+				{ id: 'lesson-01', title: 'M2L1', content_blocks: [] },
+				{ id: 'lesson-02', title: 'M2L2', content_blocks: [] }
+			]
+		}
+	]
+} as unknown as Curriculum;
+
+// Stub global fetch so the `curriculum` readable store loads our fixture
+// instead of doing a real network call (jsdom has no /curriculum.json).
+vi.stubGlobal(
+	'fetch',
+	vi.fn().mockResolvedValue({
+		ok: true,
+		status: 200,
+		statusText: 'OK',
+		json: async () => FIXTURE
+	})
+);
+
+// Import AFTER vi.mock and vi.stubGlobal so they're applied to the module
+// under test on first load.
+const {
+	curriculum,
+	currentPosition,
+	navigateTo,
+	navigateNext,
+	navigatePrev
+} = await import('./curriculum.js');
+
+// Wait for the readable store's async loader to resolve and emit the
+// fixture. Once `curriculum` has data, the derived `modules` store that
+// `navigateNext`/`navigatePrev` consume internally will have data too.
+beforeAll(async () => {
+	await new Promise<void>((resolve) => {
+		const unsub = curriculum.subscribe((c) => {
+			if (c) {
+				unsub();
+				resolve();
+			}
+		});
+	});
+});
+
+beforeEach(() => {
+	gotoMock.mockClear();
+	currentPosition.set(null);
+});
+
+describe('navigateTo', () => {
+	it('updates currentPosition and calls goto with /{moduleId}/{lessonId}', () => {
+		navigateTo('mod-01', 'lesson-01');
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-01', lessonId: 'lesson-01' });
+		expect(gotoMock).toHaveBeenCalledTimes(1);
+		expect(gotoMock).toHaveBeenCalledWith('/mod-01/lesson-01');
+	});
+});
+
+describe('navigateNext', () => {
+	it('advances to the next lesson within the same module', () => {
+		navigateTo('mod-01', 'lesson-01');
+		gotoMock.mockClear();
+
+		navigateNext();
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-01', lessonId: 'lesson-02' });
+		expect(gotoMock).toHaveBeenCalledWith('/mod-01/lesson-02');
+	});
+
+	it('crosses module boundaries to the first lesson of the next module', () => {
+		navigateTo('mod-01', 'lesson-02');
+		gotoMock.mockClear();
+
+		navigateNext();
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-02', lessonId: 'lesson-01' });
+		expect(gotoMock).toHaveBeenCalledWith('/mod-02/lesson-01');
+	});
+
+	it('is a no-op past the final lesson', () => {
+		navigateTo('mod-02', 'lesson-02');
+		gotoMock.mockClear();
+
+		navigateNext();
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-02', lessonId: 'lesson-02' });
+		expect(gotoMock).not.toHaveBeenCalled();
+	});
+
+	it('is a no-op when currentPosition is null', () => {
+		navigateNext();
+		expect(get(currentPosition)).toBeNull();
+		expect(gotoMock).not.toHaveBeenCalled();
+	});
+});
+
+describe('navigatePrev', () => {
+	it('moves to the previous lesson within the same module', () => {
+		navigateTo('mod-01', 'lesson-02');
+		gotoMock.mockClear();
+
+		navigatePrev();
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-01', lessonId: 'lesson-01' });
+		expect(gotoMock).toHaveBeenCalledWith('/mod-01/lesson-01');
+	});
+
+	it('crosses module boundaries backward', () => {
+		navigateTo('mod-02', 'lesson-01');
+		gotoMock.mockClear();
+
+		navigatePrev();
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-01', lessonId: 'lesson-02' });
+		expect(gotoMock).toHaveBeenCalledWith('/mod-01/lesson-02');
+	});
+
+	it('is a no-op before the first lesson', () => {
+		navigateTo('mod-01', 'lesson-01');
+		gotoMock.mockClear();
+
+		navigatePrev();
+		expect(get(currentPosition)).toEqual({ moduleId: 'mod-01', lessonId: 'lesson-01' });
+		expect(gotoMock).not.toHaveBeenCalled();
+	});
+
+	it('is a no-op when currentPosition is null', () => {
+		navigatePrev();
+		expect(get(currentPosition)).toBeNull();
+		expect(gotoMock).not.toHaveBeenCalled();
+	});
+});

{learningfoundry-0.28.0 → learningfoundry-0.30.0/src/learningfoundry}/sveltekit_template/src/lib/stores/curriculum.ts

@@ -1,6 +1,7 @@
 // Copyright 2026 Pointmatic
 // SPDX-License-Identifier: Apache-2.0
 import { derived, readable, writable } from 'svelte/store';
+import { goto } from '$app/navigation';
 import type { Curriculum, Lesson, Module } from '$lib/types/index.js';
 
 // ---------------------------------------------------------------------------
@@ -87,36 +88,37 @@ export const nextLesson = derived(
 
 export function navigateTo(moduleId: string, lessonId: string): void {
 	currentPosition.set({ moduleId, lessonId });
+	void goto(`/${moduleId}/${lessonId}`);
 }
 
 export function navigateNext(): void {
-	currentPosition.update(($pos) => {
-		if (!$pos) return $pos;
-		let found = false;
-		for (const mod of get(modules) as Module[]) {
-			for (const lesson of mod.lessons as Lesson[]) {
-				if (found) return { moduleId: mod.id, lessonId: lesson.id };
-				if (mod.id === $pos.moduleId && lesson.id === $pos.lessonId) found = true;
+	const pos = get(currentPosition);
+	if (!pos) return;
+	let found = false;
+	for (const mod of get(modules) as Module[]) {
+		for (const lesson of mod.lessons as Lesson[]) {
+			if (found) {
+				navigateTo(mod.id, lesson.id);
+				return;
 			}
+			if (mod.id === pos.moduleId && lesson.id === pos.lessonId) found = true;
 		}
-		return $pos;
-	});
+	}
 }
 
 export function navigatePrev(): void {
-	currentPosition.update(($pos) => {
-		if (!$pos) return $pos;
-		let prev: NavPosition | null = null;
-		for (const mod of get(modules) as Module[]) {
-			for (const lesson of mod.lessons as Lesson[]) {
-				if (mod.id === $pos.moduleId && lesson.id === $pos.lessonId) {
-					return prev ?? $pos;
-				}
-				prev = { moduleId: mod.id, lessonId: lesson.id };
+	const pos = get(currentPosition);
+	if (!pos) return;
+	let prev: NavPosition | null = null;
+	for (const mod of get(modules) as Module[]) {
+		for (const lesson of mod.lessons as Lesson[]) {
+			if (mod.id === pos.moduleId && lesson.id === pos.lessonId) {
+				if (prev) navigateTo(prev.moduleId, prev.lessonId);
+				return;
 			}
+			prev = { moduleId: mod.id, lessonId: lesson.id };
 		}
-		return $pos;
-	});
+	}
 }
 
 // get() helper for non-reactive reads inside update callbacks

learningfoundry-0.30.0/src/learningfoundry/sveltekit_template/src/routes/+layout.ts

@@ -0,0 +1,10 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// This app is a client-side SPA: it loads `curriculum.json` at runtime,
+// persists progress in IndexedDB, and uses sql.js (WASM) — none of which
+// can run during server-side rendering. Disable SSR so the prerender
+// pass does not subscribe to the curriculum store and trigger a relative
+// `fetch('/curriculum.json')` on the server.
+export const ssr = false;
+export const prerender = false;

{learningfoundry-0.28.0 → learningfoundry-0.30.0}/src/learningfoundry/sveltekit_template/vite.config.ts

@@ -1,9 +1,15 @@
 // Copyright 2026 Pointmatic
 // SPDX-License-Identifier: Apache-2.0
+/// <reference types="vitest" />
 import { sveltekit } from '@sveltejs/kit/vite';
 import tailwindcss from '@tailwindcss/vite';
 import { defineConfig } from 'vite';
 
 export default defineConfig({
-	plugins: [tailwindcss(), sveltekit()]
+	plugins: [tailwindcss(), sveltekit()],
+	test: {
+		environment: 'jsdom',
+		include: ['src/**/*.{test,spec}.{js,ts}'],
+		globals: false
+	}
 });