skuba 0.0.0-master-20231002013336 → 0.0.0-master-20240206001217

package/lib/cli/lint/external.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../src/cli/lint/external.ts"],
-  "sourcesContent": ["import stream from 'stream';\nimport { inspect } from 'util';\n\nimport { log } from '../../utils/logging';\nimport { throwOnTimeout } from '../../utils/wait';\n\nimport { createAnnotations } from './annotate';\nimport { autofix } from './autofix';\nimport { runESLintInCurrentThread, runESLintInWorkerThread } from './eslint';\nimport {\n  runPrettierInCurrentThread,\n  runPrettierInWorkerThread,\n} from './prettier';\nimport { runTscInNewProcess } from './tsc';\nimport type { Input } from './types';\n\nconst tscPrefixRegex = /^(.*?tsc\\s+\u2502.*?\\s)/gm;\n\nexport class StreamInterceptor extends stream.Transform {\n  private chunks: Uint8Array[] = [];\n\n  public output() {\n    return Buffer.concat(this.chunks).toString().replace(tscPrefixRegex, '');\n  }\n\n  _transform(\n    chunk: Uint8Array,\n    _encoding: BufferEncoding,\n    callback: stream.TransformCallback,\n  ) {\n    this.chunks.push(chunk);\n\n    callback(null, chunk);\n  }\n}\n\nconst lintConcurrently = async ({ tscOutputStream, ...input }: Input) => {\n  const [eslint, prettier, tscOk] = await Promise.all([\n    runESLintInWorkerThread(input),\n    runPrettierInWorkerThread(input),\n    runTscInNewProcess({ ...input, tscOutputStream }),\n  ]);\n\n  return { eslint, prettier, tscOk };\n};\n\n/**\n * Run linting tools `--serial`ly for resource-constrained environments.\n *\n * Note that we still run ESLint and Prettier in worker threads as a\n * counterintuitive optimisation. Memory can be more readily freed on worker\n * thread exit, which isn't as easy with a monolithic main thread.\n */\nconst lintSerially = async ({ tscOutputStream, ...input }: Input) => {\n  const eslint = await runESLintInWorkerThread(input);\n  const prettier = await runPrettierInWorkerThread(input);\n  const tscOk = await runTscInNewProcess({ ...input, tscOutputStream });\n\n  return { eslint, prettier, tscOk };\n};\n\nconst lintSeriallyWithoutWorkerThreads = async (input: Input) => {\n  const eslint = await runESLintInCurrentThread(input);\n  const prettier = await runPrettierInCurrentThread(input);\n  const tscOk = await runTscInNewProcess(input);\n\n  return { eslint, prettier, tscOk };\n};\n\nconst selectLintFunction = (input: Input) => {\n  if (!input.workerThreads) {\n    return lintSeriallyWithoutWorkerThreads;\n  }\n\n  // `--debug` implies `--serial`.\n  const isSerial = input.debug || input.serial;\n\n  return isSerial ? lintSerially : lintConcurrently;\n};\n\nexport const externalLint = async (input: Input) => {\n  const lint = selectLintFunction(input);\n\n  const tscOutputStream = new StreamInterceptor();\n  tscOutputStream.pipe(input.tscOutputStream ?? process.stdout);\n\n  const { eslint, prettier, tscOk } = await lint({ ...input, tscOutputStream });\n\n  try {\n    await throwOnTimeout(\n      createAnnotations(eslint, prettier, tscOk, tscOutputStream),\n      { s: 30 },\n    );\n  } catch (err) {\n    log.warn('Failed to annotate lint results.');\n    log.subtle(inspect(err));\n  }\n\n  if (!eslint.ok || !prettier.ok || !tscOk) {\n    const tools = [\n      ...(eslint.ok ? [] : ['ESLint']),\n      ...(prettier.ok ? [] : ['Prettier']),\n      ...(tscOk ? [] : ['tsc']),\n    ];\n\n    log.newline();\n    log.err(`${tools.join(', ')} found issues that require triage.`);\n\n    process.exitCode = 1;\n  }\n\n  await autofix({\n    debug: input.debug,\n    eslint: eslint.fixable,\n    prettier: !prettier.ok,\n  });\n};\n"],
-  "mappings": ";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,oBAAmB;AACnB,kBAAwB;AAExB,qBAAoB;AACpB,kBAA+B;AAE/B,sBAAkC;AAClC,qBAAwB;AACxB,oBAAkE;AAClE,sBAGO;AACP,iBAAmC;AAGnC,MAAM,iBAAiB;AAEhB,MAAM,0BAA0B,cAAAA,QAAO,UAAU;AAAA,EAC9C,SAAuB,CAAC;AAAA,EAEzB,SAAS;AACd,WAAO,OAAO,OAAO,KAAK,MAAM,EAAE,SAAS,EAAE,QAAQ,gBAAgB,EAAE;AAAA,EACzE;AAAA,EAEA,WACE,OACA,WACA,UACA;AACA,SAAK,OAAO,KAAK,KAAK;AAEtB,aAAS,MAAM,KAAK;AAAA,EACtB;AACF;AAEA,MAAM,mBAAmB,OAAO,EAAE,iBAAiB,GAAG,MAAM,MAAa;AACvE,QAAM,CAAC,QAAQ,UAAU,KAAK,IAAI,MAAM,QAAQ,IAAI;AAAA,QAClD,uCAAwB,KAAK;AAAA,QAC7B,2CAA0B,KAAK;AAAA,QAC/B,+BAAmB,EAAE,GAAG,OAAO,gBAAgB,CAAC;AAAA,EAClD,CAAC;AAED,SAAO,EAAE,QAAQ,UAAU,MAAM;AACnC;AASA,MAAM,eAAe,OAAO,EAAE,iBAAiB,GAAG,MAAM,MAAa;AACnE,QAAM,SAAS,UAAM,uCAAwB,KAAK;AAClD,QAAM,WAAW,UAAM,2CAA0B,KAAK;AACtD,QAAM,QAAQ,UAAM,+BAAmB,EAAE,GAAG,OAAO,gBAAgB,CAAC;AAEpE,SAAO,EAAE,QAAQ,UAAU,MAAM;AACnC;AAEA,MAAM,mCAAmC,OAAO,UAAiB;AAC/D,QAAM,SAAS,UAAM,wCAAyB,KAAK;AACnD,QAAM,WAAW,UAAM,4CAA2B,KAAK;AACvD,QAAM,QAAQ,UAAM,+BAAmB,KAAK;AAE5C,SAAO,EAAE,QAAQ,UAAU,MAAM;AACnC;AAEA,MAAM,qBAAqB,CAAC,UAAiB;AAC3C,MAAI,CAAC,MAAM,eAAe;AACxB,WAAO;AAAA,EACT;AAGA,QAAM,WAAW,MAAM,SAAS,MAAM;AAEtC,SAAO,WAAW,eAAe;AACnC;AAEO,MAAM,eAAe,OAAO,UAAiB;AAClD,QAAM,OAAO,mBAAmB,KAAK;AAErC,QAAM,kBAAkB,IAAI,kBAAkB;AAC9C,kBAAgB,KAAK,MAAM,mBAAmB,QAAQ,MAAM;AAE5D,QAAM,EAAE,QAAQ,UAAU,MAAM,IAAI,MAAM,KAAK,EAAE,GAAG,OAAO,gBAAgB,CAAC;AAE5E,MAAI;AACF,cAAM;AAAA,UACJ,mCAAkB,QAAQ,UAAU,OAAO,eAAe;AAAA,MAC1D,EAAE,GAAG,GAAG;AAAA,IACV;AAAA,EACF,SAAS,KAAK;AACZ,uBAAI,KAAK,kCAAkC;AAC3C,uBAAI,WAAO,qBAAQ,GAAG,CAAC;AAAA,EACzB;AAEA,MAAI,CAAC,OAAO,MAAM,CAAC,SAAS,MAAM,CAAC,OAAO;AACxC,UAAM,QAAQ;AAAA,MACZ,GAAI,OAAO,KAAK,CAAC,IAAI,CAAC,QAAQ;AAAA,MAC9B,GAAI,SAAS,KAAK,CAAC,IAAI,CAAC,UAAU;AAAA,MAClC,GAAI,QAAQ,CAAC,IAAI,CAAC,KAAK;AAAA,IACzB;AAEA,uBAAI,QAAQ;AACZ,uBAAI,IAAI,GAAG,MAAM,KAAK,IAAI,CAAC,oCAAoC;AAE/D,YAAQ,WAAW;AAAA,EACrB;AAEA,YAAM,wBAAQ;AAAA,IACZ,OAAO,MAAM;AAAA,IACb,QAAQ,OAAO;AAAA,IACf,UAAU,CAAC,SAAS;AAAA,EACtB,CAAC;AACH;",
+  "sourcesContent": ["import stream from 'stream';\n\nimport { runESLintInCurrentThread, runESLintInWorkerThread } from './eslint';\nimport {\n  runPrettierInCurrentThread,\n  runPrettierInWorkerThread,\n} from './prettier';\nimport { runTscInNewProcess } from './tsc';\nimport type { Input } from './types';\n\nconst tscPrefixRegex = /^(.*?tsc\\s+\u2502.*?\\s)/gm;\n\nexport class StreamInterceptor extends stream.Transform {\n  private chunks: Uint8Array[] = [];\n\n  public output() {\n    return Buffer.concat(this.chunks).toString().replace(tscPrefixRegex, '');\n  }\n\n  _transform(\n    chunk: Uint8Array,\n    _encoding: BufferEncoding,\n    callback: stream.TransformCallback,\n  ) {\n    this.chunks.push(chunk);\n\n    callback(null, chunk);\n  }\n}\n\nconst lintConcurrently = async ({ tscOutputStream, ...input }: Input) => {\n  const [eslint, prettier, tscOk] = await Promise.all([\n    runESLintInWorkerThread(input),\n    runPrettierInWorkerThread(input),\n    runTscInNewProcess({ ...input, tscOutputStream }),\n  ]);\n\n  return { eslint, prettier, tscOk };\n};\n\n/**\n * Run linting tools `--serial`ly for resource-constrained environments.\n *\n * Note that we still run ESLint and Prettier in worker threads as a\n * counterintuitive optimisation. Memory can be more readily freed on worker\n * thread exit, which isn't as easy with a monolithic main thread.\n */\nconst lintSerially = async ({ tscOutputStream, ...input }: Input) => {\n  const eslint = await runESLintInWorkerThread(input);\n  const prettier = await runPrettierInWorkerThread(input);\n  const tscOk = await runTscInNewProcess({ ...input, tscOutputStream });\n\n  return { eslint, prettier, tscOk };\n};\n\nconst lintSeriallyWithoutWorkerThreads = async (input: Input) => {\n  const eslint = await runESLintInCurrentThread(input);\n  const prettier = await runPrettierInCurrentThread(input);\n  const tscOk = await runTscInNewProcess(input);\n\n  return { eslint, prettier, tscOk };\n};\n\nconst selectLintFunction = (input: Input) => {\n  if (!input.workerThreads) {\n    return lintSeriallyWithoutWorkerThreads;\n  }\n\n  // `--debug` implies `--serial`.\n  const isSerial = input.debug || input.serial;\n\n  return isSerial ? lintSerially : lintConcurrently;\n};\n\nexport const externalLint = async (input: Input) => {\n  const lint = selectLintFunction(input);\n\n  const tscOutputStream = new StreamInterceptor();\n  tscOutputStream.pipe(input.tscOutputStream ?? process.stdout);\n\n  const { eslint, prettier, tscOk } = await lint({ ...input, tscOutputStream });\n\n  return {\n    eslint,\n    prettier,\n    tscOk,\n    tscOutputStream,\n  };\n};\n"],
+  "mappings": ";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,oBAAmB;AAEnB,oBAAkE;AAClE,sBAGO;AACP,iBAAmC;AAGnC,MAAM,iBAAiB;AAEhB,MAAM,0BAA0B,cAAAA,QAAO,UAAU;AAAA,EAC9C,SAAuB,CAAC;AAAA,EAEzB,SAAS;AACd,WAAO,OAAO,OAAO,KAAK,MAAM,EAAE,SAAS,EAAE,QAAQ,gBAAgB,EAAE;AAAA,EACzE;AAAA,EAEA,WACE,OACA,WACA,UACA;AACA,SAAK,OAAO,KAAK,KAAK;AAEtB,aAAS,MAAM,KAAK;AAAA,EACtB;AACF;AAEA,MAAM,mBAAmB,OAAO,EAAE,iBAAiB,GAAG,MAAM,MAAa;AACvE,QAAM,CAAC,QAAQ,UAAU,KAAK,IAAI,MAAM,QAAQ,IAAI;AAAA,QAClD,uCAAwB,KAAK;AAAA,QAC7B,2CAA0B,KAAK;AAAA,QAC/B,+BAAmB,EAAE,GAAG,OAAO,gBAAgB,CAAC;AAAA,EAClD,CAAC;AAED,SAAO,EAAE,QAAQ,UAAU,MAAM;AACnC;AASA,MAAM,eAAe,OAAO,EAAE,iBAAiB,GAAG,MAAM,MAAa;AACnE,QAAM,SAAS,UAAM,uCAAwB,KAAK;AAClD,QAAM,WAAW,UAAM,2CAA0B,KAAK;AACtD,QAAM,QAAQ,UAAM,+BAAmB,EAAE,GAAG,OAAO,gBAAgB,CAAC;AAEpE,SAAO,EAAE,QAAQ,UAAU,MAAM;AACnC;AAEA,MAAM,mCAAmC,OAAO,UAAiB;AAC/D,QAAM,SAAS,UAAM,wCAAyB,KAAK;AACnD,QAAM,WAAW,UAAM,4CAA2B,KAAK;AACvD,QAAM,QAAQ,UAAM,+BAAmB,KAAK;AAE5C,SAAO,EAAE,QAAQ,UAAU,MAAM;AACnC;AAEA,MAAM,qBAAqB,CAAC,UAAiB;AAC3C,MAAI,CAAC,MAAM,eAAe;AACxB,WAAO;AAAA,EACT;AAGA,QAAM,WAAW,MAAM,SAAS,MAAM;AAEtC,SAAO,WAAW,eAAe;AACnC;AAEO,MAAM,eAAe,OAAO,UAAiB;AAClD,QAAM,OAAO,mBAAmB,KAAK;AAErC,QAAM,kBAAkB,IAAI,kBAAkB;AAC9C,kBAAgB,KAAK,MAAM,mBAAmB,QAAQ,MAAM;AAE5D,QAAM,EAAE,QAAQ,UAAU,MAAM,IAAI,MAAM,KAAK,EAAE,GAAG,OAAO,gBAAgB,CAAC;AAE5E,SAAO;AAAA,IACL;AAAA,IACA;AAAA,IACA;AAAA,IACA;AAAA,EACF;AACF;",
   "names": ["stream"]
 }

