Is Nesting Schemas Inside Each Other an Anti-Pattern?

export class Product extends Schema.TaggedClass<Product>("Product")("Product", {
  id: ProductId,
  title: Schema.String,
  description: Schema.NullOr(Schema.String),
  status: Schema.Literal("draft", "published"),
  options: Schema.Array(ProductOption),
}) {}

export class Product extends Schema.TaggedClass<Product>("Product")("Product", {
  id: ProductId,
  title: Schema.String,
  description: Schema.NullOr(Schema.String),
  status: Schema.Literal("draft", "published"),
  options: Schema.Array(ProductOption),
}) {}

return new Product({
            ...product,
            id: ProductId.make(product.id),
            options: product.options.map(
              (option) =>
                new ProductOption({
                  title: option.title,
                  id: ProductOptionId.make(option.id),
                  productId: ProductId.make(option.productId),
                }),
            ),
          });

return new Product({
            ...product,
            id: ProductId.make(product.id),
            options: product.options.map(
              (option) =>
                new ProductOption({
                  title: option.title,
                  id: ProductOptionId.make(option.id),
                  productId: ProductId.make(option.productId),
                }),
            ),
          });