@dereekb/dbx-web 14.0.0 → 14.1.0

package/eslint/index.esm.js

@@ -2864,6 +2864,45 @@ function _unsupported_iterable_to_array(o, minLen) {
     }
 };
 
+/**
+ * Token the autofix appends to an action host that presents no errors.
+ */ var DBX_ACTION_ERROR_FIX_TOKEN = 'dbxActionSnackbarError';
+/**
+ * Builds the fix that appends {@link DBX_ACTION_ERROR_FIX_TOKEN} to the action host's
+ * start tag, inserting just inside the closing `>` (or before the `/` of a
+ * self-closing `/>`).
+ *
+ * Returns `null` — declining to fix — whenever the start tag is not shaped the way
+ * this expects, so a malformed or unexpected template is left alone rather than
+ * rewritten on a guess.
+ *
+ * @param context - The rule context, used for the template source text.
+ * @param node - The action host element being reported.
+ * @param fixer - The ESLint fixer.
+ * @returns The fix, or null when the start tag could not be located.
+ */ function actionErrorDirectiveFix(context, node, fixer) {
+    var _context_sourceCode_getText, _context_sourceCode;
+    var startSpan = node.startSourceSpan;
+    var text = (_context_sourceCode = context.sourceCode) === null || _context_sourceCode === void 0 ? void 0 : (_context_sourceCode_getText = _context_sourceCode.getText) === null || _context_sourceCode_getText === void 0 ? void 0 : _context_sourceCode_getText.call(_context_sourceCode);
+    var result = null;
+    if (startSpan && text) {
+        var closeOffset = startSpan.end.offset - 1;
+        if (text[closeOffset] === '>') {
+            // step back over the '/' of a self-closing tag so the token lands inside the tag
+            var insertAt = text[closeOffset - 1] === '/' ? closeOffset - 1 : closeOffset;
+            // then back over any trailing whitespace, so the token appends directly after the
+            // last attribute rather than doubling a space or landing past a newline-indented '>'
+            while(insertAt > 0 && /\s/.test(text[insertAt - 1])){
+                insertAt -= 1;
+            }
+            result = fixer.replaceTextRange([
+                insertAt,
+                insertAt
+            ], " ".concat(DBX_ACTION_ERROR_FIX_TOKEN));
+        }
+    }
+    return result;
+}
 /**
  * ESLint (Angular template) rule that flags a `dbxAction` which runs work (has a
  * handler or a trigger) but presents no errors to the user.
@@ -2873,10 +2912,22 @@ function _unsupported_iterable_to_array(o, minLen) {
  *
  * Shares the same context-scoping bail conditions as `require-action-value-source`
  * (`[dbxActionSource]` on self/ancestor, or a nested `dbxAction`).
+ *
+ * The autofix appends the bare `dbxActionSnackbarError` token to the action host's
+ * start tag — the canonical default, and the form every compliant call site uses.
+ *
+ * IMPORTANT: `DbxActionSnackbarErrorDirective` is a standalone directive from
+ * `@dereekb/dbx-web`, so the fix is only complete once the component also lists it
+ * in `imports`. A template fixer cannot add that (for an external `.html` the
+ * imports live in a different file entirely), and Angular does NOT reject an
+ * unmatched bare attribute — it degrades to an inert DOM attribute. So `--fix`
+ * silences this rule whether or not the import lands. Always add the import
+ * alongside the fix, and prefer `dbxActionErrorHandler` (which lives in
+ * `@dereekb/dbx-core`) wherever a dbx-web dependency would be a layering violation.
  */ var DBX_WEB_REQUIRE_ACTION_ERROR_HANDLER_RULE = {
     meta: {
         type: 'suggestion',
-        fixable: undefined,
+        fixable: 'code',
         docs: {
             description: 'Require an error-presentation directive on a dbxAction that runs a handler so failures surface to the user.',
             recommended: false
@@ -2915,7 +2966,10 @@ function _unsupported_iterable_to_array(o, minLen) {
                 if (!hasErrorDirective) {
                     context.report({
                         loc: actionElementLoc(parserServices, node),
-                        messageId: 'missingErrorHandler'
+                        messageId: 'missingErrorHandler',
+                        fix: function fix(fixer) {
+                            return actionErrorDirectiveFix(context, node, fixer);
+                        }
                     });
                 }
             }

package/eslint/package.json

@@ -1,16 +1,16 @@
 {
   "name": "@dereekb/dbx-web/eslint",
-  "version": "14.0.0",
+  "version": "14.1.0",
   "type": "module",
   "peerDependencies": {
-    "@dereekb/util": "14.0.0",
+    "@dereekb/util": "14.1.0",
     "@typescript-eslint/utils": "8.69.0"
   },
   "devDependencies": {
     "@angular-eslint/template-parser": "22.2.0",
     "@angular/core": "22.1.4",
-    "@dereekb/dbx-core": "14.0.0",
-    "@dereekb/rxjs": "14.0.0",
+    "@dereekb/dbx-core": "14.1.0",
+    "@dereekb/rxjs": "14.1.0",
     "@typescript-eslint/parser": "8.69.0",
     "eslint": "10.9.1",
     "rxjs": "^7.8.2"

package/eslint/src/lib/require-action-error-handler.rule.d.ts

@@ -5,7 +5,7 @@ import { type AstNode } from './util';
 export interface DbxWebRequireActionErrorHandlerRuleDefinition {
     readonly meta: {
         readonly type: 'suggestion';
-        readonly fixable: undefined;
+        readonly fixable: 'code';
         readonly docs: {
             readonly description: string;
             readonly recommended: boolean;
@@ -26,5 +26,17 @@ export interface DbxWebRequireActionErrorHandlerRuleDefinition {
  *
  * Shares the same context-scoping bail conditions as `require-action-value-source`
  * (`[dbxActionSource]` on self/ancestor, or a nested `dbxAction`).
+ *
+ * The autofix appends the bare `dbxActionSnackbarError` token to the action host's
+ * start tag — the canonical default, and the form every compliant call site uses.
+ *
+ * IMPORTANT: `DbxActionSnackbarErrorDirective` is a standalone directive from
+ * `@dereekb/dbx-web`, so the fix is only complete once the component also lists it
+ * in `imports`. A template fixer cannot add that (for an external `.html` the
+ * imports live in a different file entirely), and Angular does NOT reject an
+ * unmatched bare attribute — it degrades to an inert DOM attribute. So `--fix`
+ * silences this rule whether or not the import lands. Always add the import
+ * alongside the fix, and prefer `dbxActionErrorHandler` (which lives in
+ * `@dereekb/dbx-core`) wherever a dbx-web dependency would be a layering violation.
  */
 export declare const DBX_WEB_REQUIRE_ACTION_ERROR_HANDLER_RULE: DbxWebRequireActionErrorHandlerRuleDefinition;

package/fesm2022/dereekb-dbx-web-mapbox.mjs

@@ -597,7 +597,17 @@ class DbxMapboxMapStore extends ComponentStore {
     setMinimumVirtualViewportSize = this.updater((state, minimumVirtualViewportSize) => ({ ...state, minimumVirtualViewportSize }));
     setUseVirtualBound = this.updater((state, useVirtualBound) => ({ ...state, useVirtualBound }));
     setBoundRefreshSettings = this.updater((state, boundRefreshSettings) => ({ ...state, boundRefreshSettings: { ...state.boundRefreshSettings, ...boundRefreshSettings } }));
-    _setMapService = this.updater((state, mapService) => ({ mapService, moveState: 'init', lifecycleState: 'init', zoomState: 'init', rotateState: 'init', retainContent: state.retainContent, drawerContent: state.retainContent ? state.drawerContent : undefined, useVirtualBound: state.useVirtualBound, boundRefreshSettings: state.boundRefreshSettings }));
+    _setMapService = this.updater((state, mapService) => ({
+        mapService,
+        moveState: 'init',
+        lifecycleState: 'init',
+        zoomState: 'init',
+        rotateState: 'init',
+        retainContent: state.retainContent,
+        drawerContent: state.retainContent ? state.drawerContent : undefined,
+        useVirtualBound: state.useVirtualBound,
+        boundRefreshSettings: state.boundRefreshSettings
+    }));
     _setLifecycleState = this.updater((state, lifecycleState) => ({ ...state, lifecycleState }));
     _setMoveState = this.updater((state, moveState) => ({ ...state, moveState }));
     _setZoomState = this.updater((state, zoomState) => ({ ...state, zoomState }));