fix: sonar maintain linting #3

apps/nextjs/src/components/AppComponents/QuizDesigner/QuizContent.tsx

@@ -41,7 +41,7 @@ const QuizContent = ({
     potentialNewQuestions,
     isLoading: suggestedQuestionsLoading,
     hasError: suggestedQuestionsHasError,
-    setPotentialNewQuestion,
+    setPotentialNewQuestions,
   } = useSuggestedQuestions({
     state,
     dispatch,
@@ -85,7 +85,7 @@ const QuizContent = ({
           potentialNewQuestions={potentialNewQuestions}
           dispatch={dispatch}
           suggestedQuestionsGeneration={suggestedQuestionsGeneration}
-          setPotentialNewQuestions={setPotentialNewQuestion}
+          setPotentialNewQuestions={setPotentialNewQuestions}
           questionsWrapperRef={questionsWrapperRef}
           questionRefs={questionRefs}
         />

apps/nextjs/src/components/AppComponents/QuizDesigner/QuizDesignerPageContent.tsx

@@ -47,7 +47,7 @@ const QuizDesignerPageContent = ({
     potentialNewQuestions,
     isLoading: suggestedQuestionsLoading,
     hasError: suggestedQuestionsHasError,
-    setPotentialNewQuestion,
+    setPotentialNewQuestions,
   } = useSuggestedQuestions({
     state,
     dispatch,
@@ -101,7 +101,7 @@ const QuizDesignerPageContent = ({
               suggestedQuestionsHasError={suggestedQuestionsHasError}
               potentialNewQuestions={potentialNewQuestions}
               dispatch={dispatch}
-              setPotentialNewQuestions={setPotentialNewQuestion}
+              setPotentialNewQuestions={setPotentialNewQuestions}
               suggestedQuestionsGeneration={suggestedQuestionsGeneration}
               questionRefs={questionRefs}
               questionsWrapperRef={questionsWrapperRef}

apps/nextjs/src/hooks/useSuggestedQuestions.ts

@@ -31,15 +31,15 @@ const useSuggestedQuestions = ({
   state,
   dispatch,
 }: UseSuggestedQuestionsProps) => {
-  const [potentialNewQuestions, setPotentialNewQuestion] =
+  const [potentialNewQuestions, setPotentialNewQuestions] =
     useState<PotentialQuestionsType>([]);
 
   useEffect(() => {
     if (
       state.status === QuizAppStatus.EditingSubjectAndKS ||
       state.status === QuizAppStatus.Initial
     ) {
-      setPotentialNewQuestion([]);
+      setPotentialNewQuestions([]);
     }
   }, [state]);
 
@@ -60,7 +60,7 @@ const useSuggestedQuestions = ({
         });
       },
       onSuccess: ({ data }) => {
-        setPotentialNewQuestion(data.response);
+        setPotentialNewQuestions(data.response);
       },
     },
   );
@@ -107,7 +107,7 @@ const useSuggestedQuestions = ({
     hasError,
     potentialNewQuestions,
     medianTimeTakenForPrompt,
-    setPotentialNewQuestion,
+    setPotentialNewQuestions,
   };
 };
 

apps/nextjs/src/hooks/useTemporaryLessonPlanWithStreamingEdits.ts

@@ -106,7 +106,7 @@ export const useTemporaryLessonPlanWithStreamingEdits = ({
   // );
 
   return useMemo(() => {
-    if (!throttledAssistantMessages || !throttledAssistantMessages.length) {
+    if (!throttledAssistantMessages?.length) {
       return {
         tempLessonPlan: tempLessonPlanRef.current,
         partialPatches: [],

apps/nextjs/src/lib/hooks/use-copy-to-clipboard.tsx

@@ -2,13 +2,13 @@
 
 import * as React from "react";
 
-interface useCopyToClipboardProps {
+interface UseCopyToClipboardProps {
   timeout?: number;
 }
 
 export function useCopyToClipboard({
   timeout = 2000,
-}: useCopyToClipboardProps) {
+}: UseCopyToClipboardProps) {
   const [isCopied, setIsCopied] = React.useState<boolean>(false);
 
   const copyToClipboard = (value: string) => {
@@ -20,7 +20,7 @@ export function useCopyToClipboard({
       return;
     }
 
-    navigator.clipboard.writeText(value).then(() => {
+    void navigator.clipboard.writeText(value).then(() => {
       setIsCopied(true);
 
       setTimeout(() => {

apps/nextjs/tests-e2e/tests/aila-chat/helpers.ts

@@ -67,7 +67,7 @@ export async function getSectionsComplete(page: Page): Promise<number> {
   const progressText = await page.getByTestId("chat-progress").textContent();
   const match = (progressText ?? "").match(/(\d+) of 10 sections complete/);
 
-  if (match && match[1]) {
+  if (match?.[1]) {
     return parseInt(match[1], 10);
   } else {
     return 0;

packages/api/src/router/testSupport/prepareUser.ts

@@ -120,7 +120,7 @@ const deleteOldTestUser = async () => {
     return;
   }
 
-  const users = testUsers.sort(
+  const users = testUsers.toSorted(
     (a, b) =>
       new Date(a.lastActiveAt ?? a.createdAt).getTime() -
       new Date(b.lastActiveAt ?? b.createdAt).getTime(),
@@ -143,7 +143,7 @@ const deleteOldTestUser = async () => {
         log.info(
           `${userToDelete.primaryEmailAddress?.emailAddress} already deleted, retrying`,
         );
-        deleteOldTestUser();
+        await deleteOldTestUser();
       } else {
         throw e;
       }

packages/core/src/models/lessonPlans.ts

@@ -137,9 +137,7 @@ export class LessonPlans {
       lessonPlanJsonSchema: JSON.stringify(LessonPlanJsonSchema),
       llmResponseJsonSchema: JSON.stringify(LLMResponseJsonSchema),
       isUsingStructuredOutput:
-        process.env.NEXT_PUBLIC_STRUCTURED_OUTPUTS_ENABLED === "true"
-          ? true
-          : false,
+        process.env.NEXT_PUBLIC_STRUCTURED_OUTPUTS_ENABLED === "true",
     });
 
     const systemPrompt = compiledTemplate;

packages/core/src/models/lessons.ts

@@ -305,7 +305,7 @@ export class Lessons {
     lessonId,
     quizQuestionId,
   }: {
-    question: { answer?: string | string[] | null | undefined };
+    question: { answer?: string | string[] | null };
     answer: string;
     lessonId: string;
     quizQuestionId: string;

packages/core/src/prompts/lesson-assistant/variants.ts

@@ -1,6 +1,6 @@
 import z from "zod";
 
-import type { TemplateProps} from ".";
+import type { TemplateProps } from ".";
 import { getPromptParts } from ".";
 import type { OakPromptDefinition, OakPromptVariant } from "../types";
 
@@ -59,9 +59,7 @@ export const generateVariants = (): OakPromptVariant[] => {
         llmResponseJsonSchema: "<llmResponseJsonSchema>",
         lessonPlan: {},
         isUsingStructuredOutput:
-          process.env.NEXT_PUBLIC_STRUCTURED_OUTPUTS_ENABLED === "true"
-            ? true
-            : false,
+          process.env.NEXT_PUBLIC_STRUCTURED_OUTPUTS_ENABLED === "true",
       },
       slug,
     );

packages/ingest/src/embedding/handleEmbeddingBatchSuccess.ts

@@ -72,6 +72,9 @@ export async function handleEmbeddingBatchSuccess({
       continue;
     }
 
+    if (errors.length > 0) {
+      log.info(`Errors: ${errors.length}`);
+    }
     lessonIdsCompleted.add(lessonId);
   }