Skip to content

Commit

Permalink
Markdoc - Shiki (#7187)
Browse files Browse the repository at this point in the history
* chore: remove unused util

* chore: changeset

* deps: shiki

* wip: first stab at shiki markdoc config

* feat: get shiki working!

* refactor: return HTML string directly from transform

* chore: move shiki to markdoc dev dep

* refactor: use async cache with clear docs on why

* test: transform units with Shiki config options

* refactor: switch to `extends` model

* refactor: nodes/ -> extensions/

* feat: raise friendly error for Promise extensions

* docs: README

* chore: lint

* chore: dead file

* chore: lowercase for fuzzy find please

* fix: bad ctx spread

* chore: clean up cache, add shiki imp error

* chore: add shiki to optional peer deps

* chore: hoist those consts

* docs: more explicit "install shiki now please"

Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>

* oops bad find and replace

* chore: update changeset

* nit: period haunts me

---------

Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
  • Loading branch information
bholmesdev and sarah11918 authored May 24, 2023
1 parent 7851f92 commit 1efaef6
Show file tree
Hide file tree
Showing 13 changed files with 383 additions and 35 deletions.
17 changes: 17 additions & 0 deletions .changeset/eleven-tables-speak.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
---
'@astrojs/markdoc': patch
---

Add support for syntax highlighting with Shiki. Install `shiki` in your project with `npm i shiki`, and apply to your Markdoc config using the `extends` option:

```js
// markdoc.config.mjs
import { defineMarkdocConfig, shiki } from '@astrojs/markdoc/config';
export default defineMarkdocConfig({
extends: [
await shiki({ /** Shiki config options */ }),
],
})
```

Learn more in the [`@astrojs/markdoc` README.](https://docs.astro.build/en/guides/integrations-guide/markdoc/#syntax-highlighting)
9 changes: 8 additions & 1 deletion packages/astro/src/runtime/server/index.ts
Original file line number Diff line number Diff line change
@@ -1,7 +1,14 @@
export { createComponent } from './astro-component.js';
export { createAstro } from './astro-global.js';
export { renderEndpoint } from './endpoint.js';
export { escapeHTML, HTMLBytes, HTMLString, markHTMLString, unescapeHTML } from './escape.js';
export {
escapeHTML,
HTMLBytes,
HTMLString,
markHTMLString,
unescapeHTML,
isHTMLString,
} from './escape.js';
export { renderJSX } from './jsx.js';
export {
addAttribute,
Expand Down
35 changes: 35 additions & 0 deletions packages/integrations/markdoc/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,41 @@ export default defineMarkdocConfig({
})
```

### Syntax highlighting

`@astrojs/markdoc` provides a [Shiki](https://github.com/shikijs/shiki) extension to highlight your code blocks.

To use this extension, you must separately install `shiki` as a dependency:

```bash
npm i shiki
```

Then, apply the `shiki()` extension to your Markdoc config using the `extends` property. You can optionally pass a shiki configuration object:

```js
// markdoc.config.mjs
import { defineMarkdocConfig, shiki } from '@astrojs/markdoc/config';

export default defineMarkdocConfig({
extends: [
await shiki({
// Choose from Shiki's built-in themes (or add your own)
// Default: 'github-dark'
// https://github.com/shikijs/shiki/blob/main/docs/themes.md
theme: 'dracula',
// Enable word wrap to prevent horizontal scrolling
// Default: false
wrap: true,
// Pass custom languages
// Note: Shiki has countless langs built-in, including `.astro`!
// https://github.com/shikijs/shiki/blob/main/docs/languages.md
langs: [],
})
],
})
```

### Access frontmatter and content collection information from your templates

You can access content collection information from your Markdoc templates using the `$entry` variable. This includes the entry `slug`, `collection` name, and frontmatter `data` parsed by your content collection schema (if any). This example renders the `title` frontmatter property as a heading:
Expand Down
15 changes: 12 additions & 3 deletions packages/integrations/markdoc/components/TreeNode.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,18 @@ import type { AstroInstance } from 'astro';
import { Fragment } from 'astro/jsx-runtime';
import type { RenderableTreeNode } from '@markdoc/markdoc';
import Markdoc from '@markdoc/markdoc';
import { createComponent, renderComponent, render } from 'astro/runtime/server/index.js';
import {
createComponent,
renderComponent,
render,
HTMLString,
isHTMLString,
} from 'astro/runtime/server/index.js';

export type TreeNode =
| {
type: 'text';
content: string;
content: string | HTMLString;
}
| {
type: 'component';
Expand All @@ -25,6 +31,7 @@ export type TreeNode =
export const ComponentNode = createComponent({
factory(result: any, { treeNode }: { treeNode: TreeNode }) {
if (treeNode.type === 'text') return render`${treeNode.content}`;

const slots = {
default: () =>
render`${treeNode.children.map((child) =>
Expand All @@ -46,7 +53,9 @@ export const ComponentNode = createComponent({
});

export function createTreeNode(node: RenderableTreeNode | RenderableTreeNode[]): TreeNode {
if (typeof node === 'string' || typeof node === 'number') {
if (isHTMLString(node)) {
return { type: 'text', content: node as HTMLString };
} else if (typeof node === 'string' || typeof node === 'number') {
return { type: 'text', content: String(node) };
} else if (Array.isArray(node)) {
return {
Expand Down
11 changes: 9 additions & 2 deletions packages/integrations/markdoc/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@astrojs/markdoc",
"description": "Add support for Markdoc pages in your Astro site",
"description": "Add support for Markdoc in your Astro site",
"version": "0.2.3",
"type": "module",
"types": "./dist/index.d.ts",
Expand Down Expand Up @@ -47,7 +47,13 @@
"zod": "^3.17.3"
},
"peerDependencies": {
"astro": "workspace:^2.5.5"
"astro": "workspace:^2.5.5",
"shiki": "^0.14.1"
},
"peerDependenciesMeta": {
"shiki": {
"optional": true
}
},
"devDependencies": {
"@astrojs/markdown-remark": "^2.2.1",
Expand All @@ -61,6 +67,7 @@
"linkedom": "^0.14.12",
"mocha": "^9.2.2",
"rollup": "^3.20.1",
"shiki": "^0.14.1",
"vite": "^4.3.1"
},
"engines": {
Expand Down
15 changes: 12 additions & 3 deletions packages/integrations/markdoc/src/config.ts
Original file line number Diff line number Diff line change
@@ -1,10 +1,19 @@
import type { ConfigType as MarkdocConfig } from '@markdoc/markdoc';
import _Markdoc from '@markdoc/markdoc';
import { nodes as astroNodes } from './nodes/index.js';
import { heading } from './heading-ids.js';

export type AstroMarkdocConfig<C extends Record<string, any> = Record<string, any>> =
MarkdocConfig & {
ctx?: C;
extends?: ResolvedAstroMarkdocConfig[];
};

export type ResolvedAstroMarkdocConfig = Omit<AstroMarkdocConfig, 'extends'>;

export const Markdoc = _Markdoc;
export const nodes = { ...Markdoc.nodes, ...astroNodes };
export const nodes = { ...Markdoc.nodes, heading };
export { shiki } from './extensions/shiki.js';

export function defineMarkdocConfig(config: MarkdocConfig): MarkdocConfig {
export function defineMarkdocConfig(config: AstroMarkdocConfig): AstroMarkdocConfig {
return config;
}
138 changes: 138 additions & 0 deletions packages/integrations/markdoc/src/extensions/shiki.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,138 @@
// @ts-expect-error Cannot find module 'astro/runtime/server/index.js' or its corresponding type declarations.
import { unescapeHTML } from 'astro/runtime/server/index.js';
import type { ShikiConfig } from 'astro';
import type * as shikiTypes from 'shiki';
import type { AstroMarkdocConfig } from '../config.js';
import Markdoc from '@markdoc/markdoc';
import { MarkdocError } from '../utils.js';

// Map of old theme names to new names to preserve compatibility when we upgrade shiki
const compatThemes: Record<string, string> = {
'material-darker': 'material-theme-darker',
'material-default': 'material-theme',
'material-lighter': 'material-theme-lighter',
'material-ocean': 'material-theme-ocean',
'material-palenight': 'material-theme-palenight',
};

const normalizeTheme = (theme: string | shikiTypes.IShikiTheme) => {
if (typeof theme === 'string') {
return compatThemes[theme] || theme;
} else if (compatThemes[theme.name]) {
return { ...theme, name: compatThemes[theme.name] };
} else {
return theme;
}
};

const ASTRO_COLOR_REPLACEMENTS = {
'#000001': 'var(--astro-code-color-text)',
'#000002': 'var(--astro-code-color-background)',
'#000004': 'var(--astro-code-token-constant)',
'#000005': 'var(--astro-code-token-string)',
'#000006': 'var(--astro-code-token-comment)',
'#000007': 'var(--astro-code-token-keyword)',
'#000008': 'var(--astro-code-token-parameter)',
'#000009': 'var(--astro-code-token-function)',
'#000010': 'var(--astro-code-token-string-expression)',
'#000011': 'var(--astro-code-token-punctuation)',
'#000012': 'var(--astro-code-token-link)',
};

const PRE_SELECTOR = /<pre class="(.*?)shiki(.*?)"/;
const LINE_SELECTOR = /<span class="line"><span style="(.*?)">([\+|\-])/g;
const INLINE_STYLE_SELECTOR = /style="(.*?)"/;

/**
* Note: cache only needed for dev server reloads, internal test suites, and manual calls to `Markdoc.transform` by the user.
* Otherwise, `shiki()` is only called once per build, NOT once per page, so a cache isn't needed!
*/
const highlighterCache = new Map<string, shikiTypes.Highlighter>();

export async function shiki({
langs = [],
theme = 'github-dark',
wrap = false,
}: ShikiConfig = {}): Promise<AstroMarkdocConfig> {
let getHighlighter: (options: shikiTypes.HighlighterOptions) => Promise<shikiTypes.Highlighter>;
try {
getHighlighter = (await import('shiki')).getHighlighter;
} catch {
throw new MarkdocError({
message: 'Shiki is not installed. Run `npm install shiki` to use the `shiki` extension.',
});
}
theme = normalizeTheme(theme);

const cacheID: string = typeof theme === 'string' ? theme : theme.name;
if (!highlighterCache.has(cacheID)) {
highlighterCache.set(
cacheID,
await getHighlighter({ theme }).then((hl) => {
hl.setColorReplacements(ASTRO_COLOR_REPLACEMENTS);
return hl;
})
);
}
const highlighter = highlighterCache.get(cacheID)!;

for (const lang of langs) {
await highlighter.loadLanguage(lang);
}
return {
nodes: {
fence: {
attributes: Markdoc.nodes.fence.attributes!,
transform({ attributes }) {
let lang: string;

if (typeof attributes.language === 'string') {
const langExists = highlighter
.getLoadedLanguages()
.includes(attributes.language as any);
if (langExists) {
lang = attributes.language;
} else {
// eslint-disable-next-line no-console
console.warn(
`[Shiki highlighter] The language "${attributes.language}" doesn't exist, falling back to plaintext.`
);
lang = 'plaintext';
}
} else {
lang = 'plaintext';
}

let html = highlighter.codeToHtml(attributes.content, { lang });

// Q: Could these regexes match on a user's inputted code blocks?
// A: Nope! All rendered HTML is properly escaped.
// Ex. If a user typed `<span class="line"` into a code block,
// It would become this before hitting our regexes:
// &lt;span class=&quot;line&quot;

html = html.replace(PRE_SELECTOR, `<pre class="$1astro-code$2"`);
// Add "user-select: none;" for "+"/"-" diff symbols
if (attributes.language === 'diff') {
html = html.replace(
LINE_SELECTOR,
'<span class="line"><span style="$1"><span style="user-select: none;">$2</span>'
);
}

if (wrap === false) {
html = html.replace(INLINE_STYLE_SELECTOR, 'style="$1; overflow-x: auto;"');
} else if (wrap === true) {
html = html.replace(
INLINE_STYLE_SELECTOR,
'style="$1; overflow-x: auto; white-space: pre-wrap; word-wrap: break-word;"'
);
}

// Use `unescapeHTML` to return `HTMLString` for Astro renderer to inline as HTML
return unescapeHTML(html);
},
},
},
};
}
Original file line number Diff line number Diff line change
@@ -1,13 +1,8 @@
import Markdoc, { type ConfigType, type RenderableTreeNode, type Schema } from '@markdoc/markdoc';
import Slugger from 'github-slugger';
import { getTextContent } from '../runtime.js';

type ConfigTypeWithCtx = ConfigType & {
// TODO: decide on `ctx` as a convention for config merging
ctx: {
headingSlugger: Slugger;
};
};
import { getTextContent } from './runtime.js';
import type { AstroMarkdocConfig } from './config.js';
import { MarkdocError } from './utils.js';

function getSlug(
attributes: Record<string, any>,
Expand All @@ -24,16 +19,31 @@ function getSlug(
return slug;
}

type HeadingIdConfig = AstroMarkdocConfig<{
headingSlugger: Slugger;
}>;

/*
Expose standalone node for users to import in their config.
Allows users to apply a custom `render: AstroComponent`
and spread our default heading attributes.
*/
export const heading: Schema = {
children: ['inline'],
attributes: {
id: { type: String },
level: { type: Number, required: true, default: 1 },
},
transform(node, config: ConfigTypeWithCtx) {
transform(node, config: HeadingIdConfig) {
const { level, ...attributes } = node.transformAttributes(config);
const children = node.transformChildren(config);

if (!config.ctx?.headingSlugger) {
throw new MarkdocError({
message:
'Unexpected problem adding heading IDs to Markdoc file. Did you modify the `ctx.headingSlugger` property in your Markdoc config?',
});
}
const slug = getSlug(attributes, children, config.ctx.headingSlugger);

const render = config.nodes?.heading?.render ?? `h${level}`;
Expand All @@ -49,9 +59,9 @@ export const heading: Schema = {
},
};

export function setupHeadingConfig(): ConfigTypeWithCtx {
// Called internally to ensure `ctx` is generated per-file, instead of per-build.
export function setupHeadingConfig(): HeadingIdConfig {
const headingSlugger = new Slugger();

return {
ctx: {
headingSlugger,
Expand Down
6 changes: 5 additions & 1 deletion packages/integrations/markdoc/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,11 @@ export default function markdocIntegration(legacyConfig?: any): AstroIntegration
async getRenderModule({ entry, viteId }) {
const ast = Markdoc.parse(entry.body);
const pluginContext = this;
const markdocConfig = setupConfig(userMarkdocConfig, entry);
const markdocConfig = setupConfig(
userMarkdocConfig,
entry,
markdocConfigResult?.fileUrl.pathname
);

const validationErrors = Markdoc.validate(ast, markdocConfig).filter((e) => {
return (
Expand Down
4 changes: 0 additions & 4 deletions packages/integrations/markdoc/src/nodes/index.ts

This file was deleted.

Loading

0 comments on commit 1efaef6

Please sign in to comment.