oxlint-plugin-vize 0.429.1 → 0.429.2

package/README.md

@@ -6,7 +6,7 @@ This package lets Oxlint execute Patina through Vize's native binding while stil
 
 > [!IMPORTANT]
 > `oxlint-plugin-vize` is a terminal-first Vue SFC linting package.
-> Until upstream Vue support in Oxlint matures, prefer `oxlint-vize -f stylish` for day-to-day output and treat machine-readable / full-fidelity original-SFC reporting as best-effort.
+> Until upstream Vue support in Oxlint matures, use `oxlint-vize` for original SFC locations in terminal and JSON output. Direct `oxlint` still has JS plugin location limits.
 
 ## Main Features
 
@@ -15,7 +15,7 @@ This package lets Oxlint execute Patina through Vize's native binding while stil
 - Ships preset rule maps for JS/TS Oxlint configs: `configs.recommended`, `configs.happyPath`, `configs.essential`, `configs.ecosystem`, `configs.opinionated`, `configs.nuxt`, `configs.all`, and type-aware opt-in variants.
 - Ships `createVizeLintConfig()`, `flatConfigs`, and `defineVizeLintConfig()` for the Vite+ `lint` block in `vite.config.ts`, which is the only Oxlint configuration `vp lint` and `vp check` read.
 - Supports runtime settings through `settings.vize`, including `locale`, `preset`, and `helpLevel`.
-- Provides the `oxlint-vize` CLI wrapper, which runs Oxlint with a scriptless-SFC workaround and rewrites temporary paths back to the original `.vue` files.
+- Provides the `oxlint-vize` CLI wrapper, which runs Oxlint with a temporary location bridge for Vue and HTML files and reports their original paths and Patina locations.
 - Resolves Vize native bindings through platform-specific optional dependencies, so published installs do not need a separate `@vizejs/native` package.
 - Caches file contents and native rule results for the lifetime of the Oxlint process, reducing duplicate work when several Vize rules are enabled for the same file.
 
