faces-cli 1.8.6 → 1.8.7

package/dist/catalog.js

@@ -135,7 +135,7 @@ export class CatalogService {
                 // what kept it alive across writes; dropping it lets the next sync
                 // replace it with the real one.
                 if (existingDescription.length > MAX_FRONTMATTER_VALUE) {
-                    process.stderr.write(`warn: discarded a ${existingDescription.length}-character description on '${alias}' — ` +
+                    process.stderr.write(`warn: discarded a ${existingDescription.length}-character description on '${alias}': ` +
                         'too large to be real. It will be restored from the server on the next sync.\n');
                 }
                 else {

package/dist/commands/auth/connect.js

@@ -83,13 +83,13 @@ export default class AuthConnect extends BaseCommand {
         // 3. Show the code + URL and the one-time enable-setting guidance.
         emit('');
         emit('To connect ChatGPT:');
-        emit(`  1. Open this URL in any browser (phone, laptop — anywhere):`);
+        emit(`  1. Open this URL in any browser (phone, laptop, anywhere):`);
         emit(`       ${verificationUri}`);
         emit(`  2. Enter this code:  ${start.user_code}`);
         emit(`  3. Sign in to ChatGPT and click Continue.`);
         emit('');
         emit('If this is your first time, you may need to enable device-code authorization in ChatGPT first:');
-        emit(`  1. Open ${SETTINGS_URL}  (the settings panel can take a few seconds to appear — give it a moment)`);
+        emit(`  1. Open ${SETTINGS_URL}  (the settings panel can take a few seconds to appear, so give it a moment)`);
         emit('  2. Turn on "Enable device code authorization for Codex"');
         emit('  3. Then enter the code above');
         emit('');
@@ -193,7 +193,7 @@ export default class AuthConnect extends BaseCommand {
                 const steps = Array.isArray(d.settings_steps) && d.settings_steps.length > 0
                     ? d.settings_steps
                     : [
-                        `Open ${url}  (the settings panel can take a few seconds to appear — give it a moment)`,
+                        `Open ${url}  (the settings panel can take a few seconds to appear, so give it a moment)`,
                         'Turn on "Enable device code authorization for Codex"',
                         'Then run `faces auth:connect openai` again',
                     ];

package/dist/commands/auth/connections.js

@@ -26,7 +26,7 @@ export default class AuthConnections extends BaseCommand {
                         const tier = label ? ` (${label})` : '';
                         this.log(`openai  connected${who}${tier}  connected_at=${row.connected_at}`);
                         if (!plan || plan === 'free') {
-                            this.log('  ⚠️  This ChatGPT plan may not include API access — connecting requires a paid plan (Plus/Pro/Team, etc.).');
+                            this.log('  ⚠️  This ChatGPT plan may not include API access. Connecting requires a paid plan (Plus/Pro/Team, etc.).');
                         }
                     }
                     else {

package/dist/commands/billing/subscription/activate.js

@@ -46,7 +46,7 @@ export default class BillingSubscriptionActivate extends BaseCommand {
             }
             const result = data;
             if (!json) {
-                this.log(`Reactivated. Connect plan continues — next renewal: ${result.current_period_end ?? 'unknown'}.`);
+                this.log(`Reactivated. Connect plan continues. Next renewal: ${result.current_period_end ?? 'unknown'}.`);
             }
             return { status: 'reactivated', ...result };
         }
@@ -63,7 +63,7 @@ export default class BillingSubscriptionActivate extends BaseCommand {
         const result = data;
         const url = result.checkout_url;
         if (!json) {
-            this.log('Subscription Connect — $17/month');
+            this.log('Subscription Connect: $17/month');
             this.log('');
             this.log('Complete payment at this link:');
             this.log(`  ${url}`);

package/dist/commands/catalog/doctor.js

@@ -301,7 +301,7 @@ export default class CatalogDoctor extends BaseCommand {
                 }
                 catch (err) {
                     const msg = err instanceof FacesAPIError ? `Error (${err.statusCode}): ${err.message}` : String(err);
-                    this.log(`  ${alias}: failed — ${msg}`);
+                    this.log(`  ${alias}: failed: ${msg}`);
                 }
             }
             this.log(`Generated ${generated} description(s).`);

package/dist/commands/catalog/list.js

@@ -19,7 +19,7 @@ export default class CatalogList extends BaseCommand {
         if (this.jsonEnabled())
             return entries;
         if (entries.length === 0) {
-            this.log('(no faces in catalog — run faces catalog:doctor --fix)');
+            this.log('(no faces in catalog. Run faces catalog:doctor --fix)');
             return entries;
         }
         const nameWidth = Math.max(...entries.map((e) => String(e.alias ?? '').length));

package/dist/commands/chat/thread.js

@@ -73,7 +73,7 @@ function listThreads() {
     return out;
 }
 export default class ChatThread extends BaseCommand {
-    static description = 'Multi-turn chat thread. Conversation history is stored under ~/.faces/threads/<id>.json — resume any thread later with --id.';
+    static description = 'Multi-turn chat thread. Conversation history is stored under ~/.faces/threads/<id>.json. Resume any thread later with --id.';
     static examples = [
         '<%= config.bin %> <%= command.id %> socrates -m "What is justice?"',
         '<%= config.bin %> <%= command.id %> --id t_abc123 -m "Say more about that"',
@@ -99,7 +99,7 @@ export default class ChatThread extends BaseCommand {
     };
     static args = {
         face_username: Args.string({
-            description: 'Face alias (alias@model, or owner:alias@model for a published face) — required when starting a new thread',
+            description: 'Face alias (alias@model, or owner:alias@model for a published face). Required when starting a new thread',
             required: false,
         }),
     };

package/dist/commands/compile/all.js

@@ -64,7 +64,7 @@ export default class CompileAll extends BaseCommand {
         }
         if (items.length === 0) {
             if (!json)
-                this.log('Nothing to compile — all documents and threads are up to date.');
+                this.log('Nothing to compile. All documents and threads are up to date.');
             return { compiled: 0, failed: 0, items: [] };
         }
         if (!json)

package/dist/commands/compile/doc/index.js

@@ -17,7 +17,7 @@ export default class CompileDoc extends BaseCommand {
         ...BaseCommand.baseFlags,
         label: Flags.string({ description: 'Document label/title (single document only; with several files each is labelled by its filename)' }),
         content: Flags.string({ description: 'Inline text content', exclusive: ['file'] }),
-        file: Flags.string({ description: 'Read content from file (repeatable — each file becomes its own document)', multiple: true }),
+        file: Flags.string({ description: 'Read content from file (repeatable; each file becomes its own document)', multiple: true }),
         medium: Flags.string({ description: MEDIUM_FLAG_DESCRIPTION }),
         perspective: Flags.string({
             description: 'Perspective',
@@ -177,8 +177,8 @@ export default class CompileDoc extends BaseCommand {
 }
 /** Extensions the compile-documents endpoint takes as inline text. */
 const BINARY_HINTS = {
-    '.pdf': 'PDF is supported, but not by this command — use: faces compile:upload <alias> --file <path> --kind document',
-    '.docx': 'Word documents are supported, but not by this command — use: faces compile:upload <alias> --file <path> --kind document',
+    '.pdf': 'PDF is supported, but not by this command. Use: faces compile:upload <alias> --file <path> --kind document',
+    '.docx': 'Word documents are supported, but not by this command. Use: faces compile:upload <alias> --file <path> --kind document',
     '.doc': "Legacy '.doc' is not supported. Open it in Word or Pages and save as .docx, then upload that with: faces compile:upload <alias> --file <path> --kind document",
     '.pages': 'Pages documents are not supported. Export it as text or PDF first.',
     '.rtf': 'RTF is not supported. Convert it to plain text first (e.g. `textutil -convert txt file.rtf`).',

package/dist/commands/compile/doc/pause.js

@@ -60,13 +60,13 @@ export default class CompileDocPause extends BaseCommand {
             }
             case 'synced':
             case 'ready': {
-                return 'Compile finished before the pause took effect — nothing to resume.';
+                return 'Compile finished before the pause took effect. Nothing to resume.';
             }
             case 'failed': {
                 return 'Compile failed before the pause took effect.';
             }
             case null: {
-                return 'Stopped — the compile is no longer running.';
+                return 'Stopped. The compile is no longer running.';
             }
             default: {
                 return `Still stopping after ${timeout}s. Poll with: faces compile:doc:get ${id} --json`;

package/dist/commands/compile/import.js

@@ -56,7 +56,7 @@ export default class CompileImport extends BaseCommand {
         catch (err) {
             if (err instanceof FacesAPIError) {
                 if (err.statusCode === 422 && flags.type === 'thread') {
-                    this.error(`${err.message} — try again with --type document`);
+                    this.error(`${err.message}. Try again with --type document`);
                 }
                 this.error(`Error (${err.statusCode}): ${err.message}`);
             }

package/dist/commands/compile/thread/pause.js

@@ -61,13 +61,13 @@ export default class CompileThreadPause extends BaseCommand {
             }
             case 'synced':
             case 'ready': {
-                return 'Compile finished before the pause took effect — nothing to resume.';
+                return 'Compile finished before the pause took effect. Nothing to resume.';
             }
             case 'failed': {
                 return 'Compile failed before the pause took effect.';
             }
             case null: {
-                return 'Stopped — the compile is no longer running.';
+                return 'Stopped. The compile is no longer running.';
             }
             default: {
                 return `Still stopping after ${timeout}s. Poll with: faces compile:thread:get ${id} --json`;

package/dist/commands/face/diff.js

@@ -30,7 +30,7 @@ export default class FaceDiff extends BaseCommand {
         if (!this.jsonEnabled()) {
             const res = data;
             const faces = res.faces;
-            const fmt = (v) => (v === null || v === undefined ? '—' : v.toFixed(2));
+            const fmt = (v) => (v === null || v === undefined ? '-' : v.toFixed(2));
             for (let i = 0; i < faces.length; i++) {
                 for (let j = i + 1; j < faces.length; j++) {
                     const a = faces[i];

package/dist/commands/face/lock.js

@@ -2,7 +2,7 @@ import { Args } from '@oclif/core';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 export default class FaceLock extends BaseCommand {
-    static description = 'Lock a face (read-only). A locked face still reads and chats normally, but it — and everything it owns (documents, threads, voiceprint) — cannot be changed until unlocked.';
+    static description = 'Lock a face (read-only). A locked face still reads and chats normally, but it, and everything it owns (documents, threads, voiceprint), cannot be changed until unlocked.';
     static examples = ['<%= config.bin %> <%= command.id %> socrates'];
     static flags = {
         ...BaseCommand.baseFlags,
@@ -24,7 +24,7 @@ export default class FaceLock extends BaseCommand {
         }
         if (!this.jsonEnabled()) {
             const res = data;
-            this.log(`Locked '${res.alias ?? args.face_id}' — read only.`);
+            this.log(`Locked '${res.alias ?? args.face_id}' is now read only.`);
             const affected = res.affected ?? [];
             if (affected.length > 0)
                 this.log(`Affected: ${affected.join(', ')}`);

package/dist/commands/face/neighbors.js

@@ -6,7 +6,7 @@ export default class FaceNeighbors extends BaseCommand {
     static flags = {
         ...BaseCommand.baseFlags,
         k: Flags.integer({
-            description: 'Number of results (1–20)',
+            description: 'Number of results (1-20)',
             default: 5,
             min: 1,
             max: 20,

package/dist/commands/face/share.js

@@ -15,7 +15,7 @@ export default class FaceShare extends BaseCommand {
         ...BaseCommand.baseFlags,
         list: Flags.boolean({ description: 'Show who the face is shared with, and change nothing', default: false }),
         add: Flags.string({
-            description: 'Add an account, keeping everyone already on the list — username or email (repeatable)',
+            description: 'Add an account, keeping everyone already on the list. Username or email (repeatable)',
             multiple: true,
             exclusive: ['with', 'none'],
         }),

package/dist/commands/face/unlock.js

@@ -24,7 +24,7 @@ export default class FaceUnlock extends BaseCommand {
         }
         if (!this.jsonEnabled()) {
             const res = data;
-            this.log(`Unlocked '${res.alias ?? args.face_id}' — writable.`);
+            this.log(`Unlocked '${res.alias ?? args.face_id}' is now writable.`);
             const affected = res.affected ?? [];
             if (affected.length > 0)
                 this.log(`Affected: ${affected.join(', ')}`);

package/dist/commands/face/unpublish.js

@@ -2,7 +2,7 @@ import { Args } from '@oclif/core';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 export default class FaceUnpublish extends BaseCommand {
-    static description = 'Stop publishing a face. Anyone it is individually shared with keeps their access — publishing is an ' +
+    static description = 'Stop publishing a face. Anyone it is individually shared with keeps their access. Publishing is an ' +
         'override on top of sharing, not a replacement for it.';
     static flags = { ...BaseCommand.baseFlags };
     static args = {

package/dist/commands/face/unshare.js

@@ -2,7 +2,7 @@ import { Args, Flags } from '@oclif/core';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 export default class FaceUnshare extends BaseCommand {
-    static description = 'Revoke access to a shared face. Takes effect immediately — there is no grace period.';
+    static description = 'Revoke access to a shared face. Takes effect immediately. There is no grace period.';
     static examples = [
         '<%= config.bin %> <%= command.id %> alice --from dana --yes',
         '<%= config.bin %> <%= command.id %> alice --all --yes',

package/dist/commands/keys/create.js

@@ -55,15 +55,15 @@ export default class KeysCreate extends BaseCommand {
         }
         this.printHuman(data);
         this.log('');
-        this.log('⚠  The full key is shown only once — `faces keys:list` returns it truncated by design.');
+        this.log('⚠  The full key is shown only once. `faces keys:list` returns it truncated by design.');
         if (saved) {
-            this.log('✓  Saved to ~/.faces/config.json as api_key — faces commands will use it automatically.');
+            this.log('✓  Saved to ~/.faces/config.json as api_key. Faces commands will use it automatically.');
         }
         else if (!flags.save) {
             this.log('   Not saved (--no-save). Copy the key above now.');
         }
         else {
-            this.warn('Could not find the key in the response — nothing was saved. Copy it manually.');
+            this.warn('Could not find the key in the response. Nothing was saved. Copy it manually.');
         }
         return data;
     }

package/dist/commands/style/delete.js

@@ -4,15 +4,20 @@ import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 import { STYLE_FACES_PATH } from '../../style.js';
 /**
- * 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`.
+ * The API takes a scope. 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.
+ * `map` removes the installed style. `all` removes it and everything derived
+ * from it — versions, jobs, checkpoints, style exemplars. **Neither touches the
+ * uploaded material.** The API reference said `all` destroyed it; that
+ * documentation was wrong and has been corrected, and the behaviour is not
+ * going to change (backend memo, 2026-08-28 pm).
+ *
+ * So the reason for sending only `map` is no longer that `all` is dangerous to
+ * source text. It is that one command should do one thing: this one forgets a
+ * style, and a deeper purge of derived artefacts is a different request nobody
+ * has asked for. Source text is deleted through the commands that own it,
+ * compile:doc:delete and compile:thread:delete, and a face's uploaded corpus
+ * through the corpus route — never as a side effect of a flag value here.
  */
 const SCOPE = 'map';
 export default class StyleDelete extends BaseCommand {

package/dist/commands/style/make.js

@@ -173,14 +173,23 @@ export default class StyleMake extends BaseCommand {
         else {
             chosen = [];
             const missing = [];
+            // Naming a source twice is accepted by the server and prints it twice,
+            // which weights it double in the analysis for no stated reason. A later
+            // mention wins so `--source X --source X:essay` declares a medium rather
+            // than being a contradiction.
+            const seen = new Map();
             for (const raw of flags.source ?? []) {
                 const { id, medium } = parseSourceArg(raw);
-                const s = byId.get(id);
-                if (s)
-                    chosen.push({ s, declared: medium });
-                else
+                if (!byId.has(id)) {
                     missing.push(id);
+                    continue;
+                }
+                if (seen.has(id) && medium === undefined)
+                    continue;
+                seen.set(id, medium ?? seen.get(id));
             }
+            for (const [id, medium] of seen)
+                chosen.push({ s: byId.get(id), declared: medium });
             if (missing.length > 0) {
                 this.error(`Not a source on '${alias}': ${missing.join(', ')}\n` +
                     `List what is there with: faces face:sources ${alias}`);

package/dist/style.js

@@ -181,6 +181,16 @@ export async function pollStyleJob(client, basePath, jobId, opts = {}) {
  * A build failure is shown to the user, and a stack trace or a SQL statement
  * tells them nothing they can act on while burying whatever might have. These
  * are reported as an upstream fault with the first line kept for a bug report.
+ *
+ * The server sanitises `error` itself now (faces-backend-shared#623), so this
+ * is inert against everything it currently returns — checked against the
+ * sanitised text and against every deliberate code it documents
+ * (CHATGPT_AUTH, INSUFFICIENT_CREDITS, NOTHING_AUTHORED, HOLDOUT_UNMEETABLE,
+ * OAUTH_RATE_LIMITED), none of which match. It is kept as a backstop rather
+ * than removed: `error` is rendered straight to a user, a raw statement reached
+ * one once, and a check that costs nothing until it fires is worth more than
+ * the line it saves. It is deliberately narrow so an actionable message can
+ * never be swallowed by it.
  */
 const INTERNAL_FAULT = /sqlalchemy|asyncpg|psycopg|Traceback|\[SQL:|IntegrityError|OperationalError/i;
 /** Failure text, with the one hint that is actionable rather than descriptive. */

package/dist/utils.js

@@ -95,7 +95,7 @@ export function isMaxTokensRenameError(message) {
  * synonyms fold server-side, the set grows, and compile's own 422 names the
  * valid ones better than a stale local list could.
  */
-export const MEDIUM_FLAG_DESCRIPTION = 'What sort of writing this is — e.g. email, text message, social post, essay, academic paper, ' +
+export const MEDIUM_FLAG_DESCRIPTION = 'What sort of writing this is, e.g. email, text message, social post, essay, academic paper, ' +
     'blog post, legal document, thread reply, conversation (dialogue: transcripts, interviews, calls), ' +
     'lecture (sustained speech nobody interrupts: talks, sermons, keynotes). Common synonyms fold ' +
     'automatically. Omit it if you do not know: a wrong declaration is worse than none, because a ' +
@@ -120,7 +120,7 @@ export function warnDroppedParams(headers, flagFor) {
         .map((p) => flagFor[p] ?? `--${p.replaceAll('_', '-')}`);
     if (flags.length === 0)
         return;
-    process.stderr.write(`Note: ${flags.join(' and ')} had no effect — this request ran on your linked ChatGPT account, ` +
+    process.stderr.write(`Note: ${flags.join(' and ')} had no effect. This request ran on your linked ChatGPT account, ` +
         'which does not accept them. Use --no-oauth-only to run on the paid API instead.\n');
 }
 /** Maps the server's parameter names back to the flags a user actually typed. */