@phnx-labs/agents-cli 1.20.33 → 1.20.35

package/dist/commands/teams.js

@@ -2,7 +2,7 @@ import chalk from 'chalk';
 import * as fs from 'fs/promises';
 import { addHostOption } from '../lib/hosts/option.js';
 import * as path from 'path';
-import { AgentManager, checkAllClis, getAgentsDir, VALID_TASK_TYPES, } from '../lib/teams/agents.js';
+import { AgentManager, checkAllClis, checkCliSignedIn, getAgentsDir, resolveSignInAdvisory, VALID_TASK_TYPES, } from '../lib/teams/agents.js';
 import { resolveProvider } from '../lib/cloud/registry.js';
 import { runSupervisor } from '../lib/teams/supervisor.js';
 import { handleSpawn, handleStatus, handleStop, handleTasks, toTaskStatusSummary, } from '../lib/teams/api.js';
@@ -10,6 +10,7 @@ import { createTeam, ensureTeam, getTeam, loadTeams, removeTeam, teamExists, } f
 import { setHelpSections } from '../lib/help.js';
 import { createWorktree, isGitRepo, hasUncommittedChanges, removeWorktree, } from '../lib/teams/worktree.js';
 import { isVersionInstalled, resolveVersionAlias, resolveVersionAliasLoose } from '../lib/versions.js';
+import { AGENTS, warnAgentDeprecated } from '../lib/agents.js';
 import { discoverSessions, parseTimeFilter, resolveSessionById } from '../lib/session/discover.js';
 import { buildPreview as buildSessionPreview } from './sessions-picker.js';
 import { parseExecEnv } from '../lib/exec.js';
@@ -133,7 +134,7 @@ function parseTeammate(spec) {
         }
     }
     die(`Unknown teammate '${spec}'. Available agents: ${VALID_AGENTS.join(', ')}.\n` +
-        `  Use 'claude', 'claude@2.1.112', or the name of a profile from 'agents view'.`);
+        `  Use 'claude', 'kimi@latest', 'kimi@0.19.2' (a version from 'agents view'), or a profile name.`);
 }
 function shortId(id) {
     return id.slice(0, 8);
@@ -183,6 +184,31 @@ export function wireCloudDispatcher(mgr) {
         return { cloudSessionId: cloudTask.id };
     });
 }
