Fix markdown syntax issues (#234)

This fixes #233 among other things

examples/snowpack/src/layouts/content-with-cover.astro

@@ -80,7 +80,7 @@ export let content: any;
           </div>
         </article>
 
-        <Subnav title={content.title} headers={content.headers} />
+        <Subnav title={content.title} headers={content.astro.headers} />
       </section>
     </div>
   </BaseLayout>

examples/snowpack/src/layouts/content.astro

@@ -98,7 +98,7 @@ export let content: any;
         </aside>
 
         <div class="pt4">
-          <Subnav title={content.title} headers={content.headers} />
+          <Subnav title={content.title} headers={content.astro.headers} />
         </div>
 
         <article class="layout-main">

packages/astro/src/compiler/index.ts

@@ -112,7 +112,7 @@ export async function compileComponent(
   const site = compileOptions.astroConfig.buildOptions.site || `http://localhost:${compileOptions.astroConfig.devOptions.port}`;
 
   // return template
-  let modJsx = `
+  let moduleJavaScript = `
 import fetch from 'node-fetch';
 
 // <script astro></script>
@@ -160,7 +160,7 @@ export async function __renderPage({request, children, props}) {
 
   return {
     result,
-    contents: modJsx,
+    contents: moduleJavaScript,
     css: result.css,
   };
 }

packages/astro/src/compiler/markdown/codeblock.ts

@@ -3,7 +3,9 @@ import { visit } from 'unist-util-visit';
 /**  */
 export function remarkCodeBlock() {
   const visitor = (node: any) => {
-    const { data, lang, meta } = node;
+    const { data, meta } = node;
+    let lang = node.lang || 'html'; // default to html matches GFM behavior.
+
     let currentClassName = data?.hProperties?.class ?? '';
     node.data = node.data || {};
     node.data.hProperties = node.data.hProperties || {};

packages/astro/src/frontend/h.ts

@@ -18,9 +18,8 @@ function* _h(tag: string, attrs: HProps, children: Array<HChild>) {
 
   yield `<${tag}`;
   if (attrs) {
-    yield ' ';
     for (let [key, value] of Object.entries(attrs)) {
-      yield `${key}="${value}"`;
+      yield ` ${key}="${value}"`;
     }
   }
   yield '>';

packages/astro/test/fixtures/plain-markdown/src/pages/post.md

@@ -8,4 +8,10 @@ description: This is a post about some stuff.
 
 Hello world!
 
+```json
+{ 
+  "key": "value"
+}
+```
+
 <div id="first">Some content</div>

packages/astro/test/snowpack-integration.test.js

@@ -10,14 +10,16 @@ const { readdir, stat } = fsPromises;
 
 const SnowpackDev = suite('snowpack.dev');
 
+const snowpackDir = new URL('../../../examples/snowpack/', import.meta.url);
+
 let runtime, cwd, setupError;
 
 SnowpackDev.before(async () => {
   // Bug: Snowpack config is still loaded relative to the current working directory.
   cwd = process.cwd();
-  process.chdir(fileURLToPath(new URL('../examples/snowpack/', import.meta.url)));
+  process.chdir(fileURLToPath(snowpackDir));
 
-  const astroConfig = await loadConfig(fileURLToPath(new URL('../examples/snowpack', import.meta.url)));
+  const astroConfig = await loadConfig(fileURLToPath(snowpackDir));
 
   const logging = {
     level: 'error',
@@ -67,7 +69,7 @@ SnowpackDev('No error creating the runtime', () => {
 SnowpackDev('Can load every page', async () => {
   const failed = [];
 
-  const pageRoot = new URL('../examples/snowpack/src/pages/', import.meta.url);
+  const pageRoot = new URL('./src/pages/', snowpackDir);
   for await (let pathname of allPages(pageRoot)) {
     if (pathname.includes('proof-of-concept-dynamic')) {
       continue;