@@ -229,20 +229,18 @@ For day-to-day terminal runs, the recommended command today is:
 vp exec oxlint-vize -c .oxlintrc.json -f stylish src
 ```
 
-`oxlint-vize` is a thin wrapper around `oxlint`. Until upstream JS plugin coverage improves, it appends a temporary `<script setup>` block only for scriptless `.vue` files so Oxlint's JS plugin pipeline still invokes Vize, then rewrites reported paths back to the original files.
-`stylish` is currently the most usable compromise for mixed Oxlint + Vize output because the Patina summary can inline the original SFC location even though Oxlint still anchors JS plugin diagnostics to the extracted script program.
+`oxlint-vize` appends a temporary `<script setup>` location bridge for Vue and HTML files, including files with existing scripts. This lets Patina report the original SFC line and column through Oxlint's JS plugin pipeline. The wrapper rewrites temporary paths back to the original files; `-f json` retains those original positions for machine-readable diagnostics.
 
 ## Limitations
 
-- Raw `oxlint` still misses files without `<script>` or `<script setup>`. The temporary `oxlint-vize` wrapper works around this by generating a transient script block for scriptless `.vue` files before invoking `oxlint`.
-- Oxlint JS plugins only accept ranges inside the extracted Vue script program. For template diagnostics, Vize now inlines the original SFC block and `line:column` into the summary, while the formatter anchor still points at the script block.
-- Formatter parity is not there yet. `stylish` is recommended for human-readable terminal output, while `json` and other machine-readable outputs are best treated as debugging aids for original template/style positions.
+- Raw `oxlint` still misses files without `<script>` or `<script setup>`. It also anchors template diagnostics to an extracted script instead of the original template location. Use `oxlint-vize` when linting Vue and HTML files.
+- The wrapper's temporary files are not copied back when Oxlint fix flags are used; it warns when `--fix` or `--fix-suggestions` is passed.
 - Oxlint core rules that need JavaScript bindings extracted from Vue templates, such as template-aware unused-variable checks, still depend on upstream work in [Oxc's Better Vue Support](https://github.com/oxc-project/oxc/issues/15761).
-- Vize's own SFC diagnostics can run through the plugin, but precise original-SFC ranges across all Oxlint formatters depend on the JS plugin reporting work tracked in [oxc-project/oxc#20465](https://github.com/oxc-project/oxc/issues/20465).
+- Direct `oxlint` original-SFC ranges depend on the JS plugin reporting work tracked in [oxc-project/oxc#20465](https://github.com/oxc-project/oxc/issues/20465).
 - Type-aware Vize rules are experimental and excluded from the default exported configs. Opt into them explicitly with `configs.recommendedWithTypeAware`, `configs.opinionatedWithTypeAware`, or `createVizeRuleConfig({ includeTypeAware: true, preset: ... })`, and use `settings.vize.typeAware: true` when you want the shared full-file pass to run them eagerly.
 
 ## Current expectations
 
 - This release is meant for terminal-first workflows.
-- It is not yet a promise of precise original-SFC spans across every Oxlint formatter.
-- Once Oxlint can preserve original Vue positions for JS plugins reliably, Vize can improve formatter parity and machine-readable reporting without relying on summary fallbacks.
+- The `oxlint-vize` wrapper reports original Patina positions in `stylish` and `json`; direct `oxlint` remains limited by its extracted-script JS plugin input.
+- Once Oxlint preserves original Vue positions for JS plugins, the temporary location bridge can be removed.

package/dist/cli.mjs

@@ -1,4 +1,4 @@
-import { i as hasVueLikeExtension, n as hasScriptLikeBlock, s as isStandaloneHtmlFile, t as appendScriptlessWorkaround } from "./workaround-DRe3y2CB.mjs";
+import { i as hasVueLikeExtension, s as isStandaloneHtmlFile, t as appendScriptlessWorkaround } from "./workaround-CcnA_oWy.mjs";
 import { spawn, spawnSync } from "node:child_process";
 import path from "node:path";
 import fs from "node:fs";
@@ -200,7 +200,6 @@ function prepareScriptlessWorkaroundFiles(cwd, filenames) {
 	for (const filename of filenames) {
 		const source = fs.readFileSync(filename, "utf8");
 		const isStandaloneHtml = isStandaloneHtmlFile(filename);
-		if (!isStandaloneHtml && hasScriptLikeBlock(source)) continue;
 		const relativeFilename = path.relative(cwd, filename);
 		if (tempDir == null) {
 			const created = createWorkaroundTempDir(nodeModulesDir, tempRoot);

package/dist/index.mjs

@@ -1,9 +1,26 @@
-import { a as isPatinaFile, c as compareLineColumn, d as getDiagnosticBlock, l as extractSfcBlocks, o as isScriptLikeFile, r as resolveWorkaroundSource, u as formatBlockLabel } from "./workaround-DRe3y2CB.mjs";
+import { a as isPatinaFile, c as compareLineColumn, d as getDiagnosticBlock, l as extractSfcBlocks, n as isLocationBridgeProgram, o as isScriptLikeFile, r as resolveWorkaroundSource, u as formatBlockLabel } from "./workaround-CcnA_oWy.mjs";
 import { createRequire } from "node:module";
 import { definePlugin, defineRule } from "@oxlint/plugins";
 import { spawnSync } from "node:child_process";
 import path from "node:path";
 import fs from "node:fs";
+//#region src/plugin-option-guards.ts
+function optionField(value) {
+	return value === void 0 || value === "always" || value === "never" || value === "any";
+}
+function optionalBooleanField(value) {
+	return value === void 0 || typeof value === "boolean";
+}
+function isString(value) {
+	return typeof value === "string";
+}
+function isRecord$1(value) {
+	return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function hasOnlyKeys(value, keys) {
+	return Object.keys(value).every((key) => keys.includes(key));
+}
+//#endregion
 //#region src/native.ts
 const require = createRequire(import.meta.url);
 const FALLBACK_BINDING_PACKAGE = "@vizejs/native";
@@ -553,29 +570,36 @@ function createSummaryLine(message) {
 }
 //#endregion
 //#region src/plugin.ts
-function createOxlintDiagnostic(diagnostic, state, scriptMap, helpLevel) {
+function createOxlintDiagnostic(diagnostic, state, scriptMap, helpLevel, program) {
 	const loc = state.usesOriginalLocations ? createOriginalSfcLoc(diagnostic) : mapToScriptLoc(diagnostic, scriptMap);
 	const block = loc === null ? getDiagnosticBlock(diagnostic, getSfcBlocks(state)) : null;
+	const message = formatPatinaMessage(diagnostic, {
+		hasMappedLocation: loc !== null,
+		blockLabel: formatBlockLabel(block),
+		helpLevel
+	});
+	if (loc == null && state.extractedScript.length === 0) return {
+		node: program,
+		message
+	};
+	const fallbackColumn = state.extractedScript.length === 0 ? 0 : 1;
 	return {
 		loc: loc ?? {
 			start: {
 				line: 1,
-				column: 1
+				column: fallbackColumn
 			},
 			end: {
 				line: 1,
-				column: 1
+				column: fallbackColumn
 			}
 		},
-		message: formatPatinaMessage(diagnostic, {
-			hasMappedLocation: loc !== null,
-			blockLabel: formatBlockLabel(block),
-			helpLevel
-		})
+		message
 	};
 }
 function shouldReportForCurrentProgram(diagnostic, state, scriptMap) {
-	if (state.usesOriginalLocations || scriptMap == null) return true;
+	if (state.usesOriginalLocations) return isScriptLikeFile(state.filename) || isLocationBridgeProgram(state.extractedScript);
+	if (scriptMap == null) return true;
 	const block = getDiagnosticBlock(diagnostic, getSfcBlocks(state));
 	return !isScriptBlock(block) || block === scriptMap.block;
 }
@@ -602,7 +626,7 @@ function createPatinaRule(ruleMeta) {
 			schema: ruleOptionsSchema(ruleMeta.name)
 		},
 		createOnce(context) {
-			return { Program() {
+			return { Program(program) {
 				if (!isPatinaFile(context.filename)) return;
 				const settings = getVizeSettings(context);
 				const activePreset = getActivePreset(settings);
@@ -615,7 +639,7 @@ function createPatinaRule(ruleMeta) {
 				if (diagnostics.length === 0) return;
 				for (const diagnostic of diagnostics) {
 					if (!markDiagnosticAsReported(state, diagnostic)) continue;
-					context.report(createOxlintDiagnostic(diagnostic, state, scriptMap, helpLevel));
+					context.report(createOxlintDiagnostic(diagnostic, state, scriptMap, helpLevel, program));
 				}
 			} };
 		}
@@ -740,21 +764,6 @@ function isHtmlSelfClosingOption(value) {
 function isHyphenationStyle(value) {
 	return value === "always" || value === "never";
 }
-function optionField(value) {
-	return value === void 0 || value === "always" || value === "never" || value === "any";
-}
-function optionalBooleanField(value) {
-	return value === void 0 || typeof value === "boolean";
-}
-function isString(value) {
-	return typeof value === "string";
-}
-function isRecord$1(value) {
-	return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-function hasOnlyKeys(value, keys) {
-	return Object.keys(value).every((key) => keys.includes(key));
-}
 var plugin_default = definePlugin({
 	meta: { name: "vize" },
 	rules: Object.fromEntries(getPatinaRules().map((ruleMeta) => [ruleMeta.name, createPatinaRule(ruleMeta)]))

package/dist/{workaround-DRe3y2CB.mjs → workaround-CcnA_oWy.mjs}

@@ -141,12 +141,16 @@ function isStandaloneHtmlFile(filename) {
 const SCRIPTLESS_WORKAROUND_MARKER = "oxlint-plugin-vize-scriptless";
 const SCRIPTLESS_WORKAROUND_OPEN_TAG_PREFIX = `<script setup lang="ts" data-${SCRIPTLESS_WORKAROUND_MARKER}="`;
 const SCRIPTLESS_WORKAROUND_FILENAME_ATTR = `data-${SCRIPTLESS_WORKAROUND_MARKER}="`;
