@launch77/cli 1.4.4 → 1.6.0

package/src/modules/deploy/commands/deploy-init-action.ts

@@ -6,10 +6,10 @@
 import * as path from 'path'
 
 import chalk from 'chalk'
+import { detectLaunch77Context } from '@launch77/plugin-runtime'
 import fs from 'fs-extra'
 import ora from 'ora'
 
-import { detectMonorepoContext } from '../../../utils/monorepo.js'
 import { DeployService } from '../services/deploy-svc.js'
 import { connectGitRepository, getPackageName, getVercelInstallInstructions, isVercelCliInstalled, loadVercelProject, runVercelCommand, updateProjectRootDirectory, verifyVercelProjectExists } from '../utils/vercel-extended.js'
 
@@ -23,16 +23,15 @@ interface DeployInitOptions {
 export async function deployInit(options: DeployInitOptions = {}) {
   const deployService = new DeployService()
 
-  // Detect monorepo context
-  const context = await detectMonorepoContext(process.cwd())
+  // Detect Launch77 context
+  const context = await detectLaunch77Context(process.cwd())
 
-  if (context.location !== 'startup-app') {
-    throw new Error('Must be run from within an app directory (e.g., startups/mycompany/apps/myapp)')
+  if (context.locationType !== 'workspace-app') {
+    throw new Error('Must be run from within an app directory (e.g., apps/myapp)')
   }
 
   const appPath = process.cwd()
   const appName = context.appName!
-  const startupName = context.startupName!
 
   console.log(chalk.blue(`\n🚀 Initializing Vercel deployment for ${appName}...\n`))
 
@@ -116,23 +115,27 @@ export async function deployInit(options: DeployInitOptions = {}) {
     throw new Error('Could not read package name from package.json')
   }
 
+  // Check template type from app's package.json
+  const appPackageJsonPath = path.join(appPath, 'package.json')
+  const appPackageJson = await fs.readJson(appPackageJsonPath)
+  const templatePackage = appPackageJson.launch77?.template?.package
+
   console.log(chalk.gray(`App details:`))
   console.log(chalk.gray(`  Name: ${manifest.name}`))
-  console.log(chalk.gray(`  Type: ${manifest.type}`))
   console.log(chalk.gray(`  Package: ${packageName}`))
+  if (templatePackage) {
+    console.log(chalk.gray(`  Template: ${templatePackage}`))
+  }
   console.log()
 
-  // Check if this is an API app - Railway deployment coming soon
-  if (manifest.type === 'api') {
-    console.log(chalk.yellow('⚠️  Railway deployments for API apps are not yet supported'))
-    console.log()
-    console.log(chalk.cyan('Coming soon! For now, you can deploy manually to:'))
-    console.log(chalk.white('  • Railway.app'), chalk.gray('- Simple deployment with databases'))
-    console.log(chalk.white('  • Fly.io'), chalk.gray('- Global edge deployment'))
-    console.log(chalk.white('  • Render.com'), chalk.gray('- Managed cloud hosting'))
+  // Only allow webapp apps to deploy to Vercel
+  const isWebapp = templatePackage === 'webapp' || templatePackage === '@launch77-shared/app-template-webapp'
+
+  if (!isWebapp) {
+    console.log(chalk.yellow('⚠️  Vercel deployment only supports webapp apps'))
+    console.log(chalk.gray('Other app types coming soon'))
     console.log()
-    console.log(chalk.gray('Stay tuned for automated Railway deployment support!'))
-    process.exit(0)
+    process.exit(1)
   }
 
   // Check if vercel.json exists
@@ -160,7 +163,7 @@ export async function deployInit(options: DeployInitOptions = {}) {
   console.log()
 
   try {
-    const suggestedProjectName = `${startupName}--${appName}`
+    const suggestedProjectName = appName
     await runVercelCommand(['link', '--project', suggestedProjectName, '--yes'], appPath)
 
     console.log()
@@ -172,7 +175,7 @@ export async function deployInit(options: DeployInitOptions = {}) {
 
   // Read the created .vercel/project.json to get project info
   const vercelProject = await loadVercelProject(appPath)
-  const projectName = vercelProject?.projectName || `${startupName}--${appName}`
+  const projectName = vercelProject?.projectName || appName
 
   // Update manifest with deployment configuration
   manifest.deployment = {
@@ -187,7 +190,7 @@ export async function deployInit(options: DeployInitOptions = {}) {
 
   // Set Root Directory for monorepo deployments
   console.log()
-  const rootDirectory = `startups/${startupName}/apps/${appName}`
+  const rootDirectory = `apps/${appName}`
   await updateProjectRootDirectory(projectName, rootDirectory)
 
   // Automatically connect to Git repository

package/src/modules/deploy/commands/deploy-logs-action.ts

@@ -3,9 +3,12 @@
  */
 
 /* eslint-disable no-console */
+import * as path from 'path'
+
 import chalk from 'chalk'
+import { detectLaunch77Context } from '@launch77/plugin-runtime'
+import fs from 'fs-extra'
 
-import { detectMonorepoContext } from '../../../utils/monorepo.js'
 import { DeployService } from '../services/deploy-svc.js'
 import { getVercelInstallInstructions, isVercelCliInstalled, runVercelCommand } from '../utils/vercel-extended.js'
 
@@ -20,11 +23,11 @@ export interface DeployLogsOptions {
 export async function deployLogs(options: DeployLogsOptions) {
   const deployService = new DeployService()
 
-  // Detect monorepo context
-  const context = await detectMonorepoContext(process.cwd())
+  // Detect Launch77 context
+  const context = await detectLaunch77Context(process.cwd())
 
-  if (context.location !== 'startup-app') {
-    throw new Error('Must be run from within an app directory (e.g., startups/mycompany/apps/myapp)')
+  if (context.locationType !== 'workspace-app') {
+    throw new Error('Must be run from within an app directory (e.g., apps/myapp)')
   }
 
   const appPath = process.cwd()
@@ -34,14 +37,17 @@ export async function deployLogs(options: DeployLogsOptions) {
   // Load manifest
   const manifest = await deployService.loadManifest()
 
-  // Check if this is an API app
-  if (manifest.type === 'api') {
-    console.log(chalk.yellow('⚠️  Railway deployments for API apps are not yet supported'))
-    console.log()
-    console.log(chalk.cyan('Coming soon! For now, you can view logs at:'))
-    console.log(chalk.white('  • Railway.app dashboard'))
-    console.log(chalk.white('  • Fly.io dashboard'))
-    console.log(chalk.white('  • Render.com dashboard'))
+  // Check template type from app's package.json
+  const appPackageJsonPath = path.join(appPath, 'package.json')
+  const appPackageJson = await fs.readJson(appPackageJsonPath)
+  const templatePackage = appPackageJson.launch77?.template?.package
+
+  // Only allow webapp apps to deploy to Vercel
+  const isWebapp = templatePackage === 'webapp' || templatePackage === '@launch77-shared/app-template-webapp'
+
+  if (!isWebapp) {
+    console.log(chalk.yellow('⚠️  Vercel deployment only supports webapp apps'))
+    console.log(chalk.gray('Other app types coming soon'))
     console.log()
     return
   }

package/src/modules/deploy/commands/deploy-status-action.ts

@@ -3,10 +3,13 @@
  */
 
 /* eslint-disable no-console */
+import * as path from 'path'
+
 import chalk from 'chalk'
+import { detectLaunch77Context } from '@launch77/plugin-runtime'
+import fs from 'fs-extra'
 import ora from 'ora'
 
-import { detectMonorepoContext } from '../../../utils/monorepo.js'
 import { DeployService } from '../services/deploy-svc.js'
 import { getVercelInstallInstructions, isVercelCliInstalled, runVercelCommand, verifyVercelProjectExists } from '../utils/vercel-extended.js'
 
@@ -16,11 +19,11 @@ import { getVercelInstallInstructions, isVercelCliInstalled, runVercelCommand, v
 export async function deployStatus() {
   const deployService = new DeployService()
 
-  // Detect monorepo context
-  const context = await detectMonorepoContext(process.cwd())
+  // Detect Launch77 context
+  const context = await detectLaunch77Context(process.cwd())
 
-  if (context.location !== 'startup-app') {
-    throw new Error('Must be run from within an app directory (e.g., startups/mycompany/apps/myapp)')
+  if (context.locationType !== 'workspace-app') {
+    throw new Error('Must be run from within an app directory (e.g., apps/myapp)')
   }
 
   const appPath = process.cwd()
@@ -30,14 +33,17 @@ export async function deployStatus() {
   // Load manifest
   const manifest = await deployService.loadManifest()
 
-  // Check if this is an API app
-  if (manifest.type === 'api') {
-    console.log(chalk.yellow('⚠️  Railway deployments for API apps are not yet supported'))
-    console.log()
-    console.log(chalk.cyan('Coming soon! For now, you can check your deployment status at:'))
-    console.log(chalk.white('  • Railway.app dashboard'))
-    console.log(chalk.white('  • Fly.io dashboard'))
-    console.log(chalk.white('  • Render.com dashboard'))
+  // Check template type from app's package.json
+  const appPackageJsonPath = path.join(appPath, 'package.json')
+  const appPackageJson = await fs.readJson(appPackageJsonPath)
+  const templatePackage = appPackageJson.launch77?.template?.package
+
+  // Only allow webapp apps to deploy to Vercel
+  const isWebapp = templatePackage === 'webapp' || templatePackage === '@launch77-shared/app-template-webapp'
+
+  if (!isWebapp) {
+    console.log(chalk.yellow('⚠️  Vercel deployment only supports webapp apps'))
+    console.log(chalk.gray('Other app types coming soon'))
     console.log()
     return
   }

package/src/modules/library/commands/delete-library.ts

@@ -0,0 +1,126 @@
+/* eslint-disable no-console */
+import * as path from 'path'
+
+import chalk from 'chalk'
+import { Command } from 'commander'
+import inquirer from 'inquirer'
+import ora from 'ora'
+
+import { detectLaunch77Context } from '@launch77/plugin-runtime'
+import { LibraryService } from '../services/library-svc.js'
+
+export function libraryDeleteCommand(): Command {
+  const command = new Command('library:delete')
+    .argument('<library-name>', 'Name of the library to delete')
+    .description('Delete a library from the workspace (requires confirmation)')
+    .action(async (libraryName: string) => {
+      try {
+        console.log(chalk.yellow(`\n⚠️  WARNING: You are about to delete the library '${libraryName}'\n`))
+
+        // Detect context
+        const context = await detectLaunch77Context(process.cwd())
+
+        // Create library service
+        const libraryService = new LibraryService()
+
+        // Get library path for display (before deletion)
+        const libraryPath = path.join(context.workspaceRoot, 'libraries', libraryName)
+
+        // Display what will be deleted
+        console.log(chalk.red('Library location:'), libraryPath)
+        console.log(chalk.red('\nThis will permanently delete:'))
+        console.log(chalk.gray('  • All source code and configuration files'))
+        console.log(chalk.gray('  • All node_modules (will be cleaned from monorepo)'))
+        console.log(chalk.gray('  • All build artifacts'))
+
+        console.log(chalk.yellow('\n⚠️  Important notes:'))
+        console.log(chalk.gray('  • Check if other packages depend on this library'))
+        console.log(chalk.gray('  • If published to npm, you may want to deprecate/unpublish it manually'))
+        console.log(chalk.gray('  • Update import statements in dependent packages manually'))
+
+        console.log(chalk.red('\n⚠️  THIS ACTION CANNOT BE UNDONE!\n'))
+
+        // First confirmation: Type library name
+        const { confirmName } = await inquirer.prompt([
+          {
+            type: 'input',
+            name: 'confirmName',
+            message: `Type the library name to confirm:`,
+            validate: (input: string) => {
+              if (input === libraryName) {
+                return true
+              }
+              return `Please type '${libraryName}' exactly to confirm`
+            },
+          },
+        ])
+
+        if (confirmName !== libraryName) {
+          console.log(chalk.gray('\n✖ Deletion cancelled\n'))
+          return
+        }
+
+        // Second confirmation: Type DELETE
+        const { confirmDelete } = await inquirer.prompt([
+          {
+            type: 'input',
+            name: 'confirmDelete',
+            message: 'Type DELETE (all caps) to confirm permanent deletion:',
+            validate: (input: string) => {
+              if (input === 'DELETE') {
+                return true
+              }
+              return 'Please type DELETE (all caps) to confirm'
+            },
+          },
+        ])
+
+        if (confirmDelete !== 'DELETE') {
+          console.log(chalk.gray('\n✖ Deletion cancelled\n'))
+          return
+        }
+
+        // Delete the library
+        const deleteSpinner = ora('Deleting library directory...').start()
+        try {
+          await libraryService.deleteLibrary({ libraryName }, context)
+          deleteSpinner.succeed('Library directory deleted')
+        } catch (error) {
+          deleteSpinner.fail('Failed to delete library directory')
+          throw error
+        }
+
+        // Update dependencies
+        const installSpinner = ora('Updating workspace dependencies...').start()
+        try {
+          // This is already done by library service, just show spinner for consistency
+          installSpinner.succeed('Dependencies updated (package-lock.json cleaned)')
+        } catch (error) {
+          installSpinner.fail('Failed to update dependencies')
+          console.log(chalk.yellow('\n⚠️  You may need to run "npm install" manually at the workspace root\n'))
+        }
+
+        // Success message
+        console.log(chalk.green(`\n✅ Library '${libraryName}' has been deleted\n`))
+
+        // Reminder about cleanup
+        console.log(chalk.yellow('📝 Remember to:'))
+        console.log(chalk.gray('   • Check and update dependent packages (remove from dependencies)'))
+        console.log(chalk.gray('   • Update import statements in code that used this library'))
+        console.log(chalk.gray('   • Consider deprecating/unpublishing from npm if published'))
+        console.log()
+
+        // Reminder about Git
+        console.log(chalk.yellow('📝 Remember to commit the deletion:'))
+        console.log(chalk.gray('   git add -A'))
+        console.log(chalk.gray(`   git commit -m "Remove ${libraryName} library"`))
+        console.log()
+      } catch (error) {
+        const message = error instanceof Error ? error.message : String(error)
+        console.error(chalk.red('Error:'), message)
+        process.exit(1)
+      }
+    })
+
+  return command
+}

package/src/modules/library/errors/library-errors.ts

@@ -0,0 +1,13 @@
+export class LibraryNotFoundError extends Error {
+  constructor(libraryName: string, expectedPath: string) {
+    super(`Library '${libraryName}' does not exist.\n\n` + `Expected location: ${expectedPath}\n\n` + `Available libraries:\n` + `  cd libraries/\n` + `  ls`)
+    this.name = 'LibraryNotFoundError'
+  }
+}
+
+export class InvalidLibraryNameError extends Error {
+  constructor(message: string) {
+    super(message)
+    this.name = 'InvalidLibraryNameError'
+  }
+}

package/src/modules/library/index.ts

@@ -0,0 +1,11 @@
+// Services
+export { LibraryService } from './services/library-svc.js'
+
+// Types
+export type { DeleteLibraryRequest, DeleteLibraryResult } from './types/library-types.js'
+
+// Errors
+export { LibraryNotFoundError, InvalidLibraryNameError } from './errors/library-errors.js'
+
+// Commands
+export { libraryDeleteCommand } from './commands/delete-library.js'

package/src/modules/library/services/library-svc.ts

@@ -0,0 +1,50 @@
+import * as path from 'path'
+
+import fs from 'fs-extra'
+
+import * as npm from '../../../infrastructure/npm.js'
+import { validateWorkspaceContext } from '../../../utils/launch77-validation.js'
+import { validateAppName } from '../../../utils/validation.js'
+import { LibraryNotFoundError, InvalidLibraryNameError } from '../errors/library-errors.js'
+
+import type { DeleteLibraryRequest, DeleteLibraryResult } from '../types/library-types.js'
+import type { Launch77Context } from '@launch77/plugin-runtime'
+
+export class LibraryService {
+  /**
+   * Delete a library from the workspace
+   */
+  async deleteLibrary(request: DeleteLibraryRequest, context: Launch77Context): Promise<DeleteLibraryResult> {
+    const { libraryName } = request
+
+    // 1. Validate library name (reusing app name validation since format is the same)
+    const nameValidation = validateAppName(libraryName)
+    if (!nameValidation.valid) {
+      throw new InvalidLibraryNameError(nameValidation.errorMessage || 'Invalid library name')
+    }
+
+    // 2. Validate workspace context
+    const contextValidation = validateWorkspaceContext(context)
+    if (!contextValidation.valid) {
+      throw new Error(contextValidation.errorMessage || 'Invalid workspace context')
+    }
+
+    // 3. Determine library path
+    const libraryPath = path.join(context.workspaceRoot, 'libraries', libraryName)
+
+    // 4. Check if library exists
+    if (!(await fs.pathExists(libraryPath))) {
+      throw new LibraryNotFoundError(libraryName, libraryPath)
+    }
+
+    // 5. Delete the library directory
+    await fs.remove(libraryPath)
+
+    // 6. Update dependencies
+    await npm.install(context.workspaceRoot)
+
+    return {
+      libraryName,
+    }
+  }
+}

package/src/modules/library/types/library-types.ts

@@ -0,0 +1,7 @@
+export interface DeleteLibraryRequest {
+  libraryName: string
+}
+
+export interface DeleteLibraryResult {
+  libraryName: string
+}

package/src/modules/plugin/commands/delete-plugin.ts

@@ -0,0 +1,128 @@
+/* eslint-disable no-console */
+import * as path from 'path'
+
+import chalk from 'chalk'
+import { Command } from 'commander'
+import inquirer from 'inquirer'
+import ora from 'ora'
+
+import { detectLaunch77Context } from '@launch77/plugin-runtime'
+import { PluginService } from '../services/plugin-svc.js'
+
+export function pluginDeleteCommand(): Command {
+  const command = new Command('plugin:delete')
+    .argument('<plugin-name>', 'Name of the plugin to delete')
+    .description('Delete a plugin from the workspace (requires confirmation)')
+    .action(async (pluginName: string) => {
+      try {
+        console.log(chalk.yellow(`\n⚠️  WARNING: You are about to delete the plugin '${pluginName}'\n`))
+
+        // Detect context
+        const context = await detectLaunch77Context(process.cwd())
+
+        // Create plugin service
+        const pluginService = new PluginService()
+
+        // Get plugin path for display (before deletion)
+        const pluginPath = path.join(context.workspaceRoot, 'plugins', pluginName)
+
+        // Display what will be deleted
+        console.log(chalk.red('Plugin location:'), pluginPath)
+        console.log(chalk.red('\nThis will permanently delete:'))
+        console.log(chalk.gray('  • All source code and configuration files'))
+        console.log(chalk.gray('  • All node_modules (will be cleaned from monorepo)'))
+        console.log(chalk.gray('  • All build artifacts'))
+        console.log(chalk.gray('  • Plugin templates and generators'))
+
+        console.log(chalk.yellow('\n⚠️  Important notes:'))
+        console.log(chalk.gray('  • Check if plugin is installed in other packages'))
+        console.log(chalk.gray('  • If published to npm, you may want to deprecate/unpublish it manually'))
+        console.log(chalk.gray('  • Remove plugin from launch77.installedPlugins in dependent packages manually'))
+
+        console.log(chalk.red('\n⚠️  THIS ACTION CANNOT BE UNDONE!\n'))
+
+        // First confirmation: Type plugin name
+        const { confirmName } = await inquirer.prompt([
+          {
+            type: 'input',
+            name: 'confirmName',
+            message: `Type the plugin name to confirm:`,
+            validate: (input: string) => {
+              if (input === pluginName) {
+                return true
+              }
+              return `Please type '${pluginName}' exactly to confirm`
+            },
+          },
+        ])
+
+        if (confirmName !== pluginName) {
+          console.log(chalk.gray('\n✖ Deletion cancelled\n'))
+          return
+        }
+
+        // Second confirmation: Type DELETE
+        const { confirmDelete } = await inquirer.prompt([
+          {
+            type: 'input',
+            name: 'confirmDelete',
+            message: 'Type DELETE (all caps) to confirm permanent deletion:',
+            validate: (input: string) => {
+              if (input === 'DELETE') {
+                return true
+              }
+              return 'Please type DELETE (all caps) to confirm'
+            },
+          },
+        ])
+
+        if (confirmDelete !== 'DELETE') {
+          console.log(chalk.gray('\n✖ Deletion cancelled\n'))
+          return
+        }
+
+        // Delete the plugin
+        const deleteSpinner = ora('Deleting plugin directory...').start()
+        try {
+          await pluginService.deletePlugin({ pluginName }, context)
+          deleteSpinner.succeed('Plugin directory deleted')
+        } catch (error) {
+          deleteSpinner.fail('Failed to delete plugin directory')
+          throw error
+        }
+
+        // Update dependencies
+        const installSpinner = ora('Updating workspace dependencies...').start()
+        try {
+          // This is already done by plugin service, just show spinner for consistency
+          installSpinner.succeed('Dependencies updated (package-lock.json cleaned)')
+        } catch (error) {
+          installSpinner.fail('Failed to update dependencies')
+          console.log(chalk.yellow('\n⚠️  You may need to run "npm install" manually at the workspace root\n'))
+        }
+
+        // Success message
+        console.log(chalk.green(`\n✅ Plugin '${pluginName}' has been deleted\n`))
+
+        // Reminder about cleanup
+        console.log(chalk.yellow('📝 Remember to:'))
+        console.log(chalk.gray('   • Check packages that had this plugin installed'))
+        console.log(chalk.gray('   • Remove plugin from launch77.installedPlugins in package.json files'))
+        console.log(chalk.gray('   • Remove any plugin-generated files if needed'))
+        console.log(chalk.gray('   • Consider deprecating/unpublishing from npm if published'))
+        console.log()
+
+        // Reminder about Git
+        console.log(chalk.yellow('📝 Remember to commit the deletion:'))
+        console.log(chalk.gray('   git add -A'))
+        console.log(chalk.gray(`   git commit -m "Remove ${pluginName} plugin"`))
+        console.log()
+      } catch (error) {
+        const message = error instanceof Error ? error.message : String(error)
+        console.error(chalk.red('Error:'), message)
+        process.exit(1)
+      }
+    })
+
+  return command
+}

package/src/modules/plugin/errors/plugin-errors.ts

@@ -123,3 +123,23 @@ ${cause ? `\nOriginal error: ${cause.message}` : ''}`)
     this.name = 'NpmInstallationError'
   }
 }
+
+/**
+ * Error when plugin directory is not found for deletion
+ */
+export class PluginDirectoryNotFoundError extends Error {
+  constructor(pluginName: string, expectedPath: string) {
+    super(`Plugin '${pluginName}' does not exist.\n\n` + `Expected location: ${expectedPath}\n\n` + `Available plugins:\n` + `  cd plugins/\n` + `  ls`)
+    this.name = 'PluginDirectoryNotFoundError'
+  }
+}
+
+/**
+ * Error when plugin name validation fails
+ */
+export class InvalidPluginNameError extends Error {
+  constructor(message: string) {
+    super(message)
+    this.name = 'InvalidPluginNameError'
+  }
+}

package/src/modules/plugin/index.ts

@@ -11,6 +11,7 @@ export { PluginNotFoundError, InvalidPluginContextError, PluginInstallationError
 // Commands
 export { pluginInstallCommand } from './commands/plugin-install.js'
 export { pluginCreateCommand } from './commands/plugin-create.js'
+export { pluginDeleteCommand } from './commands/delete-plugin.js'
 
 // Utilities
 export { listAvailablePlugins } from './lib/plugin-registry.js'

package/src/modules/plugin/lib/plugin-resolver.test.ts

@@ -120,6 +120,7 @@ describe('Plugin Resolver', () => {
       await fs.ensureDir(pluginPath)
       await fs.ensureDir(path.join(pluginPath, 'dist'))
       await fs.writeFile(path.join(pluginPath, 'plugin.json'), JSON.stringify({ name: 'my-plugin', version: '1.0.0' }))
+      await fs.writeFile(path.join(pluginPath, 'package.json'), JSON.stringify({ name: 'my-plugin', version: '0.0.1' }))
       await fs.writeFile(path.join(pluginPath, 'dist/generator.js'), 'console.log("test")')
 
       const result = await resolver.resolveLocation('my-plugin', tempDir)
@@ -128,6 +129,7 @@ describe('Plugin Resolver', () => {
         source: 'local',
         resolvedName: 'my-plugin',
         localPath: pluginPath,
+        version: '0.0.1',
       })
     })
 

package/src/modules/plugin/lib/plugin-resolver.ts

@@ -21,10 +21,21 @@ export class PluginResolver extends PackageResolver {
   }
 
   protected async verify(localPath: string): Promise<boolean> {
-    // Verify it's a valid plugin (has plugin.json and dist/generator.js)
+    // Verify it's a valid plugin (has plugin.json, dist/generator.js, and package.json with version)
     const hasPluginJson = await fs.pathExists(path.join(localPath, 'plugin.json'))
     const hasGenerator = await fs.pathExists(path.join(localPath, 'dist/generator.js'))
+    const hasPackageJson = await fs.pathExists(path.join(localPath, 'package.json'))
 
-    return hasPluginJson && hasGenerator
+    if (!hasPluginJson || !hasGenerator || !hasPackageJson) {
+      return false
+    }
+
+    // Verify package.json has a version field
+    try {
+      const packageJson = await fs.readJson(path.join(localPath, 'package.json'))
+      return !!packageJson.version
+    } catch {
+      return false
+    }
   }
 }