senlinai/agent
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(web): consume api v1 contracts

Browse Source
This commit is contained in:
yanweidong
2026-07-21 16:29:42 +08:00
parent 80bec26839
commit c56ada3708
8 changed files with 311 additions and 149 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
apps/web_v1/scripts/structure-check.mjs
View File

@@ -21,6 +21,8 @@ const requiredFiles = [
'src/api/client.ts',
'src/api/projects.ts',
'src/api/mappers.tsx',
'src/api/search.ts',
'src/api/inbox.ts',
]
const failures = requiredFiles.filter((file) => !existsSync(file)).map((file) => `missing ${file}`)
@@ -47,6 +49,43 @@ if (!projectRailSource.includes('projectNameLabel(project.name)')) {
failures.push('project rail labels must derive from the project name')
}
const apiFiles = ['client.ts', 'projects.ts', 'mappers.tsx', 'search.ts', 'inbox.ts'].map((name) => `src/api/${name}`)
const apiSource = apiFiles.map((file) => readFileSync(file, 'utf8')).join('\n')
for (const forbidden of [
{ pattern: /\bID\s*\?\s*:/, label: 'optional PascalCase ID compatibility field' },
{ pattern: /\bName\s*\?\s*:/, label: 'optional PascalCase Name compatibility field' },
{ pattern: /['"`]\/api\/projects/, label: 'legacy /api/projects path' },
{
pattern: /\b(?:project|task|inbox)Id\s*\??\s*:\s*(?:string\s*\|\s*)?number\b/i,
label: 'numeric project/task/inbox identity',
},
]) {
if (forbidden.pattern.test(apiSource)) failures.push(`src/api contains ${forbidden.label}`)
}
const clientSource = readFileSync('src/api/client.ts', 'utf8')
if (!clientSource.includes("'/api/v1/auth/login'")) failures.push('login must use /api/v1/auth/login')
if (!clientSource.includes('export class ApiError extends Error')) failures.push('client must export ApiError')
const projectsSource = readFileSync('src/api/projects.ts', 'utf8')
if (!projectsSource.includes("'/api/v1/projects'")) failures.push('project API must use /api/v1/projects')
if (/['"`]\/api\/(?!v1\/)/.test(apiSource)) failures.push('src/api paths must use the /api/v1 prefix')
for (const match of apiSource.matchAll(/export type (\w+DTO)\s*=\s*\{([\s\S]*?)\n\}/g)) {
const [, name, body] = match
if (/^\s*\w+\s*\?\s*:/m.test(body)) failures.push(`${name} fields must be required`)
if (/^\s*[A-Z]\w*\s*:/m.test(body)) failures.push(`${name} fields must use camelCase`)
if (/^\s*(?:id|\w+Id)\s*:\s*number\b/im.test(body)) failures.push(`${name} identities must be strings`)
}
const searchSource = readFileSync('src/api/search.ts', 'utf8')
if (!searchSource.includes("'/api/v1/search'")) failures.push('search API must use /api/v1/search')
const inboxSource = readFileSync('src/api/inbox.ts', 'utf8')
for (const path of ['/api/v1/projects/', '/api/v1/inbox/']) {
if (!inboxSource.includes(path)) failures.push(`inbox API must use ${path}`)
}
if (existsSync('src/App.tsx')) {
failures.push('legacy src/App.tsx should be removed after page split')
}