@intellectif/lk-react 23.1.0 → 23.1.2

package/CHANGELOG.md

@@ -1,5 +1,28 @@
 # @intellectif/lk-react
 
+## 23.1.2
+
+### Patch Changes
+
+- [#98](https://github.com/intellectif/learning-kit/pull/98) [`e2b3100`](https://github.com/intellectif/learning-kit/commit/e2b310024f25b7d7e38054f833e324b2978ef7ac) Thanks [@diego28e](https://github.com/diego28e)! - **Components declare their return type**: every component's declaration now reads `React.JSX.Element`. The type is unchanged; only its spelling in the `.d.ts` files is.
+
+  **Action required:** none.
+
+  - Ten components (`ActivityPreview`, `Dictation`, `FillInTheBlanks`, `GapSelect`, `InteractiveVideo`, `MultipleChoice`, `PronunciationFeedback`, `ReadAloud`, `ThemeProvider` and `WrittenResponse`) left their return type to inference. The declarations therefore named it after the file TypeScript found it in: `import("react/jsx-runtime").JSX.Element`. `@types/react` 19.3 moved that type into `react`, so the same code built under it declared `import("react").JSX.Element` instead. They now declare `React.JSX.Element`, as the other components already did. Under `@types/react` 19.2 and 19.3, the old and new spellings are one type.
+  - The API report now rejects an exported type named through `import("…")`, so a dependency upgrade can no longer change what the package declares.
+
+## 23.1.1
+
+### Patch Changes
+
+- [#90](https://github.com/intellectif/learning-kit/pull/90) [`9b5f5ef`](https://github.com/intellectif/learning-kit/commit/9b5f5ef6243f825c1336628d38816789993455b3) Thanks [@diego28e](https://github.com/diego28e)! - **Multiple choice under `feedback: false`**: no longer says "Answer submitted." before the learner has answered.
+
+  **Action required:** none.
+
+  - With `delivery={{ feedback: false }}`, a `<MultipleChoice>` announced "Answer submitted." in its live region as soon as it rendered. It now says nothing until the learner submits, as every other activity component does; after a submit it still says only that the answer was received.
+  - **Built with tsdown**, as tsup is no longer maintained. Every entry point exports the same names in ESM and CommonJS, each module still starts with `'use client'`, and the declarations describe the same API; only the names of the internal chunk files differ.
+  - A new test walks all six activity components through the same moment, in practice and exam mode, under every delivery policy, so a component that drifts from the others fails by name.
+
 ## 23.1.0
 
 ### Minor Changes

package/README.md

@@ -151,9 +151,13 @@ subpath of their own.
 
 ### Versioning note
 
-`@intellectif/lk-core` is a **peer** dependency, so widening its range is a breaking
-change for installs and forces a `lk-react` major. Some majors here — **5.0.0**
-is the clearest — are exactly that and change no React API. Others do: 6.0.0
+`@intellectif/lk-core` is a **peer** dependency. Before 1.0 every `lk-core` minor
+forced a `lk-react` major, which is why this package is at 23.x; since 1.0 a
+`lk-core` minor or patch releases no `lk-react` at all, and when `lk-react` needs a
+newer `lk-core` its own release says which
+([Stability](https://github.com/intellectif/learning-kit/blob/main/docs/stability.md#the-two-version-numbers)).
+Some of the early majors — **5.0.0** is the clearest — are only that peer bump and
+change no React API. Others do: 6.0.0
 added five `<ActivitySequence>` props, a `restored` arm to `SequenceItemOutcome`
 that an exhaustive `switch` must handle, and a changed reset target for
 `<MultipleChoice>`. Always read the
@@ -165,7 +169,7 @@ before assuming a migration is needed.
 - **What is supported, and since when** — [Features](https://github.com/intellectif/learning-kit/blob/main/docs/features.md): every activity and capability, the `lk-core` / `lk-react` versions it arrived in, and its guide.
 - **What changed** — [GitHub Releases](https://github.com/intellectif/learning-kit/releases), one per package version, and the same notes in each package's `CHANGELOG.md` (it ships in the npm package, so it is in your `node_modules` too). Notes written since release notes gained a template — every release after lk-core 0.20.0 / lk-react 20.0.0 — start with a one-line summary and **Action required**: `none`, or what to do. Earlier notes lead with the detail; the version map in the upgrading guide says what each one asks of you.
 - **How to upgrade** — [Upgrading](https://github.com/intellectif/learning-kit/blob/main/docs/upgrading.md): which `lk-core` goes with which `lk-react`, and a section per release saying what, if anything, you need to change. A `lk-react` major is often only the `lk-core` peer bump; the map says so.
-- **Hearing about a release** — on GitHub, **Watch → Custom → Releases**, or subscribe to [the releases feed](https://github.com/intellectif/learning-kit/releases.atom). Better still, let your dependency bot bring each release to you as a pull request, with its notes — and keep the two packages together in one, since they move together:
+- **Hearing about a release** — on GitHub, **Watch → Custom → Releases**, or subscribe to [the releases feed](https://github.com/intellectif/learning-kit/releases.atom). Better still, let your dependency bot bring each release to you as a pull request, with its notes — and keep the two packages together in one, since a `lk-react` release names the `lk-core` it needs:
 
   ```yaml
   # .github/dependabot.yml — one entry per directory with a package.json
@@ -181,7 +185,7 @@ before assuming a migration is needed.
 
   With Renovate: `"packageRules": [{ "matchPackageNames": ["@intellectif/lk-core", "@intellectif/lk-react"], "groupName": "learning-kit" }]`.
 
-  `lk-core` is still `0.x`, and a caret range on a `0.x` version matches only that minor: `^0.18.0` never installs `0.19.0`. A bot is how you find out there is one.
+  A caret range installs every later minor and patch of its major — `^1.2.0` takes `1.3.0` — and never a new major. A bot is how you hear about either, with its notes.
 
 ## Documentation
 

package/dist/ActivityErrorBoundary-B379hvtL.cjs

@@ -0,0 +1,39 @@
+'use client';
+const require_useAiHelp = require("./useAiHelp-h9yOLFqt.cjs");
+const require_LkIntlProvider = require("./LkIntlProvider-oi57lYB0.cjs");
+let react = require("react");
+let react_jsx_runtime = require("react/jsx-runtime");
+//#region src/components/ActivityErrorBoundary.tsx
+/**
+* Catches render errors in an activity subtree. In development it renders the
+* full error stack to aid debugging; in production it renders an accessible,
+* generic fallback (optionally naming the activity) so a single broken
+* activity never crashes the host application.
+*/
+var ActivityErrorBoundary = class extends react.Component {
+	state = { error: null };
+	static getDerivedStateFromError(error) {
+		return { error };
+	}
+	render() {
+		const { error } = this.state;
+		if (!error) return this.props.children;
+		if (require_useAiHelp.isDevelopment()) return /* @__PURE__ */ (0, react_jsx_runtime.jsxs)("div", {
+			role: "alert",
+			children: [/* @__PURE__ */ (0, react_jsx_runtime.jsx)("strong", { children: (this.props.strings ?? require_LkIntlProvider.DEFAULT_STRINGS).activityFailed }), /* @__PURE__ */ (0, react_jsx_runtime.jsx)("pre", { children: error.stack ?? error.message })]
+		});
+		return /* @__PURE__ */ (0, react_jsx_runtime.jsx)("div", {
+			role: "alert",
+			children: this.props.activityTitle ? (this.props.strings ?? require_LkIntlProvider.DEFAULT_STRINGS).activityFailedNamed(this.props.activityTitle) : (this.props.strings ?? require_LkIntlProvider.DEFAULT_STRINGS).activityFailedUnnamed
+		});
+	}
+};
+//#endregion
+Object.defineProperty(exports, "ActivityErrorBoundary", {
+	enumerable: true,
+	get: function() {
+		return ActivityErrorBoundary;
+	}
+});
+
+//# sourceMappingURL=ActivityErrorBoundary-B379hvtL.cjs.map

package/dist/ActivityErrorBoundary-B379hvtL.cjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"ActivityErrorBoundary-B379hvtL.cjs","names":["Component","isDevelopment","DEFAULT_STRINGS"],"sources":["../src/components/ActivityErrorBoundary.tsx"],"sourcesContent":["'use client';\n\nimport { Component, type ReactNode } from 'react';\nimport type { LkStrings } from '../i18n/strings.js';\nimport { DEFAULT_STRINGS } from '../i18n/strings.js';\nimport { isDevelopment } from './_internal.js';\n\nexport interface ActivityErrorBoundaryProps {\n  /** Shown in the production fallback when a render error occurs. */\n  activityTitle?: string;\n  /**\n   * Chrome text. A class component cannot call `useLkStrings`, so the public\n   * wrapper reads the hook and hands the resolved dictionary down.\n   */\n  strings?: LkStrings;\n  children: ReactNode;\n}\n\ninterface ActivityErrorBoundaryState {\n  error: Error | null;\n}\n\n/**\n * Catches render errors in an activity subtree. In development it renders the\n * full error stack to aid debugging; in production it renders an accessible,\n * generic fallback (optionally naming the activity) so a single broken\n * activity never crashes the host application.\n */\nexport class ActivityErrorBoundary extends Component<\n  ActivityErrorBoundaryProps,\n  ActivityErrorBoundaryState\n> {\n  state: ActivityErrorBoundaryState = { error: null };\n\n  static getDerivedStateFromError(error: Error): ActivityErrorBoundaryState {\n    return { error };\n  }\n\n  render(): ReactNode {\n    const { error } = this.state;\n    if (!error) {\n      return this.props.children;\n    }\n\n    // The same answer every development check reads, from the one function\n    // that gives it: a boundary with a reading of its own showed a learner a\n    // stack trace in every browser production build.\n    if (isDevelopment()) {\n      return (\n        <div role=\"alert\">\n          <strong>{(this.props.strings ?? DEFAULT_STRINGS).activityFailed}</strong>\n          <pre>{error.stack ?? error.message}</pre>\n        </div>\n      );\n    }\n\n    return (\n      <div role=\"alert\">\n        {this.props.activityTitle\n          ? (this.props.strings ?? DEFAULT_STRINGS).activityFailedNamed(this.props.activityTitle)\n          : (this.props.strings ?? DEFAULT_STRINGS).activityFailedUnnamed}\n      </div>\n    );\n  }\n}\n"],"mappings":";;;;;;;;;;;AA4BA,IAAa,wBAAb,cAA2CA,MAAAA,UAGzC;CACA,QAAoC,EAAE,OAAO,KAAK;CAElD,OAAO,yBAAyB,OAA0C;EACxE,OAAO,EAAE,MAAM;CACjB;CAEA,SAAoB;EAClB,MAAM,EAAE,UAAU,KAAK;EACvB,IAAI,CAAC,OACH,OAAO,KAAK,MAAM;EAMpB,IAAIC,kBAAAA,cAAc,GAChB,OACE,iBAAA,GAAA,kBAAA,KAAA,CAAC,OAAD;GAAK,MAAK;GAAV,UAAA,CACE,iBAAA,GAAA,kBAAA,IAAA,CAAC,UAAD,EAAA,WAAU,KAAK,MAAM,WAAWC,uBAAAA,gBAAAA,CAAiB,eAAuB,CAAA,GACxE,iBAAA,GAAA,kBAAA,IAAA,CAAC,OAAD,EAAA,UAAM,MAAM,SAAS,MAAM,QAAa,CAAA,CACrC;;EAIT,OACE,iBAAA,GAAA,kBAAA,IAAA,CAAC,OAAD;GAAK,MAAK;GACP,UAAA,KAAK,MAAM,iBACP,KAAK,MAAM,WAAWA,uBAAAA,gBAAAA,CAAiB,oBAAoB,KAAK,MAAM,aAAa,KACnF,KAAK,MAAM,WAAWA,uBAAAA,gBAAAA,CAAiB;EACzC,CAAA;CAET;AACF"}

package/dist/ActivityErrorBoundary-Dw6SbWv3.js

@@ -0,0 +1,34 @@
+'use client';
+import { c as isDevelopment } from "./useAiHelp-B-ccDL5Q.js";
+import { a as DEFAULT_STRINGS } from "./LkIntlProvider-D_FdYXjQ.js";
+import { Component } from "react";
+import { jsx, jsxs } from "react/jsx-runtime";
+//#region src/components/ActivityErrorBoundary.tsx
+/**
+* Catches render errors in an activity subtree. In development it renders the
+* full error stack to aid debugging; in production it renders an accessible,
+* generic fallback (optionally naming the activity) so a single broken
+* activity never crashes the host application.
+*/
+var ActivityErrorBoundary = class extends Component {
+	state = { error: null };
+	static getDerivedStateFromError(error) {
+		return { error };
+	}
+	render() {
+		const { error } = this.state;
+		if (!error) return this.props.children;
+		if (isDevelopment()) return /* @__PURE__ */ jsxs("div", {
+			role: "alert",
+			children: [/* @__PURE__ */ jsx("strong", { children: (this.props.strings ?? DEFAULT_STRINGS).activityFailed }), /* @__PURE__ */ jsx("pre", { children: error.stack ?? error.message })]
+		});
+		return /* @__PURE__ */ jsx("div", {
+			role: "alert",
+			children: this.props.activityTitle ? (this.props.strings ?? DEFAULT_STRINGS).activityFailedNamed(this.props.activityTitle) : (this.props.strings ?? DEFAULT_STRINGS).activityFailedUnnamed
+		});
+	}
+};
+//#endregion
+export { ActivityErrorBoundary as t };
+
+//# sourceMappingURL=ActivityErrorBoundary-Dw6SbWv3.js.map

package/dist/ActivityErrorBoundary-Dw6SbWv3.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ActivityErrorBoundary-Dw6SbWv3.js","names":[],"sources":["../src/components/ActivityErrorBoundary.tsx"],"sourcesContent":["'use client';\n\nimport { Component, type ReactNode } from 'react';\nimport type { LkStrings } from '../i18n/strings.js';\nimport { DEFAULT_STRINGS } from '../i18n/strings.js';\nimport { isDevelopment } from './_internal.js';\n\nexport interface ActivityErrorBoundaryProps {\n  /** Shown in the production fallback when a render error occurs. */\n  activityTitle?: string;\n  /**\n   * Chrome text. A class component cannot call `useLkStrings`, so the public\n   * wrapper reads the hook and hands the resolved dictionary down.\n   */\n  strings?: LkStrings;\n  children: ReactNode;\n}\n\ninterface ActivityErrorBoundaryState {\n  error: Error | null;\n}\n\n/**\n * Catches render errors in an activity subtree. In development it renders the\n * full error stack to aid debugging; in production it renders an accessible,\n * generic fallback (optionally naming the activity) so a single broken\n * activity never crashes the host application.\n */\nexport class ActivityErrorBoundary extends Component<\n  ActivityErrorBoundaryProps,\n  ActivityErrorBoundaryState\n> {\n  state: ActivityErrorBoundaryState = { error: null };\n\n  static getDerivedStateFromError(error: Error): ActivityErrorBoundaryState {\n    return { error };\n  }\n\n  render(): ReactNode {\n    const { error } = this.state;\n    if (!error) {\n      return this.props.children;\n    }\n\n    // The same answer every development check reads, from the one function\n    // that gives it: a boundary with a reading of its own showed a learner a\n    // stack trace in every browser production build.\n    if (isDevelopment()) {\n      return (\n        <div role=\"alert\">\n          <strong>{(this.props.strings ?? DEFAULT_STRINGS).activityFailed}</strong>\n          <pre>{error.stack ?? error.message}</pre>\n        </div>\n      );\n    }\n\n    return (\n      <div role=\"alert\">\n        {this.props.activityTitle\n          ? (this.props.strings ?? DEFAULT_STRINGS).activityFailedNamed(this.props.activityTitle)\n          : (this.props.strings ?? DEFAULT_STRINGS).activityFailedUnnamed}\n      </div>\n    );\n  }\n}\n"],"mappings":";;;;;;;;;;;AA4BA,IAAa,wBAAb,cAA2C,UAGzC;CACA,QAAoC,EAAE,OAAO,KAAK;CAElD,OAAO,yBAAyB,OAA0C;EACxE,OAAO,EAAE,MAAM;CACjB;CAEA,SAAoB;EAClB,MAAM,EAAE,UAAU,KAAK;EACvB,IAAI,CAAC,OACH,OAAO,KAAK,MAAM;EAMpB,IAAI,cAAc,GAChB,OACE,qBAAC,OAAD;GAAK,MAAK;GAAV,UAAA,CACE,oBAAC,UAAD,EAAA,WAAU,KAAK,MAAM,WAAW,gBAAA,CAAiB,eAAuB,CAAA,GACxE,oBAAC,OAAD,EAAA,UAAM,MAAM,SAAS,MAAM,QAAa,CAAA,CACrC;;EAIT,OACE,oBAAC,OAAD;GAAK,MAAK;GACP,UAAA,KAAK,MAAM,iBACP,KAAK,MAAM,WAAW,gBAAA,CAAiB,oBAAoB,KAAK,MAAM,aAAa,KACnF,KAAK,MAAM,WAAW,gBAAA,CAAiB;EACzC,CAAA;CAET;AACF"}