slop-police 0.3.1 → 0.3.3

package/README.md

@@ -40,9 +40,9 @@ slop-police reads `slop-police.json` from the current directory:
     "no-redundant-value-alias": "error",
     "no-single-string-cn": "off",
     "no-pass-through-wrapper": "error",
+    "no-duplicate-code": "error",
     "no-duplicate-function": "warn",
     "no-placeholder-comment": "error",
-    "no-promise-default-fallback": "error",
     "no-stringified-unknown-error": "warn"
   }
 }
@@ -153,6 +153,26 @@ function parse(input: string) {
 }
 ```
 
+### no-duplicate-code
+
+Copy-pasted code: at least 50 tokens in a row that repeat earlier code (in the same file or one before it by path) and span at least 10 lines. Tokens are compared by their text, so a renamed variable ends a copy, but comments, formatting, imports and TypeScript-only syntax are left out: code that differs only in its types is still a copy, and repeated types aren't reported. Test files are neither reported nor compared. `// slop-police-ignore no-duplicate-code` above a statement, class member or object property takes it out of the comparison, so it is not reported as a copy or as the original. Options: `minLines` (default `10`, counting every line a copy spans), `minTokens` (default `50`) and `isolate`, folders (relative to the config) that aren't compared with each other, such as packages that can't share code: `["packages/api", "packages/frontend"]`.
+
+```ts
+// update-card.ts and delete-card.ts both check the card the same way:
+const userId = ctx.userId;
+assert(userId);
+
+const cards = await getCardsWithDeck(ctx.db, [input.id]);
+if (cards.length === 0) {
+  throw notFound("Card not found");
+}
+const card = cards[0];
+if (card.deckAuthorId !== userId) {
+  throw forbidden("You don't own this card");
+}
+// delete-card.ts:6:5  Lines 6-16 duplicate update-card.ts:6-16
+```
+
 ### no-duplicate-function
 
 A function whose body has the same shape as functions in at least two other files: a helper that was rewritten instead of reused. Shapes ignore the names of parameters and locals and the values of literals, but not the names of what the body calls or reads. Bodies with fewer than 2 statements or 16 syntax nodes, and functions in test files, don't count. Option: `minFiles` (default `3`, at least `2`), the number of files a shape has to appear in.
@@ -167,19 +187,6 @@ export async function loadUsers(id: string) {
 // Function loadUsers has the same body shape as loadPosts (posts.ts:1), loadTeams (teams.ts:1)
 ```
 
-### no-promise-default-fallback
-
-A promise `.catch()` handler that hides the rejection: an empty handler, or one that only returns `null`, `undefined`, `false`, `0`, `""`, `[]` or `{}`, optionally after `console.*` or `logger.*` calls. Only `.catch` on promises and other thenables counts.
-
-```ts
-const config = await fetchConfig().catch(() => null); // Promise rejection is replaced with null
-navigator.clipboard.writeText(text).catch(() => {}); // Promise rejection is swallowed by an empty catch handler
-const enabled = await fetchFlag().catch((error) => { // Promise rejection is logged and replaced with false
-  console.error("flag load failed", error);
-  return false;
-});
-```
-
 ### no-stringified-unknown-error
 
 `error instanceof Error ? error.message : String(error)` (in either order), which flattens a caught value into a string and drops its type, stack and cause. A plain `instanceof Error` check counts when the variable is named `error`, `err`, `e` or `cause`; an `||` of `instanceof` checks counts for any name. A function that does nothing but this conversion, like `getErrorMessage(error)`, is where it belongs and isn't reported; use it instead of repeating the expression.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slop-police",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Type-aware TypeScript linter for the dead code and needless indirection AI agents leave behind",
   "keywords": [
     "typescript",
@@ -41,9 +41,9 @@
     "node": ">=22.18"
   },
   "optionalDependencies": {
-    "slop-police-darwin-arm64": "0.3.1",
-    "slop-police-darwin-x64": "0.3.1",
-    "slop-police-linux-arm64": "0.3.1",
-    "slop-police-linux-x64": "0.3.1"
+    "slop-police-darwin-arm64": "0.3.3",
+    "slop-police-darwin-x64": "0.3.3",
+    "slop-police-linux-arm64": "0.3.3",
+    "slop-police-linux-x64": "0.3.3"
   }
 }