remove unnecessary gate

[leastbad]

Type of PR (feature, enhancement, bug fix, etc.)

"Enhancement"

Description

Back in #45 (aka callbacks) we introduced a gate around the bottom-half of the main stimulus_reflex.js to make sure it was only executed once. The thing is, I cannot for the life of me see what purpose this serves. Presumably, with or without Turbolinks, this code is only going to be executed once per page - or else the const initialize immediately above it would already be complaining loudly, right?

This PR just removes the gate. There could be a very good reason to keep it! I just don't know what we were guarding against.

Why should this be added

Code clarity

Checklist

• My code follows the style guidelines of this project
• Checks (StandardRB & Prettier-Standard) are passing
• This is not a documentation update

[leastbad]

Note that the file view on this PR looks intimidating, but it's literally just removing lines 494-496, 604.

The changeset is the result of the indentation formatting moving everything two spaces to the left.

[hopsoft]

This should be safe considering that SR is typically initialized in app/javascript/controllers/index.js or somewhere similar.