cli/pkg/cmd/skills/publish
Sam Morrow 29e55fe5b9
refactor: remove redundant nil-client fallback in skills publish (#13168)
Remove the dead code block that silently swallowed errors from
opts.HttpClient() and opts.Config() when creating an API client.
Instead, create the client with proper error propagation inside the
remote-checks block where it is actually needed.

Changes:
- Remove the error-swallowing client == nil fallback (lines 363-376)
- Create the API client inside the remote repo checks block with
  proper error returns from HttpClient() and Config()
- Resolve host from repoInfo first, then fall back to config
- Remove the now-unreachable 'client == nil' early exit before publish

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2026-04-16 15:52:53 +02:00
..
publish.go refactor: remove redundant nil-client fallback in skills publish (#13168) 2026-04-16 15:52:53 +02:00
publish_test.go Merge pull request #13167 from cli/sammorrowdrums/publish-use-shared-discovery 2026-04-16 15:48:20 +02:00