mongodb-atlas-api-client 4.10.0 → 4.13.0

package/.github/copilot-instructions.md

@@ -0,0 +1,239 @@
+# MongoDB Atlas API Client - Copilot Instructions
+
+## Project Overview
+
+This is a MongoDB Atlas API client library for Node.js that provides a programmatic interface to the MongoDB Atlas REST API. The library uses HTTP Digest Authentication and wraps the urllib HTTP client library.
+
+## Architecture and Code Organization
+
+### Module Structure
+- Each API resource (User, Cluster, Project, Organization, etc.) is implemented as a separate class in `src/`
+- Each class follows a consistent pattern:
+  - Constructor receives: `client`, `baseUrl`, and optionally `projectId`
+  - Methods return promises
+  - All API methods accept an optional `options` parameter
+- The main entry point (`src/index.js`) exports a factory function that creates instances of all API resource classes
+- TypeScript definitions (`.d.ts` files) are provided alongside each JavaScript file
+
+### Key Components
+- `httpClient.js`: HTTP client wrapper that handles digest authentication
+- `helper.js`: Utility functions for query string generation
+- `index.js`: Main entry point that instantiates and exports all API resource clients
+
+## Code Patterns and Conventions
+
+### API Method Pattern
+```javascript
+async methodName(param1, param2, options = {}) {
+  const queryString = getQueryStringFromOptions(options);
+  const httpOptions = options.httpOptions;
+  const response = await this.client_.fetch(`${this.baseUrl_}/path/${param1}?${queryString}`, {
+    "method": "METHOD",
+    "data": body,
+    ...httpOptions
+  });
+  return response;
+}
+```
+
+### Class Pattern
+- Private properties use trailing underscore: `this.client_`, `this.baseUrl_`, `this.projectId_`
+- All methods are async and return promises
+- Methods should handle `options` parameter for query strings and HTTP options
+- The `httpOptions` parameter is extracted from options and passed separately (not included in query string)
+
+### Naming Conventions
+- Use camelCase for variables and function names
+- Use PascalCase for class names
+- Use descriptive names (e.g., `getAll`, `getById`, `delete`, `create`, `update`)
+- Private instance variables end with underscore: `this.client_`, `this.baseUrl_`
+
+## Testing Requirements
+
+### Test Framework
+- Uses `@hapi/lab` for test execution
+- Uses `@hapi/code` for assertions
+- Uses `sinon` for stubbing/mocking when needed
+- Uses `urllib`'s MockAgent for HTTP mocking
+- **100% code coverage is required** - all tests must pass with full coverage
+
+### Test Structure Pattern
+```javascript
+const {describe, it, afterEach, before, beforeEach} = exports.lab = require("@hapi/lab").script();
+const {expect} = require('@hapi/code');
+const getClient = require('../src/index.js');
+const {MockAgent, setGlobalDispatcher} = require('urllib');
+
+const baseUrl = "http://localhost:7001";
+const projectId = "dummyProjectId";
+
+const client = getClient({
+  "publicKey": "dummyPublicKey",
+  "privateKey": "dummyPrivateKey",
+  "baseUrl": baseUrl,
+  "projectId": projectId
+});
+
+describe("Mongo Atlas Api Client - ResourceName", () => {
+  let mockAgent;
+  let mockPool;
+  
+  before(() => {
+    mockAgent = new MockAgent();
+    setGlobalDispatcher(mockAgent);
+  });
+
+  beforeEach(() => {
+    mockPool = mockAgent.get(baseUrl);
+  });
+
+  afterEach(() => {
+    mockAgent.assertNoPendingInterceptors();
+  });
+
+  describe("When method is called", () => {
+    it("should return expected result", async () => {
+      mockPool.intercept({
+        "path": `/expected/path`,
+        "method": "get"
+      }).reply(200, {"response": "data"});
+      
+      const result = await client.resource.method();
+      expect(result).to.equal({"response": "data"});
+    });
+  });
+});
+```
+
+### Test Guidelines
+- Test file names match source files: `user.js` → `user.test.js`
+- Each resource should have tests for:
+  - Verification that methods are exported from index
+  - Each public method with various parameter combinations
+  - Query string handling
+  - HTTP options handling
+- Use MockAgent to intercept HTTP requests
+- Always call `mockAgent.assertNoPendingInterceptors()` in afterEach
+
+## Linting and Code Style
+
+### ESLint Configuration
+- Uses ESLint 9 with `@eslint/js` recommended config
+- ES2015+ (latest) JavaScript syntax
+- CommonJS module system (not ES modules)
+- Strict rules including:
+  - No console logs (`no-console: error`)
+  - No var declarations (`no-var: error`)
+  - Prefer const (`prefer-const: error`)
+  - Prefer template literals (`prefer-template: error`)
+  - Complexity limit of 9 (`complexity: [warn, 9]`)
+  - One variable declaration per statement (`one-var: [warn, never]`)
+  - Variable name length: 2-70 characters (with exceptions for i, j, k, n, Q, _)
+
+### Code Style Guidelines
+- Use double quotes for strings (no enforcement, but check existing code)
+- Use semicolons at end of statements (no enforcement, but check existing code)
+- No trailing commas in arrays/objects (`comma-dangle: [error, never]`)
+- Use arrow functions for callbacks where appropriate
+- Prefer object method shorthand
+
+## Build and Development Commands
+
+```bash
+# Install dependencies
+npm install
+
+# Run tests (includes linting and depcheck)
+npm test
+
+# Run linting only
+npm run lint
+
+# Run dependency check
+npm run depcheck
+
+# Version bumping and publishing
+npm run patch   # Bump patch version
+npm run minor   # Bump minor version
+npm run major   # Bump major version
+```
+
+### Pre-test Steps
+The `pretest` script automatically runs:
+1. Dependency check with `depcheck`
+2. ESLint on all source and test files
+
+## TypeScript Definitions
+
+- All source files should have corresponding `.d.ts` TypeScript definition files
+- Type definitions should match the JavaScript implementation
+- Export types for all public classes and their methods
+- Method signatures should include parameter types and return types
+
+## Dependencies
+
+### Production
+- `urllib`: HTTP client library (version 4.x)
+
+### Development
+- `@hapi/lab`: Test runner
+- `@hapi/code`: Assertion library
+- `eslint`: Linting
+- `depcheck`: Dependency checker
+- `sinon`: Test stubbing/mocking
+
+## MongoDB Atlas API Specifics
+
+### Authentication
+- Uses HTTP Digest Authentication
+- Requires public key (username) and private key (password)
+- Authentication is handled by HttpClient class
+
+### Common Parameters
+- `baseUrl`: MongoDB Atlas API base URL (e.g., `https://cloud.mongodb.com/api/atlas/v1.0`)
+- `projectId`: MongoDB Atlas project/group ID
+- `options`: Object that can contain:
+  - Query parameters (e.g., `envelope`, `itemsPerPage`, `pretty`)
+  - `httpOptions`: Additional options passed to urllib (e.g., `timeout`)
+
+### URL Structure
+- Groups and projects are synonymous in MongoDB Atlas API
+- URLs typically follow pattern: `/groups/{projectId}/resource/{identifier}`
+
+## Best Practices for Contributing
+
+1. **Minimal Changes**: Make the smallest possible changes to accomplish the goal
+2. **Test Coverage**: Maintain 100% test coverage - add tests for all new code
+3. **Consistency**: Follow existing patterns in the codebase
+4. **No Breaking Changes**: Don't modify existing public APIs unless necessary
+5. **Documentation**: Update README.md if adding new features
+6. **Type Safety**: Update corresponding `.d.ts` files when modifying JavaScript
+7. **Error Handling**: Follow existing error handling patterns (let errors propagate)
+8. **Async/Await**: Use async/await for all asynchronous operations (not callbacks or raw promises)
+
+## Common Tasks
+
+### Adding a New API Resource
+
+1. Create `src/resourceName.js` with class following the established pattern
+2. Create `src/resourceName.d.ts` with TypeScript definitions
+3. Add resource to `src/index.js` initialization and exports
+4. Create `test/resourceName.test.js` with comprehensive tests
+5. Ensure 100% coverage and all tests pass
+6. Update README.md if the resource is a significant new feature
+
+### Adding a Method to Existing Resource
+
+1. Add method to the class in `src/`
+2. Update corresponding `.d.ts` file
+3. Add comprehensive tests in `test/` file
+4. Ensure 100% coverage maintained
+5. Run linter and fix any issues
+
+## Important Notes
+
+- This library is production code used by many users - stability is critical
+- All changes must pass existing tests without modification
+- HTTP interactions are mocked in tests using urllib's MockAgent
+- The library uses MongoDB Atlas API v1.0
+- Digest authentication is required for all API calls

