publishing agents requires "publisher" field

Author: charleslien

.agents/types/agent-config.d.ts

@@ -10,7 +10,7 @@
  *   const config: AgentConfig = {
  *     // ... your agent configuration with full type safety ...
  *   }
- *   
+ *
  *   export default config
  */
 
@@ -25,6 +25,9 @@ export interface AgentConfig {
   /** Version string (if not provided, will default to '0.0.1' and be bumped on each publish) */
   version?: string
 
+  /** Publisher ID for the agent. Must be provided if you want to publish the agent. */
+  publisher?: string
+
   /** Human-readable name for the agent */
   displayName: string
 

common/src/types/__tests__/dynamic-agent-template.test.ts

@@ -0,0 +1,8 @@
+import type { AgentConfig } from '../../util/types/agent-config'
+import type { DynamicAgentConfig } from '../dynamic-agent-template'
+
+// Don't remove these lines!
+const _typecheck1: AgentConfig extends DynamicAgentConfig ? true : false = true
+const _typecheck2: DynamicAgentConfig extends AgentConfig ? true : false = true
+const _keyTypecheck1: keyof AgentConfig = {} as keyof DynamicAgentConfig
+const _keyTypecheck2: keyof DynamicAgentConfig = {} as keyof AgentConfig

common/src/types/api/agents/publish.ts

@@ -4,7 +4,6 @@ import { DynamicAgentTemplateSchema } from '../../../types/dynamic-agent-templat
 
 export const publishAgentsRequestSchema = z.object({
   data: DynamicAgentTemplateSchema.array(),
-  publisherId: z.string().optional(),
   authToken: z.string(),
 })
 export type PublishAgentsRequest = z.infer<typeof publishAgentsRequestSchema>

common/src/types/dynamic-agent-template.ts

@@ -3,8 +3,6 @@ import { z } from 'zod'
 import { ALLOWED_MODEL_PREFIXES, models } from '../constants'
 import { toolNames } from '../tools/constants'
 
-import type { AgentConfig } from '../util/types/agent-config'
-
 // Filter models to only include those that begin with allowed prefixes
 const filteredModels = Object.values(models).filter((model) =>
   ALLOWED_MODEL_PREFIXES.some((prefix) => model.startsWith(prefix)),
@@ -91,6 +89,7 @@ export const DynamicAgentConfigSchema = z.object({
       'Agent ID must contain only lowercase letters, numbers, and hyphens',
     ), // The unique identifier for this agent
   version: z.string().optional(),
+  publisher: z.string().optional(),
 
   // Required fields for new agents
   displayName: z.string(),
@@ -136,20 +135,13 @@ export const DynamicAgentConfigSchema = z.object({
   systemPrompt: z.string().optional(),
   instructionsPrompt: z.string().optional(),
   stepPrompt: z.string().optional(),
-  // NOTE: Removed from AgentConfig. If there's a need, can be added back or else removed entirely.
-  parentInstructions: z.record(z.string(), z.string()).optional(),
 
   // Optional generator function for programmatic agents
   handleSteps: HandleStepsSchema,
 })
 export type DynamicAgentConfig = z.input<typeof DynamicAgentConfigSchema>
 export type DynamicAgentConfigParsed = z.infer<typeof DynamicAgentConfigSchema>
 
-// Checks that the AgentConfig and DynamicAgentConfig types are compatible
-// NOTE TO LLM: Don't remove these lines!
-const _typecheck: AgentConfig = {} as DynamicAgentConfig
-const _typecheck2: DynamicAgentConfig = {} as AgentConfig
-
 export const DynamicAgentTemplateSchema = DynamicAgentConfigSchema.extend({
   systemPrompt: z.string(),
   instructionsPrompt: z.string(),

common/src/util/types/agent-config.d.ts

@@ -10,7 +10,7 @@
  *   const config: AgentConfig = {
  *     // ... your agent configuration with full type safety ...
  *   }
- *   
+ *
  *   export default config
  */
 
@@ -25,6 +25,9 @@ export interface AgentConfig {
   /** Version string (if not provided, will default to '0.0.1' and be bumped on each publish) */
   version?: string
 
+  /** Publisher ID for the agent. Must be provided if you want to publish the agent. */
+  publisher?: string
+
   /** Human-readable name for the agent */
   displayName: string
 
@@ -57,18 +60,18 @@ export interface AgentConfig {
   }
 
   /** Whether to include conversation history from the parent agent in context.
-   * 
+   *
    * Defaults to false.
    * Use this if the agent needs to know all the previous messages in the conversation.
    */
   includeMessageHistory?: boolean
 
   /** How the agent should output a response to its parent (defaults to 'last_message')
-   * 
+   *
    * last_message: The last message from the agent, typcically after using tools.
-   * 
+   *
    * all_messages: All messages from the agent, including tool calls and results.
-   * 
+   *
    * json: Make the agent output a JSON object. Can be used with outputSchema or without if you want freeform json output.
    */
   outputMode?: 'last_message' | 'all_messages' | 'json'
@@ -81,21 +84,21 @@ export interface AgentConfig {
   // ============================================================================
 
   /** Prompt for when to spawn this agent as a subagent. Include the main purpose and use cases.
-   * 
+   *
    * This field is key if the agent is a subagent and intended to be spawned. */
   parentPrompt?: string
 
   /** Background information for the agent. Fairly optional. Prefer using instructionsPrompt for agent instructions. */
   systemPrompt?: string
 
   /** Instructions for the agent.
-   * 
+   *
    * IMPORTANT: Updating this prompt is the best way to shape the agent's behavior.
    * This prompt is inserted after each user input. */
   instructionsPrompt?: string
 
   /** Prompt inserted at each agent step.
-   * 
+   *
    * Powerful for changing the agent's behavior, but usually not necessary for smart models.
    * Prefer instructionsPrompt for most instructions. */
   stepPrompt?: string

npm-app/src/cli-definitions.ts

@@ -93,10 +93,4 @@ export const cliOptions: CliParam[] = [
     menuDescription: 'Log subagent messages to trace files',
     hidden: false,
   },
-  {
-    flags: '--publisher <publisher-id>',
-    description: 'Specify which publisher to use when publishing agents',
-    menuDescription: 'Specify publisher for agent publishing',
-    hidden: false,
-  },
 ]

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

@@ -16,11 +16,7 @@ import type { DynamicAgentTemplate } from '@codebuff/common/types/dynamic-agent-
 /**
  * Handle the publish command to upload agent templates to the backend
  * @param agentId The id of the agent to publish (required)
- * @param publisherId The id of the publisher to use (optional)
- */ export async function handlePublish(
-  agentIds: string[],
-  publisherId?: string,
-): Promise<void> {
+ */ export async function handlePublish(agentIds: string[]): Promise<void> {
   const user = getUserCredentials()
 
   if (!user) {
@@ -95,7 +91,6 @@ import type { DynamicAgentTemplate } from '@codebuff/common/types/dynamic-agent-
       const result = await publishAgentTemplates(
         Object.values(matchingTemplates),
         user.authToken!,
-        publisherId,
       )
 
       if (result.success) {
@@ -111,73 +106,27 @@ import type { DynamicAgentTemplate } from '@codebuff/common/types/dynamic-agent-
       }
 
       console.log(red(`❌ Failed to publish agents: ${result.error}`))
-      if (result.statusCode !== 403) {
-        return
-      }
 
-      // Check if this is a "no publisher" error vs "multiple publishers" error
-      if (result.error?.includes('No publisher associated with user')) {
-        console.log()
-        console.log(
-          cyan('Please visit the website to create your publisher profile:'),
-        )
-        console.log(yellow(`${websiteUrl}/publishers`))
+      // Show helpful guidance based on error type
+      if (result.error?.includes('Publisher field required')) {
         console.log()
-        console.log('A publisher profile allows you to:')
-        console.log('  • Publish and manage your agents')
-        console.log('  • Build your reputation in the community')
-        console.log('  • Organize agents under your name or organization')
+        console.log(cyan('Add a "publisher" field to your agent templates:'))
+        console.log(yellow('  "publisher": "<publisher-id>"'))
         console.log()
       } else if (
-        result.availablePublishers &&
-        result.availablePublishers.length > 0
+        result.error?.includes('Publisher not found or not accessible')
       ) {
-        // Show available publishers
+        console.log()
         console.log(
           cyan(
-            'You have access to multiple publishers. Please specify which one to use:',
+            'Check that the publisher ID is correct and you have access to it.',
           ),
         )
         console.log()
-        console.log(cyan('Available publishers:'))
-        result.availablePublishers.forEach((publisher) => {
-          const orgInfo = publisher.organizationName
-            ? ` (${publisher.organizationName})`
-            : ''
-          const typeInfo =
-            publisher.ownershipType === 'organization'
-              ? ' [Organization]'
-              : ' [Personal]'
-          console.log(
-            `  • ${yellow(publisher.id)} - ${publisher.name}${orgInfo}${typeInfo}`,
-          )
-        })
-        console.log()
-        console.log('Run one of these commands:')
-        result.availablePublishers.forEach((publisher) => {
-          console.log(
-            yellow(
-              `  codebuff publish ${agentIds.join(' ')} --publisher ${publisher.id}`,
-            ),
-          )
-        })
-        console.log()
-        console.log(cyan('Or visit the website to manage your publishers:'))
-        console.log(yellow(`${websiteUrl}/publishers`))
-        console.log()
-      } else {
-        // Generic 403 error
-        console.log(cyan('You may need to specify which publisher to use.'))
-        console.log()
-        console.log('Try running:')
-        console.log(
-          yellow(`  publish ${agentIds.join(' ')} --publisher <publisher-id>`),
-        )
-        console.log()
-        console.log(cyan('Visit the website to see your available publishers:'))
-        console.log(yellow(`${websiteUrl}/publishers`))
-        console.log()
       }
+
+      console.log(cyan('Visit the website to manage your publishers:'))
+      console.log(yellow(`${websiteUrl}/publishers`))
     } catch (error) {
       console.log(
         red(
@@ -204,7 +153,6 @@ import type { DynamicAgentTemplate } from '@codebuff/common/types/dynamic-agent-
 async function publishAgentTemplates(
   data: DynamicAgentTemplate[],
   authToken: string,
-  publisherId?: string,
 ): Promise<PublishAgentsResponse & { statusCode?: number }> {
   try {
     const response = await fetch(`${websiteUrl}/api/agents/publish`, {
@@ -215,7 +163,6 @@ async function publishAgentTemplates(
       body: JSON.stringify({
         data,
         authToken,
-        ...(publisherId && { publisherId }),
       }),
     })
 

npm-app/src/index.ts

@@ -137,8 +137,7 @@ For all commands and options, run 'codebuff' and then type 'help'.
   // Handle publish command
   if (args[0] === 'publish') {
     const agentNames = args.slice(1)
-    const publisherId = options.publisher
-    await handlePublish(agentNames, publisherId)
+    await handlePublish(agentNames)
     process.exit(0)
   }