Publish agent (Part 2) (#226)

Author: jahooma

common/src/types/publisher.ts

@@ -18,3 +18,43 @@ export const PublisherSchema = z.object({
 })
 
 export type PublisherInput = z.infer<typeof PublisherSchema>
+
+export interface Publisher {
+  id: string
+  user_id: string
+  name: string
+  email: string | null
+  verified: boolean
+  bio: string | null
+  avatar_url: string | null
+  created_at: Date
+  updated_at: Date
+}
+
+export interface CreatePublisherRequest {
+  id: string
+  name: string
+  email?: string
+  bio?: string
+  avatar_url?: string
+}
+
+export interface UpdatePublisherRequest {
+  name?: string
+  email?: string
+  bio?: string
+  avatar_url?: string
+}
+
+export interface PublisherProfileResponse {
+  id: string
+  user_id: string
+  name: string
+  email: string | null
+  verified: boolean
+  bio: string | null
+  avatar_url: string | null
+  created_at: Date
+  updated_at: Date
+  agentCount?: number
+}

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

@@ -1,10 +1,9 @@
 import { DynamicAgentTemplate } from '@codebuff/common/types/dynamic-agent-template'
 import * as fs from 'fs'
-import { green, red, yellow, cyan } from 'picocolors'
-import { websiteUrl } from '../config'
-import { logger } from '../utils/logger'
-import { loadLocalAgents } from '../agents/load-agents'
+import { cyan, green, red, yellow } from 'picocolors'
 import { getAgentsDirectory } from '../agents/agent-utils'
+import { loadLocalAgents } from '../agents/load-agents'
+import { websiteUrl } from '../config'
 import { getUserCredentials } from '../credentials'
 
 interface PublishResponse {
@@ -13,6 +12,12 @@ interface PublishResponse {
   version?: string
   message?: string
   error?: string
+  details?: string
+  validationErrors?: Array<{
+    code: string
+    message: string
+    path: (string | number)[]
+  }>
 }
 
 /**
@@ -106,34 +111,25 @@ interface PublishResponse {
           `❌ Error publishing ${template.displayName}: ${error instanceof Error ? error.message : String(error)}`
         )
       )
-      logger.error(
-        {
-          error: error instanceof Error ? error.message : String(error),
-          agentId: template.id,
-        },
-        'Error publishing agent template'
-      )
+      // Avoid logger.error here as it can cause sonic boom errors that mask the real error
+      // The error is already displayed to the user via console.log above
     }
   } catch (error) {
     console.log(
       red(
-        `Error during publish: ${error instanceof Error ? error.message : String(error)}`
+        `Error during publish: ${error instanceof Error ? error.message + '\n' + error.stack : String(error)}`
       )
     )
-    logger.error(
-      {
-        error: error instanceof Error ? error.message : String(error),
-      },
-      'Error during publish command'
-    )
+    // Avoid logger.error here as it can cause sonic boom errors that mask the real error
+    // The error is already displayed to the user via console.log above
   }
 }
 
 /**
  * Publish an agent template to the backend
  */
 async function publishAgentTemplate(
-  template: DynamicAgentTemplate,
+  data: DynamicAgentTemplate,
   authToken: string
 ): Promise<PublishResponse> {
   try {
@@ -144,17 +140,47 @@ async function publishAgentTemplate(
         Cookie: `next-auth.session-token=${authToken}`,
       },
       body: JSON.stringify({
-        template,
+        data,
       }),
     })
 
-    const result = await response.json()
+    let result: any
+    try {
+      result = await response.json()
+    } catch (jsonError) {
+      return {
+        success: false,
+        error: `Failed to parse server response: ${response.status} ${response.statusText}`,
+      }
+    }
 
     if (!response.ok) {
+      // Extract detailed error information from the response
+      let errorMessage =
+        result.error || `HTTP ${response.status}: ${response.statusText}`
+
+      // If there are validation details, include them
+      if (result.details) {
+        errorMessage += `\n\nDetails: ${result.details}`
+      }
+
+      // If there are specific validation errors, format them nicely
+      if (result.validationErrors && Array.isArray(result.validationErrors)) {
+        const formattedErrors = result.validationErrors
+          .map((err: any) => {
+            const path =
+              err.path && err.path.length > 0 ? `${err.path.join('.')}: ` : ''
+            return `  • ${path}${err.message}`
+          })
+          .join('\n')
+        errorMessage += `\n\nValidation errors:\n${formattedErrors}`
+      }
+
       return {
         success: false,
-        error:
-          result.error || `HTTP ${response.status}: ${response.statusText}`,
+        error: errorMessage,
+        details: result.details,
+        validationErrors: result.validationErrors,
       }
     }
 
@@ -165,9 +191,17 @@ async function publishAgentTemplate(
       message: result.message,
     }
   } catch (error) {
+    // Handle network errors, timeouts, etc.
+    if (error instanceof TypeError && error.message.includes('fetch')) {
+      return {
+        success: false,
+        error: `Network error: Unable to connect to ${websiteUrl}. Please check your internet connection and try again.`,
+      }
+    }
+
     return {
       success: false,
-      error: error instanceof Error ? error.message : String(error),
+      error: `Unexpected error: ${error instanceof Error ? error.message : String(error)}`,
     }
   }
 }

