@kitschpatrol/typescript-config 5.1.0 → 5.3.0

package/bin/cli.js

@@ -1978,11 +1978,11 @@ var parser = new YargsParser({
   resolve: resolve2,
   // TODO: figure  out a  way to combine ESM and CJS coverage, such  that
   // we can exercise all the lines below:
-  require: (path4) => {
+  require: (path5) => {
     if (typeof __require !== "undefined") {
-      return __require(path4);
-    } else if (path4.match(/\.json$/)) {
-      return JSON.parse(readFileSync(path4, "utf8"));
+      return __require(path5);
+    } else if (path5.match(/\.json$/)) {
+      return JSON.parse(readFileSync(path5, "utf8"));
     } else {
       throw Error("only .json config files are supported in ESM");
     }
@@ -5552,7 +5552,7 @@ var Yargs = YargsFactory(esm_default);
 var yargs_default = Yargs;
 
 // ../../package.json
-var version = "5.1.0";
+var version = "5.3.0";
 
 // ../../src/execa-utilities.ts
 function isErrorExecaError(error) {
@@ -5565,7 +5565,7 @@ import jsonColorizer from "@pinojs/json-colorizer";
 // ../../node_modules/.pnpm/decircular@1.0.0/node_modules/decircular/index.js
 function decircular(object) {
   const seenObjects = /* @__PURE__ */ new WeakMap();
-  function internalDecircular(value, path4 = []) {
+  function internalDecircular(value, path5 = []) {
     if (!(value !== null && typeof value === "object")) {
       return value;
     }
@@ -5573,10 +5573,10 @@ function decircular(object) {
     if (existingPath) {
       return `[Circular *${existingPath.join(".")}]`;
     }
-    seenObjects.set(value, path4);
+    seenObjects.set(value, path5);
     const newValue = Array.isArray(value) ? [] : {};
     for (const [key2, value2] of Object.entries(value)) {
-      newValue[key2] = internalDecircular(value2, [...path4, key2]);
+      newValue[key2] = internalDecircular(value2, [...path5, key2]);
     }
     seenObjects.delete(value);
     return newValue;
@@ -6167,19 +6167,46 @@ var DESCRIPTION = {
 };
 
 // src/command.ts
+import fse3 from "fs-extra";
+import path4 from "node:path";
+async function isSvelteProject() {
+  const packageDirectory = getPackageDirectory();
+  const svelteConfigFiles = ["svelte.config.js", "svelte.config.mjs", "svelte.config.cjs"];
+  const fileChecks = svelteConfigFiles.map(
+    async (configFile) => fse3.exists(path4.join(packageDirectory, configFile))
+  );
+  const results = await Promise.all(fileChecks);
+  return results.some(Boolean);
+}
+async function printSvelteWarningCommand(logStream) {
+  logStream.write(
+    "Skipping `tsc` since this is a Svelte project. Consider running `svelte-check` instead.\n"
+  );
+  return 0;
+}
+async function generateTypeScriptLintCommand() {
+  return await isSvelteProject() ? [
+    {
+      execute: printSvelteWarningCommand,
+      name: printSvelteWarningCommand.name
+    }
+  ] : [
+    {
+      cwdOverride: "package-dir",
+      name: "tsc",
+      optionFlags: ["--noEmit"]
+    }
+  ];
+}
 var commandDefinition = {
   commands: {
     init: {
       locationOptionFlag: false
     },
     lint: {
-      commands: [
-        {
-          cwdOverride: "package-dir",
-          name: "tsc",
-          optionFlags: ["--noEmit"]
-        }
-      ],
+      // Needs some special logic since tsc doesn't really work in Svelte projects
+      // See https://github.com/sveltejs/language-tools/issues/2527
+      commands: await generateTypeScriptLintCommand(),
       // TODO confirm monorepo behavior
       description: `Run type checking on your project. ${DESCRIPTION.packageRun} ${DESCRIPTION.monorepoRun}`,
       positionalArgumentMode: "none"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitschpatrol/typescript-config",
-  "version": "5.1.0",
+  "version": "5.3.0",
   "description": "TypeScript configuration for @kitschpatrol/shared-config.",
   "keywords": [
     "shared-config",

package/readme.md

@@ -173,6 +173,12 @@ kpi-typescript print-config
 
 ## Notes
 
+### Svelte caveat
+
+The `kpi-typescript lint` command will detect whether it's running in a Svelte project, and treat `lint` as a no-op when that's the case.
+
+This is necessary to prevent unactionable warnings because Svelte [doesn't play well](https://github.com/sveltejs/language-tools/issues/2527) with the underlying TypeScript `tsc` command.
+
 ### General
 
 - [tsconfig Cheat Sheet](https://www.totaltypescript.com/tsconfig-cheat-sheet)