refactor!(napi): use EngineId instead of our enum

* Supported executor.list() to get all available engines
* Engine is no longer present

Author: pan93412

napi/demo/demo.js

@@ -6,8 +6,9 @@ async function main() {
   UNM.enableLogging(UNM.LoggingType.ConsoleEnv);
   const executor = new UNM.Executor();
 
+  console.log(`Enabled engines: ${executor.list().join(", ")}`);
   const searchResult = await executor.search(
-    Object.values(UNM.Engine),
+    executor.list(),
     {
       id: "12345",
       name: "青花瓷",

napi/index.d.ts

@@ -3,14 +3,6 @@
 
 /* auto-generated by NAPI-RS */
 
-export const enum Engine {
-  Bilibili = 0,
-  Kugou = 1,
-  Migu = 2,
-  PyNCM = 3,
-  YtDl = 4,
-  Kuwo = 5
-}
 /** [napi-rs] The metadata of the artist of a song. */
 export interface Artist {
   /** The identifier of this artist. */
@@ -91,6 +83,7 @@ export function enableLogging(logType: LoggingType): void
 export type JsExecutor = Executor
 export class Executor {
   constructor()
-  search(engines: Array<Engine>, song: Song, ctx: Context): Promise<SongSearchInformation>
+  list(): Array<string>
+  search(engines: Array<string>, song: Song, ctx: Context): Promise<SongSearchInformation>
   retrieve(song: SongSearchInformation, ctx: Context): Promise<RetrievedSongInfo>
 }

napi/index.js

@@ -236,9 +236,8 @@ if (!nativeBinding) {
   throw new Error(`Failed to load native binding`)
 }
 
-const { Engine, Executor, LoggingType, enableLogging } = nativeBinding
+const { Executor, LoggingType, enableLogging } = nativeBinding
 
-module.exports.Engine = Engine
 module.exports.Executor = Executor
 module.exports.LoggingType = LoggingType
 module.exports.enableLogging = enableLogging

napi/src/engines.rs

@@ -4,10 +4,7 @@ use napi_derive::napi;
 use std::{borrow::Cow, sync::Arc};
 use unm_engine::executor::Executor;
 
-use crate::{
-  engines::Engine,
-  types::{Context, RetrievedSongInfo, Song, SongSearchInformation},
-};
+use crate::types::{Context, RetrievedSongInfo, Song, SongSearchInformation};
 
 #[napi(js_name = "Executor")]
 pub struct JsExecutor {
@@ -21,16 +18,21 @@ impl JsExecutor {
     Self::default()
   }
 
+  #[napi]
+  pub fn list(&self) -> Vec<&str> {
+    self.executor.list()
+  }
+
   #[napi]
   pub async fn search(
     &self,
-    engines: Vec<Engine>,
+    engines: Vec<String>,
     song: Song,
     ctx: Context,
   ) -> Result<SongSearchInformation> {
     let engines = engines
       .into_iter()
-      .map(|engine| engine.as_str().into())
+      .map(|engine| engine.into())
       .collect::<Vec<Cow<'static, str>>>();
 
     self

napi/src/executor.rs

@@ -1,4 +1,3 @@
-pub mod engines;
 pub mod executor;
 pub mod types;