chore: docs, formatting, linting, and minor changes

Author: usefulthink

.editorconfig

@@ -0,0 +1,8 @@
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+insert_final_newline = true

.prettierrc.js

@@ -0,0 +1,6 @@
+export default {
+  bracketSpacing: false,
+  singleQuote: true,
+  trailingComma: "all",
+  arrowParens: "avoid",
+};

MIGRATION.md

@@ -10,11 +10,15 @@ functional approach. This simplifies the API and aligns it with
 [the recommended way](https://developers.google.com/maps/documentation/javascript/load-maps-js-api)
 to load the Google Maps JavaScript API.
 
-- **v1.x:** You would create an instance of the `Loader` class with your configuration and then call methods like `load()` or `importLibrary()` on that instance.
-- **v2.x:** You now use two standalone functions: `setOptions()` to configure the API loader and `importLibrary()` to load specific libraries.
+- **v1.x:** You would create an instance of the `Loader` class with your 
+  configuration and then call methods like `load()` or `importLibrary()` on 
+  that instance.
+- **v2.x:** You now use two standalone functions: `setOptions()` to 
+  configure the API loader and `importLibrary()` to load specific libraries.
 
 Generally, the Loader constructor can be replaced with a call to `setOptions()`,
-and the different methods used to actually load the API
+and the different methods used to actually load the API are all replaced
+with calls to `importLibrary()`.
 
 ## Key Changes
 
@@ -49,7 +53,7 @@ loader.load().then(() => initMap());
 await loader.load();
 initMap();
 
-// c) using callback – same as a)
+// c) using callback
 loader.loadCallback(() => {
   initMap();
 });
@@ -65,7 +69,7 @@ function initMap() {
 
 ### v2.x
 
-The typical use case from versions 2.0 onwards is to use importLibrary to get
+The typical use case from versions 2.0 onwards is to use `importLibrary` to get
 access to the classes and features needed from the Maps JavaScript API.
 
 ```javascript
@@ -128,4 +132,5 @@ function initMap() {
   });
 }
 ```
+
 </details>

README.md

@@ -73,7 +73,7 @@ importLibrary("maps").then(({ Map }) => {
 
 If you use custom HTML elements from the Google Maps JavaScript API (e.g.
 `<gmp-map>`, and `<gmp-advanced-marker>`), you need to import them as well.
-Note that you do not need to await the result of `importLibrary` in this case.
+Note that you do not need to await the result of `importLibrary()` in this case.
 The custom elements will upgrade automatically once the library is loaded
 and you can use the `whenDefined()` method to wait for the upgrade to
 complete.
@@ -85,11 +85,11 @@ import { setOptions, importLibrary } from "@googlemaps/js-api-loader";
 setOptions({ key: "your-api-key-here" });
 
 // Start loading the libraries needed for custom elements.
-importLibrary("maps"); // needed for gmp-map
-importLibrary("marker"); // needed for gmp-advanced-marker
+importLibrary("maps"); // needed for <gmp-map>
+importLibrary("marker"); // needed for <gmp-advanced-marker>
 
 // Wait for gmp-map to be upgraded and interact with it.
-await customElements.whenDefined('gmp-map');
+await customElements.whenDefined("gmp-map");
 const map = document.querySelector("gmp-map");
 // ...
 ```
@@ -109,8 +109,8 @@ await importLibrary("core");
 ### `setOptions(options: APIOptions): void`
 
 Sets the options for loading the Google Maps JavaScript API and installs the
-global `google.maps.importLibrary` function that is used by the importLibrary
-function.
+global `google.maps.importLibrary()` function that is used by the
+`importLibrary()` function of this package.
 
 This function should be called as early as possible in your application and
 should only be called once. Any further calls will not have any effect and
@@ -123,7 +123,7 @@ Below is a short summary of the accepted options, see the
 - `v: string`: The version of the Maps JavaScript API to load.
 - `language: string`: The language to use.
 - `region: string`: The region code to use.
-- `libraries: string[]`: additional libraries to load.
+- `libraries: string[]`: Additional libraries to load.
 - `authReferrerPolicy: string`: Set the referrer policy for the API requests.
 - `mapIds: string[]`: An array of map IDs to preload.
 - `channel: string`: Can be used to track your usage.

eslint.config.js

@@ -2,10 +2,11 @@ import globals from "globals";
 import tseslint from "typescript-eslint";
 import pluginJest from "eslint-plugin-jest";
 import eslintConfigPrettier from "eslint-config-prettier";
+import { defineConfig } from "eslint/config";
 
-export default tseslint.config(
+export default defineConfig(
   {
-    ignores: ["dist/", "node_modules/", "docs/", "src/bootstrap.js"],
+    ignores: ["dist/", "node_modules/", "src/bootstrap.js"],
   },
   {
     languageOptions: {
@@ -19,10 +20,6 @@ export default tseslint.config(
         sourceType: "module",
       },
     },
-    rules: {
-      "no-var": "error",
-      "prefer-arrow-callback": "error",
-    },
   },
   ...tseslint.configs.recommended,
   {
@@ -47,11 +44,15 @@ export default tseslint.config(
     },
   },
   {
-    ...pluginJest.configs["flat/recommended"],
     files: ["**/*.test.ts"],
-    rules: {
-      ...pluginJest.configs["flat/recommended"].rules,
-    },
+    ...pluginJest.configs["flat/recommended"],
   },
   eslintConfigPrettier,
-);
+  {
+    rules: {
+      "no-var": "error",
+      "prefer-arrow-callback": "error",
+      curly: "error",
+    },
+  }
+);

package-lock.json

@@ -55,6 +55,7 @@
     "eslint-plugin-jest": "^29.0.1",
     "eslint-plugin-prettier": "^5.5.4",
     "fast-deep-equal": "^3.1.3",
+    "globals": "^16.4.0",
     "jest": "^30.1.3",
     "jest-environment-jsdom": "^30.1.2",
     "prettier": "^3.0.3",

package.json

@@ -2,7 +2,7 @@
  * Copyright 2025 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-/* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars, @typescript-eslint/no-empty-function */
+/* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars */
 
 import { MSG_DEPRECATED_LOADER } from "./messages.js";
 

src/deprecated.ts

@@ -63,7 +63,6 @@ type APILibraryName = keyof APILibraryMap;
 declare const __DEV__: boolean;
 
 let setOptionsWasCalled_ = false;
-let options_: APIOptions = {};
 
 /**
  * Sets the options for the Maps JavaScript API.
@@ -82,9 +81,7 @@ export function setOptions(options: APIOptions) {
     return;
   }
 
-  options_ = options;
-
-  installImportLibrary_(options_);
+  installImportLibrary_(options);
   setOptionsWasCalled_ = true;
 }
 
@@ -112,8 +109,9 @@ export async function importLibrary(libraryName: string): Promise<unknown> {
     logDevWarning(MSG_SET_OPTIONS_NOT_CALLED);
   }
 
-  if (!window?.google?.maps?.importLibrary)
+  if (!window?.google?.maps?.importLibrary) {
     throw new Error("google.maps.importLibrary is not installed.");
+  }
 
   return (await google.maps.importLibrary(
     libraryName
@@ -140,7 +138,9 @@ function installImportLibrary_(options: APIOptions) {
 
   // If the google.maps.importLibrary function already exists, bootstrap()
   // won't do anything, so we won't call it
-  if (!importLibraryExists) bootstrap(options);
+  if (!importLibraryExists) {
+    bootstrap(options);
+  }
 }
 
 // export the deprecated (and non-functional) Loader class to trigger a strong

src/index.ts

@@ -27,7 +27,7 @@ export const MSG_REPEATED_SET_OPTIONS = (options: APIOptions) =>
   `to the additional call (${JSON.stringify(options)}) will be ignored.`;
 
 export const MSG_IMPORT_LIBRARY_EXISTS = (options: APIOptions) =>
-  `The google.maps.importLibrary function is already defined, and ` +
+  `The google.maps.importLibrary() function is already defined, and ` +
   `@googlemaps/js-api-loader will use the existing function instead of ` +
   `overwriting it. The options passed to setOptions ` +
   `(${JSON.stringify(options)}) will be ignored.`;