AIGC: asset-adoption should ignore failed/incomplete tasks

[nighca]

Problem:

• After asset generation finishes, the frontend records adoption via /aigc/asset-adoption.
• The current request does not exclude failed or incomplete generation tasks (their outputs are discarded anyway).
• The backend validates task status while collecting task results; for such task IDs it returns HTTP 400.
• As a result, asset adoption fails unexpectedly when the payload contains failed/incomplete task IDs.

Expected behavior:

• Asset adoption should only include successfully completed tasks; or the backend should skip failed/incomplete tasks during aggregation instead of returning 400.

Likely cause:

• The frontend submits all task IDs related to the generation session, including tasks that ended up failed/incomplete and whose outputs were discarded.

Fix options (either or both):

1. Frontend: when calling /aigc/asset-adoption, filter out failed/incomplete tasks and only submit task IDs that produced valid outputs.
2. Backend: when collecting task results, skip failed/incomplete tasks (or treat them as ignorable) to avoid 400; optionally return partial success with a list of skipped task IDs.

Acceptance criteria:

• If a generation session contains failed/incomplete tasks, asset adoption still succeeds (at least for the successful outputs).
• No 400 is returned solely because the payload contains failed/incomplete task IDs.

[nighca]

@xgopilot Fix this problem in both frontend and backend.

[xgopilot]

✅ Fixed asset-adoption to handle failed/incomplete tasks

I've implemented fixes in both frontend and backend repositories to handle failed/incomplete tasks gracefully in asset adoption.

Changes Made

Frontend (goplus/builder)

• spx-gui/src/models/gen/animation-gen.ts: Modified getTaskIds() to filter only completed tasks
• spx-gui/src/models/gen/costume-gen.ts: Modified getTaskIds() to filter only completed tasks
• spx-gui/src/models/gen/sprite-gen.ts: Modified recordAdoption() to filter only completed tasks
• spx-gui/src/models/gen/backdrop-gen.ts: Modified recordAdoption() to filter only completed tasks

Backend (goplus/builder-backend)

• internal/controller/aigc_asset_adoption.go: Added status check to skip non-completed tasks when collecting URLs from task results

How It Works

Frontend: When calling /aigc/asset-adoption, the code now checks each task's status and only includes task IDs where status === TaskStatus.Completed. This prevents sending failed/incomplete task IDs to the backend.

Backend: When processing the asset adoption request, the code now:

1. Retrieves the task status from the database
2. Skips tasks that are not in completed status (pending, processing, cancelling, cancelled, failed)
3. Only collects URLs from completed tasks
4. Logs skipped tasks for debugging

Result

After these changes, asset adoption will succeed even when the generation session contains failed/incomplete tasks. Only successfully completed tasks will be processed, and no HTTP 400 errors will be returned for failed/incomplete task IDs.

Pull Requests Created

• Frontend: fix(spx-gui): filter failed/incomplete tasks in asset-adoption #2825
• Backend: https://github.com/goplus/builder-backend/pull/132