learningfoundry 0.45.0__tar.gz → 0.49.0__tar.gz

{learningfoundry-0.45.0 → learningfoundry-0.49.0}/CHANGELOG.md

@@ -7,6 +7,41 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.49.0] - 2026-05-01
+
+### Changed
+
+- **Finish on the last lesson now clears the active-lesson highlight and collapses the previously expanded sidebar module.** When `Navigation.goNext()` finds no next lesson it now sets `currentPosition` to `null` *before* `goto('/')` so the sidebar's auto-expand effect sees the null transition; `computeAutoExpand` was extended to emit a reset (`{expandedModuleId: null, lastAutoExpandedModuleId: null}`) when position clears after a prior auto-expand. Result: landing on the dashboard after Finish shows no module expanded and no lesson row carrying the active highlight, instead of leaving the previously focused lesson visually marked as the learner's current location. The I.f manual-toggle preservation behavior is unchanged (the reset only fires on the position-cleared transition).
+
+## [0.48.0] - 2026-05-01
+
+### Changed
+
+- **TextBlock completion now requires the bottom of the block to be in view, not just any portion.** `TextBlock.svelte` renders a zero-size `<div data-textblock-end aria-hidden="true">` sentinel at the end of the rendered markdown and observes that element rather than the wrapper. A tall lesson can no longer be marked complete simply because the top of the text was on screen on initial render — the learner must scroll until the sentinel is in the viewport for the full 1-second debounce window. The `IntersectionObserver` debounce, threshold (`0.1`), and single-fire `fired` guard are unchanged. New vitest cases cover the "tall block, sentinel never intersects" and "scrolled into view → fires 1 s later" branches.
+
+## [0.47.0] - 2026-05-01
+
+### Added
+
+- **Reset course button.** New `ResetCourseButton.svelte` pinned at the bottom of the sidebar (`mt-auto`). Disabled until any progress exists in the curriculum (any `lesson_progress` row whose status is not `not_started`); reactive activation via the existing `progressStore`. Clicking opens a `window.confirm` dialog; on accept it calls the new `resetProgress()` DB op (single-transaction `DELETE FROM lesson_progress; quiz_scores; exercise_status`), clears `currentPosition`, refreshes the progress store, and routes to `/`. Pure helpers `hasAnyProgress` (in `$lib/utils/progress.ts`) and the new `resetProgress` DB op are independently unit-tested.
+
+## [0.46.0] - 2026-05-01
+
+### Fixed
+
+- **Lesson navigation routing.** Sidebar lesson clicks, the Next/Finish button, and dashboard "Start module / Continue" buttons now call `goto()` from `$app/navigation` directly instead of going through the curriculum store helper. The previous flow updated `currentPosition` (and therefore the sidebar highlight) but left the URL untouched, so the lesson route was never re-mounted: `markLessonInProgress` ran only on the first lesson reached by direct URL, the sidebar checkmarks never updated, and the curriculum/module progress bars stayed at zero across the session. Components now route via `Navigation.svelte`, `LessonList.svelte`, `ProgressDashboard.svelte` → `goto('/${moduleId}/${lessonId}')`.
+- **Sticky LessonView state across navigations.** The dynamic lesson route now wraps `<LessonView>` in `{#key \`${moduleId}/${lessonId}\`}` so the subtree tears down and re-mounts whenever either route param changes — guaranteeing fresh `allBlocksComplete` / `completedBlocks` state and a re-run of the on-mount progress check (so revisiting a previously-completed lesson activates Next/Finish immediately).
+- **Stale video iframe across consecutive video lessons.** `LessonView`'s `{#each lesson.content_blocks}` now uses a stable identity key derived from `block.ref` or `block.content.url` (falling back to `${type}-${index}`); previously, two consecutive lessons each with a `video` block reused the same `<VideoBlock>` instance and its iframe player, leaving the previous lesson's video on screen. `VideoBlock.svelte` additionally tracks `content.url` via `$effect` and tears down / recreates its YouTube player whenever the URL changes, as belt-and-suspenders coverage.
+
+### Added
+
+- **Playwright e2e harness.** New `e2e/` directory with three regression specs (`navigation.spec.ts`, `progress.spec.ts`, `video.spec.ts`) covering the FR-P9/FR-P10 lifecycle invariants that vitest cannot exercise (because vitest mocks `$app/navigation`). New `pnpm e2e` script and `playwright.config.ts` driving `pnpm preview` against the built static site. The smoke test runs `pnpm e2e` after `pnpm build` and skips gracefully if Playwright browsers aren't installed locally; CI installs them via `pnpm exec playwright install chromium`.
+- **Vitest navigation regression coverage.** New `navigation.helpers.ts` (`resolveGoNext`, `resolveGoPrev`, `lessonHref`) and `navigation.test.ts` lock down the routing decisions made by Next/Finish, Previous, and lesson rows. New `contentBlockKey` helper (and tests) verifies the FR-P10 stable-identity convention used by `{#each}`.
+
+### Changed
+
+- `navigateTo` in `$lib/stores/curriculum.ts` is now documented as **internal route-sync only — UI code must use `goto` directly**. The function continues to set `currentPosition` for use by the dynamic lesson route's URL→store `$effect`; UI code (sidebar, dashboard, Next/Finish) routes via `goto()` so SvelteKit's full navigation lifecycle (page params, scroll restoration, `{#key}` re-mount) fires predictably.
+
 ## [0.45.0] - 2026-04-30
 
 ### Added

