fix: guard fmt.Errorf %w against nil err

errors.Wrap(nil, "msg") returned nil, but fmt.Errorf("msg: %w", nil)
returns a non-nil error. Add nil check before wrapping.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

.github/workflows/bash-frozen.yml

@@ -1,35 +0,0 @@
-name: Bash scripts frozen
-
-on:
-  pull_request:
-    paths:
-      - 'kubectx'
-      - 'kubens'
-
-jobs:
-  comment:
-    runs-on: ubuntu-latest
-    permissions:
-      pull-requests: write
-    steps:
-      - name: Comment on PR if author is not ahmetb
-        if: github.event.pull_request.user.login != 'ahmetb'
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const body = [
-              '> [!WARNING]',
-              '> **This PR will not be merged.**',
-              '>',
-              '> The bash implementation of `kubectx` and `kubens` is **frozen** and is provided only for convenience.',
-              '> We are not accepting any improvements to the bash scripts.',
-              '>',
-              '> Please propose your improvements to the **Go implementation** instead.',
-            ].join('\n');
-
-            await github.rest.issues.createComment({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              issue_number: context.issue.number,
-              body: body
-            });

.github/workflows/release.yml

@@ -24,7 +24,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout
-      uses: actions/checkout@v4
+      uses: actions/checkout@master
     - run: git fetch --tags
     - name: Setup Go
       uses: actions/setup-go@v6

.goreleaser.yml

@@ -15,7 +15,7 @@
 # limitations under the License.
 
 # This is an example goreleaser.yaml file with some sane defaults.
-# Make sure to check the documentation at https://goreleaser.com
+# Make sure to check the documentation at http://goreleaser.com
 
 version: 2
 before:
@@ -69,11 +69,11 @@ archives:
       {{- else -}}v{{- . -}}
       {{- end -}}
     {{- end -}}
-  ids:
+  builds:
     - kubectx
   format_overrides:
     - goos: windows
-      formats: [zip]
+      format: zip
   files: ["LICENSE"]
 - id: kubens-archive
   name_template: |-
@@ -89,11 +89,11 @@ archives:
       {{- else -}}v{{- . -}}
       {{- end -}}
     {{- end -}}
-  ids:
+  builds:
     - kubens
   format_overrides:
     - goos: windows
-      formats: [zip]
+      format: zip
   files: ["LICENSE"]
 checksum:
   name_template: "checksums.txt"
@@ -111,7 +111,7 @@ snapcrafts:
       kubens is a tool to switch between Kubernetes namespaces (and configure them for kubectl) easily.
     grade: stable
     confinement: classic
-    base: core24
+    base: core20
     apps:
       kubectx:
         command: kubectx

README.md

@@ -72,23 +72,139 @@ names anymore.
 
 ## Installation
 
