Skip to content

docs(vite): cite TEMPLATE_NAME_PATTERN from @samva/markup/compiler - #17

Merged
aryasaatvik merged 1 commit into
mainfrom
cursor/fix-template-name-pattern-docs-a1f2
Oct 8, 2026
Merged

aryasaatvik merged 1 commit into
mainfrom
cursor/fix-template-name-pattern-docs-a1f2

Conversation

@aryasaatvik

Copy link
Copy Markdown
Member

Summary

packages/vite/AGENTS.md cites TEMPLATE_NAME_PATTERN from @samva/markup/compiler, the subpath packages/vite/src/project.ts imports.

Before / After

- **Template ids and fixture keys match `TEMPLATE_NAME_PATTERN`** from `@samva/markup/template`,
+ **Template ids and fixture keys match `TEMPLATE_NAME_PATTERN`** from `@samva/markup/compiler`,

@samva/markup publishes no ./template export. packages/markup/src/compiler.ts re-exports the constant.

Open in Web Open in Cursor 

Co-authored-by: Saatvik Arya <aryasaatvik@gmail.com>
@pkg-pr-new

pkg-pr-new Bot commented Oct 6, 2026

Copy link
Copy Markdown

Open in StackBlitz

bun add https://pkg.pr.new/@samva/editor@17
bun add https://pkg.pr.new/@samva/markup@17
bun add https://pkg.pr.new/@samva/vite@17

commit: 3c9fe86

@greptile-apps

greptile-apps Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low risk] Updates a documentation reference to a package name.

This documentation correction appears safe to merge.

What we checked:

  • Documented import exists: Yes. The package exposes ./compiler, which re-exports the constant. project.ts already uses that import.

Summary

Corrects the source of TEMPLATE_NAME_PATTERN in packages/vite/AGENTS.md from @samva/markup/template to @samva/markup/compiler.

  • The new reference matches the public export and the existing import in project.ts.
  • No actionable issues found.

Reviews (1) · Last reviewed commit: "docs(vite): cite TEMPLATE_NAME_PATTERN f..."

@aryasaatvik
aryasaatvik merged commit 0b8f4e0 into main Oct 8, 2026
4 checks passed
@aryasaatvik
aryasaatvik deleted the cursor/fix-template-name-pattern-docs-a1f2 branch October 8, 2026 19:14
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