learningfoundry 0.46.0__tar.gz → 0.49.0__tar.gz

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

@@ -7,6 +7,24 @@ 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

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

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

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

@@ -1,4 +1,4 @@
 # Copyright 2026 Pointmatic
 # SPDX-License-Identifier: Apache-2.0
 
-__version__ = "0.46.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/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.46.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.46.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/Navigation.svelte

@@ -2,7 +2,7 @@
 <script lang="ts">
 	import { goto } from '$app/navigation';
 	import { ChevronLeft, ChevronRight } from 'lucide-svelte';
-	import { nextLesson, previousLesson } from '$lib/stores/curriculum.js';
+	import { currentPosition, nextLesson, previousLesson } from '$lib/stores/curriculum.js';
 	import { resolveGoNext, resolveGoPrev } from './navigation.helpers.js';
 
 	interface Props {
@@ -15,7 +15,12 @@
 
 	function goNext() {
 		const action = resolveGoNext(disabled, next);
-		if (action.kind === 'goto') void goto(action.url);
+		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() {

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('/');
+	});
+});

{learningfoundry-0.46.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/TextBlock.svelte

@@ -11,11 +11,17 @@
 	let { content, ontextcomplete }: Props = $props();
 
 	const html = $derived(renderMarkdown(content.markdown));
-	let blockEl: HTMLDivElement | undefined = $state();
+	// Observe a zero-size sentinel placed at the *end* of the rendered
+	// markdown rather than the wrapper itself. Otherwise a tall block fires
+	// `textcomplete` simply because the top of the block is in view on
+	// initial render — the learner would never have to scroll to the lesson
+	// body. With the sentinel, completion requires the bottom of the block
+	// to be in view for 1 s.
+	let sentinelEl: HTMLDivElement | undefined = $state();
 	let fired = false;
 
 	onMount(() => {
-		if (!blockEl || !ontextcomplete) return;
+		if (!sentinelEl || !ontextcomplete) return;
 		let timer: ReturnType<typeof setTimeout> | null = null;
 
 		const observer = new IntersectionObserver(
@@ -37,7 +43,7 @@
 			{ threshold: 0.1 }
 		);
 
-		observer.observe(blockEl);
+		observer.observe(sentinelEl);
 
 		return () => {
 			observer.disconnect();
@@ -46,6 +52,7 @@
 	});
 </script>
 
-<div bind:this={blockEl} class="prose prose-slate max-w-none">
+<div class="prose prose-slate max-w-none">
 	{@html html}
+	<div bind:this={sentinelEl} aria-hidden="true" data-textblock-end></div>
 </div>

{learningfoundry-0.46.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/components/TextBlock.test.ts

@@ -3,6 +3,12 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import { createViewportTracker } from '$lib/utils/viewport-completion.js';
 
+// FR-P13: TextBlock completion observes a sentinel at the end of the
+// rendered markdown — completion requires the *bottom* of the block to
+// be in view for 1 s, not just any portion. The viewport tracker tested
+// here is target-agnostic; the sentinel-vs-wrapper choice is made in
+// `TextBlock.svelte` by which element it `observer.observe()`s.
+
 describe('TextBlock completion (via createViewportTracker, 1 s delay)', () => {
 	beforeEach(() => {
 		vi.useFakeTimers();
@@ -58,6 +64,49 @@ describe('TextBlock completion (via createViewportTracker, 1 s delay)', () => {
 		tracker.destroy();
 	});
 
+	it('tall block: sentinel never intersects → does not fire even after 5 s', () => {
+		const callback = vi.fn();
+		const tracker = createViewportTracker(callback, 1000);
+
+		// Sentinel at the end of a tall block is never in the viewport.
+		// The component never calls `handleIntersecting` for this case.
+		vi.advanceTimersByTime(5000);
+		expect(callback).not.toHaveBeenCalled();
+
+		tracker.destroy();
+	});
+
+	it('tall block: sentinel scrolled into view fires 1 s later', () => {
+		const callback = vi.fn();
+		const tracker = createViewportTracker(callback, 1000);
+
+		// Initial state: sentinel below the fold. Simulate a scroll that
+		// brings the sentinel into the viewport.
+		vi.advanceTimersByTime(2000);
+		expect(callback).not.toHaveBeenCalled();
+
+		tracker.handleIntersecting();
+		vi.advanceTimersByTime(999);
+		expect(callback).not.toHaveBeenCalled();
+		vi.advanceTimersByTime(1);
+		expect(callback).toHaveBeenCalledOnce();
+
+		tracker.destroy();
+	});
+
+	it('sentinel briefly visible (<1 s) then hidden → does not fire', () => {
+		const callback = vi.fn();
+		const tracker = createViewportTracker(callback, 1000);
+
+		tracker.handleIntersecting();
+		vi.advanceTimersByTime(700);
+		tracker.handleNotIntersecting();
+		vi.advanceTimersByTime(2000);
+
+		expect(callback).not.toHaveBeenCalled();
+		tracker.destroy();
+	});
+
 	it('restarts timer when re-entering viewport after leaving', () => {
 		const callback = vi.fn();
 		const tracker = createViewportTracker(callback, 1000);

{learningfoundry-0.46.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/components/module-list.helpers.ts

@@ -8,18 +8,33 @@
  */
 
 /**
- * Determine whether the sidebar should auto-expand a module because the
- * current navigation position has moved to a new module.
+ * Determine how the sidebar should react to a change in the current
+ * navigation position.
  *
- * Returns the new `expandedModuleId` and `lastAutoExpandedModuleId` values,
- * or `null` if no auto-expand should happen (i.e. the module was already
- * auto-expanded or there is no current position).
+ * Returns:
+ * - `null` — no change required (position is null and nothing was
+ *   previously auto-expanded; or the current module already matches
+ *   the last auto-expanded module).
+ * - `{ expandedModuleId: null, lastAutoExpandedModuleId: null }` — the
+ *   position was just cleared (FR-P14: Finish on the last lesson);
+ *   collapse the previously expanded module and forget the auto-expand
+ *   anchor so the next manual toggle starts from a clean slate.
+ * - `{ expandedModuleId, lastAutoExpandedModuleId }` — auto-expand the
+ *   new module (and remember it as the auto-expand anchor so manual
+ *   toggles aren't reverted; see Story I.f).
  */
 export function computeAutoExpand(
-	currentModuleId: string | undefined,
+	currentModuleId: string | null | undefined,
 	lastAutoExpandedModuleId: string | null
-): { expandedModuleId: string; lastAutoExpandedModuleId: string } | null {
-	if (!currentModuleId) return null;
+): { expandedModuleId: string | null; lastAutoExpandedModuleId: string | null } | null {
+	if (!currentModuleId) {
+		// Position cleared. Only emit a reset if we previously auto-expanded
+		// — otherwise we'd loop forever rewriting the same null values.
+		if (lastAutoExpandedModuleId !== null) {
+			return { expandedModuleId: null, lastAutoExpandedModuleId: null };
+		}
+		return null;
+	}
 	if (currentModuleId === lastAutoExpandedModuleId) return null;
 	return {
 		expandedModuleId: currentModuleId,

{learningfoundry-0.46.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/components/module-list.test.ts

@@ -2,6 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 import { describe, expect, it } from 'vitest';
 import {
+	computeAutoExpand,
 	lessonStatusIcon,
 	resolveLessonClick,
 	resolveModuleHeaderClick
@@ -63,3 +64,42 @@ describe('lessonStatusIcon (optional rendering)', () => {
 		expect(lessonStatusIcon('l1', 'not_started', new Set())).toBe('○');
 	});
 });
+
+describe('computeAutoExpand (FR-P14 sidebar reset on null position)', () => {
+	it('expanding into a new module: returns expand instruction', () => {
+		const result = computeAutoExpand('mod-01', null);
+		expect(result).toEqual({
+			expandedModuleId: 'mod-01',
+			lastAutoExpandedModuleId: 'mod-01'
+		});
+	});
+
+	it('staying in the same module: returns null (no-op)', () => {
+		expect(computeAutoExpand('mod-01', 'mod-01')).toBeNull();
+	});
+
+	it('null position with no prior auto-expand: null (no-op, prevents re-run loop)', () => {
+		expect(computeAutoExpand(null, null)).toBeNull();
+	});
+
+	it('null position after auto-expand: resets both expanded and last-auto', () => {
+		expect(computeAutoExpand(null, 'mod-01')).toEqual({
+			expandedModuleId: null,
+			lastAutoExpandedModuleId: null
+		});
+	});
+
+	it('after a Finish reset, subsequent auto-expand into a new module still works', () => {
+		// First: position cleared from mod-01 → both reset to null.
+		expect(computeAutoExpand(null, 'mod-01')).toEqual({
+			expandedModuleId: null,
+			lastAutoExpandedModuleId: null
+		});
+		// Then: navigating into mod-02 should auto-expand it (regression
+		// check that I.f's manual-toggle preservation is still intact).
+		expect(computeAutoExpand('mod-02', null)).toEqual({
+			expandedModuleId: 'mod-02',
+			lastAutoExpandedModuleId: 'mod-02'
+		});
+	});
+});

{learningfoundry-0.46.0 → learningfoundry-0.49.0/src/learningfoundry}/sveltekit_template/src/lib/components/navigation.helpers.ts

@@ -9,16 +9,26 @@
  */
 import type { NavPosition } from '$lib/stores/curriculum.js';
 
-export type NavAction = { kind: 'noop' } | { kind: 'goto'; url: string };
+export type NavAction =
+	| { kind: 'noop' }
+	| { kind: 'goto'; url: string; clearPosition?: boolean };
 
-/** Resolve the action for the Next/Finish button click. */
+/**
+ * Resolve the action for the Next/Finish button click.
+ *
+ * On Finish (`next === null`) the action carries `clearPosition: true`
+ * so the caller wipes `currentPosition` *before* `goto('/')` runs —
+ * that lets the sidebar's auto-expand effect see the null transition
+ * and collapse the previously expanded module before the URL change
+ * settles (FR-P14).
+ */
 export function resolveGoNext(
 	disabled: boolean,
 	next: NavPosition | null
 ): NavAction {
 	if (disabled) return { kind: 'noop' };
 	if (next) return { kind: 'goto', url: `/${next.moduleId}/${next.lessonId}` };
-	return { kind: 'goto', url: '/' };
+	return { kind: 'goto', url: '/', clearPosition: true };
 }
 
 /** Resolve the action for the Previous button click. */

{learningfoundry-0.46.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/components/navigation.test.ts

@@ -9,9 +9,18 @@ describe('resolveGoNext (Next/Finish button)', () => {
 		expect(action).toEqual({ kind: 'goto', url: '/mod-01/lesson-02' });
 	});
 
-	it('routes to / when next is null (Finish on last lesson)', () => {
+	it('routes to / when next is null (Finish on last lesson) and signals position-clear', () => {
 		const action = resolveGoNext(false, null);
-		expect(action).toEqual({ kind: 'goto', url: '/' });
+		expect(action).toEqual({ kind: 'goto', url: '/', clearPosition: true });
+	});
+
+	it('does not signal position-clear when there is a next lesson', () => {
+		const action = resolveGoNext(false, { moduleId: 'mod-01', lessonId: 'lesson-02' });
+		// `clearPosition` is omitted (or false-y) for in-curriculum navigation.
+		expect(action.kind).toBe('goto');
+		if (action.kind === 'goto') {
+			expect(action.clearPosition).toBeFalsy();
+		}
 	});
 
 	it('disabled state is a no-op even with a next lesson', () => {

{learningfoundry-0.46.0 → learningfoundry-0.49.0}/src/learningfoundry/sveltekit_template/src/lib/db/index.ts

@@ -7,6 +7,7 @@ export {
 	getQuizScore,
 	markLessonComplete,
 	markLessonInProgress,
+	resetProgress,
 	saveQuizScore,
 	updateExerciseStatus
 } from './progress.js';

learningfoundry-0.49.0/src/learningfoundry/sveltekit_template/src/lib/db/progress.test.ts

@@ -0,0 +1,44 @@
+// Copyright 2026 Pointmatic
+// SPDX-License-Identifier: Apache-2.0
+import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+
+// `vi.mock` is hoisted to the top of the file, so any references inside
+// the factory must come from `vi.hoisted` (which is also hoisted).
+const { execMock, persistMock } = vi.hoisted(() => ({
+	execMock: vi.fn(),
+	persistMock: vi.fn().mockResolvedValue(undefined)
+}));
+
+vi.mock('./database.js', () => ({
+	getDb: vi.fn().mockResolvedValue({ exec: execMock, run: vi.fn() }),
+	persistDb: () => persistMock()
+}));
+
+import { resetProgress } from './progress.js';
+
+describe('resetProgress', () => {
+	beforeEach(() => {
+		execMock.mockClear();
+		persistMock.mockClear();
+	});
+
+	afterEach(() => {
+		vi.clearAllMocks();
+	});
+
+	it('truncates lesson_progress, quiz_scores, and exercise_status in a single transaction', async () => {
+		await resetProgress();
+		expect(execMock).toHaveBeenCalledTimes(1);
+		const sql = String(execMock.mock.calls[0][0]);
+		expect(sql).toMatch(/BEGIN;/);
+		expect(sql).toMatch(/DELETE FROM lesson_progress;/);
+		expect(sql).toMatch(/DELETE FROM quiz_scores;/);
+		expect(sql).toMatch(/DELETE FROM exercise_status;/);
+		expect(sql).toMatch(/COMMIT;/);
+	});
+
+	it('persists after the truncate', async () => {
+		await resetProgress();
+		expect(persistMock).toHaveBeenCalledTimes(1);
+	});
+});