@sap/eslint-plugin-cds 2.3.2 → 2.3.5

package/lib/impl/rules/require-2many-oncond.js

@@ -1,31 +0,0 @@
-module.exports = require("../../api").createRule({
-  meta: {
-    docs: {
-      description: `Foreign key information of a \`TO MANY\` relationship must be defined within the target and specified in an \`ON\` condition.`,
-      category: "Model Validation",
-      recommended: true,
-      version: "2.1.0",
-    },
-    severity: "error",
-    type: "problem",
-  },
-  create: function (context) {
-    const m = context.cds.model;  if (!m)  return
-    m.forall((d) => {
-      if (d.name) {
-        if (!d.elements) return;
-        for (const elementName in d.elements) {
-          const element = d.elements[elementName];
-          if (element.is2many && !element.on && typeof element.target === 'string') {
-            const loc = context.cds.getLocation(elementName, element);
-            context.report({
-              message: `You must provide an \`ON\` condition for \`TO MANY\` relationship '${element.name}'.`,
-              loc,
-              file: d.$location.file,
-            });
-          }
-        }
-      }
-    });
-  },
-});

package/lib/impl/rules/rule.hbs

@@ -1,20 +0,0 @@
-// @ts-check
-module.exports = require("../../api").createRule({
-  meta: {
-    docs: {
-      description: "{{description}}",
-      version: "{{version}}"
-      },
-      type:"{{type}}",
-  },
-  create: function(context) {
-    const m = cotext.cds.model;
-    m.forall((d)) => {
-      // Add cds logic here, for example
-      return [{
-        message: "{{messages}}",
-        loc: {{loc}}
-      }];
-    }
-  }
-})

package/lib/impl/rules/sql-cast-suggestion.js

@@ -1,52 +0,0 @@
-module.exports = require("../../api").createRule({
-  meta: {
-    docs: {
-      description: "Should make suggestions for possible missing SQL casts.",
-      category: "Model Validation",
-      recommended: true,
-      version: "1.0.8",
-    },
-    severity: "warn",
-    type: "suggestion",
-    hasSuggestions: true,
-    messages: {
-      missingSQLCast:
-        "Potential issue - Missing SQL cast for column expression?",
-    },
-  },
-  create: function (context) {
-    const m = context.cds.model;
-    if (m) {
-      const view = (d) => d.query;
-      m.foreach(view, (v) => {
-        if (v.query.SET)
-          for (const { SELECT } of v.query.SET.args) {
-            // Only in UNION cases?
-            for (const each of SELECT.columns || []) {
-              const { xpr, cast, $location: location } = each;
-              if (cast && xpr) {
-                if (xpr[0].xpr && xpr[0].xpr && xpr[0].cast) {
-                  continue;
-                } else {
-                  if (context.sourcecode.lines[location.line - 1]) {
-                    const endCol =
-                      context.sourcecode.lines[location.line - 1].length;
-                    const loc = {
-                      start: { line: location.line, column: location.col - 1 },
-                      end: { line: location.line, column: endCol },
-                    };
-                    context.report({
-                      messageId: "missingSQLCast",
-                      loc,
-                      file: location.file,
-                    });
-                  }
-                }
-              }
-            }
-          }
-      });
-    }
-    return context.report;
-  },
-});

package/lib/impl/rules/start-elements-lowercase.js

@@ -1,75 +0,0 @@
-module.exports = require("../../api").createRule({
-  meta: {
-    docs: {
-      description: "Regular element names should start with lowercase letters.",
-      category: "Model Validation",
-      version: "1.0.4",
-    },
-    type: "suggestion",
-    hasSuggestions: true,
-    messages: {
-      startLowercase:
-        "Element name '{{entityName}}.{{elementName}}' should start with a lowercase letter.",
-        fixLowercase:
-        "Start element name with a lowercase letter."
-    },
-    fixable: "code",
-  },
-  create: function (context) {
-    const m = context.cds.model;
-    if (m && m.definitions) {
-      m.forall((d) => {
-        const entityName = d.name;
-        for (const elementName in d.elements) {
-          const element = d.elements[elementName];
-          if (
-            elementName &&
-            !(
-              entityName.startsWith("localized") || entityName.endsWith("texts")
-            )
-          ) {
-            if (
-              elementName.charAt(0) !== elementName.charAt(0).toLowerCase() &&
-              !["ID"].includes(elementName)
-            ) {
-              if (element.$location && element.$location.file) {
-                const file = element.$location.file;
-                const loc = context.cds.getLocation(elementName, element);
-                const fix = (fixer) => {
-                  const elementNameSanitized =
-                    elementName.charAt(0).toLowerCase() + elementName.slice(1);
-                  const rangeEnd = context.sourcecode.getIndexFromLoc({
-                    line: loc.end.line,
-                    column: loc.end.column,
-                  });
-                  const rangeBeg = rangeEnd
-                    ? rangeEnd - elementNameSanitized.length
-                    : 0;
-                  return fixer.replaceTextRange(
-                    [rangeBeg, rangeEnd],
-                    elementNameSanitized
-                  );
-                };
-                context.report({
-                  messageId: "startLowercase",
-                  loc,
-                  file,
-                  data: {
-                    entityName,
-                    elementName,
-                  },
-                  suggest: [
-                    {
-                      messageId: "fixLowercase",
-                      fix,
-                    },
-                  ],
-                });
-              }
-            }
-          }
-        }
-      });
-    }
-  },
-});

package/lib/impl/rules/start-entities-uppercase.js

