feat: enrich agent config with workspace identity and tools
This commit is contained in:
parent
e4594c7854
commit
6cd5e3e534
|
|
@ -1,7 +1,7 @@
|
||||||
import { NextRequest, NextResponse } from 'next/server'
|
import { NextRequest, NextResponse } from 'next/server'
|
||||||
import { getDatabase, db_helpers, logAuditEvent } from '@/lib/db'
|
import { getDatabase, db_helpers, logAuditEvent } from '@/lib/db'
|
||||||
import { getUserFromRequest, requireRole } from '@/lib/auth'
|
import { getUserFromRequest, requireRole } from '@/lib/auth'
|
||||||
import { writeAgentToConfig } from '@/lib/agent-sync'
|
import { writeAgentToConfig, enrichAgentConfigFromWorkspace } from '@/lib/agent-sync'
|
||||||
import { eventBus } from '@/lib/event-bus'
|
import { eventBus } from '@/lib/event-bus'
|
||||||
import { logger } from '@/lib/logger'
|
import { logger } from '@/lib/logger'
|
||||||
|
|
||||||
|
|
@ -32,7 +32,7 @@ export async function GET(
|
||||||
|
|
||||||
const parsed = {
|
const parsed = {
|
||||||
...(agent as any),
|
...(agent as any),
|
||||||
config: (agent as any).config ? JSON.parse((agent as any).config) : {},
|
config: enrichAgentConfigFromWorkspace((agent as any).config ? JSON.parse((agent as any).config) : {}),
|
||||||
}
|
}
|
||||||
|
|
||||||
return NextResponse.json({ agent: parsed })
|
return NextResponse.json({ agent: parsed })
|
||||||
|
|
@ -154,9 +154,11 @@ export async function PUT(
|
||||||
updated_at: now,
|
updated_at: now,
|
||||||
})
|
})
|
||||||
|
|
||||||
|
const enrichedConfig = enrichAgentConfigFromWorkspace(newConfig)
|
||||||
|
|
||||||
return NextResponse.json({
|
return NextResponse.json({
|
||||||
success: true,
|
success: true,
|
||||||
agent: { ...agent, config: newConfig, role: role || agent.role, updated_at: now },
|
agent: { ...agent, config: enrichedConfig, role: role || agent.role, updated_at: now },
|
||||||
})
|
})
|
||||||
} catch (error: any) {
|
} catch (error: any) {
|
||||||
logger.error({ err: error }, 'PUT /api/agents/[id] error')
|
logger.error({ err: error }, 'PUT /api/agents/[id] error')
|
||||||
|
|
|
||||||
|
|
@ -2,7 +2,7 @@ import { NextRequest, NextResponse } from 'next/server';
|
||||||
import { getDatabase, Agent, db_helpers } from '@/lib/db';
|
import { getDatabase, Agent, db_helpers } from '@/lib/db';
|
||||||
import { eventBus } from '@/lib/event-bus';
|
import { eventBus } from '@/lib/event-bus';
|
||||||
import { getTemplate, buildAgentConfig } from '@/lib/agent-templates';
|
import { getTemplate, buildAgentConfig } from '@/lib/agent-templates';
|
||||||
import { writeAgentToConfig } from '@/lib/agent-sync';
|
import { writeAgentToConfig, enrichAgentConfigFromWorkspace } from '@/lib/agent-sync';
|
||||||
import { logAuditEvent } from '@/lib/db';
|
import { logAuditEvent } from '@/lib/db';
|
||||||
import { getUserFromRequest, requireRole } from '@/lib/auth';
|
import { getUserFromRequest, requireRole } from '@/lib/auth';
|
||||||
import { mutationLimiter } from '@/lib/rate-limit';
|
import { mutationLimiter } from '@/lib/rate-limit';
|
||||||
|
|
@ -50,7 +50,7 @@ export async function GET(request: NextRequest) {
|
||||||
// Parse JSON config field
|
// Parse JSON config field
|
||||||
const agentsWithParsedData = agents.map(agent => ({
|
const agentsWithParsedData = agents.map(agent => ({
|
||||||
...agent,
|
...agent,
|
||||||
config: agent.config ? JSON.parse(agent.config) : {}
|
config: enrichAgentConfigFromWorkspace(agent.config ? JSON.parse(agent.config) : {})
|
||||||
}));
|
}));
|
||||||
|
|
||||||
// Get task counts for each agent (prepare once, reuse per agent)
|
// Get task counts for each agent (prepare once, reuse per agent)
|
||||||
|
|
@ -357,4 +357,4 @@ export async function PUT(request: NextRequest) {
|
||||||
logger.error({ err: error }, 'PUT /api/agents error');
|
logger.error({ err: error }, 'PUT /api/agents error');
|
||||||
return NextResponse.json({ error: 'Failed to update agent' }, { status: 500 });
|
return NextResponse.json({ error: 'Failed to update agent' }, { status: 500 });
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -62,6 +62,70 @@ export interface SyncDiff {
|
||||||
onlyInMC: string[]
|
onlyInMC: string[]
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function parseIdentityFromFile(content: string): { name?: string; theme?: string; emoji?: string; content?: string } {
|
||||||
|
if (!content.trim()) return {}
|
||||||
|
const lines = content.split('\n').map((line) => line.trim()).filter(Boolean)
|
||||||
|
let name: string | undefined
|
||||||
|
let theme: string | undefined
|
||||||
|
let emoji: string | undefined
|
||||||
|
|
||||||
|
for (const line of lines) {
|
||||||
|
if (!name && line.startsWith('#')) {
|
||||||
|
name = line.replace(/^#+\s*/, '').trim()
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!theme) {
|
||||||
|
const themeMatch = line.match(/^theme\s*:\s*(.+)$/i)
|
||||||
|
if (themeMatch?.[1]) {
|
||||||
|
theme = themeMatch[1].trim()
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!emoji) {
|
||||||
|
const emojiMatch = line.match(/^emoji\s*:\s*(.+)$/i)
|
||||||
|
if (emojiMatch?.[1]) {
|
||||||
|
emoji = emojiMatch[1].trim()
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return {
|
||||||
|
...(name ? { name } : {}),
|
||||||
|
...(theme ? { theme } : {}),
|
||||||
|
...(emoji ? { emoji } : {}),
|
||||||
|
content: lines.slice(0, 8).join('\n'),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
function parseToolsFromFile(content: string): { allow?: string[]; raw?: string } {
|
||||||
|
if (!content.trim()) return {}
|
||||||
|
|
||||||
|
const parsedTools = new Set<string>()
|
||||||
|
for (const line of content.split('\n')) {
|
||||||
|
const cleaned = line.trim()
|
||||||
|
if (!cleaned || cleaned.startsWith('#')) continue
|
||||||
|
|
||||||
|
const listMatch = cleaned.match(/^[-*]\s+`?([^`]+?)`?\s*$/)
|
||||||
|
if (listMatch?.[1]) {
|
||||||
|
parsedTools.add(listMatch[1].trim())
|
||||||
|
continue
|
||||||
|
}
|
||||||
|
|
||||||
|
const inlineMatch = cleaned.match(/^`([^`]+)`$/)
|
||||||
|
if (inlineMatch?.[1]) {
|
||||||
|
parsedTools.add(inlineMatch[1].trim())
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const allow = [...parsedTools].filter(Boolean)
|
||||||
|
return {
|
||||||
|
...(allow.length > 0 ? { allow } : {}),
|
||||||
|
raw: content.split('\n').map((line) => line.trim()).filter(Boolean).slice(0, 24).join('\n'),
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
function getConfigPath(): string | null {
|
function getConfigPath(): string | null {
|
||||||
if (!config.openclawHome) return null
|
if (!config.openclawHome) return null
|
||||||
return join(config.openclawHome, 'openclaw.json')
|
return join(config.openclawHome, 'openclaw.json')
|
||||||
|
|
@ -82,6 +146,30 @@ function readWorkspaceFile(workspace: string | undefined, filename: string): str
|
||||||
return null
|
return null
|
||||||
}
|
}
|
||||||
|
|
||||||
|
export function enrichAgentConfigFromWorkspace(configData: any): any {
|
||||||
|
if (!configData || typeof configData !== 'object') return configData
|
||||||
|
const workspace = typeof configData.workspace === 'string' ? configData.workspace : undefined
|
||||||
|
if (!workspace) return configData
|
||||||
|
|
||||||
|
const identityFile = readWorkspaceFile(workspace, 'identity.md')
|
||||||
|
const toolsFile = readWorkspaceFile(workspace, 'TOOLS.md')
|
||||||
|
|
||||||
|
const mergedIdentity = {
|
||||||
|
...parseIdentityFromFile(identityFile || ''),
|
||||||
|
...((configData.identity && typeof configData.identity === 'object') ? configData.identity : {}),
|
||||||
|
}
|
||||||
|
const mergedTools = {
|
||||||
|
...parseToolsFromFile(toolsFile || ''),
|
||||||
|
...((configData.tools && typeof configData.tools === 'object') ? configData.tools : {}),
|
||||||
|
}
|
||||||
|
|
||||||
|
return {
|
||||||
|
...configData,
|
||||||
|
identity: Object.keys(mergedIdentity).length > 0 ? mergedIdentity : configData.identity,
|
||||||
|
tools: Object.keys(mergedTools).length > 0 ? mergedTools : configData.tools,
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/** Read and parse openclaw.json agents list */
|
/** Read and parse openclaw.json agents list */
|
||||||
async function readOpenClawAgents(): Promise<OpenClawAgent[]> {
|
async function readOpenClawAgents(): Promise<OpenClawAgent[]> {
|
||||||
const configPath = getConfigPath()
|
const configPath = getConfigPath()
|
||||||
|
|
@ -102,9 +190,8 @@ function mapAgentToMC(agent: OpenClawAgent): {
|
||||||
} {
|
} {
|
||||||
const name = agent.identity?.name || agent.name || agent.id
|
const name = agent.identity?.name || agent.name || agent.id
|
||||||
const role = agent.identity?.theme || 'agent'
|
const role = agent.identity?.theme || 'agent'
|
||||||
|
|
||||||
// Store the full config minus systemPrompt/soul (which can be large)
|
// Store the full config minus systemPrompt/soul (which can be large)
|
||||||
const configData = {
|
const configData = enrichAgentConfigFromWorkspace({
|
||||||
openclawId: agent.id,
|
openclawId: agent.id,
|
||||||
model: agent.model,
|
model: agent.model,
|
||||||
identity: agent.identity,
|
identity: agent.identity,
|
||||||
|
|
@ -115,7 +202,7 @@ function mapAgentToMC(agent: OpenClawAgent): {
|
||||||
workspace: agent.workspace,
|
workspace: agent.workspace,
|
||||||
agentDir: agent.agentDir,
|
agentDir: agent.agentDir,
|
||||||
isDefault: agent.default || false,
|
isDefault: agent.default || false,
|
||||||
}
|
})
|
||||||
|
|
||||||
// Read soul.md from the agent's workspace if available
|
// Read soul.md from the agent's workspace if available
|
||||||
const soul_content = readWorkspaceFile(agent.workspace, 'soul.md')
|
const soul_content = readWorkspaceFile(agent.workspace, 'soul.md')
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue