Skip to content

Move turbo/**/*.hbs oxfmt ignore to root config#1604

Draft
stipsan wants to merge 1 commit into
mainfrom
cursor/oxfmt-turbo-ignore-root-f476
Draft

Move turbo/**/*.hbs oxfmt ignore to root config#1604
stipsan wants to merge 1 commit into
mainfrom
cursor/oxfmt-turbo-ignore-root-f476

Conversation

@stipsan

@stipsan stipsan commented Jul 16, 2026

Copy link
Copy Markdown
Member

Summary

Moves the turbo/**/*.hbs oxfmt ignore pattern out of the shared @sanity/plugin-kit/oxfmt preset and into the monorepo root oxfmt.config.ts.

The plugin-kit preset should only contain formatting rules and ignore patterns that apply broadly to Sanity plugins (like dist/** and pnpm-lock.yaml). Turbo generator Handlebars templates are specific to this monorepo's scaffolding setup.

Changes

  • packages/@sanity/plugin-kit/src/oxfmt.ts: Remove turbo/**/*.hbs from ignorePatterns
  • oxfmt.config.ts: Spread the plugin-kit preset and add the monorepo-specific ignore (mirrors how oxlint.config.ts handles workspace-specific patterns)
  • AGENTS.md: Document that root oxfmt.config.ts extends the preset with workspace-specific ignores

Testing

  • pnpm format
  • pnpm lint
Open in Web Open in Cursor 

The turbo generator template ignore is monorepo-specific and should
not ship in the shared @sanity/plugin-kit/oxfmt preset consumed by
standalone plugins.
@stipsan stipsan added the 🤖 bot label Jul 16, 2026 — with Cursor
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
plugins-studio Ready Ready Preview, Comment Jul 16, 2026 9:28am

Request Review

@changeset-bot

changeset-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 39d7225

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sanity/plugin-kit Patch

Not sure what this means? Click here to learn what changesets are.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants