Adds harbor context command

[Darkhood148]

Related Issue:

#417

Description:

Adds harbor context command to manage and switch context by reading/writing from confg.yaml

[Darkhood148]

[Copilot]

Pull Request Overview

This PR adds a new harbor context command to manage and switch contexts by reading/writing from a configuration file.

• Introduces a new command (harbor context list) for listing contexts
• Adds a new view for rendering the context list
• Creates additional API types to support the context functionality

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
pkg/views/context/list/view.go	Adds a view to render the list of contexts using a table
pkg/api/types.go	Introduces the ContextListView type
cmd/harbor/root/context/list.go	Implements the cobra command for listing contexts
cmd/harbor/root/context/cmd.go	Adds a context command to the overall CLI command hierarchy
cmd/harbor/root/cmd.go	Integrates the new context command into the root command collection

Comments suppressed due to low confidence (1)

cmd/harbor/root/context/list.go:38

• [nitpick] Consider using named field initialization for ContextListView for improved clarity and maintainability (e.g., ContextListView{Name: cred.Name, Username: cred.Username, Server: cred.ServerAddress}).

cx := api.ContextListView{cred.Name, cred.Username, cred.ServerAddress}

[Darkhood148]

@Vad1mo Should I continue working on this and add harbor context switch command?

[Darkhood148]

Hi @bupd
I have rename config to context as mentioned here: #438 . I have also improved the working of the list subcommand, giving it a neater ui. Can you review it please?

[bupd]

@Darkhood148 rebase onto main branch

[bupd]

@Darkhood148 Thanks for your contributions

I have added some suggestion for improvements

Also rebase & run dagger call lint. and fix the lint issues

Thanks

[Darkhood148]

@bupd Done the changes.

[bupd]

Thanks for your contributions

I have added suggestions please fix that.

[Darkhood148]

Done with the changes @bupd !
The test was failing due to bubbletea opening a TTY by default which fails in non-interactive environments like Dagger and Github Workflows. As mentioned in the documentation, this can be disabled by adding:

if !term.IsTerminal(int(os.Stdout.Fd())) {
     opts = append(opts, tea.WithoutRenderer(), tea.WithInput(nil))
}

After adding this, the same test passes!
This could be useful while writing tests for other commands using bubbletea.

[Vad1mo]

It would make sense if the context listsub-command would display what is the current context.

Eg. with an Asterix

[Vad1mo]

context switch should allow switching context, just like currently harbor-cli login --name contextallows us to do so.

Furthermore, we should only have one way to switch the context, so we should remove harbor-cli login --name context

[Darkhood148]

[Vad1mo]

Rename CurrentCredentialName: into CurrentContext

[Vad1mo]

Mybe provide an output after context switch e..g switched context from x to y.

❯ ./harbor-cli context switch vadim2@goharbor-io
""

[Vad1mo]

Display the current context in the UI

[Vad1mo]

It would make more sense if the order of the list would not change and make the current context on the top: