@rdjperron/lint 0.1.0

package/dist/rule-options-DrAXz8BD.mjs

@@ -0,0 +1,18 @@
+//#region src/shared/rule-options.ts
+/**
+* Rule options arrive as parsed JSON from the linter config, so this is the one
+* place their shape is checked. Returns the first option when it is an object.
+*/
+function firstOptionObject(options) {
+	const first = options[0];
+	return typeof first === "object" && first !== null && !Array.isArray(first) ? first : {};
+}
+/** A string-valued key of the first option object, or undefined when absent or not a string. */
+function stringOption(options, key) {
+	const value = firstOptionObject(options)[key];
+	return typeof value === "string" ? value : void 0;
+}
+//#endregion
+export { stringOption as t };
+
+//# sourceMappingURL=rule-options-DrAXz8BD.mjs.map

package/dist/rule-options-DrAXz8BD.mjs.map

@@ -0,0 +1 @@
+{"version":3,"file":"rule-options-DrAXz8BD.mjs","names":[],"sources":["../src/shared/rule-options.ts"],"sourcesContent":["import type { Options } from \"@oxlint/plugins\";\n\ntype OptionValue = Options[number];\n\n/**\n * Rule options arrive as parsed JSON from the linter config, so this is the one\n * place their shape is checked. Returns the first option when it is an object.\n */\nfunction firstOptionObject(options: Readonly<Options>): { readonly [key: string]: OptionValue } {\n  const first = options[0];\n  return typeof first === \"object\" && first !== null && !Array.isArray(first) ? first : {};\n}\n\n/** A string-valued key of the first option object, or undefined when absent or not a string. */\nexport function stringOption(options: Readonly<Options>, key: string): string | undefined {\n  const value = firstOptionObject(options)[key];\n  return typeof value === \"string\" ? value : undefined;\n}\n"],"mappings":";;;;;AAQA,SAAS,kBAAkB,SAAqE;CAC9F,MAAM,QAAQ,QAAQ;CACtB,OAAO,OAAO,UAAU,YAAY,UAAU,QAAQ,CAAC,MAAM,QAAQ,KAAK,IAAI,QAAQ,CAAC;AACzF;;AAGA,SAAgB,aAAa,SAA4B,KAAiC;CACxF,MAAM,QAAQ,kBAAkB,OAAO,CAAC,CAAC;CACzC,OAAO,OAAO,UAAU,WAAW,QAAQ,KAAA;AAC7C"}

package/fixtures/README.md

@@ -0,0 +1,63 @@
+# fixtures: load-bearing test data, do not delete
+
+One deliberately violating file per rule, in one directory per pack:
+`anti-slop/<rule>.fixture.ts`, `react/no-use-effect.fixture.tsx`,
+`i18n/no-hardcoded-jsx-text.fixture.tsx` and `expo/static-public-env.fixture.ts`. Each file trips
+exactly the rule it is named after, and nothing else.
+
+These ship in the published package (`files` in `package.json`), so a consumer proves the rules
+fire through its own linter, not ours.
+
+## Why this exists
+
+The linter lives inside the consumer's toolchain (under Vite+, inside the `vp` binary) and moves
+independently of any lockfile. The rules run through oxlint's JS-plugin bridge. A rule can stop
+firing with no diff anywhere: after a linter upgrade, after a plugin API skew, after an ignore
+pattern swallows the wrong path.
+
+And a dead lint rule is silent. "Rule reports nothing" looks identical to "code has no
+violations". Nothing else would notice.
+
+The source repo's `scripts/verify-lint-fixtures.mjs` (in CI and in `vp run ready`) lints each
+pack's fixtures and asserts a positive: every rule must fire, each on its own fixture. If a rule
+goes dead, the run fails loudly instead of going quietly green.
+
+## How it relates to the unit tests
+
+Two layers, different questions:
+
+| Layer                            | Question it answers                                                                                            | Where                                                             |
+| -------------------------------- | -------------------------------------------------------------------------------------------------------------- | ----------------------------------------------------------------- |
+| This directory + a verify script | Does each rule fire through the real pipeline: the bundled oxlint, the `jsPlugins` wiring, the actual config? | `fixtures/`, the consumer's `scripts/verify-lint-fixtures.mjs`   |
+| Unit tests (RuleTester)          | Is each rule's logic right? Valid cases stay quiet, invalid cases report, with per-case precision.             | `tests/` in the source repo (not published)                       |
+
+The unit tests catch a locally edited rule that starts over- or under-firing. This layer catches
+the rule not running at all. Both are needed.
+
+## Rules of the house
+
+- Do not delete or "clean up" these files. They are the regression proof for rules nobody
+  upstream maintains for us.
+- In the source repo they are in `lint.ignorePatterns` and `fmt.ignorePatterns`, so a plain
+  `vp check` stays quiet. The verify script reaches them through a transient symlink at
+  `scripts/.lint-fixtures/<pack>`, because no `vp lint` flag re-includes a config-ignored path and
+  oxlint skips `node_modules` by default.
+- Never gitignore that view. oxlint honours `.gitignore`, so gitignoring anything on the
+  verification path silently disables the whole check (this happened once; the script now detects
+  it with `git check-ignore` and says so).
+- Name fixtures `*.fixture.ts` / `*.fixture.tsx`, never `*.test.ts`, or Vitest collects them and
+  the deliberately broken code fails the test run.
+- One rule per file. A fixture that trips two rules makes the per-rule assertion ambiguous.
+- Expected advisory noise in the source repo: the type-aware pass also reports `typescript(TS…)`
+  on the view, because no tsconfig covers `scripts/.lint-fixtures/` (no React types, no `process`
+  types, no `jsx` setting). The script lists those as non-pack diagnostics rather than hiding
+  them; only a pack rule that did not fire fails the run.
+
+## When a rule stops firing after a `vp` upgrade
+
+1. `vp toolchain oxlint`: did the bundled oxlint move?
+2. `vp why @oxlint/plugins`: if `vite-plus` now declares a different version, move the exact
+   pin to it and reinstall. In the source repo the parity test in `tests/` fails until you do.
+3. Re-run the verify script.
+4. Still dead? The rule's AST expectations may have changed under the new oxc version. Read the
+   rule in `src/<pack>/rules/`, fix it, add the case to `tests/rules/<pack>/`, and release.

