From 77bb8202c94bd0346282c893f6eba76df1611452 Mon Sep 17 00:00:00 2001
From: Braden MacDonald <braden@opencraft.com>
Date: Tue, 31 Mar 2026 15:35:52 -0700
Subject: [PATCH 1/6] test: expand oxlint config, enable 'react' lints, remove
 temporary disable

---
 .oxlintrc.json | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/.oxlintrc.json b/.oxlintrc.json
index 7aa3afaeef..827999be4a 100644
--- a/.oxlintrc.json
+++ b/.oxlintrc.json
@@ -3,9 +3,14 @@
   "categories": {
     "correctness": "warn"
   },
+  "plugins": ["react"],
   "rules": {
     "eslint/no-unused-vars": "off",
-    "typescript/unbound-method": "off", // 🛑 TEMPORARY
+    // We disable exhaustive-deps because: it's noisy, and we often include extra deps when we want a memoized thing to
+    // re-calculate after some change, even if we're not using that thing in the calculation.
+    "react-hooks/exhaustive-deps": "off",
+    // Rule of hooks is useful, but not on by default:
+    "react/rules-of-hooks": "warn",
     "typescript/no-floating-promises": ["error", {
       "allowForKnownSafeCalls": [
         // queryClient.invalidateQueries returns a promise that can be awaited

From ed7f02391a9e4b960694479f29d95dd0eadace96 Mon Sep 17 00:00:00 2001
From: Braden MacDonald <braden@opencraft.com>
Date: Tue, 31 Mar 2026 16:11:02 -0700
Subject: [PATCH 2/6] chore: fix missing keys

---
 plugins/course-apps/teams/GroupEditor.jsx                 | 2 +-
 src/advanced-settings/AdvancedSettings.tsx                | 2 +-
 src/course-libraries/OutOfSyncAlert.tsx                   | 4 +---
 src/course-outline/page-alerts/PageAlerts.jsx             | 3 +++
 src/course-unit/clipboard/paste-notification/index.jsx    | 2 ++
 src/course-unit/unit-sidebar/AddSidebar.tsx               | 2 ++
 src/course-updates/CourseUpdates.tsx                      | 2 ++
 src/editors/EditorContainer.tsx                           | 1 +
 .../settingsComponents/GroupFeedback/GroupFeedbackRow.jsx | 1 +
 .../components/LicenseWidget/LicenseSelector.jsx          | 6 +++---
 .../components/TranscriptWidget/LanguageSelector.jsx      | 8 +++++---
 .../components/TranscriptWidget/index.jsx                 | 1 +
 src/files-and-videos/files-page/FileValidationModal.jsx   | 2 +-
 .../sort-and-filter-modal/SortAndFilterModal.jsx          | 2 +-
 .../videos-page/info-sidebar/TranscriptTab.jsx            | 4 +++-
 .../AlertAgreementGatedFeature.tsx                        | 7 +++----
 src/generic/sidebar/BlockCardButton.tsx                   | 2 +-
 src/generic/sidebar/Sidebar.tsx                           | 2 +-
 src/grading-settings/GradingSettings.jsx                  | 2 +-
 src/group-configurations/index.jsx                        | 1 +
 src/library-authoring/LibraryContent.tsx                  | 1 +
 src/library-authoring/import-course/ImportDetailsPage.tsx | 3 +++
 src/optimizer-page/scan-results/ScanResults.tsx           | 2 +-
 src/optimizer-page/scan-results/filterModal.jsx           | 2 +-
 src/studio-home/tabs-section/courses-tab/index.tsx        | 1 +
 src/textbooks/textbook-form/TextbookForm.jsx              | 3 ++-
 26 files changed, 44 insertions(+), 24 deletions(-)

diff --git a/plugins/course-apps/teams/GroupEditor.jsx b/plugins/course-apps/teams/GroupEditor.jsx
index b43348b51b..5d4d457b5c 100644
--- a/plugins/course-apps/teams/GroupEditor.jsx
+++ b/plugins/course-apps/teams/GroupEditor.jsx
@@ -48,7 +48,7 @@ const GroupEditor = ({
         ? (
           <div className="d-flex flex-column card rounded mb-3 px-4 py-2 p-4" key="isDeleting">
             <h4 className="mb-3">{intl.formatMessage(messages.groupDeleteHeading)}</h4>
-            {intl.formatMessage(messages.groupDeleteBody).split('\n').map(text => <p>{text}</p>)}
+            {intl.formatMessage(messages.groupDeleteBody).split('\n').map(text => <p key={text}>{text}</p>)}
             <div className="d-flex flex-row justify-content-end">
               <Button variant="muted" size="sm" onClick={cancelDeletion}>
                 {intl.formatMessage(messages.cancel)}
diff --git a/src/advanced-settings/AdvancedSettings.tsx b/src/advanced-settings/AdvancedSettings.tsx
index c02b3ecd50..6c042ea0d2 100644
--- a/src/advanced-settings/AdvancedSettings.tsx
+++ b/src/advanced-settings/AdvancedSettings.tsx
@@ -282,7 +282,7 @@ const AdvancedSettings = () => {
           role="dialog"
           actions={[
             !isQueryPending ? (
-              <Button variant="tertiary" onClick={handleResetSettingsValues}>
+              <Button key="cancelBtn" variant="tertiary" onClick={handleResetSettingsValues}>
                 {intl.formatMessage(messages.buttonCancelText)}
               </Button>
             ) : /* istanbul ignore next */ null,
diff --git a/src/course-libraries/OutOfSyncAlert.tsx b/src/course-libraries/OutOfSyncAlert.tsx
index 5a365a2737..fe61e345d2 100644
--- a/src/course-libraries/OutOfSyncAlert.tsx
+++ b/src/course-libraries/OutOfSyncAlert.tsx
@@ -67,9 +67,7 @@ export const OutOfSyncAlert: React.FC<OutOfSyncAlertProps> = ({
       variant="info"
       onClose={dismissAlert}
       actions={[
-        <Button
-          onClick={onReview}
-        >
+        <Button key="review-btn" onClick={onReview}>
           {intl.formatMessage(messages.outOfSyncCountAlertReviewBtn)}
         </Button>,
       ]}
diff --git a/src/course-outline/page-alerts/PageAlerts.jsx b/src/course-outline/page-alerts/PageAlerts.jsx
index dad989010e..b949fe2e36 100644
--- a/src/course-outline/page-alerts/PageAlerts.jsx
+++ b/src/course-outline/page-alerts/PageAlerts.jsx
@@ -104,6 +104,7 @@ const PageAlerts = ({
         onClose={onDismiss}
         actions={[
           <Button
+            key="learnMore"
             href={discussionsIncontextLearnmoreUrl}
             target="_blank"
           >
@@ -267,6 +268,7 @@ const PageAlerts = ({
           onClose={onDismiss}
           actions={[
             <Button
+              key="view-files"
               as={Link}
               to={getAssetsUrl()}
             >
@@ -329,6 +331,7 @@ const PageAlerts = ({
           onClose={onDismiss}
           actions={[
             <Button
+              key="view-files"
               as={Link}
               to={getAssetsUrl()}
             >
diff --git a/src/course-unit/clipboard/paste-notification/index.jsx b/src/course-unit/clipboard/paste-notification/index.jsx
index 20eed888df..53f437a185 100644
--- a/src/course-unit/clipboard/paste-notification/index.jsx
+++ b/src/course-unit/clipboard/paste-notification/index.jsx
@@ -47,6 +47,7 @@ const PastNotificationAlert = ({ staticFileNotices, courseId }) => {
           icon={WarningIcon}
           dismissible
           actions={[
+            // oxlint-disable-next-line react/jsx-key (Paragon <Alert> adds its own key for action buttons)
             <ActionButton
               courseId={courseId}
               title={intl.formatMessage(messages.hasConflictingErrorsButtonText)}
@@ -87,6 +88,7 @@ const PastNotificationAlert = ({ staticFileNotices, courseId }) => {
           icon={InfoIcon}
           dismissible
           actions={[
+            // oxlint-disable-next-line react/jsx-key (Paragon <Alert> adds its own key for action buttons)
             <ActionButton
               courseId={courseId}
               title={intl.formatMessage(messages.hasNewFilesButtonText)}
diff --git a/src/course-unit/unit-sidebar/AddSidebar.tsx b/src/course-unit/unit-sidebar/AddSidebar.tsx
index 37c8c7ec47..e85af37118 100644
--- a/src/course-unit/unit-sidebar/AddSidebar.tsx
+++ b/src/course-unit/unit-sidebar/AddSidebar.tsx
@@ -206,6 +206,7 @@ const AddNewContent = () => {
         <Stack gap={2}>
           {templatesByType.advanced?.templates.map((advancedTypeObj) => (
             <BlockCardButton
+              key={advancedTypeObj.category}
               blockType={advancedTypeObj.category}
               name={advancedTypeObj.displayName}
               onClick={() => handleSelection('advanced', advancedTypeObj.category)}
@@ -223,6 +224,7 @@ const AddNewContent = () => {
         {blockTypes.map((blockTypeObj) => (
           <BlockCardButton
             {...blockTypeObj}
+            key={blockTypeObj.blockType}
             templates={templatesByType[blockTypeObj.blockType].templates}
             onClick={() => handleSelection(blockTypeObj.blockType)}
             onClickTemplate={(boilerplateName: string) => handleSelection(blockTypeObj.blockType, boilerplateName)}
diff --git a/src/course-updates/CourseUpdates.tsx b/src/course-updates/CourseUpdates.tsx
index 10745fdd9a..d71b31aa86 100644
--- a/src/course-updates/CourseUpdates.tsx
+++ b/src/course-updates/CourseUpdates.tsx
@@ -173,6 +173,7 @@ const CourseUpdates = () => {
                           {courseUpdates.map((courseUpdate, index) => (
                             isInnerFormOpen(courseUpdate.id) ? (
                               <UpdateForm
+                                key={courseUpdate.id}
                                 close={closeUpdateForm}
                                 requestType={requestType}
                                 isInnerForm
@@ -182,6 +183,7 @@ const CourseUpdates = () => {
                               />
                             ) : (
                               <CourseUpdate
+                                key={courseUpdate.id}
                                 dateForUpdate={courseUpdate.date}
                                 contentForUpdate={courseUpdate.content}
                                 onEdit={() => handleOpenUpdateForm(REQUEST_TYPES.edit_update, courseUpdate)}
diff --git a/src/editors/EditorContainer.tsx b/src/editors/EditorContainer.tsx
index 0df4c816fa..6b7cd9af56 100644
--- a/src/editors/EditorContainer.tsx
+++ b/src/editors/EditorContainer.tsx
@@ -63,6 +63,7 @@ const EditorContainer: React.FC<Props> = ({
         description={intl.formatMessage(messages.libraryBlockEditWarningDescription)}
         actions={[
           <Button
+            key="edit-warning"
             destination={getLibraryBlockUrl()}
             target="_blank"
             rel="noopener noreferrer"
diff --git a/src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/settingsComponents/GroupFeedback/GroupFeedbackRow.jsx b/src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/settingsComponents/GroupFeedback/GroupFeedbackRow.jsx
index eb180ef3ab..3086f0ff16 100644
--- a/src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/settingsComponents/GroupFeedback/GroupFeedbackRow.jsx
+++ b/src/editors/containers/ProblemEditor/components/EditProblemView/SettingsWidget/settingsComponents/GroupFeedback/GroupFeedbackRow.jsx
@@ -40,6 +40,7 @@ const GroupFeedbackRow = ({
           {answers.map((letter) => (
             <Form.Checkbox
               className="mr-4 mt-1"
+              key={letter.id}
               value={letter.id}
               checked={value.answers.indexOf(letter.id)}
               isValid={value.answers.indexOf(letter.id) >= 0}
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseSelector.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseSelector.jsx
index 7e21f5df30..a8727554da 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseSelector.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/LicenseWidget/LicenseSelector.jsx
@@ -42,9 +42,9 @@ const LicenseSelector = ({
           onChange={(e) => onLicenseChange(e.target.value)}
         >
           {Object.entries(LicenseNames).map(([key, text]) => {
-            if (license === key) { return (<option value={LicenseTypes[key]} selected>{text}</option>); }
-            if (key === LicenseTypes.select) { return (<option hidden>{text}</option>); }
-            return (<option value={LicenseTypes[key]}>{text}</option>);
+            if (license === key) { return (<option key={key} value={LicenseTypes[key]} selected>{text}</option>); }
+            if (key === LicenseTypes.select) { return (<option key={key} hidden>{text}</option>); }
+            return (<option key={key} value={LicenseTypes[key]}>{text}</option>);
           })}
         </Form.Control>
         {level !== LicenseLevel.course ? (
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx
index ad441c562d..7765a2ad91 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx
@@ -95,12 +95,14 @@ const LanguageSelector = ({
         <Dropdown.Menu>
           {Object.entries(videoTranscriptLanguages).map(([lang, text]) => {
             if (language === lang) {
-              return (<Dropdown.Item>{text}<Icon className="text-primary-500" src={Check} /></Dropdown.Item>);
+              return <Dropdown.Item key={lang}>{text}<Icon className="text-primary-500" src={Check} /></Dropdown.Item>;
             }
             if (openLanguages.some(row => row.includes(lang))) {
-              return (<Dropdown.Item onClick={() => onLanguageChange({ newLang: lang })}>{text}</Dropdown.Item>);
+              return (
+                <Dropdown.Item key={lang} onClick={() => onLanguageChange({ newLang: lang })}>{text}</Dropdown.Item>
+              );
             }
-            return (<Dropdown.Item className="disabled">{text}</Dropdown.Item>);
+            return (<Dropdown.Item key={lang} className="disabled">{text}</Dropdown.Item>);
           })}
         </Dropdown.Menu>
       </Dropdown>
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx
index cf8765b593..929ef11b23 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/index.jsx
@@ -147,6 +147,7 @@ const TranscriptWidget = ({
           <Form.Group className="border-primary-100 border-bottom">
             {transcripts.map((language, index) => (
               <Transcript
+                key={language}
                 language={language}
                 transcriptUrl={selectedVideoTranscriptUrls[language]}
                 index={index}
diff --git a/src/files-and-videos/files-page/FileValidationModal.jsx b/src/files-and-videos/files-page/FileValidationModal.jsx
index 52439c907a..36381f21d6 100644
--- a/src/files-and-videos/files-page/FileValidationModal.jsx
+++ b/src/files-and-videos/files-page/FileValidationModal.jsx
@@ -41,7 +41,7 @@ const FileValidationModal = ({
       <ModalDialog.Body>
         <FormattedMessage {...messages.overwriteConfirmMessage} />
         <ul className="mt-2">
-          {Object.keys(duplicateFiles).map(file => <li>{file}</li>)}
+          {Object.keys(duplicateFiles).map(file => <li key={file}>{file}</li>)}
         </ul>
       </ModalDialog.Body>
       <ModalDialog.Footer>
diff --git a/src/files-and-videos/generic/table-components/sort-and-filter-modal/SortAndFilterModal.jsx b/src/files-and-videos/generic/table-components/sort-and-filter-modal/SortAndFilterModal.jsx
index d878093439..3c3e291cfb 100644
--- a/src/files-and-videos/generic/table-components/sort-and-filter-modal/SortAndFilterModal.jsx
+++ b/src/files-and-videos/generic/table-components/sort-and-filter-modal/SortAndFilterModal.jsx
@@ -147,7 +147,7 @@ const SortAndFilterModal = ({
             isInline
           >
             {filterOptions.map(({ name, value }) => (
-              <Form.Checkbox {...{ value, key: value }}>{name}</Form.Checkbox>
+              <Form.Checkbox key={value} value={value}>{name}</Form.Checkbox>
             ))}
           </Form.CheckboxSet>
         </Form.Group>
diff --git a/src/files-and-videos/videos-page/info-sidebar/TranscriptTab.jsx b/src/files-and-videos/videos-page/info-sidebar/TranscriptTab.jsx
index 7392932318..c46fe8a305 100644
--- a/src/files-and-videos/videos-page/info-sidebar/TranscriptTab.jsx
+++ b/src/files-and-videos/videos-page/info-sidebar/TranscriptTab.jsx
@@ -112,8 +112,10 @@ const TranscriptTab = ({
             ))}
           </ul>
         </ErrorAlert>
-        {previousSelection.map(transcript => (
+        {previousSelection.map((transcript, idx) => (
           <Transcript
+            // eslint-disable-next-line react/no-array-index-key
+            key={idx}
             {...{
               languages,
               transcript,
diff --git a/src/generic/agreement-gated-feature/AlertAgreementGatedFeature.tsx b/src/generic/agreement-gated-feature/AlertAgreementGatedFeature.tsx
index 6df1f30321..8733369513 100644
--- a/src/generic/agreement-gated-feature/AlertAgreementGatedFeature.tsx
+++ b/src/generic/agreement-gated-feature/AlertAgreementGatedFeature.tsx
@@ -1,4 +1,4 @@
-import { useIntl } from '@edx/frontend-platform/i18n';
+import { FormattedMessage } from '@edx/frontend-platform/i18n';
 import { Alert, Button, Hyperlink } from '@openedx/paragon';
 import { Policy } from '@openedx/paragon/icons';
 import { AgreementGated } from '@src/constants';
@@ -11,7 +11,6 @@ import {
 import messages from './messages';
 
 const AlertAgreement = ({ agreementType }: { agreementType: string }) => {
-  const intl = useIntl();
   const { data, isLoading, isError } = useUserAgreement(agreementType);
   const mutation = useUserAgreementRecordUpdater(agreementType);
   const showAlert = data && !isLoading && !isError;
@@ -30,8 +29,8 @@ const AlertAgreement = ({ agreementType }: { agreementType: string }) => {
       variant="warning"
       icon={Policy}
       actions={[
-        <Hyperlink destination={url}>{intl.formatMessage(messages.learnMoreLinkLabel)}</Hyperlink>,
-        <Button onClick={handleAcceptAgreement}>{intl.formatMessage(messages.agreeButtonLabel)}</Button>,
+        <Hyperlink key="learn-more" destination={url}><FormattedMessage {...messages.learnMoreLinkLabel} /></Hyperlink>,
+        <Button key="agree" onClick={handleAcceptAgreement}><FormattedMessage {...messages.agreeButtonLabel} /></Button>,
       ]}
     >
       <Alert.Heading>{name}</Alert.Heading>
diff --git a/src/generic/sidebar/BlockCardButton.tsx b/src/generic/sidebar/BlockCardButton.tsx
index 569fdf0ac9..bab788f83d 100644
--- a/src/generic/sidebar/BlockCardButton.tsx
+++ b/src/generic/sidebar/BlockCardButton.tsx
@@ -52,7 +52,7 @@ export const BlockCardButton = ({
         >
           <Stack direction="horizontal" className="d-flex flex-wrap" gap={2}>
             {templates.map((template) => (
-              <Chip onClick={() => onClickTemplate?.(template.boilerplateName)}>
+              <Chip onClick={() => onClickTemplate?.(template.boilerplateName)} key={template.boilerplateName}>
                 {template.displayName}
               </Chip>
             ))}
diff --git a/src/generic/sidebar/Sidebar.tsx b/src/generic/sidebar/Sidebar.tsx
index ebfd06aee4..02de034933 100644
--- a/src/generic/sidebar/Sidebar.tsx
+++ b/src/generic/sidebar/Sidebar.tsx
@@ -163,7 +163,7 @@ export function Sidebar<T extends SidebarPages>({
             }
 
             return (
-              <IconButton {...buttonData} />
+              <IconButton key={key} {...buttonData} />
             );
           })}
         </IconButtonToggle>
diff --git a/src/grading-settings/GradingSettings.jsx b/src/grading-settings/GradingSettings.jsx
index a359087db4..a1efc34a4f 100644
--- a/src/grading-settings/GradingSettings.jsx
+++ b/src/grading-settings/GradingSettings.jsx
@@ -259,7 +259,7 @@ const GradingSettings = () => {
           role="dialog"
           actions={[
             !isQueryPending && (
-              <Button variant="tertiary" onClick={handleResetPageData}>
+              <Button key="cancel" variant="tertiary" onClick={handleResetPageData}>
                 {intl.formatMessage(messages.buttonCancelText)}
               </Button>
             ),
diff --git a/src/group-configurations/index.jsx b/src/group-configurations/index.jsx
index 9738c53020..f3c0e621b7 100644
--- a/src/group-configurations/index.jsx
+++ b/src/group-configurations/index.jsx
@@ -90,6 +90,7 @@ const GroupConfigurations = () => {
               {!!teamGroups && teamGroups.length > 0 && (
                 teamGroups.map((teamGroup) => (
                   <TeamGroupsSection
+                    key={teamGroup.id}
                     availableGroup={teamGroup}
                   />
                 ))
diff --git a/src/library-authoring/LibraryContent.tsx b/src/library-authoring/LibraryContent.tsx
index 416d1a57b3..fa20658e16 100644
--- a/src/library-authoring/LibraryContent.tsx
+++ b/src/library-authoring/LibraryContent.tsx
@@ -110,6 +110,7 @@ const LibraryContent = ({ contentType = ContentType.home }: LibraryContentProps)
       })}
       {showPlaceholderBlocks && placeholderData?.hits?.map((item) => (
         <PlaceholderCard
+          key={item.usage_key}
           displayName={item.display_name}
           blockType={item.block_type}
         />
diff --git a/src/library-authoring/import-course/ImportDetailsPage.tsx b/src/library-authoring/import-course/ImportDetailsPage.tsx
index 763712cfb3..ee12ba9b99 100644
--- a/src/library-authoring/import-course/ImportDetailsPage.tsx
+++ b/src/library-authoring/import-course/ImportDetailsPage.tsx
@@ -234,6 +234,7 @@ const ImportDetailsContent = () => {
           stacked
           actions={[
             <Button
+              key="view-content"
               variant="outline-primary"
               iconAfter={ArrowForward}
               onClick={() => navigate(collectionLink())}
@@ -289,6 +290,7 @@ const ImportDetailsContent = () => {
           stacked
           actions={[
             <Button
+              key="retry-btn"
               variant="outline-primary"
               iconAfter={ArrowForward}
               onClick={handleImportCourse}
@@ -330,6 +332,7 @@ const ImportDetailsContent = () => {
           stacked
           actions={[
             <Button
+              key="view-content"
               variant="outline-primary"
               iconAfter={ArrowForward}
               onClick={() => navigate(collectionLink())}
diff --git a/src/optimizer-page/scan-results/ScanResults.tsx b/src/optimizer-page/scan-results/ScanResults.tsx
index 2ca8d1e537..ce17593ee4 100644
--- a/src/optimizer-page/scan-results/ScanResults.tsx
+++ b/src/optimizer-page/scan-results/ScanResults.tsx
@@ -1087,7 +1087,7 @@ const ScanResults: FC<Props> = ({
                 {section.subsections.map((subsection) => (
                   <>
                     {subsection.units.map((unit) => (
-                      <div className="unit">
+                      <div className="unit" key={unit.id}>
                         <BrokenLinkTable
                           unit={unit}
                           linkType="previous"
diff --git a/src/optimizer-page/scan-results/filterModal.jsx b/src/optimizer-page/scan-results/filterModal.jsx
index 16e90901b3..0a68ac80a6 100644
--- a/src/optimizer-page/scan-results/filterModal.jsx
+++ b/src/optimizer-page/scan-results/filterModal.jsx
@@ -50,7 +50,7 @@ const FilterModal = ({
             value={filterBy}
           >
             {filterOptions.map(({ name, value }) => (
-              <Form.Checkbox {...{ value, key: value }}>
+              <Form.Checkbox key={value} value={value}>
                 <span style={{ display: 'flex', gap: '90px' }}>
                   {name}
                   { value === 'brokenLinks' && <CustomIcon icon={LinkOff} message1={messages.brokenLabel} message2={messages.brokenInfoTooltip} placement="right-end" /> }
diff --git a/src/studio-home/tabs-section/courses-tab/index.tsx b/src/studio-home/tabs-section/courses-tab/index.tsx
index 337a0e8d30..310375cd22 100644
--- a/src/studio-home/tabs-section/courses-tab/index.tsx
+++ b/src/studio-home/tabs-section/courses-tab/index.tsx
@@ -88,6 +88,7 @@ const CardList = ({
               url,
             }) => (
               <CardItem
+                key={courseKey}
                 courseKey={courseKey}
                 onClick={() => onClickCard?.(courseKey)}
                 itemId={courseKey}
diff --git a/src/textbooks/textbook-form/TextbookForm.jsx b/src/textbooks/textbook-form/TextbookForm.jsx
index a667654fef..506064bbf9 100644
--- a/src/textbooks/textbook-form/TextbookForm.jsx
+++ b/src/textbooks/textbook-form/TextbookForm.jsx
@@ -81,7 +81,8 @@ const TextbookForm = ({
               render={(arrayHelpers) => (
                 <>
                   {!!values?.chapters.length && values.chapters.map(({ title, url }, index) => (
-                    <div className="form-chapters-fields" data-testid="form-chapters-fields">
+                    // eslint-disable-next-line react/no-array-index-key
+                    <div className="form-chapters-fields" data-testid="form-chapters-fields" key={index}>
                       <Form.Group size="sm" className="form-field">
                         <Form.Label size="sm" className="form-label font-weight-bold required text-black">
                           {intl.formatMessage(messages.chapterTitleLabel)} *

From d5fcf2bb4ee0f1d3617ef0203878d54d2bd2b7a3 Mon Sep 17 00:00:00 2001
From: Braden MacDonald <braden@opencraft.com>
Date: Tue, 31 Mar 2026 17:04:45 -0700
Subject: [PATCH 3/6] chore: other miscellaneous fixes

---
 src/course-unit/CourseUnit.tsx                    | 8 +++-----
 src/group-configurations/{index.jsx => index.tsx} | 9 +++++----
 src/optimizer-page/scan-results/ScanResults.tsx   | 4 ++--
 3 files changed, 10 insertions(+), 11 deletions(-)
 rename src/group-configurations/{index.jsx => index.tsx} (92%)

diff --git a/src/course-unit/CourseUnit.tsx b/src/course-unit/CourseUnit.tsx
index 192679ab3d..4c0c45c8b7 100644
--- a/src/course-unit/CourseUnit.tsx
+++ b/src/course-unit/CourseUnit.tsx
@@ -282,14 +282,12 @@ const CourseUnit = () => {
             </TransitionReplace>
             {courseUnit.upstreamInfo?.upstreamLink && (
               <AlertMessage
-                title={intl.formatMessage(
+                description={intl.formatMessage(
                   messages.alertLibraryUnitReadOnlyText,
                   {
                     link: (
-                      <Alert.Link
-                        href={courseUnit.upstreamInfo.upstreamLink}
-                      >
-                        {intl.formatMessage(messages.alertLibraryUnitReadOnlyLinkText)}
+                      <Alert.Link href={courseUnit.upstreamInfo.upstreamLink}>
+                        <FormattedMessage {...messages.alertLibraryUnitReadOnlyLinkText} />
                       </Alert.Link>
                     ),
                   },
diff --git a/src/group-configurations/index.jsx b/src/group-configurations/index.tsx
similarity index 92%
rename from src/group-configurations/index.jsx
rename to src/group-configurations/index.tsx
index f3c0e621b7..81af1fa357 100644
--- a/src/group-configurations/index.jsx
+++ b/src/group-configurations/index.tsx
@@ -17,6 +17,7 @@ import EnrollmentTrackGroupsSection from './enrollment-track-groups-section';
 import GroupConfigurationSidebar from './group-configuration-sidebar';
 import { useGroupConfigurations } from './hooks';
 import ConnectionErrorAlert from '../generic/ConnectionErrorAlert';
+import { AvailableGroup } from './types';
 
 const GroupConfigurations = () => {
   const { formatMessage } = useIntl();
@@ -39,7 +40,7 @@ const GroupConfigurations = () => {
   } = useGroupConfigurations(courseId);
 
   document.title = getPageHeadTitle(
-    courseDetails?.name,
+    courseDetails?.name ?? '',
     formatMessage(messages.headingTitle),
   );
 
@@ -59,13 +60,13 @@ const GroupConfigurations = () => {
     );
   }
 
-  const enrollmentTrackGroup = shouldShowEnrollmentTrack
+  const enrollmentTrackGroup: AvailableGroup = shouldShowEnrollmentTrack
     ? allGroupConfigurations.find((group) => group.scheme === 'enrollment_track')
     : null;
 
-  const contentGroup = allGroupConfigurations.find((group) => group.scheme === 'cohort');
+  const contentGroup: AvailableGroup[] = allGroupConfigurations.find((group) => group.scheme === 'cohort');
 
-  const teamGroups = allGroupConfigurations.filter((group) => group.scheme === 'team');
+  const teamGroups: AvailableGroup[] = allGroupConfigurations.filter((group) => group.scheme === 'team');
 
   return (
     <>
diff --git a/src/optimizer-page/scan-results/ScanResults.tsx b/src/optimizer-page/scan-results/ScanResults.tsx
index ce17593ee4..068019d86c 100644
--- a/src/optimizer-page/scan-results/ScanResults.tsx
+++ b/src/optimizer-page/scan-results/ScanResults.tsx
@@ -23,7 +23,7 @@ import { useDispatch } from 'react-redux';
 import messages from './messages';
 import SectionCollapsible from './SectionCollapsible';
 import BrokenLinkTable from './BrokenLinkTable';
-import { LinkCheckResult } from '../types';
+import type { LinkCheckResult, Section } from '../types';
 import { countBrokenLinks, isDataEmpty } from '../utils';
 import FilterModal from './filterModal';
 import { useWaffleFlags } from '../../data/apiHooks';
@@ -131,7 +131,7 @@ const ScanResults: FC<Props> = ({
   }, [data?.courseUpdates, data?.customPages, intl]);
 
   // Combine renderable sections with regular sections
-  const allSections = useMemo(
+  const allSections: Section[] = useMemo(
     () => [...renderableSections, ...(sections || [])],
     [renderableSections, sections],
   );

From bff903dc9471b7a2203ada526572bc94c478e4de Mon Sep 17 00:00:00 2001
From: Braden MacDonald <braden@opencraft.com>
Date: Tue, 31 Mar 2026 17:14:24 -0700
Subject: [PATCH 4/6] test: re-enable oxlint check for "no-unused-vars"

---
 .oxlintrc.json                                               | 5 ++++-
 .../outline-sidebar/info-sidebar/CourseInfoSidebar.tsx       | 1 -
 src/library-authoring/import-course/ImportDetailsPage.tsx    | 2 +-
 .../import-course/stepper/ImportStepperPage.tsx              | 2 +-
 src/taxonomy/tag-list/TagListTable.tsx                       | 2 --
 webpack.dev.config.js                                        | 2 +-
 webpack.prod.config.js                                       | 2 +-
 7 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/.oxlintrc.json b/.oxlintrc.json
index 827999be4a..7aa558b4d9 100644
--- a/.oxlintrc.json
+++ b/.oxlintrc.json
@@ -5,7 +5,10 @@
   },
   "plugins": ["react"],
   "rules": {
-    "eslint/no-unused-vars": "off",
+    "eslint/no-unused-vars": ["warn", {
+      // Allow using {ignoredProp, ...keepTheRest} to omit a prop like 'ignoredProp' from an object.
+      "ignoreRestSiblings": true,
+    }],
     // We disable exhaustive-deps because: it's noisy, and we often include extra deps when we want a memoized thing to
     // re-calculate after some change, even if we're not using that thing in the calculation.
     "react-hooks/exhaustive-deps": "off",
diff --git a/src/course-outline/outline-sidebar/info-sidebar/CourseInfoSidebar.tsx b/src/course-outline/outline-sidebar/info-sidebar/CourseInfoSidebar.tsx
index a7344952ea..26b008704c 100644
--- a/src/course-outline/outline-sidebar/info-sidebar/CourseInfoSidebar.tsx
+++ b/src/course-outline/outline-sidebar/info-sidebar/CourseInfoSidebar.tsx
@@ -1,4 +1,3 @@
-import { getConfig } from '@edx/frontend-platform';
 import { useIntl } from '@edx/frontend-platform/i18n';
 import {
   Tab,
diff --git a/src/library-authoring/import-course/ImportDetailsPage.tsx b/src/library-authoring/import-course/ImportDetailsPage.tsx
index ee12ba9b99..fdb8d1d1f9 100644
--- a/src/library-authoring/import-course/ImportDetailsPage.tsx
+++ b/src/library-authoring/import-course/ImportDetailsPage.tsx
@@ -208,7 +208,7 @@ const ImportDetailsContent = () => {
       });
       navigate(`../import/${courseImportDetails.source}/${newMigrationTask.uuid}`);
       setDisableReimport(false);
-    } catch (error) {
+    } catch {
       showToast(intl.formatMessage(messages.importCourseCompleteFailedToastMessage, {
         courseName: courseDetails.title,
       }));
diff --git a/src/library-authoring/import-course/stepper/ImportStepperPage.tsx b/src/library-authoring/import-course/stepper/ImportStepperPage.tsx
index cddf341685..79990d0377 100644
--- a/src/library-authoring/import-course/stepper/ImportStepperPage.tsx
+++ b/src/library-authoring/import-course/stepper/ImportStepperPage.tsx
@@ -85,7 +85,7 @@ export const ImportStepperPage = () => {
         compositionLevel: 'section',
       });
       navigate(`../import/${selectedCourseId}/${migrationTask.uuid}`);
-    } catch (error) {
+    } catch {
       showToast(intl.formatMessage(messages.importCourseCompleteFailedToastMessage, {
         courseName: courseData?.title,
       }));
diff --git a/src/taxonomy/tag-list/TagListTable.tsx b/src/taxonomy/tag-list/TagListTable.tsx
index 01795b8917..01bfdbfe11 100644
--- a/src/taxonomy/tag-list/TagListTable.tsx
+++ b/src/taxonomy/tag-list/TagListTable.tsx
@@ -3,7 +3,6 @@ import React, {
   useMemo,
   useEffect,
 } from 'react';
-import { useIntl } from '@edx/frontend-platform/i18n';
 import type { PaginationState } from '@tanstack/react-table';
 import { useTagListData, useCreateTag } from '../data/apiHooks';
 import { TagTree } from './tagTree';
@@ -40,7 +39,6 @@ const TagListTable = ({ taxonomyId, maxDepth }: TagListTableProps) => {
   // TODO: Simpler approaches have been suggested. Two options are to just use simple React state:
   // `isCurrentlyEditingTag` and `lastCreatedTag`, or to use optimistic updates.
   // For reference, see https://github.com/openedx/frontend-app-authoring/pull/2872#discussion_r2880965005.
-  const intl = useIntl();
 
   const [creatingParentId, setCreatingParentId] = useState<RowId | null>(null);
   const [editingRowId, setEditingRowId] = useState<RowId | null>(null);
diff --git a/webpack.dev.config.js b/webpack.dev.config.js
index 34cc439e2a..199e00beb8 100644
--- a/webpack.dev.config.js
+++ b/webpack.dev.config.js
@@ -24,7 +24,7 @@ const config = createConfig('webpack-dev', {
         try {
           // Try to resolve the real package. If it exists, do nothing.
           require.resolve('@edx/frontend-plugin-notifications');
-        } catch (e) {
+        } catch {
           // Package not found → point to the stub we created.
           // eslint-disable-next-line no-param-reassign
           resource.request = path.resolve(__dirname, 'src/stubs/empty-notifications-plugin.tsx');
diff --git a/webpack.prod.config.js b/webpack.prod.config.js
index f1fec7d6b8..51faf70aed 100644
--- a/webpack.prod.config.js
+++ b/webpack.prod.config.js
@@ -24,7 +24,7 @@ const config = createConfig('webpack-prod', {
         try {
           // Try to resolve the real package. If it exists, do nothing.
           require.resolve('@edx/frontend-plugin-notifications');
-        } catch (e) {
+        } catch {
           // Package not found → point to the stub we created.
           // eslint-disable-next-line no-param-reassign
           resource.request = path.resolve(__dirname, 'src/stubs/empty-notifications-plugin.tsx');

From 82651c6ea95df1ddffdd3ea086bb11eaaf5e189c Mon Sep 17 00:00:00 2001
From: Braden MacDonald <braden@opencraft.com>
Date: Mon, 13 Apr 2026 15:15:54 -0700
Subject: [PATCH 5/6] chore: remove unused imports

---
 src/course-outline/CourseOutline.tsx             | 3 ---
 src/course-outline/CourseOutlineContext.tsx      | 2 +-
 src/course-outline/hooks.jsx                     | 2 --
 src/library-authoring/history-log/HistoryLog.tsx | 9 ---------
 4 files changed, 1 insertion(+), 15 deletions(-)
 delete mode 100644 src/library-authoring/history-log/HistoryLog.tsx

diff --git a/src/course-outline/CourseOutline.tsx b/src/course-outline/CourseOutline.tsx
index b9fb53a692..c67a5671e1 100644
--- a/src/course-outline/CourseOutline.tsx
+++ b/src/course-outline/CourseOutline.tsx
@@ -28,7 +28,6 @@ import { UnlinkModal } from '@src/generic/unlink-modal';
 import AlertMessage from '@src/generic/alert-message';
 import getPageHeadTitle from '@src/generic/utils';
 import CourseOutlineHeaderActionsSlot from '@src/plugin-slots/CourseOutlineHeaderActionsSlot';
-import { XBlock } from '@src/data/types';
 import { useCourseAuthoringContext } from '@src/CourseAuthoringContext';
 import { useCourseOutlineContext } from './CourseOutlineContext';
 import LegacyLibContentBlockAlert from '@src/course-libraries/LegacyLibContentBlockAlert';
@@ -72,8 +71,6 @@ const CourseOutline = () => {
     closeUnlinkModal,
   } = useCourseAuthoringContext();
   const {
-    handleAddBlock,
-    handleAddAndOpenUnit,
     currentSelection,
     sections,
     restoreSectionList,
diff --git a/src/course-outline/CourseOutlineContext.tsx b/src/course-outline/CourseOutlineContext.tsx
index 65098cf5fb..38c8c4824f 100644
--- a/src/course-outline/CourseOutlineContext.tsx
+++ b/src/course-outline/CourseOutlineContext.tsx
@@ -5,7 +5,7 @@ import { useDispatch, useSelector } from 'react-redux';
 import { useToggle } from '@openedx/paragon';
 import { arrayMove } from '@dnd-kit/sortable';
 
-import { SelectionState, type UnitXBlock, type XBlock } from '@src/data/types';
+import { SelectionState, type XBlock } from '@src/data/types';
 import { useToggleWithValue } from '@src/hooks';
 import { getBlockType } from '@src/generic/key-utils';
 import { COURSE_BLOCK_NAMES } from '@src/constants';
diff --git a/src/course-outline/hooks.jsx b/src/course-outline/hooks.jsx
index 2804b9b1b3..ab2d1e07c6 100644
--- a/src/course-outline/hooks.jsx
+++ b/src/course-outline/hooks.jsx
@@ -38,7 +38,6 @@ import {
   enableCourseHighlightsEmailsQuery,
   fetchCourseBestPracticesQuery,
   fetchCourseLaunchQuery,
-  fetchCourseOutlineIndexQuery,
   fetchCourseReindexQuery,
   setVideoSharingOptionQuery,
   dismissNotificationQuery,
@@ -52,7 +51,6 @@ const useCourseOutline = ({ courseId }) => {
     handleAddBlock,
     setCurrentSelection,
     currentSelection,
-    isDuplicatingItem,
     isDeleteModalOpen,
     openDeleteModal,
     closeDeleteModal,
diff --git a/src/library-authoring/history-log/HistoryLog.tsx b/src/library-authoring/history-log/HistoryLog.tsx
deleted file mode 100644
index 54260dc71c..0000000000
--- a/src/library-authoring/history-log/HistoryLog.tsx
+++ /dev/null
@@ -1,9 +0,0 @@
-import { Stack } from "@openedx/paragon";
-
-const HistoryLog = () => {
-  return (
-    <Stack>
-      History
-    </Stack>
-  )
-};

From 4cd8eabf54fa345ef087cd9a6aecbd3c7200ebc7 Mon Sep 17 00:00:00 2001
From: Braden MacDonald <braden@opencraft.com>
Date: Mon, 13 Apr 2026 15:24:47 -0700
Subject: [PATCH 6/6] test: improve test coverage slightly

---
 .../TranscriptWidget/LanguageSelector.jsx     | 19 ++-------
 .../LanguageSelector.test.jsx                 | 40 ++++++++++++++++++-
 2 files changed, 42 insertions(+), 17 deletions(-)

diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx
index 7765a2ad91..058a74a445 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.jsx
@@ -9,7 +9,7 @@ import {
 } from '@openedx/paragon';
 
 import { Check } from '@openedx/paragon/icons';
-import { connect, useDispatch } from 'react-redux';
+import { useDispatch, useSelector } from 'react-redux';
 import { useIntl } from '@edx/frontend-platform/i18n';
 import { thunkActions, selectors } from '../../../../../../data/redux';
 import { videoTranscriptLanguages } from '../../../../../../data/constants/video';
@@ -47,9 +47,8 @@ export const hooks = {
 const LanguageSelector = ({
   index, // For a unique id for the form control
   language,
-  // Redux
-  openLanguages, // Only allow those languages not already associated with a transcript to be selected
 }) => {
+  const openLanguages = useSelector(selectors.video.openLanguages);
   const intl = useIntl();
   const [localLang, setLocalLang] = React.useState(language);
   const input = fileInput({ onAddFile: hooks.addFileCallback({ dispatch: useDispatch(), localLang }) });
@@ -111,21 +110,9 @@ const LanguageSelector = ({
   );
 };
 
-LanguageSelector.defaultProps = {
-  openLanguages: [],
-};
-
 LanguageSelector.propTypes = {
-  openLanguages: PropTypes.arrayOf(PropTypes.string),
   index: PropTypes.number.isRequired,
   language: PropTypes.string.isRequired,
 };
 
-export const mapStateToProps = (state) => ({
-  openLanguages: selectors.video.openLanguages(state),
-});
-
-export const mapDispatchToProps = {};
-
-export const LanguageSelectorInternal = LanguageSelector; // For testing only
-export default connect(mapStateToProps, mapDispatchToProps)(LanguageSelector);
+export default LanguageSelector;
diff --git a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.test.jsx b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.test.jsx
index 3fc95017b7..360455f737 100644
--- a/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.test.jsx
+++ b/src/editors/containers/VideoEditor/components/VideoSettingsModal/components/TranscriptWidget/LanguageSelector.test.jsx
@@ -2,8 +2,9 @@ import React from 'react';
 import {
   render, screen, initializeMocks, fireEvent,
 } from '@src/testUtils';
+import { useDispatch } from 'react-redux';
 import LanguageSelector from './LanguageSelector';
-import { selectors } from '../../../../../../data/redux';
+import { thunkActions, selectors } from '../../../../../../data/redux';
 
 const lang1 = 'kLinGon';
 const lang1Code = 'kl';
@@ -12,6 +13,14 @@ const lang2Code = 'el';
 const lang3 = 'sImLisH';
 const lang3Code = 'sl';
 
+jest.mock('react-redux', () => {
+  const dispatchFn = jest.fn().mockName('mockDispatch');
+  return {
+    ...jest.requireActual('react-redux'),
+    useDispatch: jest.fn(() => dispatchFn),
+  };
+});
+
 jest.mock('../../../../../../data/constants/video', () => ({
   videoTranscriptLanguages: {
     [lang1Code]: lang1,
@@ -20,6 +29,20 @@ jest.mock('../../../../../../data/constants/video', () => ({
   },
 }));
 
+jest.mock('../../../../../../data/redux', () => ({
+  thunkActions: {
+    video: {
+      updateTranscriptLanguage: jest.fn((args) => ({ updateTranscriptLanguage: args })).mockName('thunkActions.video.updateTranscriptLanguage'),
+      uploadTranscript: jest.fn().mockName('thunkActions.video.uploadTranscript'),
+    },
+  },
+  selectors: {
+    video: {
+      openLanguages: jest.fn(),
+    },
+  },
+}));
+
 describe('LanguageSelector', () => {
   const props = {
     onSelect: jest.fn().mockName('props.OnSelect'),
@@ -55,4 +78,19 @@ describe('LanguageSelector', () => {
     const disabledItems = container.querySelectorAll('.disabled.dropdown-item');
     expect(disabledItems.length).toBe(3);
   });
+
+  test('clicking an open language item dispatches updateTranscriptLanguage', () => {
+    const mockDispatch = jest.fn();
+    useDispatch.mockReturnValue(mockDispatch);
+    const { video } = selectors;
+    jest.spyOn(video, 'openLanguages').mockReturnValue([[lang2Code, lang2], [lang3Code, lang3]]);
+    render(<LanguageSelector index={1} language={lang1Code} />);
+    fireEvent.click(screen.getByRole('button', { name: 'Languages' }));
+    fireEvent.click(screen.getByText(lang2));
+    expect(thunkActions.video.updateTranscriptLanguage).toHaveBeenCalledWith({
+      newLanguageCode: lang2Code,
+      languageBeforeChange: lang1Code,
+    });
+    expect(mockDispatch).toHaveBeenCalled();
+  });
 });