ReferenceError: __WC_CONFIG__ is not defined

[AriPerkkio]

Describe the bug

Freshly created tutorials without any modifications are running into __WC_CONFIG__ is not defined errors. This is coming up randomly and looks like possible race condition. When page is refreshed with cache clearing the error disappears, but normal refreshes run into this error immediatelly.

This only affects dev mode.

Link to a StackBlitz project which shows the error

No response

Steps to reproduce

1. pnpm create tutorial
2. Start in dev mode
3. Open page in browser
4. Keep refreshing page and check JS console

Expected behavior

Error should not happen.

Screenshots

Platform

• TutorialKit version: 0.2.0

Additional context

No response

[Nemikolh]

Could this be a bug with Astro? I wonder if we also see this issue with an older version of Astro.

I'm wondering because this is literally the first thing we set in the vite config and I don't think this has ever really changed:

tutorialkit/packages/astro/src/index.ts

Lines 77 to 94 in 3e7e6cb

	async 'astro:config:setup'(options) {
	const { injectRoute, updateConfig, config } = options;
	updateConfig({
	server: {
	headers: {
	'Cross-Origin-Embedder-Policy': isolation ?? 'require-corp',
	'Cross-Origin-Opener-Policy': 'same-origin',
	},
	},
	vite: {
	optimizeDeps: {
	entries: ['!**/src/(content|templates)/**'],
	include: process.env.TUTORIALKIT_DEV ? [] : ['@tutorialkit/react'],
	},
	define: {
	__ENTERPRISE__: `${!!enterprise}`,
	__WC_CONFIG__: enterprise ? JSON.stringify(enterprise) : 'undefined',

(also if that helps bisecting the combo that breaks, I've never encountered that issue when I was doing demos back in June)

[AriPerkkio]

This issue isn't present on 0.1.2 but appears first in 0.1.3.

0.1.2...0.1.3

[AriPerkkio]

This is likely introduced in #219.

When import of './OpenInStackblitzLink.astro' is removed here, the issue disappears:

tutorialkit/packages/astro/src/default/components/TopBarWrapper.astro

Lines 6 to 8 in 3e7e6cb

	import OpenInStackblitzLink from './OpenInStackblitzLink.astro';
	import Logo from './Logo.astro';
	import { useAuth } from './setup';

There's two depedency chains for the setup.ts that tries to access __WC_CONFIG__:

TopBarWrapper.astro > setup
TopBarWrapper.astro > OpenInStackblitzLink.astro > webcontainer > setup

The first one works fine. But the second on comes from <script> tag. Maybe Astro doesn't apply Vite's define transforms there? I'll keep looking... 👀

[AriPerkkio]

Maybe Astro doesn't apply Vite's define transforms there? I'll keep looking...

Something weird going on there. Tested this in minimal Astro reproduction:

import { defineConfig } from "astro/config";

export default defineConfig({
  vite: {
    define: {
      __MY_DEFINE__: '"Value set in config"',
    },
  },
});

---
const frontmatter = __MY_DEFINE__;
---

<h1>My Component</h1>

<script define:vars={{frontmatter}}>
  console.log({frontmatter, scriptScope: globalThis.__MY_DEFINE__})

  setTimeout(() => {
    console.log({frontmatter, scriptScope: globalThis.__MY_DEFINE__})
  }, 1000)
</script>

And we get:

Looks like Vite's dev-time define is loaded after the Astro component is. Looks like bug to me.

I'll try to find quick solution for us here so that we can release fix as soon as possible. Maybe convert the component into React one. 🤔

[AriPerkkio]

withastro/astro#11874

[Nemikolh]

Oh good find! Thanks for digging in 🤩