Run custom commands in interactive shell

[felixschndr]

Is your feature request related to a problem? Please describe.
When a custom command is specified in the config it runs it with bash -c "<command>". I'd like this shell to be interactive to be able to run shell aliases which lie in my .bashrc.

Describe the solution you'd like
Run bash -ic "<command">

Describe alternatives you've considered
Currently I am adding a layer of bash:

  - key: <c-d>
    context: "localBranches, remoteBranches"
    command: bash -ic "git_delete_old_branches"
    stream: true

which runs bash -c bash -ic "git_delete_old_branches"

Additional context
Maybe make this a config variable?

I tried to go through the source code to check where the bash -c (without -i) is invoked but was a bit overwhelmed by the search results. If someone could show me where this is done I'd take a crack at creating a PR.

[aaronjconway]

NewShell() in the /oscommands/cmd_obj_builder.go

I just open up another pane, do work then hop back.

[felixschndr]

There is also NewInteractiveShell()

lazygit/pkg/commands/oscommands/cmd_obj_builder.go

Line 54 in b0a766c

func (self *CmdObjBuilder) NewInteractiveShell(commandStr string) ICmdObj {

. We could just run this function instead, right?

[felixschndr]

We could just run this function instead, right?

If so I am not able to find where this function call has to be changed

[yam-liu]

There are some thoughts shared in #3299 (comment). Using -i for custom commands is not ideal due to performance concerns.

Since custom commands are typically one-off configurations in config.yml, it should be fine to write the original command directly. However, for shell commands that you frequently type and reuse, using -i is reasonable so that you can reach your aliases etc..

[melMass]

The ideal thing would be to allow custom flags for the shell.
In nushell for instance there is a concept of env vs config. env is lightweight and only adds some path to modules etc, enough to import those modules in custom commands: use mymod.nu;mymod mycommand

Currently -c being hardcoded makes that impossible

[melMass]

Also the parsing is inconsistent between macos and windows.

On macos this works fine:

command: "nu --env-config $nu.env-path -c 'use ensure.nu;ensure scope {{.Form.Type}}'"

But on windows it adds extra quotes around $nu.env-path:

I tried various things, single quotes, escaping etc to no avail.