chore: Update to async endpoints (#255)

Author: joe-yeager

Diff for: api/__tests__/projects.test.ts

@@ -30,9 +30,6 @@ import {
   queueBuild,
   uploadFileToBuild,
   uploadProject,
-  listAppsForMigration,
-  beginMigration,
-  finishMigration,
 } from '../projects';
 
 const createReadStreamMock = createReadStream as jest.MockedFunction<
@@ -637,105 +634,14 @@ describe('api/projects', () => {
     });
   });
 
-  describe('listAppsForMigration', () => {
-    it('should call http correctly', async () => {
-      await listAppsForMigration(accountId);
-      expect(http.get).toHaveBeenCalledTimes(1);
-      expect(http.get).toHaveBeenCalledWith(accountId, {
-        url: `dfs/migrations/v2/list-apps`,
-      });
-    });
-  });
-
-  describe('beginMigration', () => {
-    it('should call http correctly', async () => {
-      const applicationId = 123456;
-      await beginMigration(accountId, applicationId);
-      expect(http.post).toHaveBeenCalledTimes(1);
-      expect(http.post).toHaveBeenCalledWith(accountId, {
-        url: `dfs/migrations/v2/migrations`,
-        data: {
-          applicationId,
-        },
-      });
-    });
-  });
-
-  describe('finishMigration', () => {
-    it('should call http correctly with stable platform version', async () => {
-      const migrationId = 123456;
-      const componentUids = { 'component-1': 'uid-1' };
-      const targetPlatformVersion = '2025.2';
-      await finishMigration(
-        accountId,
-        migrationId,
-        componentUids,
-        projectName,
-        targetPlatformVersion
-      );
-      expect(http.post).toHaveBeenCalledTimes(1);
-      expect(http.post).toHaveBeenCalledWith(accountId, {
-        url: `dfs/migrations/v2/migrations/migrate/v20252`,
-        data: {
-          migrationId,
-          projectName,
-          componentUids,
-        },
-      });
-    });
-
-    it('should call http correctly with unstable platform version', async () => {
-      const migrationId = 123456;
-      const componentUids = { 'component-1': 'uid-1' };
-      const targetPlatformVersion = 'unstable';
-      await finishMigration(
-        accountId,
-        migrationId,
-        componentUids,
-        projectName,
-        targetPlatformVersion
-      );
-      expect(http.post).toHaveBeenCalledTimes(1);
-      expect(http.post).toHaveBeenCalledWith(accountId, {
-        url: `dfs/migrations/v2/migrations/migrate/unstable`,
-        data: {
-          migrationId,
-          projectName,
-          componentUids,
-        },
-      });
-    });
-  });
-
   describe('checkMigrationStatus', () => {
     it('should call v1 api for stable platform version', async () => {
       const migrationId = 123456;
-      const targetPlatformVersion = '2023.2';
-      await checkMigrationStatus(accountId, migrationId, targetPlatformVersion);
+      await checkMigrationStatus(accountId, migrationId);
       expect(http.get).toHaveBeenCalledTimes(1);
       expect(http.get).toHaveBeenCalledWith(accountId, {
         url: `dfs/migrations/v1/migrations/${migrationId}`,
       });
     });
-
-    it('should call v2 api for unstable platform version', async () => {
-      const migrationId = 123456;
-      const targetPlatformVersion = 'unstable';
-      await checkMigrationStatus(accountId, migrationId, targetPlatformVersion);
-      expect(http.get).toHaveBeenCalledTimes(1);
-      expect(http.get).toHaveBeenCalledWith(accountId, {
-        url: `dfs/migrations/v2/migrations/${migrationId}/status`,
-      });
-    });
-
-    it('should call v2 api for 2025.2 platform version', async () => {
-      const migrationId = 123456;
-      const targetPlatformVersion = '2025.2';
-      await checkMigrationStatus(accountId, migrationId, targetPlatformVersion);
-      expect(http.get).toHaveBeenCalledTimes(1);
-      expect(http.get).toHaveBeenCalledWith(accountId, {
-        url: `dfs/migrations/v2/migrations/${migrationId}/status`,
-      });
-    });
   });
 });

Diff for: api/projects.ts

@@ -9,9 +9,6 @@ import {
   FetchPlatformVersionResponse,
   WarnLogsResponse,
   UploadIRResponse,
-  ListAppsResponse,
-  BeginMigrationResponse,
-  FinishMigrationResponse,
 } from '../types/Project';
 import { Build, FetchProjectBuildsResponse } from '../types/Build';
 import {
@@ -24,7 +21,6 @@ import {
   CloneAppResponse,
   PollAppResponse,
 } from '../types/Migration';
-import { PLATFORM_VERSIONS } from '../constants/projects';
 
 const PROJECTS_API_PATH = 'dfs/v1/projects';
 const DEVELOPER_FILE_SYSTEM_PATH = 'dfs/v1';
@@ -33,7 +29,6 @@ const PROJECTS_DEPLOY_API_PATH_V3 = 'dfs/deploy/v3';
 const PROJECTS_LOGS_API_PATH = 'dfs/logging/v1';
 const DEVELOPER_PROJECTS_API_PATH = 'developer/projects/v1';
 const MIGRATIONS_API_PATH_V1 = 'dfs/migrations/v1';
-const MIGRATIONS_API_PATH_V2 = 'dfs/migrations/v2';
 
 const PROJECTS_V3_API_PATH = 'project-components-external/v3';
 
@@ -339,47 +334,9 @@ export function fetchDeployWarnLogs(
   });
 }
 
