@anzusystems/common-admin 1.47.0-beta.dev-1790143991 → 1.49.0

package/package.json

@@ -3,28 +3,24 @@
   "packageManager": "yarn@4.18.0",
   "files": [
     "dist",
-    "src/eslint/**"
+    "src/eslint"
   ],
-  "module": "./dist/common-admin.js",
+  "module": "./dist/common-admin.es.js",
   "types": "./dist/common-admin.d.ts",
   "exports": {
     ".": {
       "import": "./dist/common-admin.js",
       "types": "./dist/common-admin.d.ts"
     },
-    "./styles": {
-      "types": "./dist/common-admin.css.d.ts",
-      "default": "./dist/common-admin.css"
-    },
+    "./styles": "./dist/common-admin.css",
     "./labs": {
       "import": "./dist/labs.js",
       "types": "./dist/labs.d.ts"
     },
     "./eslint": "./src/eslint/plugin.mjs",
-    "./oxlint": "./src/eslint/oxlint.mjs",
     "./*": "./*"
   },
-  "version": "1.47.0-beta.dev-1790143991",
+  "version": "1.49.0",
   "license": "Apache-2.0",
   "repository": {
     "type": "git",
@@ -34,31 +30,29 @@
     "prepare": "node -e \"if (process.env.SKIP_PLAYWRIGHT_INSTALL !== 'true') { require('child_process').execSync('playwright install chromium --with-deps', { stdio: 'inherit' }) }\"",
     "docs:dev": "vitepress dev docs",
     "docs:build-only": "vitepress build docs",
-    "docs:build": "run-s build docs:build-only",
+    "docs:build": "yarn build ; yarn docs:build-only",
     "docs:preview": "vitepress preview docs",
     "playground:dev": "vite",
     "playground:build": "vite build",
     "playground:build:nm": "vite build --minify false",
-    "build": "vite build --config vite.config.lib.mts",
-    "build:dev": "vite build --minify false --sourcemap true --config vite.config.lib.mts",
-    "lib:build": "run-s --print-name lint:tsc build",
-    "ci": "run-p --aggregate-output --continue-on-error --print-label --silent lint:tsc format:check lint:oxlint lint:eslint lint:stylelint",
+    "build": "vite build --config vite.config.lib.ts",
+    "build:dev": "vite build --minify false --sourcemap true --config vite.config.lib.ts",
+    "lib:build": "vite build --config vite.config.lib.ts",
+    "ci": "run-s --print-name lint:tsc lint:oxlint lint:eslint lint:stylelint",
     "lint": "[ \"$*\" = \"--fix\" ] && yarn lint:fix || yarn ci",
-    "lint:fix": "run-s --print-name lint:tsc lint:oxlint:fix lint:eslint:fix lint:stylelint:fix format",
+    "lint:fix": "run-s --print-name lint:tsc format lint:oxlint:fix lint:eslint:fix lint:stylelint:fix",
     "lint:tsc": "NODE_OPTIONS=--max-old-space-size=4096 vue-tsc --build",
     "lint:tsc:test": "NODE_OPTIONS=--max-old-space-size=4096 vue-tsc --build tsconfig.test.json",
-    "lint:eslint": "eslint --cache --cache-strategy content",
-    "lint:eslint:fix": "eslint --fix --cache --cache-strategy content",
-    "lint:oxlint": "oxlint . --disable-nested-config",
-    "lint:oxlint:fix": "oxlint . --fix --disable-nested-config",
-    "lint:stylelint": "stylelint src/**/*.{scss,vue} --cache --cache-strategy content",
-    "lint:stylelint:fix": "stylelint src/**/*.{scss,vue} --fix --cache --cache-strategy content",
-    "format": "oxfmt --disable-nested-config src vite.config.lib.mts vite.config.mts vitest.config.mts eslint.config.mjs",
-    "format:check": "oxfmt --disable-nested-config src vite.config.lib.mts vite.config.mts vitest.config.mts eslint.config.mjs --check",
+    "lint:eslint": "eslint",
+    "lint:eslint:fix": "eslint --fix",
+    "lint:oxlint": "oxlint .",
+    "lint:oxlint:fix": "oxlint . --fix",
+    "lint:stylelint": "stylelint src/**/*.{scss,vue}",
+    "lint:stylelint:fix": "stylelint src/**/*.{scss,vue} --fix",
+    "format": "oxfmt src/",
+    "format:check": "oxfmt src/ --check",
     "test": "vitest",
-    "test:browser": "vitest run",
-    "test:eslint": "vitest run --config vitest.config.node.mts",
-    "test:run": "run-s --print-name test:browser test:eslint",
+    "test:run": "vitest run",
     "test:ui": "vitest --ui"
   },
   "devDependencies": {
@@ -66,75 +60,75 @@
     "@intlify/unplugin-vue-i18n": "^11.2.5",
     "@kyvg/vue3-notification": "^3.4.2",
     "@mdi/font": "^7.4.47",
-    "@microsoft/api-extractor": "^7.59.2",
-    "@sentry/vue": "^10.75.2",
+    "@microsoft/api-extractor": "^7.59.0",
+    "@sentry/vue": "^10.73.0",
     "@shikijs/vitepress-twoslash": "^4.4.3",
+    "@stylistic/eslint-plugin": "^5.10.0",
     "@tsconfig/node22": "^22.0.6",
-    "@types/node": "^24.13.6",
+    "@types/node": "^24.13.3",
     "@types/rusha": "^0.8.3",
     "@types/sortablejs": "^1.15.9",
     "@types/webfontloader": "^1.6.38",
-    "@vitejs/plugin-vue": "^6.0.9",
-    "@vitest/browser": "^5.0.1",
-    "@vitest/browser-playwright": "^5.0.1",
-    "@vitest/ui": "^5.0.1",
-    "@vue/compiler-dom": "3.5.43",
+    "@vitejs/plugin-vue": "^6.0.8",
+    "@vitest/browser": "^4.1.11",
+    "@vitest/browser-playwright": "^4.1.11",
+    "@vitest/ui": "^4.1.11",
+    "@vue/compiler-dom": "3.5.42",
     "@vue/devtools-api": "^8.2.1",
     "@vue/eslint-config-typescript": "^14.9.0",
     "@vue/language-server": "3.3.11",
-    "@vue/test-utils": "^2.5.1",
+    "@vue/test-utils": "^2.5.0",
     "@vue/tsconfig": "0.9.1",
     "@vuelidate/core": "^2.0.3",
     "@vuelidate/validators": "^2.0.4",
-    "@vueuse/core": "15.0.0",
-    "@vueuse/integrations": "15.0.0",
+    "@vueuse/core": "14.4.0",
+    "@vueuse/integrations": "14.4.0",
     "axios": "1.20.0",
     "cropperjs": "^1.6.3",
-    "cropperjs2": "npm:cropperjs@2.2.0",
     "dayjs": "1.11.23",
-    "eslint": "^10.11.0",
-    "eslint-plugin-oxlint": "1.85.0",
-    "eslint-plugin-vue": "10.11.0",
+    "eslint": "^10.9.1",
+    "eslint-plugin-oxlint": "1.80.0",
+    "eslint-plugin-vue": "10.10.0",
     "eslint-plugin-vuetify": "^2.7.2",
     "npm-run-all2": "^9.0.3",
-    "oxfmt": "^0.70.0",
-    "oxlint": "1.85.0",
+    "oxfmt": "^0.65.0",
+    "oxlint": "1.80.0",
     "pinia": "4.0.3",
-    "playwright": "^1.63.0",
-    "postcss": "^8.5.28",
+    "playwright": "^1.62.1",
+    "postcss": "^8.5.26",
     "postcss-html": "^2.0.0",
-    "postcss-prefix-selector": "^2.2.1",
+    "postcss-prefix-selector": "^2.2.0",
     "rusha": "^0.8.14",
-    "sass": "1.105.0",
+    "sass": "1.103.1",
     "socket.io-client": "4.8.3",
     "sortablejs": "^1.15.7",
-    "stylelint": "17.15.0",
+    "stylelint": "17.14.1",
     "stylelint-config-html": "^2.0.0",
     "stylelint-config-recommended": "^18.0.0",
     "stylelint-config-recommended-vue": "^2.0.0",
     "stylelint-config-standard-scss": "^17.0.0",
-    "typescript": "~6.0.3",
-    "unplugin": "3.4.0",
-    "unplugin-dts": "^1.1.1",
+    "typescript": "5.9.3",
+    "unplugin": "3.3.0",
+    "unplugin-dts": "^1.1.0",
     "uuid": "^14.0.2",
-    "vite": "8.3.0",
+    "vite": "7.3.6",
     "vite-plugin-vuetify": "^2.1.3",
     "vitepress": "1.6.4",
-    "vitest": "^5.0.1",
-    "vue": "3.5.43",
+    "vitest": "^4.1.11",
+    "vue": "3.5.42",
     "vue-eslint-parser": "^10.4.1",
-    "vue-i18n": "11.4.12",
-    "vue-router": "5.3.1",
+    "vue-i18n": "11.4.10",
+    "vue-router": "5.3.0",
     "vue-tsc": "3.3.11",
-    "vuetify": "4.2.1",
+    "vuetify": "4.1.12",
     "webfontloader": "^1.6.28"
   },
   "peerDependencies": {
     "@sentry/vue": "^10.0.0",
     "@vuelidate/core": "^2.0.3",
     "@vuelidate/validators": "^2.0.4",
-    "@vueuse/core": "^15.0.0",
-    "@vueuse/integrations": "^15.0.0",
+    "@vueuse/core": "^14.1.0",
+    "@vueuse/integrations": "^14.1.0",
     "axios": "^1.13.4",
     "dayjs": "^1.11.19",
     "pinia": "^4.0.2",

package/src/eslint/plugin.mjs

@@ -21,13 +21,6 @@ const DEFAULT_DEPRECATED_IMPORTS = [
   'Pagination',
   'makeFilterHelper',
   'apiFetchList',
-  'apiGenerateListQuery',
-  'apiFetchListBatch',
-  'usePaginationAutoHide',
-  'fetchDamUserList',
-  'fetchDamExtSystemList',
-  'fetchDamAssetLicenceList',
-  'fetchDamAssetLicenceGroupList',
   'FilterBag',
   'Filter',
   'apiFetchByIds',
@@ -41,39 +34,15 @@ const DEFAULT_DEPRECATED_IMPORTS = [
 const DEFAULT_INTERNAL_DEPRECATED_IMPORTS = [
   {
     path: '@/services/api/apiFetchList',
-    imports: ['apiFetchList', 'apiGenerateListQuery'],
+    imports: ['apiFetchList'],
   },
   {
     path: '@/services/api/apiFetchListBatch',
     imports: ['apiFetchListBatch'],
   },
-  {
-    path: '@/services/api/apiFetchOne',
-    imports: ['apiFetchOne'],
-  },
-  {
-    path: '@/services/api/apiCreateOne',
-    imports: ['apiCreateOne'],
-  },
-  {
-    path: '@/services/api/apiUpdateOne',
-    imports: ['apiUpdateOne'],
-  },
-  {
-    path: '@/services/api/apiDeleteOne',
-    imports: ['apiDeleteOne'],
-  },
-  {
-    path: '@/services/api/apiFetchByIds',
-    imports: ['apiFetchByIds'],
-  },
-  {
-    path: '@/services/api/apiAnyRequest',
-    imports: ['apiAnyRequest'],
-  },
   {
     path: '@/composables/system/pagination',
-    imports: ['usePagination', 'Pagination', 'usePaginationAutoHide'],
+    imports: ['usePagination', 'Pagination'],
   },
   {
     path: '@/composables/filter/filterHelpers',
@@ -153,39 +122,6 @@ const DEFAULT_INTERNAL_DEPRECATED_IMPORTS = [
   },
 ]
 
-// Shared by `prefer-api-command` and `prefer-api-fetch-items`: both ask the same question -- is this
-// call the `useApiRequest` helper, and what did it say it answers with.
-const isHelperSource = (source) =>
-  typeof source === 'string' && (source.includes('labs/api/useApiRequest') || source.endsWith('common-admin/labs'))
-
-// Resolved through the scope rather than matched by name. Matching the name fires on anyone
-// else's function called `useApiRequest` and on a parameter that shadows the import inside
-// one function, and stays silent when the real helper is imported under another name.
-const isHelperBinding = (context, node) => {
-  let scope = context.sourceCode.getScope(node)
-  while (scope !== null) {
-    const variable = scope.variables.find((candidate) => candidate.name === node.name)
-    if (variable) {
-      return variable.defs.some(
-        (def) =>
-          def.type === 'ImportBinding' &&
-          isHelperSource(def.parent?.source?.value) &&
-          // The imported name, not the local one: an alias may call it anything, and a
-          // different export from the same module is a different function.
-          def.node?.imported?.name === 'useApiRequest'
-      )
-    }
-    scope = scope.upper
-  }
-
-  return false
-}
-
-const firstTypeArgument = (node) => {
-  const args = node.typeArguments ?? node.typeParameters
-  return args?.params?.[0]
-}
-
 const anzuPlugin = {
   rules: {
     'no-ts-extension': {
@@ -267,52 +203,41 @@ const anzuPlugin = {
             return rule.path
           })
 
-        // Index rules by the module specifier they match, so one import is
-        // resolved by a single Map lookup instead of scanning every rule.
-        const rulesBySource = new Map()
-        for (const rule of deprecationRules) {
-          const matchPath = rule.path || rule.module
-          if (!matchPath) continue
-          const entry = { matchPath, imports: new Set(rule.imports) }
-          const existing = rulesBySource.get(matchPath)
-          if (existing) existing.push(entry)
-          else rulesBySource.set(matchPath, [entry])
-        }
-
-        // Skipping depends only on the file name, so resolve it once per file
-        // instead of once per import declaration.
-        let fileSkipped = null
-        const isFileSkipped = () => {
-          if (fileSkipped !== null) return fileSkipped
-          const normalizedFilename = context.filename.replace(/\\/g, '/')
-          fileSkipped =
-            skipFiles.some((skip) => normalizedFilename.endsWith(skip)) ||
-            ruleFilePaths.some(
-              (rulePath) =>
-                normalizedFilename.endsWith(rulePath + '.ts') ||
-                normalizedFilename.endsWith(rulePath + '.js') ||
-                normalizedFilename.endsWith(rulePath + '.vue') ||
-                normalizedFilename.endsWith(rulePath)
-            )
-          return fileSkipped
-        }
-
         return {
           ImportDeclaration(node) {
+            const filename = context.filename
+            const normalizedFilename = filename.replace(/\\/g, '/')
+
+            // Check manual skip list
+            if (skipFiles.some((skip) => normalizedFilename.endsWith(skip))) return
+
+            // Auto-skip source files of path-based rules
+            if (
+              ruleFilePaths.some(
+                (rulePath) =>
+                  normalizedFilename.endsWith(rulePath + '.ts') ||
+                  normalizedFilename.endsWith(rulePath + '.js') ||
+                  normalizedFilename.endsWith(rulePath + '.vue') ||
+                  normalizedFilename.endsWith(rulePath),
+              )
+            )
+              return
+
             const source = node.source.value
             if (typeof source !== 'string') return
 
-            const entries = rulesBySource.get(source)
-            if (!entries) return
-            if (isFileSkipped()) return
+            for (const rule of deprecationRules) {
+              const matchPath = rule.path || rule.module
+              if (!matchPath || source !== matchPath) continue
 
-            for (const { matchPath, imports } of entries) {
-              for (const spec of node.specifiers) {
-                if (spec.type !== 'ImportSpecifier') continue
-                if (!imports.has(spec.imported.name)) continue
+              const deprecatedImports = node.specifiers
+                .filter((spec) => spec.type === 'ImportSpecifier')
+                .filter((spec) => rule.imports.includes(spec.imported.name))
+
+              for (const importSpec of deprecatedImports) {
                 context.report({
-                  node: spec,
-                  message: `'${spec.imported.name}' from '${matchPath}' is deprecated`,
+                  node: importSpec,
+                  message: `'${importSpec.imported.name}' from '${matchPath}' is deprecated`,
                 })
               }
             }
@@ -321,170 +246,26 @@ const anzuPlugin = {
       },
     },
 
-    'prefer-api-command': {
-      meta: {
-        type: 'problem',
-        docs: {
-          description:
-            'A request that reads nothing back is `useApiCommand`. `useApiRequest` is for an endpoint ' +
-            'that always answers with a body, and refuses a response without one.',
-        },
-        schema: [],
-      },
-      create(context) {
-        const reportedMethods = new Set(['DELETE', 'delete'])
-
-        // `any` and `never` satisfy the response constraint without meaning anything, and a call
-        // with no type argument at all infers it -- so the type cannot speak for these three.
-        const unhelpfulTypeArgument = (node) => {
-          const first = firstTypeArgument(node)
-          if (!first) return true
-
-          return first.type === 'TSAnyKeyword' || first.type === 'TSNeverKeyword'
-        }
-
-        const methodLiteral = (node) => {
-          const arg = node.arguments[0]
-          if (!arg || arg.type !== 'ObjectExpression') return null
-          const property = arg.properties.find(
-            (candidate) =>
-              candidate.type === 'Property' &&
-              !candidate.computed &&
-              (candidate.key.name ?? candidate.key.value) === 'method'
-          )
-          if (!property || property.value.type !== 'Literal') return null
-
-          return property.value.value
-        }
-
-        return {
-          CallExpression(node) {
-            if (node.callee.type !== 'Identifier' || !isHelperBinding(context, node.callee)) return
-
-            const method = methodLiteral(node)
-            const isDelete = typeof method === 'string' && reportedMethods.has(method)
-            if (!isDelete && !unhelpfulTypeArgument(node)) return
-
-            context.report({
-              node,
-              message: isDelete
-                ? 'A delete usually reads nothing back: use `useApiCommand`. If this endpoint really answers ' +
-                  'with the deleted entity, disable this line and say so.'
-                : 'State what this answers with. `useApiRequest<Entity>` for a body, `useApiCommand` for none, ' +
-                  '`optionalBody: true` for either.',
-            })
-          },
-        }
-      },
-    },
-
-    'prefer-api-fetch-items': {
-      meta: {
-        type: 'problem',
-        docs: {
-          description:
-            'A request whose answer is a list -- a bare array or a list envelope -- is ' +
-            '`useApiFetchItems`; `useApiRequest` checks nothing about the body it hands back.',
-        },
-        schema: [],
-      },
-      create(context) {
-        const isArrayType = (node) => {
-          if (!node) return false
-          if (node.type === 'TSArrayType') return true
-
-          return (
-            node.type === 'TSTypeReference' &&
-            node.typeName?.type === 'Identifier' &&
-            (node.typeName.name === 'Array' || node.typeName.name === 'ReadonlyArray') &&
-            (node.typeArguments ?? node.typeParameters)?.params?.length === 1
-          )
-        }
-
-        const envelopeGeneric = (node) => {
-          if (node.type !== 'TSTypeReference' || node.typeName?.type !== 'Identifier') return false
-          if (node.typeName.name !== 'ApiResponseList' && node.typeName.name !== 'ApiInfiniteResponseList') {
-            return false
-          }
-          const params = (node.typeArguments ?? node.typeParameters)?.params
-
-          // `ApiResponseList<T>` is `{ totalCount; data: T }`, so only an array in that slot makes
-          // `data` a list. `ApiResponseList<Item>` is something this helper would refuse.
-          return params?.length === 1 && isArrayType(params[0])
-        }
-
-        const envelopeLiteral = (node) => {
-          if (node.type !== 'TSTypeLiteral') return false
-
-          // Exactly one member, on purpose: a literal that also spells out `hasNextPage` or
-          // `totalCount` is an author saying they read the metadata, which is a reason to stay on
-          // `useApiRequest`. The generic form carries no such signal, so it is reported and the one
-          // legitimate reader takes a disable with a reason.
-          if (node.members.length !== 1) return false
-          const member = node.members[0]
-
-          return (
-            member.type === 'TSPropertySignature' &&
-            member.computed === false &&
-            (member.key?.name ?? member.key?.value) === 'data' &&
-            isArrayType(member.typeAnnotation?.typeAnnotation)
-          )
-        }
-
-        return {
-          CallExpression(node) {
-            if (node.callee.type !== 'Identifier' || !isHelperBinding(context, node.callee)) return
-
-            // No type argument at all is `prefer-api-command`'s case, not this one.
-            const first = firstTypeArgument(node)
-            if (!first) return
-
-            if (isArrayType(first)) {
-              context.report({
-                node,
-                message:
-                  "This answers with a bare array: use `useApiFetchItems` with `shape: 'array'`. If the " +
-                  'endpoint really answers with something that only looks like a list, disable this line and say so.',
-              })
-
-              return
-            }
-
-            if (envelopeGeneric(first) || envelopeLiteral(first)) {
-              context.report({
-                node,
-                message:
-                  'This answers with a list envelope: use `useApiFetchItems`, or `useApiFetchList` when the ' +
-                  "user drives the paging. If this call reads the envelope's metadata, disable this line and say so.",
-              })
-            }
-          },
-        }
-      },
-    },
-
     'url-params-match-template': {
       meta: {
         type: 'problem',
         docs: {
           description:
             'Ensure urlParams keys match the :placeholders declared in urlTemplate ' +
-            'for useApiRequest / useApiCommand / useApiFetchList / useApiFetchByIds / ' +
-            'useApiFetchItems / useApiFetchListBatch calls.',
+            'for useApiRequest / useApiFetchList / useApiFetchByIds / useApiFetchListBatch calls.',
         },
         schema: [],
       },
       create(context) {
         const TARGET_CALLEES = new Set([
           'useApiRequest',
-          // The commands are where most of the fleet's `:id` templates are -- every delete is one.
-          'useApiCommand',
           'useApiFetchList',
           'useApiFetchByIds',
-          'useApiFetchItems',
           'useApiFetchListBatch',
         ])
 
+        const PLACEHOLDER_RE = /:([a-zA-Z_][\w]*)/g
+
         const getCalleeName = (callee) => {
           if (callee.type === 'Identifier') return callee.name
           if (callee.type === 'MemberExpression' && callee.property.type === 'Identifier') {
@@ -497,7 +278,8 @@ const anzuPlugin = {
           for (const prop of objectExpr.properties) {
             if (prop.type !== 'Property' || prop.computed) continue
             const key = prop.key
-            const keyName = key.type === 'Identifier' ? key.name : key.type === 'Literal' ? key.value : null
+            const keyName =
+              key.type === 'Identifier' ? key.name : key.type === 'Literal' ? key.value : null
             if (keyName === name) return prop
           }
           return null
@@ -579,27 +361,11 @@ const anzuPlugin = {
             const resolved = resolveToString(templateProp.value)
             if (resolved === null) return
 
-            // The path only, because that is all `stringUrlTemplateReplace` substitutes into: it
-            // splits the template on `?` and rejoins the query untouched. Scanning the query too
-            // would fail CI on a perfectly good `...?locale=:locale`, and adding `locale` to
-            // `urlParams` to silence it would change nothing at runtime.
-            const [path] = resolved.split('?')
-
-            // Exactly what `stringUrlTemplateReplace` does, and nothing else: split the path on `/`,
-            // skip any part that does not START with `:`, and take everything after that colon as the
-            // key. Both halves matter and both were got wrong in turn. Hunting for `:name` anywhere
-            // matches inside `prefix-:id`, which the runtime leaves alone, so the rule demanded a key
-            // that is never filled in. Requiring the key to look like an identifier misses `:asset-id`,
-            // which the runtime substitutes happily, so the rule reported the caller's correct
-            // `urlParams` as having no placeholder to match.
             const placeholders = new Set()
-            for (const segment of path.split('/')) {
-              if (!segment.startsWith(':')) continue
-              // No guard on the name, because the runtime has none: a bare `:` takes `''` as its key
-              // and fills the segment in from `params['']`. Nobody writes that on purpose, which is
-              // the point -- it is a typo, and the rule reporting a missing key is how it gets seen
-              // rather than shipped as a literal colon in the url.
-              placeholders.add(segment.slice(1))
+            let match
+            PLACEHOLDER_RE.lastIndex = 0
+            while ((match = PLACEHOLDER_RE.exec(resolved)) !== null) {
+              placeholders.add(match[1])
             }
 
             const paramKeys = collectStaticKeys(paramsProp.value)
@@ -611,7 +377,9 @@ const anzuPlugin = {
               if (!paramKeySet.has(placeholder)) {
                 context.report({
                   node: paramsProp,
-                  message: `urlParams is missing key '${placeholder}' required by urlTemplate ` + `'${resolved}'.`,
+                  message:
+                    `urlParams is missing key '${placeholder}' required by urlTemplate ` +
+                    `'${resolved}'.`,
                 })
               }
             }
@@ -621,7 +389,8 @@ const anzuPlugin = {
                 context.report({
                   node: paramsProp,
                   message:
-                    `urlParams key '${key}' has no matching ':${key}' placeholder in urlTemplate ` + `'${resolved}'.`,
+                    `urlParams key '${key}' has no matching ':${key}' placeholder in urlTemplate ` +
+                    `'${resolved}'.`,
                 })
               }
             }
@@ -655,21 +424,21 @@ const anzuPlugin = {
             parts.unshift(current)
 
             const hasFatalCheck = parts.some(
-              (part) => part.type === 'CallExpression' && part.callee.name === 'isAnzuFatalError'
+              (part) => part.type === 'CallExpression' && part.callee.name === 'isAnzuFatalError',
             )
             const hasInstanceofErrorCheck = parts.some(
               (part) =>
                 part.type === 'BinaryExpression' &&
                 part.operator === 'instanceof' &&
                 part.right.type === 'Identifier' &&
-                part.right.name === 'Error'
+                part.right.name === 'Error',
             )
             const hasAxiosCheck = parts.some(
               (part) =>
                 part.type === 'CallExpression' &&
                 part.callee.type === 'MemberExpression' &&
                 part.callee.object.name === 'axios' &&
-                part.callee.property.name === 'isAxiosError'
+                part.callee.property.name === 'isAxiosError',
             )
 
             if (hasAxiosCheck && (hasFatalCheck || hasInstanceofErrorCheck)) {
@@ -695,8 +464,6 @@ const anzuPlugin = {
  * @param {boolean|'error'|'warn'|'off'} [options.noTsExtension='error'] - Severity for no-ts-extension rule.
  * @param {boolean|'error'|'warn'|'off'} [options.noFatalErrorAxiosCheck='error']
  *   - Severity for no-fatal-error-axios-check rule.
- * @param {boolean|'error'|'warn'|'off'} [options.preferApiFetchItems='error']
- *   - Severity for prefer-api-fetch-items rule.
  * @param {boolean|'error'|'warn'|'off'|Object} [options.deprecatedImports='error'] - Severity or config object.
  * @param {string[]} [options.deprecatedImports.exclude] - Import names to remove from the default list.
  * @param {string[]} [options.deprecatedImports.include] - Additional import names to add to the default list.
@@ -712,8 +479,6 @@ export function recommended(options = {}) {
   const {
     noTsExtension = 'error',
     noFatalErrorAxiosCheck = 'error',
-    preferApiCommand = 'error',
-    preferApiFetchItems = 'error',
     urlParamsMatchTemplate = 'error',
     deprecatedImports = 'error',
   } = options
@@ -732,18 +497,6 @@ export function recommended(options = {}) {
     rules['anzu/no-fatal-error-axios-check'] = fatalSeverity
   }
 
-  // prefer-api-command
-  const preferApiCommandSeverity = normalizeSeverity(preferApiCommand)
-  if (preferApiCommandSeverity) {
-    rules['anzu/prefer-api-command'] = preferApiCommandSeverity
-  }
-
-  // prefer-api-fetch-items
-  const preferApiFetchItemsSeverity = normalizeSeverity(preferApiFetchItems)
-  if (preferApiFetchItemsSeverity) {
-    rules['anzu/prefer-api-fetch-items'] = preferApiFetchItemsSeverity
-  }
-
   // url-params-match-template
   const urlParamsSeverity = normalizeSeverity(urlParamsMatchTemplate)
   if (urlParamsSeverity) {