{learningfoundry-0.45.0 → learningfoundry-0.49.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: learningfoundry
-Version: 0.45.0
+Version: 0.49.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.45.0 → learningfoundry-0.49.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "learningfoundry"
-version = "0.45.0"
+version = "0.49.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.45.0 → learningfoundry-0.49.0}/src/learningfoundry/__init__.py

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

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/e2e/finish.spec.ts

@@ -0,0 +1,31 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// FR-P14 regression coverage: Finish on the last lesson clears the
+// active-lesson highlight and collapses the previously expanded sidebar
+// module, so the dashboard re-displays in a clean state.
+//
+// Reaching the last lesson and force-completing it requires viewport
+// scroll for the FR-P13 sentinel; pending a richer e2e fixture, this
+// spec validates the underlying invariant that the curriculum-title
+// link still works and that landing at `/` shows no sidebar lesson row
+// with the active highlight class.
+import { expect, test } from '@playwright/test';
+
+test.describe('Finish-on-last-lesson sidebar state', () => {
+	test('dashboard renders no active lesson highlight', async ({ page }) => {
+		await page.goto('/');
+		// The active-lesson highlight class is `bg-blue-100 text-blue-700` on
+		// a lesson row. On the dashboard with no current position, no lesson
+		// row should carry it.
+		const active = page.locator('aside nav ul ul button.bg-blue-100');
+		await expect(active).toHaveCount(0);
+	});
+
+	test('dashboard renders no expanded module by default', async ({ page }) => {
+		await page.goto('/');
+		// Module list panels render a nested `<ul>` only when expanded.
+		const expandedPanels = page.locator('aside nav > ul > li > div > ul');
+		await expect(expandedPanels).toHaveCount(0);
+	});
+});

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/e2e/navigation.spec.ts

@@ -0,0 +1,30 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// FR-P9 regression coverage: in-app navigation must update the URL.
+// Before v0.46.0 the sidebar / Next button updated `currentPosition` but
+// not the route, so `LessonView` was never re-mounted on lesson change.
+import { expect, test } from '@playwright/test';
+
+test.describe('lesson navigation routing', () => {
+	test('sidebar lesson click updates URL', async ({ page }) => {
+		await page.goto('/');
+		// Open the first module by clicking its header in the sidebar.
+		const firstModuleHeader = page.locator('aside nav button').first();
+		await firstModuleHeader.click();
+
+		// Click the first lesson row in the now-expanded module.
+		const firstLessonRow = page.locator('aside nav ul ul button').first();
+		await firstLessonRow.click();
+
+		// URL must reflect the click; before v0.46.0 it stayed at "/".
+		await expect(page).toHaveURL(/\/[^/]+\/[^/]+$/);
+	});
+
+	test('dashboard "Start module" deep-links into a lesson', async ({ page }) => {
+		await page.goto('/');
+		const startBtn = page.getByRole('button', { name: /start module|continue/i }).first();
+		await startBtn.click();
+		await expect(page).toHaveURL(/\/[^/]+\/[^/]+$/);
+	});
+});

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/e2e/progress.spec.ts

@@ -0,0 +1,24 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// FR-P9 regression: progress reactivity. Reaching a lesson page must mark
+// it `in_progress` and the sidebar status icon must reflect that on the
+// next sidebar render — without a page reload.
+import { expect, test } from '@playwright/test';
+
+test.describe('progress reactivity', () => {
+	test('navigating into a lesson marks it in_progress in the sidebar', async ({ page }) => {
+		await page.goto('/');
+
+		// Expand the first module and click into its first lesson.
+		await page.locator('aside nav button').first().click();
+		await page.locator('aside nav ul ul button').first().click();
+		await expect(page).toHaveURL(/\/[^/]+\/[^/]+$/);
+
+		// Sidebar status icons: ○ = not_started, … = in_progress, ✓ = complete.
+		// After landing on the lesson page, the active row should not still
+		// show ○ (regression check; the previous bug left status forever ○).
+		const activeIcon = page.locator('aside nav ul ul button.bg-blue-100 span').first();
+		await expect(activeIcon).not.toHaveText('○');
+	});
+});

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/e2e/reset.spec.ts

@@ -0,0 +1,32 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// FR-P12 regression coverage: Reset course button.
+//
+// The full round-trip from the story task list (complete a lesson →
+// button enables → reset → checkmark gone, % returns to 0, dashboard
+// reads "0 of N completed", URL is `/`) requires either a tall scroll
+// to satisfy the FR-P13 sentinel or a fixture without that constraint.
+// Until the e2e fixture story (FR-P11 fixture file) lands, these tests
+// validate the smaller invariants that don't require completion:
+//   - The button exists in the DOM at the bottom of the sidebar.
+//   - On a fresh load (no progress), it is disabled.
+//   - Clicking it while disabled is a no-op (URL unchanged).
+import { expect, test } from '@playwright/test';
+
+test.describe('Reset course button', () => {
+	test('renders disabled when no progress exists', async ({ page }) => {
+		await page.goto('/');
+		const btn = page.getByRole('button', { name: /Reset course progress/i });
+		await expect(btn).toBeVisible();
+		await expect(btn).toBeDisabled();
+	});
+
+	test('clicking the disabled button does not navigate', async ({ page }) => {
+		await page.goto('/');
+		const btn = page.getByRole('button', { name: /Reset course progress/i });
+		await btn.click({ force: true }).catch(() => {});
+		// Still on the dashboard.
+		expect(new URL(page.url()).pathname).toBe('/');
+	});
+});

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/e2e/text-block-bottom.spec.ts

@@ -0,0 +1,27 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// FR-P13 regression coverage: TextBlock completion requires the
+// end-of-block sentinel to be in view for 1 s. A tall text block where
+// the learner never scrolls past the top must NOT mark the lesson
+// complete — only scrolling to the bottom should.
+//
+// The smoke fixture's text content may not always exceed viewport
+// height (curriculum authors control content); we verify the sentinel
+// element is present on the lesson page and is positioned at the end
+// of the rendered markdown, which is the structural invariant FR-P13
+// depends on.
+import { expect, test } from '@playwright/test';
+
+test.describe('TextBlock end-of-block sentinel', () => {
+	test('sentinel element is rendered after the markdown', async ({ page }) => {
+		await page.goto('/');
+		await page.locator('aside nav button').first().click();
+		await page.locator('aside nav ul ul button').first().click();
+		await expect(page).toHaveURL(/\/[^/]+\/[^/]+$/);
+
+		// Wait for at least one TextBlock to render its prose and sentinel.
+		const sentinel = page.locator('[data-textblock-end]').first();
+		await expect(sentinel).toBeAttached();
+	});
+});

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/e2e/video.spec.ts

@@ -0,0 +1,27 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// FR-P10 regression coverage: when navigating between two lessons that
+// each contain a video block, only the new lesson's iframe should be
+// present — the previous player must be destroyed. The {#key} wrapper
+// in `[module]/[lesson]/+page.svelte` plus the stable block key in
+// `LessonView` enforce this.
+import { expect, test } from '@playwright/test';
+
+test.describe('video block lifecycle', () => {
+	test('lesson page renders at most one YouTube iframe per video block', async ({ page }) => {
+		await page.goto('/');
+		await page.locator('aside nav button').first().click();
+		await page.locator('aside nav ul ul button').first().click();
+		await expect(page).toHaveURL(/\/[^/]+\/[^/]+$/);
+
+		// Wait briefly for the YouTube IFrame API to upgrade the placeholder
+		// `<div id="yt-player-…">` into an `<iframe>`. If the API is blocked
+		// (no network in CI), there will be zero iframes — that's still a
+		// pass for "no leaked iframes from a prior lesson", which is what
+		// the regression cares about.
+		await page.waitForTimeout(2000);
+		const iframes = await page.locator('iframe[src*="youtube"]').count();
+		expect(iframes).toBeLessThanOrEqual(1);
+	});
+});

{learningfoundry-0.45.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/package.json

@@ -9,6 +9,7 @@
     "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
     "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
     "test": "vitest run",
+    "e2e": "playwright test",
     "postinstall": "node -e \"const fs=require('fs'); fs.mkdirSync('static',{recursive:true}); fs.copyFileSync(require.resolve('sql.js/dist/sql-wasm.wasm'),'static/sql-wasm.wasm')\""
   },
   "dependencies": {
@@ -21,6 +22,7 @@
     "svelte": "^5.0.0"
   },
   "devDependencies": {
+    "@playwright/test": "^1.48.0",
     "@sveltejs/adapter-static": "^3.0.0",
     "@sveltejs/vite-plugin-svelte": "^7.0.0",
     "@tailwindcss/typography": "^0.5.16",

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/playwright.config.ts

@@ -0,0 +1,27 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+import { defineConfig, devices } from '@playwright/test';
+
+const PORT = Number(process.env.PLAYWRIGHT_PORT ?? 4173);
+
+export default defineConfig({
+	testDir: './e2e',
+	fullyParallel: false,
+	reporter: [['list']],
+	use: {
+		baseURL: `http://localhost:${PORT}`,
+		trace: 'retain-on-failure'
+	},
+	webServer: {
+		command: `pnpm preview --port ${PORT} --strictPort`,
+		port: PORT,
+		reuseExistingServer: !process.env.CI,
+		timeout: 60_000
+	},
+	projects: [
+		{
+			name: 'chromium',
+			use: { ...devices['Desktop Chrome'] }
+		}
+	]
+});

{learningfoundry-0.45.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/LessonList.svelte

@@ -1,8 +1,10 @@
 <!-- Copyright 2026 Pointmatic — SPDX-License-Identifier: Apache-2.0 -->
 <script lang="ts">
-	import { currentPosition, navigateTo } from '$lib/stores/curriculum.js';
+	import { goto } from '$app/navigation';
+	import { currentPosition } from '$lib/stores/curriculum.js';
 	import type { Lesson, LessonProgress } from '$lib/types/index.js';
 	import { lessonStatusIcon, resolveLessonClick } from './module-list.helpers.js';
+	import { lessonHref } from './navigation.helpers.js';
 
 	interface Props {
 		moduleId: string;
@@ -34,7 +36,7 @@
 
 	function handleClick(lessonId: string) {
 		if (resolveLessonClick(lessonId, lockedLessons) === 'noop') return;
-		navigateTo(moduleId, lessonId);
+		void goto(lessonHref(moduleId, lessonId));
 	}
 </script>
 

{learningfoundry-0.45.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/LessonView.svelte

@@ -6,6 +6,7 @@
 	import type { Lesson, QuizScore } from '$lib/types/index.js';
 	import ContentBlock from './ContentBlock.svelte';
 	import Navigation from './Navigation.svelte';
+	import { contentBlockKey } from './lesson-view.helpers.js';
 	import { onMount } from 'svelte';
 
 	interface Props {
@@ -65,7 +66,7 @@
 	</header>
 
 	<div class="space-y-8">
-		{#each lesson.content_blocks as block, i (i)}
+		{#each lesson.content_blocks as block, i (contentBlockKey(block, i))}
 			<section>
 				<ContentBlock
 					{block}

{learningfoundry-0.45.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/components/LessonView.test.ts

@@ -1,7 +1,8 @@
 // Copyright 2026 Pointmatic
 // SPDX-License-Identifier: Apache-2.0
 import { describe, expect, it } from 'vitest';
-import { createBlockTracker } from './lesson-view.helpers.js';
+import { contentBlockKey, createBlockTracker } from './lesson-view.helpers.js';
+import type { ContentBlock } from '$lib/types/index.js';
 
 describe('LessonView block tracking (via createBlockTracker)', () => {
 	it('all blocks complete → returns true on final markBlockComplete', () => {
@@ -66,3 +67,51 @@ describe('LessonView block tracking (via createBlockTracker)', () => {
 		expect(tracker.allComplete).toBe(true);
 	});
 });
+
+describe('contentBlockKey (FR-P10 stable identity)', () => {
+	it('uses ref when present', () => {
+		const block = {
+			type: 'text',
+			source: null,
+			ref: 'content/mod-01/lesson-01.md',
+			content: { markdown: '', path: '' }
+		} as ContentBlock;
+		expect(contentBlockKey(block, 0)).toBe('text:content/mod-01/lesson-01.md');
+	});
+
+	it('uses content.url for video blocks (no ref)', () => {
+		const block = {
+			type: 'video',
+			source: null,
+			ref: null,
+			content: { url: 'https://www.youtube.com/watch?v=abc' }
+		} as ContentBlock;
+		expect(contentBlockKey(block, 0)).toBe('video:https://www.youtube.com/watch?v=abc');
+	});
+
+	it('falls back to type + index when neither ref nor url is present', () => {
+		const block = {
+			type: 'visualization',
+			source: null,
+			ref: null,
+			content: {}
+		} as unknown as ContentBlock;
+		expect(contentBlockKey(block, 2)).toBe('visualization-2');
+	});
+
+	it('two video blocks with different URLs produce different keys', () => {
+		const a = {
+			type: 'video',
+			source: null,
+			ref: null,
+			content: { url: 'https://youtu.be/AAA' }
+		} as ContentBlock;
+		const b = {
+			type: 'video',
+			source: null,
+			ref: null,
+			content: { url: 'https://youtu.be/BBB' }
+		} as ContentBlock;
+		expect(contentBlockKey(a, 0)).not.toBe(contentBlockKey(b, 0));
+	});
+});

{learningfoundry-0.45.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/ModuleList.svelte

@@ -3,7 +3,7 @@
 	import { currentPosition } from '$lib/stores/curriculum.js';
 	import type { Curriculum, Module, ModuleProgress } from '$lib/types/index.js';
 	import { getOptionalLessons, lockedLessonIds } from '$lib/utils/locking.js';
-	import { resolveModuleHeaderClick } from './module-list.helpers.js';
+	import { computeAutoExpand, resolveModuleHeaderClick } from './module-list.helpers.js';
 	import LessonList from './LessonList.svelte';
 	import ProgressBar from './ProgressBar.svelte';
 	import Lock from 'lucide-svelte/icons/lock';
@@ -42,12 +42,15 @@
 	// Auto-expand the module containing the current lesson.
 	// Only fire when `currentPosition.moduleId` changes to a *new* value;
 	// `lastAutoExpandedModuleId` breaks the self-dependency that previously
-	// caused manual toggles to revert immediately.
+	// caused manual toggles to revert immediately. When the position is
+	// cleared (Finish on the last lesson, FR-P14), collapse the previously
+	// expanded module so the dashboard sidebar starts from a clean slate.
 	$effect(() => {
 		const pos = $currentPosition;
-		if (pos && pos.moduleId !== lastAutoExpandedModuleId) {
-			expandedModuleId = pos.moduleId;
-			lastAutoExpandedModuleId = pos.moduleId;
+		const next = computeAutoExpand(pos?.moduleId ?? null, lastAutoExpandedModuleId);
+		if (next) {
+			expandedModuleId = next.expandedModuleId;
+			lastAutoExpandedModuleId = next.lastAutoExpandedModuleId;
 		}
 	});
 </script>

{learningfoundry-0.45.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/Navigation.svelte

@@ -2,7 +2,8 @@
 <script lang="ts">
 	import { goto } from '$app/navigation';
 	import { ChevronLeft, ChevronRight } from 'lucide-svelte';
-	import { nextLesson, previousLesson, navigateTo } from '$lib/stores/curriculum.js';
+	import { currentPosition, nextLesson, previousLesson } from '$lib/stores/curriculum.js';
+	import { resolveGoNext, resolveGoPrev } from './navigation.helpers.js';
 
 	interface Props {
 		disabled?: boolean;
@@ -13,16 +14,18 @@
 	const next = $derived($nextLesson);
 
 	function goNext() {
-		if (disabled) return;
-		if (next) {
-			navigateTo(next.moduleId, next.lessonId);
-		} else {
-			void goto('/');
-		}
+		const action = resolveGoNext(disabled, next);
+		if (action.kind === 'noop') return;
+		// Clear the position *before* `goto` so the sidebar's auto-expand
+		// effect sees the null transition and collapses the previously
+		// expanded module before the route change settles.
+		if (action.clearPosition) currentPosition.set(null);
+		void goto(action.url);
 	}
 
 	function goPrev() {
-		if (prev) navigateTo(prev.moduleId, prev.lessonId);
+		const action = resolveGoPrev(prev);
+		if (action.kind === 'goto') void goto(action.url);
 	}
 </script>
 

{learningfoundry-0.45.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/components/ProgressDashboard.svelte

@@ -1,6 +1,6 @@
 <!-- Copyright 2026 Pointmatic — SPDX-License-Identifier: Apache-2.0 -->
 <script lang="ts">
-	import { navigateTo } from '$lib/stores/curriculum.js';
+	import { goto } from '$app/navigation';
 	import type { Curriculum, Module, ModuleProgress, QuizScore } from '$lib/types/index.js';
 	import { getOptionalLessons, isModuleComplete } from '$lib/utils/locking.js';
 	import ProgressBar from './ProgressBar.svelte';
@@ -45,7 +45,7 @@
 			(l) => !optional.has(l.id) && mp?.lessons[l.id]?.status !== 'complete'
 		);
 		const target = firstIncomplete ?? mod.lessons[0];
-		if (target) navigateTo(mod.id, target.id);
+		if (target) void goto(`/${mod.id}/${target.id}`);
 	}
 
 	const totalLessons = $derived(modules.reduce((n, m) => n + m.lessons.length, 0));

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/src/lib/components/ResetCourseButton.svelte

@@ -0,0 +1,46 @@
+<!-- Copyright 2026 Pointmatic — SPDX-License-Identifier: Apache-2.0 -->
+<script lang="ts">
+	import { goto } from '$app/navigation';
+	import { resetProgress } from '$lib/db/index.js';
+	import { currentPosition, curriculum } from '$lib/stores/curriculum.js';
+	import { invalidateProgress } from '$lib/stores/progress.js';
+	import { RotateCcw } from 'lucide-svelte';
+
+	interface Props {
+		disabled?: boolean;
+		/** Override for unit tests; defaults to `window.confirm`. */
+		confirmFn?: (message: string) => boolean;
+	}
+	let {
+		disabled = false,
+		confirmFn = (msg: string) => window.confirm(msg)
+	}: Props = $props();
+
+	const PROMPT = 'Reset all progress for this curriculum? This cannot be undone.';
+
+	async function handleClick() {
+		if (disabled) return;
+		if (!confirmFn(PROMPT)) return;
+		await resetProgress();
+		// Clearing the position triggers the FR-P14 sidebar collapse path
+		// in `ModuleList`'s auto-expand `$effect` once Story I.n ships;
+		// pre-I.n it is a harmless no-op.
+		currentPosition.set(null);
+		await invalidateProgress($curriculum);
+		await goto('/');
+	}
+</script>
+
+<button
+	type="button"
+	onclick={handleClick}
+	disabled={disabled}
+	aria-disabled={disabled}
+	class="flex w-full items-center justify-center gap-2 rounded border border-transparent px-3 py-2 text-xs font-medium transition-colors
+		{disabled
+		? 'cursor-not-allowed text-gray-300'
+		: 'text-red-600 hover:bg-red-50'}"
+>
+	<RotateCcw size={14} aria-hidden="true" />
+	Reset course progress
+</button>

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/src/lib/components/ResetCourseButton.test.ts

@@ -0,0 +1,95 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+//
+// Logic-level coverage for the Reset Course button. We don't mount the
+// Svelte component (the existing test convention is to extract testable
+// logic instead). Instead we validate the click-handler contract:
+//
+//   - disabled → no DB writes, no goto
+//   - enabled + cancelled confirm → no DB writes, no goto
+//   - enabled + accepted confirm → resetProgress + invalidateProgress + goto('/')
+//
+// These three branches are the meaningful failure modes; the Svelte
+// rendering is a thin shell over them.
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+
+const { resetMock, invalidateMock, gotoMock, setPosition } = vi.hoisted(() => ({
+	resetMock: vi.fn().mockResolvedValue(undefined),
+	invalidateMock: vi.fn().mockResolvedValue(undefined),
+	gotoMock: vi.fn(),
+	setPosition: vi.fn()
+}));
+
+vi.mock('$app/navigation', () => ({ goto: gotoMock }));
+vi.mock('$lib/db/index.js', () => ({ resetProgress: resetMock }));
+vi.mock('$lib/stores/curriculum.js', () => ({
+	curriculum: { subscribe: (fn: (v: null) => void) => (fn(null), () => {}) },
+	currentPosition: { set: setPosition }
+}));
+vi.mock('$lib/stores/progress.js', () => ({ invalidateProgress: invalidateMock }));
+
+const PROMPT = 'Reset all progress for this curriculum? This cannot be undone.';
+
+/**
+ * Inline copy of the click-handler logic in `ResetCourseButton.svelte`.
+ * Kept in lockstep with the component; if you change the component, mirror
+ * the change here.
+ */
+async function clickHandler(
+	disabled: boolean,
+	confirmFn: (msg: string) => boolean
+): Promise<void> {
+	const { resetProgress } = await import('$lib/db/index.js');
+	const { currentPosition, curriculum } = await import('$lib/stores/curriculum.js');
+	const { invalidateProgress } = await import('$lib/stores/progress.js');
+	const { goto } = await import('$app/navigation');
+	if (disabled) return;
+	if (!confirmFn(PROMPT)) return;
+	await resetProgress();
+	currentPosition.set(null);
+	let cur = null;
+	curriculum.subscribe((v: unknown) => (cur = v as null))();
+	await invalidateProgress(cur);
+	await goto('/');
+}
+
+describe('ResetCourseButton click handler', () => {
+	beforeEach(() => {
+		resetMock.mockClear();
+		invalidateMock.mockClear();
+		gotoMock.mockClear();
+		setPosition.mockClear();
+	});
+
+	afterEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('disabled → no reset, no navigation', async () => {
+		const confirmFn = vi.fn(() => true);
+		await clickHandler(true, confirmFn);
+		expect(confirmFn).not.toHaveBeenCalled();
+		expect(resetMock).not.toHaveBeenCalled();
+		expect(setPosition).not.toHaveBeenCalled();
+		expect(invalidateMock).not.toHaveBeenCalled();
+		expect(gotoMock).not.toHaveBeenCalled();
+	});
+
+	it('enabled + cancelled confirm → no reset, no navigation', async () => {
+		const confirmFn = vi.fn(() => false);
+		await clickHandler(false, confirmFn);
+		expect(confirmFn).toHaveBeenCalledOnce();
+		expect(resetMock).not.toHaveBeenCalled();
+		expect(setPosition).not.toHaveBeenCalled();
+		expect(gotoMock).not.toHaveBeenCalled();
+	});
+
+	it('enabled + accepted confirm → resets, clears position, invalidates, navigates home', async () => {
+		const confirmFn = vi.fn(() => true);
+		await clickHandler(false, confirmFn);
+		expect(resetMock).toHaveBeenCalledOnce();
+		expect(setPosition).toHaveBeenCalledWith(null);
+		expect(invalidateMock).toHaveBeenCalledOnce();
+		expect(gotoMock).toHaveBeenCalledWith('/');
+	});
+});