feat(v2.4): 切换根数据领域主写
This commit is contained in:
110
apps/web/lib/domain-api.ts
Normal file
110
apps/web/lib/domain-api.ts
Normal file
@@ -0,0 +1,110 @@
|
||||
import { api } from './api';
|
||||
|
||||
export interface RootProject {
|
||||
id: string;
|
||||
name: string;
|
||||
description: string;
|
||||
createdAt: string;
|
||||
}
|
||||
|
||||
export interface RootVersion {
|
||||
id: string;
|
||||
productId?: string;
|
||||
projectId?: string | null;
|
||||
name: string;
|
||||
status?: string;
|
||||
releaseDate: string | null;
|
||||
createdAt: string;
|
||||
currentStage?: string | null;
|
||||
startDate?: string | null;
|
||||
expectedReleaseDate?: string | null;
|
||||
members?: unknown[];
|
||||
progress?: unknown[];
|
||||
priority?: string;
|
||||
links?: unknown;
|
||||
}
|
||||
|
||||
export interface RootProduct {
|
||||
id: string;
|
||||
name: string;
|
||||
description: string;
|
||||
createdAt: string;
|
||||
updatedAt: string;
|
||||
projects: RootProject[];
|
||||
versions: RootVersion[];
|
||||
_count?: { requirements: number; projects: number; versions?: number };
|
||||
}
|
||||
|
||||
export async function createProductRoot(data: { name: string; description?: string }): Promise<RootProduct> {
|
||||
const product = await api.post<Omit<RootProduct, 'projects' | 'versions'>>('/products', data);
|
||||
return normalizeProductRoot({ ...product, projects: [], versions: [] });
|
||||
}
|
||||
|
||||
export async function updateProductRoot(
|
||||
productId: string,
|
||||
data: { name?: string; description?: string },
|
||||
): Promise<Partial<RootProduct>> {
|
||||
return api.patch<Partial<RootProduct>>(`/products/${productId}`, data);
|
||||
}
|
||||
|
||||
export async function deleteProductRoot(productId: string): Promise<void> {
|
||||
await api.delete(`/products/${productId}`);
|
||||
}
|
||||
|
||||
export async function createProjectByProductId(
|
||||
productId: string,
|
||||
data: { name: string; description?: string },
|
||||
): Promise<RootProject> {
|
||||
return api.post<RootProject>(`/products/${productId}/projects`, data);
|
||||
}
|
||||
|
||||
export async function updateProjectByProductId(
|
||||
productId: string,
|
||||
projectId: string,
|
||||
data: { name?: string; description?: string },
|
||||
): Promise<RootProject> {
|
||||
return api.patch<RootProject>(`/products/${productId}/projects/${projectId}`, data);
|
||||
}
|
||||
|
||||
export async function deleteProjectByProductId(productId: string, projectId: string): Promise<void> {
|
||||
await api.delete(`/products/${productId}/projects/${projectId}`);
|
||||
}
|
||||
|
||||
export async function createVersionByProductId(
|
||||
productId: string,
|
||||
data: { name: string; status?: string; projectId?: string },
|
||||
): Promise<RootVersion> {
|
||||
return normalizeVersionRoot(await api.post<RootVersion>(`/products/${productId}/versions`, data));
|
||||
}
|
||||
|
||||
export async function updateVersionByProductId(
|
||||
productId: string,
|
||||
versionId: string,
|
||||
data: Record<string, unknown>,
|
||||
): Promise<RootVersion> {
|
||||
return normalizeVersionRoot(await api.patch<RootVersion>(`/products/${productId}/versions/${versionId}`, data));
|
||||
}
|
||||
|
||||
export async function deleteVersionByProductId(productId: string, versionId: string): Promise<void> {
|
||||
await api.delete(`/products/${productId}/versions/${versionId}`);
|
||||
}
|
||||
|
||||
function normalizeProductRoot(product: RootProduct): RootProduct {
|
||||
return {
|
||||
...product,
|
||||
projects: product.projects ?? [],
|
||||
versions: (product.versions ?? []).map(normalizeVersionRoot),
|
||||
_count: product._count ?? { requirements: 0, projects: product.projects?.length ?? 0, versions: product.versions?.length ?? 0 },
|
||||
};
|
||||
}
|
||||
|
||||
function normalizeVersionRoot(version: RootVersion): RootVersion {
|
||||
return {
|
||||
...version,
|
||||
status: version.status ?? 'planned',
|
||||
releaseDate: version.releaseDate ?? null,
|
||||
members: Array.isArray(version.members) ? version.members : [],
|
||||
progress: Array.isArray(version.progress) ? version.progress : [],
|
||||
links: version.links && typeof version.links === 'object' ? version.links : {},
|
||||
};
|
||||
}
|
||||
61
apps/web/lib/product-domain-write-source.test.ts
Normal file
61
apps/web/lib/product-domain-write-source.test.ts
Normal file
@@ -0,0 +1,61 @@
|
||||
import assert from 'node:assert/strict';
|
||||
import { readFileSync } from 'node:fs';
|
||||
import { join } from 'node:path';
|
||||
import test from 'node:test';
|
||||
|
||||
const source = () => readFileSync(join(process.cwd(), 'stores/useProductStore.ts'), 'utf8');
|
||||
|
||||
function storeMethodBody(text: string, name: string) {
|
||||
const start = text.indexOf(`\n ${name}: async`);
|
||||
assert.notEqual(start, -1, `missing store method ${name}`);
|
||||
|
||||
let depth = 0;
|
||||
let sawFirstBrace = false;
|
||||
for (let i = start; i < text.length; i += 1) {
|
||||
const char = text[i];
|
||||
if (char === '{') {
|
||||
depth += 1;
|
||||
sawFirstBrace = true;
|
||||
}
|
||||
if (char === '}') {
|
||||
depth -= 1;
|
||||
if (sawFirstBrace && depth === 0) {
|
||||
return text.slice(start, i + 1);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
throw new Error(`could not extract store method ${name}`);
|
||||
}
|
||||
|
||||
test('product root store imports domain root write helpers', () => {
|
||||
assert.match(source(), /from '@\/lib\/domain-api'/);
|
||||
});
|
||||
|
||||
test('project mutations use domain APIs instead of products-overview AppData saves', () => {
|
||||
const text = source();
|
||||
const createProject = storeMethodBody(text, 'createProject');
|
||||
const updateProject = storeMethodBody(text, 'updateProject');
|
||||
const deleteProject = storeMethodBody(text, 'deleteProject');
|
||||
|
||||
assert.match(createProject, /createProjectByProductId\(productId, data\)/);
|
||||
assert.match(updateProject, /updateProjectByProductId\(productId, projectId, data\)/);
|
||||
assert.match(deleteProject, /deleteProjectByProductId\(productId, projectId\)/);
|
||||
assert.doesNotMatch(createProject, /saveStoredOverview\(updated\)/);
|
||||
assert.doesNotMatch(updateProject, /saveStoredOverview\(updated\)/);
|
||||
assert.doesNotMatch(deleteProject, /saveStoredOverview\(updated\)/);
|
||||
});
|
||||
|
||||
test('version mutations use domain APIs instead of products-overview AppData saves', () => {
|
||||
const text = source();
|
||||
const createVersion = storeMethodBody(text, 'createVersion');
|
||||
const updateVersion = storeMethodBody(text, 'updateVersion');
|
||||
const deleteVersion = storeMethodBody(text, 'deleteVersion');
|
||||
|
||||
assert.match(createVersion, /createVersionByProductId\(productId, data\)/);
|
||||
assert.match(updateVersion, /updateVersionByProductId\(productId, versionId, data\)/);
|
||||
assert.match(deleteVersion, /deleteVersionByProductId\(productId, versionId\)/);
|
||||
assert.doesNotMatch(createVersion, /saveStoredOverview\(updated\)/);
|
||||
assert.doesNotMatch(updateVersion, /saveStoredOverview\(updated\)/);
|
||||
assert.doesNotMatch(deleteVersion, /saveStoredOverview\(updated\)/);
|
||||
});
|
||||
@@ -3,8 +3,18 @@
|
||||
import { create } from 'zustand';
|
||||
import { Product } from '@ftb/shared';
|
||||
import { api } from '@/lib/api';
|
||||
import {
|
||||
createProductRoot,
|
||||
createProjectByProductId,
|
||||
createVersionByProductId,
|
||||
deleteProductRoot,
|
||||
deleteProjectByProductId,
|
||||
deleteVersionByProductId,
|
||||
updateProductRoot,
|
||||
updateProjectByProductId,
|
||||
updateVersionByProductId,
|
||||
} from '@/lib/domain-api';
|
||||
import { saveWithOptimisticRollback } from '@/lib/optimistic-persistence';
|
||||
import { shouldPersistRemoteOverview } from '@/lib/product-overview-persistence';
|
||||
import { loadServerData, saveServerData, SERVER_DATA_CACHE_MS } from '@/lib/server-data';
|
||||
import type { Stage, Role } from '@/lib/stage';
|
||||
import type { Priority, VersionLinks } from '@/lib/derive';
|
||||
@@ -86,51 +96,39 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
fetchProducts: async () => {
|
||||
set({ loading: true, error: null });
|
||||
try {
|
||||
const products = await api.get<ProductWithCount[]>('/products');
|
||||
set({ products, loading: false });
|
||||
} catch {
|
||||
const overview = await loadStoredOverview();
|
||||
const products = overview
|
||||
? overview.map(({ projects: _projects, versions: _versions, ...product }) => product)
|
||||
: await api.get<ProductWithCount[]>('/products');
|
||||
set({ products, loading: false });
|
||||
} catch {
|
||||
set({ products: [], error: null, loading: false });
|
||||
: [];
|
||||
set({ products, error: null, loading: false });
|
||||
}
|
||||
},
|
||||
|
||||
fetchOverview: async () => {
|
||||
if (get().overviewLoaded && Date.now() - lastOverviewFetchAt < SERVER_DATA_CACHE_MS) return;
|
||||
const cached = await loadStoredOverview();
|
||||
if (get().overviewLoaded && Date.now() - lastOverviewFetchAt < SERVER_DATA_CACHE_MS) return;
|
||||
lastOverviewFetchAt = Date.now();
|
||||
if (cached) {
|
||||
// 有本地缓存,直接用,不再调远端覆盖(mock 模式核心数据在本地)
|
||||
set({ overview: cached, loading: false, overviewLoaded: true });
|
||||
return;
|
||||
}
|
||||
set({ loading: true, error: null });
|
||||
try {
|
||||
const overview = await api.get<ProductOverview[]>('/products/overview');
|
||||
set({ overview, loading: false, overviewLoaded: true });
|
||||
if (shouldPersistRemoteOverview(overview)) {
|
||||
void saveStoredOverview(overview).catch(() => {});
|
||||
}
|
||||
} catch {
|
||||
set({ overview: MOCK_OVERVIEW, error: null, loading: false, overviewLoaded: true });
|
||||
const cached = await loadStoredOverview();
|
||||
if (get().overviewLoaded && Date.now() - lastOverviewFetchAt < SERVER_DATA_CACHE_MS) return;
|
||||
set({ overview: cached ?? MOCK_OVERVIEW, error: null, loading: false, overviewLoaded: true });
|
||||
}
|
||||
},
|
||||
|
||||
fetchProduct: async (id) => {
|
||||
set({ loading: true, error: null });
|
||||
try {
|
||||
const overview = await loadStoredOverview();
|
||||
const found = overview?.find((p) => p.id === id);
|
||||
if (found) {
|
||||
set({ currentProduct: found as any, loading: false });
|
||||
return;
|
||||
}
|
||||
const product = await api.get<Product>(`/products/${id}`);
|
||||
set({ currentProduct: product, loading: false });
|
||||
} catch {
|
||||
const found = get().overview.find((p) => p.id === id);
|
||||
const overview = await loadStoredOverview();
|
||||
const found = overview?.find((p) => p.id === id) ?? get().overview.find((p) => p.id === id);
|
||||
if (found) {
|
||||
set({ currentProduct: found as any, loading: false });
|
||||
} else {
|
||||
@@ -154,7 +152,18 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
const updated = [newProduct, ...previousOverview];
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
const created = await createProductRoot(data);
|
||||
set({
|
||||
overview: get().overview.map((product) => (product.id === newProduct.id ? created as ProductOverview : product)),
|
||||
products: [created, ...get().products] as ProductWithCount[],
|
||||
overviewLoaded: true,
|
||||
});
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -174,7 +183,13 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
previousCurrentProduct?.id === id ? ({ ...previousCurrentProduct, ...data } as any) : previousCurrentProduct,
|
||||
});
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
await updateProductRoot(id, data);
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({
|
||||
@@ -197,7 +212,13 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
currentProduct: previousCurrentProduct?.id === id ? null : previousCurrentProduct,
|
||||
});
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
await deleteProductRoot(id);
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({
|
||||
@@ -269,7 +290,24 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
});
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
const created = await createProjectByProductId(productId, data);
|
||||
set({
|
||||
overview: get().overview.map((p) => {
|
||||
if (p.id !== productId) return p;
|
||||
return {
|
||||
...p,
|
||||
projects: p.projects.map((project) => (project.id === newProject.id ? created : project)),
|
||||
_count: { ...p._count, projects: p.projects.length } as any,
|
||||
};
|
||||
}),
|
||||
overviewLoaded: true,
|
||||
});
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -312,7 +350,13 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
});
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
await updateProjectByProductId(productId, projectId, data);
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -339,7 +383,24 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
});
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
const created = await createVersionByProductId(productId, data);
|
||||
set({
|
||||
overview: get().overview.map((p) => {
|
||||
if (p.id !== productId) return p;
|
||||
return {
|
||||
...p,
|
||||
versions: p.versions.map((version) => (version.id === newVersion.id ? created as VersionItem : version)),
|
||||
_count: { ...p._count, versions: p.versions.length } as any,
|
||||
};
|
||||
}),
|
||||
overviewLoaded: true,
|
||||
});
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -359,7 +420,13 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
});
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
await updateVersionByProductId(productId, versionId, data);
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -378,7 +445,13 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
});
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
await deleteVersionByProductId(productId, versionId);
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -397,7 +470,13 @@ export const useProductStore = create<ProductState>((set, get) => ({
|
||||
});
|
||||
set({ overview: updated, overviewLoaded: true });
|
||||
await saveWithOptimisticRollback({
|
||||
save: () => saveStoredOverview(updated),
|
||||
save: async () => {
|
||||
try {
|
||||
await deleteProjectByProductId(productId, projectId);
|
||||
} catch {
|
||||
await saveRootOverviewFallback(updated);
|
||||
}
|
||||
},
|
||||
expected: updated,
|
||||
getCurrent: () => get().overview,
|
||||
rollback: () => set({ overview: previousOverview, overviewLoaded: true }),
|
||||
@@ -409,6 +488,10 @@ async function saveStoredOverview(data: ProductOverview[]) {
|
||||
await saveServerData('products-overview', data);
|
||||
}
|
||||
|
||||
async function saveRootOverviewFallback(data: ProductOverview[]) {
|
||||
await saveStoredOverview(data);
|
||||
}
|
||||
|
||||
async function loadStoredOverview(): Promise<ProductOverview[] | null> {
|
||||
try {
|
||||
return await loadServerData<ProductOverview[]>('products-overview');
|
||||
|
||||
Reference in New Issue
Block a user