SocketCluster / socketcluster

Highly scalable realtime pub/sub and RPC framework
https://socketcluster.io
MIT License
6.15k stars 314 forks source link

`socketcluster create` screws up app name #572

Closed MegaGM closed 1 year ago

MegaGM commented 1 year ago

Today I've created an app and got something unexpected as a directory name

npm i -g socketcluster@17.2.0
socketcluster create Pyaterochka-013152-SKU-auditor

Expected result

Pyaterochka-013152-SKU-auditor/

Actual result

-pyaterochka013152-s-k-u-auditor/

I've tried something like git bisect but, instead of commits, with socketcluster npm versions. And have found out that v16.1.3 socketcluster create doesn't screw up app name, and all versions starting from v16.2.0 do.

MegaGM commented 1 year ago

So far I've narrowed the possible culprits to
@maartennnn/cli-builder/src/index.js

camelCaseToKebab()
kebabCaseToCamel()
jondubois commented 1 year ago

Thanks for raising this issue @MegaGM. Nice to see you again.