В целом, ваш код выглядит неплохо, однако есть несколько замечаний и предложений по улучшению:

Имя метода ToHash не является самым информативным. Рекомендуется использовать более понятные имена, такие как GenerateUrlHashAsync или CreateUrlHashAsync.

Вместо InvalidEnumArgumentException лучше использовать ArgumentException или создать собственный класс исключения, например InvalidUrlException, так как InvalidEnumArgumentException обычно используется для проверки аргументов типа перечисления (enum).

В качестве CreatedBy используется пустая строка. Если это поле не является обязательным, возможно, стоит использовать null или добавить код для получения имени пользователя или системного аккаунта.

Добавьте XML-комментарии к методу, чтобы объяснить его назначение и параметры.

В блоке catch вы выбрасываете исключение снова, без всякой обработки. В зависимости от ситуации, вы можете добавить логирование или обработку ошибок.

Вот пример кода с учетом предложенных изменений:

/// <summary>
/// Generates a hash for the given URL, saves it to the database, and returns the hash.
/// </summary>
/// <param name="request">A CreateHashCommand containing the URL to hash.</param>
/// <param name="cancellationToken">A CancellationToken to observe while waiting for the task to complete.</param>
/// <returns>A Task that represents the asynchronous operation, containing the generated URL hash as a string.</returns>
public async Task<string> GenerateUrlHashAsync(CreateHashCommand request, CancellationToken cancellationToken)
{
    try
    {
        if (string.IsNullOrWhiteSpace(request.Url))
            throw new ArgumentException("Url cannot be null or empty.", nameof(request.Url));

        var urlInDb = new Url()
        {
            Created = DateTime.UtcNow,
            CreatedBy = null,
            OriginalUrl = request.Url
        };

        await _context.Urls.AddAsync(urlInDb, cancellationToken);
        await _context.SaveChangesAsync(cancellationToken);

        var urlHash = _hashgen.EncodeLong(urlInDb.Id);

        return urlHash;
    }
    catch (Exception ex)
    {
        // Add logging or error handling here.
        throw;
    }
}

 CREATE SEQUENCE [dbo].[MySequence] START WITH 1 INCREMENT BY 10;

 SELECT NEXT VALUE FOR [dbo].[MySequence]; -- (1) возвращается 1

 SELECT NEXT VALUE FOR [dbo].[MySequence]; -- (2) возвращается 11

lock(syncLock) {
  var old = current
  current += 10;
  return old;
}

• Command vs request в названиях
  
• Странному исключению при проверке URL (у которого может быть разумное объяснение)
  
• Тому, что проверка должна быть в конструкторе Url, а может и не должна, если он больше нигде не вызвается
  
• Тому, что значения для Created/CreatedBy похожи на дефолтные, которые надо выставлять в инициализаторах полей, или конструкторе (в зависимости от пожелания левой пятки тимлида)
  
• Тому, что new Url можно переместить в Context.Urls
  
• Тому, что SaveChangesAsync() и AddAsync() логично бы иметь на одном уровне (или на нескольких, в зависимости от того, что еще есть в _context)
  
• Тому, что считать string-хэш от ulong довольно странно
  
• Тому, что сохранять изменения после каждого добавления может быть жутко неоптимально (а может быть, оптимально, по куче причин, которые мы никогда не угадаем)
  
• Тому, что делать catch и тут же throw как-то странно (а может и не странно, если автор любит ставить там breakpoint)
  
• Пустой строке перед throw

G>    public async Task<string> ToHash(CreateHashCommand request, CancellationToken cancellationToken)
G>    {
G>        try
G>        {
G>            if (string.IsNullOrWhiteSpace(request.Url))
G>                throw new InvalidEnumArgumentException();
G>            var urlInDb = new  Url()
G>            {
G>                Created = DateTime.UtcNow,
G>                CreatedBy = "",
G>                OriginalUrl = request.Url
G>            };
G>            await _context.Urls.AddAsync(urlInDb, cancellationToken);
G>            await _context.SaveChangesAsync(cancellationToken);
G>            var urlHash = _hashgen.EncodeLong(urlInDb.Id);

G>            return urlHash;
G>        }
G>        catch (Exception ex)
G>        {
G>            throw;
G>        }
G>    }

