Skip to content

feat: init addon#1

Open
paoloricciuti wants to merge 2 commits into
mainfrom
init-addon
Open

feat: init addon#1
paoloricciuti wants to merge 2 commits into
mainfrom
init-addon

Conversation

@paoloricciuti
Copy link
Copy Markdown
Collaborator

This initialize the addon with the minimum amount of file touched. I've added test coverage both for a clean install AND for an install where the files where already there.

Right now I didn't add any workflow to run those test but it's easy to do in a future PR.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Jun 4, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedvitest@​4.1.7981007999100
Added@​sveltejs/​sv-utils@​0.2.2811008195100
Added@​types/​node@​25.9.11001008195100
Addedsv@​0.15.3971008297100
Addedtsdown@​0.21.10981008896100
Added@​playwright/​test@​1.60.010010010099100

View full report

Copy link
Copy Markdown

@jycouet jycouet left a comment

Choose a reason for hiding this comment

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

Looks great already 🎉

Comment thread package.json
@@ -0,0 +1,42 @@
{
"name": "@msw/sveltekit",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
"name": "@msw/sveltekit",
"name": "@msw/sv",

Like this you can do : npx add @msw if you keep your name you will have to do npx add @msw/sveltekit

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yeah we talked about it with @kettanaito but he prefers this as it's more explicit

Comment thread src/index.js Outdated
Co-authored-by: jyc.dev <jycouet@gmail.com>
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