chore(astro): todos cleanup (#14405)

Author: florian-lefebvre

packages/astro/src/assets/fonts/orchestrate.ts

@@ -72,7 +72,6 @@ export async function orchestrate({
 	fontMetricsResolver: FontMetricsResolver;
 	fontTypeExtractor: FontTypeExtractor;
 	fontFileReader: FontFileReader;
-	// TODO: follow this implementation: https://github.com/withastro/astro/pull/13756/commits/e30ac2b7082a3eed36225da6e88449890cbcbe6b
 	logger: Logger;
 	createUrlProxy: (params: CreateUrlProxyParams) => UrlProxy;
 	defaults: Defaults;

packages/astro/src/core/compile/compile.ts

@@ -47,9 +47,6 @@ export async function compile({
 			normalizedFilename: normalizeFilename(filename, astroConfig.root),
 			sourcemap: 'both',
 			internalURL: 'astro/compiler-runtime',
-			// TODO: this is no longer necessary for `Astro.site`
-			// but it somehow allows working around caching issues in content collections for some tests
-			astroGlobalArgs: JSON.stringify(astroConfig.site),
 			scopedStyleStrategy: astroConfig.scopedStyleStrategy,
 			resultScopedSlot: true,
 			transitionsAnimationURL: 'astro/components/viewtransitions.css',

packages/astro/src/core/dev/container.ts

@@ -134,7 +134,6 @@ export async function createContainer({
 		handle(req, res) {
 			viteServer.middlewares.handle(req, res, Function.prototype);
 		},
-		// TODO deprecate and remove
 		close() {
 			return closeContainer(container);
 		},

packages/astro/src/runtime/server/astro-global.ts

@@ -34,8 +34,6 @@ Use import.meta.glob instead: https://vitejs.dev/guide/features.html#glob-import
 // inside of getStaticPaths. See the `astroGlobalArgs` option for parameter type.
 export function createAstro(site: string | undefined): AstroGlobalPartial {
 	return {
-		// TODO: this is no longer necessary for `Astro.site`
-		// but it somehow allows working around caching issues in content collections for some tests
 		site: site ? new URL(site) : undefined,
 		generator: `Astro v${ASTRO_VERSION}`,
 		glob: createAstroGlobFn(),