@valtzu/codemirror-lang-el 1.1.0 → 1.3.0

package/CHANGELOG.md

@@ -1,6 +1,19 @@
 CHANGELOG
 =========
 
+1.3
+---
+
+* Lint left- and right-side arguments of `contains`, `starts with`, `ends with` and `matches` operators
+* Add eslint to make contributing/review easier
+* Add type checking for left-side argument in `in` expression
+
+1.2
+---
+
+* Add support for typed arrays
+* Add Codex CLI and Docker Compose integration
+
 1.1
 ---
 

package/README.md

@@ -42,7 +42,7 @@ If you're using Bootstrap UI, check the [Web Component](https://github.com/valtz
 
 ```html
 <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/css/bootstrap.min.css" rel="stylesheet">
-<script type="module" src="https://esm.sh/symfony-expression-editor@0.1.0"></script>
+<script type="module" src="https://esm.sh/symfony-expression-editor"></script>
 <textarea class="form-control" is="expression-editor" rows="1">'foobar' starts with 'foo'</textarea>
 ```
 
@@ -72,7 +72,7 @@ See [CONFIGURATION.md](CONFIGURATION.md)
 
 ### Example
 
-[Live demo](https://jsfiddle.net/turse2xq/)
+[Live demo](https://jsfiddle.net/32ybaetz/)
 
 ```html
 <div id="editor"></div>
@@ -118,6 +118,15 @@ See [CONFIGURATION.md](CONFIGURATION.md)
 </script>
 ```
 
+### Using OpenAI codex
+
+Create `.env` file with
+```
+OPENAI_API_KEY="..."
+```
+
+Then use `docker compose run --rm codex` to develop with AI.
+
 ### Contributing
 
 Contributions are welcome.

package/dist/index.cjs

@@ -8,7 +8,7 @@ var autocomplete = require('@codemirror/autocomplete');
 var state = require('@codemirror/state');
 var view = require('@codemirror/view');
 
-// @ts-ignore
+// @ts-expect-error TS2739
 const t = {
     deserialize: (str) => str,
 };
@@ -117,7 +117,7 @@ const resolveIdentifier = (nodeTypeId, identifier, config) => {
 };
 function resolveTypes(state, node, config) {
     var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l;
-    let types = new Set();
+    const types = new Set();
     if (!node) {
         return types;
     }
@@ -130,19 +130,19 @@ function resolveTypes(state, node, config) {
     }
     else if (node.type.is(Variable)) {
         const varName = state.sliceDoc(node.from, node.to) || '';
-        // @ts-ignore
+        // @ts-expect-error TS2339
         (_b = (_a = resolveIdentifier(node.type.id, varName, config)) === null || _a === void 0 ? void 0 : _a.type) === null || _b === void 0 ? void 0 : _b.forEach((x) => types.add(x));
     }
     else if (node.type.is(Function)) {
         const varName = state.sliceDoc(node.from, node.to) || '';
-        // @ts-ignore
+        // @ts-expect-error TS2339
         (_d = (_c = resolveIdentifier(node.type.id, varName, config)) === null || _c === void 0 ? void 0 : _c.returnType) === null || _d === void 0 ? void 0 : _d.forEach((x) => types.add(x));
     }
     else if (node.type.is(PropertyAccess) && node.firstChild && ((_e = node.lastChild) === null || _e === void 0 ? void 0 : _e.type.is(Property))) {
         const varName = state.sliceDoc(node.lastChild.from, node.lastChild.to) || '';
         (_f = resolveTypes(state, node.firstChild, config)) === null || _f === void 0 ? void 0 : _f.forEach(baseType => {
             var _a, _b, _c, _d;
-            // @ts-ignore
+            // @ts-expect-error TS2339
             (_d = (_c = resolveIdentifier((_a = node.lastChild) === null || _a === void 0 ? void 0 : _a.type.id, varName, (_b = config.types) === null || _b === void 0 ? void 0 : _b[baseType])) === null || _c === void 0 ? void 0 : _c.type) === null || _d === void 0 ? void 0 : _d.forEach((x) => types.add(x));
         });
     }
@@ -150,10 +150,22 @@ function resolveTypes(state, node, config) {
         const varName = state.sliceDoc(node.lastChild.from, node.lastChild.to) || '';
         (_h = resolveTypes(state, node.firstChild, config)) === null || _h === void 0 ? void 0 : _h.forEach(baseType => {
             var _a, _b, _c, _d;
-            // @ts-ignore
+            // @ts-expect-error TS2339
             (_d = (_c = resolveIdentifier((_a = node.lastChild) === null || _a === void 0 ? void 0 : _a.type.id, varName, (_b = config.types) === null || _b === void 0 ? void 0 : _b[baseType])) === null || _c === void 0 ? void 0 : _c.returnType) === null || _d === void 0 ? void 0 : _d.forEach((x) => types.add(x));
         });
     }
+    // Array indexing: for typed arrays (e.g. Foo[]) return element type, for generic arrays return any
+    else if (node.type.is(ArrayAccess) && node.firstChild) {
+        const left = node.firstChild.node;
+        resolveTypes(state, left, config).forEach(baseType => {
+            if (baseType.endsWith('[]')) {
+                types.add(baseType.slice(0, -2));
+            }
+            else if (baseType === exports.ELScalar.Array) {
+                types.add(exports.ELScalar.Any);
+            }
+        });
+    }
     else if (node.type.is(Application) && node.firstChild) {
         resolveTypes(state, node.firstChild, config).forEach(x => types.add(x));
     }
@@ -222,13 +234,13 @@ var utils = /*#__PURE__*/Object.freeze({
  */
 const expressionLanguageLinterSource = (state) => {
     const config = getExpressionLanguageConfig(state);
-    let diagnostics = [];
+    const diagnostics = [];
     language.syntaxTree(state).cursor().iterate(node => {
         var _a, _b, _c, _d, _e, _f;
         const { from, to, type: { id } } = node;
         let identifier;
         switch (id) {
-            case 0:
+            case 0: {
                 if (state.doc.length === 0 || from === 0) {
                     // Don't show error on empty doc (even though it is an error)
                     return;
@@ -242,7 +254,8 @@ const expressionLanguageLinterSource = (state) => {
                     diagnostics.push({ from, to, severity: 'error', message: `Unexpected ${type} <code>${identifier}</code>` });
                 }
                 return;
-            case Arguments:
+            }
+            case Arguments: {
                 const fn = resolveFunctionDefinition(node.node.prevSibling, state, config);
                 const args = fn === null || fn === void 0 ? void 0 : fn.args;
                 if (!args) {
@@ -263,7 +276,12 @@ const expressionLanguageLinterSource = (state) => {
                     const typesUsed = Array.from(resolveTypes(state, n, config));
                     const typesExpected = args[i].type;
                     if (typesExpected && !typesExpected.includes(exports.ELScalar.Any) && !typesUsed.some(x => typesExpected.includes(x))) {
-                        diagnostics.push({ from: n.from, to: n.to, severity: 'error', message: `<code>${typesExpected.join('|')}</code> expected, got <code>${typesUsed.join('|')}</code>` });
+                        diagnostics.push({
+                            from: n.from,
+                            to: n.to,
+                            severity: 'error',
+                            message: `<code>${typesExpected.join('|')}</code> expected, got <code>${typesUsed.join('|')}</code>`
+                        });
                     }
                     i++;
                 }
@@ -271,8 +289,9 @@ const expressionLanguageLinterSource = (state) => {
                     diagnostics.push({ from: node.from, to: node.to, severity: 'error', message: `Too few arguments – ${argumentCountHintFn()}` });
                 }
                 break;
+            }
             case Property:
-            case Method:
+            case Method: {
                 const leftArgument = (_e = (_d = node.node.parent) === null || _d === void 0 ? void 0 : _d.firstChild) === null || _e === void 0 ? void 0 : _e.node;
                 const types = Array.from(resolveTypes(state, leftArgument, config));
                 identifier = state.sliceDoc(from, to);
@@ -280,26 +299,46 @@ const expressionLanguageLinterSource = (state) => {
                     diagnostics.push({ from, to, severity: 'error', message: `${node.name} <code>${identifier}</code> not found in <code>${types.join('|')}</code>` });
                 }
                 break;
+            }
             case Variable:
-            case Function:
+            case Function: {
                 identifier = state.sliceDoc(from, node.node.firstChild ? node.node.firstChild.from - 1 : to);
                 if (!resolveIdentifier(id, identifier, config)) {
                     diagnostics.push({ from, to, severity: 'error', message: `${node.node.name} <code>${identifier}</code> not found` });
                 }
                 break;
-            case BinaryExpression:
+            }
+            case BinaryExpression: {
                 const operatorNode = node.node.getChild(OperatorKeyword);
                 if (operatorNode) {
                     const operator = state.sliceDoc(operatorNode.from, operatorNode.to);
+                    const leftArgument = node.node.firstChild;
+                    const rightArgument = node.node.lastChild;
                     if (operator === 'in') {
-                        const rightArgument = node.node.lastChild;
-                        const types = resolveTypes(state, rightArgument, config);
-                        if (!types.has(exports.ELScalar.Array)) {
-                            diagnostics.push({ from: rightArgument.from, to: rightArgument.to, severity: 'error', message: `<code>${exports.ELScalar.Array}</code> expected, got <code>${[...types].join('|')}</code>` });
+                        const leftTypes = resolveTypes(state, leftArgument, config);
+                        const rightTypes = resolveTypes(state, rightArgument, config);
+                        const allowsAny = rightTypes.has(exports.ELScalar.Array) || rightTypes.has(exports.ELScalar.Any);
+                        if (!allowsAny && ![...rightTypes].some(x => x.endsWith('[]'))) {
+                            diagnostics.push({ from: rightArgument.from, to: rightArgument.to, severity: 'error', message: `<code>${exports.ELScalar.Array}</code> expected, got <code>${[...rightTypes].join('|')}</code>` });
+                        }
+                        else if (!allowsAny && !rightTypes.has(`${exports.ELScalar.Any}[]`) && ![...leftTypes].some(type => rightTypes.has(`${type}[]`))) {
+                            diagnostics.push({ from: leftArgument.from, to: rightArgument.to, severity: 'warning', message: `Expression is always <code>false</code> because <code>${[...leftTypes].join('|')}</code> not found in <code>${[...rightTypes].join('|')}</code>` });
+                        }
+                    }
+                    else if (["contains", "starts with", "ends with", "matches"].includes(operator)) {
+                        // Both sides must be string
+                        const leftTypes = resolveTypes(state, leftArgument, config);
+                        const rightTypes = resolveTypes(state, rightArgument, config);
+                        if (!leftTypes.has(exports.ELScalar.String)) {
+                            diagnostics.push({ from: leftArgument.from, to: leftArgument.to, severity: 'error', message: `<code>string</code> expected, got <code>${[...leftTypes].join('|')}</code>` });
+                        }
+                        if (!rightTypes.has(exports.ELScalar.String)) {
+                            diagnostics.push({ from: rightArgument.from, to: rightArgument.to, severity: 'error', message: `<code>string</code> expected, got <code>${[...rightTypes].join('|')}</code>` });
                         }
                     }
                 }
                 break;
+            }
         }
         if (identifier && ((_f = node.node.parent) === null || _f === void 0 ? void 0 : _f.type.isError)) {
             diagnostics.push({ from, to, severity: 'error', message: `Unexpected identifier <code>${identifier}</code>` });
@@ -371,7 +410,7 @@ function completeIdentifier(state, config, tree, from, to) {
         validFor: /^[a-zA-Z_]+[a-zA-Z_0-9]*$/,
     };
 }
-function completeMember(state, config, tree, from, to, explicit) {
+function completeMember(state, config, tree, from, to) {
     var _a, _b, _c, _d, _e, _f;
     if (!(((_a = tree.parent) === null || _a === void 0 ? void 0 : _a.type.is(PropertyAccess)) || ((_b = tree.parent) === null || _b === void 0 ? void 0 : _b.type.is(MethodAccess))) || !((_c = tree.parent) === null || _c === void 0 ? void 0 : _c.firstChild)) {
         return null;
@@ -380,7 +419,7 @@ function completeMember(state, config, tree, from, to, explicit) {
     if (!(types === null || types === void 0 ? void 0 : types.size)) {
         return null;
     }
-    let options = [];
+    const options = [];
     for (const type of types) {
         const typeDeclaration = (_d = config.types) === null || _d === void 0 ? void 0 : _d[type];
         options.push(...(((_e = typeDeclaration === null || typeDeclaration === void 0 ? void 0 : typeDeclaration.identifiers) === null || _e === void 0 ? void 0 : _e.map(autocompleteIdentifier)) || []), ...(((_f = typeDeclaration === null || typeDeclaration === void 0 ? void 0 : typeDeclaration.functions) === null || _f === void 0 ? void 0 : _f.map(autocompleteFunction)) || []));
@@ -442,7 +481,6 @@ function resolveArguments(node) {
 }
 function getCursorTooltips(state) {
     const config = getExpressionLanguageConfig(state);
-    // @ts-ignore
     return state.selection.ranges
         .filter(range => range.empty)
         .map(range => {
@@ -468,7 +506,7 @@ function getCursorTooltips(state) {
             strictSide: false,
             arrow: true,
             create: () => {
-                let dom = document.createElement("div");
+                const dom = document.createElement("div");
                 dom.className = "cm-tooltip-cursor";
                 dom.textContent = `${argName}`;
                 return { dom };

package/dist/index.js

@@ -6,7 +6,7 @@ import { insertCompletionText } from '@codemirror/autocomplete';
 import { StateField } from '@codemirror/state';
 import { showTooltip, hoverTooltip, EditorView } from '@codemirror/view';
 
-// @ts-ignore
+// @ts-expect-error TS2739
 const t = {
     deserialize: (str) => str,
 };
@@ -114,7 +114,7 @@ const resolveIdentifier = (nodeTypeId, identifier, config) => {
 };
 function resolveTypes(state, node, config) {
     var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l;
-    let types = new Set();
+    const types = new Set();
     if (!node) {
         return types;
     }
@@ -127,19 +127,19 @@ function resolveTypes(state, node, config) {
     }
     else if (node.type.is(Variable)) {
         const varName = state.sliceDoc(node.from, node.to) || '';
-        // @ts-ignore
+        // @ts-expect-error TS2339
         (_b = (_a = resolveIdentifier(node.type.id, varName, config)) === null || _a === void 0 ? void 0 : _a.type) === null || _b === void 0 ? void 0 : _b.forEach((x) => types.add(x));
     }
     else if (node.type.is(Function)) {
         const varName = state.sliceDoc(node.from, node.to) || '';
-        // @ts-ignore
+        // @ts-expect-error TS2339
         (_d = (_c = resolveIdentifier(node.type.id, varName, config)) === null || _c === void 0 ? void 0 : _c.returnType) === null || _d === void 0 ? void 0 : _d.forEach((x) => types.add(x));
     }
     else if (node.type.is(PropertyAccess) && node.firstChild && ((_e = node.lastChild) === null || _e === void 0 ? void 0 : _e.type.is(Property))) {
         const varName = state.sliceDoc(node.lastChild.from, node.lastChild.to) || '';
         (_f = resolveTypes(state, node.firstChild, config)) === null || _f === void 0 ? void 0 : _f.forEach(baseType => {
             var _a, _b, _c, _d;
-            // @ts-ignore
+            // @ts-expect-error TS2339
             (_d = (_c = resolveIdentifier((_a = node.lastChild) === null || _a === void 0 ? void 0 : _a.type.id, varName, (_b = config.types) === null || _b === void 0 ? void 0 : _b[baseType])) === null || _c === void 0 ? void 0 : _c.type) === null || _d === void 0 ? void 0 : _d.forEach((x) => types.add(x));
         });
     }
@@ -147,10 +147,22 @@ function resolveTypes(state, node, config) {
         const varName = state.sliceDoc(node.lastChild.from, node.lastChild.to) || '';
         (_h = resolveTypes(state, node.firstChild, config)) === null || _h === void 0 ? void 0 : _h.forEach(baseType => {
             var _a, _b, _c, _d;
-            // @ts-ignore
+            // @ts-expect-error TS2339
             (_d = (_c = resolveIdentifier((_a = node.lastChild) === null || _a === void 0 ? void 0 : _a.type.id, varName, (_b = config.types) === null || _b === void 0 ? void 0 : _b[baseType])) === null || _c === void 0 ? void 0 : _c.returnType) === null || _d === void 0 ? void 0 : _d.forEach((x) => types.add(x));
         });
     }
+    // Array indexing: for typed arrays (e.g. Foo[]) return element type, for generic arrays return any
+    else if (node.type.is(ArrayAccess) && node.firstChild) {
+        const left = node.firstChild.node;
+        resolveTypes(state, left, config).forEach(baseType => {
+            if (baseType.endsWith('[]')) {
+                types.add(baseType.slice(0, -2));
+            }
+            else if (baseType === ELScalar.Array) {
+                types.add(ELScalar.Any);
+            }
+        });
+    }
     else if (node.type.is(Application) && node.firstChild) {
         resolveTypes(state, node.firstChild, config).forEach(x => types.add(x));
     }
@@ -219,13 +231,13 @@ var utils = /*#__PURE__*//*@__PURE__*/Object.freeze({
  */
 const expressionLanguageLinterSource = (state) => {
     const config = getExpressionLanguageConfig(state);
-    let diagnostics = [];
+    const diagnostics = [];
     syntaxTree(state).cursor().iterate(node => {
         var _a, _b, _c, _d, _e, _f;
         const { from, to, type: { id } } = node;
         let identifier;
         switch (id) {
-            case 0:
+            case 0: {
                 if (state.doc.length === 0 || from === 0) {
                     // Don't show error on empty doc (even though it is an error)
                     return;
@@ -239,7 +251,8 @@ const expressionLanguageLinterSource = (state) => {
                     diagnostics.push({ from, to, severity: 'error', message: `Unexpected ${type} <code>${identifier}</code>` });
                 }
                 return;
-            case Arguments:
+            }
+            case Arguments: {
                 const fn = resolveFunctionDefinition(node.node.prevSibling, state, config);
                 const args = fn === null || fn === void 0 ? void 0 : fn.args;
                 if (!args) {
@@ -260,7 +273,12 @@ const expressionLanguageLinterSource = (state) => {
                     const typesUsed = Array.from(resolveTypes(state, n, config));
                     const typesExpected = args[i].type;
                     if (typesExpected && !typesExpected.includes(ELScalar.Any) && !typesUsed.some(x => typesExpected.includes(x))) {
-                        diagnostics.push({ from: n.from, to: n.to, severity: 'error', message: `<code>${typesExpected.join('|')}</code> expected, got <code>${typesUsed.join('|')}</code>` });
+                        diagnostics.push({
+                            from: n.from,
+                            to: n.to,
+                            severity: 'error',
+                            message: `<code>${typesExpected.join('|')}</code> expected, got <code>${typesUsed.join('|')}</code>`
+                        });
                     }
                     i++;
                 }
@@ -268,8 +286,9 @@ const expressionLanguageLinterSource = (state) => {
                     diagnostics.push({ from: node.from, to: node.to, severity: 'error', message: `Too few arguments – ${argumentCountHintFn()}` });
                 }
                 break;
+            }
             case Property:
-            case Method:
+            case Method: {
                 const leftArgument = (_e = (_d = node.node.parent) === null || _d === void 0 ? void 0 : _d.firstChild) === null || _e === void 0 ? void 0 : _e.node;
                 const types = Array.from(resolveTypes(state, leftArgument, config));
                 identifier = state.sliceDoc(from, to);
@@ -277,26 +296,46 @@ const expressionLanguageLinterSource = (state) => {
                     diagnostics.push({ from, to, severity: 'error', message: `${node.name} <code>${identifier}</code> not found in <code>${types.join('|')}</code>` });
                 }
                 break;
+            }
             case Variable:
-            case Function:
+            case Function: {
                 identifier = state.sliceDoc(from, node.node.firstChild ? node.node.firstChild.from - 1 : to);
                 if (!resolveIdentifier(id, identifier, config)) {
                     diagnostics.push({ from, to, severity: 'error', message: `${node.node.name} <code>${identifier}</code> not found` });
                 }
                 break;
-            case BinaryExpression:
+            }
+            case BinaryExpression: {
                 const operatorNode = node.node.getChild(OperatorKeyword);
                 if (operatorNode) {
                     const operator = state.sliceDoc(operatorNode.from, operatorNode.to);
+                    const leftArgument = node.node.firstChild;
+                    const rightArgument = node.node.lastChild;
                     if (operator === 'in') {
-                        const rightArgument = node.node.lastChild;
-                        const types = resolveTypes(state, rightArgument, config);
-                        if (!types.has(ELScalar.Array)) {
-                            diagnostics.push({ from: rightArgument.from, to: rightArgument.to, severity: 'error', message: `<code>${ELScalar.Array}</code> expected, got <code>${[...types].join('|')}</code>` });
+                        const leftTypes = resolveTypes(state, leftArgument, config);
+                        const rightTypes = resolveTypes(state, rightArgument, config);
+                        const allowsAny = rightTypes.has(ELScalar.Array) || rightTypes.has(ELScalar.Any);
+                        if (!allowsAny && ![...rightTypes].some(x => x.endsWith('[]'))) {
+                            diagnostics.push({ from: rightArgument.from, to: rightArgument.to, severity: 'error', message: `<code>${ELScalar.Array}</code> expected, got <code>${[...rightTypes].join('|')}</code>` });
+                        }
+                        else if (!allowsAny && !rightTypes.has(`${ELScalar.Any}[]`) && ![...leftTypes].some(type => rightTypes.has(`${type}[]`))) {
+                            diagnostics.push({ from: leftArgument.from, to: rightArgument.to, severity: 'warning', message: `Expression is always <code>false</code> because <code>${[...leftTypes].join('|')}</code> not found in <code>${[...rightTypes].join('|')}</code>` });
+                        }
+                    }
+                    else if (["contains", "starts with", "ends with", "matches"].includes(operator)) {
+                        // Both sides must be string
+                        const leftTypes = resolveTypes(state, leftArgument, config);
+                        const rightTypes = resolveTypes(state, rightArgument, config);
+                        if (!leftTypes.has(ELScalar.String)) {
+                            diagnostics.push({ from: leftArgument.from, to: leftArgument.to, severity: 'error', message: `<code>string</code> expected, got <code>${[...leftTypes].join('|')}</code>` });
+                        }
+                        if (!rightTypes.has(ELScalar.String)) {
+                            diagnostics.push({ from: rightArgument.from, to: rightArgument.to, severity: 'error', message: `<code>string</code> expected, got <code>${[...rightTypes].join('|')}</code>` });
                         }
                     }
                 }
                 break;
+            }
         }
         if (identifier && ((_f = node.node.parent) === null || _f === void 0 ? void 0 : _f.type.isError)) {
             diagnostics.push({ from, to, severity: 'error', message: `Unexpected identifier <code>${identifier}</code>` });
@@ -368,7 +407,7 @@ function completeIdentifier(state, config, tree, from, to) {
         validFor: /^[a-zA-Z_]+[a-zA-Z_0-9]*$/,
     };
 }
-function completeMember(state, config, tree, from, to, explicit) {
+function completeMember(state, config, tree, from, to) {
     var _a, _b, _c, _d, _e, _f;
     if (!(((_a = tree.parent) === null || _a === void 0 ? void 0 : _a.type.is(PropertyAccess)) || ((_b = tree.parent) === null || _b === void 0 ? void 0 : _b.type.is(MethodAccess))) || !((_c = tree.parent) === null || _c === void 0 ? void 0 : _c.firstChild)) {
         return null;
@@ -377,7 +416,7 @@ function completeMember(state, config, tree, from, to, explicit) {
     if (!(types === null || types === void 0 ? void 0 : types.size)) {
         return null;
     }
-    let options = [];
+    const options = [];
     for (const type of types) {
         const typeDeclaration = (_d = config.types) === null || _d === void 0 ? void 0 : _d[type];
         options.push(...(((_e = typeDeclaration === null || typeDeclaration === void 0 ? void 0 : typeDeclaration.identifiers) === null || _e === void 0 ? void 0 : _e.map(autocompleteIdentifier)) || []), ...(((_f = typeDeclaration === null || typeDeclaration === void 0 ? void 0 : typeDeclaration.functions) === null || _f === void 0 ? void 0 : _f.map(autocompleteFunction)) || []));
@@ -439,7 +478,6 @@ function resolveArguments(node) {
 }
 function getCursorTooltips(state) {
     const config = getExpressionLanguageConfig(state);
-    // @ts-ignore
     return state.selection.ranges
         .filter(range => range.empty)
         .map(range => {
@@ -465,7 +503,7 @@ function getCursorTooltips(state) {
             strictSide: false,
             arrow: true,
             create: () => {
-                let dom = document.createElement("div");
+                const dom = document.createElement("div");
                 dom.className = "cm-tooltip-cursor";
                 dom.textContent = `${argName}`;
                 return { dom };

package/package.json

@@ -4,7 +4,9 @@
   "scripts": {
     "pretest": "npm run-script prepare",
     "test": "cm-runtests",
-    "prepare": "cm-buildhelper src/index.ts"
+    "prepare": "cm-buildhelper src/index.ts",
+    "lint-fix": "eslint . --fix --ext .ts,.js",
+    "lint": "eslint . --ext .ts,.js"
   },
   "type": "module",
   "main": "dist/index.cjs",
@@ -28,7 +30,10 @@
     "@codemirror/buildhelper": "^1.0.0",
     "@types/mocha": "^10.0.10",
     "@types/node": "^22.10.5",
-    "tsdoc-markdown": "^1.1.1"
+    "tsdoc-markdown": "^1.1.1",
+    "eslint": "^8.48.0",
+    "@typescript-eslint/parser": "^6.5.0",
+    "@typescript-eslint/eslint-plugin": "^6.5.0"
   },
   "license": "MIT",
   "repository": {
@@ -39,5 +44,5 @@
     "access": "public",
     "registry": "https://registry.npmjs.org/"
   },
-  "version": "1.1.0"
+  "version": "1.3.0"
 }