Skip to content

feat(workspaces): Add set command to switch default workspace#10

Merged
pthurlow merged 1 commit intomainfrom
feat/workspace-set
Mar 23, 2026
Merged

feat(workspaces): Add set command to switch default workspace#10
pthurlow merged 1 commit intomainfrom
feat/workspace-set

Conversation

@pthurlow
Copy link
Collaborator

No description provided.

}
};

let default_id = profile_config.workspaces.first().map(|w| w.public_id.as_str()).unwrap_or("").to_string();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: list could use the new load_client and fetch_all_workspaces helpers introduced in this same PR, which would eliminate the duplicated auth/client setup here (lines 107–138). The only extra piece list needs that load_client doesn't return is default_id from the profile's workspace list — you could either return the full ProfileConfig from load_client, or call config::load("default") once and build the client from that.

@pthurlow pthurlow merged commit 29aca6d into main Mar 23, 2026
7 checks passed
@pthurlow pthurlow deleted the feat/workspace-set branch March 23, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant