fix: dynamic linter ruler

Author: shortcuts

docker-compose.yml

@@ -13,9 +13,9 @@ services:
         - PHP_VERSION=${PHP_VERSION}
         - PYTHON_VERSION=${PYTHON_VERSION}
     command: tail -f /dev/null
-    volumes: [ ./:/app ]
-#    ports:
-#      - "5009:5009" # So we can debug the OpenAPI Generator process
+    volumes: [./:/app]
+  #    ports:
+  #      - "5009:5009" # So we can debug the OpenAPI Generator process
 
   ruby:
     container_name: apic_ruby
@@ -27,7 +27,7 @@ services:
         - NODE_VERSION=${NODE_VERSION}
         - RUBY_VERSION=${RUBY_VERSION}
     command: tail -f /dev/null
-    volumes: [ ./:/app ]
+    volumes: [./:/app]
 
   swift:
     container_name: apic_swift
@@ -39,4 +39,4 @@ services:
         - NODE_VERSION=${NODE_VERSION}
         - SWIFT_VERSION=${SWIFT_VERSION}
     command: tail -f /dev/null
-    volumes: [ ./:/app ]
+    volumes: [./:/app]

eslint/src/rules/refCommon.ts

@@ -3,18 +3,7 @@ import { createRule } from 'eslint-plugin-yml/lib/utils';
 
 import { isPairWithKey, isScalar } from '../utils.js';
 
-const allSpecs = [
-  'abtesting',
-  'analytics',
-  'crawler',
-  'ingestion',
-  'insights',
-  'monitoring',
-  'personalization',
-  'query-suggestions',
-  'recommend',
-  'search',
-];
+import clientsConfig from '../../../config/clients.config.json' with { type: 'json' };
 
 export const refCommon = createRule('refCommon', {
   meta: {
@@ -58,7 +47,13 @@ export const refCommon = createRule('refCommon', {
         while (ref.startsWith('../')) {
           ref = ref.slice(3);
         }
-        if (allSpecs.filter((s) => s !== spec).every((s) => !ref.startsWith(s))) {
+
+        if (clientsConfig.javascript.clients.filter((s) => s.name !== spec).every((s) => !ref.startsWith(s.name))) {
+          return;
+        }
+
+        // it's expected for composition to import the full version
+        if (spec === 'composition' && ref.includes('composition-full')) {
           return;
         }
 

scripts/common.ts

@@ -73,7 +73,7 @@ export const LANGUAGES = [...new Set(Object.values(GENERATORS).map((gen) => gen.
 
 // `crawler` is manually added so we can still bundled and validate the specs
 // the entry can be removed once at least one client is generated
-export const CLIENTS = [...new Set(Object.values(GENERATORS).map((gen) => gen.client)), 'crawler', 'abtesting-v3'];
+export const CLIENTS = [...new Set(Object.values(GENERATORS).map((gen) => gen.client)), 'crawler'];
 
 export async function run(command: string, { errorMessage, cwd, language }: RunOptions = {}): Promise<string> {
   const realCwd = path.resolve(ROOT_DIR, cwd ?? '.');