Browse Source

fix(plugin): correct parameters of datascript query api

charlie 2 years ago
parent
commit
488d376262
4 changed files with 54 additions and 43 deletions
  1. 6 0
      libs/CHANGELOG.md
  2. 1 1
      libs/package.json
  3. 2 1
      libs/src/LSPlugin.user.ts
  4. 45 41
      libs/webpack.config.js

+ 6 - 0
libs/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
+
+## [0.0.14]
+
+### Fixed
+- missing arguments of `DB.datascriptQuery`
+
 ## [0.0.13]
 
 ### Added

+ 1 - 1
libs/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@logseq/libs",
-  "version": "0.0.13",
+  "version": "0.0.14",
   "description": "Logseq SDK libraries",
   "main": "dist/lsplugin.user.js",
   "typings": "index.d.ts",

+ 2 - 1
libs/src/LSPlugin.user.ts

@@ -410,6 +410,7 @@ const db: Partial<IDBProxy> = {
     query: string,
     ...inputs: Array<any>
   ): Promise<T> {
+    // force remove proxy ns flag `db`
     inputs.pop()
 
     if (inputs?.some(it => (typeof it === 'function'))) {
@@ -419,7 +420,7 @@ const db: Partial<IDBProxy> = {
 
     return this._execCallableAPIAsync(
       `datascript_query`,
-      ...inputs
+      ...[query, ...inputs]
     )
   }
 }

+ 45 - 41
libs/webpack.config.js

@@ -4,46 +4,50 @@ const webpack = require('webpack')
 const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin
 const TerserPlugin = require('terser-webpack-plugin')
 
-module.exports = {
-  entry: './src/LSPlugin.user.ts',
-  module: {
-    rules: [
-      {
-        test: /\.tsx?$/,
-        use: [
-          {
-            loader: 'babel-loader'
-          },
-          {
-            loader: 'ts-loader'
-          }
-        ],
-        exclude: /node_modules/,
-      }
+module.exports = (env, argv) => {
+  const isProd = argv.mode === 'production'
+
+  return {
+    entry: './src/LSPlugin.user.ts',
+    module: {
+      rules: [
+        {
+          test: /\.tsx?$/,
+          use: [
+            {
+              loader: 'babel-loader'
+            },
+            {
+              loader: 'ts-loader'
+            }
+          ],
+          exclude: /node_modules/,
+        }
+      ],
+    },
+    resolve: {
+      extensions: ['.tsx', '.ts', '.js'],
+    },
+    optimization: {
+      minimize: isProd,
+      minimizer: [
+        new TerserPlugin()
+      ]
+    },
+    plugins: [
+      new webpack.ProvidePlugin({
+        process: 'process/browser',
+      }),
+      new webpack.DefinePlugin({
+        LIB_VERSION: JSON.stringify(pkg.version)
+      })
+      // new BundleAnalyzerPlugin()
     ],
-  },
-  resolve: {
-    extensions: ['.tsx', '.ts', '.js'],
-  },
-  optimization: {
-    minimize: true,
-    minimizer: [
-      new TerserPlugin()
-    ]
-  },
-  plugins: [
-    new webpack.ProvidePlugin({
-      process: 'process/browser',
-    }),
-    new webpack.DefinePlugin({
-      LIB_VERSION: JSON.stringify(pkg.version)
-    })
-    // new BundleAnalyzerPlugin()
-  ],
-  output: {
-    library: 'LSPluginEntry',
-    libraryTarget: 'umd',
-    filename: 'lsplugin.user.js',
-    path: path.resolve(__dirname, 'dist')
-  },
+    output: {
+      library: 'LSPluginEntry',
+      libraryTarget: 'umd',
+      filename: 'lsplugin.user.js',
+      path: path.resolve(__dirname, 'dist')
+    },
+  }
 }