faces-cli 1.8.0 → 1.8.1

package/dist/commands/style/delete.d.ts

@@ -3,7 +3,6 @@ export default class StyleDelete extends BaseCommand {
     static description: string;
     static examples: string[];
     static flags: {
-        scope: import("@oclif/core/interfaces").OptionFlag<string, import("@oclif/core/interfaces").CustomOptions>;
         yes: import("@oclif/core/interfaces").BooleanFlag<boolean>;
         'base-url': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
         token: import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;

package/dist/commands/style/delete.js

@@ -3,26 +3,28 @@ import { createInterface } from 'node:readline';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 import { STYLE_FACES_PATH } from '../../style.js';
-/** What each scope destroys, in the words the user needs before choosing. */
-const SCOPES = {
-    map: 'the captured style and everything derived from it. Your uploaded material is kept, so it can be captured again without re-uploading.',
-    all: 'the captured style AND the material it was built from. The uploaded corpus is deleted. This cannot be undone and nothing can rebuild it.',
-};
+/**
+ * The API takes a scope: `map` clears the captured style, `all` clears it and
+ * destroys the uploaded material with it. This CLI only ever sends `map`.
+ *
+ * Nothing that removes a style should be able to take source text with it. A
+ * user reaching for "delete the style" is not asking to lose the writing it was
+ * learned from, and the two are one keystroke apart on the same command. Source
+ * text is deleted through the commands that own it — compile:doc:delete and
+ * compile:thread:delete — where that is the whole point of the call rather than
+ * a side effect of a flag value.
+ */
+const SCOPE = 'map';
 export default class StyleDelete extends BaseCommand {
-    static description = 'Delete a face\'s captured style. --scope map forgets the style and keeps the material it came from. ' +
-        '--scope all deletes the uploaded material too, permanently. There is no default because the two ' +
-        'differ by whether your material survives.';
+    static description = "Delete a face's captured style. The material it was learned from is kept, so the style can be " +
+        'captured again without re-uploading. This never deletes documents or threads: remove those with ' +
+        'compile:doc:delete or compile:thread:delete.';
     static examples = [
-        '<%= config.bin %> <%= command.id %> alice --scope map',
-        '<%= config.bin %> <%= command.id %> alice --scope all --yes',
+        '<%= config.bin %> <%= command.id %> alice',
+        '<%= config.bin %> <%= command.id %> alice --yes',
     ];
     static flags = {
         ...BaseCommand.baseFlags,
-        scope: Flags.string({
-            description: 'map: forget the style, keep the material. all: delete the material with it, permanently.',
-            options: ['map', 'all'],
-            required: true,
-        }),
         yes: Flags.boolean({ description: 'Skip confirmation', default: false }),
     };
     static args = {
@@ -31,32 +33,32 @@ export default class StyleDelete extends BaseCommand {
     async run() {
         const { args, flags } = await this.parse(StyleDelete);
         const client = this.makeClient(flags);
-        const scope = flags.scope;
         if (!flags.yes) {
-            const what = `This deletes ${SCOPES[scope]}`;
+            const what = `'${args.alias}' will forget its captured style. Its documents, threads and uploaded material are kept.`;
             if (this.jsonEnabled())
                 this.error(`${what}\nRe-run with --yes to confirm.`);
-            const ok = await this.confirm(`${what}\nDelete for '${args.alias}'?`);
+            const ok = await this.confirm(`${what}\nDelete the style?`);
             if (!ok)
                 this.error('Aborted. Nothing was deleted.');
         }
         let data;
         try {
-            data = await client.delete(`${STYLE_FACES_PATH}/${encodeURIComponent(args.alias)}?scope=${scope}`);
+            data = await client.delete(`${STYLE_FACES_PATH}/${encodeURIComponent(args.alias)}?scope=${SCOPE}`);
         }
         catch (err) {
             if (err instanceof FacesAPIError) {
                 if (err.statusCode === 404)
                     this.error(`No face named '${args.alias}'. It does not exist, or is not yours.`);
+                if (err.statusCode === 409)
+                    this.error(`Error (409): ${err.message}\nThe face may be locked. Unlock it first.`);
                 this.error(`Error (${err.statusCode}): ${err.message}`);
             }
             throw err;
         }
         if (this.jsonEnabled())
             return data;
-        this.log(scope === 'all'
-            ? `Deleted the style and the uploaded material for '${args.alias}'.`
-            : `Deleted the style for '${args.alias}'. The material it came from is still there, so you can capture it again: faces style:make ${args.alias} --all`);
+        this.log(`'${args.alias}' has forgotten its style. Its source material is untouched.`);
+        this.log(`Capture it again with: faces style:make ${args.alias} --all`);
         return data;
     }
     confirm(message) {