faces-cli 1.8.3 → 1.8.4

package/dist/commands/style/revert.js

@@ -3,9 +3,10 @@ import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 import { VERSIONS_PATH } from '../../style.js';
 export default class StyleRevert extends BaseCommand {
-    static description = 'Go back to the style a face had before the last style:make. A face holds one style per medium, so ' +
-        'name which one with --medium when it has more than one. Two versions are kept per medium, so this ' +
-        'goes back exactly one step. It changes how the face writes immediately, so it asks first.';
+    static description = 'Go back to the style a face had before the last style:make. Two versions are kept per medium, so ' +
+        'this is a toggle: running it again returns to where you started. A face holds one style per medium, ' +
+        'so name which one with --medium when it has more than one. It changes how the face writes ' +
+        'immediately, so it asks first.';
     static examples = [
         '<%= config.bin %> <%= command.id %> alice --yes',
         '<%= config.bin %> <%= command.id %> alice --medium email --yes',
@@ -50,13 +51,6 @@ export default class StyleRevert extends BaseCommand {
                 if (err.statusCode === 409) {
                     this.error(`Error (409): ${err.message}\nSee what is there: faces style:versions ${args.alias}`);
                 }
-                // Reverting a named medium currently fails this way even when the
-                // server itself reports can_revert: true (faces-backend-shared#590).
-                if (err.statusCode === 500) {
-                    this.error(`Error (500): the server failed to revert '${args.alias}'.\n` +
-                        'Reverting a per-medium style is currently broken upstream and nothing was changed. ' +
-                        'Tracked as faces-backend-shared#590.');
-                }
                 this.error(`Error (${err.statusCode}): ${err.message}`);
             }
             throw err;
@@ -71,8 +65,11 @@ export default class StyleRevert extends BaseCommand {
         else {
             this.log(`'${args.alias}' reverted.`);
         }
-        this.log(`To undo this, capture the style again: faces style:make ${args.alias} --all`);
-        this.log(`List versions: faces style:versions ${args.alias}`);
+        // Reverting again returns to where you started — it is a toggle between the
+        // two kept versions, not a walk backwards. Name the version it goes back to
+        // so the user does not have to hold it in their head.
+        const back = from === undefined ? '' : ` to return to version ${from}`;
+        this.log(`Run it again${back}. List versions: faces style:versions ${args.alias}`);
         return data;
     }
 }