Feature request: global/universal actions

These might better be defined as ‘hooks’.

I spent a while trying to create my own prehook on actions, something that would get run against every action. Something like the following:

old_action = prog.action.bind(prog);
prog.action = function () {
    always_gets_called_on_every_action();
    return old_action.apply(prog, arguments)
}

The problem is there’s still no way to get at the opts argument in this prog.action overwrite, which is what I need.

Use case: change the config file for every action (-c, --config) without manually baking it into every action.

I decided to stop wasting time on this and make hooks or something like that an actual feature request, or at least a point of conversation.

Thanks.

[arxpoetica]

@tunnckoCore

But it’s working right?

I couldn’t get Object.assign to work, no, but I don’t have that code anymore to really compare notes…

Regardless…

@lukeed your wrapper suggestion worked great. Still trying to wrap my mind around it (no pun intended), but it’s a working solution, so closing.

[lukeed]

If you wanted to do the prog.action route, it’d have to look like this:

// save the old
const add_action = prog.action.bind(prog);

prog.action = function (handler) {
  add_action(
    // use the same `wrapper` util before
    // because it actually wraps the handler
    wrapper(handler)
  );
}

What you had modified sade’s construct. All that method does is build the tree, but what you (originally) showed was trying to execute/wrap the handler on initialization