chore: Add error message for projects linked to GH Repo

lang/en.ts

@@ -1,12 +1,7 @@
 // @ts-nocheck
 import chalk from 'chalk';
 import { uiAccountDescription, uiCommandReference, uiLink } from '../lib/ui';
-
-type LangFunction = (...args: (string | number)[]) => string;
-
-type LangObject = {
-  [key: string]: string | LangFunction | LangObject;
-};
+import { getProjectSettingsUrl } from '../lib/projects/urls';
 
 type LangFunction = (...args: (string | number)[]) => string;
 
@@ -3452,6 +3447,11 @@ export const lib = {
         upToDate: 'App is already up to date',
         isPrivateApp: 'Private apps are not currently migratable',
         listedInMarketplace: 'Listed apps are not currently migratable',
+        projectConnectedToGitHub: (
+          projectName: string | undefined,
+          accountId: number
+        ) =>
+          `The project is linked to a GitHub repository.  ${uiLink('Visit the project settings page to unlink it', getProjectSettingsUrl(projectName, accountId))}`,
         partOfProjectAlready: `This app is part of a project, run ${uiCommandReference('hs project migrate')} from the project directory to migrate it`,
         generic: reasonCode => `Unable to migrate app: ${reasonCode}`,
       },

lib/app/migrate.ts

@@ -57,14 +57,23 @@ export type MigrateAppArgs = CommonArgs &
     platformVersion: string;
   };
 
-function getUnmigratableReason(reasonCode: string): string {
+function getUnmigratableReason(
+  reasonCode: string,
+  projectName: string | undefined,
+  accountId: number
+): string {
   switch (reasonCode) {
     case UNMIGRATABLE_REASONS.UP_TO_DATE:
       return lib.migrate.errors.unmigratableReasons.upToDate;
     case UNMIGRATABLE_REASONS.IS_A_PRIVATE_APP:
       return lib.migrate.errors.unmigratableReasons.isPrivateApp;
     case UNMIGRATABLE_REASONS.LISTED_IN_MARKETPLACE:
       return lib.migrate.errors.unmigratableReasons.listedInMarketplace;
+    case UNMIGRATABLE_REASONS.PROJECT_CONNECTED_TO_GITHUB:
+      return lib.migrate.errors.unmigratableReasons.projectConnectedToGitHub(
+        projectName,
+        accountId
+      );
     case CLI_UNMIGRATABLE_REASONS.PART_OF_PROJECT_ALREADY:
       return lib.migrate.errors.unmigratableReasons.partOfProjectAlready;
     default:
@@ -128,7 +137,7 @@ async function fetchMigrationApps(
   if (allApps.length === 0 || !allApps.some(app => app.isMigratable)) {
     const reasons = filteredUnmigratableApps.map(
       app =>
-        `${chalk.bold(app.appName)}: ${getUnmigratableReason(app.unmigratableReason)}`
+        `${chalk.bold(app.appName)}: ${getUnmigratableReason(app.unmigratableReason, app.projectName, derivedAccountId)}`
     );
 
     throw new Error(
@@ -151,15 +160,22 @@ async function fetchMigrationApps(
   return allApps;
 }
 
-async function promptForAppToMigrate(allApps: MigrationApp[]) {
+async function promptForAppToMigrate(
+  allApps: MigrationApp[],
+  derivedAccountId: number
+) {
   const appChoices = allApps.map(app => ({
     name: app.isMigratable
       ? app.appName
       : `[${chalk.yellow('DISABLED')}] ${app.appName} `,
     value: app,
     disabled: app.isMigratable
       ? false
-      : getUnmigratableReason(app.unmigratableReason),
+      : getUnmigratableReason(
+          app.unmigratableReason,
+          app.projectName,
+          derivedAccountId
+        ),
   }));
 
   const enabledChoices = appChoices.filter(app => !app.disabled);
@@ -180,6 +196,7 @@ async function promptForAppToMigrate(allApps: MigrationApp[]) {
 }
 async function selectAppToMigrate(
   allApps: MigrationApp[],
+  derivedAccountId: number,
   appId?: number,
   projectConfig?: LoadedProjectConfig
 ): Promise<{ proceed: boolean; appIdToMigrate?: number }> {
@@ -194,7 +211,7 @@ async function selectAppToMigrate(
 
   let appIdToMigrate = appId;
   if (!appIdToMigrate) {
-    appIdToMigrate = await promptForAppToMigrate(allApps);
+    appIdToMigrate = await promptForAppToMigrate(allApps, derivedAccountId);
   }
 
   const selectedApp = allApps.find(app => app.appId === appIdToMigrate);
@@ -259,6 +276,7 @@ async function handleMigrationSetup(
 
   const { proceed, appIdToMigrate } = await selectAppToMigrate(
     allApps,
+    derivedAccountId,
     appId,
     projectConfig
   );

lib/projects/urls.ts

@@ -18,6 +18,14 @@ export function getProjectDetailUrl(
   return `${getProjectHomeUrl(accountId)}/project/${projectName}`;
 }
 
+export function getProjectSettingsUrl(
+  projectName: string,
+  accountId: number
+): string | undefined {
+  if (!projectName) return;
+  return `${getProjectDetailUrl(projectName, accountId)}/settings`;
+}
+
 export function getProjectActivityUrl(
   projectName: string,
   accountId: number