Ryan Schultz
8afacf268a
- Hover provider showing entity information and type - Go-to-definition (F12) for entity references - Basic IFC file validation (ISO-10303-21 header check) - Entity parsing with regex-based detection - Proper CommonJS module system (avoiding ES module issues) This replaces the broken baseline from ifc-developer-tools which had: - Non-functional ES module configuration - Circular dependency issues - Parser crashes - Non-working PositionVisitor Built on Microsoft's LSP example template for a clean, maintainable foundation. Next: Add hierarchical entity dependency tree in hover tooltip."
55 lines
1.6 KiB
JavaScript
55 lines
1.6 KiB
JavaScript
'use strict';
|
|
|
|
var utils = require('../utils.js');
|
|
require('eslint-visitor-keys');
|
|
require('espree');
|
|
require('estraverse');
|
|
|
|
function isConditional(node) {
|
|
return node.type === "ConditionalExpression";
|
|
}
|
|
var noConfusingArrow = utils.createRule({
|
|
name: "no-confusing-arrow",
|
|
package: "js",
|
|
meta: {
|
|
type: "layout",
|
|
docs: {
|
|
description: "Disallow arrow functions where they could be confused with comparisons"
|
|
},
|
|
fixable: "code",
|
|
schema: [{
|
|
type: "object",
|
|
properties: {
|
|
allowParens: { type: "boolean", default: true },
|
|
onlyOneSimpleParam: { type: "boolean", default: false }
|
|
},
|
|
additionalProperties: false
|
|
}],
|
|
messages: {
|
|
confusing: "Arrow function used ambiguously with a conditional expression."
|
|
}
|
|
},
|
|
create(context) {
|
|
const config = context.options[0] || {};
|
|
const allowParens = config.allowParens || config.allowParens === void 0;
|
|
const onlyOneSimpleParam = config.onlyOneSimpleParam;
|
|
const sourceCode = context.sourceCode;
|
|
function checkArrowFunc(node) {
|
|
const body = node.body;
|
|
if (isConditional(body) && !(allowParens && utils.isParenthesised(sourceCode, body)) && !(onlyOneSimpleParam && !(node.params.length === 1 && node.params[0].type === "Identifier"))) {
|
|
context.report({
|
|
node,
|
|
messageId: "confusing",
|
|
fix(fixer) {
|
|
return allowParens ? fixer.replaceText(node.body, `(${sourceCode.getText(node.body)})`) : null;
|
|
}
|
|
});
|
|
}
|
|
}
|
|
return {
|
|
ArrowFunctionExpression: checkArrowFunc
|
|
};
|
|
}
|
|
});
|
|
|
|
module.exports = noConfusingArrow;
|