Fix async state management in ReassignModal

[Copilot]

The ReassignModal component was managing isSubmitting state locally and resetting it immediately after emitting the reassign event, causing the submit button to re-enable before the parent's async API request completed.

Changes:

• ReassignModal.vue: Accept isSubmitting as prop instead of managing it locally; removed async wrapper and state reset from handleSubmit
• ProjectDetail.vue: Added isReassigningTask ref; wrapped handleReassignTask with try-finally to control state lifecycle; passed state as prop to modal

Before:

// Modal manages its own state
const handleSubmit = async () => {
  isSubmitting.value = true
  try {
    emit('reassign', data)  // Fire-and-forget
  } finally {
    isSubmitting.value = false  // Resets immediately
  }
}

After:

// Parent manages state across async boundary
const handleReassignTask = async (data) => {
  isReassigningTask.value = true
  try {
    await fetch(...)  // Actual async operation
  } finally {
    isReassigningTask.value = false  // Resets after completion
  }
}

Button now stays disabled until API request completes or fails.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

This PR fixes a race condition in the ReassignModal component where the submit button would re-enable prematurely before the parent's async API request completed. The solution moves the loading state management from the child modal to the parent component, ensuring the button stays disabled until the API operation finishes.

Changes:

• Moved isSubmitting state management from ReassignModal to parent ProjectDetail component
• Added try-finally block in parent's handleReassignTask to properly control state lifecycle
• Passed loading state as prop to modal for controlled button disabling

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File	Description
web/src/components/ReassignModal.vue	Converted to controlled component by accepting isSubmitting prop; removed local async state management
web/src/views/ProjectDetail.vue	Added isReassigningTask ref with proper lifecycle management in try-finally block; passes state to modal
web/package-lock.json	Automated npm lock file updates (removal of peer: true flags from several dependencies)

Files not reviewed (1)

• web/package-lock.json: Language not supported

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The local form state is being reset immediately after emitting the reassign event, before the parent's async operation completes. This causes the selectedAgentId to clear while the modal is still visible (if an error occurs and the modal stays open).

The watch on props.show already resets selectedAgentId when the modal opens, so this line should be removed. Let the parent control when the modal closes, and the watch will naturally reset the form when it reopens.

Suggested change

selectedAgentId.value = ''