WORK-615
ID:WORK-615Status:done

Remove snippet's standalone figure wrapper

SPEC-141 D5/D6, shipped first and on its own. snippet runs in two phases: preprocess, and a postProcess pass, wrapStandaloneSnippets in packages/runes/src/config.ts, which wraps a standalone <pre> in <figure class="rf-snippet">. The wrapper adds no information: data-source and data-lines are already on the <pre>, and .rf-snippet's CSS only resets the margins the figure itself introduced.

Removing it makes snippet single-phase, so WORK-618 moves one concern into the tag module rather than half of one.

Breaking output change. Downstream CSS or tooling selecting .rf-snippet or [data-source-path] must move to pre[data-source]. Authors who want chrome compose it: {% codegroup title="…" %}, or {% figure %} once BUG-028 lands in this milestone.

Priority:mediumComplexity:simpleMilestone:v0.39.0Source:SPEC-141
main View source

Criteria completion

Criteria completion: 7 of 7 (100%) checked; history from Oct 7 to Oct 70%25%50%75%100%Oct 7Oct 7

Relationships

Implements 1
Related 4
Branches 5
main current done
claude/post-v0.38-roadmap-jb7pie donechangeset-release/main readyclaude/v039-tree-order-preprocess doneclaude/v039-registers doneclaude/v039-slot-labelling done
History 5
  1. 71499f7
    • ☑ `wrapStandaloneSnippets` is removed; a standalone snippet renders as `<pre data-source>` with no `<figure>` wrapper
    • ☑ Lumina's `.rf-snippet` rules are removed, and the CSS coverage test passes
    • ☑ A changeset records the output change and names the replacement selector, `pre[data-source]`
    • ☑ `{% snippet %}` inside `{% codegroup %}` and `{% diff %}` is unchanged
    • ☑ `refrakt contracts --check` drift is limited to the wrapper removal, both committed copies regenerated
    • ☑ `npm run seo:baseline:check` reports no drift
    • ☑ Docs that show or mention the figure wrapper (snippet rune page, SPEC-062-derived content) are updated — `refrakt.stale` on the touched files names them
    by bjornolofandersson
  2. 9870c2a
    Content editedby Claude
    docs(plan): close WORK-615, WORK-618 and BUG-027 with resolutions and PR
  3. b77900b
    Content editedby Claude
    test(runes): preprocessors resolve in tree order — failing (BUG-027)
  4. 50daca8
    Created (ready)by bjornolofandersson
  5. 1046f41
    Content editedby Claude
    docs(plan): add v0.39.0 milestone, Declarations part two

Acceptance Criteria

  • wrapStandaloneSnippets is removed; a standalone snippet renders as <pre data-source> with no <figure> wrapper
  • Lumina's .rf-snippet rules are removed, and the CSS coverage test passes
  • A changeset records the output change and names the replacement selector, pre[data-source]
  • {% snippet %} inside {% codegroup %} and {% diff %} is unchanged
  • refrakt contracts --check drift is limited to the wrapper removal, both committed copies regenerated
  • npm run seo:baseline:check reports no drift
  • Docs that show or mention the figure wrapper (snippet rune page, SPEC-062-derived content) are updated — refrakt.stale on the touched files names them

References

  • SPEC-141 — "Prerequisite: snippet must become single-phase", D5, D6
  • SPEC-062 — the snippet rune and the wrapper's original rationale

Resolution

Completed: 2026-10-07

Branch: claude/v039-tree-order-preprocess PR: refrakt-md/refrakt#667

What was done

  • packages/runes/src/snippet-pipeline.ts: removed wrapStandaloneSnippets / walkAndWrap / wrapPreInFigure and the fence-container allowlist (the module itself later went away in WORK-618).
  • packages/runes/src/config.ts: postProcess no longer calls the wrap step; the Snippet theme entry is kept for inspect/contracts tooling, like Data and Include.
  • packages/lumina/styles/runes/snippet.css deleted, import removed from index.css; snippet added to UNSTYLED_BLOCKS in the CSS coverage test.
  • packages/runes/src/file-ref-resolve.ts: the file-ref preview drawer reused figure.rf-snippet for its body; without the CSS that figure would take the UA 40px inline margin, so the drawer body is now the same bare <pre data-source data-lines>. Test, runes/file-ref.md and a drawer.css comment updated.
  • site/content/runes/snippet.md: new "Output" section — pre[data-source], the removed wrapper, and composition with {% codegroup %} (figure not documented until BUG-028 lands).
  • Changeset snippet-renders-as-a-bare-pre.md (breaking output change, names pre[data-source]).

Notes

  • Contracts: no drift on either copy. Because the Snippet config entry stays, the contract never described the figure, so there was nothing to regenerate. SEO baseline: no drift.
  • Tests: the snippet-pipeline wrap tests are replaced by one that runs the full core postProcess and asserts a bare <pre data-source data-lines>. It and the updated file-ref drawer assertion both fail on the pre-change code (verified by stashing the src change).
  • SPEC-062 still describes the wrapper; it is a shipped historical spec, superseded on this point by SPEC-141 D5, and was left as written.