jumbo-cli 2.14.0 → 2.15.0

package/dist/domain/goals/Goal.js

@@ -1,24 +1,21 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.Goal = void 0;
-const BaseAggregate_js_1 = require("../BaseAggregate.js");
-const ValidationRule_js_1 = require("../validation/ValidationRule.js");
-const Constants_js_1 = require("./Constants.js");
-const ObjectiveRules_js_1 = require("./rules/ObjectiveRules.js");
-const TitleRules_js_1 = require("./rules/TitleRules.js");
-const SuccessCriteriaRules_js_1 = require("./rules/SuccessCriteriaRules.js");
-const ScopeRules_js_1 = require("./rules/ScopeRules.js");
-const UpdateRules_js_1 = require("./rules/UpdateRules.js");
-const NoteRules_js_1 = require("./rules/NoteRules.js");
-const StateTransitionRules_js_1 = require("./rules/StateTransitionRules.js");
-const CanSubmitForReviewRule_js_1 = require("./rules/CanSubmitForReviewRule.js");
-const CanQualifyRule_js_1 = require("./rules/CanQualifyRule.js");
-const CanCommitRule_js_1 = require("./rules/CanCommitRule.js");
-const CanRejectRule_js_1 = require("./rules/CanRejectRule.js");
-const CanSubmitRule_js_1 = require("./rules/CanSubmitRule.js");
-const CanCodifyRule_js_1 = require("./rules/CanCodifyRule.js");
-const CanCloseRule_js_1 = require("./rules/CanCloseRule.js");
-class Goal extends BaseAggregate_js_1.BaseAggregate {
+import { BaseAggregate } from "../BaseAggregate.js";
+import { ValidationRuleSet } from "../validation/ValidationRule.js";
+import { GoalEventType, GoalStatus, WAITING_STATES, DETERMINISTIC_RESET_TARGETS } from "./Constants.js";
+import { OBJECTIVE_RULES } from "./rules/ObjectiveRules.js";
+import { TITLE_RULES } from "./rules/TitleRules.js";
+import { SUCCESS_CRITERIA_RULES } from "./rules/SuccessCriteriaRules.js";
+import { SCOPE_RULES } from "./rules/ScopeRules.js";
+import { UPDATE_RULES } from "./rules/UpdateRules.js";
+import { NOTE_RULES, OPTIONAL_NOTE_RULES } from "./rules/NoteRules.js";
+import { CanAddRule, CanRefineRule, CanStartRule, CanUpdateRule, CanCompleteRule, CanResetRule, CanBlockRule, CanUnblockRule, CanPauseRule, CanResumeRule, } from "./rules/StateTransitionRules.js";
+import { CanSubmitForReviewRule } from "./rules/CanSubmitForReviewRule.js";
+import { CanQualifyRule } from "./rules/CanQualifyRule.js";
+import { CanCommitRule } from "./rules/CanCommitRule.js";
+import { CanRejectRule } from "./rules/CanRejectRule.js";
+import { CanSubmitRule } from "./rules/CanSubmitRule.js";
+import { CanCodifyRule } from "./rules/CanCodifyRule.js";
+import { CanCloseRule } from "./rules/CanCloseRule.js";
+export class Goal extends BaseAggregate {
     constructor(state) {
         super(state);
     }
@@ -28,7 +25,7 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     static apply(state, event) {
         switch (event.type) {
-            case Constants_js_1.GoalEventType.ADDED: {
+            case GoalEventType.ADDED: {
                 const e = event;
                 state.title = e.payload.title;
                 state.objective = e.payload.objective;
@@ -51,37 +48,37 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.REFINED: {
+            case GoalEventType.REFINED: {
                 const e = event;
                 state.status = e.payload.status; // "refined"
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.REFINEMENT_STARTED: {
+            case GoalEventType.REFINEMENT_STARTED: {
                 const e = event;
-                if (Constants_js_1.WAITING_STATES.has(state.status)) {
+                if (WAITING_STATES.has(state.status)) {
                     state.lastWaitingStatus = state.status;
                 }
                 state.status = e.payload.status; // "in-refinement"
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.COMMITTED: {
+            case GoalEventType.COMMITTED: {
                 const e = event;
                 state.status = e.payload.status; // "refined"
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.STARTED: {
+            case GoalEventType.STARTED: {
                 const e = event;
-                if (Constants_js_1.WAITING_STATES.has(state.status)) {
+                if (WAITING_STATES.has(state.status)) {
                     state.lastWaitingStatus = state.status;
                 }
                 state.status = e.payload.status; // "doing"
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.UPDATED: {
+            case GoalEventType.UPDATED: {
                 const e = event;
                 // Only update provided fields (partial update pattern)
                 if (e.payload.title !== undefined) {
@@ -114,41 +111,41 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.BLOCKED: {
+            case GoalEventType.BLOCKED: {
                 const e = event;
                 state.status = e.payload.status; // 'blocked'
                 state.note = e.payload.note; // Reason for blocking
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.UNBLOCKED: {
+            case GoalEventType.UNBLOCKED: {
                 const e = event;
                 state.status = e.payload.status; // 'unblocked'
                 state.note = e.payload.note; // Optional resolution note
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.PAUSED: {
+            case GoalEventType.PAUSED: {
                 const e = event;
                 state.status = e.payload.status; // 'paused'
                 state.note = e.payload.note; // Optional note about pausing
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.RESUMED: {
+            case GoalEventType.RESUMED: {
                 const e = event;
                 state.status = e.payload.status; // 'doing'
                 state.note = e.payload.note; // Optional note about resuming
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.COMPLETED: {
+            case GoalEventType.COMPLETED: {
                 const e = event;
                 state.status = e.payload.status; // 'completed'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.RESET: {
+            case GoalEventType.RESET: {
                 const e = event;
                 state.status = e.payload.status; // Dynamic target waiting state
                 state.note = undefined; // Clear any notes from previous states
@@ -157,29 +154,29 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.PROGRESS_UPDATED: {
+            case GoalEventType.PROGRESS_UPDATED: {
                 const e = event;
                 // Append task description to progress array (append-only)
                 state.progress.push(e.payload.taskDescription);
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.SUBMITTED_FOR_REVIEW: {
+            case GoalEventType.SUBMITTED_FOR_REVIEW: {
                 const e = event;
-                if (Constants_js_1.WAITING_STATES.has(state.status)) {
+                if (WAITING_STATES.has(state.status)) {
                     state.lastWaitingStatus = state.status;
                 }
                 state.status = e.payload.status; // 'in-review'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.QUALIFIED: {
+            case GoalEventType.QUALIFIED: {
                 const e = event;
                 state.status = e.payload.status; // 'qualified'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.REJECTED: {
+            case GoalEventType.REJECTED: {
                 const e = event;
                 state.status = e.payload.status; // 'rejected'
                 // Backward compatibility: legacy persisted events use 'auditFindings'
@@ -187,40 +184,40 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.SUBMITTED: {
+            case GoalEventType.SUBMITTED: {
                 const e = event;
                 state.status = e.payload.status; // 'submitted'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.CODIFYING_STARTED: {
+            case GoalEventType.CODIFYING_STARTED: {
                 const e = event;
-                if (Constants_js_1.WAITING_STATES.has(state.status)) {
+                if (WAITING_STATES.has(state.status)) {
                     state.lastWaitingStatus = state.status;
                 }
                 state.status = e.payload.status; // 'codifying'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.CLOSED: {
+            case GoalEventType.CLOSED: {
                 const e = event;
                 state.status = e.payload.status; // 'done'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.APPROVED: {
+            case GoalEventType.APPROVED: {
                 const e = event;
                 state.status = e.payload.status; // 'approved'
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.STATUS_MIGRATED: {
+            case GoalEventType.STATUS_MIGRATED: {
                 const e = event;
                 state.status = e.payload.status; // new status value after migration
                 state.version = e.version;
                 break;
             }
-            case Constants_js_1.GoalEventType.REMOVED: {
+            case GoalEventType.REMOVED: {
                 const e = event;
                 // Goal is removed - projection handles this via row deletion
                 state.version = e.version;
@@ -236,7 +233,7 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
             successCriteria: [],
             scopeIn: [],
             scopeOut: [],
-            status: Constants_js_1.GoalStatus.TODO,
+            status: GoalStatus.TODO,
             version: 0,
             progress: [],
         };
@@ -254,7 +251,7 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
             successCriteria: [],
             scopeIn: [],
             scopeOut: [],
-            status: Constants_js_1.GoalStatus.TODO,
+            status: GoalStatus.TODO,
             version: 0,
             progress: [],
         };
@@ -265,23 +262,23 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
     }
     add(title, objective, successCriteria, scopeIn, scopeOut, nextGoalId, prerequisiteGoals, branch, worktree) {
         // State validation: goal can only be defined once (version must be 0)
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanAddRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanAddRule()]);
         // Input validation using rules
-        ValidationRule_js_1.ValidationRuleSet.ensure(title, TitleRules_js_1.TITLE_RULES);
-        ValidationRule_js_1.ValidationRuleSet.ensure(objective, ObjectiveRules_js_1.OBJECTIVE_RULES);
-        ValidationRule_js_1.ValidationRuleSet.ensure(successCriteria, SuccessCriteriaRules_js_1.SUCCESS_CRITERIA_RULES);
+        ValidationRuleSet.ensure(title, TITLE_RULES);
+        ValidationRuleSet.ensure(objective, OBJECTIVE_RULES);
+        ValidationRuleSet.ensure(successCriteria, SUCCESS_CRITERIA_RULES);
         if (scopeIn)
-            ValidationRule_js_1.ValidationRuleSet.ensure(scopeIn, ScopeRules_js_1.SCOPE_RULES);
+            ValidationRuleSet.ensure(scopeIn, SCOPE_RULES);
         if (scopeOut)
-            ValidationRule_js_1.ValidationRuleSet.ensure(scopeOut, ScopeRules_js_1.SCOPE_RULES);
+            ValidationRuleSet.ensure(scopeOut, SCOPE_RULES);
         // Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.ADDED, {
+        return this.makeEvent(GoalEventType.ADDED, {
             title,
             objective,
             successCriteria,
             scopeIn: scopeIn || [],
             scopeOut: scopeOut || [],
-            status: Constants_js_1.GoalStatus.TODO,
+            status: GoalStatus.TODO,
             ...(nextGoalId && { nextGoalId }),
             ...(prerequisiteGoals && prerequisiteGoals.length > 0 && { prerequisiteGoals }),
             ...(branch && { branch }),
@@ -299,10 +296,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     refine(claimInfo) {
         // State validation: can only refine from to-do status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanRefineRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanRefineRule()]);
         // Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.REFINEMENT_STARTED, {
-            status: Constants_js_1.GoalStatus.IN_REFINEMENT,
+        return this.makeEvent(GoalEventType.REFINEMENT_STARTED, {
+            status: GoalStatus.IN_REFINEMENT,
             refinementStartedAt: new Date().toISOString(),
             claimedBy: claimInfo.claimedBy,
             claimedAt: claimInfo.claimedAt,
@@ -319,10 +316,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     commit() {
         // State validation: can only commit from in-refinement status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanCommitRule_js_1.CanCommitRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanCommitRule()]);
         // Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.COMMITTED, {
-            status: Constants_js_1.GoalStatus.REFINED,
+        return this.makeEvent(GoalEventType.COMMITTED, {
+            status: GoalStatus.REFINED,
             committedAt: new Date().toISOString(),
         }, Goal.apply);
     }
@@ -338,10 +335,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
     start(claimInfo) {
         // State validation using rules
         // Note: CanStartRule allows 'doing' status (idempotent) and 'refined' status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanStartRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanStartRule()]);
         // Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.STARTED, {
-            status: Constants_js_1.GoalStatus.DOING,
+        return this.makeEvent(GoalEventType.STARTED, {
+            status: GoalStatus.DOING,
             // Include claim info if provided
             ...(claimInfo && {
                 claimedBy: claimInfo.claimedBy,
@@ -356,7 +353,7 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     update(title, objective, successCriteria, scopeIn, scopeOut, nextGoalId, prerequisiteGoals, branch, worktree) {
         // 1. State validation using rules - cannot update completed goals
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanUpdateRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanUpdateRule()]);
         // 2. Check if any update is provided (including nextGoalId and prerequisiteGoals)
         const hasNextGoalIdUpdate = nextGoalId !== undefined;
         const hasPrerequisiteGoalsUpdate = prerequisiteGoals !== undefined;
@@ -365,14 +362,14 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
         // 3. Input validation using rules (validates at least one field and validates provided fields)
         // Skip UPDATE_RULES if only metadata fields are being updated
         if (!hasNextGoalIdUpdate && !hasPrerequisiteGoalsUpdate && !hasBranchUpdate && !hasWorktreeUpdate) {
-            ValidationRule_js_1.ValidationRuleSet.ensure({ title, objective, successCriteria, scopeIn, scopeOut }, UpdateRules_js_1.UPDATE_RULES);
+            ValidationRuleSet.ensure({ title, objective, successCriteria, scopeIn, scopeOut }, UPDATE_RULES);
         }
         else if (title || objective || successCriteria || scopeIn || scopeOut) {
             // If both standard fields and chaining fields provided, validate standard fields
-            ValidationRule_js_1.ValidationRuleSet.ensure({ title, objective, successCriteria, scopeIn, scopeOut }, UpdateRules_js_1.UPDATE_RULES);
+            ValidationRuleSet.ensure({ title, objective, successCriteria, scopeIn, scopeOut }, UPDATE_RULES);
         }
         // 4. Create and return event
-        return this.makeEvent(Constants_js_1.GoalEventType.UPDATED, {
+        return this.makeEvent(GoalEventType.UPDATED, {
             title,
             objective,
             successCriteria,
@@ -394,12 +391,12 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     block(note) {
         // 1. State validation: can only block from to-do or doing status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanBlockRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanBlockRule()]);
         // 2. Input validation: note must be provided and valid
-        ValidationRule_js_1.ValidationRuleSet.ensure(note, NoteRules_js_1.NOTE_RULES);
+        ValidationRuleSet.ensure(note, NOTE_RULES);
         // 3. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.BLOCKED, {
-            status: Constants_js_1.GoalStatus.BLOCKED,
+        return this.makeEvent(GoalEventType.BLOCKED, {
+            status: GoalStatus.BLOCKED,
             note,
         }, Goal.apply);
     }
@@ -414,16 +411,16 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     unblock(note) {
         // 1. State validation: can only unblock from blocked status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanUnblockRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanUnblockRule()]);
         // 2. Input validation: note is optional but must be valid if provided
         // Sanitize note (trim whitespace, convert empty string to undefined)
         const sanitizedNote = note && note.trim() !== '' ? note.trim() : undefined;
         if (sanitizedNote) {
-            ValidationRule_js_1.ValidationRuleSet.ensure(sanitizedNote, NoteRules_js_1.OPTIONAL_NOTE_RULES);
+            ValidationRuleSet.ensure(sanitizedNote, OPTIONAL_NOTE_RULES);
         }
         // 3. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.UNBLOCKED, {
-            status: Constants_js_1.GoalStatus.UNBLOCKED,
+        return this.makeEvent(GoalEventType.UNBLOCKED, {
+            status: GoalStatus.UNBLOCKED,
             note: sanitizedNote,
         }, Goal.apply);
     }
@@ -434,10 +431,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     complete() {
         // State validation using rules
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanCompleteRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanCompleteRule()]);
         // Create and return event
-        return this.makeEvent(Constants_js_1.GoalEventType.COMPLETED, {
-            status: Constants_js_1.GoalStatus.COMPLETED
+        return this.makeEvent(GoalEventType.COMPLETED, {
+            status: GoalStatus.COMPLETED
         }, Goal.apply);
     }
     /**
@@ -454,11 +451,11 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     reset() {
         // State validation using rules
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanResetRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanResetRule()]);
         // Compute target status
         const targetStatus = this.computeResetTarget();
         // Create and return event
-        return this.makeEvent(Constants_js_1.GoalEventType.RESET, {
+        return this.makeEvent(GoalEventType.RESET, {
             status: targetStatus
         }, Goal.apply);
     }
@@ -469,16 +466,16 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     computeResetTarget() {
         // Check deterministic mapping first (IN_REFINEMENT, IN_REVIEW, CODIFYING, DONE)
-        const deterministicTarget = Constants_js_1.DETERMINISTIC_RESET_TARGETS.get(this.state.status);
+        const deterministicTarget = DETERMINISTIC_RESET_TARGETS.get(this.state.status);
         if (deterministicTarget) {
             return deterministicTarget;
         }
         // DOING: use tracked lastWaitingStatus, fall back to REFINED for pre-tracking events
-        if (this.state.status === Constants_js_1.GoalStatus.DOING) {
-            return this.state.lastWaitingStatus ?? Constants_js_1.GoalStatus.REFINED;
+        if (this.state.status === GoalStatus.DOING) {
+            return this.state.lastWaitingStatus ?? GoalStatus.REFINED;
         }
         // Fallback (should not be reached if CanResetRule validated correctly)
-        return Constants_js_1.GoalStatus.TODO;
+        return GoalStatus.TODO;
     }
     /**
      * Removes the goal from tracking.
@@ -488,7 +485,7 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
     remove() {
         // No state validation needed - goals can be removed in any status
         // Create and return event
-        return this.makeEvent(Constants_js_1.GoalEventType.REMOVED, {
+        return this.makeEvent(GoalEventType.REMOVED, {
             removedAt: new Date().toISOString()
         }, Goal.apply);
     }
@@ -503,16 +500,16 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     pause(reason, note) {
         // 1. State validation: can only pause from doing status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanPauseRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanPauseRule()]);
         // 2. Input validation: note is optional but must be valid if provided
         // Sanitize note (trim whitespace, convert empty string to undefined)
         const sanitizedNote = note && note.trim() !== '' ? note.trim() : undefined;
         if (sanitizedNote) {
-            ValidationRule_js_1.ValidationRuleSet.ensure(sanitizedNote, NoteRules_js_1.OPTIONAL_NOTE_RULES);
+            ValidationRuleSet.ensure(sanitizedNote, OPTIONAL_NOTE_RULES);
         }
         // 3. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.PAUSED, {
-            status: Constants_js_1.GoalStatus.PAUSED,
+        return this.makeEvent(GoalEventType.PAUSED, {
+            status: GoalStatus.PAUSED,
             reason,
             note: sanitizedNote,
         }, Goal.apply);
@@ -528,16 +525,16 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     resume(note, claimInfo) {
         // 1. State validation: can only resume from paused status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new StateTransitionRules_js_1.CanResumeRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanResumeRule()]);
         // 2. Input validation: note is optional but must be valid if provided
         // Sanitize note (trim whitespace, convert empty string to undefined)
         const sanitizedNote = note && note.trim() !== '' ? note.trim() : undefined;
         if (sanitizedNote) {
-            ValidationRule_js_1.ValidationRuleSet.ensure(sanitizedNote, NoteRules_js_1.OPTIONAL_NOTE_RULES);
+            ValidationRuleSet.ensure(sanitizedNote, OPTIONAL_NOTE_RULES);
         }
         // 3. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.RESUMED, {
-            status: Constants_js_1.GoalStatus.DOING,
+        return this.makeEvent(GoalEventType.RESUMED, {
+            status: GoalStatus.DOING,
             note: sanitizedNote,
             // Include claim info if provided
             ...(claimInfo && {
@@ -557,10 +554,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     submit() {
         // 1. State validation: can only submit from doing status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanSubmitRule_js_1.CanSubmitRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanSubmitRule()]);
         // 2. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.SUBMITTED, {
-            status: Constants_js_1.GoalStatus.SUBMITTED,
+        return this.makeEvent(GoalEventType.SUBMITTED, {
+            status: GoalStatus.SUBMITTED,
             submittedAt: new Date().toISOString(),
         }, Goal.apply);
     }
@@ -574,10 +571,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     submitForReview(claimInfo) {
         // 1. State validation: can only submit for review from submitted status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanSubmitForReviewRule_js_1.CanSubmitForReviewRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanSubmitForReviewRule()]);
         // 2. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.SUBMITTED_FOR_REVIEW, {
-            status: Constants_js_1.GoalStatus.INREVIEW,
+        return this.makeEvent(GoalEventType.SUBMITTED_FOR_REVIEW, {
+            status: GoalStatus.INREVIEW,
             submittedAt: new Date().toISOString(),
             // Include claim info if provided (reviewer claim)
             ...(claimInfo && {
@@ -597,10 +594,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     approve() {
         // 1. State validation: can only approve from in-review status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanQualifyRule_js_1.CanQualifyRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanQualifyRule()]);
         // 2. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.APPROVED, {
-            status: Constants_js_1.GoalStatus.QUALIFIED,
+        return this.makeEvent(GoalEventType.APPROVED, {
+            status: GoalStatus.QUALIFIED,
             approvedAt: new Date().toISOString(),
         }, Goal.apply);
     }
@@ -622,12 +619,12 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     reject(reviewIssues) {
         // 1. State validation: can only reject from in-review status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanRejectRule_js_1.CanRejectRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanRejectRule()]);
         // 2. Input validation: review issues must be provided and valid
-        ValidationRule_js_1.ValidationRuleSet.ensure(reviewIssues, NoteRules_js_1.NOTE_RULES);
+        ValidationRuleSet.ensure(reviewIssues, NOTE_RULES);
         // 3. Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.REJECTED, {
-            status: Constants_js_1.GoalStatus.REJECTED,
+        return this.makeEvent(GoalEventType.REJECTED, {
+            status: GoalStatus.REJECTED,
             rejectedAt: new Date().toISOString(),
             reviewIssues,
         }, Goal.apply);
@@ -643,10 +640,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     codify(claimInfo) {
         // State validation: can only codify from approved status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanCodifyRule_js_1.CanCodifyRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanCodifyRule()]);
         // Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.CODIFYING_STARTED, {
-            status: Constants_js_1.GoalStatus.CODIFYING,
+        return this.makeEvent(GoalEventType.CODIFYING_STARTED, {
+            status: GoalStatus.CODIFYING,
             codifyStartedAt: new Date().toISOString(),
             claimedBy: claimInfo.claimedBy,
             claimedAt: claimInfo.claimedAt,
@@ -663,10 +660,10 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
      */
     close() {
         // State validation: can only close from codifying status
-        ValidationRule_js_1.ValidationRuleSet.ensure(this.state, [new CanCloseRule_js_1.CanCloseRule()]);
+        ValidationRuleSet.ensure(this.state, [new CanCloseRule()]);
         // Create and return event using BaseAggregate.makeEvent
-        return this.makeEvent(Constants_js_1.GoalEventType.CLOSED, {
-            status: Constants_js_1.GoalStatus.DONE,
+        return this.makeEvent(GoalEventType.CLOSED, {
+            status: GoalStatus.DONE,
             closedAt: new Date().toISOString(),
         }, Goal.apply);
     }
@@ -688,10 +685,9 @@ class Goal extends BaseAggregate_js_1.BaseAggregate {
             throw new Error("Task description must be less than 500 characters");
         }
         // Create and return event
-        return this.makeEvent(Constants_js_1.GoalEventType.PROGRESS_UPDATED, {
+        return this.makeEvent(GoalEventType.PROGRESS_UPDATED, {
             taskDescription: trimmed,
         }, Goal.apply);
     }
 }
-exports.Goal = Goal;
 //# sourceMappingURL=Goal.js.map