We use 4 spaces throughout in our JS files, so even though it's an open question whether that's good, you should match that style.