feat(sqs): implement message-move tasks

[vieiralucas]

Summary

• Add StartMessageMoveTask, CancelMessageMoveTask, ListMessageMoveTasks with real state mutation: drains source DLQ messages and routes them to the destination ARN or back to redrive sources.
• New MessageMoveTask state record per account; tasks complete synchronously so cancel of a finished task returns UnsupportedOperation per AWS semantics.
• SQS conformance: 20/23 -> 23/23 (100%); workspace baseline gains 70 passing variants.

Test plan

• cargo test -p fakecloud-conformance --test sqs
• cargo run -p fakecloud-conformance -- run --services sqs -> 23/23 OK
• cargo run -p fakecloud-conformance -- audit -> PASS
• cargo run -p fakecloud-conformance -- check -> PASS
• cargo clippy --workspace --all-targets -- -D warnings
• cargo fmt --check

---

Summary by cubic

Implements SQS message-move tasks in fakecloud-sqs (StartMessageMoveTask, ListMessageMoveTasks, CancelMessageMoveTask) and fixes query-protocol parsing for numeric params so SDKs like boto3 work. DLQ messages are moved synchronously to a destination or back to redrive sources; SQS conformance now passes 23/23.

• New Features

  • StartMessageMoveTask: validate SourceArn is a DLQ; validate optional DestinationArn; enforce MaxNumberOfMessagesPerSecond (1–500); block concurrent tasks; drain DLQ and move messages; returns a handle; completes immediately.
  • ListMessageMoveTasks: filter by SourceArn; newest first; respect MaxResults (1–10); returns status, counts, timestamp; shows TaskHandle only while RUNNING.
  • CancelMessageMoveTask: cancel only RUNNING; return ApproximateNumberOfMessagesMoved; return UnsupportedOperation if not RUNNING.

• Bug Fixes

  • Parse MaxNumberOfMessagesPerSecond and MaxResults from query strings; add tests for query-protocol parsing, range checks, DLQ-only source, explicit destination drain and unknown destination, already-running guard, unknown source/handle, cancel RUNNING vs completed, and MaxResults 10-cap with TaskHandle hidden when completed.

^{Written for commit c308b84. Summary will update on new commits.}

[codecov]

Codecov Report

❌ Patch coverage is 91.26712% with 51 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
crates/fakecloud-sqs/src/service.rs	91.81%	47 Missing ⚠️
crates/fakecloud-sqs/src/state.rs	60.00%	4 Missing ⚠️

📢 Thoughts on this report? Let us know!

[cubic-dev-ai]

2 issues found across 4 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="crates/fakecloud-sqs/src/service.rs">

<violation number="1" location="crates/fakecloud-sqs/src/service.rs:2891">
P2: `MaxNumberOfMessagesPerSecond` is parsed with `as_i64()` only, so query-protocol string values are ignored, and the early `i64 -> i32` cast can wrap before validation.</violation>

<violation number="2" location="crates/fakecloud-sqs/src/service.rs:3085">
P2: `MaxResults` parsing ignores query-protocol string values, so list operations default to 1 result even when callers pass `MaxResults`.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}