@sap/cds-dk 7.0.2 → 7.0.3

package/lib/init/template/html5-repo/index.js

@@ -13,7 +13,7 @@ module.exports = class Html5RepoTemplate extends require('../templateBase') {
     }
 
     async canRun() {
-        const project = await readProject(this.options)
+        const project = await readProject()
         const { hasMta, hasHelm } = project
         if (!hasHelm && hasMta) {
             throw `'cds add html5-repo' is not available for Cloud Foundry yet`
@@ -26,12 +26,8 @@ module.exports = class Html5RepoTemplate extends require('../templateBase') {
         return exists(join('chart', 'templates', 'html5-apps-deployer-configmap.yaml'))
     }
 
-    async run() {
-        await this.runDependentMerging()
-    }
-
     async runDependentMerging(context = {}) {
-        const { appName, hasHelm } = await readProject(this.options)
+        const { appName, hasHelm } = await readProject()
 
         if (hasHelm) {
             //in case facet is being added to the already exisiting charts folder
@@ -78,7 +74,7 @@ module.exports = class Html5RepoTemplate extends require('../templateBase') {
 
             // copy content deployment chart if it is not present
             const HelmTemplate = require(`../helm`);
-            const helmTemplate = new HelmTemplate(this.generator, context.oldTrackingData, context.newTrackingData);
+            const helmTemplate = new HelmTemplate(context.oldTrackingData, context.newTrackingData);
             isIndependentCommand = isIndependentCommand | await helmTemplate.addSubChart('content-deployment');
 
             // add dependency entry in chart.yaml

package/lib/init/template/java/index.js

@@ -23,13 +23,13 @@ module.exports = class JavaTemplate extends require('../templateBase') {
     }
 
     async run() {
-        const { cmdLine, artifactId, archetypeVersion } = await mvn.getGenerateCmdArgs(this.projectName, this.options)
+        const { cmdLine, artifactId, archetypeVersion } = await mvn.getGenerateCmdArgs(path.basename(cds.root))
         console.log(`Using Maven archetype version ${archetypeVersion}`)
 
-        const temp = await fs.promises.mkdtemp(join(os.tmpdir(), `${this.projectName}_`))
+        const temp = await fs.promises.mkdtemp(join(os.tmpdir(), `${path.basename(cds.root)}_`))
         try {
             await cmd.spawnCommand('mvn', cmdLine, { cwd: temp })
-            await copy(join(temp, artifactId), this.projectPath);
+            await copy(join(temp, artifactId), cds.root);
         } catch (err) {
             if (err.code === 'ENOENT' && err.path === 'mvn') {
                 throw `Maven executable 'mvn' not found, follow ${term.info(URLS.MAVEN_INSTALL_HELP)} and install Maven on your machine.`

package/lib/init/template/kibana/index.js

@@ -10,21 +10,20 @@ module.exports = class KibanaTemplate extends require('../templateBase') {
     }
 
     async canRun() {
-        const { hasMta, hasHelm, isJava } = await readProject(this.options)
+        const { hasMta, hasHelm, isJava } = await readProject()
         if (hasHelm && !hasMta) throw `'cds add kibana' is not available for Kyma yet`
         if (isJava) throw `'cds add kibana' is not available for Java yet`
         return true
     }
 
     async run() {
-        const project = await readProject(this.options)
+        const project = await readProject()
         const { configFile } = project
         await mergeJSON(configFile, join(__dirname, 'files', 'package.json.hbs'), project)
-        await this.runDependentMerging()
     }
 
     async runDependentMerging() {
-        const project = await readProject(this.options)
+        const project = await readProject()
         const { hasMta, hasApprouter, isJava, isMultitenant, srvPath } = project
 
         if (hasMta) {

package/lib/init/template/lint/index.js

@@ -19,8 +19,8 @@ const VSCODE_FILE_ASSOCIATIONS = {
 }
 
 module.exports = class LintTemplate extends require('../templateBase') {
-  constructor(generator) {
-    super(generator);
+  constructor() {
+    super();
     this.customPath = path.join(cds.root, ".eslint");
     this.docsPath = path.join(this.customPath, "docs");
     this.rulesPath = path.join(this.customPath, "rules");
@@ -32,13 +32,14 @@ module.exports = class LintTemplate extends require('../templateBase') {
   }
 
   async canRun() {
-    if (this.options.lint) {
+    const { lint } = cds.cli.options
+    if (lint) {
       // If custom rule example is requested, also trigger
       // "custom" rule type of setup
-      if (this.options.lint.has(LINT_DEV)) {
+      if (lint.has(LINT_DEV)) {
         this.customRuleExample = true;
       }
-      if (this.options.lint.has(LINT_GLOBAL)) {
+      if (lint.has(LINT_GLOBAL)) {
         this.lintType = "global";
       }
     }
@@ -98,7 +99,7 @@ module.exports = class LintTemplate extends require('../templateBase') {
           "names"
         ].join(" ")}`;
 
-        if (this.options.force) {
+        if (cds.cli.options.force) {
           LOG.info(
             `\n\nAlmost done - installing ${this.missingNpmDependencies["names"].length} npm dependencies:\n${task}`
           );
@@ -187,7 +188,7 @@ module.exports = class LintTemplate extends require('../templateBase') {
         clearTimeout(timerID);
         received = answer.trim().toLowerCase();
         // If user confirms or force flag is set, install dependencies
-        if (["y", "yes"].includes(received) || this.options.force) {
+        if (["y", "yes"].includes(received) || cds.cli.options.force) {
           resolve(this._runTask(task));
           // If use declines or question timeout is reached, print todos
         } else if (["n", "no"].includes(received.toLowerCase())) {

package/lib/init/template/mta/index.js

@@ -7,7 +7,8 @@ const { mergeJSON, sort } = require('../../util/merge')
 
 module.exports = class MtaTemplate extends require('../templateBase') {
 
-    static hasFacet(_, options) {
+    static hasFacet() {
+        const { options } = cds.cli
         if (!options?.add.has('mta') && options?.add.has('helm')) return false
         return exists('mta.yaml')
     }
@@ -17,7 +18,7 @@ module.exports = class MtaTemplate extends require('../templateBase') {
     }
 
     async run() {
-        const project = await readProject(this.options)
+        const project = await readProject()
         if (!exists('mta.yaml')) {
             await copyRenderedYAML(join(__dirname, 'files', 'mta.yaml.hbs'), 'mta.yaml', project)
         }

package/lib/init/template/multitenancy/files/package.sidecar.json.hbs

@@ -13,7 +13,7 @@
   },
   "scripts": {
     "start": "cds-serve"{{#isJava}},
-    "build": "cds build ../.. --for mtx-sidecar --production && cd gen && npm ci"{{/isJava}}
+    "build": "cds build ../.. --for mtx-sidecar --production && npm ci --prefix gen"{{/isJava}}
   },
   "cds": {
     "profile": "mtx-sidecar"

package/lib/init/template/multitenancy/index.js

@@ -15,8 +15,9 @@ const {
 
 module.exports = class MultitenancyTemplate extends require('../templateBase') {
 
-  static hasFacet(env, options) {
+  static hasFacet(env) {
     // REVISIT: Check if this can be simplified
+    const { options } = cds.cli.options
     return options?.add?.has('mtx') || options?.add?.has('multitenancy') ||
            !!env.requires?.multitenancy || !!env.requires?.db?.multiTenant ||
            exists(join('mtx', 'sidecar')) || false
@@ -27,7 +28,7 @@ module.exports = class MultitenancyTemplate extends require('../templateBase') {
   }
 
   async run() {
-    const project = await readProject(this.options)
+    const project = await readProject()
     const { isNodejs, isJava, configFile } = project
     await mergeJSON(configFile, join(__dirname, 'files', `package.json.hbs`), project)
     if (isJava) {
@@ -37,11 +38,10 @@ module.exports = class MultitenancyTemplate extends require('../templateBase') {
     }
     if (isNodejs) await sort('package.json', 'dependencies')
     await copyRenderedJSON(join(__dirname, 'files', 'package.sidecar.json.hbs'), join('mtx', 'sidecar', 'package.json'), project)
-    await this.runDependentMerging()
   }
 
   async runDependentMerging(context = {}) {
-    const project = await readProject(this.options)
+    const project = await readProject()
     const { isNodejs, isJava, hasMta, hasHelm, hasXsuaa, hasHana, srvPath } = project
 
     if (hasMta) {
@@ -93,11 +93,7 @@ module.exports = class MultitenancyTemplate extends require('../templateBase') {
         context.oldTrackingData = context.newTrackingData = await parseMd5File(join('chart', '.cds-add-helm-files.md5'))
       }
 
-      await mergeYAML(
-        join('chart', 'values.yaml'),
-        join(__dirname, 'files', 'values.yaml.hbs'),
-        project
-      )
+      await mergeYAML(join('chart', 'values.yaml'), join(__dirname, 'files', 'values.yaml.hbs'), project)
 
       const envFromObject = {
         envFrom: [
@@ -130,7 +126,7 @@ module.exports = class MultitenancyTemplate extends require('../templateBase') {
 
       // REVISIT: Don't reference other templates except for the safe `runDependentMerging` -> low coupling, high cohesion
       const HelmTemplate = require(`../helm`);
-      const helmTemplate = new HelmTemplate(this.generator, context.oldTrackingData, context.newTrackingData);
+      const helmTemplate = new HelmTemplate(context.oldTrackingData, context.newTrackingData);
 
       await helmTemplate.addDependency('web-application', 'sidecar');
 
@@ -159,33 +155,16 @@ module.exports = class MultitenancyTemplate extends require('../templateBase') {
     }
 
     if (hasXsuaa) {
-      await mergeJSON(
-        'xs-security.json',
-        join(__dirname, 'files', 'xs-security.json.hbs'),
-        project,
-        {
-          additions: [{
-            in: 'scopes',
-            where: [{ property: 'name', isEqualTo: '$XSAPPNAME.mtcallback' }],
-          }]
-        }
-      )
-    }
-
-    if (hasXsuaa && hasHelm) {
-      await mergeYAML(
-          join('chart', 'values.yaml'),
-          {
-              xsuaa: {
-                  parameters: {
-                      'tenant-mode': 'shared'
-                  }
-              }
-          },
-          null,
-          { forceOverwrite: true }
-      )
-      if (isIndependentCommand) await writeMd5File(join('chart', '.cds-add-helm-files.md5'), context.newTrackingData)
+      await mergeJSON('xs-security.json',join(__dirname, 'files', 'xs-security.json.hbs'), project, {
+        additions: [{
+          in: 'scopes',
+          where: [{ property: 'name', isEqualTo: '$XSAPPNAME.mtcallback' }],
+        }]
+      })
+      if (hasHelm) {
+        await mergeYAML(join('chart', 'values.yaml'), { xsuaa: { parameters: { 'tenant-mode': 'shared' }}}, null, { forceOverwrite: true })
+        if (isIndependentCommand) await writeMd5File(join('chart', '.cds-add-helm-files.md5'), context.newTrackingData)
+      }
     }
   }
 }

package/lib/init/template/nodejs/index.js

@@ -1,6 +1,6 @@
 const { join } = require('path')
 const cds = require('../../../cds')
-const { exists, mkdirp } = cds.utils
+const { exists, mkdirp, path } = cds.utils
 const { URLS, OPTIONS: { NODEJS, JAVA }, COMMAND_ADD } = require('../../constants')
 const term = require('../../../util/term')
 const { copyFiles } = require('../../util/template')
@@ -24,8 +24,7 @@ module.exports = class NodejsTemplate extends require('../templateBase') {
 
     async run() {
         const { db, srv, app } = (await this.getEnv()).folders
-        const { projectPath, projectName } = this
-        await copyFiles(join(__dirname, 'files'), projectPath, { projectName }, this.options.force)
+        await copyFiles(join(__dirname, 'files'), cds.root, { projectName: path.basename(cds.root) }, cds.cli.options.force)
         await Promise.all([db, srv, app].map(p => mkdirp(p)))
     }
 

package/lib/init/template/pipeline/index.js

@@ -9,7 +9,7 @@ module.exports = class PipelineTemplate extends require('../templateBase') {
 
 
     async canRun() {
-        if (this.options.force) {
+        if (cds.cli.options.force) {
             return true;
         }
 
@@ -21,7 +21,7 @@ module.exports = class PipelineTemplate extends require('../templateBase') {
     }
 
     async run() {
-        await copyFiles(join(__dirname, 'files'), cds.root, {}, this.options.force);
+        await copyFiles(join(__dirname, 'files'), cds.root, {}, true);
     }
 
     async finalize() {

package/lib/init/template/redis-messaging/index.js

@@ -8,7 +8,7 @@ const { srvNode4, srvJava4, redis } = require('../_merging/registry-mta')
 module.exports = class RedisMessagingTemplate extends require('../templateBase') {
 
   async canRun() {
-    const { hasMta, hasHelm, isJava } = await readProject(this.options)
+    const { hasMta, hasHelm, isJava } = await readProject()
     if (isJava)             throw `'cds add redis-messaging' is not available for Java yet`
     if (hasHelm && !hasMta) throw `'cds add redis-messaging' is not available for Kyma yet`
     return true
@@ -19,15 +19,14 @@ module.exports = class RedisMessagingTemplate extends require('../templateBase')
   }
 
   async run() {
-    const project = await readProject(this.options)
+    const project = await readProject()
     const { configFile } = project
     project.shortcut = !(await read(configFile)).cds?.requires?.messaging?.kind
     await mergeJSON(configFile, join(__dirname, 'files', 'package.json.hbs'), project)
-    await this.runDependentMerging()
   }
 
   async runDependentMerging() {
-    const project = await readProject(this.options)
+    const project = await readProject()
     const { hasMta, isNodejs, srvPath } = project
 
     if (hasMta) {

package/lib/init/template/sample/files/app/admin-books/webapp/manifest.json

@@ -11,7 +11,7 @@
       "i18n": "i18n/i18n.properties",
       "dataSources": {
         "AdminService": {
-          "uri": "/admin/",
+          "uri": "/odata/v4/admin/",
           "type": "OData",
           "settings": {
             "odataVersion": "4.0"

package/lib/init/template/sample/files/app/browse/webapp/manifest.json

@@ -11,7 +11,7 @@
     "i18n": "i18n/i18n.properties",
     "dataSources": {
       "CatalogService": {
-        "uri": "/browse/",
+        "uri": "/odata/v4/catalog/",
         "type": "OData",
         "settings": {
           "odataVersion": "4.0"

package/lib/init/template/sample/files/srv/admin-service.cds

@@ -1,5 +1,5 @@
 using { sap.capire.bookshop as my } from '../db/schema';
-service AdminService @(requires:'admin', path:'/admin') {
+service AdminService @(requires:'admin') {
   entity Books as projection on my.Books;
   entity Authors as projection on my.Authors;
 }

package/lib/init/template/sample/files/srv/cat-service.cds

@@ -1,5 +1,5 @@
 using { sap.capire.bookshop as my } from '../db/schema';
-service CatalogService @(path:'/browse') {
+service CatalogService {
 
   /** For displaying lists of Books */
   @readonly entity ListOfBooks as projection on Books

package/lib/init/template/sample/index.js

@@ -6,7 +6,7 @@ const { copyFiles } = require('../../util/template')
 module.exports = class SampleTemplate extends require('../templateBase') {
 
     async canRun() {
-        const { isJava } = await readProject(this.options)
+        const { isJava } = await readProject()
         if (isJava) throw `'cds add sample' is not available for Java yet`
         return true
     }
@@ -14,11 +14,12 @@ module.exports = class SampleTemplate extends require('../templateBase') {
     async run() {
         const { db, srv, app } = (await this.getEnv()).folders
         const language = exists('pom.xml') ? 'java' : 'nodejs'
+        const { force } = cds.cli.options
         await Promise.all([
-          copyFiles(join(__dirname, 'files', 'db'),  db, {}, this.options.force),
-          copyFiles(join(__dirname, 'files', 'app'), app, {}, this.options.force),
-          copyFiles(join(__dirname, 'files', 'srv'), srv, {}, this.options.force),
-          copyFiles(join(__dirname, 'files', language), srv, {}, this.options.force)
+          copyFiles(join(__dirname, 'files', 'db'),  db, {}, force),
+          copyFiles(join(__dirname, 'files', 'app'), app, {}, force),
+          copyFiles(join(__dirname, 'files', 'srv'), srv, {}, force),
+          copyFiles(join(__dirname, 'files', language), srv, {}, force)
         ])
     }
 }

package/lib/init/template/sqlite/index.js

@@ -7,7 +7,7 @@ const { mergeJSON } = require('../../util/merge')
 module.exports = class SQLiteTemplate extends require('../templateBase') {
 
     async run() {
-      const project = await readProject(this.options)
+      const project = await readProject()
       const { configFile } = project
       project.shortcut = !(await read(configFile)).cds?.requires?.db?.kind
       await mergeJSON(configFile, join(__dirname, 'files', `package.json.hbs`), project)

package/lib/init/template/templateBase.js

@@ -1,25 +1,14 @@
-const cds = require('../../cds');
-const path = require('path');
-
-const { PROJECT_TYPE } = require('../constants');
-const { exists } = require('../../cds').utils;
+const cds = require('../../cds')
+const { exists } = require('../../cds').utils
+const { PROJECT_TYPE } = require('../constants')
 
 module.exports = class TemplateBase {
+
     /**
      * @constructor
      */
-    constructor(generator) {
-        if (generator.options?.for && typeof generator.options?.for !== 'string') {
-            throw 'The --for argument must not be empty.'
-        }
-        this.projectPath = cds.root;
-        this.options = generator.options || {};
-        this.generator = generator;
+    constructor() {
         this.name = this.constructor.name.replace(/template/i, '').replace(/([a-z0-9])([A-Z])/g, '$1-$2').toLowerCase()
-
-        this.cwd = this.options.cwd || process.cwd();
-
-        this.projectName = path.basename(cds.root);
     }
 
     /**
@@ -51,7 +40,7 @@ module.exports = class TemplateBase {
     }
 
     // eslint-disable-next-line no-unused-vars
-    static hasFacet(env, options) {
+    static hasFacet(env) {
         return true;
     }
 
@@ -85,13 +74,14 @@ module.exports = class TemplateBase {
     }
 
     /**
+     * @abstract
      * Executes steps for related facets that are set up for the project.
      */
     async runRelatedMerging() {
         for (const facet of this.getRelatedFacets()) {
             const Template = require('./'+facet)
-            const template = new Template(this.generator)
-            if (Template.hasFacet(await this.getEnv(), this.options)) await template.runDependentMerging()
+            const template = new Template()
+            if (Template.hasFacet(await this.getEnv())) await template.runDependentMerging()
         }
     }
 

package/lib/init/template/tiny-sample/index.js

@@ -16,11 +16,11 @@ module.exports = class TinySampleTemplate extends require('../templateBase') {
     }
 
     async _addJavaSamples() {
-        if (!this.options.add.has(JAVA)) {
-            const mvnAddSamplesArgs = mvnArchetypes.getAddSamplesCmdArgs(this.options)
+        if (!cds.cli.options.add.has(JAVA)) {
+            const mvnAddSamplesArgs = mvnArchetypes.getAddSamplesCmdArgs()
             try {
                 await commandUtil.spawnCommand('mvn', mvnAddSamplesArgs, { cwd: cds.root })
-                const mvnAddTestsArgs = mvnArchetypes.getAddTestsCmdArgs(this.options)
+                const mvnAddTestsArgs = mvnArchetypes.getAddTestsCmdArgs()
                 await commandUtil.spawnCommand('mvn', mvnAddTestsArgs, { cwd: cds.root })
             } catch (err) {
                 if (err.code === 'ENOENT' && err.path === 'mvn') {
@@ -33,7 +33,7 @@ module.exports = class TinySampleTemplate extends require('../templateBase') {
 
     async _addNodejsSamples() {
         const { db, srv } = (await this.getEnv()).folders
-        await copyFiles(join(__dirname, 'files', 'nodejs', 'db'), db, {}, this.options.force)
-        await copyFiles(join(__dirname, 'files', 'nodejs', 'srv'), srv, { dbFolder: db.replace(/[\\/]+$/, '') }, this.options.force)
+        await copyFiles(join(__dirname, 'files', 'nodejs', 'db'), db, {}, cds.cli.options.force)
+        await copyFiles(join(__dirname, 'files', 'nodejs', 'srv'), srv, { dbFolder: db.replace(/[\\/]+$/, '') }, cds.cli.options.force)
     }
 }

package/lib/init/template/toggles/index.js

@@ -9,10 +9,9 @@ module.exports = class TogglesTemplate extends require('../templateBase') {
   }
 
   async run() {
-    const project = await readProject(this.options)
+    const project = await readProject()
     const { isNodejs , configFile} = project
     await mergeJSON(configFile, join(__dirname, 'files', 'package.json.hbs'), project)
     if (isNodejs) await sort('package.json', 'dependencies')
-    await this.runDependentMerging()
   }
 }

package/lib/init/template/typer/files/xsconfig.json

@@ -1,6 +1,5 @@
 {
   "compilerOptions": {
-    "checkJs": true,
     "moduleResolution": "nodenext",
     "paths": {
       "#cds-models/*": ["./@cds-models/*"]

package/lib/init/template/typer/index.js

@@ -7,7 +7,7 @@ const { exists, read } = require('../../../cds').utils
 
 module.exports = class TyperTemplate extends require('../templateBase') {
   async canRun() {
-    const { isJava } = await readProject(this.options)
+    const { isJava } = await readProject()
     if (isJava) {
       console.warn(warn(`This appears to be a Java project. 'cds add ${TYPER}' is intended to be used in Node.js projects. Proceeding anyway.\n`))
     }

package/lib/init/template/xsuaa/index.js

@@ -17,15 +17,14 @@ module.exports = class XsuaaTemplate extends require('../templateBase') {
     }
 
     async run() {
-        const project = await readProject(this.options)
+        const project = await readProject()
         const { configFile } = project
         project.shortcut = !(await read(configFile)).cds?.requires?.auth?.kind
         await mergeJSON(configFile, join(__dirname, 'files', 'package.json.hbs'), project, { forceOverwrite: true })
-        await this.runDependentMerging()
     }
 
     async runDependentMerging( context = {}) {
-        const project = await readProject(this.options)
+        const project = await readProject()
         const { isMultitenant, hasMta, hasHelm, isJava, srvPath } = project
 
         const cds = require('../../../../lib')
@@ -71,7 +70,7 @@ module.exports = class XsuaaTemplate extends require('../templateBase') {
 
             // copy service-instance chart
             const HelmTemplate = require(`../helm`);
-            const helmTemplate = new HelmTemplate(this.generator, context.oldTrackingData, context.newTrackingData);
+            const helmTemplate = new HelmTemplate(context.oldTrackingData, context.newTrackingData);
             isIndependentCommand = isIndependentCommand | await helmTemplate.addSubChart('service-instance');
 
             // add dependency entry in chart.yaml

package/lib/init/util/indent.js

@@ -0,0 +1,57 @@
+const indentRegEx = new RegExp(/^(\s*)/)
+const leadingSpacesRegEx = new RegExp(/^( +)/)
+const leadingTabsRegEx = new RegExp(/^(\t+)/)
+
+/**
+ * Greatest Common Divisor Euclidean algorithm
+ * @param {number[]} xs - list of number to find the gcd of.
+ * @returns {number} - greatest common divisor of all numbers in xs.
+ */
+function gcd (xs) {
+    const euclid = (a, b) => b === 0 ? a : euclid(b, a % b)
+    const [a, b, ...tail] = xs
+    return tail.length
+        ? gcd([euclid(a, b)].concat(tail))
+        : euclid(a, b ?? 0)  // coalesce in case xs consists of exactly one element
+}
+
+/**
+ * Detects indent used for a file.
+ * The algorithm inspects the indentation of every line to find whether tabs or spaces are more prevalent.
+ * Once that has been determined, it derives the spacing by calculating the gcd of all relevant lines.
+ * I.e. if there are lines indented with 2, 4, or 8 spaces, it will find '  ' as indent.
+ * If lines are indented using 1 or 2 tabs, '\t' will be returned.
+ * TODO: filter out outliers, i.e. if the user used three spaces in one line, should ' ' space be used for spacing? Or should that be ignored?
+ * @param {string[]} file - lines of a file.
+ * @returns {string | null} - the string used to space (e.g. 2 spaces, 4 spaces, 1 tab, etc.) or null if the file was empty.
+ */
+function detectIndent (file) {
+    if (file.length === 0) return null
+    if (file.length === 1) return indentRegEx.exec(file[0])[0]
+
+    const indents = file
+        .map(line => indentRegEx.exec(line)[0])
+        .filter(Boolean)
+        .reduce((bins, indent) => {
+            const type = leadingSpacesRegEx.test(indent)
+                ? 'spaces'
+                : leadingTabsRegEx.test(indent)
+                ? 'tabs'
+                : undefined
+            if (type) {
+                const length = indent.split('').length
+                const bin = bins[type]
+                bin.total++
+                bin.indents[length] = (bin.indents[length] ?? 0) + 1
+            }
+            return bins
+        }, { tabs: { indents: {}, total: 0, character: '\t' }, spaces: { indents: {}, total: 0, character: ' ' } })
+
+    const preferred = indents.tabs.total > indents.spaces.total ? indents.tabs : indents.spaces
+    const width = gcd(Object.keys(preferred.indents))
+    return preferred.character.repeat(width)
+}
+
+module.exports = {
+    detectIndent
+}