Skip to content

Commit 602842d

Browse files
committed
docs: Add HTML authoring skill release artifact
1 parent f5d7a71 commit 602842d

6 files changed

Lines changed: 167 additions & 0 deletions

File tree

.agents/SKILL.md

Lines changed: 102 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,102 @@
1+
---
2+
name: obsidian-html-docs
3+
description: Author HTML files for the Obsidian HTML Docs plugin (smcllns/obsidian-plugin-html-docs). Use when creating .html docs intended to render inline in Obsidian: theme tokens, sandbox constraints, asset rules, embed sizing, and things that silently do not work.
4+
---
5+
6+
# Obsidian HTML Docs authoring guide
7+
8+
The HTML Docs plugin lets users view `.html` files inside Obsidian tabs, markdown embeds, and Canvas cards. It renders each file in a sandboxed iframe via Blob URL. The iframe is sealed (`sandbox="allow-scripts allow-popups allow-forms"`, no `allow-same-origin`) so it cannot reach into Obsidian or the vault. Everything below works within that envelope.
9+
10+
## Default to Obsidian context unless asked otherwise
11+
12+
Before creating the Blob URL, the plugin appends a `<style data-html-docs-theme>` block to your HTML containing a snapshot of the user's current Obsidian design tokens. The iframe does not reach out for these values. They are written as static CSS into your document at load time, then the sandboxed iframe loads that document. **Security boundary unchanged**: no `allow-same-origin`, no reads from the iframe back into Obsidian.
13+
14+
If the user just wants a styled doc that "fits" their vault, use these as your defaults. If they're asking for a specific aesthetic (brutalist poster, retro terminal, Linear-style, an exact brand palette), design freely — Obsidian context is a *hint*, not a constraint.
15+
16+
Injected as static CSS:
17+
18+
- A `color-scheme: light | dark` declaration matching the user's current Obsidian theme
19+
- `--obsidian-color-scheme`
20+
- `--obsidian-bg`
21+
- `--obsidian-bg-2`
22+
- `--obsidian-text`
23+
- `--obsidian-text-muted`
24+
- `--obsidian-accent`
25+
- `--obsidian-border`
26+
- `--obsidian-font`
27+
- `--obsidian-font-mono`
28+
29+
Use Obsidian tokens if present, fall back if not:
30+
31+
```css
32+
:root {
33+
color-scheme: light dark;
34+
--bg: var(--obsidian-bg, light-dark(#ffffff, #0e1014));
35+
--text: var(--obsidian-text, light-dark(#16161a, #e7e9ec));
36+
}
37+
```
38+
39+
Open HTML tabs and embeds re-render when the Obsidian theme changes, so the injected snapshot follows theme switches.
40+
41+
## Assets: vault paths do not cross into the iframe
42+
43+
The iframe has no base URL pointing into the vault. Obsidian themes, snippets, and `attachments/...` images don't reach the iframe.
44+
45+
| Pattern | Works? |
46+
|---|---|
47+
| `<img src="attachments/foo.png">` | No — fails silently |
48+
| `<img src="data:image/png;base64,...">` | Yes — fully self-contained |
49+
| `<img src="https://example.com/foo.png">` | Yes — CORS permitting |
50+
| Inline `<svg>...</svg>` | Yes — best for icons / diagrams |
51+
| `<link rel="stylesheet" href="https://cdn...">` | Yes — HTTPS only |
52+
53+
Rules of thumb:
54+
55+
- Small graphics & icons → inline SVG or `data:` URL
56+
- Photos / large images → upload to a host (R2, CDN) and reference HTTPS
57+
- Fonts → system stack, or HTTPS CDN
58+
- Never reference `attachments/` or any vault path — the iframe can't see them
59+
60+
## What works
61+
62+
- HTML / CSS (grid, `light-dark()`, custom properties, animations, gradients, SVG with CSS animations)
63+
- JavaScript (ES2020+, fetch with CORS, Promises, DOM events, requestAnimationFrame, Canvas 2D)
64+
- Forms (`allow-forms` is set; intercept `submit` if you don't want navigation)
65+
- `window.parent.postMessage(msg, '*')` — works even with opaque origin
66+
- External HTTPS resources (images, fonts on CDNs, fetch APIs that allow CORS)
67+
- Anchor links (`#section`) and the History API — Blob URL preserves both
68+
69+
## What's blocked
70+
71+
- `localStorage`, `sessionStorage`, `IndexedDB``SecurityError`
72+
- `document.cookie` — throws or silently no-ops
73+
- Reading `window.parent.*` — cross-origin (postMessage still works)
74+
- Service workers, geolocation, clipboard, notifications, most permission-gated APIs
75+
- Top-level navigation from inside the iframe (no `allow-top-navigation`)
76+
- Vault-relative URLs (see Assets above)
77+
78+
## Linking from markdown to HTML
79+
80+
Wikilinks to `.html` files need the explicit extension:
81+
82+
```markdown
83+
See: [[my-doc.html]]
84+
```
85+
86+
`[[my-doc]]` won't resolve to `.html` in stock Obsidian.
87+
88+
## Embed sizing
89+
90+
```markdown
91+
![[doc.html|600x400]]
92+
```
93+
94+
Sets the embed width and height. Default markdown embed height is about 600px; tab views fill the pane.
95+
96+
## Pitfalls (skip the turn cost)
97+
98+
- **Theme scripts that read `window.parent`** — always throw under this plugin's sandbox. Don't write them.
99+
- **Images via `attachments/foo.png`** — won't resolve. Inline as data URL or use HTTPS.
100+
- **`localStorage` / cookies for state** — blocked. Use URL hash or postMessage to parent.
101+
- **Hard-coding a palette when Obsidian tokens are available** — defeats the contextual fit. Use `var(--obsidian-*)` with `light-dark()` fallbacks.
102+
- **Assuming `prefers-color-scheme` == Obsidian theme** — use `--obsidian-color-scheme` or injected colors instead.
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
# HTML Docs authoring skill release
2+
3+
Scope:
4+
- [x] Sync `main` to merged PR #8.
5+
- [x] Rebase PR #7 branch onto theme-token work.
6+
- [x] Update canonical `.agents/SKILL.md` for real theme-token injection.
7+
- [x] Bundle `SKILL.md` into `dist/html-docs/` during build.
8+
- [x] Include bundled `SKILL.md` in release upload/provenance.
9+
- [x] Clarify README optional agent-skill install/discovery.
10+
- [x] Mirror skill to `smcllns/skills`.
11+
- [x] Run build/lint/test checks.
12+
- [x] Push PR branches.
13+
- [x] Merge `smcllns/skills` mirror first so PR #7 docs can point at a real install URL.
14+
- [x] Add `skills.sh` install command to README.
15+
16+
Decisions:
17+
- Plugin repo is canonical source.
18+
- `smcllns/skills` is the installable mirror.
19+
- Obsidian plugin install does not install agent skills.
20+
- Plugin runtime must not write to agent install roots or dotfiles.
21+
- Merge/install order matters: public skills mirror first, plugin docs/release second.
22+
23+
Unresolved questions:
24+
- None.

.github/workflows/release.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ jobs:
3838
subject-path: |
3939
dist/html-docs/main.js
4040
dist/html-docs/manifest.json
41+
dist/html-docs/SKILL.md
4142
dist/html-docs/styles.css
4243
4344
- name: Create GitHub Release
@@ -50,4 +51,5 @@ jobs:
5051
--generate-notes \
5152
dist/html-docs/main.js \
5253
dist/html-docs/manifest.json \
54+
dist/html-docs/SKILL.md \
5355
dist/html-docs/styles.css

README.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,13 @@ A demo page (`test/fixture.html`) demonstrates all the passing HTML features.
3030

3131
Releases are built and signed by GitHub Actions ([.github/workflows/release.yml](.github/workflows/release.yml)) so the binaries carry a [build attestation](https://docs.github.com/en/actions/security-for-github-actions/using-artifact-attestations/using-artifact-attestations-to-establish-provenance-for-builds) you can verify against the source.
3232

33+
### Optional Agent Skill
34+
35+
There is an optional agent skill so your agent can author HTML that fits this plugin's sandbox, theme tokens, assets, and embeds.
36+
37+
- Skills CLI: `npx skills add smcllns/skills --skill obsidian-html-docs`
38+
- Manual: [smcllns/skills/skills/obsidian-html-docs/SKILL.md](https://github.com/smcllns/skills/blob/main/skills/obsidian-html-docs/SKILL.md)
39+
3340
### Build and install from source
3441

3542
```bash
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
# Authoring Skill Release Handoff
2+
3+
Goal:
4+
- Finish PR #7 as a small follow-up to merged PR #8.
5+
- Make the HTML authoring skill useful after theme-token injection shipped.
6+
- Be honest that Obsidian plugin installs do not activate agent skills.
7+
8+
Implemented shape:
9+
- `.agents/SKILL.md` is the canonical source copy in this plugin repo.
10+
- `esbuild.config.mjs` copies `.agents/SKILL.md` to `dist/html-docs/SKILL.md`.
11+
- The release workflow uploads and attests `dist/html-docs/SKILL.md` alongside runtime files.
12+
- README has an optional agent-skill note explaining discovery roots and the `smcllns/skills` mirror.
13+
- `smcllns/skills` mirrors the canonical skill at `skills/obsidian-html-docs/SKILL.md`.
14+
- `smcllns/skills` PR #1 was merged first so README can point to the real `skills.sh` install URL.
15+
16+
Important constraint:
17+
- Plugin runtime must not write to agent install roots, dotfiles, or any other user-level configuration. Agent skill activation belongs to the user's agent tooling, not the Obsidian plugin.
18+
- Merge order matters: public skills mirror first, then plugin docs/release work.
19+
20+
Verification target:
21+
- `bun run build`
22+
- `bun run lint`
23+
- `bun run test`
24+
- Confirm `dist/html-docs/SKILL.md` matches `.agents/SKILL.md`.
25+
26+
Current verification:
27+
- `bun run build` passed.
28+
- `bun run lint` passed.
29+
- `bun run test` passed against live Obsidian.
30+
- `.agents/SKILL.md`, generated `dist/html-docs/SKILL.md`, and the `smcllns/skills` mirror match.
31+
- Haiku critical review found one stale `light-dark()` wording mismatch; fixed by keeping `light-dark()` in the supported CSS list.

esbuild.config.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ const distDir = path.join("dist", "html-docs");
1616
fs.mkdirSync(distDir, { recursive: true });
1717
fs.copyFileSync("manifest.json", path.join(distDir, "manifest.json"));
1818
fs.copyFileSync("styles.css", path.join(distDir, "styles.css"));
19+
fs.copyFileSync(path.join(".agents", "SKILL.md"), path.join(distDir, "SKILL.md"));
1920

2021
const context = await esbuild.context({
2122
banner: { js: banner },

0 commit comments

Comments
 (0)