feat: add wake-up command and curl workspace #152

OliverTrautvetter · 2026-01-21T15:45:58Z

Adds the ability to wake up a workspace using the CLI.

Clickup

Adds the ability to send HTTP requests to my workspace's dev domain, so that I can test connectivity.

Clickup

…e tests

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

…oud/cs-go into wake_up_workspaces

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

… devDomain

…ce URL construction

…oud/cs-go into wake_up_workspaces

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

Copilot

Pull request overview

This PR adds CLI support for waking up on-demand workspaces and sending authenticated HTTP requests to a workspace’s dev domain, plus corresponding client APIs, tests, and documentation updates.

Changes:

Introduces a cs wake-up command with API support (ScaleWorkspace) and both unit and integration tests to start an on-demand workspace by scaling replicas and waiting until it is running.
Adds a cs curl command that constructs an authenticated URL to a workspace’s dev domain and shells out to curl, including comprehensive unit and integration tests and new documentation.
Updates client interfaces, mocks, and workflows (integration test cleanup and docs) to support the new commands and to slightly improve workspace readiness polling behavior.

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
`int/util/workspace.go`	Adds a helper to call the new `ScaleWorkspace` API from integration utilities.
`int/integration_test.go`	Adds integration suites for `wake-up` and `curl` commands, and extends generic error-handling coverage to the new commands.
`docs/cs_wake-up.md`	Documents the new `cs wake-up` command, including usage, examples, and options.
`docs/cs_list_workspaces.md`	Corrects the list-workspaces example to use the `-t` flag, aligning docs with the CLI.
`docs/cs_curl.md`	Documents the new `cs curl` command, including path and curl-arg syntax, examples, and options.
`docs/cs.md`	Adds links for the new `curl` and `wake-up` subcommands in the main `cs` docs index.
`docs/README.md`	Mirrors the `cs.md` updates for the CLI command index in the docs README.
`cli/cmd/wakeup_test.go`	Introduces unit tests for the `WakeUpCmd.WakeUpWorkspace` logic, covering normal, already-running, and error cases.
`cli/cmd/wakeup.go`	Implements the `cs wake-up` command, wiring it into the root command and using the API client to scale and wait for workspace readiness.
`cli/cmd/root.go`	Registers the new `wake-up` and `curl` subcommands with the root CLI.
`cli/cmd/mocks.go`	Extends the CLI `MockClient` with `ScaleWorkspace`, `WaitForWorkspaceRunning`, and `WorkspaceStatus` methods to support new command tests.
`cli/cmd/list_workspaces.go`	Updates the example string to use `-t <team-id>` for listing workspaces.
`cli/cmd/curl_test.go`	Adds unit tests for `CurlCmd.CurlWorkspace`, covering URL construction, dev-domain absence, and workspace lookup errors.
`cli/cmd/curl.go`	Implements the `cs curl` command, building an authenticated URL from the workspace dev domain and delegating the HTTP request to `curl`.
`cli/cmd/client.go`	Extends the `Client` interface and `NewClient` to expose `WorkspaceStatus`, `WaitForWorkspaceRunning`, and `ScaleWorkspace` to CLI commands.
`api/workspace.go`	Adds the `ScaleWorkspace` API wrapper and adjusts `WaitForWorkspaceRunning` to retry on transient errors until timeout.
`api/openapi_client/mocks.go`	Regenerates/updates openapi client mocks with simpler type assertions in `Run` helpers.
`.mockery.yml`	Adjusts mockery configuration for the openapi client package (removing `include-auto-generated`).
`.github/workflows/integration-test.yml`	Extends orphaned workspace cleanup to include the new `cli-wakeup-test-` and `cli-curl-test-` workspace name prefixes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

cli/cmd/curl_test.go

api/workspace.go

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…oud/cs-go into wake_up_workspaces

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

siherrmann

A few comments, please revisit.

siherrmann · 2026-01-30T18:37:16Z

cli/cmd/curl.go

+	curl.Port = curl.cmd.Flags().IntP("port", "p", 3000, "Port to connect to")
+	curl.Timeout = curl.cmd.Flags().DurationP("timeout", "", 30*time.Second, "Timeout for the request")
+	curl.cmd.Flags().BoolVar(&curl.Insecure, "insecure", false, "skip TLS certificate verification (for testing only)")


Our nornal way is the last (so curl.cmd.Flags().BoolVar). Please update the other flags to the same structure (IntVar and DurationVar)

siherrmann · 2026-01-30T18:38:33Z

cli/cmd/curl.go

+	port := 3000
+	if c.Port != nil {
+		port = *c.Port
+	}


This can also be done purely with a flag that has a default, would be cleaner.

siherrmann · 2026-01-30T18:42:40Z

cli/cmd/curl.go

+		url = fmt.Sprintf("https://%s%s", devDomain, path)
+	}
+
+	fmt.Fprintf(os.Stderr, "Sending request to workspace %d (%s) at %s\n", wsId, workspace.Name, url)


Why not use log.Printf()?

siherrmann · 2026-01-30T18:43:17Z

cli/cmd/curl.go

+
+	fmt.Fprintf(os.Stderr, "Sending request to workspace %d (%s) at %s\n", wsId, workspace.Name, url)
+
+	timeout := 30 * time.Second


Again we could use a flag with a default, we should never have nil then.

Ah, they even have the default already, I think you could remove the nil check then.

siherrmann · 2026-01-30T18:45:24Z