npm-app/src/index.ts

@@ -10,24 +10,23 @@ import { AnalyticsEvent } from '@codebuff/common/constants/analytics-events'
 import { displayLoadedAgents, loadLocalAgents } from './agents/load-agents'
 import { CLI } from './cli'
 import { cliArguments, cliOptions } from './cli-definitions'
+import { handlePublish } from './cli-handlers/publish'
 import { npmAppVersion } from './config'
 import { createTemplateProject } from './create-template-project'
-import { handlePublish } from './cli-handlers/publish'
 import { printModeLog, setPrintMode } from './display/print-mode'
+import { loadCodebuffConfig } from './json-config/parser'
 import {
-  getStartingDirectory,
+  getProjectRoot,
+  getWorkingDirectory,
+  initializeProjectRootAndWorkingDir,
   initProjectFileContextWithWorker,
-  setProjectRoot,
-  setWorkingDirectory,
 } from './project-files'
 import { rageDetectors } from './rage-detectors'
 import { logAndHandleStartup } from './startup-process-handler'
 import { recreateShell } from './terminal/run-command'
 import { CliOptions } from './types'
 import { initAnalytics, trackEvent } from './utils/analytics'
-import { findGitRoot } from './utils/git'
 import { logger } from './utils/logger'
-import { loadCodebuffConfig } from './json-config/parser'
 
 async function codebuff({
   initialInput,
@@ -42,15 +41,8 @@ async function codebuff({
   trace,
 }: CliOptions) {
   enableSquashNewlines()
-
-  // Initialize starting directory
-  const { cwd: workingDir, shouldSearch } = getStartingDirectory(cwd)
-  const gitRoot = shouldSearch
-    ? findGitRoot(workingDir) ?? workingDir
-    : workingDir
-  const projectRoot = setProjectRoot(gitRoot)
-  setWorkingDirectory(workingDir)
-
+  const workingDir = getWorkingDirectory()
+  const projectRoot = getProjectRoot()
   await recreateShell(workingDir)
 
   // Kill all processes we failed to kill before
@@ -130,6 +122,10 @@ For all commands and options, run 'codebuff' and then type 'help'.
 
   const options = program.opts()
   const args = program.args // Handle template creation
+
+  // Initialize project root and working directory
+  initializeProjectRootAndWorkingDir(options.cwd)
+
   if (options.create) {
     const template = options.create
     const projectDir = args[0] || '.'

npm-app/src/project-files.ts

@@ -29,11 +29,11 @@ import {
   codebuffConfigFile,
   codebuffConfigFileBackup,
 } from '@codebuff/common/json-config/constants'
-import { loadCodebuffConfig } from './json-config/parser'
 import { loadedAgents, loadLocalAgents } from './agents/load-agents'
 import { checkpointManager } from './checkpoints/checkpoint-manager'
 import { CONFIG_DIR } from './credentials'
-import { gitCommandIsAvailable } from './utils/git'
+import { loadCodebuffConfig } from './json-config/parser'
+import { findGitRoot, gitCommandIsAvailable } from './utils/git'
 import { logger } from './utils/logger'
 import { getSystemInfo } from './utils/system-info'
 import { getScrapedContentBlocks, parseUrlsFromContent } from './web-scraper'
@@ -156,6 +156,24 @@ export function getStartingDirectory(dir: string | undefined = undefined): {
   return { cwd: dirAbsolute, shouldSearch: false }
 }
 
+/**
+ * Initialize project root for standalone commands that don't go through normal CLI setup
+ * @param cwd Optional working directory override
+ * @returns Object with projectRoot and workingDir paths
+ */
+export function initializeProjectRootAndWorkingDir(cwd?: string): {
+  projectRoot: string
+  workingDir: string
+} {
+  const { cwd: workingDir, shouldSearch } = getStartingDirectory(cwd)
+  const gitRoot = shouldSearch
+    ? findGitRoot(workingDir) ?? workingDir
+    : workingDir
+  const projectRoot = setProjectRoot(gitRoot)
+  setWorkingDirectory(workingDir)
+  return { projectRoot, workingDir }
+}
+
 /**
  * Transforms a relative filepath into an absolute one, using the project root as the base.
  * Handles '..' and '.' in paths correctly. Also handles Windows paths.

web/.gitignore

@@ -48,3 +48,6 @@ sitemap-*.xml
 /playwright-report/
 /blob-report/
 /playwright/.cache/
+
+# uploads
+/public/uploads/