Load credentials directly for publish command

Author: jahooma

npm-app/src/cli-handlers/publish.ts

@@ -1,11 +1,11 @@
 import { DynamicAgentTemplate } from '@codebuff/common/types/dynamic-agent-template'
 import * as fs from 'fs'
 import { green, red, yellow, cyan } from 'picocolors'
-import { Client } from '../client'
 import { websiteUrl } from '../config'
 import { logger } from '../utils/logger'
 import { loadLocalAgents } from '../agents/load-agents'
 import { getAgentsDirectory } from '../agents/agent-utils'
+import { getUserCredentials } from '../credentials'
 
 interface PublishResponse {
   success: boolean
@@ -17,18 +17,17 @@ interface PublishResponse {
 
 /**
  * Handle the publish command to upload agent templates to the backend
- * @param agentName The name of the agent to publish (required)
- */
-export async function handlePublish(agentName?: string): Promise<void> {
-  const client = Client.getInstance()
+ * @param agentId The id of the agent to publish (required)
+ */ export async function handlePublish(agentId?: string): Promise<void> {
+  const user = getUserCredentials()
 
-  if (!client.user) {
+  if (!user) {
     console.log(red('Please log in first using "login".'))
     return
   }
 
-  if (!agentName) {
-    console.log(red('Agent name is required. Usage: publish <agent-name>'))
+  if (!agentId) {
+    console.log(red('Agent id is required. Usage: publish <agent-id>'))
     console.log(
       yellow('This prevents accidentally publishing all agents at once.')
     )
@@ -69,13 +68,13 @@ export async function handlePublish(agentName?: string): Promise<void> {
     // Find the specific agent
     const matchingTemplate = Object.entries(agentTemplates).find(
       ([key, template]) =>
-        key === agentName ||
-        template.id === agentName ||
-        template.displayName === agentName
+        key === agentId ||
+        template.id === agentId ||
+        template.displayName === agentId
     )
 
     if (!matchingTemplate) {
-      console.log(red(`Agent "${agentName}" not found. Available agents:`))
+      console.log(red(`Agent "${agentId}" not found. Available agents:`))
       Object.values(agentTemplates).forEach((template) => {
         console.log(`  - ${template.displayName} (${template.id})`)
       })
@@ -87,10 +86,7 @@ export async function handlePublish(agentName?: string): Promise<void> {
     )
 
     try {
-      const result = await publishAgentTemplate(
-        template,
-        client.user.authToken!
-      )
+      const result = await publishAgentTemplate(template, user.authToken!)
 
       if (result.success) {
         console.log(

npm-app/src/credentials.ts

@@ -3,6 +3,7 @@ import path from 'node:path'
 import os from 'os'
 import { z } from 'zod'
 import { logger } from './utils/logger'
+import { existsSync, readFileSync } from 'fs'
 
 const credentialsSchema = z
   .object({
@@ -48,3 +49,28 @@ export const CONFIG_DIR = path.join(
 // Ensure config directory exists
 ensureDirectoryExists(CONFIG_DIR)
 export const CREDENTIALS_PATH = path.join(CONFIG_DIR, 'credentials.json')
+
+/**
+ * Get user credentials from file system
+ * @returns User object or null if not found/authenticated
+ */
+export const getUserCredentials = (): User | null => {
+  // Read user credentials directly from file
+  if (!existsSync(CREDENTIALS_PATH)) {
+    return null
+  }
+
+  try {
+    const credentialsFile = readFileSync(CREDENTIALS_PATH, 'utf8')
+    const user = userFromJson(credentialsFile)
+    return user || null
+  } catch (error) {
+    logger.error(
+      {
+        error: error instanceof Error ? error.message : String(error),
+      },
+      'Error reading credentials'
+    )
+    return null
+  }
+}