@@ -1,65 +0,0 @@
-module.exports = require("../../api").createRule({
-  meta: {
-    docs: {
-      description: "Regular entity names should start with uppercase letters.",
-      category: "Model Validation",
-      version: "1.0.4",
-    },
-    type: "suggestion",
-    hasSuggestions: true,
-    messages: {
-      startUppercase:
-        "Entity name '{{entityName}}' should start with an uppercase letter.",
-      fixUppercase: "Start entity name with an uppercase letter.",
-    },
-    fixable: "code",
-  },
-  create: function (context) {
-    const m = context.cds.model;
-    if (m) {
-      m.foreach("entity", (e) => {
-        let entityName = e.name;
-        const names = entityName.split(".");
-        entityName = names[names.length - 1];
-        if (
-          entityName &&
-          !(entityName.startsWith("localized") || entityName.endsWith("texts"))
-        ) {
-          if (entityName.charAt(0) !== entityName.charAt(0).toUpperCase()) {
-            if (e.$location && e.$location.file) {
-              const file = e.$location.file;
-              const loc = context.cds.getLocation(entityName, e);
-              const fix = (fixer) => {
-                const entityNameSanitized =
-                  entityName.charAt(0).toUpperCase() + entityName.slice(1);
-                const rangeEnd = context.sourcecode.getIndexFromLoc({
-                  line: loc.end.line,
-                  column: loc.end.column,
-                });
-                const rangeBeg = rangeEnd
-                  ? rangeEnd - entityNameSanitized.length
-                  : 0;
-                return fixer.replaceTextRange(
-                  [rangeBeg, rangeEnd],
-                  entityNameSanitized
-                );
-              };
-              context.report({
-                messageId: "startUppercase",
-                loc,
-                file,
-                data: { entityName },
-                suggest: [
-                  {
-                    messageId: "fixUppercase",
-                    fix,
-                  },
-                ],
-              });
-            }
-          }
-        }
-      });
-    }
-  },
-});

package/lib/impl/types.d.ts

@@ -1,48 +0,0 @@
-import { Rule, RuleTester, SourceCode } from "eslint";
-
-export interface CDSRuleMetaData extends Rule.RuleMetaData {
-    docs: {
-        /** provides the short description of the rule in the [rules index](https://eslint.org/docs/rules/) */
-        description: Rule.RuleMetaData['docs']['description'];
-        /** specifies version of @sap/eslint-plugin-cds at which the rule was first implemented */
-        version: string;
-    };
-    messages?: Rule.RuleMetaData['messages'];
-    fixable?: Rule.RuleMetaData['fixable'];
-    schema?: Rule.RuleMetaData['schema'];
-    deprecated?: Rule.RuleMetaData['deprecated'];
-    type?: Rule.RuleMetaData['type'];
-}
-
-export type CDSReport = Rule.ReportDescriptor & { file?: string };
-
-export interface CDSRuleContext extends Rule.RuleContext {
-    cds: any;
-    configPath: string;
-    code: string;
-    filePath: string;
-    options: [];
-    ruleID: string;
-    sourcecode: SourceCode
-}
-export interface CDSRuleSpec {
-    meta: CDSRuleMetaData,
-    create: (arg0: CDSRuleContext) => Rule.ReportDescriptor;
-}
-
-export interface CDSTestCaseError extends RuleTester.TestCaseError {
-    message: string | RegExp;
-}
-
-export interface CDSRuleTestOpts {
-    /** specifies __dirname */
-    root: string;
-    /** requires your rule .js here */
-    rule?: string;
-    /** filename ('schema.cds' for model, 'package.json' for env) */
-    filename: string;
-    /** resolves cds parser path */
-    parser?: string;
-    /** List of warnings/errors from ESLint's [ruleTester](https://eslint.org/docs/developer-guide/nodejs-api#ruletester) */
-    errors: CDSTestCaseError[]
-}

package/lib/impl/utils/helpers.js

@@ -1,68 +0,0 @@
-const { files, envFiles, modelFiles } = require("../constants");
-
-module.exports = {
-
-  /**
-   * Checks whether the given file path contains a file extension allowed by
-   * the plugin
-   * @param {string} filePath
-   * @returns boolean
-   */
-  isValidFile: function (filePath, key = "") {
-    function genRegex(key) {
-      return new RegExp(
-        `${key
-          .map((file) => {
-            return file.replace("*", "");
-          })
-          .join("$|")}$`
-      );
-    }
-    if (key === "model") {
-      return genRegex(modelFiles).test(filePath);
-    } else if (key === "env") {
-      return genRegex(envFiles).test(filePath);
-    } else {
-      return genRegex(files).test(filePath);
-    }
-  },
-
-  /**
-   * Checks whether the plugin is run via the VS Code ESLint extension (editor)
-   * @returns boolean
-   */
-  isEditor() {
-    return process.argv.join(" ").includes("dbaeumer.vscode-eslint");
-  },
-
-  /**
-   * Returns an array of allowed file extensions
-   * the plugin can parse of the form "*.ext"
-   * @returns {ConfigOverrideFiles} Array of file extensions
-   */
-  getFileExtensions: function () {
-    return files;
-  },
-
-  /**
-   * Prints a formatted message string according to the styles provided
-   * @param msg message to print
-   * @param styles array of styles for apply
-   * @returns
-   */
-  styleText: function (msg, styles) {
-    const types = {
-      reset: "\x1b[0m", // Default
-      bold: "\x1b[1m", // Bold/Bright
-      link: "\x1b[4m", // underline
-      red: "\x1b[31m", // Foreground Red
-      green: "\x1b[32m", // Foreground Green
-      yellow: "\x1b[33m", // Foreground Yellow
-    };
-    let msgStyle = "";
-    styles.forEach((style) => {
-      msgStyle += types[style];
-    });
-    return `${msgStyle}${msg}${types.reset}`;
-  },
-};