-export async function listAppsForMigration(
-  accountId: number
-): HubSpotPromise<ListAppsResponse> {
-  return http.get<ListAppsResponse>(accountId, {
-    url: `${MIGRATIONS_API_PATH_V2}/list-apps`,
-  });
-}
-
-export async function beginMigration(
-  accountId: number,
-  applicationId: number
-): HubSpotPromise<BeginMigrationResponse> {
-  return http.post(accountId, {
-    url: `${MIGRATIONS_API_PATH_V2}/migrations`,
-    data: {
-      applicationId,
-    },
-  });
-}
-
-export async function finishMigration(
-  portalId: number,
-  migrationId: number,
-  componentUids: Record<string, string>,
-  projectName: string,
-  targetPlatformVersion: string
-): HubSpotPromise<FinishMigrationResponse> {
-  const pathPlatformVersion =
-    targetPlatformVersion === PLATFORM_VERSIONS.unstable
-      ? targetPlatformVersion
-      : `v${targetPlatformVersion.replace('.', '')}`;
-  return http.post(portalId, {
-    url: `${MIGRATIONS_API_PATH_V2}/migrations/migrate/${pathPlatformVersion}`,
-    data: {
-      migrationId,
-      projectName,
-      componentUids,
-    },
-  });
-}
-
+/**
+ * @deprecated
+ */
 export function migrateApp(
   accountId: number,
   appId: number,
@@ -395,25 +352,21 @@ export function migrateApp(
   });
 }
 
+/**
+ * @deprecated
+ */
 export function checkMigrationStatus(
   accountId: number,
-  id: number,
-  targetPlatformVersion: string = PLATFORM_VERSIONS.v2023_2
+  id: number
 ): HubSpotPromise<PollAppResponse> {
-  if (
-    targetPlatformVersion === PLATFORM_VERSIONS.unstable ||
-    targetPlatformVersion === PLATFORM_VERSIONS.v2025_2
-  ) {
-    return http.get<PollAppResponse>(accountId, {
-      url: `${MIGRATIONS_API_PATH_V2}/migrations/${id}/status`,
-    });
-  }
-
   return http.get<PollAppResponse>(accountId, {
     url: `${MIGRATIONS_API_PATH_V1}/migrations/${id}`,
   });
 }
 
+/**
+ * @deprecated
+ */
 export function cloneApp(
   accountId: number,
   appId: number
@@ -427,6 +380,9 @@ export function cloneApp(
   });
 }
 
+/**
+ * @deprecated
+ */
 export function checkCloneStatus(
   accountId: number,
   exportId: number
@@ -436,6 +392,9 @@ export function checkCloneStatus(
   });
 }
 
+/**
+ * @deprecated
+ */
 export function downloadClonedProject(
   accountId: number,
   exportId: number

Diff for: package.json

@@ -54,7 +54,10 @@
     "./config": "./config/index.js",
     "./constants/*": "./constants/*.js",
     "./models/*": "./models/*.js",
-    "./types/*": "./types/*.d.ts"
+    "./types/*": {
+      "types": "./types/*.d.ts",
+      "default": "./types/*.js"
+    }
   },
   "dependencies": {
     "address": "2.0.2",

Diff for: types/Migration.ts

@@ -7,6 +7,8 @@ export const MIGRATION_STATUS = {
   PREPARING: 'PREPARING',
   PENDING: 'PENDING',
   SUCCESS: 'SUCCESS',
+  IN_PROGRESS: 'IN_PROGRESS',
+  INPUT_REQUIRED: 'INPUT_REQUIRED',
 } as const;
 
 export type MigrateAppResponse = {

Diff for: types/Project.ts

@@ -1,7 +1,6 @@
 import { Build } from './Build';
 import { GithubSourceData } from './Github';
 import { ProjectLog } from './ProjectLog';
-import { UNMIGRATABLE_REASONS } from '../constants/projects';
 
 export type Project = {
   createdAt: number;
@@ -73,44 +72,3 @@ export type ProjectStandardError = {
 export type WarnLogsResponse = {
   logs: Array<ProjectLog>;
 };
-
-interface BaseMigrationApp {
-  appId: number;
-  appName: string;
-  isMigratable: boolean;
-  migrationComponents: ListAppsMigrationComponent[];
-}
-
-export interface MigratableApp extends BaseMigrationApp {
-  isMigratable: true;
-}
-
-export interface UnmigratableApp extends BaseMigrationApp {
-  isMigratable: false;
-  unmigratableReason: keyof typeof UNMIGRATABLE_REASONS;
-}
-
-export type MigrationApp = MigratableApp | UnmigratableApp;
-
-export interface ListAppsResponse {
-  migratableApps: MigratableApp[];
-  unmigratableApps: UnmigratableApp[];
-}
-
-export interface BeginMigrationResponse {
-  migrationId: number;
-  componentsRequiringUids: Record<
-    string,
-    { componentType: string; componentHint: string | null }
-  >;
-}
-export interface ListAppsMigrationComponent {
-  id: string;
-  componentType: string;
-  isSupported: boolean;
-}
-
-export type FinishMigrationResponse = {
-  projectName: string;
-  buildId: number;
-};