package/fixtures/anti-slop/no-array-filter-map.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-array-filter-map.
+export function activeEmails(users: { active: boolean; email: string }[]): string[] {
+  return users.filter((user) => user.active).map((user) => user.email);
+}

package/fixtures/anti-slop/no-chained-type-assertions.fixture.ts

@@ -0,0 +1,7 @@
+// Fixture for anti-slop/no-chained-type-assertions.
+const rawCount = "12";
+
+// SAFETY: fixture file; the assertion chain is the behaviour under test.
+const parsedCount = rawCount as unknown as number;
+
+export { parsedCount };

package/fixtures/anti-slop/no-conditional-empty-object-spread.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-conditional-empty-object-spread.
+const includeExtra = true;
+
+export const requestBody = { name: "widget", ...(includeExtra ? { extra: 1 } : {}) };

package/fixtures/anti-slop/no-known-value-widening.fixture.ts

@@ -0,0 +1,2 @@
+// Fixture for anti-slop/no-known-value-widening.
+export const retrySettings: unknown = { retries: 3 };

package/fixtures/anti-slop/no-module-mocking.fixture.ts

@@ -0,0 +1,6 @@
+// Fixture for anti-slop/no-module-mocking.
+// `vi` is deliberately left undeclared: the rule's global/unresolved-reference
+// path is what recognises `vi.mock` without importing from "vitest" (which
+// `vite-plus/prefer-vite-plus-imports` forbids in this repo).
+// @ts-expect-error -- `vi` is intentionally not declared in this fixture.
+vi.mock("./clock.ts");

package/fixtures/anti-slop/no-object-parameters.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-object-parameters.
+export function serialize(payload: object): string {
+  return JSON.stringify(payload);
+}

package/fixtures/anti-slop/no-reduce-accumulator-copy.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-reduce-accumulator-copy.
+export function indexById(items: { id: string }[]): Record<string, { id: string }> {
+  return items.reduce((acc, item) => Object.assign({}, acc, { [item.id]: item }), {});
+}

package/fixtures/anti-slop/no-reflect-apply.fixture.ts

@@ -0,0 +1,6 @@
+// Fixture for anti-slop/no-reflect-apply.
+function add(left: number, right: number): number {
+  return left + right;
+}
+
+export const total = Reflect.apply(add, undefined, [1, 2]);

package/fixtures/anti-slop/no-reflect-get.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-reflect-get.
+const entry = { id: "a" };
+
+export const identifier = Reflect.get(entry, "id");