-| Package manager | Command |
-|---|---|
-| [Homebrew](https://brew.sh/) (macOS & Linux) | `brew install kubectx` |
-| [MacPorts](https://www.macports.org) (macOS) | `sudo port install kubectx` |
-| apt (Debian/Ubuntu) | `sudo apt install kubectx` |
-| pacman (Arch Linux) | `sudo pacman -S kubectx` |
-| [Chocolatey](https://chocolatey.org/) (Windows) | `choco install kubens kubectx` |
-| [Scoop](https://scoop.sh/) (Windows) | `scoop bucket add main && scoop install main/kubens main/kubectx` |
-| [winget](https://learn.microsoft.com/en-us/windows/package-manager/) (Windows) | `winget install --id ahmetb.kubectx && winget install --id ahmetb.kubens` |
-| [Krew](https://github.com/kubernetes-sigs/krew/) (kubectl plugin) | `kubectl krew install ctx && kubectl krew install ns` |
+Stable versions of `kubectx` and `kubens` are small bash scripts that you
+can find in this repository.
 
-Alternatively, download binaries from the [**Releases page →**](https://github.com/ahmetb/kubectx/releases) and add them to somewhere in your `PATH`.
+Starting with v0.9.0, `kubectx` and `kubens` **are now rewritten in Go**.  They
+should work the same way (and we'll keep the bash-based implementations around)
+but the new features will be added to the new Go programs.  Please help us test
+this new Go implementation by downloading the binaries from the [**Releases page
+→**](https://github.com/ahmetb/kubectx/releases)
 
-<details>
-<summary>Shell completion scripts</summary>
+**Installation options:**
 
-#### zsh (with [antibody](https://getantibody.github.io))
+- [as kubectl plugins (macOS & Linux)](#kubectl-plugins-macos-and-linux)
+- [with Homebrew (macOS & Linux)](#homebrew-macos-and-linux)
+- [with MacPorts (macOS)](#macports-macos)
+- [with apt (Debian)](#apt-debian)
+- [with pacman (Arch Linux)](#pacman-arch-linux)
+- [with Chocolatey (Windows)](#windows-installation-using-chocolatey)
+- [Windows Installation (using Scoop)](#windows-installation-using-scoop)
+- [with winget (Windows)](#windows-installation-using-winget)
+- [manually (macOS & Linux)](#manual-installation-macos-and-linux)
+
+If you like to add context/namespace information to your shell prompt (`$PS1`),
+you can try out [kube-ps1].
+
+[kube-ps1]: https://github.com/jonmosco/kube-ps1
+
+### Kubectl Plugins (macOS and Linux)
+
+You can install and use the [Krew](https://github.com/kubernetes-sigs/krew/) kubectl
+plugin manager to get `kubectx` and `kubens`.
+
+**Note:** This will not install the shell completion scripts. If you want them,
+*choose another installation method
+or install the scripts [manually](#manual-installation-macos-and-linux).
+
+```sh
+kubectl krew install ctx
+kubectl krew install ns
+```
+
+After installing, the tools will be available as `kubectl ctx` and `kubectl ns`.
+
+### Homebrew (macOS and Linux)
+
+If you use [Homebrew](https://brew.sh/) you can install like this:
+
+```sh
+brew install kubectx
+```
+
+This command will set up bash/zsh/fish completion scripts automatically. Make sure you [configure your shell](https://docs.brew.sh/Shell-Completion) to load completions for installed Homebrew formulas.
+
+
+### MacPorts (macOS)
+
+If you use [MacPorts](https://www.macports.org) you can install like this:
+
+```sh
+sudo port install kubectx
+```
+
+### apt (Debian)
+
+``` bash
+sudo apt install kubectx
+```
+Newer versions might be available on repos like
+[Debian Buster (testing)](https://packages.debian.org/buster/kubectx),
+[Sid (unstable)](https://packages.debian.org/sid/kubectx)
+(_if you are unfamiliar with the Debian release process and how to enable
+testing/unstable repos, check out the
+[Debian Wiki](https://wiki.debian.org/DebianReleases)_):
+
+
+### pacman (Arch Linux)
+
+Available as official Arch Linux package. Install it via:
+
+```bash
+sudo pacman -S kubectx
+```
+
+### Windows Installation (using Chocolatey)
+
+Available as packages on [Chocolatey](https://chocolatey.org/why-chocolatey)
+```pwsh
+choco install kubens kubectx
+```
+
+### Windows Installation (using Scoop)
+
+Available as packages on [Scoop](https://scoop.sh/)
+```pwsh
+scoop bucket add main
+scoop install main/kubens main/kubectx
+```
+
+### Windows Installation (using winget)
+
+Available as packages on [winget](https://learn.microsoft.com/en-us/windows/package-manager/)
+```pwsh
+winget install --id ahmetb.kubectx
+winget install --id ahmetb.kubens
+```
+
+### Manual Installation (macOS and Linux)
+
+Since `kubectx` and `kubens` are written in Bash, you should be able to install
+them to any POSIX environment that has Bash installed.
+
+- Download the `kubectx`, and `kubens` scripts.
+- Either:
+  - save them all to somewhere in your `PATH`,
+  - or save them to a directory, then create symlinks to `kubectx`/`kubens` from
+    somewhere in your `PATH`, like `/usr/local/bin`
+- Make `kubectx` and `kubens` executable (`chmod +x ...`)
+
+Example installation steps:
+
+``` bash
+sudo git clone https://github.com/ahmetb/kubectx /opt/kubectx
+sudo ln -s /opt/kubectx/kubectx /usr/local/bin/kubectx
+sudo ln -s /opt/kubectx/kubens /usr/local/bin/kubens
+```
+
+If you also want to have shell completions, pick an installation method for the
+[completion scripts](completion/) that fits your system best: [`zsh` with
+`antibody`](#completion-scripts-for-zsh-with-antibody), [plain
+`zsh`](#completion-scripts-for-plain-zsh),
+[`bash`](#completion-scripts-for-bash) or
+[`fish`](#completion-scripts-for-fish).
+
+#### Completion scripts for `zsh` with [antibody](https://getantibody.github.io)
 
 Add this line to your [Plugins File](https://getantibody.github.io/usage/) (e.g.
 `~/.zsh_plugins.txt`):
@@ -101,9 +217,9 @@ Depending on your setup, you might or might not need to call `compinit` or
 `autoload -U compinit && compinit` in your `~/.zshrc` after you load the Plugins
 file. If you use [oh-my-zsh](https://github.com/ohmyzsh/ohmyzsh), load the
 completions before you load `oh-my-zsh` because `oh-my-zsh` will call
-`compinit`.
+`compinit`. 
 
-#### zsh (plain)
+#### Completion scripts for plain `zsh`
 
 The completion scripts have to be in a path that belongs to `$fpath`. Either
 link or copy them to an existing folder.
@@ -128,7 +244,7 @@ depending on the `$fpath` of your zsh installation.
 
 In case of errors, calling `compaudit` might help.
 
-#### bash
+#### Completion scripts for `bash`
 
 ```bash
 git clone https://github.com/ahmetb/kubectx.git ~/.kubectx
@@ -143,7 +259,7 @@ export PATH=~/.kubectx:\$PATH
 EOF
 ```
 
-#### fish
+#### Completion scripts for `fish`
 
 ```fish
 mkdir -p ~/.config/fish/completions
@@ -151,12 +267,6 @@ ln -s /opt/kubectx/completion/kubectx.fish ~/.config/fish/completions/
 ln -s /opt/kubectx/completion/kubens.fish ~/.config/fish/completions/
 ```
 
-</details>
-
-> [!NOTE]
-> Tip: Show context/namespace in your shell prompt with [oh-my-posh](https://ohmyposh.dev/) or
-> simply with [kube-ps1](https://github.com/jonmosco/kube-ps1).
-
 -----
 
 ### Interactive mode
@@ -167,12 +277,12 @@ with fuzzy searching, you just need to [install
 
 ![kubectx interactive search with fzf](img/kubectx-interactive.gif)
 
-Caveats:
-- If you have `fzf` installed, but want to opt out of using this feature, set the
-  environment variable `KUBECTX_IGNORE_FZF=1`.
-- If you want to keep `fzf` interactive mode but need the default behavior of the
-  command, you can do it by piping the output to another command (e.g. `kubectx |
-  cat `).
+If you have `fzf` installed, but want to opt out of using this feature, set the
+environment variable `KUBECTX_IGNORE_FZF=1`.
+
+If you want to keep `fzf` interactive mode but need the default behavior of the
+command, you can do it by piping the output to another command (e.g. `kubectx |
+cat `).
 
 -----
 
@@ -196,7 +306,7 @@ Colors in the output can be disabled by setting the
 If you liked `kubectx`, you may like my
 [`kubectl-aliases`](https://github.com/ahmetb/kubectl-aliases) project, too. I
 recommend pairing kubectx and kubens with [fzf](#interactive-mode) and
-[kube-ps1](https://github.com/jonmosco/kube-ps1).
+[kube-ps1].
 
 #### Stargazers over time
 

cmd/kubectx/current.go

@@ -35,14 +35,12 @@ func (_op CurrentOp) Run(stdout, _ io.Writer) error {
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	v, err := kc.GetCurrentContext()
-	if err != nil {
-		return fmt.Errorf("failed to get current context: %w", err)
-	}
+	v := kc.GetCurrentContext()
 	if v == "" {
 		return errors.New("current-context is not set")
 	}
-	if _, err := fmt.Fprintln(stdout, v); err != nil {
+	_, err := fmt.Fprintln(stdout, v)
+	if err != nil {
 		return fmt.Errorf("write error: %w", err)
 	}
 	return nil

cmd/kubectx/delete.go

@@ -44,7 +44,7 @@ func (op DeleteOp) Run(_, stderr io.Writer) error {
 				selfName())
 		}
 
-		_ = printer.Success(stderr, `Deleted context %s.`, printer.SuccessColor.Sprint(deletedName))
+		printer.Success(stderr, `Deleted context %s.`, printer.SuccessColor.Sprint(deletedName))
 	}
 	return nil
 }
@@ -58,10 +58,7 @@ func deleteContext(name string) (deleteName string, wasActiveContext bool, err e
 		return deleteName, false, fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	cur, err := kc.GetCurrentContext()
-	if err != nil {
-		return deleteName, false, fmt.Errorf("failed to get current context: %w", err)
-	}
+	cur := kc.GetCurrentContext()
 	// resolve "." to a real name
 	if name == "." {
 		if cur == "" {
@@ -71,11 +68,7 @@ func deleteContext(name string) (deleteName string, wasActiveContext bool, err e
 		name = cur
 	}
 
-	exists, err := kc.ContextExists(name)
-	if err != nil {
-		return name, false, fmt.Errorf("failed to check context: %w", err)
-	}
-	if !exists {
+	if !kc.ContextExists(name) {
 		return name, false, errors.New("context does not exist")
 	}
 

cmd/kubectx/fzf.go

@@ -43,7 +43,6 @@ func (op InteractiveSwitchOp) Run(_, stderr io.Writer) error {
 	}
 	// parse kubeconfig just to see if it can be loaded
 	kc := new(kubeconfig.Kubeconfig).WithLoader(kubeconfig.DefaultLoader)
-	defer kc.Close()
 	if err := kc.Parse(); err != nil {
 		if cmdutil.IsNotFoundErr(err) {
 			printer.Warning(stderr, "kubeconfig file not found")
@@ -51,6 +50,7 @@ func (op InteractiveSwitchOp) Run(_, stderr io.Writer) error {
 		}
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
+	kc.Close()
 
 	cmd := exec.Command("fzf", "--ansi", "--no-preview")
 	var out bytes.Buffer
@@ -75,7 +75,7 @@ func (op InteractiveSwitchOp) Run(_, stderr io.Writer) error {
 	if err != nil {
 		return fmt.Errorf("failed to switch context: %w", err)
 	}
-	_ = printer.Success(stderr, "Switched to context \"%s\".", printer.SuccessColor.Sprint(name))
+	printer.Success(stderr, "Switched to context \"%s\".", printer.SuccessColor.Sprint(name))
 	return nil
 }
 
@@ -85,7 +85,6 @@ func (op InteractiveDeleteOp) Run(_, stderr io.Writer) error {
 	}
 	// parse kubeconfig just to see if it can be loaded
 	kc := new(kubeconfig.Kubeconfig).WithLoader(kubeconfig.DefaultLoader)
-	defer kc.Close()
 	if err := kc.Parse(); err != nil {
 		if cmdutil.IsNotFoundErr(err) {
 			printer.Warning(stderr, "kubeconfig file not found")
@@ -93,12 +92,9 @@ func (op InteractiveDeleteOp) Run(_, stderr io.Writer) error {
 		}
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
+	kc.Close()
 
-	ctxNames, err := kc.ContextNames()
-	if err != nil {
-		return fmt.Errorf("failed to get context names: %w", err)
-	}
-	if len(ctxNames) == 0 {
+	if len(kc.ContextNames()) == 0 {
 		return errors.New("no contexts found in config")
 	}
 
@@ -133,7 +129,7 @@ func (op InteractiveDeleteOp) Run(_, stderr io.Writer) error {
 			selfName())
 	}
 
-	_ = printer.Success(stderr, `Deleted context %s.`, printer.SuccessColor.Sprint(name))
+	printer.Success(stderr, `Deleted context %s.`, printer.SuccessColor.Sprint(name))
 
 	return nil
 }

cmd/kubectx/isolated_shell_guard.go

@@ -19,6 +19,6 @@ func checkIsolatedMode() error {
 		return fmt.Errorf("you are in a locked single-context shell, use 'exit' to leave")
 	}
 
-	cur, _ := kc.GetCurrentContext()
+	cur := kc.GetCurrentContext()
 	return fmt.Errorf("you are in a locked single-context shell (\"%s\"), use 'exit' to leave", cur)
 }

cmd/kubectx/list.go

@@ -42,16 +42,10 @@ func (_ ListOp) Run(stdout, stderr io.Writer) error {
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	ctxs, err := kc.ContextNames()
-	if err != nil {
-		return fmt.Errorf("failed to get context names: %w", err)
-	}
+	ctxs := kc.ContextNames()
 	natsort.Sort(ctxs)
 
-	cur, err := kc.GetCurrentContext()
-	if err != nil {
-		return fmt.Errorf("failed to get current context: %w", err)
-	}
+	cur := kc.GetCurrentContext()
 	for _, c := range ctxs {
 		s := c
 		if c == cur {

cmd/kubectx/rename.go

@@ -52,27 +52,16 @@ func (op RenameOp) Run(_, stderr io.Writer) error {
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	cur, err := kc.GetCurrentContext()
-	if err != nil {
-		return fmt.Errorf("failed to get current context: %w", err)
-	}
+	cur := kc.GetCurrentContext()
 	if op.Old == "." {
 		op.Old = cur
 	}
 
-	oldExists, err := kc.ContextExists(op.Old)
-	if err != nil {
-		return fmt.Errorf("failed to check context: %w", err)
-	}
-	if !oldExists {
+	if !kc.ContextExists(op.Old) {
 		return fmt.Errorf("context \"%s\" not found, can't rename it", op.Old)
 	}
 
-	newExists, err := kc.ContextExists(op.New)
-	if err != nil {
-		return fmt.Errorf("failed to check context: %w", err)
-	}
-	if newExists {
+	if kc.ContextExists(op.New) {
 		printer.Warning(stderr, "context \"%s\" exists, overwriting it.", op.New)
 		if err := kc.DeleteContextEntry(op.New); err != nil {
 			return fmt.Errorf("failed to delete new context to overwrite it: %w", err)
@@ -90,7 +79,7 @@ func (op RenameOp) Run(_, stderr io.Writer) error {
 	if err := kc.Save(); err != nil {
 		return fmt.Errorf("failed to save modified kubeconfig: %w", err)
 	}
-	_ = printer.Success(stderr, "Context %s renamed to %s.",
+	printer.Success(stderr, "Context %s renamed to %s.",
 		printer.SuccessColor.Sprint(op.Old),
 		printer.SuccessColor.Sprint(op.New))
 	return nil

cmd/kubectx/shell.go

@@ -35,17 +35,10 @@ func (op ShellOp) Run(_, stderr io.Writer) error {
 	if err := kc.Parse(); err != nil {
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
-	exists, err := kc.ContextExists(op.Target)
-	if err != nil {
-		return fmt.Errorf("failed to check context: %w", err)
-	}
-	if !exists {
+	if !kc.ContextExists(op.Target) {
 		return fmt.Errorf("no context exists with the name: \"%s\"", op.Target)
 	}
-	previousCtx, err := kc.GetCurrentContext()
-	if err != nil {
-		return fmt.Errorf("failed to get current context: %w", err)
-	}
+	previousCtx := kc.GetCurrentContext()
 
 	// Extract minimal kubeconfig using kubectl
 	data, err := extractMinimalKubeconfig(kubectlPath, op.Target)

cmd/kubectx/switch.go

@@ -61,15 +61,8 @@ func switchContext(name string) (string, error) {
 		return "", fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	prev, err := kc.GetCurrentContext()
-	if err != nil {
-		return "", fmt.Errorf("failed to get current context: %w", err)
-	}
-	exists, err := kc.ContextExists(name)
-	if err != nil {
-		return "", fmt.Errorf("failed to check context: %w", err)
-	}
-	if !exists {
+	prev := kc.GetCurrentContext()
+	if !kc.ContextExists(name) {
 		return "", fmt.Errorf("no context exists with the name: \"%s\"", name)
 	}
 	if err := kc.ModifyCurrentContext(name); err != nil {

cmd/kubens/current.go

@@ -31,10 +31,7 @@ func (c CurrentOp) Run(stdout, _ io.Writer) error {
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	ctx, err := kc.GetCurrentContext()
-	if err != nil {
-		return fmt.Errorf("failed to get current context: %w", err)
-	}
+	ctx := kc.GetCurrentContext()
 	if ctx == "" {
 		return errors.New("current-context is not set")
 	}

cmd/kubens/fzf.go

@@ -37,7 +37,6 @@ type InteractiveSwitchOp struct {
 func (op InteractiveSwitchOp) Run(_, stderr io.Writer) error {
 	// parse kubeconfig just to see if it can be loaded
 	kc := new(kubeconfig.Kubeconfig).WithLoader(kubeconfig.DefaultLoader)
-	defer kc.Close()
 	if err := kc.Parse(); err != nil {
 		if cmdutil.IsNotFoundErr(err) {
 			printer.Warning(stderr, "kubeconfig file not found")
@@ -45,6 +44,7 @@ func (op InteractiveSwitchOp) Run(_, stderr io.Writer) error {
 		}
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
+	defer kc.Close()
 
 	cmd := exec.Command("fzf", "--ansi", "--no-preview")
 	var out bytes.Buffer
@@ -69,6 +69,6 @@ func (op InteractiveSwitchOp) Run(_, stderr io.Writer) error {
 	if err != nil {
 		return fmt.Errorf("failed to switch namespace: %w", err)
 	}
-	_ = printer.Success(stderr, "Active namespace is \"%s\".", printer.SuccessColor.Sprint(name))
+	printer.Success(stderr, "Active namespace is \"%s\".", printer.SuccessColor.Sprint(name))
 	return nil
 }

cmd/kubens/list.go

@@ -20,7 +20,6 @@ import (
 	"fmt"
 	"io"
 	"os"
-	"slices"
 
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/client-go/kubernetes"
@@ -40,10 +39,7 @@ func (op ListOp) Run(stdout, stderr io.Writer) error {
 		return fmt.Errorf("kubeconfig error: %w", err)
 	}
 
-	ctx, err := kc.GetCurrentContext()
-	if err != nil {
-		return fmt.Errorf("failed to get current context: %w", err)
-	}
+	ctx := kc.GetCurrentContext()
 	if ctx == "" {
 		return errors.New("current-context is not set")
 	}
@@ -90,7 +86,6 @@ func queryNamespaces(kc *kubeconfig.Kubeconfig) ([]string, error) {
 			return nil, fmt.Errorf("failed to list namespaces from k8s API: %w", err)
 		}
 		next = list.Continue
-		out = slices.Grow(out, len(list.Items))
 		for _, it := range list.Items {
 			out = append(out, it.Name)
 		}

cmd/kubens/switch.go

@@ -49,10 +49,7 @@ func (s SwitchOp) Run(_, stderr io.Writer) error {
 }
 
 func switchNamespace(kc *kubeconfig.Kubeconfig, ns string, force bool) (string, error) {
-	ctx, err := kc.GetCurrentContext()
-	if err != nil {
-		return "", fmt.Errorf("failed to get current context: %w", err)
-	}
+	ctx := kc.GetCurrentContext()
 	if ctx == "" {
 		return "", errors.New("current-context is not set")
 	}

cmd/kubens/unset.go

@@ -42,10 +42,7 @@ func (_ UnsetOp) Run(_, stderr io.Writer) error {
 }
 
 func clearNamespace(kc *kubeconfig.Kubeconfig) (string, error) {
-	ctx, err := kc.GetCurrentContext()
-	if err != nil {
-		return "", fmt.Errorf("failed to get current context: %w", err)
-	}
+	ctx := kc.GetCurrentContext()
 	ns := "default"
 	if ctx == "" {
 		return "", errors.New("current-context is not set")

internal/kubeconfig/contexts.go

@@ -50,25 +50,18 @@ func (k *Kubeconfig) contextNode(name string) (*yaml.RNode, error) {
 	return context, nil
 }
 
-func (k *Kubeconfig) ContextNames() ([]string, error) {
+func (k *Kubeconfig) ContextNames() []string {
 	contexts, err := k.config.Pipe(yaml.Get("contexts"))
 	if err != nil {
-		return nil, fmt.Errorf("failed to get contexts: %w", err)
-	}
-	if contexts == nil {
-		return nil, nil
+		return nil
 	}
 	names, err := contexts.ElementValues("name")
 	if err != nil {
-		return nil, fmt.Errorf("failed to get context names: %w", err)
+		return nil
 	}
-	return names, nil
+	return names
 }
 
-func (k *Kubeconfig) ContextExists(name string) (bool, error) {
-	names, err := k.ContextNames()
-	if err != nil {
-		return false, err
-	}
-	return slices.Contains(names, name), nil
+func (k *Kubeconfig) ContextExists(name string) bool {
+	return slices.Contains(k.ContextNames(), name)
 }

internal/kubeconfig/contexts_test.go

@@ -33,10 +33,7 @@ func TestKubeconfig_ContextNames(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	ctx, err := kc.ContextNames()
-	if err != nil {
-		t.Fatal(err)
-	}
+	ctx := kc.ContextNames()
 	expected := []string{"abc", "def", "ghi"}
 	if diff := cmp.Diff(expected, ctx); diff != "" {
 		t.Fatalf("%s", diff)
@@ -49,10 +46,7 @@ func TestKubeconfig_ContextNames_noContextsEntry(t *testing.T) {
 	if err := kc.Parse(); err != nil {
 		t.Fatal(err)
 	}
-	ctx, err := kc.ContextNames()
-	if err != nil {
-		t.Fatal(err)
-	}
+	ctx := kc.ContextNames()
 	var expected []string = nil
 	if diff := cmp.Diff(expected, ctx); diff != "" {
 		t.Fatalf("%s", diff)
@@ -65,9 +59,10 @@ func TestKubeconfig_ContextNames_nonArrayContextsEntry(t *testing.T) {
 	if err := kc.Parse(); err != nil {
 		t.Fatal(err)
 	}
-	_, err := kc.ContextNames()
-	if err == nil {
-		t.Fatal("expected error for non-array contexts entry")
+	ctx := kc.ContextNames()
+	var expected []string = nil
+	if diff := cmp.Diff(expected, ctx); diff != "" {
+		t.Fatalf("%s", diff)
 	}
 }
 
@@ -82,15 +77,13 @@ func TestKubeconfig_CheckContextExists(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	if exists, err := kc.ContextExists("c1"); err != nil || !exists {
+	if !kc.ContextExists("c1") {
 		t.Fatal("c1 actually exists; reported false")
 	}
-	if exists, err := kc.ContextExists("c2"); err != nil || !exists {
+	if !kc.ContextExists("c2") {
 		t.Fatal("c2 actually exists; reported false")
 	}
-	if exists, err := kc.ContextExists("c3"); err != nil {
-		t.Fatal(err)
-	} else if exists {
+	if kc.ContextExists("c3") {
 		t.Fatal("c3 does not exist; but reported true")
 	}
 }

internal/kubeconfig/currentcontext.go

@@ -15,19 +15,17 @@
 package kubeconfig
 
 import (
-	"fmt"
-
 	"sigs.k8s.io/kustomize/kyaml/yaml"
 )
 
 // GetCurrentContext returns "current-context" value in given
-// kubeconfig object Node, or returns ("", nil) if not found.
-func (k *Kubeconfig) GetCurrentContext() (string, error) {
+// kubeconfig object Node, or returns "" if not found.
+func (k *Kubeconfig) GetCurrentContext() string {
 	v, err := k.config.Pipe(yaml.Get("current-context"))
 	if err != nil {
-		return "", fmt.Errorf("failed to read current-context: %w", err)
+		return ""
 	}
-	return yaml.GetValue(v), nil
+	return yaml.GetValue(v)
 }
 
 func (k *Kubeconfig) UnsetCurrentContext() error {

internal/kubeconfig/currentcontext_test.go

@@ -26,10 +26,7 @@ func TestKubeconfig_GetCurrentContext(t *testing.T) {
 	if err := kc.Parse(); err != nil {
 		t.Fatal(err)
 	}
-	v, err := kc.GetCurrentContext()
-	if err != nil {
-		t.Fatal(err)
-	}
+	v := kc.GetCurrentContext()
 
 	expected := "foo"
 	if v != expected {
@@ -43,10 +40,7 @@ func TestKubeconfig_GetCurrentContext_missingField(t *testing.T) {
 	if err := kc.Parse(); err != nil {
 		t.Fatal(err)
 	}
-	v, err := kc.GetCurrentContext()
-	if err != nil {
-		t.Fatal(err)
-	}
+	v := kc.GetCurrentContext()
 
 	expected := ""
 	if v != expected {

internal/kubeconfig/kubeconfig.go

@@ -87,8 +87,5 @@ func (k *Kubeconfig) Save() error {
 	}
 	enc := yaml.NewEncoder(k.f)
 	enc.SetIndent(0)
-	if err := enc.Encode(k.config.YNode()); err != nil {
-		return err
-	}
-	return enc.Close()
+	return enc.Encode(k.config.YNode())
 }