faces-cli 1.8.5 → 1.8.7

package/dist/style.d.ts

@@ -24,12 +24,12 @@ export declare const STYLE_FACES_PATH = "/v1/voiceprint/faces";
  */
 export declare const DEFAULT_STYLE_MODEL = "gpt-5.6-terra";
 /**
- * Medium assumed for a room that does not declare one.
+ * Medium assumed for a room that declares nothing at all.
  *
- * Corpus rooms carry `corpus_kind`. Ordinary threads do not, because they are
- * live conversations rather than an imported corpus — so a thread with no
- * declared medium is a conversation, which is the one case where the medium
- * follows from the source type rather than from a guess about content.
+ * Only live threads reach this. An imported corpus always reports a
+ * `corpus_medium`, so a room with none is a live conversation, and that is the
+ * one case where the medium follows from the source type rather than from a
+ * guess about content.
  */
 export declare const ROOM_DEFAULT_MEDIUM = "conversation";
 /** Exit code for "no such face", so a caller can tell it from a face with no style. */
@@ -48,8 +48,10 @@ export declare function faceExists(client: FacesClient, alias: string): Promise<
 export interface Selectable {
     id: string;
     sourceType: 'room' | 'document';
-    /** Resolved from the listing. Undefined means the caller has to declare it. */
+    /** What the source itself declares. Undefined means nobody has said. */
     medium?: string;
+    /** A room with no imported corpus behind it, i.e. a live conversation. */
+    isLiveThread: boolean;
     label: string;
     synced: boolean;
     /**
@@ -67,10 +69,20 @@ export interface Selectable {
     printedAt?: string;
     printedMedium?: string;
 }
-/** What goes on the wire. `medium`, not `kind` — renamed 2026-08-27, no alias. */
+/**
+ * What goes on the wire.
+ *
+ * `medium` is optional and is sent only when the caller chose one. Omitting it
+ * lets the server resolve from what the source already declares, which is
+ * strictly safer than resolving here: reading the wrong field once filed three
+ * corpus rooms as conversations, and a value we do not send cannot be wrong.
+ *
+ * `kind` was the old name and is refused rather than ignored, so there is no
+ * chance of sending it by accident.
+ */
 export interface SourceRef {
     id: string;
-    medium: string;
+    medium?: string;
     source_type: 'room' | 'document';
 }
 /**

package/dist/style.js

@@ -24,12 +24,12 @@ export const STYLE_FACES_PATH = '/v1/voiceprint/faces';
  */
 export const DEFAULT_STYLE_MODEL = 'gpt-5.6-terra';
 /**
- * Medium assumed for a room that does not declare one.
+ * Medium assumed for a room that declares nothing at all.
  *
- * Corpus rooms carry `corpus_kind`. Ordinary threads do not, because they are
- * live conversations rather than an imported corpus — so a thread with no
- * declared medium is a conversation, which is the one case where the medium
- * follows from the source type rather than from a guess about content.
+ * Only live threads reach this. An imported corpus always reports a
+ * `corpus_medium`, so a room with none is a live conversation, and that is the
+ * one case where the medium follows from the source type rather than from a
+ * guess about content.
  */
 export const ROOM_DEFAULT_MEDIUM = 'conversation';
 /**
@@ -76,10 +76,13 @@ export async function faceExists(client, alias) {
 function mediumOf(r, sourceType) {
     if (sourceType === 'document')
         return r.medium?.trim() || undefined;
-    const kind = r.corpus_kind?.trim();
-    if (!kind)
-        return ROOM_DEFAULT_MEDIUM;
-    return kind === UNIDENTIFIED_CORPUS ? undefined : kind;
+    // A room carries two: `medium` is the user's correction, `corpus_medium` is
+    // what its messages arrived as. The correction wins, which is the same order
+    // the server resolves in.
+    const declared = r.medium?.trim() || r.corpus_medium?.trim();
+    // `unknown` is what an import reports when a message declared nothing. It is
+    // not a medium and the server refuses it, so it counts as undeclared here.
+    return !declared || declared === UNIDENTIFIED_CORPUS ? undefined : declared;
 }
 function rowToSelectable(r, sourceType) {
     const stamp = r.voiceprint ?? undefined;
@@ -91,7 +94,8 @@ function rowToSelectable(r, sourceType) {
         medium: mediumOf(r, sourceType),
         label: r.label?.trim() || '(untitled)',
         synced: Boolean(r.synced),
-        isCorpus: sourceType === 'document' || Boolean(r.corpus_kind?.trim()),
+        isCorpus: sourceType === 'document' || Boolean(r.corpus_medium?.trim()),
+        isLiveThread: sourceType === 'room' && !r.corpus_medium?.trim(),
         authored: r.authored_message_count ?? undefined,
         // The stamp was renamed alongside the request field, but stored stamps kept
         // the old key (faces-backend-shared#589), so both are read.
@@ -171,9 +175,35 @@ export async function pollStyleJob(client, basePath, jobId, opts = {}) {
             return data;
     }
 }
+/**
+ * Markers of a server-side fault that leaked its internals into the job error.
+ *
+ * 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. */
 export function jobFailureMessage(error) {
     const e = typeof error === 'string' && error ? error : 'unknown error';
+    if (INTERNAL_FAULT.test(e)) {
+        // Keep a short, quotable head so a bug report can name the fault, and drop
+        // the rest rather than printing a query at somebody.
+        const head = e.split('\n')[0].slice(0, 200);
+        return (`the server failed internally, so nothing was installed.\n${head}\n` +
+            'This is a fault on the server, not something you did. Re-running is unlikely to help; ' +
+            'report it with the job id from above.');
+    }
     if (e.toUpperCase().startsWith('CHATGPT_AUTH')) {
         return (`${e}\n` +
             'The ChatGPT link has expired. Reconnect it with: faces auth:connect openai\n' +

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. */