Improve performance by allowing commands and namespaces to be `require`d dinamically about findhelp HOT 5 CLOSED

I was just thinking about this yesterday, because vtex startup time is now more than one second. My idea is also in the way of a module property, as you suggested. Would it be something along these lines?

apps/index.js

export default {
  list: {
    description: 'List apps',
    module: './commands'
  }
}

apps/commands.js

export default {
  list: async () => {
    console.log('Hello list')
  }
}

from findhelp.

Why have the list key twice? I'd think it would be more natural/interchangeable with the default API to have the new module behave precisely as if the {module} object was replaced by the module. In your example, the child would have description and handler properties.

from findhelp.

@rdumont it dawned on me today that we might fix the performance problem by simply bundling the entire lib/ with webpack on prepublish. What do you think?

from findhelp.

Well, we have to test it, but I wouldn't give up on the dynamic modules.

from findhelp.

Done on 3793ed7

from findhelp.