+/**
+ * Advisory: warn once per agent type of any still-pending (staged `--after`)
+ * teammate whose CLI may not be signed in. Warn-only — never blocks `start`.
+ * Local teammates only; cloud teammates authenticate through their provider.
+ */
+async function warnUnsignedTeammates(mgr, team) {
+    let pending;
+    try {
+        pending = (await mgr.listByTask(team)).filter((a) => a.status === 'pending' && !a.cloudProvider);
+    }
+    catch {
+        return; // team not loadable yet — nothing to warn about
+    }
+    const seen = new Set();
+    for (const a of pending) {
+        const agent = a.agentType;
+        if (seen.has(agent) || !AGENT_NAMES[agent])
+            continue;
+        seen.add(agent);
+        if (!(await checkCliSignedIn(agent))) {
+            console.error(chalk.yellow(`⚠ ${AGENT_NAMES[agent]} may not be signed in (detection is unreliable). Launching anyway.`) +
+                chalk.gray(`\n  If it fails to start, run \`${AGENTS[agent].cliCommand}\` to log in, or pass --force to silence this.`));
+        }
+    }
+}
 /** Single-wave start used by `teams start` without --watch. */
 async function runOneWave(mgr, team, json) {
     const launched = await mgr.startReady(team);
@@ -839,6 +865,7 @@ export function registerTeamsCommands(program) {
         .option('--cloud <provider>', `Dispatch to cloud backend instead of local CLI: ${VALID_CLOUD_PROVIDERS.join('|')}`)
         .option('--repo <owner/repo>', 'GitHub repository (required for --cloud rush)')
         .option('--branch <name>', 'Target git branch for cloud dispatch')
+        .option('--force', "Skip the advisory 'may not be signed in' warning (detection is unreliable)")
         .option('--json', 'Output machine-readable JSON')
         .action(async (team, teammate, task, opts) => {
         if (!VALID_MODES.includes(opts.mode)) {
@@ -865,10 +892,18 @@ export function registerTeamsCommands(program) {
             }
         }
         const { agent, version, profileName } = parseTeammate(teammate);
+        warnAgentDeprecated(agent);
         if (version && !isVersionInstalled(agent, version)) {
             die(`${AGENT_NAMES[agent]} ${version} isn't installed.\n` +
                 `  Install it:  agents add ${agent}@${version}\n` +
-                `  Or see what's installed:  agents view ${agent}`);
+                `  Or see what's installed (incl. @latest):  agents view ${agent}`);
+        }
+        // Advisory sign-in check: warn but NEVER block. Detection is unreliable
+        // for opaque-cred agents, so a false negative must not stop a team. Cloud
+        // dispatch authenticates through the provider, not the local CLI — skip it.
+        if (!opts.force && !cloudProviderId && !(await checkCliSignedIn(agent))) {
+            console.error(chalk.yellow(`⚠ ${AGENT_NAMES[agent]} may not be signed in (detection is unreliable). Adding anyway.`) +
+                chalk.gray(`\n  If it fails to start, run \`${AGENTS[agent].cliCommand}\` to log in, or pass --force to silence this.`));
         }
         if (opts.name !== undefined) {
             if (!opts.name || !/^[A-Za-z0-9_-]+$/.test(opts.name)) {
@@ -1139,6 +1174,7 @@ export function registerTeamsCommands(program) {
         .option('--watch', 'Keep running: poll every --interval seconds, fire new waves, exit when the DAG drains.')
         .option('--interval <seconds>', 'Seconds between waves in --watch mode (default 8)', '8')
         .option('--max-waves <n>', 'Safety cap on waves in --watch mode (default 1000)', '1000')
+        .option('--force', "Skip the advisory 'may not be signed in' warning for staged teammates (detection is unreliable)")
         .action(async (team, opts) => {
         const mgr = mkManager();
         wireCloudDispatcher(mgr);
@@ -1148,6 +1184,8 @@ export function registerTeamsCommands(program) {
                 return;
             team = picked;
         }
+        if (!opts.force && !isJsonMode(opts))
+            await warnUnsignedTeammates(mgr, team);
         if (!opts.watch) {
             await runOneWave(mgr, team, Boolean(opts.json));
             return;
@@ -1446,19 +1484,45 @@ export function registerTeamsCommands(program) {
     teams
         .command('doctor')
         .alias('dr')
-        .description('Check which agents are installed and available to join a team. Verifies CLI paths.')
+        .description('Check which agents are installed and available to join a team. Verifies CLI paths and shows an advisory sign-in hint.')
         .option('--json', 'Output machine-readable JSON')
         .action(async (opts) => {
         const info = checkAllClis();
+        // Advisory enrichment only. Sign-in detection is UNRELIABLE, so it never
+        // changes the authoritative installed/ready column — it annotates. And an
+        // agent that is actually running in a team is treated as signed in
+        // regardless of the probe, so doctor never reports a working agent as
+        // logged out ("don't show wrong stuff").
+        const running = new Set();
+        try {
+            for (const a of await mkManager().listRunning())
+                running.add(a.agentType);
+        }
+        catch { /* no teams yet — leave running empty */ }
+        const auth = {};
+        await Promise.all(Object.entries(info).map(async ([name, entry]) => {
+            const isRunning = running.has(name);
+            const probe = entry.installed && !isRunning ? await checkCliSignedIn(name) : false;
+            auth[name] = resolveSignInAdvisory(entry.installed, isRunning, probe);
+        }));
         if (isJsonMode(opts)) {
-            console.log(JSON.stringify(info, null, 2));
+            const merged = {};
+            for (const [name, entry] of Object.entries(info))
+                merged[name] = { ...entry, ...auth[name] };
+            console.log(JSON.stringify(merged, null, 2));
             return;
         }
         console.log(chalk.bold('Who can join a team:'));
         for (const [name, entry] of Object.entries(info)) {
             const pretty = AGENT_NAMES[name] || name;
             if (entry.installed) {
-                console.log(`  ${chalk.green('ready')}  ${pretty.padEnd(10)} ${chalk.gray(entry.path || '')}`);
+                const { signedIn, running: isRunning } = auth[name];
+                const hint = isRunning
+                    ? chalk.gray('in use')
+                    : signedIn
+                        ? chalk.gray('signed in')
+                        : chalk.gray('sign-in unverified');
+                console.log(`  ${chalk.green('ready')}  ${pretty.padEnd(10)} ${chalk.gray(entry.path || '')}  ${hint}`);
             }
             else {
                 console.log(`  ${chalk.red('no   ')}  ${pretty.padEnd(10)} ${chalk.gray(entry.error || 'not installed')}`);

package/dist/commands/versions.js

@@ -4,7 +4,7 @@ import ora from 'ora';
 import * as fs from 'fs';
 import * as path from 'path';
 import { select, confirm, checkbox } from '@inquirer/prompts';
-import { AGENTS, ALL_AGENT_IDS, getAccountEmail, getAccountInfo, agentLabel, } from '../lib/agents.js';
+import { AGENTS, ALL_AGENT_IDS, getAccountEmail, getAccountInfo, agentLabel, warnAgentDeprecated, } from '../lib/agents.js';
 import { formatUsageSummary, getUsageInfoForIdentity, getUsageInfoByIdentity, getUsageLookupKey, } from '../lib/usage.js';
 import { viewAction } from './view.js';
 import { readManifest, writeManifest, createDefaultManifest } from '../lib/manifest.js';
@@ -273,6 +273,7 @@ export function registerVersionsCommands(program) {
             }
             const { agent, version } = parsed;
             const agentConfig = AGENTS[agent];
+            warnAgentDeprecated(agent);
             if (!agentConfig.npmPackage && !agentConfig.installScript) {
                 console.log(chalk.yellow(`${agentLabel(agentConfig.id)} has no npm package. Install manually.`));
                 continue;

package/dist/commands/view.d.ts

@@ -9,6 +9,7 @@
 import type { Command } from 'commander';
 import type { AgentId } from '../lib/types.js';
 import { type ProfileSummary } from '../lib/profiles.js';
+type SyncState = 'synced' | 'new' | 'modified' | 'deleted';
 /** Per-section filter flags. When any are true, only those sections render. */
 export interface ViewSectionFilter {
     commands?: boolean;
@@ -21,12 +22,16 @@ export interface ViewSectionFilter {
     promptcuts?: boolean;
     cli?: boolean;
 }
+/** Trim a description to a column-friendly snippet. Strips newlines, collapses whitespace. */
+export declare function summarizeDescription(desc: string | undefined, maxLen?: number): string;
+export declare function descriptionForPrefix(desc: string | undefined, prefix: string): string;
 /** Machine-readable entry for a single installed version. */
 export interface ViewJsonVersion {
     version: string;
     isDefault: boolean;
     signedIn: boolean;
     email: string | null;
+    accountId?: string | null;
     plan: string | null;
     usageStatus: 'available' | 'rate_limited' | 'out_of_credits' | null;
     overageCredits?: {
@@ -40,6 +45,9 @@ export interface ViewJsonVersion {
     }>;
     lastActive: string | null;
     path: string;
+    /** Present only when --resources / --detailed (or --json with a section
+     *  flag) is passed: this version's resource inventory with git sync-state. */
+    resources?: VersionResourcesJson;
 }
 /** Machine-readable entry for one agent's installed versions. */
 export interface ViewJsonAgent {
@@ -47,6 +55,37 @@ export interface ViewJsonAgent {
     versions: ViewJsonVersion[];
     profiles: ProfileSummary[];
 }
+/** Resource sections that --resources can include in --json output. */
+export type ResourceSection = 'commands' | 'skills' | 'mcp' | 'memory' | 'hooks' | 'workflows' | 'plugins';
+/** One resource entry in `--json --resources` output. */
+export interface ResourceItemJson {
+    name: string;
+    scope?: 'user' | 'project';
+    /** Git sync-state vs ~/.agents. Omitted for project-scoped resources or when
+     *  ~/.agents isn't a git repo. When queried via --host it reflects the
+     *  remote's ~/.agents — i.e. per-host resource drift. */
+    syncState?: SyncState;
+    description?: string;
+    /** Skills only, when > 0. */
+    ruleCount?: number;
+}
+/** Per-version resource inventory; one key per requested section. */
+export interface VersionResourcesJson {
+    commands?: ResourceItemJson[];
+    skills?: ResourceItemJson[];
+    mcp?: ResourceItemJson[];
+    memory?: ResourceItemJson[];
+    hooks?: ResourceItemJson[];
+    workflows?: ResourceItemJson[];
+    plugins?: ResourceItemJson[];
+}
+/** Build the set of resource sections to include in `--json` from the
+ *  `--resources` / `--detailed` flags, plus (in --json mode) the per-section
+ *  boolean filters (`--skills` etc.) that plain `--json` historically ignored. */
+export declare function parseResourceSections(options: {
+    resources?: string | boolean;
+    detailed?: boolean;
+} & ViewSectionFilter, jsonMode: boolean): Set<ResourceSection>;
 /**
  * Prune older installed versions that share an email with a newer installed
  * version. Keeps the highest semver per email, skips the global default (with
@@ -66,6 +105,9 @@ export declare function viewAction(agentArg?: string, options?: {
     prune?: boolean;
     yes?: boolean;
     dryRun?: boolean;
+    resources?: string | boolean;
+    detailed?: boolean;
 } & ViewSectionFilter): Promise<void>;
 /** Register the `agents view` command. */
 export declare function registerViewCommand(program: Command): void;
+export {};