update generator-jhipster to 8.7.3 (#47)

.blueprint/generate-sample/command.mjs

@@ -17,7 +17,7 @@
  * limitations under the License.
  */
 import { GENERATOR_APP } from 'generator-jhipster/generators';
-import { getSamples } from './get-samples.mjs';
+import { getGithubSamplesGroup } from 'generator-jhipster/testing';
 
 /**
  * @type {import('generator-jhipster').JHipsterCommandDefinition}
@@ -29,12 +29,20 @@ const command = {
     },
   },
   configs: {
+    samplesFolder: {
+      description: 'Path to the samples folder',
+      cli: {
+        type: String,
+      },
+      default: 'samples',
+      scope: 'generator',
+    },
     sampleName: {
       prompt: gen => ({
         when: !gen.all,
         type: 'list',
         message: 'which sample do you want to generate?',
-        choices: async () => getSamples(gen.templatePath(gen.samplesFolder)),
+        choices: async () => getGithubSamplesGroup(gen.templatePath(gen.samplesFolder)),
       }),
       scope: 'generator',
     },
@@ -45,14 +53,6 @@ const command = {
       },
       scope: 'generator',
     },
-    samplesFolder: {
-      description: 'Path to the samples folder',
-      cli: {
-        type: String,
-      },
-      default: 'samples',
-      scope: 'generator',
-    },
   },
   options: {},
   import: [GENERATOR_APP],

.blueprint/generate-sample/generator.mjs

@@ -1,6 +1,8 @@
 import { readdir } from 'node:fs/promises';
 import { readFileSync } from 'node:fs';
+import { extname, join } from 'node:path';
 import BaseGenerator from 'generator-jhipster/generators/base';
+import { getGithubSamplesGroup } from 'generator-jhipster/testing';
 
 export default class extends BaseGenerator {
   sampleName;
@@ -14,11 +16,22 @@ export default class extends BaseGenerator {
   get [BaseGenerator.WRITING]() {
     return this.asWritingTaskGroup({
       async copySample() {
-        const samplesFolder = `${this.samplesFolder ?? 'samples'}/`;
-        if (this.all) {
-          this.copyTemplate(`${samplesFolder}*.jdl`, '');
+        const { samplesFolder, all, sampleName } = this;
+        if (all) {
+          this.copyTemplate(`${samplesFolder}/*.jdl`, '');
+        } else if (extname(sampleName) === '.jdl') {
+          this.copyTemplate(join(samplesFolder, sampleName), sampleName, { noGlob: true });
         } else {
-          this.copyTemplate(`${samplesFolder}${this.sampleName}`, this.sampleName, { noGlob: true });
+          const { samples } = await getGithubSamplesGroup(this.templatePath(), samplesFolder);
+          const { 'sample-type': sampleType } = samples[sampleName];
+          if (sampleType === 'jdl') {
+            const jdlFile = `${sampleName}.jdl`;
+            this.copyTemplate(join(samplesFolder, jdlFile), jdlFile, { noGlob: true });
+          } else if (sampleType === 'yo-rc') {
+            this.copyTemplate(join(samplesFolder, sampleName, '**'), '', {
+              fromBasePath: this.templatesPath(samplesFolder, sampleName),
+            });
+          }
         }
       },
     });

.blueprint/github-build-matrix/generator.mjs

@@ -1,10 +1,8 @@
-import { appendFileSync, existsSync } from 'node:fs';
-import os from 'node:os';
 import BaseGenerator from 'generator-jhipster/generators/base';
-import { getSamples } from '../generate-sample/get-samples.mjs';
-import { buildMatrix } from './build-matrix.mjs';
+import { convertToGitHubMatrix, getGithubOutputFile, getGithubSamplesGroup, setGithubTaskOutput } from 'generator-jhipster/testing';
 
 export default class extends BaseGenerator {
+  /** @type {string} */
   samplesFolder;
 
   constructor(args, opts, features) {
@@ -14,14 +12,16 @@ export default class extends BaseGenerator {
   get [BaseGenerator.WRITING]() {
     return this.asWritingTaskGroup({
       async buildMatrix() {
-        const samplesFolder = this.samplesFolder ?? 'samples';
-        const samples = await getSamples(this.templatePath(`../../generate-sample/templates/${samplesFolder}`));
-        const matrix = buildMatrix({ samples, samplesFolder });
-        const matrixoutput = `matrix<<EOF${os.EOL}${JSON.stringify(matrix)}${os.EOL}EOF${os.EOL}`;
-        const filePath = process.env.GITHUB_OUTPUT;
-        console.log(matrixoutput);
-        if (filePath && existsSync(filePath)) {
-          appendFileSync(filePath, matrixoutput, { encoding: 'utf8' });
+        const { samplesFolder } = this;
+        const { samples, warnings } = await getGithubSamplesGroup(this.templatePath('../../generate-sample/templates/'), samplesFolder);
+        if (warnings.length > 0) {
+          this.info(warnings.join('\n'));
+        }
+        const matrix = JSON.stringify(convertToGitHubMatrix(samples));
+        const githubOutputFile = getGithubOutputFile(matrix);
+        this.log.info('matrix', matrix);
+        if (githubOutputFile) {
+          setGithubTaskOutput('matrix', matrix);
         }
       },
     });

.github/workflows/generator.yml

@@ -11,7 +11,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
           cache: 'npm'
       - run: npm ci
       - run: npm run test

.github/workflows/samples.yml

@@ -20,7 +20,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 20
+          node-version: 22
       - id: build
         run: |
           npm install
@@ -50,22 +50,22 @@ jobs:
           binary-dir: ${{ github.workspace }}/generator-jhipster-jooq/cli/
       - run: npm install
         working-directory: ${{ github.workspace }}/generator-jhipster-jooq
-      - run: cli.cjs generate-sample ${{ matrix.sample-name }} --skip-jhipster-dependencies ${{ matrix.extra-args }}
+      - run: cli.cjs generate-sample ${{ matrix.sample }}.jdl --skip-jhipster-dependencies --skip-install ${{ matrix.extra-args }}
       - uses: jhipster/actions/compare-sample@v0
         id: compare
         if: >-
           github.event.pull_request &&
           !contains(github.event.pull_request.labels.*.name, 'pr: disable-compare')
         with:
           generator-path: generator-jhipster-jooq
-          cmd: cli.cjs generate-sample ${{ matrix.sample-name }} --skip-jhipster-dependencies --skip-install ${{ matrix.extra-args }}
+          cmd: cli.cjs generate-sample ${{ matrix.sample }}.jdl --skip-jhipster-dependencies --skip-install ${{ matrix.extra-args }}
       - run: npm run ci:backend:test
         if: steps.compare.outputs.equals != 'true'
         id: backend
-      - run: npm run ci:frontend:test --if-present
-        if: steps.compare.outputs.equals != 'true'
       - run: npm run ci:e2e:package
         if: steps.compare.outputs.equals != 'true'
+      - run: npm run ci:frontend:test --if-present
+        if: steps.compare.outputs.equals != 'true'
       - run: npm run ci:e2e:prepare
         if: steps.compare.outputs.equals != 'true'
       - run: npm run ci:e2e:run --if-present

.yo-rc.json

@@ -20,7 +20,7 @@
       }
     },
     "githubWorkflows": true,
-    "jhipsterVersion": "8.7.1",
+    "jhipsterVersion": "8.7.3",
     "js": false,
     "localBlueprint": false,
     "packageJsonType": "module",

eslint.config.js

@@ -1,9 +1,6 @@
 import globals from 'globals';
 import prettier from 'eslint-plugin-prettier/recommended';
-import jhipster from 'generator-jhipster/eslint/recommended';
-
-// Workaround for jhipster export issue.
-delete jhipster.recommended.recommended;
+import jhipster from 'generator-jhipster/eslint';
 // jhipster-needle-eslint-add-import - JHipster will add additional import here
 
 export default [