faces-cli 1.6.2 → 1.6.4

package/dist/commands/face/tag/add.js

@@ -1,6 +1,7 @@
 import { Args, Flags } from '@oclif/core';
 import { BaseCommand } from '../../../base.js';
 import { FacesAPIError } from '../../../client.js';
+import { CatalogService } from '../../../catalog.js';
 export default class FaceTagAdd extends BaseCommand {
     static description = 'Add tags to a face (appends, idempotent)';
     static flags = {
@@ -18,10 +19,20 @@ export default class FaceTagAdd extends BaseCommand {
             data = await client.post(`/v1/iam/${args.alias}/tags`, { body: { tags: flags.tag } });
         }
         catch (err) {
-            if (err instanceof FacesAPIError)
+            if (err instanceof FacesAPIError) {
+                if (err.statusCode === 403) {
+                    this.error(`Forbidden: face '${args.alias}' is not owned by the current account. Run \`faces catalog:doctor --fix\` to sync.`);
+                }
                 this.error(`Error (${err.statusCode}): ${err.message}`);
+            }
             throw err;
         }
+        // Sync tags to local catalog
+        const tags = data.tags ?? [];
+        try {
+            new CatalogService().updateTags(args.alias, tags);
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled())
             this.printHuman(data);
         return data;

package/dist/commands/face/tag/all.d.ts

@@ -0,0 +1,10 @@
+import { BaseCommand } from '../../../base.js';
+export default class FaceTagAll extends BaseCommand {
+    static description: string;
+    static flags: {
+        '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>;
+        'api-key': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/face/tag/all.js

@@ -0,0 +1,31 @@
+import { BaseCommand } from '../../../base.js';
+import { FacesAPIError } from '../../../client.js';
+export default class FaceTagAll extends BaseCommand {
+    static description = 'List all tags across all your faces';
+    static flags = {
+        ...BaseCommand.baseFlags,
+    };
+    async run() {
+        const { flags } = await this.parse(FaceTagAll);
+        const client = this.makeClient(flags);
+        let data;
+        try {
+            data = await client.get('/v1/me/tags');
+        }
+        catch (err) {
+            if (err instanceof FacesAPIError)
+                this.error(`Error (${err.statusCode}): ${err.message}`);
+            throw err;
+        }
+        if (!this.jsonEnabled()) {
+            const tags = data.tags ?? [];
+            if (tags.length === 0) {
+                this.log('(no tags)');
+            }
+            else {
+                this.log(tags.join(', '));
+            }
+        }
+        return data;
+    }
+}

package/dist/commands/face/tag/remove.js

@@ -1,6 +1,7 @@
 import { Args } from '@oclif/core';
 import { BaseCommand } from '../../../base.js';
 import { FacesAPIError } from '../../../client.js';
+import { CatalogService } from '../../../catalog.js';
 export default class FaceTagRemove extends BaseCommand {
     static description = 'Remove a tag from a face';
     static flags = {
@@ -18,10 +19,20 @@ export default class FaceTagRemove extends BaseCommand {
             data = await client.delete(`/v1/iam/${args.alias}/tags/${encodeURIComponent(args.tag)}`);
         }
         catch (err) {
-            if (err instanceof FacesAPIError)
+            if (err instanceof FacesAPIError) {
+                if (err.statusCode === 403) {
+                    this.error(`Forbidden: face '${args.alias}' is not owned by the current account. Run \`faces catalog:doctor --fix\` to sync.`);
+                }
                 this.error(`Error (${err.statusCode}): ${err.message}`);
+            }
             throw err;
         }
+        // Refresh tags in local catalog
+        try {
+            const tagResp = await client.get(`/v1/iam/${args.alias}/tags`);
+            new CatalogService().updateTags(args.alias, tagResp.tags ?? []);
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled())
             this.log(`Removed tag '${args.tag}' from ${args.alias}`);
         return data;

package/dist/commands/face/tag/set.js

@@ -1,6 +1,7 @@
 import { Args, Flags } from '@oclif/core';
 import { BaseCommand } from '../../../base.js';
 import { FacesAPIError } from '../../../client.js';
+import { CatalogService } from '../../../catalog.js';
 export default class FaceTagSet extends BaseCommand {
     static description = 'Replace all tags on a face';
     static flags = {
@@ -18,10 +19,20 @@ export default class FaceTagSet extends BaseCommand {
             data = await client.put(`/v1/iam/${args.alias}/tags`, { body: { tags: flags.tag } });
         }
         catch (err) {
-            if (err instanceof FacesAPIError)
+            if (err instanceof FacesAPIError) {
+                if (err.statusCode === 403) {
+                    this.error(`Forbidden: face '${args.alias}' is not owned by the current account. Run \`faces catalog:doctor --fix\` to sync.`);
+                }
                 this.error(`Error (${err.statusCode}): ${err.message}`);
+            }
             throw err;
         }
+        // Sync tags to local catalog
+        const tags = data.tags ?? [];
+        try {
+            new CatalogService().updateTags(args.alias, tags);
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled())
             this.printHuman(data);
         return data;

package/dist/commands/face/teams.d.ts

@@ -0,0 +1,14 @@
+import { BaseCommand } from '../../base.js';
+export default class FaceTeams extends BaseCommand {
+    static description: string;
+    static flags: {
+        team: import("@oclif/core/interfaces").OptionFlag<string[], import("@oclif/core/interfaces").CustomOptions>;
+        '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>;
+        'api-key': import("@oclif/core/interfaces").OptionFlag<string | undefined, import("@oclif/core/interfaces").CustomOptions>;
+    };
+    static args: {
+        alias: import("@oclif/core/interfaces").Arg<string, Record<string, unknown>>;
+    };
+    run(): Promise<unknown>;
+}

package/dist/commands/face/teams.js

@@ -0,0 +1,29 @@
+import { Args, Flags } from '@oclif/core';
+import { BaseCommand } from '../../base.js';
+import { FacesAPIError } from '../../client.js';
+export default class FaceTeams extends BaseCommand {
+    static description = "Set a face's team memberships (replaces all)";
+    static flags = {
+        ...BaseCommand.baseFlags,
+        team: Flags.string({ description: 'Team ID (repeatable, replaces all memberships)', multiple: true, required: true }),
+    };
+    static args = {
+        alias: Args.string({ description: 'Face alias', required: true }),
+    };
+    async run() {
+        const { args, flags } = await this.parse(FaceTeams);
+        const client = this.makeClient(flags);
+        let data;
+        try {
+            data = await client.put(`/v1/faces/${args.alias}/teams`, { body: { team_ids: flags.team } });
+        }
+        catch (err) {
+            if (err instanceof FacesAPIError)
+                this.error(`Error (${err.statusCode}): ${err.message}`);
+            throw err;
+        }
+        if (!this.jsonEnabled())
+            this.printHuman(data);
+        return data;
+    }
+}

package/dist/commands/face/update.js

@@ -2,7 +2,7 @@ import { Args, Flags } from '@oclif/core';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
 import { CatalogService } from '../../catalog.js';
-import { flattenBasicFacts } from '../../utils.js';
+import { renameFaceFields } from '../../utils.js';
 export default class FaceUpdate extends BaseCommand {
     static description = "Update a face's metadata";
     static flags = {
@@ -67,9 +67,11 @@ export default class FaceUpdate extends BaseCommand {
             }
         }
         // Replace tags if provided
+        let currentTags;
         if (hasTags) {
             try {
-                await client.put(`/v1/iam/${args.face_id}/tags`, { body: { tags: flags.tag } });
+                const tagResp = await client.put(`/v1/iam/${args.face_id}/tags`, { body: { tags: flags.tag } });
+                currentTags = tagResp.tags ?? flags.tag;
             }
             catch (err) {
                 if (err instanceof FacesAPIError)
@@ -81,12 +83,22 @@ export default class FaceUpdate extends BaseCommand {
             for (const w of face.warnings)
                 this.warn(w);
         }
-        if (face.basic_facts && typeof face.basic_facts === 'object') {
-            face.basic_facts = flattenBasicFacts(face.basic_facts);
-        }
+        // Add tags to response if we set them
+        if (currentTags)
+            face.tags = currentTags;
+        // Rename basic_facts → attributes
+        renameFaceFields(face);
+        // Write to local catalog
         try {
             const catalog = new CatalogService();
-            catalog.writeFace(face, flags.description);
+            catalog.writeFace({
+                ...face,
+                basic_facts: face.attributes,
+                tags: face.tags ?? undefined,
+                description: face.description ?? undefined,
+                default_model: face.default_model,
+                formula: face.formula,
+            });
         }
         catch { /* catalog errors are non-fatal */ }
         if (!this.jsonEnabled())

package/dist/commands/team/add.js

@@ -1,6 +1,7 @@
 import { Args, Flags } from '@oclif/core';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
+import { TeamCatalogService } from '../../team-catalog.js';
 export default class TeamAdd extends BaseCommand {
     static description = 'Add face(s) to a team';
     static flags = {
@@ -43,6 +44,22 @@ export default class TeamAdd extends BaseCommand {
                 this.error(`Error (${err.statusCode}): ${err.message}`);
             throw err;
         }
+        // Update local TEAM.md — find by team ID and add aliases
+        try {
+            const teamCatalog = new TeamCatalogService();
+            for (const name of teamCatalog.listTeams()) {
+                const team = teamCatalog.readTeam(name);
+                if (team?.frontmatter.id === args.team_id) {
+                    const members = new Set(team.frontmatter.members ?? []);
+                    for (const alias of flags.face)
+                        members.add(alias);
+                    team.frontmatter.members = [...members];
+                    teamCatalog.writeTeam(name, team.frontmatter, team.body || undefined);
+                    break;
+                }
+            }
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled()) {
             this.log(`Added ${flags.face.join(', ')} to team.`);
         }

package/dist/commands/team/create.js

@@ -2,6 +2,7 @@ import { Flags } from '@oclif/core';
 import fs from 'node:fs';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
+import { TeamCatalogService } from '../../team-catalog.js';
 export default class TeamCreate extends BaseCommand {
     static description = 'Create a new team';
     static flags = {
@@ -37,15 +38,30 @@ export default class TeamCreate extends BaseCommand {
         }
         const team = data;
         // Post tags if provided
-        if (flags.tag && flags.tag.length > 0) {
+        const tags = flags.tag ?? [];
+        if (tags.length > 0) {
             try {
-                await client.post(`/v1/teams/${team.id}/tags`, { body: { tags: flags.tag } });
+                await client.post(`/v1/teams/${team.id}/tags`, { body: { tags } });
+                team.tags = tags;
             }
             catch (err) {
                 if (err instanceof FacesAPIError)
                     this.warn(`Tags failed (${err.statusCode}): ${err.message}`);
             }
         }
+        // Write local TEAM.md
+        try {
+            const teamCatalog = new TeamCatalogService();
+            const slug = String(team.name).toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/^-|-$/g, '');
+            teamCatalog.writeTeam(slug, {
+                id: team.id,
+                name: team.name,
+                description: team.description ?? undefined,
+                tags: team.tags ?? undefined,
+                members: [],
+            }, team.protocol ?? protocol ?? undefined);
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled())
             this.printHuman(data);
         return data;

package/dist/commands/team/list.js

@@ -8,14 +8,6 @@ export default class TeamList extends BaseCommand {
     async run() {
         const { flags } = await this.parse(TeamList);
         const client = this.makeClient(flags);
-        // Get current username to filter out personal Guild
-        let username;
-        try {
-            const resp = await client.get('/auth/me');
-            const whoami = (resp.data ?? resp);
-            username = whoami.username ?? undefined;
-        }
-        catch { /* proceed without filtering */ }
         let data;
         try {
             data = await client.get('/v1/me/teams');
@@ -27,10 +19,8 @@ export default class TeamList extends BaseCommand {
         }
         const resp = data;
         let teams = resp.data ?? data;
-        // Filter out personal Guild
-        if (username) {
-            teams = teams.filter(t => t.name !== username);
-        }
+        // Filter out personal Guild using kind field
+        teams = teams.filter(t => t.kind !== 'personal');
         if (this.jsonEnabled())
             return teams;
         if (teams.length === 0) {
@@ -40,8 +30,9 @@ export default class TeamList extends BaseCommand {
             const nameWidth = Math.max(...teams.map(t => String(t.name ?? '').length));
             for (const t of teams) {
                 const name = String(t.name ?? '').padEnd(nameWidth);
-                const desc = t.description ? `  ${String(t.description).slice(0, 60)}` : '';
-                this.log(`${t.id}  ${name}${desc}`);
+                const members = t.member_count != null ? `  [${t.member_count} members]` : '';
+                const desc = t.description ? `  ${String(t.description).slice(0, 50)}` : '';
+                this.log(`${t.id}  ${name}${members}${desc}`);
             }
         }
         return teams;

package/dist/commands/team/remove.js

@@ -1,6 +1,7 @@
 import { Args } from '@oclif/core';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
+import { TeamCatalogService } from '../../team-catalog.js';
 export default class TeamRemove extends BaseCommand {
     static description = 'Remove a face from a team';
     static flags = {
@@ -35,6 +36,19 @@ export default class TeamRemove extends BaseCommand {
                 this.error(`Error (${err.statusCode}): ${err.message}`);
             throw err;
         }
+        // Update local TEAM.md — find by team ID and remove alias
+        try {
+            const teamCatalog = new TeamCatalogService();
+            for (const name of teamCatalog.listTeams()) {
+                const team = teamCatalog.readTeam(name);
+                if (team?.frontmatter.id === args.team_id) {
+                    team.frontmatter.members = (team.frontmatter.members ?? []).filter(m => m !== args.alias);
+                    teamCatalog.writeTeam(name, team.frontmatter, team.body || undefined);
+                    break;
+                }
+            }
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled())
             this.log(`Removed ${args.alias} from team.`);
         return data;

package/dist/commands/team/update.js

@@ -2,6 +2,7 @@ import { Args, Flags } from '@oclif/core';
 import fs from 'node:fs';
 import { BaseCommand } from '../../base.js';
 import { FacesAPIError } from '../../client.js';
+import { TeamCatalogService } from '../../team-catalog.js';
 export default class TeamUpdate extends BaseCommand {
     static description = 'Update a team';
     static flags = {
@@ -41,6 +42,35 @@ export default class TeamUpdate extends BaseCommand {
                 this.error(`Error (${err.statusCode}): ${err.message}`);
             throw err;
         }
+        // Update local TEAM.md
+        try {
+            const team = data;
+            const teamCatalog = new TeamCatalogService();
+            const slug = String(team.name).toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/^-|-$/g, '');
+            // Fetch members for the local file
+            let members = [];
+            try {
+                const membersResp = await client.get(`/v1/teams/${args.team_id}/members`);
+                // Members come as iam_ids — we'd need to resolve, but for now store what we have
+                members = (membersResp.data ?? []).map(m => m.iam_id);
+            }
+            catch { /* proceed without members */ }
+            // Fetch tags
+            let tags = [];
+            try {
+                const tagResp = await client.get(`/v1/teams/${args.team_id}/tags`);
+                tags = tagResp.tags ?? [];
+            }
+            catch { /* proceed */ }
+            teamCatalog.writeTeam(slug, {
+                id: args.team_id,
+                name: team.name,
+                description: team.description ?? undefined,
+                tags,
+                members,
+            }, team.protocol ?? undefined);
+        }
+        catch { /* non-fatal */ }
         if (!this.jsonEnabled())
             this.printHuman(data);
         return data;

package/dist/team-catalog.d.ts

@@ -1,5 +1,6 @@
 export declare const TEAMS_DIR: string;
 export interface TeamFrontmatter {
+    id?: string;
     name: string;
     description?: string;
     tags?: string[];

package/dist/team-catalog.js

@@ -10,6 +10,8 @@ function quoteYaml(value) {
 }
 function serializeFrontmatter(fm) {
     const lines = ['---'];
+    if (fm.id)
+        lines.push(`id: ${fm.id}`);
     lines.push(`name: ${quoteYaml(fm.name)}`);
     if (fm.description)
         lines.push(`description: ${quoteYaml(fm.description)}`);
@@ -35,7 +37,10 @@ function parseFrontmatter(content) {
             continue;
         const key = line.slice(0, colonIdx).trim();
         const val = line.slice(colonIdx + 1).trim();
-        if (key === 'name') {
+        if (key === 'id') {
+            fm.id = val.replace(/^"(.*)"$/, '$1');
+        }
+        else if (key === 'name') {
             fm.name = val.replace(/^"(.*)"$/, '$1');
         }
         else if (key === 'description') {

package/dist/utils.d.ts

@@ -1,5 +1,12 @@
 /**
- * Flatten basic_facts from API's nested {value, status, identification} format
+ * Flatten basic_facts/attributes from API's nested {value, status, identification} format
  * to simple {key: value} strings.
  */
-export declare function flattenBasicFacts(facts: Record<string, unknown>): Record<string, string>;
+export declare function flattenAttributes(facts: Record<string, unknown>): Record<string, string>;
+/** @deprecated Use flattenAttributes */
+export declare const flattenBasicFacts: typeof flattenAttributes;
+/**
+ * Rename basic_facts → attributes in a face response object.
+ * Mutates the object in place and returns it.
+ */
+export declare function renameFaceFields(face: Record<string, unknown>): Record<string, unknown>;

package/dist/utils.js

@@ -1,8 +1,8 @@
 /**
- * Flatten basic_facts from API's nested {value, status, identification} format
+ * Flatten basic_facts/attributes from API's nested {value, status, identification} format
  * to simple {key: value} strings.
  */
-export function flattenBasicFacts(facts) {
+export function flattenAttributes(facts) {
     const flat = {};
     for (const [k, v] of Object.entries(facts)) {
         if (typeof v === 'object' && v !== null && 'value' in v) {
@@ -14,3 +14,18 @@ export function flattenBasicFacts(facts) {
     }
     return flat;
 }
+/** @deprecated Use flattenAttributes */
+export const flattenBasicFacts = flattenAttributes;
+/**
+ * Rename basic_facts → attributes in a face response object.
+ * Mutates the object in place and returns it.
+ */
+export function renameFaceFields(face) {
+    if (face.basic_facts !== undefined) {
+        face.attributes = typeof face.basic_facts === 'object' && face.basic_facts !== null
+            ? flattenAttributes(face.basic_facts)
+            : face.basic_facts;
+        delete face.basic_facts;
+    }
+    return face;
+}