From 787395fccd26bab8db1ab25391eff734a4f9a202 Mon Sep 17 00:00:00 2001
From: riccardoperra <riccardo.perra@icloud.com>
Date: Mon, 11 Nov 2024 21:59:31 +0100
Subject: [PATCH] fix

---
 README.md                                       |  2 ++
 .../app/src/components/Editor/Header/Header.tsx | 14 ++++++++++----
 .../WorkflowDispatchItemForm.tsx                | 17 -----------------
 .../Home/CurrentUser/CurrentUser.css.ts         |  3 ++-
 packages/app/src/store/editor/actions/helper.ts |  8 ++++++--
 .../store/editor/actions/workflow-dispatch.ts   | 15 ++++++++++++++-
 6 files changed, 34 insertions(+), 25 deletions(-)

diff --git a/README.md b/README.md
index de64aa9..afcb343 100644
--- a/README.md
+++ b/README.md
@@ -134,6 +134,8 @@ This project uses [ungh](https://github.com/unjs/ungh) from UnJS in order to cal
 
 This project uses [statebuilder](https://github.com/riccardoperra/statebuilder) to handle local state.
 
+The editor store uses a command-based approach which granularly update the yaml code imperatively via [yaml](https://www.npmjs.com/package/yaml) package.
+
 See some implementations in `store` folder:
 
 - [packages/app/src/store/editor](packages/app/src/store/editor)
diff --git a/packages/app/src/components/Editor/Header/Header.tsx b/packages/app/src/components/Editor/Header/Header.tsx
index 6e0d875..95c1bd2 100644
--- a/packages/app/src/components/Editor/Header/Header.tsx
+++ b/packages/app/src/components/Editor/Header/Header.tsx
@@ -62,7 +62,8 @@ function EditorHeaderForkButton() {
         {_(
           msg`Forking this source will create a new scratch remotely connected to your profile, that can be modified only by you`,
         )}
-        <div>
+        <div style={{'margin-top': '8px'}}>
+          {/* TODO: fix this inline style */}
           <form
             method={'post'}
             action={createScratchFork.with(
@@ -74,10 +75,15 @@ function EditorHeaderForkButton() {
             <Button size={'xs'} theme={'primary'} type={'submit'}>
               {_(msg`Confirm`)}
             </Button>
+            <Button
+              size={'xs'}
+              theme={'secondary'}
+              type={'button'}
+              onClick={() => setOpen(false)}
+            >
+              {_(msg`Cancel`)}
+            </Button>
           </form>
-          <Button size={'xs'} theme={'secondary'}>
-            {_(msg`Cancel`)}
-          </Button>
         </div>
       </PopoverContent>
     </Popover>
diff --git a/packages/app/src/components/Editor/Properties/WorkflowDispatchForm/WorkflowDispatchItemForm.tsx b/packages/app/src/components/Editor/Properties/WorkflowDispatchForm/WorkflowDispatchItemForm.tsx
index 998611b..089ec0b 100644
--- a/packages/app/src/components/Editor/Properties/WorkflowDispatchForm/WorkflowDispatchItemForm.tsx
+++ b/packages/app/src/components/Editor/Properties/WorkflowDispatchForm/WorkflowDispatchItemForm.tsx
@@ -65,23 +65,6 @@ export function WorkflowDispatchItemForm(props: WorkflowDispatchItemFormProps) {
         />
       </FullWidthPanelRow>
 
-      <FullWidthPanelRow>
-        <TextArea
-          slotClasses={{
-            root: styles.inlineInputRoot,
-            label: styles.inlineInputLabel,
-          }}
-          options={{
-            autoResize: true,
-          }}
-          size={'sm'}
-          theme={'filled'}
-          label={'Deprecation message'}
-          value={form()?.deprecationMessage}
-          onChange={value => update({deprecationMessage: value})}
-        />
-      </FullWidthPanelRow>
-
       <FullWidthPanelRow>
         <Select
           aria-label={'Type'}
diff --git a/packages/app/src/components/Home/CurrentUser/CurrentUser.css.ts b/packages/app/src/components/Home/CurrentUser/CurrentUser.css.ts
index 7c6ffa9..5ffa1e5 100644
--- a/packages/app/src/components/Home/CurrentUser/CurrentUser.css.ts
+++ b/packages/app/src/components/Home/CurrentUser/CurrentUser.css.ts
@@ -14,5 +14,6 @@ export const badge = style({
 });
 
 export const currentUser = style({
-  fontSize: '14px',
+  // TODO: fix this style ordering
+  fontSize: '14px !important',
 });
diff --git a/packages/app/src/store/editor/actions/helper.ts b/packages/app/src/store/editor/actions/helper.ts
index b82669f..e104468 100644
--- a/packages/app/src/store/editor/actions/helper.ts
+++ b/packages/app/src/store/editor/actions/helper.ts
@@ -122,14 +122,18 @@ export const convertEnvItemFieldToYaml = (
   return value;
 };
 
-export function removeObjectYamlPrivateProperties(object: Record<string, any>) {
+export function removeObjectYamlPrivateProperties<
+  T extends Record<string, any>,
+>(object: T) {
   const clone = structuredClone(object);
   for (const key in clone) {
     if (key[0] === '$') {
       delete clone[key];
     }
   }
-  return clone;
+  return clone as {
+    [K in keyof T]: K extends `$${string}` ? never : T[K];
+  };
 }
 
 export function orderYamlMapItemByKey(
diff --git a/packages/app/src/store/editor/actions/workflow-dispatch.ts b/packages/app/src/store/editor/actions/workflow-dispatch.ts
index 02920a9..a211257 100644
--- a/packages/app/src/store/editor/actions/workflow-dispatch.ts
+++ b/packages/app/src/store/editor/actions/workflow-dispatch.ts
@@ -30,7 +30,7 @@ export const setWorkflowDispatch =
           workflowDispatch.add(new Pair(new Scalar('inputs'), inputs));
         }
 
-        let {name, ...others} = removeObjectYamlPrivateProperties(data);
+        let {name, ...others} = removeObjectYamlPrivateProperties({...data});
         const inputAtIndex = inputs.items.at(index);
         const input = new YAMLMap<Scalar<string>, unknown>();
         if (!name && fallbackMissingName) {
@@ -47,6 +47,19 @@ export const setWorkflowDispatch =
           return;
         }
         const pair = new Pair(new Scalar(name), input);
+        switch (others.type) {
+          case 'boolean':
+            others.default =
+              others.default === 'true' || others.default === true;
+            break;
+          case 'string':
+            others.default = String(others.default);
+            break;
+          case 'number':
+            others.default = parseInt(others.default, 10);
+            break;
+        }
+
         for (const [key, value] of Object.entries(others)) {
           if (value !== null && value !== undefined) {
             input.set(new Scalar(key), value);