Какой вариант правильнее?
Первый вариант позволяет отделить библиотеки от связующего кода. В этом случае вы someFunction можете вынести в отдельный пакет и переиспользовать из проекта в проект. Ну и просто помогает держать код в чистоте.
Второй вариант хорош тем, что функция не вытекает за границы замыкания, но в реальности это сомнительный плюс.
При втором варианте не засирается глобальная область видимости, но, при этом, внешний код не сможет использовать someFunction(). Если вам так и надо, то юзайте его.