harbor convert update (superseded)

[nancyjlau]

No description provided.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 6568d5f. Configure here.}

[cursor]

Slug collision resolution is a no-op for long names

Low Severity

In _make_task_slug, when len(base) > 100, line 112 rewrites base to f"{base[:91].rstrip('-')}-{digest}". The subsequent collision check on line 115–116 computes f"{base[:91].rstrip('-')}-{digest}" again, which in the common case (no trailing dashes near position 91) reproduces the same value as base, leaving slug unchanged and the collision unresolved. The while loop on lines 118–123 catches it, so uniqueness is preserved, but the if branch is effectively dead code for truncated slugs.

 

^{Reviewed by Cursor Bugbot for commit 6568d5f. Configure here.}