package/fixtures/anti-slop/no-runtime-typeof.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-runtime-typeof.
+export function describeValue(value: string | number): string {
+  return typeof value === "string" ? "text" : "number";
+}

package/fixtures/anti-slop/no-shape-in-symbol-names.fixture.ts

@@ -0,0 +1,2 @@
+// Fixture for anti-slop/no-shape-in-symbol-names.
+export const shapeCount = 1;

package/fixtures/anti-slop/no-unknown-parameters.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-unknown-parameters.
+export function record(payload: unknown): string {
+  return String(payload);
+}

package/fixtures/anti-slop/no-unknown-returns.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for anti-slop/no-unknown-returns.
+export function load(serialized: string): unknown {
+  return JSON.parse(serialized);
+}

package/fixtures/anti-slop/no-unknown-type-aliases.fixture.ts

@@ -0,0 +1,2 @@
+// Fixture for anti-slop/no-unknown-type-aliases.
+export type Payload = unknown;

package/fixtures/anti-slop/no-unsafe-dictionary-type.fixture.ts

@@ -0,0 +1,2 @@
+// Fixture for anti-slop/no-unsafe-dictionary-type.
+export type Registry = Record<string, unknown>;

package/fixtures/anti-slop/no-widen-then-assert.fixture.ts

@@ -0,0 +1,9 @@
+// Fixture for anti-slop/no-widen-then-assert.
+function project(entry: { id: string }): string {
+  const widened: unknown = entry;
+  // SAFETY: fixture file; the widen-then-assert flow is the behaviour under test.
+  const narrowed = widened as { id: string };
+  return narrowed.id;
+}
+
+export const projected = project({ id: "a" });

package/fixtures/anti-slop/require-safety-comment-for-type-assertion.fixture.ts

@@ -0,0 +1,5 @@
+// Fixture for anti-slop/require-safety-comment-for-type-assertion.
+export function lengthOf(entry: { id: string }): number {
+  const widerEntry = entry as { id: string; extra?: number };
+  return widerEntry.id.length;
+}

package/fixtures/expo/static-public-env.fixture.ts

@@ -0,0 +1,4 @@
+// Fixture for expo/static-public-env.
+const { EXPO_PUBLIC_API_URL } = process.env;
+
+export const apiUrl = EXPO_PUBLIC_API_URL;

package/fixtures/i18n/no-hardcoded-jsx-text.fixture.tsx

@@ -0,0 +1,4 @@
+// Fixture for i18n/no-hardcoded-jsx-text.
+export function Fixture() {
+  return <button type="button">Save changes</button>;
+}

package/fixtures/react/no-use-effect.fixture.tsx

@@ -0,0 +1,9 @@
+// Fixture for react-policy/no-use-effect.
+import { useEffect } from "react";
+
+export function Fixture({ title }: { title: string }) {
+  useEffect(() => {
+    document.title = title;
+  }, [title]);
+  return null;
+}

package/package.json

@@ -0,0 +1,50 @@
+{
+  "name": "@rdjperron/lint",
+  "version": "0.1.0",
+  "description": "Oxlint JS plugins: the anti-slop, react-policy, i18n and expo rule packs, shipped with the fixture that proves each rule fires.",
+  "license": "MIT",
+  "files": [
+    "dist",
+    "fixtures",
+    "README.md"
+  ],
+  "type": "module",
+  "exports": {
+    "./anti-slop": {
+      "types": "./dist/anti-slop.d.mts",
+      "default": "./dist/anti-slop.mjs"
+    },
+    "./react": {
+      "types": "./dist/react.d.mts",
+      "default": "./dist/react.mjs"
+    },
+    "./i18n": {
+      "types": "./dist/i18n.d.mts",
+      "default": "./dist/i18n.mjs"
+    },
+    "./expo": {
+      "types": "./dist/expo.d.mts",
+      "default": "./dist/expo.mjs"
+    },
+    "./fixtures/*": "./fixtures/*",
+    "./package.json": "./package.json"
+  },
+  "devDependencies": {
+    "@oxlint/plugins": "1.79.0",
+    "@types/node": "^24",
+    "typescript": "^7.0.2",
+    "vite": "npm:@voidzero-dev/vite-plus-core@0.3.1",
+    "vite-plus": "0.3.1"
+  },
+  "peerDependencies": {
+    "@oxlint/plugins": "^1.79.0"
+  },
+  "engines": {
+    "node": ">=24"
+  },
+  "scripts": {
+    "build": "vp pack",
+    "test": "vp test",
+    "check": "vp check"
+  }
+}