diff --git a/internal/mailbox/message.go b/internal/mailbox/message.go index d8f2820..0cbc18a 100644 --- a/internal/mailbox/message.go +++ b/internal/mailbox/message.go @@ -10,9 +10,6 @@ import ( type MessageType string const ( - // Auto-complete message types (log only) - MessageTypeLogError MessageType = "LOG_ERROR" // Errors - // Inbox message types MessageTypePromptResponse MessageType = "PROMPT_RESPONSE" // Bot response to user prompt MessageTypePromptRequest MessageType = "PROMPT_REQUEST" // User instructions to bot @@ -24,12 +21,11 @@ const ( MessageTypeApprovalResponse MessageType = "APPROVAL_RESPONSE" // Provide test results // Deprecated types (for backward compatibility) - MessageTypeLogInfo MessageType = "LOG_INFO" // Deprecated: use PROMPT_RESPONSE MessageTypePrompt MessageType = "PROMPT" // Deprecated: use PROMPT_REQUEST MessageTypeQuestion MessageType = "QUESTION" // Deprecated: use QUERY_REQUEST MessageTypeAnswer MessageType = "ANSWER" // Deprecated: use QUERY_RESPONSE MessageTypeStatusUpdate MessageType = "STATUS_UPDATE" // Deprecated: use PROMPT_RESPONSE - MessageTypeErrorReport MessageType = "ERROR_REPORT" // Deprecated: use LOG_ERROR + MessageTypeErrorReport MessageType = "ERROR_REPORT" // Deprecated: use PROMPT_RESPONSE ) // Message represents a structured message between sessions @@ -78,17 +74,13 @@ func generateID() string { // ValidateMessageType checks if the message type is valid and returns an error if deprecated or invalid func ValidateMessageType(msgType MessageType) error { switch msgType { - // Valid ephemeral types - case MessageTypePromptResponse, MessageTypeLogError: - return nil - // Valid persistent types - case MessageTypePromptRequest, MessageTypeQueryRequest, MessageTypeQueryResponse, + // Valid types + case MessageTypePromptResponse, + MessageTypePromptRequest, MessageTypeQueryRequest, MessageTypeQueryResponse, MessageTypeReviewRequest, MessageTypeReviewResponse, MessageTypeApprovalRequest, MessageTypeApprovalResponse: return nil // Deprecated types - case MessageTypeLogInfo: - return fmt.Errorf("deprecated type: use PROMPT_RESPONSE instead") case MessageTypePrompt: return fmt.Errorf("deprecated type: use PROMPT_REQUEST instead") case MessageTypeQuestion: @@ -98,8 +90,8 @@ func ValidateMessageType(msgType MessageType) error { case MessageTypeStatusUpdate: return fmt.Errorf("deprecated type: use PROMPT_RESPONSE instead") case MessageTypeErrorReport: - return fmt.Errorf("deprecated type: use LOG_ERROR instead") + return fmt.Errorf("deprecated type: use PROMPT_RESPONSE instead") default: - return fmt.Errorf("invalid message type. valid types are: PROMPT_RESPONSE, LOG_ERROR, PROMPT_REQUEST, QUERY_REQUEST, QUERY_RESPONSE, REVIEW_REQUEST, REVIEW_RESPONSE, APPROVAL_REQUEST, APPROVAL_RESPONSE") + return fmt.Errorf("invalid message type. valid types are: PROMPT_RESPONSE, PROMPT_REQUEST, QUERY_REQUEST, QUERY_RESPONSE, REVIEW_REQUEST, REVIEW_RESPONSE, APPROVAL_REQUEST, APPROVAL_RESPONSE") } } diff --git a/skills/anvillm-communication/SKILL.md b/skills/anvillm-communication/SKILL.md index ee6e03d..b866f6d 100644 --- a/skills/anvillm-communication/SKILL.md +++ b/skills/anvillm-communication/SKILL.md @@ -35,8 +35,6 @@ Use the `send_message` tool with these parameters: - `REVIEW_REQUEST` / `REVIEW_RESPONSE` - Request/provide reviews - `APPROVAL_REQUEST` / `APPROVAL_RESPONSE` - Request/provide approvals -**Error Reporting:** -- `LOG_ERROR` - Report errors (auto-completed) ### Examples