Skip to content

Add workspace flag to CLI args#164

Open
lelia wants to merge 5 commits intomainfrom
lelia/add-workspace-support
Open

Add workspace flag to CLI args#164
lelia wants to merge 5 commits intomainfrom
lelia/add-workspace-support

Conversation

@lelia
Copy link

@lelia lelia commented Feb 24, 2026

🚨 IMPORTANT: This change will not work without the associated Python SDK PR: SocketDev/socket-sdk-python#68. Please ensure the SDK PR is merged in before this PR is prepared for release.

Changes

  • Added --workspace <string> flag to support creating full scans with the Python CLI.
  • Updated README.md to document the new flag, as well as explain the difference between workspace and workspace-name to avoid confusion

Testing

  • Added 3 unit tests to test_cli_config.py: flag parses correctly, defaults to None when omitted, and is independent of --workspace-name
  • Tested the CLI locally using my locally-modified copy of the SDK via make first-time-local-setup
  • Also used mitmproxy to confirm that &workspace is now included in the HTTP requests the Socket API

Public Changelog

Add support for --workspace flag when creating scans

refs: SocketDev/socket-sdk-python#68

Signed-off-by: lelia <lelia@socket.dev>
Signed-off-by: lelia <lelia@socket.dev>
…sting workspace-name flag

Signed-off-by: lelia <lelia@socket.dev>
Signed-off-by: lelia <lelia@socket.dev>
@lelia lelia requested a review from a team as a code owner February 24, 2026 23:13
@lelia lelia requested review from dacoburn and removed request for a team February 24, 2026 23:14
@github-actions
Copy link

github-actions bot commented Feb 24, 2026

🚀 Preview package published!

Install with:

pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple socketsecurity==2.2.73.dev2

Docker image: socketdev/cli:pr-164

Signed-off-by: lelia <lelia@socket.dev>
repo_group.add_argument(
"--workspace",
metavar="<string>",
help="The workspace of the repository to associate the full-scan with.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Lets do the "The workspace in the Socket Organization that the repository is in to associate with the full scan" or something along those lines.

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.

2 participants