tackbox 0.1.79 → 0.1.81

package/bin/tackbox-eslint.js

@@ -115,7 +115,10 @@ function validateDeclarations(decls) {
 // line: null (location-unknown) - the caller over-reports, never drops it.
 function emitMachine(results) {
   for (const r of results) {
-    const file = path.relative(process.cwd(), r.filePath)
+    // 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

@@ -158,10 +158,18 @@ function declaredReporters(context) {
   return Array.isArray(s) ? s : []
 }
 
+// Declarations are POSIX; path.relative hands back `\` on Windows, and a key
+// 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.split(path.sep).join('/')
+}
+
 function relFile(context) {
   const fn = context.filename || (context.getFilename && context.getFilename()) || ''
   const cwd = context.cwd || process.cwd()
-  return path.isAbsolute(fn) ? path.relative(cwd, fn) : fn
+  return toPosix(path.isAbsolute(fn) ? path.relative(cwd, fn) : fn)
 }
 
 // Module extensions, compound first: a specifier that omits the extension must
@@ -235,7 +243,7 @@ function resolveAlias(context, source, absImporter) {
     dir = up
   }
   if (root === null) return null
-  return path.relative(context.cwd || process.cwd(), path.join(root, 'src', 'lib', rest))
+  return toPosix(path.relative(context.cwd || process.cwd(), path.join(root, 'src', 'lib', rest)))
 }
 
 // resolveDeclTarget resolves an Identifier callee to the {file, name} of its
@@ -252,7 +260,7 @@ function resolveDeclTarget(context, idNode) {
     const importedName = info.kind === 'named' ? info.imported : idNode.name
     let resolved
     if (source.startsWith('.')) {
-      resolved = path.normalize(path.join(path.dirname(relFile(context)), source))
+      resolved = path.posix.normalize(path.posix.join(path.posix.dirname(relFile(context)), source))
     } else {
       resolved = resolveAlias(context, source, absFile(context))
       if (resolved === null) return null

package/package.json

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