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

package/dist/commands/update.command.js

@@ -1,16 +1,15 @@
 import { getMigrationsToRun, runMigrations } from '../codemods/migrations/manager.js';
 import { getPackageManagerWithFallback, getPackageManagerExecCmd, getPackageManagerSilentInstallCmd } from '../utils/utils.packageManager.js';
 import { lt, gte } from 'semver';
-import { isGitDirectory, isGitDirectoryClean } from '../utils/utils.git.js';
+import { performPreCodemodChecks } from '../utils/utils.checks.js';
 import { CURRENT_APP_VERSION } from '../utils/utils.version.js';
 import { LEGACY_UPDATE_CUTOFF_VERSION } from '../constants.js';
 import { getConfig } from '../utils/utils.config.js';
-import { isPluginDirectory } from '../utils/utils.plugin.js';
 import { output } from '../utils/utils.console.js';
 import { spawnSync } from 'node:child_process';
 
 const update = async (argv) => {
-  await performPreUpdateChecks(argv);
+  await performPreCodemodChecks(argv);
   const { version } = getConfig();
   if (lt(version, LEGACY_UPDATE_CUTOFF_VERSION)) {
     preparePluginForMigrations(argv);
@@ -22,9 +21,12 @@ const update = async (argv) => {
       });
       process.exit(0);
     }
-    const commitEachMigration = argv.commit;
     const migrations = getMigrationsToRun(version, CURRENT_APP_VERSION);
-    await runMigrations(migrations, { commitEachMigration });
+    const { _, $0, ...codemodOptions } = argv;
+    await runMigrations(migrations, {
+      commitEachMigration: !!argv.commit,
+      codemodOptions
+    });
     output.success({
       title: `Successfully updated create-plugin from ${version} to ${CURRENT_APP_VERSION}.`
     });
@@ -38,42 +40,6 @@ const update = async (argv) => {
     process.exit(1);
   }
 };