package/lib/cli/lint/index.d.ts

@@ -1,2 +1,3 @@
+/// <reference types="node" />
 import type { Writable } from 'stream';
-export declare const lint: (args?: string[], tscOutputStream?: Writable | undefined, workerThreads?: boolean) => Promise<void>;
+export declare const lint: (args?: string[], tscWriteable?: Writable | undefined, workerThreads?: boolean) => Promise<void>;

package/lib/cli/lint/index.js

@@ -21,28 +21,60 @@ __export(lint_exports, {
   lint: () => lint
 });
 module.exports = __toCommonJS(lint_exports);
+var import_util = require("util");
 var import_args = require("../../utils/args");
-var import_addEmptyExports = require("../configure/addEmptyExports");
-var import_patchRenovateConfig = require("../configure/patchRenovateConfig");
-var import_patchServerListener = require("../configure/patchServerListener");
-var import_refreshIgnoreFiles = require("../configure/refreshIgnoreFiles");
+var import_logging = require("../../utils/logging");
+var import_packageManager = require("../../utils/packageManager");
+var import_wait = require("../../utils/wait");
+var import_annotate = require("./annotate");
+var import_autofix = require("./autofix");
 var import_external = require("./external");
 var import_internal = require("./internal");
-const lint = async (args = process.argv.slice(2), tscOutputStream = void 0, workerThreads = true) => {
-  await Promise.all([
-    (0, import_addEmptyExports.tryAddEmptyExports)(),
-    (0, import_patchRenovateConfig.tryPatchRenovateConfig)(),
-    (0, import_patchServerListener.tryPatchServerListener)(),
-    (0, import_refreshIgnoreFiles.tryRefreshIgnoreFiles)()
-  ]);
+const lint = async (args = process.argv.slice(2), tscWriteable = void 0, workerThreads = true) => {
   const opts = {
     debug: (0, import_args.hasDebugFlag)(args),
     serial: (0, import_args.hasSerialFlag)(args),
-    tscOutputStream,
+    tscOutputStream: tscWriteable,
     workerThreads
   };
-  await (0, import_external.externalLint)(opts);
-  await (0, import_internal.internalLint)();
+  const { eslint, prettier, tscOk, tscOutputStream } = await (0, import_external.externalLint)(opts);
+  const internal = await (0, import_internal.internalLint)("lint", opts);
+  try {
+    await (0, import_wait.throwOnTimeout)(
+      (0, import_annotate.createAnnotations)(internal, eslint, prettier, tscOk, tscOutputStream),
+      { s: 30 }
+    );
+  } catch (err) {
+    import_logging.log.warn("Failed to annotate lint results.");
+    import_logging.log.subtle((0, import_util.inspect)(err));
+  }
+  if (!internal.ok || !eslint.ok || !prettier.ok || !tscOk) {
+    process.exitCode = 1;
+    const tools = [
+      ...internal.ok ? [] : ["skuba"],
+      ...eslint.ok ? [] : ["ESLint"],
+      ...prettier.ok ? [] : ["Prettier"],
+      ...tscOk ? [] : ["tsc"]
+    ];
+    import_logging.log.err(`${tools.join(", ")} found issues that require triage.`);
+    if (internal.fixable || eslint.fixable || !prettier.ok) {
+      const packageManager = await (0, import_packageManager.detectPackageManager)();
+      import_logging.log.newline();
+      import_logging.log.warn(
+        `Try running ${import_logging.log.bold(
+          packageManager.exec,
+          "skuba",
+          "format"
+        )} to fix them.`
+      );
+    }
+  }
+  await (0, import_autofix.autofix)({
+    debug: opts.debug,
+    eslint: eslint.fixable,
+    prettier: !prettier.ok,
+    internal: internal.fixable
+  });
 };
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {

package/lib/cli/lint/index.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../src/cli/lint/index.ts"],
-  "sourcesContent": ["import type { Writable } from 'stream';\n\nimport { hasDebugFlag, hasSerialFlag } from '../../utils/args';\nimport { tryAddEmptyExports } from '../configure/addEmptyExports';\nimport { tryPatchRenovateConfig } from '../configure/patchRenovateConfig';\nimport { tryPatchServerListener } from '../configure/patchServerListener';\nimport { tryRefreshIgnoreFiles } from '../configure/refreshIgnoreFiles';\n\nimport { externalLint } from './external';\nimport { internalLint } from './internal';\nimport type { Input } from './types';\n\nexport const lint = async (\n  args = process.argv.slice(2),\n  tscOutputStream: Writable | undefined = undefined,\n  workerThreads = true,\n) => {\n  await Promise.all([\n    tryAddEmptyExports(),\n    tryPatchRenovateConfig(),\n    tryPatchServerListener(),\n    tryRefreshIgnoreFiles(),\n  ]);\n\n  const opts: Input = {\n    debug: hasDebugFlag(args),\n    serial: hasSerialFlag(args),\n    tscOutputStream,\n    workerThreads,\n  };\n\n  await externalLint(opts);\n\n  await internalLint();\n};\n"],
-  "mappings": ";;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAEA,kBAA4C;AAC5C,6BAAmC;AACnC,iCAAuC;AACvC,iCAAuC;AACvC,gCAAsC;AAEtC,sBAA6B;AAC7B,sBAA6B;AAGtB,MAAM,OAAO,OAClB,OAAO,QAAQ,KAAK,MAAM,CAAC,GAC3B,kBAAwC,QACxC,gBAAgB,SACb;AACH,QAAM,QAAQ,IAAI;AAAA,QAChB,2CAAmB;AAAA,QACnB,mDAAuB;AAAA,QACvB,mDAAuB;AAAA,QACvB,iDAAsB;AAAA,EACxB,CAAC;AAED,QAAM,OAAc;AAAA,IAClB,WAAO,0BAAa,IAAI;AAAA,IACxB,YAAQ,2BAAc,IAAI;AAAA,IAC1B;AAAA,IACA;AAAA,EACF;AAEA,YAAM,8BAAa,IAAI;AAEvB,YAAM,8BAAa;AACrB;",
+  "sourcesContent": ["import type { Writable } from 'stream';\nimport { inspect } from 'util';\n\nimport { hasDebugFlag, hasSerialFlag } from '../../utils/args';\nimport { log } from '../../utils/logging';\nimport { detectPackageManager } from '../../utils/packageManager';\nimport { throwOnTimeout } from '../../utils/wait';\n\nimport { createAnnotations } from './annotate';\nimport { autofix } from './autofix';\nimport { externalLint } from './external';\nimport { internalLint } from './internal';\nimport type { Input } from './types';\n\nexport const lint = async (\n  args = process.argv.slice(2),\n  tscWriteable: Writable | undefined = undefined,\n  workerThreads = true,\n) => {\n  const opts: Input = {\n    debug: hasDebugFlag(args),\n    serial: hasSerialFlag(args),\n    tscOutputStream: tscWriteable,\n    workerThreads,\n  };\n\n  const { eslint, prettier, tscOk, tscOutputStream } = await externalLint(opts);\n  const internal = await internalLint('lint', opts);\n\n  try {\n    await throwOnTimeout(\n      createAnnotations(internal, eslint, prettier, tscOk, tscOutputStream),\n      { s: 30 },\n    );\n  } catch (err) {\n    log.warn('Failed to annotate lint results.');\n    log.subtle(inspect(err));\n  }\n\n  if (!internal.ok || !eslint.ok || !prettier.ok || !tscOk) {\n    process.exitCode = 1;\n\n    const tools = [\n      ...(internal.ok ? [] : ['skuba']),\n      ...(eslint.ok ? [] : ['ESLint']),\n      ...(prettier.ok ? [] : ['Prettier']),\n      ...(tscOk ? [] : ['tsc']),\n    ];\n\n    log.err(`${tools.join(', ')} found issues that require triage.`);\n\n    if (internal.fixable || eslint.fixable || !prettier.ok) {\n      const packageManager = await detectPackageManager();\n      log.newline();\n      log.warn(\n        `Try running ${log.bold(\n          packageManager.exec,\n          'skuba',\n          'format',\n        )} to fix them.`,\n      );\n    }\n  }\n\n  await autofix({\n    debug: opts.debug,\n    eslint: eslint.fixable,\n    prettier: !prettier.ok,\n    internal: internal.fixable,\n  });\n};\n"],
+  "mappings": ";;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AACA,kBAAwB;AAExB,kBAA4C;AAC5C,qBAAoB;AACpB,4BAAqC;AACrC,kBAA+B;AAE/B,sBAAkC;AAClC,qBAAwB;AACxB,sBAA6B;AAC7B,sBAA6B;AAGtB,MAAM,OAAO,OAClB,OAAO,QAAQ,KAAK,MAAM,CAAC,GAC3B,eAAqC,QACrC,gBAAgB,SACb;AACH,QAAM,OAAc;AAAA,IAClB,WAAO,0BAAa,IAAI;AAAA,IACxB,YAAQ,2BAAc,IAAI;AAAA,IAC1B,iBAAiB;AAAA,IACjB;AAAA,EACF;AAEA,QAAM,EAAE,QAAQ,UAAU,OAAO,gBAAgB,IAAI,UAAM,8BAAa,IAAI;AAC5E,QAAM,WAAW,UAAM,8BAAa,QAAQ,IAAI;AAEhD,MAAI;AACF,cAAM;AAAA,UACJ,mCAAkB,UAAU,QAAQ,UAAU,OAAO,eAAe;AAAA,MACpE,EAAE,GAAG,GAAG;AAAA,IACV;AAAA,EACF,SAAS,KAAK;AACZ,uBAAI,KAAK,kCAAkC;AAC3C,uBAAI,WAAO,qBAAQ,GAAG,CAAC;AAAA,EACzB;AAEA,MAAI,CAAC,SAAS,MAAM,CAAC,OAAO,MAAM,CAAC,SAAS,MAAM,CAAC,OAAO;AACxD,YAAQ,WAAW;AAEnB,UAAM,QAAQ;AAAA,MACZ,GAAI,SAAS,KAAK,CAAC,IAAI,CAAC,OAAO;AAAA,MAC/B,GAAI,OAAO,KAAK,CAAC,IAAI,CAAC,QAAQ;AAAA,MAC9B,GAAI,SAAS,KAAK,CAAC,IAAI,CAAC,UAAU;AAAA,MAClC,GAAI,QAAQ,CAAC,IAAI,CAAC,KAAK;AAAA,IACzB;AAEA,uBAAI,IAAI,GAAG,MAAM,KAAK,IAAI,CAAC,oCAAoC;AAE/D,QAAI,SAAS,WAAW,OAAO,WAAW,CAAC,SAAS,IAAI;AACtD,YAAM,iBAAiB,UAAM,4CAAqB;AAClD,yBAAI,QAAQ;AACZ,yBAAI;AAAA,QACF,eAAe,mBAAI;AAAA,UACjB,eAAe;AAAA,UACf;AAAA,UACA;AAAA,QACF,CAAC;AAAA,MACH;AAAA,IACF;AAAA,EACF;AAEA,YAAM,wBAAQ;AAAA,IACZ,OAAO,KAAK;AAAA,IACZ,QAAQ,OAAO;AAAA,IACf,UAAU,CAAC,SAAS;AAAA,IACpB,UAAU,SAAS;AAAA,EACrB,CAAC;AACH;",
   "names": []
 }

package/lib/cli/lint/internal.d.ts

@@ -1 +1,12 @@
-export declare const internalLint: () => Promise<void>;
+import type { Input } from './types';
+export type InternalLintResult = {
+    ok: boolean;
+    fixable: boolean;
+    annotations?: Array<{
+        start_line?: number;
+        end_line?: number;
+        path: string;
+        message: string;
+    }>;
+};
+export declare const internalLint: (mode: 'format' | 'lint', input?: Input) => Promise<InternalLintResult>;

package/lib/cli/lint/internal.js

@@ -31,32 +31,68 @@ __export(internal_exports, {
   internalLint: () => internalLint
 });
 module.exports = __toCommonJS(internal_exports);
-var import_path = __toESM(require("path"));
+var import_util = require("util");
 var import_chalk = __toESM(require("chalk"));
-var import_fs_extra = require("fs-extra");
 var import_logging = require("../../utils/logging");
-var import_manifest = require("../../utils/manifest");
-const noSkubaTemplateJs = async () => {
-  const manifest = await (0, import_manifest.getConsumerManifest)();
-  if (!manifest) {
-    return;
+var import_upgrade = require("../configure/upgrade");
+var import_deleteFiles = require("./internalLints/deleteFiles");
+var import_noSkubaTemplateJs = require("./internalLints/noSkubaTemplateJs");
+var import_refreshConfigFiles = require("./internalLints/refreshConfigFiles");
+const lints = [
+  // Run upgradeSkuba before refreshConfigFiles for npmrc handling
+  [import_upgrade.upgradeSkuba],
+  [import_deleteFiles.deleteFilesLint, import_noSkubaTemplateJs.noSkubaTemplateJs, import_refreshConfigFiles.tryRefreshConfigFiles]
+];
+const lintSerially = async (mode, logger) => {
+  const results = [];
+  for (const lintGroup of lints) {
+    for (const lint of lintGroup) {
+      results.push(await lint(mode, logger));
+    }
   }
-  const templateConfigPath = import_path.default.join(
-    import_path.default.dirname(manifest.path),
-    "skuba.template.js"
-  );
-  if (await (0, import_fs_extra.pathExists)(templateConfigPath)) {
-    import_logging.log.err(
-      `Template is incomplete; run ${import_chalk.default.bold(
-        "yarn skuba configure"
-      )}. ${import_chalk.default.dim("no-skuba-template-js")}`
+  return results;
+};
+const lintConcurrently = async (mode, logger) => {
+  const results = [];
+  for (const lintGroup of lints) {
+    results.push(
+      ...await Promise.all(lintGroup.map((lint) => lint(mode, logger)))
     );
-    process.exitCode = 1;
   }
+  return results;
+};
+const selectLintFunction = (input) => {
+  const isSerial = input?.debug || input?.serial;
+  return isSerial ? lintSerially : lintConcurrently;
 };
-const internalLint = async () => {
-  await noSkubaTemplateJs();
+const internalLint = async (mode, input) => {
+  const start = process.hrtime.bigint();
+  const logger = (0, import_logging.createLogger)(
+    input?.debug ?? false,
+    ...mode === "lint" ? [import_chalk.default.blueBright("skuba    \u2502")] : []
+  );
+  try {
+    const lint = selectLintFunction(input);
+    const results = await lint(mode, logger);
+    const result = combineResults(results);
+    const end = process.hrtime.bigint();
+    logger.plain(`Processed skuba lints in ${logger.timing(start, end)}.`);
+    return result;
+  } catch (err) {
+    logger.err(logger.bold("Failed to run skuba lints."));
+    logger.subtle((0, import_util.inspect)(err));
+    process.exitCode = 1;
+    return { ok: false, fixable: false, annotations: [] };
+  }
 };
+const combineResults = (results) => results.reduce(
+  (cur, next) => ({
+    ok: cur.ok && next.ok,
+    fixable: cur.fixable || next.fixable,
+    annotations: [...cur.annotations ?? [], ...next.annotations ?? []]
+  }),
+  { ok: true, fixable: false }
+);
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   internalLint

package/lib/cli/lint/internal.js.map

@@ -1,7 +1,7 @@
 {
   "version": 3,
   "sources": ["../../../src/cli/lint/internal.ts"],
-  "sourcesContent": ["import path from 'path';\n\nimport chalk from 'chalk';\nimport { pathExists } from 'fs-extra';\n\nimport { log } from '../../utils/logging';\nimport { getConsumerManifest } from '../../utils/manifest';\n\nconst noSkubaTemplateJs = async () => {\n  const manifest = await getConsumerManifest();\n\n  if (!manifest) {\n    return;\n  }\n\n  const templateConfigPath = path.join(\n    path.dirname(manifest.path),\n    'skuba.template.js',\n  );\n\n  if (await pathExists(templateConfigPath)) {\n    log.err(\n      `Template is incomplete; run ${chalk.bold(\n        'yarn skuba configure',\n      )}. ${chalk.dim('no-skuba-template-js')}`,\n    );\n\n    process.exitCode = 1;\n  }\n};\n\nexport const internalLint = async () => {\n  await noSkubaTemplateJs();\n};\n"],
-  "mappings": ";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,kBAAiB;AAEjB,mBAAkB;AAClB,sBAA2B;AAE3B,qBAAoB;AACpB,sBAAoC;AAEpC,MAAM,oBAAoB,YAAY;AACpC,QAAM,WAAW,UAAM,qCAAoB;AAE3C,MAAI,CAAC,UAAU;AACb;AAAA,EACF;AAEA,QAAM,qBAAqB,YAAAA,QAAK;AAAA,IAC9B,YAAAA,QAAK,QAAQ,SAAS,IAAI;AAAA,IAC1B;AAAA,EACF;AAEA,MAAI,UAAM,4BAAW,kBAAkB,GAAG;AACxC,uBAAI;AAAA,MACF,+BAA+B,aAAAC,QAAM;AAAA,QACnC;AAAA,MACF,CAAC,KAAK,aAAAA,QAAM,IAAI,sBAAsB,CAAC;AAAA,IACzC;AAEA,YAAQ,WAAW;AAAA,EACrB;AACF;AAEO,MAAM,eAAe,YAAY;AACtC,QAAM,kBAAkB;AAC1B;",
-  "names": ["path", "chalk"]
+  "sourcesContent": ["import { inspect } from 'util';\n\nimport chalk from 'chalk';\n\nimport { type Logger, createLogger } from '../../utils/logging';\nimport { upgradeSkuba } from '../configure/upgrade';\n\nimport { deleteFilesLint } from './internalLints/deleteFiles';\nimport { noSkubaTemplateJs } from './internalLints/noSkubaTemplateJs';\nimport { tryRefreshConfigFiles } from './internalLints/refreshConfigFiles';\nimport type { Input } from './types';\n\nexport type InternalLintResult = {\n  ok: boolean;\n  fixable: boolean;\n  annotations?: Array<{\n    start_line?: number;\n    end_line?: number;\n    path: string;\n    message: string;\n  }>;\n};\n\nconst lints: Array<\n  Array<\n    (mode: 'format' | 'lint', logger: Logger) => Promise<InternalLintResult>\n  >\n> = [\n  // Run upgradeSkuba before refreshConfigFiles for npmrc handling\n  [upgradeSkuba],\n  [deleteFilesLint, noSkubaTemplateJs, tryRefreshConfigFiles],\n];\n\nconst lintSerially = async (mode: 'format' | 'lint', logger: Logger) => {\n  const results: InternalLintResult[] = [];\n  for (const lintGroup of lints) {\n    for (const lint of lintGroup) {\n      results.push(await lint(mode, logger));\n    }\n  }\n  return results;\n};\n\nconst lintConcurrently = async (mode: 'format' | 'lint', logger: Logger) => {\n  const results: InternalLintResult[] = [];\n\n  for (const lintGroup of lints) {\n    results.push(\n      ...(await Promise.all(lintGroup.map((lint) => lint(mode, logger)))),\n    );\n  }\n\n  return results;\n};\n\nconst selectLintFunction = (input?: Input) => {\n  const isSerial = input?.debug || input?.serial;\n  return isSerial ? lintSerially : lintConcurrently;\n};\n\nexport const internalLint = async (\n  mode: 'format' | 'lint',\n  input?: Input,\n): Promise<InternalLintResult> => {\n  const start = process.hrtime.bigint();\n  const logger = createLogger(\n    input?.debug ?? false,\n    ...(mode === 'lint' ? [chalk.blueBright('skuba    \u2502')] : []),\n  );\n\n  try {\n    const lint = selectLintFunction(input);\n    const results = await lint(mode, logger);\n    const result = combineResults(results);\n    const end = process.hrtime.bigint();\n    logger.plain(`Processed skuba lints in ${logger.timing(start, end)}.`);\n    return result;\n  } catch (err) {\n    logger.err(logger.bold('Failed to run skuba lints.'));\n    logger.subtle(inspect(err));\n\n    process.exitCode = 1;\n\n    return { ok: false, fixable: false, annotations: [] };\n  }\n};\n\nconst combineResults = (results: InternalLintResult[]): InternalLintResult =>\n  results.reduce(\n    (cur, next) => ({\n      ok: cur.ok && next.ok,\n      fixable: cur.fixable || next.fixable,\n      annotations: [...(cur.annotations ?? []), ...(next.annotations ?? [])],\n    }),\n    { ok: true, fixable: false },\n  );\n"],
+  "mappings": ";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,kBAAwB;AAExB,mBAAkB;AAElB,qBAA0C;AAC1C,qBAA6B;AAE7B,yBAAgC;AAChC,+BAAkC;AAClC,gCAAsC;AActC,MAAM,QAIF;AAAA;AAAA,EAEF,CAAC,2BAAY;AAAA,EACb,CAAC,oCAAiB,4CAAmB,+CAAqB;AAC5D;AAEA,MAAM,eAAe,OAAO,MAAyB,WAAmB;AACtE,QAAM,UAAgC,CAAC;AACvC,aAAW,aAAa,OAAO;AAC7B,eAAW,QAAQ,WAAW;AAC5B,cAAQ,KAAK,MAAM,KAAK,MAAM,MAAM,CAAC;AAAA,IACvC;AAAA,EACF;AACA,SAAO;AACT;AAEA,MAAM,mBAAmB,OAAO,MAAyB,WAAmB;AAC1E,QAAM,UAAgC,CAAC;AAEvC,aAAW,aAAa,OAAO;AAC7B,YAAQ;AAAA,MACN,GAAI,MAAM,QAAQ,IAAI,UAAU,IAAI,CAAC,SAAS,KAAK,MAAM,MAAM,CAAC,CAAC;AAAA,IACnE;AAAA,EACF;AAEA,SAAO;AACT;AAEA,MAAM,qBAAqB,CAAC,UAAkB;AAC5C,QAAM,WAAW,OAAO,SAAS,OAAO;AACxC,SAAO,WAAW,eAAe;AACnC;AAEO,MAAM,eAAe,OAC1B,MACA,UACgC;AAChC,QAAM,QAAQ,QAAQ,OAAO,OAAO;AACpC,QAAM,aAAS;AAAA,IACb,OAAO,SAAS;AAAA,IAChB,GAAI,SAAS,SAAS,CAAC,aAAAA,QAAM,WAAW,iBAAY,CAAC,IAAI,CAAC;AAAA,EAC5D;AAEA,MAAI;AACF,UAAM,OAAO,mBAAmB,KAAK;AACrC,UAAM,UAAU,MAAM,KAAK,MAAM,MAAM;AACvC,UAAM,SAAS,eAAe,OAAO;AACrC,UAAM,MAAM,QAAQ,OAAO,OAAO;AAClC,WAAO,MAAM,4BAA4B,OAAO,OAAO,OAAO,GAAG,CAAC,GAAG;AACrE,WAAO;AAAA,EACT,SAAS,KAAK;AACZ,WAAO,IAAI,OAAO,KAAK,4BAA4B,CAAC;AACpD,WAAO,WAAO,qBAAQ,GAAG,CAAC;AAE1B,YAAQ,WAAW;AAEnB,WAAO,EAAE,IAAI,OAAO,SAAS,OAAO,aAAa,CAAC,EAAE;AAAA,EACtD;AACF;AAEA,MAAM,iBAAiB,CAAC,YACtB,QAAQ;AAAA,EACN,CAAC,KAAK,UAAU;AAAA,IACd,IAAI,IAAI,MAAM,KAAK;AAAA,IACnB,SAAS,IAAI,WAAW,KAAK;AAAA,IAC7B,aAAa,CAAC,GAAI,IAAI,eAAe,CAAC,GAAI,GAAI,KAAK,eAAe,CAAC,CAAE;AAAA,EACvE;AAAA,EACA,EAAE,IAAI,MAAM,SAAS,MAAM;AAC7B;",
+  "names": ["chalk"]
 }

package/lib/cli/lint/internalLints/deleteFiles.d.ts

@@ -0,0 +1,3 @@
+import type { Logger } from '../../../utils/logging';
+import type { InternalLintResult } from '../internal';
+export declare const deleteFilesLint: (mode: 'format' | 'lint', logger: Logger) => Promise<InternalLintResult>;

package/lib/cli/lint/internalLints/deleteFiles.js

@@ -0,0 +1,102 @@
+"use strict";
+var __create = Object.create;
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var deleteFiles_exports = {};
+__export(deleteFiles_exports, {
+  deleteFilesLint: () => deleteFilesLint
+});
+module.exports = __toCommonJS(deleteFiles_exports);
+var import_path = __toESM(require("path"));
+var import_util = require("util");
+var import_fs_extra = require("fs-extra");
+var import_packageManager = require("../../../utils/packageManager");
+const AUTOFIX_DELETE_FILES = [
+  // Try to delete this SEEK-Jobs/gutenberg automation file that may have been
+  // accidentally committed in old versions of skuba.
+  "Dockerfile-incunabulum"
+];
+const deleteFilesLint = async (mode, logger) => {
+  const dir = process.cwd();
+  const toDelete = (await Promise.all(
+    AUTOFIX_DELETE_FILES.map(
+      async (filename) => [filename, await (0, import_fs_extra.pathExists)(import_path.default.join(dir, filename))]
+    )
+  )).filter(([, exists]) => exists).map(([filename]) => filename);
+  if (mode === "format") {
+    if (toDelete.length === 0) {
+      return { ok: true, fixable: false };
+    }
+    try {
+      await Promise.all(
+        toDelete.map((filename) => {
+          logger.warn(`Deleting ${logger.bold(filename)}.`);
+          return (0, import_fs_extra.rm)(import_path.default.join(dir, filename), { force: true });
+        })
+      );
+      return {
+        ok: true,
+        fixable: false
+      };
+    } catch (err) {
+      logger.warn(logger.bold("Failed to delete files."));
+      logger.subtle((0, import_util.inspect)(err));
+      return {
+        ok: true,
+        // It's not really a huge deal if we can't delete these files
+        fixable: false
+      };
+    }
+  }
+  if (toDelete.length) {
+    const packageManager = await (0, import_packageManager.detectPackageManager)();
+    logger.warn(
+      `Some files should be deleted. Run ${logger.bold(
+        packageManager.exec,
+        "skuba",
+        "format"
+      )} to delete them. ${logger.dim("delete-files")}`
+    );
+    return {
+      ok: false,
+      fixable: true,
+      annotations: toDelete.map((filename) => ({
+        path: filename,
+        message: "This file should be deleted."
+      }))
+    };
+  }
+  return {
+    ok: true,
+    fixable: false
+  };
+};
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  deleteFilesLint
+});
+//# sourceMappingURL=deleteFiles.js.map

package/lib/cli/lint/internalLints/deleteFiles.js.map

@@ -0,0 +1,7 @@
+{
+  "version": 3,
+  "sources": ["../../../../src/cli/lint/internalLints/deleteFiles.ts"],
+  "sourcesContent": ["import path from 'path';\nimport { inspect } from 'util';\n\nimport { pathExists, rm } from 'fs-extra';\n\nimport type { Logger } from '../../../utils/logging';\nimport { detectPackageManager } from '../../../utils/packageManager';\nimport type { InternalLintResult } from '../internal';\n\nconst AUTOFIX_DELETE_FILES = [\n  // Try to delete this SEEK-Jobs/gutenberg automation file that may have been\n  // accidentally committed in old versions of skuba.\n  'Dockerfile-incunabulum',\n];\n\nexport const deleteFilesLint = async (\n  mode: 'format' | 'lint',\n  logger: Logger,\n): Promise<InternalLintResult> => {\n  const dir = process.cwd();\n\n  const toDelete = (\n    await Promise.all(\n      AUTOFIX_DELETE_FILES.map(\n        async (filename) =>\n          [filename, await pathExists(path.join(dir, filename))] as const,\n      ),\n    )\n  )\n    .filter(([, exists]) => exists)\n    .map(([filename]) => filename);\n\n  if (mode === 'format') {\n    if (toDelete.length === 0) {\n      return { ok: true, fixable: false };\n    }\n\n    try {\n      await Promise.all(\n        toDelete.map((filename) => {\n          logger.warn(`Deleting ${logger.bold(filename)}.`);\n          return rm(path.join(dir, filename), { force: true });\n        }),\n      );\n\n      return {\n        ok: true,\n        fixable: false,\n      };\n    } catch (err) {\n      logger.warn(logger.bold('Failed to delete files.'));\n      logger.subtle(inspect(err));\n\n      return {\n        ok: true, // It's not really a huge deal if we can't delete these files\n        fixable: false,\n      };\n    }\n  }\n\n  if (toDelete.length) {\n    const packageManager = await detectPackageManager();\n\n    logger.warn(\n      `Some files should be deleted. Run ${logger.bold(\n        packageManager.exec,\n        'skuba',\n        'format',\n      )} to delete them. ${logger.dim('delete-files')}`,\n    );\n\n    return {\n      ok: false,\n      fixable: true,\n      annotations: toDelete.map((filename) => ({\n        path: filename,\n        message: 'This file should be deleted.',\n      })),\n    };\n  }\n\n  return {\n    ok: true,\n    fixable: false,\n  };\n};\n"],
+  "mappings": ";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,kBAAiB;AACjB,kBAAwB;AAExB,sBAA+B;AAG/B,4BAAqC;AAGrC,MAAM,uBAAuB;AAAA;AAAA;AAAA,EAG3B;AACF;AAEO,MAAM,kBAAkB,OAC7B,MACA,WACgC;AAChC,QAAM,MAAM,QAAQ,IAAI;AAExB,QAAM,YACJ,MAAM,QAAQ;AAAA,IACZ,qBAAqB;AAAA,MACnB,OAAO,aACL,CAAC,UAAU,UAAM,4BAAW,YAAAA,QAAK,KAAK,KAAK,QAAQ,CAAC,CAAC;AAAA,IACzD;AAAA,EACF,GAEC,OAAO,CAAC,CAAC,EAAE,MAAM,MAAM,MAAM,EAC7B,IAAI,CAAC,CAAC,QAAQ,MAAM,QAAQ;AAE/B,MAAI,SAAS,UAAU;AACrB,QAAI,SAAS,WAAW,GAAG;AACzB,aAAO,EAAE,IAAI,MAAM,SAAS,MAAM;AAAA,IACpC;AAEA,QAAI;AACF,YAAM,QAAQ;AAAA,QACZ,SAAS,IAAI,CAAC,aAAa;AACzB,iBAAO,KAAK,YAAY,OAAO,KAAK,QAAQ,CAAC,GAAG;AAChD,qBAAO,oBAAG,YAAAA,QAAK,KAAK,KAAK,QAAQ,GAAG,EAAE,OAAO,KAAK,CAAC;AAAA,QACrD,CAAC;AAAA,MACH;AAEA,aAAO;AAAA,QACL,IAAI;AAAA,QACJ,SAAS;AAAA,MACX;AAAA,IACF,SAAS,KAAK;AACZ,aAAO,KAAK,OAAO,KAAK,yBAAyB,CAAC;AAClD,aAAO,WAAO,qBAAQ,GAAG,CAAC;AAE1B,aAAO;AAAA,QACL,IAAI;AAAA;AAAA,QACJ,SAAS;AAAA,MACX;AAAA,IACF;AAAA,EACF;AAEA,MAAI,SAAS,QAAQ;AACnB,UAAM,iBAAiB,UAAM,4CAAqB;AAElD,WAAO;AAAA,MACL,qCAAqC,OAAO;AAAA,QAC1C,eAAe;AAAA,QACf;AAAA,QACA;AAAA,MACF,CAAC,oBAAoB,OAAO,IAAI,cAAc,CAAC;AAAA,IACjD;AAEA,WAAO;AAAA,MACL,IAAI;AAAA,MACJ,SAAS;AAAA,MACT,aAAa,SAAS,IAAI,CAAC,cAAc;AAAA,QACvC,MAAM;AAAA,QACN,SAAS;AAAA,MACX,EAAE;AAAA,IACJ;AAAA,EACF;AAEA,SAAO;AAAA,IACL,IAAI;AAAA,IACJ,SAAS;AAAA,EACX;AACF;",
+  "names": ["path"]
+}

package/lib/cli/lint/internalLints/noSkubaTemplateJs.d.ts

@@ -0,0 +1,3 @@
+import type { Logger } from '../../../utils/logging';
+import type { InternalLintResult } from '../internal';
+export declare const noSkubaTemplateJs: (_mode: 'format' | 'lint', logger: Logger) => Promise<InternalLintResult>;

package/lib/cli/lint/internalLints/noSkubaTemplateJs.js

@@ -0,0 +1,75 @@
+"use strict";
+var __create = Object.create;
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var noSkubaTemplateJs_exports = {};
+__export(noSkubaTemplateJs_exports, {
+  noSkubaTemplateJs: () => noSkubaTemplateJs
+});
+module.exports = __toCommonJS(noSkubaTemplateJs_exports);
+var import_path = __toESM(require("path"));
+var import_fs_extra = require("fs-extra");
+var import_manifest = require("../../../utils/manifest");
+var import_packageManager = require("../../../utils/packageManager");
+const noSkubaTemplateJs = async (_mode, logger) => {
+  const [manifest, packageManager] = await Promise.all([
+    (0, import_manifest.getConsumerManifest)(),
+    (0, import_packageManager.detectPackageManager)()
+  ]);
+  if (!manifest) {
+    return { ok: true, fixable: false };
+  }
+  const templateConfigPath = import_path.default.join(
+    import_path.default.dirname(manifest.path),
+    "skuba.template.js"
+  );
+  if (await (0, import_fs_extra.pathExists)(templateConfigPath)) {
+    logger.err(
+      `Template is incomplete; run ${logger.bold(
+        packageManager.exec,
+        "skuba",
+        "configure"
+      )}. ${logger.dim("no-skuba-template-js")}`
+    );
+    return {
+      ok: false,
+      fixable: false,
+      annotations: [
+        {
+          path: "skuba.template.js",
+          message: `Template is incomplete; run ${packageManager.exec} skuba configure.`
+        }
+      ]
+    };
+  }
+  return { ok: true, fixable: false };
+};
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  noSkubaTemplateJs
+});
+//# sourceMappingURL=noSkubaTemplateJs.js.map

package/lib/cli/lint/internalLints/noSkubaTemplateJs.js.map

@@ -0,0 +1,7 @@
+{
+  "version": 3,
+  "sources": ["../../../../src/cli/lint/internalLints/noSkubaTemplateJs.ts"],
+  "sourcesContent": ["import path from 'path';\n\nimport { pathExists } from 'fs-extra';\n\nimport type { Logger } from '../../../utils/logging';\nimport { getConsumerManifest } from '../../../utils/manifest';\nimport { detectPackageManager } from '../../../utils/packageManager';\nimport type { InternalLintResult } from '../internal';\n\nexport const noSkubaTemplateJs = async (\n  _mode: 'format' | 'lint',\n  logger: Logger,\n): Promise<InternalLintResult> => {\n  const [manifest, packageManager] = await Promise.all([\n    getConsumerManifest(),\n    detectPackageManager(),\n  ]);\n\n  if (!manifest) {\n    // This will throw elsewhere\n    return { ok: true, fixable: false };\n  }\n\n  const templateConfigPath = path.join(\n    path.dirname(manifest.path),\n    'skuba.template.js',\n  );\n\n  if (await pathExists(templateConfigPath)) {\n    logger.err(\n      `Template is incomplete; run ${logger.bold(\n        packageManager.exec,\n        'skuba',\n        'configure',\n      )}. ${logger.dim('no-skuba-template-js')}`,\n    );\n\n    return {\n      ok: false,\n      fixable: false,\n      annotations: [\n        {\n          path: 'skuba.template.js',\n          message: `Template is incomplete; run ${packageManager.exec} skuba configure.`,\n        },\n      ],\n    };\n  }\n\n  return { ok: true, fixable: false };\n};\n"],
+  "mappings": ";;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA,kBAAiB;AAEjB,sBAA2B;AAG3B,sBAAoC;AACpC,4BAAqC;AAG9B,MAAM,oBAAoB,OAC/B,OACA,WACgC;AAChC,QAAM,CAAC,UAAU,cAAc,IAAI,MAAM,QAAQ,IAAI;AAAA,QACnD,qCAAoB;AAAA,QACpB,4CAAqB;AAAA,EACvB,CAAC;AAED,MAAI,CAAC,UAAU;AAEb,WAAO,EAAE,IAAI,MAAM,SAAS,MAAM;AAAA,EACpC;AAEA,QAAM,qBAAqB,YAAAA,QAAK;AAAA,IAC9B,YAAAA,QAAK,QAAQ,SAAS,IAAI;AAAA,IAC1B;AAAA,EACF;AAEA,MAAI,UAAM,4BAAW,kBAAkB,GAAG;AACxC,WAAO;AAAA,MACL,+BAA+B,OAAO;AAAA,QACpC,eAAe;AAAA,QACf;AAAA,QACA;AAAA,MACF,CAAC,KAAK,OAAO,IAAI,sBAAsB,CAAC;AAAA,IAC1C;AAEA,WAAO;AAAA,MACL,IAAI;AAAA,MACJ,SAAS;AAAA,MACT,aAAa;AAAA,QACX;AAAA,UACE,MAAM;AAAA,UACN,SAAS,+BAA+B,eAAe,IAAI;AAAA,QAC7D;AAAA,MACF;AAAA,IACF;AAAA,EACF;AAEA,SAAO,EAAE,IAAI,MAAM,SAAS,MAAM;AACpC;",
+  "names": ["path"]
+}

package/lib/cli/lint/internalLints/refreshConfigFiles.d.ts

@@ -0,0 +1,11 @@
+import type { Logger } from '../../../utils/logging';
+import type { InternalLintResult } from '../internal';
+export declare const refreshConfigFiles: (mode: 'format' | 'lint', logger: Logger) => Promise<{
+    ok: boolean;
+    fixable: boolean;
+    annotations: {
+        path: string;
+        message: string;
+    }[];
+}>;
+export declare const tryRefreshConfigFiles: (mode: 'format' | 'lint', logger: Logger) => Promise<InternalLintResult>;

package/lib/cli/lint/internalLints/refreshConfigFiles.js

@@ -0,0 +1,147 @@
+"use strict";
+var __create = Object.create;
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var refreshConfigFiles_exports = {};
+__export(refreshConfigFiles_exports, {
+  refreshConfigFiles: () => refreshConfigFiles,
+  tryRefreshConfigFiles: () => tryRefreshConfigFiles
+});
+module.exports = __toCommonJS(refreshConfigFiles_exports);
+var import_path = __toESM(require("path"));
+var import_util = require("util");
+var import_fs_extra = require("fs-extra");
+var import_strip_ansi = __toESM(require("strip-ansi"));
+var import_npmrc = require("../../../utils/npmrc");
+var import_packageManager = require("../../../utils/packageManager");
+var import_template = require("../../../utils/template");
+var import_package = require("../../configure/analysis/package");
+var import_project = require("../../configure/analysis/project");
+var import_ignoreFile = require("../../configure/processing/ignoreFile");
+const ensureNoAuthToken = (fileContents) => fileContents.split("\n").filter((line) => !(0, import_npmrc.hasNpmrcSecret)(line)).join("\n");
+const REFRESHABLE_CONFIG_FILES = [
+  { name: ".eslintignore" },
+  { name: ".gitignore" },
+  { name: ".prettierignore" },
+  {
+    name: ".npmrc",
+    additionalMapping: ensureNoAuthToken,
+    if: (packageManager) => packageManager.command === "pnpm"
+  }
+];
+const refreshConfigFiles = async (mode, logger) => {
+  const manifest = await (0, import_package.getDestinationManifest)();
+  const destinationRoot = import_path.default.dirname(manifest.path);
+  const readDestinationFile = (0, import_project.createDestinationFileReader)(destinationRoot);
+  const refreshConfigFile = async (filename, packageManager2, additionalMapping = (s) => s, condition = () => true) => {
+    if (!condition(packageManager2)) {
+      return { needsChange: false };
+    }
+    const [inputFile, templateFile] = await Promise.all([
+      readDestinationFile(filename),
+      (0, import_template.readBaseTemplateFile)(`_${filename}`)
+    ]);
+    const data = additionalMapping(
+      inputFile ? (0, import_ignoreFile.mergeWithIgnoreFile)(templateFile)(inputFile) : templateFile,
+      packageManager2
+    );
+    const filepath = import_path.default.join(destinationRoot, filename);
+    if (mode === "format") {
+      if (data === inputFile) {
+        return { needsChange: false };
+      }
+      await (0, import_fs_extra.writeFile)(filepath, data);
+      return {
+        needsChange: false,
+        msg: `Refreshed ${logger.bold(filename)}.`,
+        filename
+      };
+    }
+    if (data !== inputFile) {
+      return {
+        needsChange: true,
+        msg: `The ${logger.bold(
+          filename
+        )} file is out of date. Run \`${logger.bold(
+          packageManager2.exec,
+          "skuba",
+          "format"
+        )}\` to update it.`,
+        filename
+      };
+    }
+    return { needsChange: false };
+  };
+  const packageManager = await (0, import_packageManager.detectPackageManager)(destinationRoot);
+  const results = await Promise.all(
+    REFRESHABLE_CONFIG_FILES.map(
+      (conf) => refreshConfigFile(
+        conf.name,
+        packageManager,
+        conf.additionalMapping,
+        conf.if
+      )
+    )
+  );
+  results.forEach((result) => {
+    if (result.msg) {
+      logger.warn(result.msg, logger.dim("refresh-config-files"));
+    }
+  });
+  const anyNeedChanging = results.some(({ needsChange }) => needsChange);
+  return {
+    ok: !anyNeedChanging,
+    fixable: anyNeedChanging,
+    annotations: results.flatMap(
+      ({ needsChange, filename, msg }) => needsChange && msg ? [
+        {
+          path: filename,
+          message: (0, import_strip_ansi.default)(msg)
+        }
+      ] : []
+    )
+  };
+};
+const tryRefreshConfigFiles = async (mode, logger) => {
+  try {
+    return await refreshConfigFiles(mode, logger);
+  } catch (err) {
+    logger.warn("Failed to refresh config files.");
+    logger.subtle((0, import_util.inspect)(err));
+    return {
+      ok: false,
+      fixable: false,
+      annotations: []
+    };
+  }
+};
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  refreshConfigFiles,
+  tryRefreshConfigFiles
+});
+//# sourceMappingURL=refreshConfigFiles.js.map