From 9318bfa0e8d8001e1d30c9f7ae9032c8ad9c7877 Mon Sep 17 00:00:00 2001
From: Shigma <1700011071@pku.edu.cn>
Date: Sun, 16 Feb 2020 23:26:29 +0800
Subject: [PATCH] feat(core): optimize typings

---
 packages/koishi-core/src/context.ts  |  2 ++
 packages/koishi-core/src/database.ts | 22 +++++++++++-----------
 2 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/packages/koishi-core/src/context.ts b/packages/koishi-core/src/context.ts
index 2ee3705875..11c1229f68 100644
--- a/packages/koishi-core/src/context.ts
+++ b/packages/koishi-core/src/context.ts
@@ -127,6 +127,8 @@ export class Context {
     })
   }
 
+  plugin <T extends PluginFunction<this>> (plugin: T, options?: T extends PluginFunction<this, infer U> ? U : never): this
+  plugin <T extends PluginObject<this>> (plugin: T, options?: T extends PluginObject<this, infer U> ? U : never): this
   plugin <T extends Plugin<this>> (plugin: T, options?: T extends Plugin<this, infer U> ? U : never) {
     if (options === false) return
     const ctx = Object.create(this)
diff --git a/packages/koishi-core/src/database.ts b/packages/koishi-core/src/database.ts
index 06647bdcf2..c4b0f92de2 100644
--- a/packages/koishi-core/src/database.ts
+++ b/packages/koishi-core/src/database.ts
@@ -86,24 +86,24 @@ export function createGroup (id: number, assignee: number) {
 }
 
 export interface UserMethods {
-  getUser <K extends UserField> (userId: number, fields?: K[]): Promise<Pick<UserData, K | 'id'>>
-  getUser <K extends UserField> (userId: number, defaultAuthority?: number, fields?: K[]): Promise<Pick<UserData, K | 'id'>>
+  getUser <K extends UserField> (userId: number, fields?: readonly K[]): Promise<Pick<UserData, K | 'id'>>
+  getUser <K extends UserField> (userId: number, defaultAuthority?: number, fields?: readonly K[]): Promise<Pick<UserData, K | 'id'>>
   getUsers <K extends UserField> (fields?: K[]): Promise<Pick<UserData, K>[]>
-  getUsers <K extends UserField> (ids: number[], fields?: K[]): Promise<Pick<UserData, K>[]>
+  getUsers <K extends UserField> (ids: readonly number[], fields?: readonly K[]): Promise<Pick<UserData, K>[]>
   setUser (userId: number, data: Partial<UserData>): Promise<any>
-  observeUser <K extends UserField> (user: number | UserData, fields?: K[]): Promise<User<K>>
-  observeUser <K extends UserField> (user: number | UserData, defaultAuthority?: number, fields?: K[]): Promise<User<K>>
+  observeUser <K extends UserField> (user: number | UserData, fields?: readonly K[]): Promise<User<K>>
+  observeUser <K extends UserField> (user: number | UserData, defaultAuthority?: number, fields?: readonly K[]): Promise<User<K>>
   getUserCount (): Promise<number>
 }
 
 export interface GroupMethods {
-  getGroup <K extends GroupField> (groupId: number, fields?: K[]): Promise<Pick<GroupData, K | 'id'>>
-  getGroup <K extends GroupField> (groupId: number, selfId?: number, fields?: K[]): Promise<Pick<GroupData, K | 'id'>>
-  getAllGroups <K extends GroupField> (assignees?: number[]): Promise<Pick<GroupData, K>[]>
-  getAllGroups <K extends GroupField> (fields?: K[], assignees?: number[]): Promise<Pick<GroupData, K>[]>
+  getGroup <K extends GroupField> (groupId: number, fields?: readonly K[]): Promise<Pick<GroupData, K | 'id'>>
+  getGroup <K extends GroupField> (groupId: number, selfId?: number, fields?: readonly K[]): Promise<Pick<GroupData, K | 'id'>>
+  getAllGroups <K extends GroupField> (assignees?: readonly number[]): Promise<Pick<GroupData, K>[]>
+  getAllGroups <K extends GroupField> (fields?: readonly K[], assignees?: readonly number[]): Promise<Pick<GroupData, K>[]>
   setGroup (groupId: number, data: Partial<GroupData>): Promise<any>
-  observeGroup <K extends GroupField> (group: number | GroupData, fields?: K[]): Promise<Group<K>>
-  observeGroup <K extends GroupField> (group: number | GroupData, selfId?: number, fields?: K[]): Promise<Group<K>>
+  observeGroup <K extends GroupField> (group: number | GroupData, fields?: readonly K[]): Promise<Group<K>>
+  observeGroup <K extends GroupField> (group: number | GroupData, selfId?: number, fields?: readonly K[]): Promise<Group<K>>
   getGroupCount (): Promise<number>
 }