Skip to content

Conversation

@dopry
Copy link
Owner

@dopry dopry commented Nov 1, 2025

fixes #56

Add missing exports condition in package.json per
https://svelte.dev/docs/kit/packaging#TypeScript

@dopry dopry requested a review from Copilot November 1, 2025 23:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the package configuration to align with modern SvelteKit packaging conventions by reorganizing the exports field and updating documentation. The changes improve package structure and remove trailing whitespace issues.

Key changes:

  • Adds a proper exports field with Svelte-specific entry points
  • Moves the svelte field to a more logical position in package.json
  • Updates documentation with references to official Svelte packaging guidelines
  • Cleans up trailing whitespace in README

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
package.json Reorganizes package exports by adding an exports field and relocating the svelte field for better adherence to SvelteKit packaging standards
README.md Adds documentation referencing official Svelte packaging guidelines and removes trailing whitespace from multiple lines

@dopry dopry force-pushed the fix/56-missing-export-condition branch from 282aed2 to f53f407 Compare November 2, 2025 01:14
@dopry dopry requested a review from Copilot November 2, 2025 01:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

fixes #56

- Add missing exports condition in package.json per
https://svelte.dev/docs/kit/packaging#TypeScript

- Add export conditions for cjs and mjs as well.
@dopry dopry force-pushed the fix/56-missing-export-condition branch from f53f407 to cd374df Compare November 2, 2025 01:18
@dopry dopry requested a review from Copilot November 2, 2025 01:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@dopry dopry merged commit e2243a4 into next Nov 2, 2025
7 checks passed
@dopry dopry deleted the fix/56-missing-export-condition branch November 2, 2025 01:20
@github-actions
Copy link

github-actions bot commented Nov 2, 2025

🎉 This PR is included in version 1.2.0-next.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Nov 2, 2025

🎉 This PR is included in version 1.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Missing exports condition warning from vite-plugin-svelte

2 participants