syncpack 12.3.1 → 12.3.2

package/dist/bin-update/effects.js

@@ -160,7 +160,7 @@ export const updateEffects = {
             ...doState.premajorAnswers,
             ...doState.majorAnswers,
             ...doState.prereleaseAnswers,
-        ], Effect.forEach((release) => pipe(release.instance.versionGroup.instances, Effect.forEach((instance) => pipe(instance.semverGroup.getFixed(Specifier.create(instance, release.versions.latest)), Effect.flatMap((latestWithRange) => instance.write(latestWithRange.raw)), Effect.catchTag('NonSemverError', Effect.logError))))), Effect.flatMap(() => Effect.void))));
+        ], Effect.forEach(({ instance, versions }) => pipe(instance.semverGroup.getFixed(Specifier.create(instance, versions.latest)), Effect.flatMap((latestWithRange) => instance.write(latestWithRange.raw)), Effect.catchTag('NonSemverError', Effect.logError))), Effect.flatMap(() => Effect.void))));
     },
 };
 function promptForReleaseType(releaseType, doState) {

package/dist/bin-update/update.js

@@ -13,12 +13,13 @@ import { withLogger } from '../lib/with-logger.js';
 import { Specifier } from '../specifier/index.js';
 import { updateEffects } from './effects.js';
 export function update(io, cli, effects = updateEffects, errorHandlers = defaultErrorHandlers) {
-    return pipe(Effect.Do, Effect.bind('ctx', () => getContext({ io, cli, errorHandlers })), Effect.bind('instances', ({ ctx }) => getInstances(ctx, io, errorHandlers)), Effect.bind('updateable', ({ instances }) => {
+    return pipe(Effect.Do, Effect.bind('ctx', () => getContext({ io, cli, errorHandlers })), Effect.bind('instances', ({ ctx }) => getInstances(ctx, io, errorHandlers)), Effect.bind('update', ({ instances }) => pipe(Effect.succeed(instances.all), Effect.map((instances) => {
         const isVisitedByName = {};
         const updateable = [];
-        instances.all.forEach((instance) => {
-            const _tag = instance.versionGroup._tag;
-            if (!isVisitedByName[instance.name] && (_tag === 'SameRange' || _tag === 'Standard')) {
+        instances.forEach((instance) => {
+            if (!isVisitedByName[instance.name] &&
+                (instance.versionGroup._tag === 'SameRange' ||
+                    instance.versionGroup._tag === 'Standard')) {
                 const specifier = Specifier.create(instance, instance.rawSpecifier.raw);
                 if (specifier._tag === 'Range' || specifier._tag === 'Exact') {
                     isVisitedByName[instance.name] = true;
@@ -26,15 +27,15 @@ export function update(io, cli, effects = updateEffects, errorHandlers = default
                 }
             }
         });
-        return Effect.succeed(updateable);
-    }), Effect.bind('update', ({ updateable }) => pipe(Effect.succeed(updateable), Effect.tap(effects.onFetchAllStart), Effect.flatMap((instances) => pipe(instances, Effect.partition((instance) => pipe(Effect.succeed(instance), Effect.tap(() => effects.onFetchStart(instance, instances.length)), Effect.flatMap(effects.fetchLatestVersions), Effect.tapBoth({
-        onFailure: () => effects.onFetchEnd(instance),
-        onSuccess: ({ versions }) => effects.onFetchEnd(instance, versions),
+        return updateable;
+    }), Effect.tap(updateEffects.onFetchAllStart), Effect.flatMap((instances) => pipe(instances, Effect.partition((instance) => pipe(Effect.succeed(instance), Effect.tap(() => updateEffects.onFetchStart(instance, instances.length)), Effect.flatMap(effects.fetchLatestVersions), Effect.tapBoth({
+        onFailure: () => updateEffects.onFetchEnd(instance),
+        onSuccess: ({ versions }) => updateEffects.onFetchEnd(instance, versions),
     }), 
     // move up to date dependencies to error channel
     Effect.flatMap((updateable) => gtr(updateable.versions.latest, String(instance.rawSpecifier.raw))
-        ? pipe(effects.onOutdated(instance, updateable.versions.latest), Effect.map(() => updateable))
-        : pipe(effects.onUpToDate(instance), Effect.flatMap(() => Effect.fail(updateable)))), 
+        ? pipe(updateEffects.onOutdated(instance, updateable.versions.latest), Effect.map(() => updateable))
+        : pipe(updateEffects.onUpToDate(instance), Effect.flatMap(() => Effect.fail(updateable)))), 
     // log error but don't catch it
     Effect.tapErrorTag('HttpError', ({ error }) => Effect.logError(chalk `{red ${ICON.cross} ${error}}`)), 
     // log error but don't catch it
@@ -43,11 +44,11 @@ export function update(io, cli, effects = updateEffects, errorHandlers = default
     Effect.flatMap(([_, outOfDate]) => Effect.succeed(outOfDate)))), 
     // always remove the spinner when we're done
     Effect.tapBoth({
-        onFailure: effects.onFetchAllEnd,
-        onSuccess: effects.onFetchAllEnd,
+        onFailure: updateEffects.onFetchAllEnd,
+        onSuccess: updateEffects.onFetchAllEnd,
     }), 
     // ask the user which updates they want
-    Effect.flatMap(effects.promptForUpdates), 
+    Effect.flatMap(updateEffects.promptForUpdates), 
     // if we think the user cancelled, say so
     Effect.catchTag('PromptCancelled', () => Effect.logInfo(chalk `{red ${ICON.panic}} aborting after {blue syncpack update} was cancelled`)))), Effect.flatMap(({ ctx }) => pipe(writeIfChanged(ctx), Effect.catchTags({
         WriteFileError: flow(errorHandlers.WriteFileError, Effect.map(() => {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "syncpack",
   "description": "Consistent dependency versions in large JavaScript Monorepos",
-  "version": "12.3.1",
+  "version": "12.3.2",
   "author": "Jamie Mason <jamie@foldleft.io> (https://github.com/JamieMason)",
   "bin": {
     "syncpack": "dist/bin.js",