Merge pull request #286 from objectstack-ai/copilot/fix-action-step-error

Author: hotlong

examples/crm/src/ui/dashboards.ts

@@ -198,7 +198,7 @@ export const ServiceDashboard: Dashboard = {
     // Row 2: Case Distribution
     {
       title: 'Cases by Status',
-      type: 'donut',
+      type: 'pie',
       object: 'case',
       filter: { is_closed: false },
       categoryField: 'status',

packages/spec/src/data/field.test.ts

@@ -23,7 +23,7 @@ describe('FieldType', () => {
       'lookup', 'master_detail',
       'image', 'file', 'avatar',
       'formula', 'summary', 'autonumber',
-      'location', 'geolocation', 'address', 'code', 'color', 'rating', 'slider', 'signature', 'qrcode'
+      'location', 'address', 'code', 'color', 'rating', 'slider', 'signature', 'qrcode'
     ];
 
     validTypes.forEach(type => {
@@ -495,10 +495,6 @@ describe('Field Factory Helpers', () => {
       expect(() => FieldType.parse('location')).not.toThrow();
     });
 
-    it('should accept geolocation field type', () => {
-      expect(() => FieldType.parse('geolocation')).not.toThrow();
-    });
-
     it('should accept address field type', () => {
       expect(() => FieldType.parse('address')).not.toThrow();
     });
@@ -647,19 +643,6 @@ describe('Field Factory Helpers', () => {
       expect(qrcodeField.displayValue).toBe(true);
       expect(qrcodeField.allowScanning).toBe(true);
     });
-
-    it('should create geolocation field', () => {
-      const geolocationField = Field.geolocation({
-        label: 'Current Location',
-        displayMap: true,
-        allowGeocoding: false,
-      });
-      
-      expect(geolocationField.type).toBe('geolocation');
-      expect(geolocationField.label).toBe('Current Location');
-      expect(geolocationField.displayMap).toBe(true);
-      expect(geolocationField.allowGeocoding).toBe(false);
-    });
   });
 
   describe('Currency Field Type with Multi-Currency Support', () => {

packages/spec/src/data/field.zod.ts

@@ -363,9 +363,4 @@ export const Field = {
     type: 'qrcode', 
     ...config 
   } as const),
-  
-  geolocation: (config: FieldInput = {}) => ({ 
-    type: 'geolocation', 
-    ...config 
-  } as const),
 };

packages/spec/src/ui/dashboard.test.ts

@@ -10,15 +10,15 @@ import {
 
 describe('ChartType', () => {
   it('should accept all chart types', () => {
-    const types = ['metric', 'bar', 'line', 'pie', 'donut', 'funnel', 'table', 'text'];
+    const types = ['metric', 'bar', 'line', 'pie', 'funnel', 'table', 'text'];
 
     types.forEach(type => {
       expect(() => ChartType.parse(type)).not.toThrow();
     });
   });
 
   it('should reject invalid chart types', () => {
-    expect(() => ChartType.parse('scatter')).toThrow();
+    expect(() => ChartType.parse('bubble')).toThrow();
     expect(() => ChartType.parse('invalid')).toThrow();
   });
 });
@@ -255,7 +255,7 @@ describe('DashboardSchema', () => {
           },
           {
             title: 'Opportunities by Type',
-            type: 'donut',
+            type: 'pie',
             object: 'opportunity',
             categoryField: 'type',
             aggregate: 'count',