tackbox 0.1.80 → 0.1.82

package/bin/tackbox-eslint.js

@@ -115,8 +115,10 @@ function validateDeclarations(decls) {
 // line: null (location-unknown) - the caller over-reports, never drops it.
 function emitMachine(results) {
   for (const r of results) {
-    // The contract path is POSIX; path.relative yields `\` on Windows.
-    const file = path.relative(process.cwd(), r.filePath).replace(/\\/g, '/')
+    // The contract path is POSIX; path.relative yields `\` on Windows. Split on
+    // the platform separator, never a blanket replace: off Windows a `\` in a
+    // name is part of the name and rewriting it misfiles the finding.
+    const file = path.relative(process.cwd(), r.filePath).split(path.sep).join('/')
     for (const m of r.messages) {
       if (m.severity !== 2) continue
       process.stdout.write(JSON.stringify({ file, line: m.line ?? null, rule: m.ruleId, message: m.message }) + '\n')

package/js/rules/_shared.js

@@ -159,9 +159,11 @@ function declaredReporters(context) {
 }
 
 // Declarations are POSIX; path.relative hands back `\` on Windows, and a key
-// that keeps it stops matching its own declaration.
+// that keeps it stops matching its own declaration. Only the platform separator
+// is rewritten - off Windows `\` is a legal file name character, and replacing
+// it would invent a key that never matches git's own spelling.
 function toPosix(p) {
-  return p.replace(/\\/g, '/')
+  return p.split(path.sep).join('/')
 }
 
 function relFile(context) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tackbox",
-  "version": "0.1.80",
+  "version": "0.1.82",
   "description": "ESLint and Markdown lint plugins plus direct error-reporting helpers for JavaScript and TypeScript.",
   "license": "MIT",
   "main": "./js/eslint-plugin.js",