oxlint-plugin-deslopify 0.1.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Brian Estrada
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -0,0 +1,90 @@
+# oxlint-plugin-deslopify
+
+[Oxlint](https://oxc.rs/docs/guide/usage/linter) rules that keep AI agents from writing sloppy code.
+
+Agents follow a linter more reliably than a style guide in a prompt. This plugin turns the patterns they reach for (multi-line ternaries, one-line objects, `T["key"]` type lookups, unchecked casts) into lint errors, and ships a preset that pairs them with Oxlint's built-in rules for the same problems.
+
+## Install
+
+```sh
+bun add -d oxlint oxlint-plugin-deslopify
+# For the type-aware rules in the preset:
+bun add -d oxlint-tsgolint
+```
+
+## Usage
+
+Extend the recommended preset in `oxlint.config.ts`:
+
+```ts
+import { defineConfig } from "oxlint";
+import deslopify from "oxlint-plugin-deslopify";
+
+export default defineConfig({
+  extends: [deslopify.configs.recommended],
+  options: {
+    // Without this, the preset's type-aware rules are skipped.
+    typeAware: true,
+  },
+});
+```
+
+Or load the plugin and pick rules yourself, in `oxlint.config.ts` or `.oxlintrc.json`:
+
+```json
+{
+  "jsPlugins": ["oxlint-plugin-deslopify"],
+  "rules": {
+    "deslopify/simple-ternary": "error"
+  }
+}
+```
+
+## Rules
+
+✅ in `configs.recommended` · 🔧 fixable with `--fix`
+
+| Rule                                                               | Description                                                                         | ✅  | 🔧  |
+| ------------------------------------------------------------------ | ----------------------------------------------------------------------------------- | --- | --- |
+| [`multiline-object`](docs/rules/multiline-object.md)               | Put each property of an object literal with two or more properties on its own line. | ✅  | 🔧  |
+| [`no-graphql-cast`](docs/rules/no-graphql-cast.md)                 | Require `graphql<unknown>` so GraphQL responses are parsed, not cast.               |     |     |
+| [`no-property-type-lookup`](docs/rules/no-property-type-lookup.md) | Disallow looking up a property's type with `T["key"]`; name the type instead.       | ✅  |     |
+| [`simple-ternary`](docs/rules/simple-ternary.md)                   | Keep ternaries on one line, testing one condition, without interpolated strings.    | ✅  |     |
+
+## What the preset turns on
+
+Besides the ✅ rules above, `configs.recommended` sets these built-in Oxlint rules to `"error"`:
+
+| Rule                                     | Setting       | Why                                                                          |
+| ---------------------------------------- | ------------- | ---------------------------------------------------------------------------- |
+| `func-style`                             | `declaration` | Named functions are `function` declarations, not arrows assigned to a const. |
+| `object-shorthand`                       | `never`       | `{ key: key }` and `key: function () {}`, never the shorthand.               |
+| `curly`                                  | `all`         | Braces on every block body, so extending one is a clean diff.                |
+| `arrow-body-style`                       | `always`      | Arrow functions get a block body and an explicit `return`.                   |
+| `typescript/consistent-type-definitions` | `interface`   | Object types are interfaces, not type aliases.                               |
+| `typescript/no-deprecated`               | type-aware    | Flags `@deprecated` APIs, which `tsc` never reports.                         |
+| `typescript/no-unsafe-type-assertion`    | type-aware    | Unchecked casts let unvalidated data through.                                |
+| `typescript/no-unsafe-assignment`        | type-aware    | Stops `any` spreading into typed variables.                                  |
+| `typescript/no-unsafe-member-access`     | type-aware    | Stops property access on `any`.                                              |
+| `typescript/no-unsafe-argument`          | type-aware    | Stops `any` being passed as a typed argument.                                |
+
+Override any of them in your own `rules` after `extends`.
+
+## Requirements
+
+- Oxlint 1.86.0 or later. Oxlint's JS plugin support is still in alpha and not covered by semver.
+- Node.js `^20.19.0` or `>=22.12.0`.
+- `multiline-object`'s fix only adds a newline after `{`. Run [oxfmt](https://oxc.rs/docs/guide/usage/formatter) (or Prettier) with the default `objectWrap: "preserve"` to lay out the rest.
+
+## Development
+
+```sh
+bun install
+bun run check
+```
+
+See [AGENTS.md](AGENTS.md) for how rules are laid out and released.
+
+## License
+
+[MIT](LICENSE)

package/dist/index.d.mts

@@ -0,0 +1,11 @@
+import { Plugin } from "@oxlint/plugins";
+import { OxlintConfig } from "oxlint";
+//#region src/index.d.ts
+interface DeslopifyPlugin extends Plugin {
+  configs: {
+    recommended: OxlintConfig;
+  };
+}
+declare const plugin: DeslopifyPlugin;
+//#endregion
+export { plugin as default };

package/dist/index.mjs

@@ -0,0 +1,141 @@
+import { eslintCompatPlugin } from "@oxlint/plugins";
+//#region src/rules/docs-url.ts
+function docsUrl(ruleName) {
+	return `https://github.com/BrianEstrada/oxlint-deslopify/blob/main/docs/rules/${ruleName}.md`;
+}
+//#endregion
+//#region src/index.ts
+const rules = {
+	"multiline-object": {
+		meta: {
+			type: "layout",
+			docs: {
+				description: "Put each property of an object literal with two or more properties on its own line.",
+				recommended: true,
+				url: docsUrl("multiline-object")
+			},
+			fixable: "whitespace",
+			messages: { singleLine: "Put each property of this object on its own line." }
+		},
+		createOnce: function(context) {
+			return { ObjectExpression: function(node) {
+				if (node.properties.length < 2 || node.loc.start.line !== node.loc.end.line) return;
+				context.report({
+					node,
+					messageId: "singleLine",
+					fix: function(fixer) {
+						return fixer.insertTextAfterRange([node.range[0], node.range[0] + 1], "\n");
+					}
+				});
+			} };
+		}
+	},
+	"no-graphql-cast": {
+		meta: {
+			type: "problem",
+			docs: {
+				description: "Require `graphql<unknown>` so GraphQL responses are parsed, not cast.",
+				recommended: false,
+				url: docsUrl("no-graphql-cast")
+			},
+			messages: { cast: "`graphql<T>` casts the response without checking it. Call `graphql<unknown>` and parse the result with a schema (e.g. Zod)." }
+		},
+		createOnce: function(context) {
+			return { CallExpression: function(node) {
+				const { callee } = node;
+				if (callee.type !== "MemberExpression" || callee.computed || callee.property.name !== "graphql") return;
+				const params = node.typeArguments?.params ?? [];
+				if (params.length === 1 && params[0]?.type === "TSUnknownKeyword") return;
+				context.report({
+					node,
+					messageId: "cast"
+				});
+			} };
+		}
+	},
+	"no-property-type-lookup": {
+		meta: {
+			type: "suggestion",
+			docs: {
+				description: "Disallow looking up a property's type with `T[\"key\"]`; name the type instead.",
+				recommended: true,
+				url: docsUrl("no-property-type-lookup")
+			},
+			messages: { lookup: "Don't look up a property's type with `T[\"{{key}}\"]`. Name that type (an interface, or `z.infer` of its own schema) and use the name." }
+		},
+		createOnce: function(context) {
+			return { TSIndexedAccessType: function(node) {
+				const { indexType } = node;
+				if (indexType.type !== "TSLiteralType" || indexType.literal.type !== "Literal") return;
+				const key = indexType.literal.value;
+				if (typeof key !== "string") return;
+				context.report({
+					node,
+					messageId: "lookup",
+					data: { key }
+				});
+			} };
+		}
+	},
+	"simple-ternary": {
+		meta: {
+			type: "suggestion",
+			docs: {
+				description: "Keep ternaries on one line, testing one condition, without interpolated strings.",
+				recommended: true,
+				url: docsUrl("simple-ternary")
+			},
+			messages: {
+				multiline: "Ternary spans multiple lines. Use if/else or extract a function.",
+				compoundTest: "Ternary test combines conditions with `{{operator}}`. Name the condition or use if/else.",
+				templateBranch: "Ternary branch builds an interpolated string. Build it outside the ternary."
+			}
+		},
+		createOnce: function(context) {
+			return { ConditionalExpression: function(node) {
+				if (node.loc.start.line !== node.loc.end.line) context.report({
+					node,
+					messageId: "multiline"
+				});
+				if (node.test.type === "LogicalExpression") context.report({
+					node: node.test,
+					messageId: "compoundTest",
+					data: { operator: node.test.operator }
+				});
+				for (const branch of [node.consequent, node.alternate]) if (branch.type === "TemplateLiteral" && branch.expressions.length > 0) context.report({
+					node: branch,
+					messageId: "templateBranch"
+				});
+			} };
+		}
+	}
+};
+const recommended = {
+	jsPlugins: ["oxlint-plugin-deslopify"],
+	rules: {
+		...Object.fromEntries(Object.entries(rules).filter(([, rule]) => {
+			return rule.meta?.docs?.recommended === true;
+		}).map(([name]) => {
+			return [`deslopify/${name}`, "error"];
+		})),
+		"func-style": ["error", "declaration"],
+		"object-shorthand": ["error", "never"],
+		curly: ["error", "all"],
+		"typescript/consistent-type-definitions": ["error", "interface"],
+		"arrow-body-style": ["error", "always"],
+		"typescript/no-deprecated": "error",
+		"typescript/no-unsafe-type-assertion": "error",
+		"typescript/no-unsafe-assignment": "error",
+		"typescript/no-unsafe-member-access": "error",
+		"typescript/no-unsafe-argument": "error"
+	}
+};
+const plugin = {
+	...eslintCompatPlugin({
+		meta: { name: "oxlint-plugin-deslopify" },
+		rules
+	}),
+	configs: { recommended }
+};
+//#endregion
+export { plugin as default };

package/package.json

@@ -0,0 +1,62 @@
+{
+  "name": "oxlint-plugin-deslopify",
+  "version": "0.1.0",
+  "description": "Oxlint rules that keep AI agents from writing sloppy code",
+  "keywords": [
+    "ai",
+    "eslint",
+    "eslint-plugin",
+    "eslintplugin",
+    "lint",
+    "oxlint",
+    "oxlint-plugin"
+  ],
+  "homepage": "https://github.com/BrianEstrada/oxlint-deslopify#readme",
+  "bugs": "https://github.com/BrianEstrada/oxlint-deslopify/issues",
+  "license": "MIT",
+  "author": "Brian Estrada",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/BrianEstrada/oxlint-deslopify.git"
+  },
+  "files": [
+    "dist"
+  ],
+  "type": "module",
+  "types": "./dist/index.d.mts",
+  "exports": {
+    ".": "./dist/index.mjs",
+    "./package.json": "./package.json"
+  },
+  "scripts": {
+    "build": "tsdown",
+    "typecheck": "tsc --noEmit",
+    "fmt": "oxfmt",
+    "fmt:check": "oxfmt --check",
+    "lint": "oxlint",
+    "lint:fix": "oxlint --fix",
+    "test": "node --test 'src/**/*.test.ts'",
+    "check": "bun run typecheck && bun run lint && bun run fmt:check && bun run test && bun run build",
+    "prepublishOnly": "bun run build"
+  },
+  "dependencies": {
+    "@oxlint/plugins": "^1.86.0"
+  },
+  "devDependencies": {
+    "@commitlint/cli": "21.2.3",
+    "@commitlint/config-conventional": "21.2.3",
+    "@types/node": "26.6.4",
+    "lefthook": "2.1.16",
+    "oxfmt": "0.71.0",
+    "oxlint": "1.86.0",
+    "oxlint-tsgolint": "7.0.2003",
+    "tsdown": "0.23.0",
+    "typescript": "7.0.2"
+  },
+  "peerDependencies": {
+    "oxlint": ">=1.86.0"
+  },
+  "engines": {
+    "node": "^20.19.0 || >=22.12.0"
+  }
+}