fix: Ensure we set jupytext metadata to model

* When selectedFormts are empty, we return before setting jupytext metadata

* So, the changes are not being comitted to notebook model

* This commit fixes it by ensuring we always set metadata before returning

* Use console.debug instead of console.log to reduce log pollution

Signed-off-by: Mahendra Paipuri <[email protected]>

Author: mahendrapaipuri

jupyterlab/packages/jupyterlab-jupytext-extension/src/commands.ts

@@ -182,7 +182,7 @@ export function executePairCommand(
   let selectedFormats = getSelectedFormats(notebookTracker);
 
   // Toggle the selected format
-  console.log('Jupytext: executing command=' + command, selectedFormats);
+  console.debug('Jupytext: executing command=' + command);
   if (format === 'custom') {
     showErrorMessage(
       trans.__('Error'),
@@ -256,8 +256,8 @@ export function executePairCommand(
     }
     if (Object.keys(jupytext).length === 0) {
       (model as any).deleteMetadata('jupytext');
-      (model as any).setMetadata('jupytext', jupytext);
     }
+    (model as any).setMetadata('jupytext', jupytext);
     return;
   }
 
@@ -332,7 +332,7 @@ export function isMetadataCommandEnabled(
 export function executeMetadataCommand(
   notebookTracker: INotebookTracker
 ): void {
-  console.log('Jupytext: toggling YAML header');
+  console.debug('Jupytext: toggling YAML header');
   if (!notebookTracker.currentWidget) {
     return;
   }