Fix up style nits and code review comments

This is a follow-up commit addressing the comments left on the previous
commits in code review. In a nutshell:

  * Don't use $1 in SpaceBeforeScript linter
  * Better format multi-line array in tag_with_inline_text
  * Changed spelling of behaviour
  * Aligned `autoload` statements
  * Renamed `continue` variable to `descend`
  * Used more concise regex match syntax
  * Changed indent code to specify number of spaces instead of hard-coding
  * Changed source map creator to use `respond_to?` instead of checking
    if class was `Fixnum`
  * Tightened up a regex to include a word boundary to avoid false matches
  * Removed unnecessary `protected`
  * Replaced use of `next if ...` with a `select` on the list
  * Add support for `until` loops

Change-Id: I4b461fbad01639a09ec8084b58822da89e165f8d
Reviewed-on: https://gerrit.causes.com/28524
Reviewed-by: Shane da Silva <shane@causes.com>
Tested-by: Shane da Silva <shane@causes.com>