Phase 8: Remove deprecated fields (formula, encryption, geoSpatial, stateMachine)

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/spec/src/data/driver-sql.test.ts

@@ -186,7 +186,6 @@ describe('SQLDriverConfigSchema', () => {
         jsonFields: true,
         arrayFields: true,
         vectorSearch: true,
-        geoSpatial: false,
         schemaSync: true,
         migrations: true,
         indexes: true,
@@ -240,7 +239,6 @@ describe('SQLDriverConfigSchema', () => {
         jsonFields: true,
         arrayFields: false,
         vectorSearch: false,
-        geoSpatial: false,
         schemaSync: true,
         migrations: true,
         indexes: true,
@@ -330,7 +328,6 @@ describe('SQLDriverConfigSchema', () => {
         jsonFields: true,
         arrayFields: false,
         vectorSearch: false,
-        geoSpatial: false,
         schemaSync: true,
         migrations: false,
         indexes: true,

packages/spec/src/data/driver-sql.zod.ts

@@ -132,7 +132,6 @@ export type SSLConfig = z.infer<typeof SSLConfigSchema>;
  *     jsonFields: true,
  *     arrayFields: true,
  *     vectorSearch: true,
- *     geoSpatial: false,
  *     schemaSync: true,
  *     migrations: true,
  *     indexes: true,

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

@@ -205,12 +205,6 @@ export const DriverCapabilitiesSchema = z.object({
    */
   vectorSearch: z.boolean().default(false).describe('Supports vector embeddings and similarity search'),
 
-  /**
-   * Whether the driver supports geospatial queries.
-   * @deprecated Use geospatialQuery instead
-   */
-  geoSpatial: z.boolean().default(false).describe('Supports geospatial queries (deprecated: use geospatialQuery)'),
-
   // ============================================================================
   // Schema Management
   // ============================================================================
@@ -248,14 +242,6 @@ export const DriverCapabilitiesSchema = z.object({
    * Whether the driver supports query result caching.
    */
   queryCache: z.boolean().default(false).describe('Supports query result caching'),
-}).refine((data) => {
-  // Ensure deprecated geoSpatial and new geospatialQuery are consistent if both are provided
-  if (data.geoSpatial !== undefined && data.geospatialQuery !== undefined && data.geoSpatial !== data.geospatialQuery) {
-    return false;
-  }
-  return true;
-}, {
-  message: 'Deprecated geoSpatial and geospatialQuery must have the same value if both are provided',
 });
 
 /**

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

@@ -208,7 +208,6 @@ describe('FieldSchema', () => {
       expect(result.unique).toBe(false);
       expect(result.hidden).toBe(false);
       expect(result.readonly).toBe(false);
-      expect(result.encryption).toBe(false);
       expect(result.index).toBe(false);
       expect(result.externalId).toBe(false);
     });
@@ -355,7 +354,6 @@ describe('FieldSchema', () => {
         type: 'password',
         hidden: true,
         readonly: true,
-        encryption: true,
       };
 
       expect(() => FieldSchema.parse(secureField)).not.toThrow();

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

@@ -377,9 +377,6 @@ export const FieldSchema = z.object({
 
   /** Calculation */
   expression: z.string().optional().describe('Formula expression'),
-  /** @deprecated Use `expression` instead. Will be removed in v2.0.0 */
-  formula: z.string().optional()
-    .describe('DEPRECATED: Use `expression` field instead. Scheduled for removal in v2.0.0'),
   summaryOperations: z.object({
     object: z.string(),
     field: z.string(),
@@ -460,9 +457,6 @@ export const FieldSchema = z.object({
   trackFeedHistory: z.boolean().optional().describe('Track field changes in Chatter/activity feed (Salesforce pattern)'),
   caseSensitive: z.boolean().optional().describe('Whether text comparisons are case-sensitive'),
   autonumberFormat: z.string().optional().describe('Auto-number display format pattern (e.g., "CASE-{0000}")'),
-  /** @deprecated Use `encryptionConfig` instead. Will be removed in v2.0.0 */
-  encryption: z.boolean().default(false).describe('DEPRECATED: Use `encryptionConfig` for enhanced encryption features. Scheduled for removal in v2.0.0'),
-  
   /** Indexing */
   index: z.boolean().default(false).describe('Create standard database index'),
   externalId: z.boolean().default(false).describe('Is external ID for upsert operations'),

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

@@ -493,20 +493,22 @@ describe('ObjectSchema', () => {
         fields: {
           status: { type: 'text' }
         },
-        stateMachine: {
-          id: 'leave_flow',
-          initial: 'draft',
-          states: {
-            draft: { on: { SUBMIT: 'pending' } },
-            pending: { on: { APPROVE: 'approved' } },
-            approved: { type: 'final' }
+        stateMachines: {
+          leave_flow: {
+            id: 'leave_flow',
+            initial: 'draft',
+            states: {
+              draft: { on: { SUBMIT: 'pending' } },
+              pending: { on: { APPROVE: 'approved' } },
+              approved: { type: 'final' }
+            }
           }
         }
       };
 
       const result = ObjectSchema.parse(objectWithState);
-      expect(result.stateMachine).toBeDefined();
-      expect(result.stateMachine?.initial).toBe('draft');
+      expect(result.stateMachines).toBeDefined();
+      expect(result.stateMachines!.leave_flow.initial).toBe('draft');
     });
 
     it('should support multiple parallel state machines via stateMachines', () => {
@@ -559,21 +561,19 @@ describe('ObjectSchema', () => {
       expect(result.stateMachines!.approval.initial).toBe('pending');
     });
 
-    it('should allow both stateMachine and stateMachines to coexist', () => {
+    it('should allow multiple stateMachines', () => {
       const obj = {
         name: 'contract',
         fields: { status: { type: 'text' } },
-        // Legacy single shorthand
-        stateMachine: {
-          id: 'contract_lifecycle',
-          initial: 'draft',
-          states: {
-            draft: { on: { ACTIVATE: 'active' } },
-            active: { type: 'final' },
-          }
-        },
-        // Additional parallel machines
         stateMachines: {
+          contract_lifecycle: {
+            id: 'contract_lifecycle',
+            initial: 'draft',
+            states: {
+              draft: { on: { ACTIVATE: 'active' } },
+              active: { type: 'final' },
+            }
+          },
           billing: {
             id: 'contract_billing',
             initial: 'unbilled',
@@ -587,7 +587,7 @@ describe('ObjectSchema', () => {
       };
 
       const result = ObjectSchema.parse(obj);
-      expect(result.stateMachine?.initial).toBe('draft');
+      expect(result.stateMachines?.contract_lifecycle.initial).toBe('draft');
       expect(result.stateMachines?.billing.initial).toBe('unbilled');
     });
   });

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

@@ -292,14 +292,11 @@ const ObjectSchemaBase = z.object({
 
   /**
    * State Machine(s)
-   * Supports a single machine (legacy) or a named record of machines.
+   * Named record of state machines, where each key is a unique machine identifier.
    * Multiple machines allow parallel lifecycles (e.g., status + payment_status + approval_status).
    * 
-   * @example Single: stateMachine: { id: 'lifecycle', initial: 'draft', states: {...} }
-   * @example Multiple: stateMachines: { lifecycle: {...}, payment: {...}, approval: {...} }
+   * @example stateMachines: { lifecycle: {...}, payment: {...}, approval: {...} }
    */
-  /** @deprecated Use `stateMachines` (plural) instead. Will be removed in v2.0.0 */
-  stateMachine: StateMachineSchema.optional().describe('DEPRECATED: Use stateMachines (plural). Single state machine shorthand.'),
   stateMachines: z.record(z.string(), StateMachineSchema).optional().describe('Named state machines for parallel lifecycles (e.g., status, payment, approval)'),
 
   /**