cli/cmd/curl.go

+	cmdArgs := []string{"curl"}
+
+	// Add authentication header
+	cmdArgs = append(cmdArgs, "-H", fmt.Sprintf("x-forward-security: %s", token))


I think it's fair to just do:
cmdArgs := []string{"curl", "-H", fmt.Sprintf("x-forward-security: %s", token)}

Or even:
cmdArgs := []string{"curl", "-H", fmt.Sprintf("x-forward-security: %s", token), curlArgs...}
Then you can remove the // Add user's curl arguments

siherrmann · 2026-01-30T18:49:05Z

cli/cmd/curl.go

+	if err != nil {
+		if ctx.Err() == context.DeadlineExceeded {
+			return fmt.Errorf("timeout exceeded while requesting workspace %d", wsId)
+		}
+		return fmt.Errorf("curl command failed: %w", err)
+	}


I'm not sure but wouldn't be the err itself be context.DeadlineExceeded?
If so we could do:

if err != nil && err == context.DeadlineExceeded { ... } else if err != nil { ... }

Would be a little bit cleaner I think (but please verify, I'm not sure, error could be wrapped)

siherrmann · 2026-01-30T18:52:03Z

cli/cmd/wakeup.go

+	timeout := 120 * time.Second
+	if c.Timeout != nil {
+		timeout = *c.Timeout
+	}


Again, if default in flag is set this value should never be nil as far as I know.

OliverTrautvetter added 12 commits January 21, 2026 16:45

feat: add wake-up command to manage on-demand workspaces and integrat…

a16bb72

…e tests

chore(docs): Auto-update docs and licenses

38522d1

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

chore(docs): Auto-update docs and licenses

3b9e8a2

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

fix: logging and lint error

2eaf8ce

Merge branch 'wake_up_workspaces' of https://github.com/codesphere-cl…

2122bb7

…oud/cs-go into wake_up_workspaces

fix: tests

34e7c70

chore(docs): Auto-update docs and licenses

35a0e8f

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

chore(docs): Auto-update docs and licenses

acc7015

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

chore(docs): Auto-update docs and licenses

e704b92

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

chore(docs): Auto-update docs and licenses

18bee67

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

chore(docs): Auto-update docs and licenses

7f96b04

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

fix: Improve error handling in WaitForWorkspaceRunning function

165d31b

OliverTrautvetter changed the title ~~feat: add wake-up command to manage on-demand workspaces and integrate tests~~ feat: add wake-up command Jan 22, 2026

OliverTrautvetter added 2 commits January 22, 2026 10:56

Merge branch 'main' into wake_up_workspaces

c963cc1

fix: list workspace example

647ec6c

OliverTrautvetter changed the title ~~feat: add wake-up command~~ feat: add wake-up command and curl workspace Jan 22, 2026

OliverTrautvetter added 11 commits January 22, 2026 15:02

feat: add curl command

99ea553

chore(docs): Auto-update docs and licenses

b06abda

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

fix: enhance URL construction by stripping workspace-port prefix from…

51b37b5

… devDomain

feat: enhance workspace management by adding team retrieval and servi…

5e41b59

…ce URL construction

Merge branch 'main' into wake_up_workspaces

8300eb0

fix: wake up and curl

5dc44be

Merge branch 'wake_up_workspaces' of https://github.com/codesphere-cl…

9f9e691

…oud/cs-go into wake_up_workspaces

chore(docs): Auto-update docs and licenses

98a1b93

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

ref: cleanup

e874153

chore(docs): Auto-update docs and licenses

d03ba68

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

fix: integration test

12bdf29

OliverTrautvetter requested a review from Copilot January 28, 2026 13:18

Copilot started reviewing on behalf of OliverTrautvetter January 28, 2026 13:18 View session

Copilot AI reviewed Jan 28, 2026

View reviewed changes

cli/cmd/curl_test.go Outdated Show resolved Hide resolved

api/workspace.go Outdated Show resolved Hide resolved

OliverTrautvetter and others added 5 commits January 28, 2026 14:44

Update api/workspace.go

401eb63

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

ref: curl tests

ca57a7c

Merge branch 'wake_up_workspaces' of https://github.com/codesphere-cl…

bbbb9c4

…oud/cs-go into wake_up_workspaces

chore(docs): Auto-update docs and licenses

d499ebf

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>

Merge branch 'main' into wake_up_workspaces

db439c3

OliverTrautvetter requested a review from Copilot January 28, 2026 15:06

Copilot started reviewing on behalf of OliverTrautvetter January 28, 2026 15:07 View session

Copilot AI reviewed Jan 28, 2026

View reviewed changes

OliverTrautvetter marked this pull request as ready for review January 28, 2026 16:15

OliverTrautvetter requested a review from a team as a code owner January 28, 2026 16:15

OliverTrautvetter requested a review from siherrmann January 28, 2026 16:15

Merge branch 'main' into wake_up_workspaces

e917713

siherrmann reviewed Jan 30, 2026

View reviewed changes


		fmt.Fprintf(os.Stderr, "Sending request to workspace %d (%s) at %s\n", wsId, workspace.Name, url)

		timeout := 30 * time.Second

feat: add wake-up command and curl workspace #152

Are you sure you want to change the base?

feat: add wake-up command and curl workspace #152

Uh oh!

Conversation

OliverTrautvetter commented Jan 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

siherrmann left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

OliverTrautvetter commented Jan 21, 2026 •

edited

Loading