package/eslint.config.js

@@ -1,251 +1,205 @@
 // eslint.config.js
 const pluginJs = require("@eslint/js");
 
-module.export = [
+module.exports = [
   pluginJs.configs.recommended,
   {
-    "rules": {
+    languageOptions: {
+      ecmaVersion: "latest",
+      sourceType: "commonjs",
+      globals: {
+        process: "readonly",
+        __dirname: "readonly",
+        __filename: "readonly",
+        require: "readonly",
+        module: "readonly",
+        exports: "writable",
+        console: "readonly",
+        Buffer: "readonly",
+        URLSearchParams: "readonly"
+      }
+    },
+    rules: {
       // Possible Errors
-      // The following rules point out areas where you might have made mistakes.
-
-      "comma-dangle": [2, "never"],               // - disallow or enforce trailing commas (recommended)
-      "getter-return": 2,                         // - enforces that a return statement is present in property getters
-      "no-async-promise-executor": 2,             // - disallow using an async function as a Promise executor
-      "no-await-in-loop": 2,                      // - disallow await inside of loops
-      "no-compare-neg-zero": 2,                   // - disallow checking for equality against negative zero
-      "no-cond-assign": 2,                        // - disallow assignment in conditional expressions (recommended)
-      "no-console": 2,                            // - disallow use of console in the node environment (recommended)
-      "no-constant-condition": 2,                 // - disallow use of constant expressions in conditions (recommended)
-      "no-control-regex": 2,                      // - disallow control characters in regular expressions (recommended)
-      "no-debugger": 2,                           // - disallow use of debugger (recommended)
-      "no-dupe-args": 2,                          // - disallow duplicate arguments in functions (recommended)
-      "no-dupe-keys": 2,                          // - disallow duplicate keys when creating object literals (recommended)
-      "no-duplicate-case": 2,                     // - disallow a duplicate case label. (recommended)
-      "no-empty-character-class": 2,              // - disallow the use of empty character classes in regular expressions (recommended)
-      "no-empty": 2,                              // - disallow empty statements (recommended)
-      "no-ex-assign": 2,                          // - disallow assigning to the exception in a catch block (recommended)
-      "no-extra-boolean-cast": 2,                 // - disallow double-negation boolean casts in a boolean context (recommended)
-      "no-extra-parens": 0,                       // - disallow unnecessary parentheses
-      "no-extra-semi": 2,                         // - disallow unnecessary semicolons (recommended)
-      "no-func-assign": 2,                        // - disallow overwriting functions written as function declarations (recommended)
-      "no-inner-declarations": [2, "functions"],  // - disallow function or variable declarations in nested blocks (recommended)
-      "no-invalid-regexp": 2,                     // - disallow invalid regular expression strings in the RegExp constructor (recommended)
-      "no-irregular-whitespace": 2,               // - disallow irregular whitespace outside of strings and comments (recommended)
-      "no-negated-in-lhs": 2,                     // - disallow negation of the left operand of an in expression (recommended)
-      "no-obj-calls": 2,                          // - disallow the use of object properties of the global object (Math and JSON) as functions (recommended)
-      "no-regex-spaces": 2,                       // - disallow multiple spaces in a regular expression literal (recommended)
-      "no-sparse-arrays": 0,                      // - disallow sparse arrays (recommended) Note: set to false because array destructuring in ES6
-      "no-unreachable": 2,                        // - disallow unreachable statements after a return, throw, continue, or break statement (recommended)
-      "use-isnan": 2,                             // - disallow comparisons with the value NaN (recommended)
-      "valid-typeof": 2,                          // - Ensure that the results of typeof are compared against a valid string (recommended)
-      "no-unexpected-multiline": 2,               // - Avoid code that looks like two expressions but is actually one
+      "comma-dangle": ["error", "never"],
+      "getter-return": "error",
+      "no-async-promise-executor": "error",
+      "no-await-in-loop": "error",
+      "no-compare-neg-zero": "error",
+      "no-cond-assign": "error",
+      "no-console": "error",
+      "no-constant-condition": "error",
+      "no-control-regex": "error",
+      "no-debugger": "error",
+      "no-dupe-args": "error",
+      "no-dupe-keys": "error",
+      "no-duplicate-case": "error",
+      "no-empty-character-class": "error",
+      "no-empty": "error",
+      "no-ex-assign": "error",
+      "no-extra-boolean-cast": "error",
+      "no-extra-parens": "off",
+      "no-extra-semi": "error",
+      "no-func-assign": "error",
+      "no-inner-declarations": ["error", "functions"],
+      "no-invalid-regexp": "error",
+      "no-irregular-whitespace": "error",
+      "no-obj-calls": "error",
+      "no-regex-spaces": "error",
+      "no-sparse-arrays": "off",
+      "no-unreachable": "error",
+      "use-isnan": "error",
+      "valid-typeof": "error",
+      "no-unexpected-multiline": "error",
 
       // Best Practices
-      // These are rules designed to prevent you from making mistakes. They either prescribe a better way of doing something or help you avoid footguns.
-
-      "accessor-pairs": 2,                        // - Enforces getter/setter pairs in objects
-      "block-scoped-var": 0,                      // - treat var statements as if they were block scoped
-      "complexity": [1, 9],                       // - specify the maximum cyclomatic complexity allowed in a program
-      "consistent-return": 1,                     // - require return statements to either always or never specify values
-      "curly": [2, "all"],                       // - specify curly brace conventions for all control statements
-      "default-case": 0,                          // - require default case in switch statements
-      "dot-notation": 2,                          // - encourages use of dot notation whenever possible
-      "dot-location": [2, "property"],            // - enforces consistent newlines before or after dots
-      "eqeqeq": 2,                                // - require the use of === and !==
-      "guard-for-in": 2,                          // - make sure for-in loops have an if statement
-      "no-alert": 2,                              // - disallow the use of alert, confirm, and prompt
-      "no-caller": 2,                             // - disallow use of arguments.caller or arguments.callee
-      "no-div-regex": 2,                          // - disallow division operators explicitly at beginning of regular expression
-      "no-else-return": 2,                        // - disallow else after a return in an if
-      "no-eq-null": 2,                            // - disallow comparisons to null without a type-checking operator
-      "no-eval": 2,                               // - disallow use of eval()
-      "no-extend-native": 2,                      // - disallow adding to native types
-      "no-extra-bind": 2,                         // - disallow unnecessary function binding
-      "no-fallthrough": 2,                        // - disallow fallthrough of case statements (recommended)
-      "no-floating-decimal": 2,                   // - disallow the use of leading or trailing decimal points in numeric literals
-      "no-implicit-coercion": 2,                  // - disallow the type conversions with shorter notations
-      "no-implied-eval": 2,                       // - disallow use of eval()-like methods
-      "no-invalid-this": 2,                       // - disallow this keywords outside of classes or class-like objects
-      "no-iterator": 2,                           // - disallow usage of __iterator__ property
-      "no-labels": 2,                             // - disallow use of labeled statements
-      "no-lone-blocks": 2,                        // - disallow unnecessary nested blocks
-      "no-loop-func": 2,                          // - disallow creation of functions within loops
-      "no-multi-spaces": [
-        2,
-        {
-          "ignoreEOLComments": true
-        }
-      ],                                          // - disallow use of multiple spaces
-      "no-multi-str": 2,                          // - disallow use of multiline strings
-      "no-native-reassign": 2,                    // - disallow reassignments of native objects
-      "no-new-func": 2,                           // - disallow use of new operator for Function object
-      "no-new-wrappers": 2,                       // - disallows creating new instances of String,Number, and Boolean
-      "no-new": 2,                                // - disallow use of the new operator when not part of an assignment or comparison
-      "no-octal-escape": 2,                       // - disallow use of octal escape sequences in string literals, such as var foo = "Copyright \251";
-      "no-octal": 2,                              // - disallow use of octal literals (recommended)
-      "no-param-reassign": 2,                     // - disallow reassignment of function parameters
-      "no-process-env": 2,                        // - disallow use of process.env
-      "no-proto": 2,                              // - disallow usage of __proto__ property
-      "no-redeclare": 2,                          // - disallow declaring the same variable more than once (recommended)
-      "no-return-assign": 2,                      // - disallow use of assignment in return statement
-      "no-return-await": 2,                       // - disallows unnecessary return await
-      "no-script-url": 2,                         // - disallow use of javascript: urls.
-      "no-self-assign": 2,                        // - disallow Self Assignment
-      "no-self-compare": 2,                       // - disallow comparisons where both sides are exactly the same
-      "no-sequences": 2,                          // - disallow use of the comma operator
-      "no-throw-literal": 2,                      // - restrict what can be thrown as an exception
-      "no-unused-expressions": 0,                 // - disallow usage of expressions in statement position
-      "no-useless-call": 2,                       // - disallow unnecessary .call() and .apply()
-      "no-void": 2,                               // - disallow use of the void operator
-      "no-warning-comments": 0,                   // - disallow usage of configurable warning terms in comments - e.g. TODO or FIXME
-      "no-with": 2,                               // - disallow use of the with statement
-      "radix": 2,                                 // - require use of the second argument for parseInt()
-      "vars-on-top": 2,                           // - require declaration of all vars at the top of their containing scope
-      "wrap-iife": 2,                             // - require immediate function invocation to be wrapped in parentheses
-      "yoda": 2,                                  // - require or disallow Yoda conditions
-
-      // Strict Mode
-      // These rules relate to using strict mode.
-
-      "strict": [2, "global"],                    // - controls location of Use Strict Directives
+      "accessor-pairs": "error",
+      "block-scoped-var": "off",
+      "complexity": ["warn", 9],
+      "consistent-return": "warn",
+      "curly": ["error", "all"],
+      "default-case": "off",
+      "dot-notation": "error",
+      "dot-location": ["error", "property"],
+      "eqeqeq": "error",
+      "guard-for-in": "error",
+      "no-alert": "error",
+      "no-caller": "error",
+      "no-div-regex": "error",
+      "no-else-return": "error",
+      "no-eq-null": "error",
+      "no-eval": "error",
+      "no-extend-native": "error",
+      "no-extra-bind": "error",
+      "no-fallthrough": "error",
+      "no-floating-decimal": "error",
+      "no-implicit-coercion": "error",
+      "no-implied-eval": "error",
+      "no-invalid-this": "error",
+      "no-iterator": "error",
+      "no-labels": "error",
+      "no-lone-blocks": "error",
+      "no-loop-func": "error",
+      "no-multi-spaces": ["error", { "ignoreEOLComments": true }],
+      "no-multi-str": "error",
+      "no-new-func": "error",
+      "no-new-wrappers": "error",
+      "no-new": "error",
+      "no-octal-escape": "error",
+      "no-octal": "error",
+      "no-param-reassign": "error",
+      "no-proto": "error",
+      "no-redeclare": "error",
+      "no-return-assign": "error",
+      "no-return-await": "error",
+      "no-script-url": "error",
+      "no-self-assign": "error",
+      "no-self-compare": "error",
+      "no-sequences": "error",
+      "no-throw-literal": "error",
+      "no-unused-expressions": "off",
+      "no-useless-call": "error",
+      "no-void": "error",
+      "no-warning-comments": "off",
+      "no-with": "error",
+      "radix": "error",
+      "vars-on-top": "error",
+      "wrap-iife": "error",
+      "yoda": "error",
 
       // Variables
-      // These rules have to do with variable declarations.
+      "init-declarations": "off",
+      "no-delete-var": "error",
+      "no-label-var": "error",
+      "no-shadow-restricted-names": "error",
+      "no-shadow": "error",
+      "no-undef-init": "error",
+      "no-undef": "error",
+      "no-undefined": "off",
+      "no-unused-vars": "error",
+      "no-use-before-define": "error",
 
-      "init-declarations": 0,                     // - enforce or disallow variable initializations at definition
-      "no-catch-shadow": 2,                       // - disallow the catch clause parameter name being the same as a variable in the outer scope
-      "no-delete-var": 2,                         // - disallow deletion of variables (recommended)
-      "no-label-var": 2,                          // - disallow labels that share a name with a variable
-      "no-shadow-restricted-names": 2,            // - disallow shadowing of names such as arguments
-      "no-shadow": 2,                             // - disallow declaration of variables already declared in the outer scope
-      "no-undef-init": 2,                         // - disallow use of undefined when initializing variables
-      "no-undef": 2,                              // - disallow use of undeclared variables unless mentioned in a /*global */ block (recommended)
-      "no-undefined": 0,                          // - disallow use of undefined variable
-      "no-unused-vars": 2,                        // - disallow declaration of variables that are not used in the code (recommended)
-      "no-use-before-define": 2,                  // - disallow use of variables before they are defined
+      // Node.js (using stylistic rules where available in ESLint 9)
+      "callback-return": "off", // Removed in ESLint 9
+      "global-require": "off",  // Removed in ESLint 9
+      "handle-callback-err": "off", // Removed in ESLint 9
+      "no-mixed-requires": "off", // Removed in ESLint 9
+      "no-new-require": "off", // Removed in ESLint 9
+      "no-path-concat": "off", // Removed in ESLint 9
+      "no-process-env": "off", // Removed in ESLint 9
+      "no-process-exit": "off", // Removed in ESLint 9
+      "no-restricted-modules": "off",
+      "no-sync": "off", // Removed in ESLint 9
+      "no-buffer-constructor": "error",
 
-      // Node.js
-      // These rules are specific to JavaScript running on Node.js.
-
-      "callback-return": 2,                       // - enforce return after a callback
-      "global-require": 2,                        // - enforce require() on the top-level module scope
-      "handle-callback-err": 2,                   // - enforce error handling in callbacks
-      "no-mixed-requires": 2,                     // - disallow mixing regular variable and require declarations
-      "no-new-require": 2,                        // - disallow use of new operator with the require function
-      "no-path-concat": 2,                        // - disallow string concatenation with __dirname and __filename
-      "no-process-exit": 2,                       // - disallow process.exit()
-      "no-restricted-modules": 2,                 // - restrict usage of specified node modules
-      "no-sync": 2,                               // - disallow use of synchronous methods
-      "no-buffer-constructor": 2,                 // - disallow use of the deprecated Buffer() constructor
-
-      // Stylistic Issues
-      // These rules are purely matters of style and are quite subjective.
-
-      "array-bracket-spacing": 2,                 // - enforce spacing inside array brackets
-      "brace-style": 2,                           // - enforce one true brace style
-      "camelcase": 2,                             // - require camel case names
-      "comma-spacing": 2,                         // - enforce spacing before and after comma
-      "comma-style": 2,                           // - enforce one true comma style
-      "computed-property-spacing": 2,             // - require or disallow padding inside computed properties
-      "consistent-this": 2,                       // - enforce consistent naming when capturing the current execution context
-      "eol-last": 2,                              // - enforce newline at the end of file, with no multiple empty lines
-      "func-names": 0,                            // - require function expressions to have a name
-      "func-style": [2, "declaration"],           // - enforce use of function declarations or expressions
-      "id-length": [2,
-        {
-          "min": 2,
-          "max": 70,
-          "exceptions": ["i", "j", "k", "n", "Q", "_"]
-        }
-      ],                                          // - this option enforces minimum and maximum identifier lengths (variable names, property names etc.) (off by default)
-      "id-match": 0,                              // - require identifiers to match the provided regular expression
-      "indent": [2, 2,                            // - specify tab or space width for your code
-        {
-          "SwitchCase": 1
-        }
-      ],
-      "key-spacing": 2,                           // - enforce spacing between keys and values in object literal properties
-      "lines-around-comment": [2,
-        {
-          "beforeBlockComment": true,
-          "afterBlockComment": false,
-          "beforeLineComment": false,
-          "afterLineComment": false,
-          "allowBlockStart": true,
-          "allowBlockEnd": true
-        }
-      ],                                          // - enforce empty lines around comments
-      "linebreak-style": 2,                       // - disallow mixed 'LF' and 'CRLF' as linebreaks
-      "max-nested-callbacks": [2, 6],             // - specify the maximum depth callbacks can be nested
-      "max-statements-per-line": [
-        2,
-        {
-          "max": 1
-        }
-      ],                                          // - enforce a maximum number of statements per line
-      "new-cap": 0,                               // - require a capital letter for constructors
-      "new-parens": 2,                            // - disallow the omission of parentheses when invoking a constructor with no arguments
-      "newline-after-var": 0,                     // - require or disallow an empty newline after variable declarations
-      "no-array-constructor": 2,                  // - disallow use of the Array constructor
-      "no-continue": 2,                           // - disallow use of the continue statement
-      "no-inline-comments": 0,                    // - disallow comments inline after code
-      "no-lonely-if": 2,                          // - disallow if as the only statement in an else block
-      "no-mixed-spaces-and-tabs": 2,              // - disallow mixed spaces and tabs for indentation (recommended)
-      "no-multiple-empty-lines": [
-        2,
-        {
-          "max": 2,
-          "maxEOF": 1,
-          "maxBOF": 0
-        }
-      ],                                          // - disallow multiple empty lines
-      "no-nested-ternary": 2,                     // - disallow nested ternary expressions
-      "no-new-object": 2,                         // - disallow the use of the Object constructor
-      "no-spaced-func": 2,                        // - disallow space between function identifier and application
-      "no-ternary": 0,                            // - disallow the use of ternary operators
-      "no-trailing-spaces": 2,                    // - disallow trailing whitespace at the end of lines
-      "no-underscore-dangle": 0,                  // - disallow dangling underscores in identifiers
-      "no-unneeded-ternary": 2,                   // - disallow the use of Boolean literals in conditional expressions
-      "object-curly-spacing": 2,                  // - require or disallow padding inside curly braces
-      "one-var": [1, "never"],                    // - require or disallow one variable declaration per function
-      "operator-assignment": [2, "always"],       // - require assignment operator shorthand where possible or prohibit it entirely
-      "operator-linebreak": 2,                    // - enforce operators to be placed before or after line breaks
-      "padded-blocks": 0,                         // - enforce padding within blocks
-      "quote-props": 2,                           // - require quotes around object literal property names
-      "quotes": [2, "double"],                    // - specify whether backticks, double or single quotes should be used
-      "semi-spacing": 2,                          // - enforce spacing before and after semicolons
-      "semi": 2,                                  // - require or disallow use of semicolons instead of ASI
-      "sort-vars": 0,                             // - sort variables within the same declaration block
-      "keyword-spacing": 2,                       // - require a space after certain keywords
-      "space-before-blocks": 2,                   // - require or disallow a space before blocks
-      "space-before-function-paren": [2,
-        {
-          "anonymous": "always",
-          "named": "never"
-        }
-      ],                                          //  - require or disallow a space before function opening parenthesis
-      "space-in-parens": 2,                       //  - require or disallow spaces inside parentheses
-      "space-infix-ops": 2,                       // - require spaces around operators
-      "space-unary-ops": 2,                       // - require or disallow spaces before/after unary operators
-      "spaced-comment": 2,                        // - require or disallow a space immediately following the // or /* in a comment
-      "wrap-regex": 2,                            // - require regex literals to be wrapped in parentheses
+      // Stylistic Issues (many moved to @stylistic/eslint-plugin in ESLint 9)
+      "array-bracket-spacing": "off",
+      "brace-style": "off",
+      "camelcase": "error",
+      "comma-spacing": "off",
+      "comma-style": "off",
+      "computed-property-spacing": "off",
+      "consistent-this": "off",
+      "eol-last": "off",
+      "func-names": "off",
+      "func-style": "off",
+      "id-length": ["error", { "min": 2, "max": 70, "exceptions": ["i", "j", "k", "n", "Q", "_"] }],
+      "id-match": "off",
+      "indent": "off",
+      "key-spacing": "off",
+      "lines-around-comment": "off",
+      "linebreak-style": "off",
+      "max-nested-callbacks": ["error", 6],
+      "max-statements-per-line": "off",
+      "new-cap": "off",
+      "new-parens": "off",
+      "newline-after-var": "off",
+      "no-array-constructor": "error",
+      "no-continue": "error",
+      "no-inline-comments": "off",
+      "no-lonely-if": "error",
+      "no-mixed-spaces-and-tabs": "error",
+      "no-multiple-empty-lines": "off",
+      "no-nested-ternary": "error",
+      "no-new-object": "error",
+      "no-spaced-func": "off",
+      "no-ternary": "off",
+      "no-trailing-spaces": "off",
+      "no-underscore-dangle": "off",
+      "no-unneeded-ternary": "error",
+      "object-curly-spacing": "off",
+      "one-var": ["warn", "never"],
+      "operator-assignment": ["error", "always"],
+      "operator-linebreak": "off",
+      "padded-blocks": "off",
+      "quote-props": "off",
+      "quotes": "off",
+      "semi-spacing": "off",
+      "semi": "off",
+      "sort-vars": "off",
+      "keyword-spacing": "off",
+      "space-before-blocks": "off",
+      "space-before-function-paren": "off",
+      "space-in-parens": "off",
+      "space-infix-ops": "off",
+      "space-unary-ops": "off",
+      "spaced-comment": "off",
+      "wrap-regex": "off",
 
       // ECMAScript 6
-      // These rules are only relevant to ES6 environments.
-
-      "arrow-parens": [2, "as-needed"],           // - require parens in arrow function arguments
-      "arrow-spacing": 2,                         // - require space before/after arrow function's arrow
-      "constructor-super": 2,                     // - verify calls of super() in constructors
-      "generator-star-spacing": 2,                // - enforce spacing around the * in generator functions
-      "no-class-assign": 2,                       // - disallow modifying variables of class declarations
-      "no-const-assign": 2,                       // - disallow modifying variables that are declared using const
-      "no-this-before-super": 2,                  // - disallow use of this/super before calling super() in constructors.
-      "no-var": 2,                                // - require let or const instead of var
-      "object-shorthand": [2, "methods"],         // - require method and property shorthand syntax for object literals
-      "prefer-const": 2,                          // - suggest using const declaration for variables that are never modified after declared
-      "prefer-spread": 2,                         // - suggest using the spread operator instead of .apply().
-      "require-yield": 2,                         // - disallow generator functions that do not have yield
-      "prefer-template": 2
+      "arrow-parens": "off",
+      "arrow-spacing": "off",
+      "constructor-super": "error",
+      "generator-star-spacing": "off",
+      "no-class-assign": "error",
+      "no-const-assign": "error",
+      "no-this-before-super": "error",
+      "no-var": "error",
+      "object-shorthand": ["error", "methods"],
+      "prefer-const": "error",
+      "prefer-spread": "error",
+      "require-yield": "error",
+      "prefer-template": "error"
     }
   }
 ];

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "mongodb-atlas-api-client",
-  "version": "4.10.0",
+  "version": "4.13.0",
   "description": "A mongodb atlas api client for nodejs.",
   "main": "src/index.js",
   "scripts": {
     "pretest": "npm run depcheck && eslint --cache \"src/**/*.js\" \"test/**/*.js\"",
     "depcheck": "depcheck",
-    "test": "./node_modules/@hapi/lab/bin/lab -I '@@any-promise/REGISTRATION,Symbol(undici.globalDispatcher.1)' ./test/ -v -S --assert @hapi/code --threshold 100 -p 1 -o test-results/result.json -r json -r console -o stdout",
+    "test": "./node_modules/@hapi/lab/bin/lab -I '@@any-promise/REGISTRATION,Symbol(undici.globalDispatcher.1),File' ./test/ -v -S --assert @hapi/code --threshold 100 -p 1 -o test-results/result.json -r json -r console -o stdout",
     "lint": "./node_modules/.bin/eslint ./src --fix",
     "premajor": "npm run test",
     "major": "npm version major -m \"published to npm as v%s\" && git push --follow-tags && npm publish",
@@ -36,14 +36,14 @@
   },
   "homepage": "https://github.com/montumodi/mongodb-atlas-api-client#readme",
   "devDependencies": {
-    "@eslint/js": "^9.10.0",
+    "@eslint/js": "^9.38.0",
     "@hapi/code": "^9.0.3",
-    "@hapi/lab": "^25.3.1",
+    "@hapi/lab": "^25.3.2",
     "depcheck": "^1.4.7",
-    "eslint": "^9.10.0",
-    "sinon": "^18.0.0"
+    "eslint": "^9.38.0",
+    "sinon": "^21.0.0"
   },
   "dependencies": {
-    "urllib": "^4.2.0"
+    "urllib": "^4.8.2"
   }
 }

package/test/helper.test.js

@@ -1,6 +1,6 @@
 const {describe, it} = exports.lab = require("@hapi/lab").script();
 const {expect} = require('@hapi/code');
-const {getQueryStringFromOptions}  = require('../src/helper.js');
+const {getQueryStringFromOptions} = require('../src/helper.js');
 
 describe("Helper Methods", () => {