Merge branch 'master' into vkarpov15/gh-15779

Author: vkarpov15

docs/compatibility.md

@@ -11,14 +11,15 @@
     }
 </style>
 
-Mongoose relies on the [MongoDB Node.js Driver](http://mongodb.github.io/node-mongodb-native/) to talk to MongoDB.
-You can refer to [this table](https://www.mongodb.com/docs/drivers/node/current/compatibility/) for up-to-date information as to which version of the MongoDB driver supports which version of MongoDB.
+Mongoose relies on the [MongoDB Node.js Driver](https://www.mongodb.com/docs/drivers/node/current/) to communicate with MongoDB.
 
-Below are the [semver](http://semver.org/) ranges representing which versions of mongoose are compatible with the listed versions of MongoDB server.
+You can refer to [this table](https://www.mongodb.com/docs/drivers/compatibility/?driver-language=javascript&javascript-driver-framework=nodejs) for up-to-date information as to which version of the MongoDB driver supports which version of the MongoDB server.
+
+Below are the [SemVer](http://semver.org/) ranges representing which versions of mongoose are compatible with the listed versions of MongoDB server.
 
 | MongoDB Server |                    Mongoose                    |
 | :------------: | :--------------------------------------------: |
-|     `8.x`      |                 `^8.7.0 | ^9.0.0`              |
+|     `8.x`      |                 `^8.7.0 \| ^9.0.0`             |
 |     `7.x`      |            `^7.4.0 \| ^8.0.0 \| ^9.0.0`        |
 |     `6.x`      |            `^7.0.0 \| ^8.0.0 \| ^9.0.0`        |
 |     `5.x`      |            `^6.0.0 \| ^7.0.0 \| ^8.0.0`        |
@@ -27,4 +28,4 @@ Below are the [semver](http://semver.org/) ranges representing which versions of
 |    `4.0.x`     |       `^6.0.0 \| ^7.0.0 \| ^8.0.0 <8.16.0`     |
 |    `3.6.x`     |       `^6.0.0 \| ^7.0.0 \| ^8.0.0 <8.8.0`      |
 
-Mongoose `^6.5.0` also works with MongoDB server 7.x. But not all new MongoDB server 7.x features are supported by Mongoose 6.x.
+Mongoose `^6.5.0` also works with MongoDB server 7.x. But not all new MongoDB server 7.x features are supported by Mongoose 6.x. To verify that your version of Mongoose is compatible based on the table above, use the [online SemVer checker](https://jubianchi.github.io/semver-check/#/).

docs/source/api.js

@@ -269,7 +269,7 @@ function processFile(props) {
     if ('constructor' in ctx && ctx.constructor === undefined) {
       ctx.constructorWasUndefined = true;
     }
-
+    if (!prop.tags) continue;
     for (const __tag of prop.tags) {
       // the following has been done, because otherwise no type could be applied for intellisense
       /** @type {TagObject} */

scripts/generateSearch.js

@@ -63,14 +63,17 @@ function generateContents() {
       content.validateSync();
 
       const $ = cheerio.load(text);
-
       contents.push(content);
 
-      // Break up individual h3's into separate content for more fine grained search
+      // Break up h3's into additional content entries
       $('h3').each((index, el) => {
         el = $(el);
         const title = el.text();
         const html = el.nextUntil('h3').html();
+
+        // *** FIXED: Skip empty HTML blocks ***
+        if (!html || html.trim() === '') return;
+
         const id = el.prop('id');
         const baseUrl = filename.replace('.md', '.html').replace(/^docs/, '');
         const content = new Content({
@@ -82,6 +85,7 @@ function generateContents() {
         content.validateSync();
         contents.push(content);
       });
+
     } else if (file.guide) {
       let text = fs.readFileSync(filename, 'utf8');
       text = text.substring(text.indexOf('block content') + 'block content\n'.length);
@@ -96,14 +100,17 @@ function generateContents() {
       content.validateSync();
 
       const $ = cheerio.load(text);
-
       contents.push(content);
 
-      // Break up individual h3's into separate content for more fine grained search
+      // Break up h3's into additional content entries
       $('h3').each((index, el) => {
         el = $(el);
         const title = el.text();
         const html = el.nextUntil('h3').html();
+
+        // *** FIXED: Skip empty HTML blocks ***
+        if (!html || html.trim() === '') return;
+
         const id = el.prop('id');
         const baseUrl = filename.replace('.pug', '.html').replace(/^docs/, '');
         const content = new Content({
@@ -137,56 +144,47 @@ async function generateSearch(config) {
 
   const promises = [];
   let lastPrint = 0;
-
   let doneCount = 0;
+
   console.log('Search Content to save:', contents.length);
+
   for (const content of contents) {
-    if (version === '9.x') {
-      let url = content.url.startsWith('/') ? content.url : `/${content.url}`;
-      if (!url.startsWith('/docs')) {
-        url = '/docs' + url;
-      }
-      content.url = url;
-    } else {
-      let url = content.url.startsWith('/') ? content.url : `/${content.url}`;
-      if (!url.startsWith('/docs')) {
-        url = '/docs' + url;
-      }
-      content.url = `/docs/${version}${url}`;
+    let url = content.url.startsWith('/') ? content.url : `/${content.url}`;
+    if (!url.startsWith('/docs')) {
+      url = '/docs' + url;
     }
+    content.url = version === '9.x' ? url : `/docs/${version}${url}`;
+
     const promise = content.save().then(() => {
       doneCount += 1;
       const nowDate = Date.now();
-      // only print every 2 seconds, or if it is the first or last element
       if (nowDate - lastPrint > 2000 || doneCount === contents.length || doneCount === 1) {
         lastPrint = nowDate;
         console.log(`${doneCount} / ${contents.length}`);
       }
     });
+
     promises.push(promise);
   }
 
   await Promise.allSettled(promises);
 
-  const results = await Content.
-    find({ $text: { $search: 'validate' }, version }, { score: { $meta: 'textScore' } }).
-    sort({ score: { $meta: 'textScore' } }).
-    limit(10);
+  const results = await Content
+    .find({ $text: { $search: 'validate' }, version }, { score: { $meta: 'textScore' } })
+    .sort({ score: { $meta: 'textScore' } })
+    .limit(10);
 
   console.log(results.map(res => res.url));
 
   console.log(`Added ${contents.length} Search Content`);
-
-  // this likely should not be done as part of this script, but by the caller,
-  // but this script is currently the only one that connects in the website generation.
   await mongoose.disconnect();
 }
 
 function getConfig() {
   const config = require('../.config.js');
 
   if (!config || !config.uri) {
-    throw new Error('No Config or config.URI given, please create a .config.js file with those values in the root of the repository');
+    throw new Error('No Config or config.uri given, please create a .config.js file with those values in the root of the repository');
   }
 
   return config;
@@ -195,7 +193,6 @@ function getConfig() {
 module.exports.generateSearch = generateSearch;
 module.exports.getConfig = getConfig;
 
-// only run the following code if this file is the main module / entry file
 if (isMain) {
   (async function main() {
     const config = getConfig();

test/types/connection.test.ts

@@ -167,5 +167,6 @@ async function gh15359() {
     { model: 'Test', name: 'updateOne', filter: { name: 'test4' }, update: { $set: { num: 'not a number' } } }
   ], { ordered: false });
   expectType<number>(res3.insertedCount);
+  expectError(res3.validationErrors);
   expectType<Error[] | undefined>(res3.mongoose?.validationErrors);
 }

test/types/document.test.ts

@@ -166,8 +166,6 @@ function gh11085(): void {
 
   const newUser = new UserModel();
 
-  let _id: number;
-  expectError(_id = newUser._id);
   const _id2: Types.ObjectId = newUser._id;
 }
 

test/types/queries.test.ts

@@ -851,3 +851,17 @@ async function gh15779() {
   TestModel.findOne(query);
   TestModel.deleteMany(query);
 }
+  
+async function gh15786() {
+  interface IDoc {
+    nmae: string;
+  }
+
+  interface DocStatics {
+    m1(): void;
+    m2(): void;
+  }
+
+  const schema = new Schema<IDoc, Model<IDoc>, {}, {}, {}, DocStatics>({});
+  schema.static({ m1() {} } as DocStatics);
+}

test/util.js

@@ -19,8 +19,10 @@ exports.clearTestData = async function clearTestData(db) {
     retries -= 1;
     try {
       await _inner();
+      return;
     } catch (err) {
-      if (err instanceof mongoose.mongo.MongoWriteConcernError && /operation was interrupted/.test(err.message)) {
+      const retryable = err instanceof mongoose.mongo.MongoWriteConcernError && /operation was interrupted/.test(err.message);
+      if (retryable && retries > 0) {
         console.log('DropDB operation interrupted, retrying'); // log that a error was thrown to know that it is going to re-try
         continue;
       }

types/index.d.ts

@@ -623,6 +623,7 @@ declare module 'mongoose' {
 
     /** Adds static "class" methods to Models compiled from this schema. */
     static<K extends keyof TStaticMethods>(name: K, fn: TStaticMethods[K]): this;
+    static(obj: { [F in keyof TStaticMethods]: TStaticMethods[F] }): this;
     static(obj: { [F in keyof TStaticMethods]: TStaticMethods[F] } & { [name: string]: (this: TModelType, ...args: any[]) => any }): this;
     static(name: string, fn: (this: TModelType, ...args: any[]) => any): this;