refactor: clean code

This commit is contained in:
Fu Diwei 2024-12-18 10:27:55 +08:00
parent 2374bb56fa
commit d693d26323
17 changed files with 51 additions and 46 deletions

View File

@ -46,7 +46,7 @@ import AccessEditFormTencentCloudConfig from "./AccessEditFormTencentCloudConfig
import AccessEditFormVolcEngineConfig from "./AccessEditFormVolcEngineConfig"; import AccessEditFormVolcEngineConfig from "./AccessEditFormVolcEngineConfig";
import AccessEditFormWebhookConfig from "./AccessEditFormWebhookConfig"; import AccessEditFormWebhookConfig from "./AccessEditFormWebhookConfig";
type AccessEditFormModelType = Partial<Omit<AccessModel, "id" | "created" | "updated" | "deleted">>; type AccessEditFormModelType = Partial<MaybeModelRecord<AccessModel>>;
export type AccessEditFormProps = { export type AccessEditFormProps = {
className?: string; className?: string;
@ -79,9 +79,9 @@ const AccessEditForm = forwardRef<AccessEditFormInstance, AccessEditFormProps>((
const formRule = createSchemaFieldRule(formSchema); const formRule = createSchemaFieldRule(formSchema);
const [form] = Form.useForm<z.infer<typeof formSchema>>(); const [form] = Form.useForm<z.infer<typeof formSchema>>();
const [initialValues, setInitialValues] = useState<Partial<z.infer<typeof formSchema>>>(model ?? {}); const [initialValues, setInitialValues] = useState<Partial<z.infer<typeof formSchema>>>(model as Partial<z.infer<typeof formSchema>>);
useEffect(() => { useEffect(() => {
setInitialValues(model ?? {}); setInitialValues(model as Partial<z.infer<typeof formSchema>>);
}, [model]); }, [model]);
const [configType, setConfigType] = useState(model?.configType); const [configType, setConfigType] = useState(model?.configType);

View File

@ -9,7 +9,7 @@ import { getErrMsg } from "@/utils/error";
import AccessEditForm, { type AccessEditFormInstance } from "./AccessEditForm"; import AccessEditForm, { type AccessEditFormInstance } from "./AccessEditForm";
export type AccessEditModalProps = { export type AccessEditModalProps = {
data?: Partial<AccessModel>; data?: Partial<MaybeModelRecord<AccessModel>>;
loading?: boolean; loading?: boolean;
mode: "add" | "edit" | "copy"; mode: "add" | "edit" | "copy";
open?: boolean; open?: boolean;

View File

@ -1,4 +0,0 @@
declare module "antd/locale/zh_CN" {
import zhCN from "antd/locale/zh_CN";
export default zhCN;
}

View File

@ -1,5 +1,3 @@
import { type BaseModel } from "pocketbase";
/* /*
ASCII ASCII
NOTICE: If you add new constant, please keep ASCII order. NOTICE: If you add new constant, please keep ASCII order.

View File

@ -1,8 +1,6 @@
import { type BaseModel } from "pocketbase"; import { type WorkflowModel } from "./workflow";
import { WorkflowModel } from "./workflow"; export interface CertificateModel extends BaseModel {
export interface CertificateModel extends Omit<BaseModel, "created" | "updated"> {
san: string; san: string;
certificate: string; certificate: string;
privateKey: string; privateKey: string;

View File

@ -1,6 +1,4 @@
import { type BaseModel } from "pocketbase"; export interface SettingsModel<T> extends BaseModel {
export interface SettingsModel<T> extends Omit<BaseModel, "created" | "updated"> {
name: string; name: string;
content: T; content: T;
} }

View File

@ -1,6 +1,5 @@
import { produce } from "immer"; import { produce } from "immer";
import { nanoid } from "nanoid"; import { nanoid } from "nanoid";
import { type BaseModel } from "pocketbase";
import i18n from "@/i18n"; import i18n from "@/i18n";
import { deployTargets, KVType } from "./domain"; import { deployTargets, KVType } from "./domain";
@ -28,7 +27,7 @@ export type WorkflowOutput = {
error: string; error: string;
}; };
export interface WorkflowModel extends Omit<BaseModel, "created" | "updated"> { export interface WorkflowModel extends BaseModel {
name: string; name: string;
description?: string; description?: string;
type: string; type: string;
@ -152,6 +151,8 @@ export const initWorkflow = (): WorkflowModel => {
crontab: "0 0 * * *", crontab: "0 0 * * *",
enabled: false, enabled: false,
draft: rs, draft: rs,
created: new Date().toUTCString(),
updated: new Date().toUTCString(),
}; };
}; };

View File

@ -24,11 +24,6 @@
--input: 20 5.9% 90%; --input: 20 5.9% 90%;
--ring: 24.6 95% 53.1%; --ring: 24.6 95% 53.1%;
--radius: 0.5rem; --radius: 0.5rem;
--chart-1: 12 76% 61%;
--chart-2: 173 58% 39%;
--chart-3: 197 37% 24%;
--chart-4: 43 74% 66%;
--chart-5: 27 87% 67%;
} }
.dark { .dark {
@ -51,10 +46,5 @@
--border: 12 6.5% 15.1%; --border: 12 6.5% 15.1%;
--input: 12 6.5% 15.1%; --input: 12 6.5% 15.1%;
--ring: 20.5 90.2% 48.2%; --ring: 20.5 90.2% 48.2%;
--chart-1: 220 70% 50%;
--chart-2: 160 60% 45%;
--chart-3: 30 80% 55%;
--chart-4: 280 65% 60%;
--chart-5: 340 75% 55%;
} }
} }

13
ui/src/global.d.ts vendored Normal file
View File

@ -0,0 +1,13 @@
import { type BaseModel as PbBaseModel } from "pocketbase";
declare global {
declare interface BaseModel extends PbBaseModel {
deleted?: string;
}
declare type MaybeModelRecord<T extends BaseModel = BaseModel> = T | Omit<T, "id" | "created" | "updated" | "deleted">;
declare type MaybeModelRecordWithId<T extends BaseModel = BaseModel> = T | Pick<T, "id">;
}
export {};

View File

@ -12,7 +12,7 @@ export const list = async () => {
}); });
}; };
export const save = async (record: AccessModel | Omit<AccessModel, "id" | "created" | "updated" | "deleted">) => { export const save = async (record: MaybeModelRecord<AccessModel>) => {
if (record.id) { if (record.id) {
return await getPocketBase().collection(COLLECTION_NAME).update<AccessModel>(record.id, record); return await getPocketBase().collection(COLLECTION_NAME).update<AccessModel>(record.id, record);
} }
@ -20,7 +20,7 @@ export const save = async (record: AccessModel | Omit<AccessModel, "id" | "creat
return await getPocketBase().collection(COLLECTION_NAME).create<AccessModel>(record); return await getPocketBase().collection(COLLECTION_NAME).create<AccessModel>(record);
}; };
export const remove = async (record: AccessModel) => { export const remove = async (record: MaybeModelRecordWithId<AccessModel>) => {
record = { ...record, deleted: dayjs.utc().format("YYYY-MM-DD HH:mm:ss") }; record = { ...record, deleted: dayjs.utc().format("YYYY-MM-DD HH:mm:ss") };
await getPocketBase().collection(COLLECTION_NAME).update<AccessModel>(record.id!, record); await getPocketBase().collection(COLLECTION_NAME).update<AccessModel>(record.id!, record);
}; };

View File

@ -1,4 +1,4 @@
import { SettingsModel } from "@/domain/settings"; import { type SettingsModel } from "@/domain/settings";
import { getPocketBase } from "./pocketbase"; import { getPocketBase } from "./pocketbase";
export const get = async <T>(name: string) => { export const get = async <T>(name: string) => {
@ -13,7 +13,7 @@ export const get = async <T>(name: string) => {
} }
}; };
export const save = async <T>(record: SettingsModel<T>) => { export const save = async <T>(record: MaybeModelRecordWithId<SettingsModel<T>>) => {
if (record.id) { if (record.id) {
return await getPocketBase().collection("settings").update<SettingsModel<T>>(record.id, record); return await getPocketBase().collection("settings").update<SettingsModel<T>>(record.id, record);
} }

View File

@ -39,7 +39,7 @@ export const save = async (record: Record<string, string | boolean | WorkflowNod
return await getPocketBase().collection(COLLECTION_NAME).create<WorkflowModel>(record); return await getPocketBase().collection(COLLECTION_NAME).create<WorkflowModel>(record);
}; };
export const remove = async (record: WorkflowModel) => { export const remove = async (record: MaybeModelRecordWithId<WorkflowModel>) => {
return await getPocketBase().collection(COLLECTION_NAME).delete(record.id); return await getPocketBase().collection(COLLECTION_NAME).delete(record.id);
}; };

View File

@ -6,9 +6,9 @@ import { list as listAccess, save as saveAccess, remove as removeAccess } from "
export interface AccessState { export interface AccessState {
accesses: AccessModel[]; accesses: AccessModel[];
createAccess: (access: Omit<AccessModel, "id" | "created" | "udpated" | "deleted">) => void; createAccess: (access: MaybeModelRecord<AccessModel>) => void;
updateAccess: (access: AccessModel) => void; updateAccess: (access: MaybeModelRecordWithId<AccessModel>) => void;
deleteAccess: (access: AccessModel) => void; deleteAccess: (access: MaybeModelRecordWithId<AccessModel>) => void;
fetchAccesses: () => Promise<void>; fetchAccesses: () => Promise<void>;
} }

View File

@ -36,14 +36,14 @@ export const useWorkflowStore = create<WorkflowState>((set, get) => ({
id: "", id: "",
name: "placeholder", name: "placeholder",
type: WorkflowNodeType.Start, type: WorkflowNodeType.Start,
}, } as WorkflowModel,
initialized: false, initialized: false,
init: async (id?: string) => { init: async (id?: string) => {
let data: WorkflowModel = { let data = {
id: "", id: "",
name: "placeholder", name: "placeholder",
type: "auto", type: "auto",
}; } as WorkflowModel;
if (!id) { if (!id) {
data = initWorkflow(); data = initWorkflow();

View File

@ -4,12 +4,18 @@
"tsBuildInfoFile": "./node_modules/.tmp/tsconfig.app.tsbuildinfo", "tsBuildInfoFile": "./node_modules/.tmp/tsconfig.app.tsbuildinfo",
"target": "ES2020", "target": "ES2020",
"useDefineForClassFields": true, "useDefineForClassFields": true,
"lib": ["ES2020", "DOM", "DOM.Iterable"], "lib": [
"ES2020",
"DOM",
"DOM.Iterable"
],
"module": "ESNext", "module": "ESNext",
"skipLibCheck": true, "skipLibCheck": true,
"baseUrl": ".", "baseUrl": ".",
"paths": { "paths": {
"@/*": ["./src/*"] "@/*": [
"./src/*"
]
}, },
/* Bundler mode */ /* Bundler mode */
"moduleResolution": "bundler", "moduleResolution": "bundler",
@ -25,5 +31,7 @@
"noUnusedParameters": true, "noUnusedParameters": true,
"noFallthroughCasesInSwitch": true "noFallthroughCasesInSwitch": true
}, },
"include": ["src", "src/declarations.d.ts"] "include": [
"src"
]
} }

View File

@ -11,8 +11,9 @@
"compilerOptions": { "compilerOptions": {
"baseUrl": ".", "baseUrl": ".",
"paths": { "paths": {
"@/*": ["./src/*"] "@/*": [
"./src/*"
]
} }
} }
} }

View File

@ -9,5 +9,7 @@
"strict": true, "strict": true,
"noEmit": true "noEmit": true
}, },
"include": ["vite.config.ts"] "include": [
"vite.config.ts"
]
} }