DEV: Debundle plugin javascript assets and don't load if disabled (PR #7566)

GitHub

You’ve signed the CLA, vinothkannans. Thank you! This pull request is ready for review.

Teeny-weeny typo

# ignore auto-generated plugin js assets

Using equality (==) is slightly easier to read than inequality (!=)

      next if args[:include_disabled] == false && !plugin.enabled?

I tend to prefer select over find_all because it’s shorter and I don’t confuse it with all?

    plugins.select do |plugin|

Is it possible to keep a test to ensure safe mode is still loading the proper javascript assets?

I spent many hours to keep existing or create new spec tests for this PR. But it looks it’s not possible since all the dynamic file creations are happening only on initial app booting time.

LGTM :+1: