editor/sagas: fix signature trigger characters

The intended behavior is that comma should trigger signature help when
it is not showing. retrigger only triggers if already showing.

Author: dlech

CHANGELOG.md

@@ -4,6 +4,9 @@
 
 ## [Unreleased]
 
+### Changed
+- Code completion trigger improvements.
+
 ### Fixed
 - Fixed terminal layout issues when resizing window vertically.
 - Fixed editor find widget visible over modal dialogs.

src/editor/sagas.ts

@@ -635,8 +635,8 @@ function* runJedi(): Generator {
             const subscription = monaco.languages.registerSignatureHelpProvider(
                 pybricksMicroPythonId,
                 {
-                    signatureHelpTriggerCharacters: ['('],
-                    signatureHelpRetriggerCharacters: [','],
+                    signatureHelpTriggerCharacters: ['(', ','],
+                    signatureHelpRetriggerCharacters: [')'],
                     provideSignatureHelp(model, position, token, context) {
                         return new Promise((resolve) => {
                             emit({ model, position, token, context, resolve });