• …, CancellationToken cancellationToken = default) — так обычно удобнее объявлять CancellationToken-параметры.
  
• Если CreateHashCommand это ссылочный тип, то аргумент не мешало бы проверить на null.
  
• Вместо InvalidEnumArgumentException уместнее бросать ArgumentException или ArgumentNullException, как это делается, например, здесь.
  
• С сущностью (Url). Я б спросил, а это так и нужно, что CreatedBy = "" или это ошибка? Если это ошибка, то что там должно быть? Если это не ошибка и так и нужно, чтобы ORM сам подставил это поле нужное значение, возможно, ORM сам мог бы подставлять и Created?
  
• _context мне кажется лучше заводить локальный, раз уж мы его сохраняем здесь же.
  
• Подумать и решить, не нужно ли нам добавить .ConfigureAwait(continueOnCapturedContext: false). Скорее всего, нужно.

[Info] Order #100 created
[Info] Order item #101 for Order #100 added
[Info] Order item #102 for Order #100 added
[Info] Saving order #100 to DB
[Error] Db unavailable exception

    public async Task<string> ToHash(CreateHashCommand request, CancellationToken cancellationToken)
    {
        try
        {
            if (string.IsNullOrWhiteSpace(request.Url))
                throw new InvalidEnumArgumentException();
            var urlInDb = new  Url()
            {
                Created = DateTime.UtcNow,
                CreatedBy = "",
                OriginalUrl = request.Url
            };
            await _context.Urls.AddAsync(urlInDb, cancellationToken);
            await _context.SaveChangesAsync(cancellationToken);
            var urlHash = _hashgen.EncodeLong(urlInDb.Id);

            return urlHash;
        }
        catch (Exception ex)
        {

            throw;
        }
    }

class Url
{
  public Url()
  {
  }
}

...

var url = new Url{OriginalUrl = foo, Created = DateTime.Now);  //Повторяется 5 раз в разных местах

class Url
{
  public Url(string url)
  {
    Created = DateTime.Now;
    OriginalUrl = url;
  }
}

...

var url = new Url{foo);  //Повторяется 5 раз в разных местах

G>    public async Task<string> ToHash(CreateHashCommand request, CancellationToken cancellationToken)
G>    {
G>        try
G>        {
G>            if (string.IsNullOrWhiteSpace(request.Url))
G>                throw new InvalidEnumArgumentException();
G>            var urlInDb = new  Url()
G>            {
G>                Created = DateTime.UtcNow,
G>                CreatedBy = "",
G>                OriginalUrl = request.Url
G>            };
G>            await _context.Urls.AddAsync(urlInDb, cancellationToken);
G>            await _context.SaveChangesAsync(cancellationToken);
G>            var urlHash = _hashgen.EncodeLong(urlInDb.Id);

G>            return urlHash;
G>        }
G>        catch (Exception ex)
G>        {

G>            throw;
G>        }
G>    }

G>

static Func<long,string> _hashgen = ....;
public static string ToHash(this Url url) => _hashgen.EncodeLong(url.Id);

async Task<Url> SaveRequest(string requestUrl) 
{
     var urlInDb = new  Url()
            {
               Created = DateTime.UtcNow,
               CreatedBy = "",
               OriginalUrl = requestUrl
            };
           await _context.Urls.AddAsync(urlInDb, cancellationToken);
           await _context.SaveChangesAsync(cancellationToken);
         return urlInDb;
}

app.MapGet("/hash", async (string requestUrl) =>
{
    if (string.IsNullOrWhiteSpace(requestUrl)) return String.Empty;
    var url = await SaveRequest(requestUrl);
    return url.ToHash();
});

> null |> hash;;
val it: int = 0