Skip to content

refactor(plugin-seo): rename non-JSX files from .tsx to .ts#14418

Merged
paulpopus merged 5 commits into
payloadcms:mainfrom
jhb-dev:refactor/plugin-seo/index.tsx
Apr 8, 2026
Merged

refactor(plugin-seo): rename non-JSX files from .tsx to .ts#14418
paulpopus merged 5 commits into
payloadcms:mainfrom
jhb-dev:refactor/plugin-seo/index.tsx

Conversation

@jhb-dev

@jhb-dev jhb-dev commented Oct 30, 2025

Copy link
Copy Markdown
Contributor

What?

Renamed three non-JSX source files in @payloadcms/plugin-seo from .tsx to .ts extension and updated package.json accordingly.

Why?

These files don't contain any JSX syntax, they only export TypeScript functions and type definitions. Using the .tsx extension was incorrect and inconsistent with all other Payload plugins, which use .ts for their main entry points.

How?

  • Simple file renames with zero code changes
  • Updated package.json exports to reference ./src/index.ts instead of ./src/index.tsx

@paulpopus paulpopus enabled auto-merge (squash) April 8, 2026 13:59
@paulpopus paulpopus merged commit b67882c into payloadcms:main Apr 8, 2026
157 checks passed
@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

🚀 This is included in version v3.82.0

milamer pushed a commit to milamer/payload that referenced this pull request Apr 20, 2026
…ms#14418)

### What?

Renamed three non-JSX source files in `@payloadcms/plugin-seo` from
`.tsx` to `.ts` extension and updated `package.json` accordingly.

###  Why?

These files don't contain any JSX syntax, they only export TypeScript
functions and type definitions. Using the `.tsx` extension was incorrect
and inconsistent with all other Payload plugins, which use `.ts` for
their main entry points.

###  How?
- Simple file renames with zero code changes
- Updated package.json exports to reference `./src/index.ts` instead of
`./src/index.tsx`

Co-authored-by: Paul Popus <paul@payloadcms.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants