fix(astro compiler): check vite command to decide on "annotateSourceFile" (#9275)

lilnasy · bluwy · web-flow · commit 0968cb1a373b · 2023-12-05T01:29:35.000+05:30
* prevent annotations in build

* add changeset

* Apply suggestions from code review

Co-authored-by: Bjorn Lu &lt;bjornlu.dev@gmail.com&gt;

---------

Co-authored-by: Bjorn Lu &lt;bjornlu.dev@gmail.com&gt;
diff --git a/.changeset/chatty-poems-doubt.md b/.changeset/chatty-poems-doubt.md
@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Fixes an issue where html annotations relevant only to the dev server were included in the production build.
diff --git a/packages/astro/src/core/compile/compile.ts b/packages/astro/src/core/compile/compile.ts
@@ -48,7 +48,7 @@ export async function compile({
 			resultScopedSlot: true,
 			transitionsAnimationURL: 'astro/components/viewtransitions.css',
 			annotateSourceFile:
-				!viteConfig.isProduction && astroConfig.devToolbar && astroConfig.devToolbar.enabled,
+				viteConfig.command === "serve" && astroConfig.devToolbar && astroConfig.devToolbar.enabled,
 			preprocessStyle: createStylePreprocessor({
 				filename,
 				viteConfig,

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'astro': patch
 +---
++
 +Fixes an issue where html annotations relevant only to the dev server were included in the production build.