fix: Increase ACTOR_ENUM_MAX_LENGTH from 200 to 2000 to ensure that geocode are not removed (#292)

* fix: Increase ACTOR_ENUM_MAX_LENGTH from 200 to 2000 to ensure that geocode are not removed

Author: jirispilka

src/const.ts

@@ -1,10 +1,9 @@
 // Actor input const
 export const ACTOR_README_MAX_LENGTH = 5_000;
-export const ACTOR_ENUM_MAX_LENGTH = 200;
+// Actor enum property max length, we need to make sure that most of the enum values fit into the input (such as geocodes)
+export const ACTOR_ENUM_MAX_LENGTH = 2000;
 export const ACTOR_MAX_DESCRIPTION_LENGTH = 500;
 
-export const ACTOR_RUN_DATASET_OUTPUT_MAX_ITEMS = 5;
-
 // Actor run const
 export const ACTOR_MAX_MEMORY_MBYTES = 4_096; // If the Actor requires 8GB of memory, free users can't run actors-mcp-server and requested Actor
 

src/tools/utils.ts

@@ -278,6 +278,8 @@ export function shortenEnum(enumList: string[]): string[] | undefined {
 
 /**
  * Shortens the description, enum, and items.enum properties of the schema properties.
+ * This is mostly problem with compass/crawler-google-places, which has large number of categories
+ * such as ( 'abbey', 'accountant', 'accounting',  'acupuncturist', .... )
  * @param properties
  */
 export function shortenProperties(properties: { [key: string]: ISchemaProperties }): {

tests/unit/tools.utils.test.ts

@@ -356,7 +356,8 @@ describe('shortenProperties', () => {
 
     it('should shorten enum values if they exceed the limit', () => {
         // Create an enum with many values to exceed the character limit
-        const enumValues = Array.from({ length: 50 }, (_, i) => `enum-value-${i}`);
+        const value = 'enum-value-';
+        const enumValues = Array.from({ length: Math.ceil(ACTOR_ENUM_MAX_LENGTH / value.length) + 1 }, (_, i) => `${value}${i}`);
         const properties: Record<string, ISchemaProperties> = {
             prop1: {
                 type: 'string',
@@ -371,7 +372,7 @@ describe('shortenProperties', () => {
         // Check that enum was shortened
         expect(result.prop1.enum).toBeDefined();
         if (result.prop1.enum) {
-            expect(result.prop1.enum.length).toBeLessThan(30);
+            expect(result.prop1.enum.length).toBeLessThan(enumValues.length);
             const totalEnumLen = result.prop1.enum.reduce((sum, v) => sum + v.length, 0);
             expect(totalEnumLen).toBeLessThanOrEqual(ACTOR_ENUM_MAX_LENGTH);
 
@@ -385,7 +386,8 @@ describe('shortenProperties', () => {
 
     it('should shorten items.enum values if they exceed the limit', () => {
         // Create an enum with many values to exceed the character limit
-        const enumValues = Array.from({ length: 50 }, (_, i) => `enum-value-${i}`);
+        const value = 'enum-value-';
+        const enumValues = Array.from({ length: Math.ceil(ACTOR_ENUM_MAX_LENGTH / value.length) + 1 }, (_, i) => `${value}${i}`);
         const properties: Record<string, ISchemaProperties> = {
             prop1: {
                 type: 'array',
@@ -889,8 +891,8 @@ describe('inferArrayItemsTypeIfMissing', () => {
 describe('shortenEnum', () => {
     it('shorten enum list', () => {
         const enumList: string[] = [];
-        const wordLength = 10;
-        const wordCount = 30;
+        const wordLength = 100;
+        const wordCount = ACTOR_ENUM_MAX_LENGTH / wordLength + 1; // exceed the limit
 
         for (let i = 0; i < wordCount; i++) {
             enumList.push('a'.repeat(wordLength));