Skip to content

docs: fix printers.md claiming a printer is the last pipeline step - #165

Merged
stijnvanhulle merged 1 commit into
mainfrom
claude/docs-validation-audit-8upeg0
Jul 19, 2026
Merged

docs: fix printers.md claiming a printer is the last pipeline step#165
stijnvanhulle merged 1 commit into
mainfrom
claude/docs-validation-audit-8upeg0

Conversation

@stijnvanhulle

Copy link
Copy Markdown
Contributor

🎯 Changes

snippets/how-to/printers.md (included into docs/5.x/guide/going-further/printers.md) opened with "A printer is the last step of Kubb's pipeline." That's wrong: a printer turns one schema node into code inside a plugin's generator, well before the pipeline actually ends. Traced the real order in kubb-labs/kubb:

  1. A printer runs per schema node, inside a plugin's generator (plugin-ts/plugin-zod/plugin-faker's printer.nodes).
  2. A parser then converts the assembled FileNode into source text (confirmed against packages/parser-ts, which uses the TypeScript compiler's own printer internally to stringify the AST — a different, lower-level "printer" than Kubb's PrinterNodes concept).
  3. Storage writes that text to disk.
  4. Only after all of that does the CLI run the formatter, linter, and postGenerate commands (packages/cli/src/runners/generate/run.ts's processOutput).

Corrected the opening sentence and added the missing steps after printing, so the page no longer implies printing is the final thing that happens to generated code.

✅ Checklist

  • I have followed the steps in the Contributing guide.
  • I am using USA English (serialize, color, center, canceled, analyze, license).
  • I ran the humanizer check on any new documentation I wrote.

Generated by Claude Code

A printer runs inside a plugin's generator, well before the pipeline ends:
a parser still has to turn the assembled file into source text, storage
writes it to disk, and only then do the CLI's formatter, linter, and
postGenerate commands run (packages/cli/src/runners/generate/run.ts).
@changeset-bot

changeset-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5f8e0e5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@stijnvanhulle
stijnvanhulle merged commit a3d01fc into main Jul 19, 2026
1 check passed
@github-actions
github-actions Bot deleted the claude/docs-validation-audit-8upeg0 branch July 19, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants