Intermediate DI/service locator solution for JavaScript modules

di-plus-sl-modules.js

//////////////////////////
// Pure "DI style" modules

// BackEndAnalytics.js
define(function () {
    return function BackEndAnalytics() {
        this.send = function () { };
    };
});

// FrontEndAnalytics.js
define(function () {
    return function FrontEndAnalytics() {
        this.send = function () { };
        this.sendScreenInfo = function (colorDepth, width, height) { };
    };
});

// Analytics.js
define(function () {
    return function Analytics(backEndAnalytics, frontEndAnalytics, screen) {
        this.send = function () {
            backendAnalytics.send();
            frontEndAnalytics.send();
            frontEndAnalytics.sendScreenInfo(screen.colorDepth, screen.width, screen.height);
        };
    };
});

// NullAnalytics.js
define(function () {
    // Null object pattern
    return function NullAnalytics() {
        this.send = function () { };
    }
});

/////////////////////////
// Service-located singleton modules that lots of legacy code depends on.
// In a pure DI system, these would not exist; e.g. the legacy code would get an Analytics instance injected into it,
// and the code that is inside of these module would get moved up to the composition root.

// configuration.js
define(function () {
    return {
        isInProduction: true
    };
});

// analytics.js
define(["BackEndAnalytics", "FrontEndAnalytics", "Analytics", "NullAnalytics", "configuration"],
       function (BackEndAnalytics, FrontEndAnalytics, Analytics, NullAnalytics, configuration) {

    var analytics = configuration.isInProduction
        ? new Analytics(new BackEndAnalytics(), new FrontEndAnalytics(), window.screen)
        : new NullAnalytics();
        
    return {
        send: analytics.send
    };
});

//////////////////////
// Client legacy code

// frobber.js
define(["analytics"], function (analytics) {
    return {
        frob: function () {
            console.log("frobbing");
            analytics.send();
        }
    };
});

line 13: should it be FrontEndAnalytics instead of BackEndAnalytics?

@bolshchikov thanks, fixed