generator-jhipster-sentry-module 1.0.7 → 1.0.9

package/generators/app/index.js

@@ -5,7 +5,6 @@ const semver = require('semver');
 const Generator = require('yeoman-generator');
 const packageJson = require('../../package.json');
 
-// JHipster constants
 const jhipsterConstants = {
   SERVER_MAIN_SRC_DIR: 'src/main/java/',
   SERVER_MAIN_RES_DIR: 'src/main/resources/',
@@ -13,25 +12,34 @@ const jhipsterConstants = {
 };
 
 module.exports = class extends Generator {
+
   async initializing() {
-    // Read JHipster config
-    this.jhipsterAppConfig = this.fs.readJSON(this.destinationPath('.yo-rc.json'), {})['generator-jhipster'];
+    const yoConfig = this.fs.readJSON(this.destinationPath('.yo-rc.json'), {});
+    this.jhipsterAppConfig = yoConfig['generator-jhipster'];
+
     if (!this.jhipsterAppConfig) {
       this.log(chalk.red('Cannot read .yo-rc.json or JHipster config missing'));
-      process.exit(1);
+      return;
     }
 
-    this.log(`\nWelcome to the ${chalk.bold.yellow('JHipster Sentry Module')} generator! ${chalk.yellow(`v${packageJson.version}\n`)}`);
-
-    // Check JHipster version
-    const currentJhipsterVersion = this.jhipsterAppConfig.jhipsterVersion;
-    if (!semver.satisfies(currentJhipsterVersion, '>=8.0.0')) {
-      this.log(chalk.yellow(`Warning: Your JHipster version is ${currentJhipsterVersion}, minimum required is 8.0.0\n`));
+    this.log(
+      `\nWelcome to the ${chalk.bold.yellow('JHipster Sentry Module')} generator! ${chalk.yellow(
+        `v${packageJson.version}\n`
+      )}`
+    );
+
+    const currentVersion = this.jhipsterAppConfig.jhipsterVersion || '0.0.0';
+    if (!semver.satisfies(currentVersion, '>=8.0.0')) {
+      this.log(
+        chalk.yellow(
+          `Warning: Your JHipster version is ${currentVersion}, minimum required is 8.0.0\n`
+        )
+      );
     }
   }
 
   async prompting() {
-    const prompts = [
+    this.promptAnswers = await this.prompt([
       {
         type: 'confirm',
         name: 'enableSentry',
@@ -44,102 +52,86 @@ module.exports = class extends Generator {
         name: 'sentryDsn',
         message: 'Enter your Sentry DSN'
       }
-    ];
-    this.promptAnswers = await this.prompt(prompts);
+    ]);
   }
 
   writing() {
-    // Read JHipster config
-    this.baseName = this.jhipsterAppConfig.baseName;
-    this.packageName = this.jhipsterAppConfig.packageName;
-    this.packageFolder = this.jhipsterAppConfig.packageFolder;
-    this.clientFramework = this.jhipsterAppConfig.clientFramework;
-    this.clientPackageManager = this.jhipsterAppConfig.clientPackageManager;
-    this.buildTool = this.jhipsterAppConfig.buildTool;
-
-    const javaDir = `${jhipsterConstants.SERVER_MAIN_SRC_DIR}${this.packageFolder}/`;
-    const resourceDir = jhipsterConstants.SERVER_MAIN_RES_DIR;
+    if (!this.jhipsterAppConfig) return;
+
+    const {
+      baseName,
+      packageName,
+      packageFolder,
+      clientFramework,
+      buildTool
+    } = this.jhipsterAppConfig;
+
     const webappDir = jhipsterConstants.CLIENT_MAIN_SRC_DIR;
+    const resourceDir = jhipsterConstants.SERVER_MAIN_RES_DIR;
 
-    // Persist Sentry config
+    // Save config
     this.jhipsterAppConfig.sentry = {
       enabled: this.promptAnswers.enableSentry,
       dsn: this.promptAnswers.sentryDsn
     };
+
     const configPath = this.destinationPath('.yo-rc.json');
     const config = this.fs.readJSON(configPath, {});
-    config['generator-jhipster'] = config['generator-jhipster'] || {};
     config['generator-jhipster'].sentry = this.jhipsterAppConfig.sentry;
     this.fs.writeJSON(configPath, config, null, 2);
 
-    // Logging
-    this.log('\n--- JHipster Config ---');
-    this.log(`baseName=${this.baseName}`);
-    this.log(`packageName=${this.packageName}`);
-    this.log(`clientFramework=${this.clientFramework}`);
-    this.log(`buildTool=${this.buildTool}`);
-    this.log('\n--- Sentry ---');
-    this.log(`enabled=${this.promptAnswers.enableSentry}`);
-    this.log('----------------------\n');
-
     if (!this.promptAnswers.enableSentry) {
-      this.log(chalk.yellow('Sentry is disabled — skipping integration'));
+      this.log(chalk.yellow('Sentry disabled — skipping integration'));
       return;
     }
 
-    // Backend dependencies
-    if (this.buildTool === 'maven') {
+    // ======================
+    // Backend (Maven only safe injection)
+    // ======================
+    if (buildTool === 'maven') {
       const pomPath = this.destinationPath('pom.xml');
       if (fs.existsSync(pomPath)) {
         let pom = fs.readFileSync(pomPath, 'utf8');
         if (!pom.includes('sentry-spring-boot-starter')) {
           pom = pom.replace(
             /<\/dependencies>/,
-            `    <dependency>
-        <groupId>io.sentry</groupId>
-        <artifactId>sentry-spring-boot-starter</artifactId>
-        <version>7.3.0</version>
-    </dependency>
-</dependencies>`
+            `
+        <dependency>
+            <groupId>io.sentry</groupId>
+            <artifactId>sentry-spring-boot-starter</artifactId>
+            <version>7.3.0</version>
+        </dependency>
+    </dependencies>`
           );
           fs.writeFileSync(pomPath, pom);
         }
       }
     }
 
-    if (this.buildTool === 'gradle') {
-      const buildGradle = this.destinationPath('build.gradle');
-      if (fs.existsSync(buildGradle)) {
-        let gradle = fs.readFileSync(buildGradle, 'utf8');
-        if (!gradle.includes('sentry-spring-boot-starter')) {
-          gradle += '\ndependencies {\n    implementation "io.sentry:sentry-spring-boot-starter:7.3.0"\n}\n';
-          fs.writeFileSync(buildGradle, gradle);
-        }
-      }
-    }
-
-    // Frontend templates
-    if (this.clientFramework === 'react') {
-      this.fs.copyTpl(this.templatePath('client/react/sentry.ts.ejs'), `${webappDir}app/sentry.ts`);
-    } else if (this.clientFramework === 'angularX') {
-      this.fs.copyTpl(this.templatePath('client/angular/sentry.ts.ejs'), `${webappDir}app/sentry.ts`);
+    // ======================
+    // Frontend
+    // ======================
+    if (clientFramework === 'react') {
+      this.fs.copyTpl(
+        this.templatePath('client/react/sentry.ts.ejs'),
+        this.destinationPath(`${webappDir}app/sentry.ts`),
+        { sentryDsn: this.promptAnswers.sentryDsn }
+      );
+
+      this.addPackageJsonDependency('@sentry/react', '^7.3.0');
     }
-  }
 
-  install() {
-    if (!this.promptAnswers.enableSentry) return;
+    if (clientFramework === 'angularX') {
+      this.fs.copyTpl(
+        this.templatePath('client/angular/sentry.ts.ejs'),
+        this.destinationPath(`${webappDir}app/sentry.ts`),
+        { sentryDsn: this.promptAnswers.sentryDsn }
+      );
 
-    const packages = [];
-    if (this.clientFramework === 'react') {
-      packages.push('@sentry/react@^7.3.0');
-    } else if (this.clientFramework === 'angularX') {
-      packages.push('@sentry/angular@^7.3.0');
+      this.addPackageJsonDependency('@sentry/angular', '^7.3.0');
     }
 
-    if (packages.length > 0) {
-      const cmd = this.clientPackageManager === 'yarn' ? 'add' : 'install';
-      this.spawnCommandSync(this.clientPackageManager, [...(this.clientPackageManager === 'yarn' ? ['add'] : ['install']), ...packages]);
-    }
+    this.log(chalk.green('✔ Sentry files added successfully'));
   }
 
   end() {

package/generators/app/templates/client/react/sentry.ts.ejs

@@ -1,6 +1,6 @@
 import * as Sentry from '@sentry/react';
 
 Sentry.init({
-  dsn: '<%= jhipsterAppConfig.sentry.dsn %>',
+  dsn: '<%= sentryDsn %>',
   environment: 'dev',
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "generator-jhipster-sentry-module",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "Adding Sentry integration to JHipster projects",
   "keywords": [
     "yeoman-generator",