tackbox 0.1.78 → 0.1.80

package/bin/tackbox-eslint.js

@@ -115,7 +115,8 @@ 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.
+    const file = path.relative(process.cwd(), r.filePath).replace(/\\/g, '/')
     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/report.js

@@ -1,6 +1,5 @@
 // Browser report helper: empty DSN = log-only no-op.
-// Mirrors the gmux/sts-wand pattern — single funnel for toast,
-// Sentry capture, optional diagnostic stream.
+// Single funnel for toast, Sentry capture, optional diagnostic stream.
 
 const Sentry = require('@sentry/browser')
 

package/js/rules/_shared.js

@@ -158,10 +158,16 @@ 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.
+function toPosix(p) {
+  return p.replace(/\\/g, '/')
+}
+
 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 +241,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 +258,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
@@ -438,9 +444,9 @@ function hasMarkerAbove(context, node, prefix) {
 // One coherent path analysis for all three legal catch exits: throw and a
 // Result-boundary return terminate a path; a recognized reporter call is a
 // sticky event (statements after it on the path are fine). A path reaching the
-// end of the handler without a terminator or event swallows. Ported from gmux
-// makeHandledAnalysis; reporter recognition stays tackbox origin-gating.
-// Result-boundary is kin to the policy layer (specs/general/error-policies.md):
+// end of the handler without a terminator or event swallows. Reporter
+// recognition stays tackbox origin-gating.
+// Result-boundary is kin to the error-policy layer:
 // annotation-based (no type program) - only a syntactic Result / Attempt /
 // Promise<Result|Attempt> return type earns the boundary credit.
 
@@ -633,7 +639,7 @@ function isBareErrReturn(expr, errName) {
 // return), 'bad' (some path exits unhandled), { reported } (falls through;
 // reported true when every falling path passed the sticky event). Constructs
 // not modeled (switch, loops, nested try) are opaque: a hidden return fails
-// closed, reporters inside do not count. Ported from gmux. returnIdentity
+// closed, reporters inside do not count. returnIdentity
 // credits `return <errName>` as terminal (promise handlers only: the settled
 // value is the error object itself).
 function makeHandledAnalysis(opts) {

package/js/rules/no-swallow-catch.js

@@ -8,7 +8,7 @@ const {
 module.exports = {
   meta: {
     type: 'problem',
-    docs: { description: 'every path out of a catch must throw, call a reporter, convert to a Result boundary (return { ok: false, cause: err } when the function returns Result/Attempt), or the try must carry a // no-report: marker. Boundary conversion is kin to the policy layer (specs/general/error-policies.md).' },
+    docs: { description: 'every path out of a catch must throw, call a reporter, convert to a Result boundary (return { ok: false, cause: err } when the function returns Result/Attempt), or the try must carry a // no-report: marker. Boundary conversion is kin to the policy layer.' },
     messages: {
       swallow: 'every catch path must throw, call a reporter, or convert to a Result boundary',
     },

package/package.json

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