feat: made Control.value mandatory in all controls

src/commonControls/DateControl.ts

@@ -293,7 +293,7 @@ export class DateControlState implements ControlState {
     /**
      * The value, an ISO date string.
      */
-    value?: string;
+    value: string;
 
     /**
      * Tracks if the control is actively asking the user to set or change the value.

src/commonControls/ValueControl.ts

@@ -240,7 +240,7 @@ export class ValueControlState implements ControlState {
      * If `erMatch = true` the value is a slot value ID for the slot type `this.slotType`.
      * If `erMatch = false` the value may be an arbitrary string.
      */
-    value?: string;
+    value: string;
 
     /**
      * Tracks whether the value is an Entity Resolution match.

src/commonControls/dateRangeControl/DateRangeControl.ts

@@ -794,23 +794,23 @@ export class DateRangeControl extends ContainerControl implements InteractionMod
         }
     }
 
-    private getStartDateFromChild(): string | undefined {
+    private getStartDateFromChild(): string {
         return this.startDateControl.state.value;
     }
 
-    private getEndDateFromChild(): string | undefined {
+    private getEndDateFromChild(): string {
         return this.endDateControl.state.value;
     }
 
-    private setStartDate(date: string | undefined): void {
+    private setStartDate(date: string): void {
         this.startDateControl.state.value = date;
         this.state.startDate = date;
 
         // Clean open question once a value set
         this.state.isChangingRange = false;
     }
 
-    private setEndDate(date: string | undefined): void {
+    private setEndDate(date: string): void {
         this.endDateControl.state.value = date;
         this.state.endDate = date;
 

src/commonControls/listControl/ListControl.ts

@@ -348,7 +348,7 @@ export class ListControlState implements ControlState {
      * If `erMatch = true` the value is a slot value ID for the slot type `this.slotType`.
      * If `erMatch = false` the value may be an arbitrary string.
      */
-    value?: string;
+    value: string;
 
     /**
      * Tracks whether the value is an Entity Resolution match.

src/commonControls/multiValueListControl/MultiValueListControl.ts

@@ -412,7 +412,7 @@ export class MultiValueListControlState implements ControlState {
     /**
      * The list of values as [ (id1, erMatch), (id2, erMatch) ]
      */
-    value?: MultiValueListStateValue[];
+    value: MultiValueListStateValue[];
 
     /**
      * Tracks the most recent elicitation action.
@@ -1213,7 +1213,7 @@ export class MultiValueListControl extends Control implements InteractionModelCo
 
     // tsDoc - see ControlStateDiagramming
     stringifyStateForDiagram(): string {
-        let text = this.state.value ? this.state.value.join(', ') : '<none>';
+        let text = this.state.value.length > 0 ? this.state.value.join(', ') : '<none>';
         if (this.state.elicitationAction !== undefined) {
             text += `[eliciting, ${this.state.elicitationAction}]`;
         }

src/commonControls/numberControl/NumberControl.ts

@@ -301,7 +301,7 @@ export class NumberControlState implements ControlState {
     /**
      * The value, an integer.
      */
-    value?: number;
+    value: number;
 
     /**
      * Tracks whether the value has been explicitly confirmed by the user.

src/controls/ContainerControl.ts

@@ -35,7 +35,7 @@ interface ChildActivityRecord {
  * Container state for use in arbitration
  */
 export class ContainerControlState implements ControlState {
-    value?: any;
+    value: any;
     lastHandlingControl?: ChildActivityRecord; // TODO: naming: change to lastHandlingControlInfo | lastHandlingControlRecord
     lastInitiativeChild?: ChildActivityRecord; // ditto.
 }

src/controls/Control.ts

@@ -44,7 +44,7 @@ export interface ControlState {
      *   controls and other consumers should `control.isReady()` to determine if the value
      *   is ready for use.
      */
-    value?: any; //TODO: make mandatory to tighten the convention.
+    value: any;
 }
 
 export interface ControlInputHandler {