fix: types

Author: shortcuts

scripts/config.ts

@@ -8,6 +8,10 @@ export function getClientsConfigField(
   pathToField: string[] | string,
   required: boolean = true,
 ): any {
+  if (typeof clientsConfig[language] !== 'object') {
+    throw new Error(`${language} doesn't exist in clients.config.json`);
+  }
+
   const config: LanguageConfig = clientsConfig[language];
   const path = Array.isArray(pathToField) ? pathToField : [pathToField];
 
@@ -42,11 +46,11 @@ export function getTestOutputFolder(language: Language): string {
 }
 
 export function getDockerImage(language?: Language): string | undefined {
-  if (CI || !language || !('dockerImage' in clientsConfig[language])) {
+  if (CI || !language) {
     return undefined;
   }
 
-  return getClientsConfigField(language, 'dockerImage');
+  return getClientsConfigField(language, 'dockerImage', false);
 }
 
 /**
@@ -57,7 +61,7 @@ export function getPackageVersionDefault(language: Language): string {
 }
 
 export function getGitHubUrl(language: Language, options?: { token: string }): string {
-  const { gitRepoId } = clientsConfig[language];
+  const gitRepoId = getClientsConfigField(language, ['gitRepoId']);
 
   // GitHub Action provides a default token for authentication
   // https://docs.github.com/en/actions/security-guides/automatic-token-authentication

scripts/husky/pre-commit.mjs

@@ -18,11 +18,14 @@ async function run(command) {
 
 export function getPatterns() {
   const entries = patterns;
-  for (const [language, { tests }] of Object.entries(clientConfig)) {
-    entries.unshift(`tests/output/${language}/${tests.outputFolder}/client/**`);
-    entries.unshift(`tests/output/${language}/${tests.outputFolder}/requests/**`);
-    entries.unshift(`tests/output/${language}/${tests.outputFolder}/e2e/**`);
-    entries.unshift(`tests/output/${language}/${tests.outputFolder}/benchmark/**`);
+  for (const [language, opts] of Object.entries(clientConfig)) {
+    if (typeof opts !== 'object') {
+      continue;
+    }
+    entries.unshift(`tests/output/${language}/${opts.tests.outputFolder}/client/**`);
+    entries.unshift(`tests/output/${language}/${opts.tests.outputFolder}/requests/**`);
+    entries.unshift(`tests/output/${language}/${opts.tests.outputFolder}/e2e/**`);
+    entries.unshift(`tests/output/${language}/${opts.tests.outputFolder}/benchmark/**`);
   }
   return entries;
 }

scripts/release/updateAPIVersions.ts

@@ -13,7 +13,7 @@ import type { Changelog, Versions } from './types.ts';
 async function updateConfigFiles(versionsToRelease: Versions): Promise<void> {
   // update the other versions in clients.config.json
   for (const lang of Object.keys(versionsToRelease) as Language[]) {
-    if (versionsToRelease[lang]?.next) {
+    if (typeof clientsConfig[lang] == 'object' && versionsToRelease[lang]?.next) {
       clientsConfig[lang].packageVersion = versionsToRelease[lang].next;
     }
   }