👍 allows readonly array for arguments

Author: Milly

autocmd/_utils.ts

@@ -1,8 +1,8 @@
 import type { AutocmdEvent, DefineOptions, RemoveOptions } from "./types.ts";
 
 export function buildDefineExpr(
-  event: AutocmdEvent | AutocmdEvent[],
-  pat: string | string[],
+  event: AutocmdEvent | readonly AutocmdEvent[],
+  pat: string | readonly string[],
   cmd: string,
   options: DefineOptions = {},
 ): string {
@@ -13,12 +13,12 @@ export function buildDefineExpr(
   if (Array.isArray(event)) {
     terms.push(event.join(","));
   } else {
-    terms.push(event);
+    terms.push(event as AutocmdEvent);
   }
   if (Array.isArray(pat)) {
     terms.push(pat.join(","));
   } else {
-    terms.push(pat);
+    terms.push(pat as string);
   }
   if (options.once) {
     terms.push("++once");
@@ -31,8 +31,8 @@ export function buildDefineExpr(
 }
 
 export function buildRemoveExpr(
-  event?: "*" | AutocmdEvent | AutocmdEvent[],
-  pat?: string | string[],
+  event?: "*" | AutocmdEvent | readonly AutocmdEvent[],
+  pat?: string | readonly string[],
   options: RemoveOptions = {},
 ): string {
   const terms = ["au!"];
@@ -43,13 +43,13 @@ export function buildRemoveExpr(
     if (Array.isArray(event)) {
       terms.push(event.join(","));
     } else {
-      terms.push(event);
+      terms.push(event as AutocmdEvent);
     }
     if (pat) {
       if (Array.isArray(pat)) {
         terms.push(pat.join(","));
       } else {
-        terms.push(pat);
+        terms.push(pat as string);
       }
     }
   }

autocmd/common.ts

@@ -41,8 +41,8 @@ import { buildDefineExpr, buildRemoveExpr } from "./_utils.ts";
  */
 export async function define(
   denops: Denops,
-  event: AutocmdEvent | AutocmdEvent[],
-  pat: string | string[],
+  event: AutocmdEvent | readonly AutocmdEvent[],
+  pat: string | readonly string[],
   cmd: string,
   options: DefineOptions = {},
 ): Promise<void> {
@@ -76,8 +76,8 @@ export async function define(
  */
 export async function remove(
   denops: Denops,
-  event?: "*" | AutocmdEvent | AutocmdEvent[],
-  pat?: string | string[],
+  event?: "*" | AutocmdEvent | readonly AutocmdEvent[],
+  pat?: string | readonly string[],
   options: RemoveOptions = {},
 ): Promise<void> {
   const expr = buildRemoveExpr(event, pat, options);
@@ -108,8 +108,8 @@ export async function remove(
  */
 export async function list(
   denops: Denops,
-  event?: "*" | AutocmdEvent | AutocmdEvent[],
-  pat?: string | string[],
+  event?: "*" | AutocmdEvent | readonly AutocmdEvent[],
+  pat?: string | readonly string[],
   options: ListOptions = {},
 ): Promise<unknown> {
   const terms = ["au"];
@@ -120,13 +120,13 @@ export async function list(
     if (Array.isArray(event)) {
       terms.push(event.join(","));
     } else {
-      terms.push(event);
+      terms.push(event as AutocmdEvent);
     }
     if (pat) {
       if (Array.isArray(pat)) {
         terms.push(pat.join(","));
       } else {
-        terms.push(pat);
+        terms.push(pat as string);
       }
     }
   }
@@ -155,7 +155,7 @@ export async function list(
  */
 export async function emit(
   denops: Denops,
-  event: AutocmdEvent | AutocmdEvent[],
+  event: AutocmdEvent | readonly AutocmdEvent[],
   fname?: string,
   options: EmitOptions = {},
 ): Promise<unknown> {
@@ -169,7 +169,7 @@ export async function emit(
   if (Array.isArray(event)) {
     terms.push(event.join(","));
   } else {
-    terms.push(event);
+    terms.push(event as AutocmdEvent);
   }
   if (fname) {
     terms.push(fname);
@@ -199,7 +199,7 @@ export async function emit(
  */
 export async function emitAll(
   denops: Denops,
-  event: AutocmdEvent | AutocmdEvent[],
+  event: AutocmdEvent | readonly AutocmdEvent[],
   fname?: string,
   options: EmitOptions = {},
 ): Promise<unknown> {
@@ -213,7 +213,7 @@ export async function emitAll(
   if (Array.isArray(event)) {
     terms.push(event.join(","));
   } else {
-    terms.push(event);
+    terms.push(event as AutocmdEvent);
   }
   if (fname) {
     terms.push(fname);

autocmd/group.ts

@@ -72,8 +72,8 @@ class GroupHelper {
    * Define an autocmd
    */
   define(
-    event: AutocmdEvent | AutocmdEvent[],
-    pat: string | string[],
+    event: AutocmdEvent | readonly AutocmdEvent[],
+    pat: string | readonly string[],
     cmd: string,
     options: GroupDefineOptions = {},
   ): void {
@@ -84,8 +84,8 @@ class GroupHelper {
    * Remove an autocmd
    */
   remove(
-    event?: "*" | AutocmdEvent | AutocmdEvent[],
-    pat?: string | string[],
+    event?: "*" | AutocmdEvent | readonly AutocmdEvent[],
+    pat?: string | readonly string[],
     options: GroupRemoveOptions = {},
   ): void {
     this.#commands.push(buildRemoveExpr(event, pat, options));