Nested arrays using `of` result in the incorrect schema type

[forivall]

Prerequisites

• I have written a descriptive issue title
• I have searched existing issues to ensure the bug has not already been reported

Mongoose version

7.6.9

Node.js version

20.6.1

MongoDB server version

5.x

Typescript version (if applicable)

5.2.2

Description

With a schema definition of

    const schema = new Schema({
      nums: [{ type: Array, of: Number }],
      tags: [{ type: 'Array', of: String }],
      subdocs: [{ type: Array, of: Schema({ name: String }) }]
    });

the resulting casters result in Mixed rather than the appropriate type.

    assert.equal(schema.path('nums.$').caster.instance, 'Number'); // actually Mixed
    assert.equal(schema.path('tags.$').caster.instance, 'String'); // actually Mixed
    assert.equal(schema.path('subdocs.$').casterConstructor.schema.path('name').instance, 'String'); // casterConstructor is actually a Mixed instance

Steps to Reproduce

See my updated tests at master...forivall:mongoose:forivall/nested-array-of

.patch

From fe53ac241a266afa08a1e5963e7ff53eac18c469 Mon Sep 17 00:00:00 2001
From: Emily M Klassen <forivall@gmail.com>
Date: Tue, 5 Mar 2024 17:50:52 -0800
Subject: [PATCH] test: add test for nested array of issue

---
 test/schema.test.js | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/test/schema.test.js b/test/schema.test.js
index 0df19a65790..d57fb8e528f 100644
--- a/test/schema.test.js
+++ b/test/schema.test.js
@@ -2672,6 +2672,18 @@ describe('schema', function() {
     assert.equal(schema.path('subdocs').casterConstructor.schema.path('name').instance, 'String');
   });
 
+  it('supports `of` for nested array type definition', function() {
+    const schema = new Schema({
+      nums: [{ type: Array, of: Number }],
+      tags: [{ type: 'Array', of: String }],
+      subdocs: [{ type: Array, of: Schema({ name: String }) }]
+    });
+
+    assert.equal(schema.path('nums.$').caster.instance, 'Number');
+    assert.equal(schema.path('tags.$').caster.instance, 'String');
+    assert.equal(schema.path('subdocs.$').casterConstructor.schema.path('name').instance, 'String');
+  });
+
   it('should use the top-most class\'s getter/setter gh-8892', function() {
     class C1 {
       get hello() {

Expected Behavior

The casters should be the correct schema types.

[IslandRhythms]

const mongoose = require('mongoose');
const { Schema } = mongoose;
const assert = require('assert');

(async () => {
  const schema = new Schema({
    nums: [{ type: Array, of: Number }],
    tags: [{ type: 'Array', of: String }],
    subdocs: [{ type: Array, of: Schema({ name: String }) }]
  });

  assert.equal(schema.path('nums.$').caster.instance, 'Number'); // actually Mixed
  assert.equal(schema.path('tags.$').caster.instance, 'String'); // actually Mixed
  assert.equal(schema.path('subdocs.$').casterConstructor.schema.path('name').instance, 'String'); // actually Mixed
  console.log('done');
})();

[forivall]

quick correction: in the third assert, the casterConstructor is actually an instance of mixed, so it doesnt have a schema property, so it results in the error

  schema.path('subdocs.$').casterConstructor.schema.path('name').instance,
                                                   ^

TypeError: Cannot read properties of undefined (reading 'path')

[vkarpov15]

I took a look and the issue is that we don't support type: Array, of: Number syntax currently. We'll add support for that in 8.3. But for older versions of Mongoose, of is only supported for Maps.

[forivall]

Yeah, it's been in the documentation here for a while, and that's how I hit the issue

Example:

// `arr` is an array of numbers.
new Schema({ arr: [Number] });
// Equivalent way to define `arr` as an array of numbers
new Schema({ arr: { type: Array, of: Number } });

(i thought i mentioned that in my original comment, but looks like i missed that. whoops!)