Files
supabase/apps/studio/lib/ai/tools/report-tools.test.ts
Gildas GarciaandAlaister Young 737b8595f2 Update API types (#50234)
## Problem

platform, v1 and v2 have been already completely migrated and introduced
some changes.

Some types have been renamed, some outputs and inputs updated.

## Solution

- Update the API types
- Fix the TS errors

## Update

Taking this over to unblock #50134, which needs the new scoped token
permission ids from the regenerated types.

- Merged `master`.
- Regenerated `api-v2.d.ts` from the production spec. The previous files
came from a local API that exposed a webhook events endpoint production
doesn't have yet. Production has since added standardized 400 error
responses on the v2 organization endpoints. `api-v1.d.ts` and
`platform.d.ts` already matched production.
- Fixed `verify-production-types`. It formatted the regenerated files in
a temp directory outside the repository, so Prettier fell back to its
defaults and the comparison could never match the committed files. It
now passes the repository config explicitly. `pnpm api:verify-types`
passes on this branch.
- Verified locally: `pnpm typecheck`, `pnpm api:verify-types`, Studio
unit tests.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Bug Fixes**
* Preserved descriptions when saving, sharing, moving, or unsharing
notebooks, reports, SQL snippets, and saved queries.
* Improved handling of empty or null values across notebook
descriptions, billing usage, pooler settings, and infrastructure fields.
* Improved read-replica connection handling, including read-only
connection strings.
* Updated storage configuration and capability handling to match current
settings.

* **API and Compatibility**
* Updated organization, project, storage, OAuth, billing, and
infrastructure data handling to match current API responses.
  * OAuth app creation and updates now require scopes.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Alaister Young <10985857+alaister@users.noreply.github.com>
2026-09-11 12:17:49 +08:00

219 lines
7.2 KiB
TypeScript

import { components } from 'api-types'
import { HttpResponse } from 'msw'
import { describe, expect, it } from 'vitest'
import { getReportTools } from './report-tools'
import { addAPIMock, type APIErrorBody } from '@/tests/lib/msw'
type GetUserContentByIdResponse = components['schemas']['GetUserContentByIdResponse_Output']
describe('ai/tools/report-tools', () => {
describe('getReportTools', () => {
it('should return list_reports and get_report tools', () => {
const tools = getReportTools()
expect(Object.keys(tools)).toEqual(['list_reports', 'get_report'])
})
it('should not require approval to read reports', () => {
const tools = getReportTools()
expect(tools.list_reports.needsApproval).toBeUndefined()
expect(tools.get_report.needsApproval).toBeUndefined()
})
})
describe('list_reports', () => {
it('should list reports with summary fields, forwarding the authorization header', async () => {
let capturedRequest: Request | undefined
addAPIMock({
method: 'get',
path: '/platform/projects/:ref/content',
response: ({ request }) => {
capturedRequest = request
return HttpResponse.json({
data: [
{
id: 'report-1',
name: 'Home',
description: undefined,
visibility: 'project',
favorite: false,
folder_id: null,
inserted_at: '2026-01-01T00:00:00.000Z',
updated_at: '2026-01-01T00:00:00.000Z',
owner_id: 1,
owner: { id: 1, username: 'test' },
updated_by: { id: 1, username: 'test' },
project_id: 1,
type: 'report',
content: {
schema_version: 1,
period_start: { time_period: '7d' },
period_end: { time_period: 'today' },
interval: '1d',
layout: [{ id: 'a' }, { id: 'b' }],
},
},
],
})
},
})
const tools = getReportTools({ projectRef: 'test-project', authorization: 'Bearer token' })
if (!tools.list_reports.execute) throw new Error('execute is undefined')
const result = await tools.list_reports.execute(
{ limit: 20 },
{ toolCallId: 'test', messages: [], context: {} }
)
expect(capturedRequest?.headers.get('authorization')).toBe('Bearer token')
const url = new URL(capturedRequest!.url)
expect(url.pathname).toContain('/projects/test-project/content')
expect(url.searchParams.get('type')).toBe('report')
expect(url.searchParams.get('limit')).toBe('20')
expect(result).toEqual([
{
id: 'report-1',
name: 'Home',
description: undefined,
visibility: 'project',
updated_at: '2026-01-01T00:00:00.000Z',
chart_count: 2,
},
])
})
})
describe('get_report', () => {
it('should resolve snippet_ chart blocks to their SQL', async () => {
addAPIMock({
method: 'get',
path: '/platform/projects/:ref/content/item/:id',
response: ({ params }) => {
if (params.id === 'report-1') {
return HttpResponse.json<GetUserContentByIdResponse>({
id: 'report-1',
name: 'My report',
description: undefined,
visibility: 'project',
favorite: false,
folder_id: null,
inserted_at: '2026-01-01T00:00:00.000Z',
updated_at: '2026-01-01T00:00:00.000Z',
owner_id: 1,
project_id: 1,
type: 'report',
content: {
schema_version: 1,
period_start: { time_period: '7d' },
period_end: { time_period: 'today' },
interval: '1d',
layout: [
{
id: 'snippet-1',
attribute: 'snippet_snippet-1',
x: 0,
y: 0,
w: 1,
h: 1,
label: 'My query',
provider: 'daily-stats',
chart_type: 'bar',
},
{
id: 'total_egress',
attribute: 'total_egress',
x: 1,
y: 0,
w: 1,
h: 1,
label: 'Egress',
provider: 'daily-stats',
chart_type: 'bar',
},
],
},
})
}
if (params.id === 'snippet-1') {
return HttpResponse.json<GetUserContentByIdResponse>({
id: 'snippet-1',
name: 'My query',
description: undefined,
visibility: 'user',
favorite: false,
folder_id: null,
inserted_at: '2026-01-01T00:00:00.000Z',
updated_at: '2026-01-01T00:00:00.000Z',
owner_id: 1,
project_id: 1,
type: 'sql',
content: { content_id: 'snippet-1', sql: 'select 1', schema_version: '1' },
})
}
return HttpResponse.json<APIErrorBody>(
{ message: `Unexpected id: ${params.id}` },
{ status: 404 }
)
},
})
const tools = getReportTools({ projectRef: 'test-project' })
if (!tools.get_report.execute) throw new Error('execute is undefined')
const result = (await tools.get_report.execute(
{ id: 'report-1' },
{ toolCallId: 'test', messages: [], context: {} }
)) as { layout: Array<{ id: string; attribute: string; sql?: string }> }
expect(result.layout).toEqual([
expect.objectContaining({
id: 'snippet-1',
attribute: 'snippet_snippet-1',
sql: 'select 1',
}),
expect.objectContaining({ id: 'total_egress', attribute: 'total_egress' }),
])
expect(result.layout[1].sql).toBeUndefined()
})
it('should throw when the content id is not a report', async () => {
addAPIMock({
method: 'get',
path: '/platform/projects/:ref/content/item/:id',
response: () =>
HttpResponse.json<GetUserContentByIdResponse>({
id: 'snippet-1',
name: 'My query',
description: undefined,
visibility: 'user',
favorite: false,
folder_id: null,
inserted_at: '2026-01-01T00:00:00.000Z',
updated_at: '2026-01-01T00:00:00.000Z',
owner_id: 1,
project_id: 1,
type: 'sql',
content: { content_id: 'snippet-1', sql: 'select 1', schema_version: '1' },
}),
})
const tools = getReportTools({ projectRef: 'test-project' })
if (!tools.get_report.execute) throw new Error('execute is undefined')
await expect(
tools.get_report.execute(
{ id: 'snippet-1' },
{ toolCallId: 'test', messages: [], context: {} }
)
).rejects.toThrow('is not a report')
})
})
})