fix a sync issue on deploy

Author: josephjclark

packages/cli/src/deploy/beta.ts

@@ -29,6 +29,8 @@ export async function handler(options: DeployOptionsBeta, logger: Logger) {
   // TMP use options.path to set the directory for now
   // We'll need to manage this a bit better
   const project = await Project.from('fs', { root: options.path || '.' });
+  // Why is there an id on openfn here?
+  console.log({ openfn: project.openfn });
 
   // TODO: work out if there's any diff
 

packages/cli/src/pull/beta.ts

@@ -77,6 +77,10 @@ export async function handler(options: PullOptionsBeta, logger: Logger) {
     {
       endpoint: cfg.endpoint,
       env: name,
+
+      // TODO this is NOT an openfn metadata key
+      // (it should not be sent back to lighting)
+      // should add it to the local or meta objects instead
       fetched_at: new Date().toISOString(),
     },
     config

packages/project/src/parse/from-fs.ts

@@ -12,6 +12,7 @@ import {
   findWorkspaceFile,
 } from '../util/config';
 import fromProject from './from-project';
+import { omit } from 'lodash-es';
 
 export type FromFsConfig = {
   root: string;
@@ -50,7 +51,7 @@ export const parseProject = async (options: FromFsConfig) => {
 
   const proj: any = {
     name: state?.name,
-    openfn: context.project,
+    openfn: omit(context.project, ['id']),
     config: config,
     workflows: [],
   };

packages/project/src/serialize/to-app-state.ts

@@ -18,7 +18,14 @@ export default function (
   project: Project,
   options: Options = {}
 ): Provisioner.Project | string {
-  const { uuid, endpoint, env, ...rest } = project.openfn ?? {};
+  const {
+    uuid,
+    endpoint,
+    env,
+    id /* shouldn't be there but will cause problems if it's set*/,
+    fetched_at /* remove this metadata as it causes problems */,
+    ...rest
+  } = project.openfn ?? {};
 
   const state = omitBy(
     pick(project, ['name', 'description', 'collections']),