Slug the theme name from package.json

[andreascreten]

The zipping fails when you have a more complex name defined in your package.json. Slugging prevents the zipping from crashing.

For example, we had a package named madewithlove/theme.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The changes introduce the slug package as a new dependency and integrate it into the theme name derivation logic. In index.js, when no zip path is provided, the code now processes the package name through the slug() function before assigning it as the theme name. The package.json dependency section is updated to include "slug": "^8.2.2". All other functionality in the zip creation and deployment workflow remains unaltered.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}