-async function performPreUpdateChecks(argv) {
-  if (!await isGitDirectory() && !argv.force) {
-    output.error({
-      title: "You are not inside a git directory",
-      body: [
-        `In order to proceed please run ${output.formatCode("git init")} in the root of your project and commit your changes.`,
-        `(This check is necessary to make sure that the updates are easy to revert and don't interfere with any changes you currently have.`,
-        `In case you want to proceed as is please use the ${output.formatCode("--force")} flag.)`
-      ]
-    });
-    process.exit(1);
-  }
-  if (!await isGitDirectoryClean() && !argv.force) {
-    output.error({
-      title: "Please clean your repository working tree before updating.",
-      body: [
-        "Commit your changes or stash them.",
-        `(This check is necessary to make sure that the updates are easy to revert and don't mess with any changes you currently have.`,
-        `In case you want to proceed as is please use the ${output.formatCode("--force")} flag.)`
-      ]
-    });
-    process.exit(1);
-  }
-  if (!isPluginDirectory() && !argv.force) {
-    output.error({
-      title: "Are you inside a plugin directory?",
-      body: [
-        `We couldn't find a "src/plugin.json" file under your current directory.`,
-        `(Please make sure to run this command from the root of your plugin folder. In case you want to proceed as is please use the ${output.formatCode(
-          "--force"
-        )} flag.)`
-      ]
-    });
-    process.exit(1);
-  }
-}
 function preparePluginForMigrations(argv) {
   const { packageManagerName, packageManagerVersion } = getPackageManagerWithFallback();
   const packageManagerExecCmd = getPackageManagerExecCmd(packageManagerName, packageManagerVersion);

package/dist/utils/utils.checks.js

@@ -0,0 +1,40 @@
+import { isGitDirectory, isGitDirectoryClean } from './utils.git.js';
+import { isPluginDirectory } from './utils.plugin.js';
+import { output } from './utils.console.js';
+
+async function performPreCodemodChecks(argv) {
+  if (!await isGitDirectory() && !argv.force) {
+    output.error({
+      title: "You are not inside a git directory",
+      body: [
+        `In order to proceed please run ${output.formatCode("git init")} in the root of your project and commit your changes.`,
+        `(This check is necessary to make sure that changes are easy to revert and don't interfere with any changes you currently have.`,
+        `In case you want to proceed as is please use the ${output.formatCode("--force")} flag.)`
+      ]
+    });
+    process.exit(1);
+  }
+  if (!await isGitDirectoryClean() && !argv.force) {
+    output.error({
+      title: "Please clean your repository working tree before making changes.",
+      body: [
+        "Commit your changes or stash them.",
+        `(This check is necessary to make sure that changes are easy to revert and don't mess with any changes you currently have.`,
+        `In case you want to proceed as is please use the ${output.formatCode("--force")} flag.)`
+      ]
+    });
+    process.exit(1);
+  }
+  if (!isPluginDirectory() && !argv.force) {
+    output.error({
+      title: "Are you inside a plugin directory?",
+      body: [
+        `We couldn't find a "src/plugin.json" file under your current directory.`,
+        `(Please make sure to run this command from the root of your plugin folder. In case you want to proceed as is please use the ${output.formatCode("--force")} flag.)`
+      ]
+    });
+    process.exit(1);
+  }
+}
+
+export { performPreCodemodChecks };

package/dist/utils/utils.config.js

@@ -8,9 +8,6 @@ import path from 'node:path';
 import { writeFile } from 'node:fs/promises';
 import { EOL } from 'node:os';
 
-function isFeatureEnabled(features, featureName) {
-  return features[featureName] === true;
-}
 let hasShownConfigWarnings = false;
 function getConfig(workDir = process.cwd()) {
   const rootConfig = getRootConfig(workDir);
@@ -95,17 +92,5 @@ async function setRootConfig(configOverride = {}) {
   await writeFile(rootConfigPath, JSON.stringify(updatedConfig, null, 2) + EOL);
   return updatedConfig;
 }
-async function setFeatureFlag(featureName, enabled = true) {
-  const userConfig = getUserConfig() || { features: {} };
-  const userConfigPath = path.resolve(process.cwd(), ".cprc.json");
-  const updatedConfig = {
-    ...userConfig,
-    features: {
-      ...userConfig.features,
-      [featureName]: enabled
-    }
-  };
-  await writeFile(userConfigPath, JSON.stringify(updatedConfig, null, 2) + EOL);
-}
 
-export { getConfig, isFeatureEnabled, setFeatureFlag, setRootConfig };
+export { getConfig, setRootConfig };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@grafana/create-plugin",
-  "version": "6.2.0-canary.2233.19133609453.0",
+  "version": "6.2.0-canary.2233.19368311379.0",
   "repository": {
     "directory": "packages/create-plugin",
     "url": "https://github.com/grafana/plugin-tools"
@@ -38,6 +38,7 @@
     "recast": "^0.23.11",
     "semver": "^7.3.5",
     "title-case": "^4.3.0",
+    "valibot": "^1.1.0",
     "which": "^5.0.0",
     "yaml": "^2.7.0"
   },
@@ -59,5 +60,5 @@
   "engines": {
     "node": ">=20"
   },
-  "gitHead": "d84f5bcb2be40efa1f28ad4a86b52a8a1566e85c"
+  "gitHead": "265f74e1e4a843bdeaad5300a13b94391ffd5fff"
 }

package/src/codemods/additions/additions.test.ts

@@ -0,0 +1,12 @@
+import defaultAdditions from './additions.js';
+
+describe('additions json', () => {
+  // as addition scripts are imported dynamically when add is run we assert the path is valid
+  defaultAdditions.forEach((addition) => {
+    it(`should have a valid addition script path for ${addition.name}`, () => {
+      expect(async () => {
+        await import(addition.scriptPath);
+      }).not.toThrow();
+    });
+  });
+});

package/src/codemods/additions/additions.ts

@@ -1,23 +1,10 @@
-export type AdditionMeta<TFeatureName extends string = string> = {
-  name: string;
-  description: string;
-  scriptPath: string;
-  featureName: TFeatureName;
-};
-
-const additions = {
-  i18n: {
-    name: 'i18n',
-    description: 'Add internationalization (i18n) support to your plugin',
-    scriptPath: './scripts/add-i18n.js',
-    featureName: 'i18nEnabled',
+import { Codemod } from '../types.js';
+import { resolveScriptPath } from '../utils.js';
+
+export default [
+  {
+    name: 'example-addition',
+    description: 'Example addition demonstrating Valibot schema with type inference',
+    scriptPath: resolveScriptPath(import.meta.url, './scripts/example-addition.js'),
   },
-};
-
-export default { additions };
-
-// extract feature names for type-safe FeatureFlags in utils.config.ts
-type AdditionValues = (typeof additions)[keyof typeof additions];
-export type AdditionFeatureName = AdditionValues['featureName'];
-
-export type TypedAdditionMeta = AdditionMeta<AdditionFeatureName>;
+] satisfies Codemod[];

package/src/codemods/additions/scripts/example-addition.test.ts

@@ -4,23 +4,12 @@ import { Context } from '../../context.js';
 import migrate from './example-addition.js';
 
 describe('example-addition', () => {
-  it('should be idempotent', async () => {
-    const context = new Context('/virtual');
-
-    // Set up a minimal project structure
-    context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
-    context.addFile('src/index.ts', 'export const foo = "bar";');
-
-    const migrateWithOptions = (ctx: Context) => migrate(ctx, { featureName: 'testFeature', enabled: true });
-    await expect(migrateWithOptions).toBeIdempotent(context);
-  });
-
   it('should add example script to package.json', () => {
     const context = new Context('/virtual');
 
     context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
 
-    const result = migrate(context, { featureName: 'testFeature', enabled: true });
+    const result = migrate(context, { featureName: 'testFeature', enabled: true, frameworks: ['react'] });
 
     const packageJson = JSON.parse(result.getFile('package.json') || '{}');
     expect(packageJson.scripts['example-script']).toBe('echo "Running testFeature"');
@@ -31,39 +20,31 @@ describe('example-addition', () => {
 
     context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
 
-    const result = migrate(context, { featureName: 'myFeature', enabled: false });
+    const result = migrate(context, { featureName: 'myFeature', enabled: false, frameworks: ['react'] });
 
     const packageJson = JSON.parse(result.getFile('package.json') || '{}');
     expect(packageJson.devDependencies['example-dev-dep']).toBe('^1.0.0');
   });
 
-  it('should create config file with feature settings', () => {
+  it('should create feature TypeScript file with options', () => {
     const context = new Context('/virtual');
 
     context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
 
-    const result = migrate(context, { featureName: 'coolFeature', enabled: true });
-
-    expect(result.doesFileExist('src/config.json')).toBe(true);
-    const config = JSON.parse(result.getFile('src/config.json') || '{}');
-    expect(config.features.coolFeature).toEqual({
-      enabled: true,
-      description: 'Example feature configuration',
+    const result = migrate(context, {
+      featureName: 'myFeature',
+      enabled: false,
+      port: 4000,
+      frameworks: ['react', 'vue'],
     });
-  });
-
-  it('should create feature TypeScript file', () => {
-    const context = new Context('/virtual');
-
-    context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
-
-    const result = migrate(context, { featureName: 'myFeature', enabled: false });
 
     expect(result.doesFileExist('src/features/myFeature.ts')).toBe(true);
     const featureCode = result.getFile('src/features/myFeature.ts');
     expect(featureCode).toContain('export const myFeature');
     expect(featureCode).toContain('enabled: false');
-    expect(featureCode).toContain('myFeature initialized');
+    expect(featureCode).toContain('port: 4000');
+    expect(featureCode).toContain('frameworks: ["react","vue"]');
+    expect(featureCode).toContain('myFeature initialized on port 4000');
   });
 
   it('should delete deprecated file if it exists', () => {
@@ -72,7 +53,7 @@ describe('example-addition', () => {
     context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
     context.addFile('src/deprecated.ts', 'export const old = true;');
 
-    const result = migrate(context, { featureName: 'testFeature', enabled: true });
+    const result = migrate(context, { featureName: 'testFeature', enabled: true, frameworks: ['react'] });
 
     expect(result.doesFileExist('src/deprecated.ts')).toBe(false);
   });
@@ -83,7 +64,7 @@ describe('example-addition', () => {
     context.addFile('package.json', JSON.stringify({ scripts: {}, dependencies: {}, devDependencies: {} }));
     context.addFile('src/old-config.json', JSON.stringify({ old: true }));
 
-    const result = migrate(context, { featureName: 'testFeature', enabled: true });
+    const result = migrate(context, { featureName: 'testFeature', enabled: true, frameworks: ['react'] });
 
     expect(result.doesFileExist('src/old-config.json')).toBe(false);
     expect(result.doesFileExist('src/new-config.json')).toBe(true);
@@ -103,7 +84,7 @@ describe('example-addition', () => {
       })
     );
 
-    const result = migrate(context, { featureName: 'testFeature', enabled: true });
+    const result = migrate(context, { featureName: 'testFeature', enabled: true, frameworks: ['react'] });
 
     const packageJson = JSON.parse(result.getFile('package.json') || '{}');
     expect(packageJson.scripts['example-script']).toBe('existing command');

package/src/codemods/additions/scripts/example-addition.ts

@@ -1,37 +1,30 @@
+import * as v from 'valibot';
 import type { Context } from '../../context.js';
-import type { FlagDefinition } from '../../types.js';
 import { addDependenciesToPackageJson } from '../../utils.js';
 
-export type ExampleOptions = {
-  featureName: string;
-  enabled: boolean;
-};
-
-export const flags: FlagDefinition[] = [
-  {
-    name: 'feature-name',
-    description: 'Name of the feature to add',
-    required: false,
-  },
-  {
-    name: 'enabled',
-    description: 'Whether the feature should be enabled by default',
-    required: false,
-  },
-];
-
-export function parseFlags(argv: any): ExampleOptions {
-  return {
-    featureName: argv['feature-name'] || 'myFeature',
-    enabled: argv.enabled === 'true' || argv.enabled === true,
-  };
-}
-
-export default function migrate(
-  context: Context,
-  options: ExampleOptions = { featureName: 'myFeature', enabled: true }
-): Context {
-  const { featureName, enabled } = options;
+/**
+ * Example addition demonstrating Valibot schema with type inference
+ * Schema defines validation rules, defaults and types are automatically inferred
+ */
+export const schema = v.object({
+  featureName: v.pipe(
+    v.string(),
+    v.minLength(3, 'Feature name must be at least 3 characters'),
+    v.maxLength(50, 'Feature name must be at most 50 characters')
+  ),
+  enabled: v.optional(v.boolean(), true),
+  port: v.optional(
+    v.pipe(v.number(), v.minValue(1000, 'Port must be at least 1000'), v.maxValue(65535, 'Port must be at most 65535'))
+  ),
+  frameworks: v.optional(v.array(v.string()), ['react']),
+});
+
+// Type is automatically inferred from the schema
+type ExampleOptions = v.InferOutput<typeof schema>;
+
+export default function exampleAddition(context: Context, options: ExampleOptions): Context {
+  // These options have been validated by the framework
+  const { featureName, enabled, port, frameworks } = options;
 
   const rawPkgJson = context.getFile('./package.json') ?? '{}';
   const packageJson = JSON.parse(rawPkgJson);
@@ -43,24 +36,14 @@ export default function migrate(
 
   addDependenciesToPackageJson(context, {}, { 'example-dev-dep': '^1.0.0' });
 
-  if (!context.doesFileExist('./src/config.json')) {
-    const config = {
-      features: {
-        [featureName]: {
-          enabled,
-          description: 'Example feature configuration',
-        },
-      },
-    };
-    context.addFile('./src/config.json', JSON.stringify(config, null, 2));
-  }
-
   if (!context.doesFileExist(`./src/features/${featureName}.ts`)) {
     const featureCode = `export const ${featureName} = {
   name: '${featureName}',
   enabled: ${enabled},
+  port: ${port ?? 3000},
+  frameworks: ${JSON.stringify(frameworks)},
   init() {
-    console.log('${featureName} initialized');
+    console.log('${featureName} initialized on port ${port ?? 3000}');
   },
 };
 `;

package/src/codemods/migrations/fixtures/migrations.ts

@@ -1,19 +1,20 @@
-export default {
-  migrations: {
-    'migration-key1': {
-      version: '5.0.0',
-      description: 'Update project to use new cache directory',
-      migrationScript: './5-0-0-cache-directory.js',
-    },
-    'migration-key2': {
-      version: '5.4.0',
-      description: 'Update project to use new cache directory',
-      migrationScript: './5-4-0-cache-directory.js',
-    },
-    'migration-key3': {
-      version: '6.0.0',
-      description: 'Update project to use new cache directory',
-      migrationScript: './5-4-0-cache-directory.js',
-    },
+export default [
+  {
+    name: 'migration-key1',
+    version: '5.0.0',
+    description: 'Update project to use new cache directory',
+    scriptPath: './5-0-0-cache-directory.js',
   },
-};
+  {
+    name: 'migration-key2',
+    version: '5.4.0',
+    description: 'Update project to use new cache directory',
+    scriptPath: './5-4-0-cache-directory.js',
+  },
+  {
+    name: 'migration-key3',
+    version: '6.0.0',
+    description: 'Update project to use new cache directory',
+    scriptPath: './5-4-0-cache-directory.js',
+  },
+];