finalize code

Author: MQ37

src/tools/apify-properties.ts

@@ -1,5 +1,9 @@
 import type { ISchemaProperties } from '../types.js';
 
+/**
+ * Adds resource picker schema structure to array properties with editor === 'resourcePicker'.
+ * The resource picker allows users to select resources from their Apify account.
+ */
 export function addResourcePickerProperties(property: ISchemaProperties): ISchemaProperties {
     return {
         ...property,
@@ -39,16 +43,12 @@ export function addKeyValueProperties(property: ISchemaProperties): ISchemaPrope
     };
 }
 
-/**
- * Adds globs schema structure to array properties with editor === 'globs'.
- */
-/**
- * Adds pseudoUrls schema structure to array properties with items.editor === 'pseudoUrls'.
- */
-
 const USER_DATA_DESCRIPTION = `User data object. A JSON object with custom user data that will be passed in the userData property of the Request object for each URL`;
 const HEADERS_DESCRIPTION = `Headers object. A JSON object whose properties and values contain HTTP headers that will sent with the request.`;
 
+/**
+ * Adds globs schema structure to array properties with editor === 'globs'.
+ */
 export function addGlobsProperties(property: ISchemaProperties): ISchemaProperties {
     return {
         ...property,
@@ -90,6 +90,9 @@ export function addGlobsProperties(property: ISchemaProperties): ISchemaProperti
     };
 }
 
+/**
+ * Adds pseudoUrls schema structure to array properties with items.editor === 'pseudoUrls'.
+ */
 export function addPseudoUrlsProperties(property: ISchemaProperties): ISchemaProperties {
     return {
         ...property,

src/tools/utils.ts

@@ -78,14 +78,6 @@ export function buildApifySpecificProperties(properties: Record<string, ISchemaP
     return clonedProperties;
 }
 
-/**
- * Filters schema properties to include only the necessary fields.
- *
- * This is done to reduce the size of the input schema and to make it more readable.
- *
- * @param properties
- */
-
 /**
  * Filters schema properties to include only the necessary fields.
  * This is done to reduce the size of the input schema and to make it more readable.

tests/unit/tools.utils.test.ts

@@ -374,13 +374,13 @@ describe('shortenProperties', () => {
             expect(result.prop1.enum.length).toBeLessThan(30);
             const totalEnumLen = result.prop1.enum.reduce((sum, v) => sum + v.length, 0);
             expect(totalEnumLen).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
+
+            // Calculate total character length of enum values
+            const totalLength = result.prop1.enum.reduce((sum, val) => sum + val.length, 0);
+            expect(totalLength).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
         } else {
             expect(result.prop1.enum).toBeUndefined();
         }
-
-        // Calculate total character length of enum values
-        const totalLength = result.prop1.enum!.reduce((sum, val) => sum + val.length, 0);
-        expect(totalLength).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
     });
 
     it('should shorten items.enum values if they exceed the limit', () => {
@@ -408,13 +408,12 @@ describe('shortenProperties', () => {
             expect(result.prop1.items.enum.length).toBeLessThan(enumValues.length);
             const totalLength = result.prop1.items.enum.reduce((sum, val) => sum + val.length, 0);
             expect(totalLength).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
+
+            // Calculate total character length of enum values
+            expect(totalLength).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
         } else {
             expect(result.prop1.items?.enum).toBeUndefined();
         }
-
-        // Calculate total character length of enum values
-        const totalLength = result.prop1.items!.enum!.reduce((sum, val) => sum + val.length, 0);
-        expect(totalLength).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
     });
 
     it('should handle properties without enum or items.enum', () => {