Dynamic Import of images using import.meta.glob() as described in the Documentation is causing error: ts(2322)

[scharf-dev]

In a project i started on v3, i used dynamic image importing as described in the Astro Documentation and it worked as expected.

This is the code:

---
import { Image } from "astro:assets";
const images =  import.meta.glob('../assets/images/slider/*.{jpeg,jpg,png,gif}');
---

<div id="slider" class="full-width img-wrapper">
  {Object.keys(images).map((path, index) => {
    return <Image src={images[path]()} alt="test"/>
  })}
</div>

Since i upgraded the project to Astro 4.0, i get the following error on the src property:

type 'Promise' is not assignable to type 'string | ImageMetadata | Promise<{ default: ImageMetadata; }>'.
Type 'Promise' is not assignable to type 'Promise<{ default: ImageMetadata; }>'.ts(2322)
types.d.ts(136, 5): The expected type comes from property 'src' which is declared here on type 'IntrinsicAttributes & Props'
(property) src: Promise

Despite the error, the images still show up on the dev server. However, astro build fails because of this.

[lilnasy]

Since Vite 5, import.meta.glob became stricter, any became unknown. You can tell it the expected shape of the module like this:

import type { ImageMetadata } from "astro"
import { Image, ImageMetadata } from "astro:assets";
const images =  import.meta.glob<{ default: ImageMetadata }>('../assets/images/slider/*.{jpeg,jpg,png,gif}');

[VoxelMC]

Since Vite 5, import.meta.glob became stricter, any became unknown. You can tell it the expected shape of the module like this:

import type { ImageMetadata } from "astro"
import { Image, ImageMetadata } from "astro:assets";
const images =  import.meta.glob<{ default: ImageMetadata }>('../assets/images/slider/*.{jpeg,jpg,png,gif}');

Shall I draft up a PR to reflect this suggestion?
I will test it later to make sure. After slight correction, is this correct?

import type { ImageMetadata } from "astro";
import { Image } from "astro:assets";
const images = import.meta.glob<{ default: ImageMetadata }>('../assets/images/slider/*.{jpeg,jpg,png,gif}');

[sarah11918]

@VoxelMC Thanks for tackling this! A PR would be great, then you can tag @Princesseuh to confirm!

[Princesseuh]

Funnily enough it's not that it became stricter, it's that the import was bugged in Vite 4, so it resolved to any