yeoman-environment 4.0.0-beta.0 → 4.0.0-beta.1

package/dist/commit.js

@@ -1,15 +1,18 @@
 import { createConflicterTransform, createYoResolveTransform, forceYoFiles } from '@yeoman/conflicter';
+import createdLogger from 'debug';
 import { create as createMemFsEditor } from 'mem-fs-editor';
 // eslint-disable-next-line n/file-extension-in-import
 import { isFilePending } from 'mem-fs-editor/state';
 // eslint-disable-next-line n/file-extension-in-import
 import { createCommitTransform } from 'mem-fs-editor/transform';
+const debug = createdLogger('yeoman:environment:commit');
 /**
  * Commits the MemFs to the disc.
  * @param {Stream} [stream] - files stream, defaults to this.sharedFs.stream().
  * @return {Promise}
  */
 export const commitSharedFsTask = ({ adapter, conflicterOptions, sharedFs, stream, }) => {
+    debug('Running commitSharedFsTask');
     const fs = createMemFsEditor(sharedFs);
     stream = stream ?? fs.store.stream({ filter: file => isFilePending(file) });
     return fs.commit([

package/dist/composed-store.d.ts

@@ -13,9 +13,9 @@ export declare class ComposedStore {
     getGenerators(): Record<string, BaseGenerator>;
     addGenerator(generator: any): {
         uniqueBy: any;
+        identifier: any;
         added: boolean;
         generator: BaseGenerator | undefined;
-        identifier?: undefined;
     } | {
         identifier: any;
         added: boolean;

package/dist/composed-store.js

@@ -1,5 +1,7 @@
 import crypto from 'node:crypto';
 import { toNamespace } from '@yeoman/namespace';
+import createdLogger from 'debug';
+const debug = createdLogger('yeoman:environment:composed-store');
 const uniqueFeatureValues = ['customCommitTask', 'customInstallTask'];
 export class ComposedStore {
     log;
@@ -41,7 +43,7 @@ export class ComposedStore {
         const generatorRoot = generator.destinationRoot();
         const uniqueByMap = uniqueGlobally ? this.uniqueGloballyMap : this.getUniqueByPathMap(generatorRoot);
         if (uniqueByMap.has(uniqueBy)) {
-            return { uniqueBy, added: false, generator: uniqueByMap.get(uniqueBy) };
+            return { uniqueBy, identifier, added: false, generator: uniqueByMap.get(uniqueBy) };
         }
         uniqueByMap.set(uniqueBy, generator);
         for (const featureName of uniqueFeatureValues) {
@@ -49,6 +51,7 @@ export class ComposedStore {
             if (feature) {
                 const existingFeature = this.uniqueFeatures.get(feature);
                 if (typeof existingFeature !== 'function') {
+                    debug(`Feature ${featureName} provided by ${uniqueBy}`);
                     this.uniqueFeatures.set(featureName, feature);
                 }
                 else if (typeof feature === 'function') {

package/dist/environment-base.js

@@ -526,12 +526,14 @@ export default class EnvironmentBase extends EventEmitter {
              * would be killed if an error is thrown to environment.
              * Make sure to not rely on that behavior.
              */
-            this.on('error', error => {
-                reject(error);
+            this.on('error', async (error) => {
                 this.runLoop.pause();
+                await this.adapter.onIdle?.();
+                reject(error);
                 this.adapter.close();
             });
-            this.once('end', () => {
+            this.once('end', async () => {
+                await this.adapter.onIdle?.();
                 resolve();
                 this.adapter.close();
             });
@@ -568,13 +570,15 @@ export default class EnvironmentBase extends EventEmitter {
         const queueCommit = () => {
             debug('Queueing conflicts task');
             this.queueTask('environment:conflicts', async () => {
+                debug('Running conflicts');
                 const { customCommitTask = () => commitSharedFsTask(this) } = this.composedStore;
-                if (typeof customCommitTask !== 'function') {
-                    // There is a custom commit task or just disabled
-                    return;
+                if (typeof customCommitTask === 'function') {
+                    await customCommitTask();
                 }
-                await customCommitTask();
-                debug('Adding queueCommit event listener');
+                else {
+                    debug('Ignoring commit, custom commit was provided');
+                }
+                debug('Adding queueCommit listener');
                 this.sharedFs.once('change', queueCommit);
             }, {
                 once: 'write memory fs to disk',
@@ -602,6 +606,10 @@ export default class EnvironmentBase extends EventEmitter {
                 nodePackageManager,
                 customInstallTask,
             });
+            memFs.once('change', file => {
+                if (file === join(this.cwd, 'package.json'))
+                    this.queuePackageManagerInstall();
+            });
         }, { once: 'package manager install' });
     }
     /**

package/dist/package-manager.js

@@ -15,6 +15,7 @@ const debug = createdLogger('yeoman:environment:package-manager');
 
 */
 export async function packageManagerInstallTask({ memFs, packageJsonLocation, customInstallTask, adapter, nodePackageManager, skipInstall, }) {
+    debug('Running packageManagerInstallTask');
     packageJsonLocation = resolve(packageJsonLocation);
     /**
      * @private

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yeoman-environment",
-  "version": "4.0.0-beta.0",
+  "version": "4.0.0-beta.1",
   "description": "Handles the lifecyle and bootstrapping of generators in a specific environment",
   "keywords": [
     "development",