Feature 131/remove paso tutorial #138

noe.trevino · 2026-02-24T23:30:50-06:00

noe.trevino commented

2026-02-24 23:30:50 -06:00

Checklist

Read CONTRIBUTING.md
Ran tests (go test ./... -race) locally
Formatted and linted with gofmt -w . and golangci-lint run ./... respectively
Added tests if necessary (significant changes/complex logic)
Updated documentation if applicable
Updated README.md if applicable

Summary

Closes: #131

## Checklist - [x] Read `CONTRIBUTING.md` - [x] Ran tests (`go test ./... -race`) locally - [x] Formatted and linted with `gofmt -w .` and `golangci-lint run ./...` respectively - [x] Added tests if necessary (significant changes/complex logic) - [x] Updated documentation if applicable - [x] Updated `README.md` if applicable ## Summary  Closes: #131

github-actions[bot] commented

2026-02-24 23:31:02 -06:00

(Migrated from github.com)

Informational Warning:

The snapshot tests have been changed.
Please add a screenshot showing what has been changed, and a description of why it was changed.

If you do not provide an explanation and screenshot in 3 days, this PR will be closed.

@TheNoeTrevino

Changed snapshot files (5 files)

internal/testing/snapshots/cli/testdata/help/paso-setup-claude.golden
internal/testing/snapshots/cli/testdata/help/paso-setup-opencode.golden
internal/testing/snapshots/cli/testdata/help/paso-setup.golden
internal/testing/snapshots/cli/testdata/help/paso-tutorial.golden
internal/testing/snapshots/cli/testdata/help/paso.golden

**Informational Warning:** The snapshot tests have been changed. Please add a screenshot showing what has been changed, and a description of why it was changed. If you do not provide an explanation and screenshot in 3 days, this PR will be closed. @TheNoeTrevino <details> <summary>Changed snapshot files (5 files)</summary> - `internal/testing/snapshots/cli/testdata/help/paso-setup-claude.golden` - `internal/testing/snapshots/cli/testdata/help/paso-setup-opencode.golden` - `internal/testing/snapshots/cli/testdata/help/paso-setup.golden` - `internal/testing/snapshots/cli/testdata/help/paso-tutorial.golden` - `internal/testing/snapshots/cli/testdata/help/paso.golden` </details>

noe.trevino commented

2026-02-24 23:33:51 -06:00

added the setup command

<img width="997" height="984" alt="image" src="https://github.com/user-attachments/assets/733c5c3f-7772-49c5-97e7-c13fbac50524" /> added the setup command

claude[bot] (Migrated from github.com) reviewed

2026-02-24 23:34:57 -06:00

internal/cli/setup/setup.go

					
				@ -20,0 +213,4 @@

									return nil

								}

								return cli.NewExitErr(cli.ExitError, fmt.Sprintf("prompt error: %v", err))

							}

huh.ErrUserAborted is not handled here. runInteractive (line 168) returns nil on abort, but this confirmation prompt returns a non-zero exit error when the user presses Ctrl+C or Escape. Consistent behavior requires the same guard:

			if err := confirm.Run(); err != nil {
				if errors.Is(err, huh.ErrUserAborted) {
					return nil
				}
				return cli.NewExitErr(cli.ExitError, fmt.Sprintf("prompt error: %v", err))
			}

`huh.ErrUserAborted` is not handled here. `runInteractive` (line 168) returns `nil` on abort, but this confirmation prompt returns a non-zero exit error when the user presses Ctrl+C or Escape. Consistent behavior requires the same guard: ```suggestion if err := confirm.Run(); err != nil { if errors.Is(err, huh.ErrUserAborted) { return nil } return cli.NewExitErr(cli.ExitError, fmt.Sprintf("prompt error: %v", err)) } ```

Sign in to join this conversation.

No reviewers