Skip to content

Commit 2acc72e

Browse files
committed
improvement(persona): validate report template ID before request
1 parent b082119 commit 2acc72e

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

apps/sim/tools/persona/create_report.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,11 @@ export const personaCreateReportTool: ToolConfig<PersonaCreateReportParams, Pers
9696
throw new Error(`Report type must be one of: ${SUPPORTED_REPORT_TYPES.join(', ')}`)
9797
}
9898

99+
const reportTemplateId = params.reportTemplateId?.trim()
100+
if (!reportTemplateId) {
101+
throw new Error('Report template ID is required (starts with rptp_)')
102+
}
103+
99104
const query: Record<string, unknown> = {}
100105
if (params.term?.trim()) query.term = params.term.trim()
101106
if (params.nameFirst?.trim()) query['name-first'] = params.nameFirst.trim()
@@ -114,7 +119,7 @@ export const personaCreateReportTool: ToolConfig<PersonaCreateReportParams, Pers
114119
}
115120

116121
const attributes: Record<string, unknown> = {
117-
'report-template-id': params.reportTemplateId?.trim(),
122+
'report-template-id': reportTemplateId,
118123
query,
119124
}
120125
if (params.accountId?.trim()) {

0 commit comments

Comments
 (0)