@grafana/create-plugin 6.2.0-canary.2233.19133609453.0 → 6.2.0-canary.2233.19368311379.0

package/src/codemods/migrations/manager.test.ts

@@ -1,24 +1,23 @@
-import { flushChanges, formatFiles } from '../utils.js';
-import { getMigrationsToRun, runMigration, runMigrations } from './manager.js';
+import { flushChanges, formatFiles, printChanges } from '../utils.js';
+import { getMigrationsToRun, runMigrations } from './manager.js';
 
 import { Context } from '../context.js';
-import { MigrationMeta } from './migrations.js';
+import { Migration } from './migrations.js';
 import { gitCommitNoVerify } from '../../utils/utils.git.js';
 import migrationFixtures from './fixtures/migrations.js';
-import { printChanges } from './utils.js';
 import { setRootConfig } from '../../utils/utils.config.js';
 import { vi } from 'vitest';
 
-vi.mock('./utils.js', () => ({
-  printChanges: vi.fn(),
-  migrationsDebug: vi.fn(),
-}));
-
-vi.mock('../utils.js', () => ({
-  flushChanges: vi.fn(),
-  formatFiles: vi.fn(),
-  installNPMDependencies: vi.fn(),
-}));
+vi.mock('../utils.js', async (importOriginal) => {
+  const actual = (await importOriginal()) as any;
+  return {
+    ...actual,
+    flushChanges: vi.fn(),
+    formatFiles: vi.fn(),
+    installNPMDependencies: vi.fn(),
+    printChanges: vi.fn(),
+  };
+});
 
 // Silence terminal output during tests.
 vi.mock('../../utils/utils.console.js', () => ({
@@ -50,92 +49,83 @@ describe('Migrations', () => {
     it('should return the migrations that need to be run', () => {
       const fromVersion = '3.0.0';
       const toVersion = '5.0.0';
-      const migrations = getMigrationsToRun(fromVersion, toVersion, migrationFixtures.migrations);
-      expect(migrations).toEqual({
-        'migration-key1': {
+      const migrations = getMigrationsToRun(fromVersion, toVersion, migrationFixtures);
+      expect(migrations).toEqual([
+        {
+          name: 'migration-key1',
           version: '5.0.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './5-0-0-cache-directory.js',
+          scriptPath: './5-0-0-cache-directory.js',
         },
-      });
+      ]);
 
       const fromVersion2 = '5.0.0';
       const toVersion2 = '5.5.0';
-      const migrations2 = getMigrationsToRun(fromVersion2, toVersion2, migrationFixtures.migrations);
-      expect(migrations2).toEqual({
-        'migration-key1': {
+      const migrations2 = getMigrationsToRun(fromVersion2, toVersion2, migrationFixtures);
+      expect(migrations2).toEqual([
+        {
+          name: 'migration-key1',
           version: '5.0.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './5-0-0-cache-directory.js',
+          scriptPath: './5-0-0-cache-directory.js',
         },
-        'migration-key2': {
+        {
+          name: 'migration-key2',
           version: '5.4.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './5-4-0-cache-directory.js',
+          scriptPath: './5-4-0-cache-directory.js',
         },
-      });
+      ]);
 
       const fromVersion3 = '5.5.0';
       const toVersion3 = '6.0.0';
-      const migrations3 = getMigrationsToRun(fromVersion3, toVersion3, migrationFixtures.migrations);
-      expect(migrations3).toEqual({
-        'migration-key3': {
+      const migrations3 = getMigrationsToRun(fromVersion3, toVersion3, migrationFixtures);
+      expect(migrations3).toEqual([
+        {
+          name: 'migration-key3',
           version: '6.0.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './5-4-0-cache-directory.js',
+          scriptPath: './5-4-0-cache-directory.js',
         },
-      });
+      ]);
     });
 
     it('should sort migrations by version', () => {
       const fromVersion = '2.0.0';
       const toVersion = '6.0.0';
-      const migrations = getMigrationsToRun(fromVersion, toVersion, {
-        'migration-key1': {
+      const migrations = getMigrationsToRun(fromVersion, toVersion, [
+        {
+          name: 'migration-key1',
           version: '5.3.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './5.3.0-migration.js',
+          scriptPath: './5.3.0-migration.js',
         },
-        'migration-key2': {
+        {
+          name: 'migration-key2',
           version: '2.3.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './2.3.0-migration.js',
+          scriptPath: './2.3.0-migration.js',
         },
-        'migration-key3': {
+        {
+          name: 'migration-key3',
           version: '2.0.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './2.0.0-migration.js',
+          scriptPath: './2.0.0-migration.js',
         },
-        'migration-key4': {
+        {
+          name: 'migration-key4',
           version: '2.0.0',
           description: 'Update project to use new cache directory',
-          migrationScript: './2.0.0-migration.js',
+          scriptPath: './2.0.0-migration.js',
         },
-      });
-
-      expect(Object.keys(migrations)).toEqual(['migration-key3', 'migration-key4', 'migration-key2', 'migration-key1']);
-    });
-  });
-
-  describe('runMigration', () => {
-    it('should pass a context to the migration script', async () => {
-      const mockContext = new Context('/virtual');
-      const migrationFn = vi.fn().mockResolvedValue(mockContext);
-
-      vi.doMock('./test-migration.js', () => ({
-        default: migrationFn,
-      }));
-
-      const migration: MigrationMeta = {
-        version: '1.0.0',
-        description: 'test migration',
-        migrationScript: './test-migration.js',
-      };
-
-      const result = await runMigration(migration, mockContext);
-
-      expect(migrationFn).toHaveBeenCalledWith(mockContext);
-      expect(result).toBe(mockContext);
+      ]);
+
+      expect(migrations.map((m) => m.name)).toEqual([
+        'migration-key3',
+        'migration-key4',
+        'migration-key2',
+        'migration-key1',
+      ]);
     });
   });
 
@@ -144,25 +134,27 @@ describe('Migrations', () => {
     const migrationTwoFn = vi.fn();
     const consoleMock = vi.spyOn(console, 'log').mockImplementation(() => undefined);
 
-    vi.doMock('./migration-one.js', () => ({
+    vi.doMock('virtual-test-migration.js', async () => ({
       default: migrationOneFn,
     }));
-    vi.doMock('./migration-two.js', () => ({
+    vi.doMock('virtual-test-migration2.js', async () => ({
       default: migrationTwoFn,
     }));
 
-    const migrations: Record<string, MigrationMeta> = {
-      'migration-one': {
+    const migrations: Migration[] = [
+      {
+        name: 'migration-one',
         version: '1.0.0',
         description: '...',
-        migrationScript: './migration-one.js',
+        scriptPath: 'virtual-test-migration.js',
       },
-      'migration-two': {
+      {
+        name: 'migration-two',
         version: '1.2.0',
         description: '...',
-        migrationScript: './migration-two.js',
+        scriptPath: 'virtual-test-migration2.js',
       },
-    };
+    ];
 
     beforeEach(() => {
       migrationOneFn.mockImplementation(async (context: Context) => {
@@ -208,6 +200,7 @@ describe('Migrations', () => {
     it('should commit the changes for each migration if the CLI arg is present', async () => {
       await runMigrations(migrations, { commitEachMigration: true });
 
+      // 2 migration commits + 1 version update commit = 3 total
       expect(gitCommitNoVerify).toHaveBeenCalledTimes(3);
     });
 
@@ -216,6 +209,7 @@ describe('Migrations', () => {
 
       await runMigrations(migrations, { commitEachMigration: true });
 
+      // 1 migration commit (only migration-one has changes) + 1 version update commit = 2 total
       expect(gitCommitNoVerify).toHaveBeenCalledTimes(2);
     });
 

package/src/codemods/migrations/manager.ts

@@ -1,11 +1,7 @@
-import defaultMigrations, { MigrationMeta } from './migrations.js';
-import { flushChanges, formatFiles, installNPMDependencies } from '../utils.js';
+import defaultMigrations, { Migration } from './migrations.js';
+import { runCodemod } from '../runner.js';
 import { gte, satisfies } from 'semver';
-import { migrationsDebug, printChanges } from './utils.js';
-
 import { CURRENT_APP_VERSION } from '../../utils/utils.version.js';
-import { Context } from '../context.js';
-import type { MigrationModule } from '../types.js';
 import { gitCommitNoVerify } from '../../utils/utils.git.js';
 import { output } from '../../utils/utils.console.js';
 import { setRootConfig } from '../../utils/utils.config.js';
@@ -13,59 +9,36 @@ import { setRootConfig } from '../../utils/utils.config.js';
 export function getMigrationsToRun(
   fromVersion: string,
   toVersion: string,
-  migrations: Record<string, MigrationMeta> = defaultMigrations.migrations
-): Record<string, MigrationMeta> {
+  migrations: Migration[] = defaultMigrations
+): Migration[] {
   const semverRange = `${fromVersion} - ${toVersion}`;
 
-  const migrationsToRun = Object.entries(migrations)
+  return migrations
+    .filter((meta) => satisfies(meta.version, semverRange))
     .sort((a, b) => {
-      return gte(a[1].version, b[1].version) ? 1 : -1;
-    })
-    .reduce<Record<string, MigrationMeta>>((acc, [key, meta]) => {
-      if (satisfies(meta.version, semverRange)) {
-        acc[key] = meta;
-      }
-      return acc;
-    }, {});
-
-  return migrationsToRun;
+      return gte(a.version, b.version) ? 1 : -1;
+    });
 }
 
 type RunMigrationsOptions = {
   commitEachMigration?: boolean;
+  codemodOptions?: Record<string, any>;
 };
 
-export async function runMigrations(migrations: Record<string, MigrationMeta>, options: RunMigrationsOptions = {}) {
-  const basePath = process.cwd();
-  const migrationList = Object.entries(migrations).map(
-    ([key, migrationMeta]) => `${key} (${migrationMeta.description})`
-  );
+export async function runMigrations(migrations: Migration[], options: RunMigrationsOptions = {}) {
+  const migrationList = migrations.map((meta) => `${meta.name} (${meta.description})`);
 
   const migrationListBody = migrationList.length > 0 ? output.bulletList(migrationList) : ['No migrations to run.'];
 
   output.log({ title: 'Running the following migrations:', body: migrationListBody });
 
-  for (const [key, migration] of Object.entries(migrations)) {
-    try {
-      const context = await runMigration(migration, new Context(basePath));
-      const shouldCommit = options.commitEachMigration && context.hasChanges();
-
-      migrationsDebug(`context for "${key} (${migration.migrationScript})":`);
-      migrationsDebug('%O', context.listChanges());
-
-      await formatFiles(context);
-      flushChanges(context);
-      printChanges(context, key, migration);
+  // run migrations sequentially in version order where lowest version runs first
+  for (const migration of migrations) {
+    const context = await runCodemod(migration, options.codemodOptions);
+    const shouldCommit = options.commitEachMigration && context.hasChanges();
 
-      installNPMDependencies(context);
-
-      if (shouldCommit) {
-        await gitCommitNoVerify(`chore: run create-plugin migration - ${key} (${migration.migrationScript})`);
-      }
-    } catch (error) {
-      if (error instanceof Error) {
-        throw new Error(`Error running migration "${key} (${migration.migrationScript})": ${error.message}`);
-      }
+    if (shouldCommit) {
+      await gitCommitNoVerify(`chore: run create-plugin migration - ${migration.name} (${migration.scriptPath})`);
     }
   }
 
@@ -75,9 +48,3 @@ export async function runMigrations(migrations: Record<string, MigrationMeta>, o
     await gitCommitNoVerify(`chore: update .config/.cprc.json to version ${CURRENT_APP_VERSION}.`);
   }
 }
-
-export async function runMigration(migration: MigrationMeta, context: Context): Promise<Context> {
-  const module = (await import(migration.migrationScript)) as MigrationModule;
-
-  return module.default(context);
-}

package/src/codemods/migrations/migrations.test.ts

@@ -1,12 +1,15 @@
+import { existsSync } from 'node:fs';
 import defaultMigrations from './migrations.js';
 
 describe('migrations json', () => {
   // As migration scripts are imported dynamically when update is run we assert the path is valid
-  Object.entries(defaultMigrations.migrations).forEach(([key, migration]) => {
-    it(`should have a valid migration script path for ${key}`, () => {
-      expect(async () => {
-        await import(migration.migrationScript);
-      }).not.toThrow();
+  // Vitest 4 reimplemented its workers, which caused the previous dynamic import tests to fail.
+  // This test now only asserts that the migration script source file exists.
+  defaultMigrations.forEach((migration) => {
+    it(`should have a valid migration script path for ${migration.name}`, () => {
+      // ensure migration script exists
+      const sourceFilePath = migration.scriptPath.replace('.js', '.ts');
+      expect(existsSync(sourceFilePath)).toBe(true);
     });
   });
 });

package/src/codemods/migrations/migrations.ts

@@ -1,39 +1,43 @@
 import { LEGACY_UPDATE_CUTOFF_VERSION } from '../../constants.js';
+import { Codemod } from '../types.js';
+import { resolveScriptPath } from '../utils.js';
 
-export type MigrationMeta = {
+export interface Migration extends Codemod {
   version: string;
-  description: string;
-  migrationScript: string;
-};
+}
 
-type Migrations = {
-  migrations: Record<string, MigrationMeta>;
-};
-
-export default {
-  migrations: {
-    '001-update-grafana-compose-extend': {
-      version: LEGACY_UPDATE_CUTOFF_VERSION,
-      description: 'Update ./docker-compose.yaml to extend from ./.config/docker-compose-base.yaml.',
-      migrationScript: './scripts/001-update-grafana-compose-extend.js',
-    },
-    '002-update-is-compatible-workflow': {
-      version: LEGACY_UPDATE_CUTOFF_VERSION,
-      description:
-        'Update ./.github/workflows/is-compatible.yml to use is-compatible github action instead of calling levitate directly',
-      migrationScript: './scripts/002-update-is-compatible-workflow.js',
-    },
-    '003-update-eslint-deprecation-rule': {
-      version: LEGACY_UPDATE_CUTOFF_VERSION,
-      description: 'Replace deprecated eslint-plugin-deprecation with @typescript-eslint/no-deprecated rule.',
-      migrationScript: './scripts/003-update-eslint-deprecation-rule.js',
-    },
-    '004-eslint9-flat-config': {
-      version: LEGACY_UPDATE_CUTOFF_VERSION,
-      description: 'Migrate eslint config to flat config format and update devDependencies to latest versions.',
-      migrationScript: './scripts/004-eslint9-flat-config.js',
-    },
-    // Do not use LEGACY_UPDATE_CUTOFF_VERSION for new migrations. It is only used above to force migrations to run
-    // for those written before the switch to updates as migrations.
+export default [
+  {
+    name: '001-update-grafana-compose-extend',
+    version: LEGACY_UPDATE_CUTOFF_VERSION,
+    description: 'Update ./docker-compose.yaml to extend from ./.config/docker-compose-base.yaml.',
+    scriptPath: resolveScriptPath(import.meta.url, './scripts/001-update-grafana-compose-extend.js'),
+  },
+  {
+    name: '002-update-is-compatible-workflow',
+    version: LEGACY_UPDATE_CUTOFF_VERSION,
+    description:
+      'Update ./.github/workflows/is-compatible.yml to use is-compatible github action instead of calling levitate directly',
+    scriptPath: resolveScriptPath(import.meta.url, './scripts/002-update-is-compatible-workflow.js'),
+  },
+  {
+    name: '003-update-eslint-deprecation-rule',
+    version: LEGACY_UPDATE_CUTOFF_VERSION,
+    description: 'Replace deprecated eslint-plugin-deprecation with @typescript-eslint/no-deprecated rule.',
+    scriptPath: resolveScriptPath(import.meta.url, './scripts/003-update-eslint-deprecation-rule.js'),
+  },
+  {
+    name: '004-eslint9-flat-config',
+    version: LEGACY_UPDATE_CUTOFF_VERSION,
+    description: 'Migrate eslint config to flat config format and update devDependencies to latest versions.',
+    scriptPath: resolveScriptPath(import.meta.url, './scripts/004-eslint9-flat-config.js'),
+  },
+  {
+    name: '005-react-18-3',
+    version: '6.1.9',
+    description: 'Update React and ReactDOM 18.x versions to ^18.3.0 to surface React 19 compatibility issues.',
+    scriptPath: resolveScriptPath(import.meta.url, './scripts/005-react-18-3.js'),
   },
-} as Migrations;
+  // Do not use LEGACY_UPDATE_CUTOFF_VERSION for new migrations. It is only used above to force migrations to run
+  // for those written before the switch to updates as migrations.
+] satisfies Migration[];

package/src/codemods/migrations/scripts/004-eslint9-flat-config.ts

@@ -7,7 +7,7 @@ import { dirname, relative, resolve } from 'node:path';
 import * as recast from 'recast';
 import type { Context } from '../../context.js';
 import { addDependenciesToPackageJson } from '../../utils.js';
-import { migrationsDebug } from '../utils.js';
+// migrationsDebug removed - was from deleted migrations/utils.js
 
 type Imports = Map<string, { name?: string; bindings?: string[] }>;
 
@@ -407,7 +407,7 @@ function getIgnorePaths(context: Context): string[] {
         }
       }
     } catch (error) {
-      migrationsDebug('Error parsing package.json: %s', error);
+      console.log('Error parsing package.json: %s', error);
     }
   }
 

package/src/codemods/migrations/scripts/005-react-18-3.test.ts

@@ -0,0 +1,145 @@
+import { describe, it, expect } from 'vitest';
+import { Context } from '../../context.js';
+import migrate from './005-react-18-3.js';
+
+describe('005-react-18-3', () => {
+  it('should not modify anything if package.json does not exist', async () => {
+    const context = new Context('/virtual');
+    await migrate(context);
+    expect(context.listChanges()).toEqual({});
+  });
+
+  it('should not modify anything if there are no React dependencies', async () => {
+    const context = new Context('/virtual');
+    context.addFile(
+      './package.json',
+      JSON.stringify({
+        dependencies: {
+          lodash: '^4.17.21',
+        },
+      })
+    );
+    const initialChanges = context.listChanges();
+    await migrate(context);
+    expect(context.listChanges()).toEqual(initialChanges);
+  });
+
+  it('should not update React if version is below 18.0.0', async () => {
+    const context = new Context('/virtual');
+    const packageJson = {
+      dependencies: {
+        react: '^17.0.2',
+        'react-dom': '^17.0.2',
+      },
+      devDependencies: {
+        '@types/react': '^17.0.0',
+        '@types/react-dom': '^17.0.0',
+      },
+    };
+    context.addFile('./package.json', JSON.stringify(packageJson, null, 2));
+    const initialPackageJson = context.getFile('./package.json');
+
+    await migrate(context);
+
+    expect(context.getFile('./package.json')).toBe(initialPackageJson);
+  });
+
+  it('should update React 18.0.0 to ^18.3.0', async () => {
+    const context = new Context('/virtual');
+    context.addFile(
+      './package.json',
+      JSON.stringify({
+        dependencies: {
+          react: '^18.0.0',
+        },
+      })
+    );
+
+    await migrate(context);
+
+    const updatedPackageJson = JSON.parse(context.getFile('./package.json') || '{}');
+    expect(updatedPackageJson.dependencies.react).toBe('^18.3.0');
+    expect(updatedPackageJson.devDependencies?.['@types/react']).toBe('^18.3.0');
+  });
+
+  it('should not update React if version is already 18.3.0 or higher', async () => {
+    const context = new Context('/virtual');
+    const packageJson = {
+      dependencies: {
+        react: '^18.3.0',
+        'react-dom': '^18.3.0',
+      },
+      devDependencies: {
+        '@types/react': '^18.3.0',
+        '@types/react-dom': '^18.3.0',
+      },
+    };
+    context.addFile('./package.json', JSON.stringify(packageJson, null, 2));
+
+    await migrate(context);
+
+    const updatedPackageJson = JSON.parse(context.getFile('./package.json') || '{}');
+    expect(updatedPackageJson.dependencies.react).toBe('^18.3.0');
+    expect(updatedPackageJson.dependencies['react-dom']).toBe('^18.3.0');
+  });
+
+  it('should not downgrade React 19.0.0 to 18.3.0', async () => {
+    const context = new Context('/virtual');
+    const packageJson = {
+      dependencies: {
+        react: '^19.0.0',
+        'react-dom': '^19.0.0',
+      },
+      devDependencies: {
+        '@types/react': '^19.0.0',
+        '@types/react-dom': '^19.0.0',
+      },
+    };
+    context.addFile('./package.json', JSON.stringify(packageJson, null, 2));
+
+    await migrate(context);
+
+    const updatedPackageJson = JSON.parse(context.getFile('./package.json') || '{}');
+    expect(updatedPackageJson.dependencies.react).toBe('^19.0.0');
+    expect(updatedPackageJson.dependencies['react-dom']).toBe('^19.0.0');
+    expect(updatedPackageJson.devDependencies['@types/react']).toBe('^19.0.0');
+    expect(updatedPackageJson.devDependencies['@types/react-dom']).toBe('^19.0.0');
+  });
+
+  it('should handle version ranges correctly', async () => {
+    const context = new Context('/virtual');
+    context.addFile(
+      './package.json',
+      JSON.stringify({
+        dependencies: {
+          react: '~18.1.0',
+          'react-dom': '18.2.0',
+        },
+      })
+    );
+
+    await migrate(context);
+
+    const updatedPackageJson = JSON.parse(context.getFile('./package.json') || '{}');
+    expect(updatedPackageJson.dependencies.react).toBe('^18.3.0');
+    expect(updatedPackageJson.dependencies['react-dom']).toBe('^18.3.0');
+  });
+
+  it('should be idempotent', async () => {
+    const context = new Context('/virtual');
+    context.addFile(
+      './package.json',
+      JSON.stringify({
+        dependencies: {
+          react: '^18.2.0',
+          'react-dom': '^18.2.0',
+        },
+        devDependencies: {
+          '@types/react': '^18.2.0',
+          '@types/react-dom': '^18.2.0',
+        },
+      })
+    );
+    await expect(migrate).toBeIdempotent(context);
+  });
+});

package/src/codemods/migrations/scripts/005-react-18-3.ts

@@ -0,0 +1,19 @@
+import type { Context } from '../../context.js';
+import { addDependenciesToPackageJson, isVersionGreater } from '../../utils.js';
+
+export default function migrate(context: Context) {
+  if (context.doesFileExist('package.json')) {
+    const packageJson = JSON.parse(context.getFile('package.json') || '{}');
+    if (packageJson.dependencies?.react) {
+      if (isVersionGreater(packageJson.dependencies.react, '18.0.0', true)) {
+        addDependenciesToPackageJson(context, { react: '^18.3.0' }, { '@types/react': '^18.3.0' });
+      }
+    }
+    if (packageJson.dependencies?.['react-dom']) {
+      if (isVersionGreater(packageJson.dependencies['react-dom'], '18.0.0', true)) {
+        addDependenciesToPackageJson(context, { 'react-dom': '^18.3.0' }, { '@types/react-dom': '^18.3.0' });
+      }
+    }
+  }
+  return context;
+}

package/src/codemods/migrations/scripts/example-migration.test.ts

@@ -14,7 +14,7 @@ describe('Migration - append profile to webpack', () => {
       })
     );
 
-    const updatedContext = await migrate(context);
+    const updatedContext = await migrate(context, { profile: true, skipBackup: false, verbose: false });
 
     expect(updatedContext.getFile('./package.json')).toMatch(
       'webpack -c ./.config/webpack/webpack.config.ts --profile --env production'

package/src/codemods/migrations/scripts/example-migration.ts

@@ -1,6 +1,23 @@
 import type { Context } from '../../context.js';
 
-export default function migrate(context: Context): Context {
+/**
+ * Example migration that demonstrates basic context operations
+ * This example shows how to modify package.json, add/delete files, and rename files
+ */
+
+type MigrateOptions = {
+  profile?: boolean;
+  skipBackup?: boolean;
+  verbose?: boolean;
+};
+
+export default function migrate(context: Context, options: MigrateOptions = {}): Context {
+  const { profile = false, skipBackup = false, verbose = false } = options;
+
+  if (verbose) {
+    console.log('Running migration with options:', options);
+  }
+
   const rawPkgJson = context.getFile('./package.json') ?? '{}';
   const packageJson = JSON.parse(rawPkgJson);
 
@@ -9,14 +26,14 @@ export default function migrate(context: Context): Context {
 
     const pattern = /(webpack.+-c\s.+\.ts)\s(.+)/;
 
-    if (pattern.test(buildScript) && !buildScript.includes('--profile')) {
+    if (profile && pattern.test(buildScript) && !buildScript.includes('--profile')) {
       packageJson.scripts.build = buildScript.replace(pattern, `$1 --profile $2`);
     }
 
     context.updateFile('./package.json', JSON.stringify(packageJson, null, 2));
   }
 
-  if (context.doesFileExist('./src/README.md')) {
+  if (!skipBackup && context.doesFileExist('./src/README.md')) {
     context.deleteFile('./src/README.md');
   }