+const LOCATION_BRIDGE_MARKER = "/*vize-location-bridge*/";
 function hasScriptLikeBlock(source) {
 	return extractSfcBlocks(source).some((block) => block.kind === "script" || block.kind === "script-setup");
 }
 function appendScriptlessWorkaround(source, filename) {
 	return `${createWorkaroundScript(source, filename)}${source}`;
 }
+function isLocationBridgeProgram(extractedScript) {
+	return extractedScript.trimEnd().endsWith(LOCATION_BRIDGE_MARKER);
+}
 function resolveWorkaroundSource(source, fallbackFilename) {
 	const workaroundBlock = getPrependedWorkaroundBlock(source);
 	if (workaroundBlock == null) return {
@@ -167,7 +171,7 @@ function getPrependedWorkaroundBlock(source) {
 	if (!source.startsWith(SCRIPTLESS_WORKAROUND_OPEN_TAG_PREFIX)) return null;
 	const [firstBlock] = extractSfcBlocks(source);
 	if (!firstBlock || firstBlock.kind !== "script-setup") return null;
-	if (!isWhitespaceOnly(firstBlock.content)) return null;
+	if (!firstBlock.content.endsWith(LOCATION_BRIDGE_MARKER) || !isWhitespaceOnly(firstBlock.content.slice(0, -24))) return null;
 	const contentStart = offsetFromLineColumn(source, firstBlock.contentStart);
 	const contentEnd = offsetFromLineColumn(source, firstBlock.contentEnd);
 	const encodedFilename = encodedFilenameFromWorkaroundOpenTag(source.slice(0, contentStart));
@@ -175,7 +179,7 @@ function getPrependedWorkaroundBlock(source) {
 	const closeTagEnd = contentEnd + 9;
 	const strippedSourceStart = sourceStartAfterSyntheticBlock(source, closeTagEnd);
 	if (strippedSourceStart == null) return null;
-	if (firstBlock.content !== createWhitespaceMirror(source.slice(strippedSourceStart))) return null;
+	if (firstBlock.content !== createWhitespaceMirror(source.slice(strippedSourceStart)) + LOCATION_BRIDGE_MARKER) return null;
 	return {
 		closeTagEnd,
 		encodedFilename
@@ -194,7 +198,7 @@ function encodedFilenameFromWorkaroundOpenTag(openTag) {
 	return openTag.slice(encodedFilenameStart, encodedFilenameEnd);
 }
 function createWorkaroundScript(source, filename) {
-	return `${SCRIPTLESS_WORKAROUND_OPEN_TAG_PREFIX}${encodeWorkaroundFilename(filename)}">${createWhitespaceMirror(source)}<\/script>\n`;
+	return `${SCRIPTLESS_WORKAROUND_OPEN_TAG_PREFIX}${encodeWorkaroundFilename(filename)}">${createWhitespaceMirror(source)}${LOCATION_BRIDGE_MARKER}<\/script>\n`;
 }
 function createWhitespaceMirror(source) {
 	return source.replaceAll(/[^\r\n]/gu, " ");
@@ -251,4 +255,4 @@ if (import.meta.vitest) {
 	});
 }
 //#endregion
-export { isPatinaFile as a, compareLineColumn as c, getDiagnosticBlock as d, hasVueLikeExtension as i, extractSfcBlocks as l, hasScriptLikeBlock as n, isScriptLikeFile as o, resolveWorkaroundSource as r, isStandaloneHtmlFile as s, appendScriptlessWorkaround as t, formatBlockLabel as u };
+export { isPatinaFile as a, compareLineColumn as c, getDiagnosticBlock as d, hasVueLikeExtension as i, extractSfcBlocks as l, isLocationBridgeProgram as n, isScriptLikeFile as o, resolveWorkaroundSource as r, isStandaloneHtmlFile as s, appendScriptlessWorkaround as t, formatBlockLabel as u };

package/package.json

@@ -12,7 +12,7 @@
   "devDependencies": {
     "@tsdown/css": "0.22.14",
     "@types/node": "25.9.2",
-    "@vizejs/native": "0.429.1",
+    "@vizejs/native": "0.429.2",
     "tsdown": "0.22.14",
     "typescript": "6.0.3",
     "vite": "npm:@voidzero-dev/vite-plus-core@0.1.24",
@@ -49,14 +49,14 @@
   },
   "name": "oxlint-plugin-vize",
   "optionalDependencies": {
-    "@vizejs/native-darwin-arm64": "0.429.1",
-    "@vizejs/native-darwin-x64": "0.429.1",
-    "@vizejs/native-linux-arm64-gnu": "0.429.1",
-    "@vizejs/native-linux-arm64-musl": "0.429.1",
-    "@vizejs/native-linux-x64-gnu": "0.429.1",
-    "@vizejs/native-linux-x64-musl": "0.429.1",
-    "@vizejs/native-win32-arm64-msvc": "0.429.1",
-    "@vizejs/native-win32-x64-msvc": "0.429.1"
+    "@vizejs/native-darwin-arm64": "0.429.2",
+    "@vizejs/native-darwin-x64": "0.429.2",
+    "@vizejs/native-linux-arm64-gnu": "0.429.2",
+    "@vizejs/native-linux-arm64-musl": "0.429.2",
+    "@vizejs/native-linux-x64-gnu": "0.429.2",
+    "@vizejs/native-linux-x64-musl": "0.429.2",
+    "@vizejs/native-win32-arm64-msvc": "0.429.2",
+    "@vizejs/native-win32-x64-msvc": "0.429.2"
   },
   "publishConfig": {
     "access": "public"
@@ -68,7 +68,7 @@
   },
   "type": "module",
   "types": "./dist/index.d.mts",
-  "version": "0.429.1",
+  "version": "0.429.2",
   "scripts": {
     "build": "vp pack",
     "check": "vp check src vite.config.ts",