Bulk import silently queues zero assets for asset_groups due to double UUID-to-name conversion

Problem/Motivation

When bulk import is configured with asset_groups as the source, no assets are ever queued. The Drush command (drush ad-ia) completes with exit 0, no errors, nothing in watchdog and the queue stays empty. Categories work correctly; only asset_groups are affected.

In AssetQueueService::addAssetsToQueueBySource(), the asset group UUID gets converted to a name before being passed to buildQuery():

$source_uuid = $source == 'categories'
  ? rawurldecode($source_uuid)
  : $this->userClientFactory->getSiteClient()->convertAssetGroupUuidToName($source_uuid);

buildQuery() then calls convertAssetGroupUuidToName() again on what is now already a name, which returns NULL. $query stays NULL and the fetch loop is skipped.
Introduced in commit d740ef49 (Jan 2025).

Steps to reproduce

1. Configure bulk import with Source type "Asset Groups" against a populated group.
2. Run drush ad-ia.
3. Queue acquia_dam_asset_import stays empty. No errors logged.

Proposed resolution

I'm not sure what the original intent of the conversion in the caller was. Perhaps to use a human-readable name as the key for $import_details later. There may be a better fix that preserves that, but this seemed the simplest change to just drop the conversion in the caller and let buildQuery() handle it once:

if ($source == 'categories') {
  $source_uuid = rawurldecode($source_uuid);
}

Remaining tasks

- review patch
- suggest alternate approach if desired