fix(skills): retry ClawHub requests on HTTP 429

[qs3c]

Summary

• reuse utils.DoRequestWithRetry for ClawHub search/download HTTP requests
• remove duplicated retry implementation in ClawHubRegistry
• keep 429 retry behavior covered by existing ClawHub registry tests
• document registries.clawhub.auth_token in skills tool config docs

Testing

• go test ./pkg/skills ./pkg/utils

Closes #1134

[afjcjsbx]

good point about adding retry on status_codes 429, but rather than reimplement the client with retry I would use the one already existing in the utils package 🙏

[qs3c]

Thanks for the review. Updated in 536e9ac: removed custom retry implementation in ClawHub registry and switched search/download requests to use utils.DoRequestWithRetry, keeping behavior covered by existing tests.

[CLAassistant]

All committers have signed the CLA.

[afjcjsbx]

LGTM!

[Orgmar]

@qs3c Nice work reusing DoRequestWithRetry for ClawHub requests and cleaning up the duplicated retry logic. Keeping things DRY while making sure 429 retries still work properly is solid.

We have a PicoClaw Dev Group on Discord where contributors collaborate and share ideas. If you're interested, send an email to [email protected] with the subject [Join PicoClaw Dev Group] qs3c and we'll get you the invite link!