UI: Remove unnecessary useCallback in useMonacoTheme

Address review feedback: React Compiler handles memoization,
so the useCallback wrapper around defineAirflowMonacoThemes
is redundant. Pass the function reference directly instead.
Also fix prettier formatting in tests.

Author: shivaam

airflow-core/src/airflow/ui/src/context/colorMode/useMonacoTheme.test.ts

@@ -99,10 +99,7 @@ describe("useMonacoTheme", () => {
 
     expect(defineTheme).toHaveBeenCalledTimes(2);
     expect(defineTheme).toHaveBeenCalledWith("airflow-light", expect.objectContaining({ base: "vs" }));
-    expect(defineTheme).toHaveBeenCalledWith(
-      "airflow-dark",
-      expect.objectContaining({ base: "vs-dark" }),
-    );
+    expect(defineTheme).toHaveBeenCalledWith("airflow-dark", expect.objectContaining({ base: "vs-dark" }));
   });
 
   it("does not re-register themes on subsequent beforeMount calls", async () => {

airflow-core/src/airflow/ui/src/context/colorMode/useMonacoTheme.ts

@@ -17,7 +17,6 @@
  * under the License.
  */
 import type { Monaco } from "@monaco-editor/react";
-import { useCallback } from "react";
 
 import { useColorMode } from "./useColorMode";
 
@@ -115,12 +114,8 @@ const defineAirflowMonacoThemes = (monaco: Monaco) => {
 export const useMonacoTheme = () => {
   const { colorMode } = useColorMode();
 
-  const beforeMount = useCallback((monaco: Monaco) => {
-    defineAirflowMonacoThemes(monaco);
-  }, []);
-
   return {
-    beforeMount,
+    beforeMount: defineAirflowMonacoThemes,
     theme: colorMode === "dark" ? DARK_THEME_NAME : LIGHT_THEME_NAME,
   };
 };