Policy Field: Add MaxLen, Required Priority and Fix Decimal issues

Author: asmasarw

workspaces/dcm/plugins/dcm/src/pages/policies/components/PolicyFormFields.tsx

@@ -71,11 +71,15 @@ export function PolicyFormFields({
 
       <TextField
         label="Description"
-        helperText="Optional — describe the purpose of this policy"
+        helperText={
+          err('description') ?? 'Optional — describe the purpose of this policy'
+        }
+        error={Boolean(err('description'))}
         value={form.description}
         onChange={e =>
           setForm(prev => ({ ...prev, description: e.target.value }))
         }
+        onBlur={() => touch('description')}
         fullWidth
         variant="outlined"
         size="small"
@@ -111,19 +115,28 @@ export function PolicyFormFields({
       </FormControl>
 
       <TextField
-        label="Priority"
+        label="Priority *"
         helperText={
           err('priority') ?? '1 (highest) – 1000 (lowest), default 500'
         }
         error={Boolean(err('priority'))}
         value={form.priority}
-        onChange={e => setForm(prev => ({ ...prev, priority: e.target.value }))}
+        onChange={e => {
+          if (/^\d*$/.test(e.target.value)) {
+            setForm(prev => ({ ...prev, priority: e.target.value }));
+          }
+        }}
+        onKeyDown={e => {
+          if (e.key === '.' || e.key === 'e' || e.key === 'E') {
+            e.preventDefault();
+          }
+        }}
         onBlur={() => touch('priority')}
         fullWidth
         variant="outlined"
         size="small"
         type="number"
-        inputProps={{ min: 1, max: 1000 }}
+        inputProps={{ min: 1, max: 1000, step: 1 }}
       />
 
       <TextField

workspaces/dcm/plugins/dcm/src/pages/policies/policyFormTypes.test.ts

@@ -52,6 +52,23 @@ describe('validatePolicyForm', () => {
 
     const ok = validatePolicyForm({ ...valid(), priority: '500' });
     expect(ok.priority).toBeUndefined();
+
+    const decimal = validatePolicyForm({ ...valid(), priority: '500.5' });
+    expect(decimal.priority).toBeDefined();
+
+    const empty = validatePolicyForm({ ...valid(), priority: '' });
+    expect(empty.priority).toBeDefined();
+  });
+
+  it('rejects description longer than 255 characters', () => {
+    const errors = validatePolicyForm({
+      ...valid(),
+      description: 'a'.repeat(256),
+    });
+    expect(errors.description).toBeDefined();
+
+    const ok = validatePolicyForm({ ...valid(), description: 'a'.repeat(255) });
+    expect(ok.description).toBeUndefined();
   });
 
   it('accepts only GLOBAL or USER as policy_type', () => {

workspaces/dcm/plugins/dcm/src/pages/policies/policyFormTypes.ts

@@ -37,13 +37,18 @@ const policySchema = yup.object({
     .required('Display name is required')
     .min(1, 'Display name cannot be empty')
     .max(255, 'Display name must be at most 255 characters'),
+  description: yup
+    .string()
+    .max(255, 'Description must be at most 255 characters'),
   policy_type: yup
     .string()
     .required('Policy type is required')
     .oneOf(['GLOBAL', 'USER'], 'Must be GLOBAL or USER'),
   priority: yup
     .number()
     .typeError('Priority must be a number')
+    .required('Priority is required')
+    .integer('Priority must be a whole number')
     .min(1, 'Priority must be at least 1')
     .max(1000, 'Priority must be at most 1000'),
   rego_code: yup