You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
sgpt -s is a great feature but I would like to propose the ability to edit the output before execution, sort of like whatever happens with ctrl +l (shell integration). Maybe now I am answering my own question, but the difference here is that since I can pipe to here and cannot pipe to ctrl +l I can totally see how this would be valuable. For example, mistakes happen and it may be easier to correct it right away.
Current Situation
$ sgpt --shell "find all json files in current folder"# -> find . -type f -name "*.json"# -> [E]xecute, [D]escribe, [A]bort:
Proposed Future Option 1 ( add Modify to list of options)
$ sgpt --shell "find all json files in current folder"# -> find . -type f -name "*.json"# -> [M]odify, [E]xecute, [D]escribe, [A]bort:
Pros: Current Functionality remains
Proposed Future Option 2 (remove Execute, replace with Edit)
$ sgpt --shell "find all json files in current folder"# -> find . -type f -name "*.json"# -> [E]dit, [D]escribe, [A]bort:
Pros: Current User Behavior is safe and product doesn't just allow to execute on autopilot.
Cons: You have to press enter twice instead of once
Proposed Future Option 3 (remove Execute, replace with Modify)
$ sgpt --shell "find all json files in current folder"# -> find . -type f -name "*.json"# -> [M]odify, [D]escribe, [A]bort:
I don't actually like this one, because Option 2 is better 😄
The text was updated successfully, but these errors were encountered:
sgpt -s
is a great feature but I would like to propose the ability to edit the output before execution, sort of like whatever happens withctrl +l
(shell integration). Maybe now I am answering my own question, but the difference here is that since I can pipe to here and cannot pipe toctrl +l
I can totally see how this would be valuable. For example, mistakes happen and it may be easier to correct it right away.Current Situation
Proposed Future Option 1 ( add
Modify
to list of options)Pros: Current Functionality remains
Proposed Future Option 2 (remove
Execute
, replace withEdit
)Pros: Current User Behavior is safe and product doesn't just allow to execute on autopilot.
Cons: You have to press enter twice instead of once
Proposed Future Option 3 (remove
Execute
, replace withModify
)I don't actually like this one, because Option 2 is better 😄
The text was updated successfully, but these errors were encountered: