Skip to content

Add workspace param support#68

Merged
lelia merged 9 commits intomainfrom
lelia/add-workspace-support
Feb 26, 2026
Merged

Add workspace param support#68
lelia merged 9 commits intomainfrom
lelia/add-workspace-support

Conversation

@lelia
Copy link
Contributor

@lelia lelia commented Feb 24, 2026

Changes

  • In order to add workspace support to the Socket Python CLI, the Socket SDK for Python must first be updated to add the parameter to the FullScanParams dataclass.
  • Due to some recent hatch and virtualenv compatibility issues, some GitHub workflow files (pr-preview.yml and release.yml) also need to be updated to unblock PR checks.

Testing

  • Added unit test test_fullscans_post_with_workspace_unit asserting workspace=grofers appears in the outgoing request URL when set on FullScanParams
  • Validated end-to-end in combination with the companion CLI PR: workspace=grofers confirmed present in the raw HTTP request to api.socket.dev via mitmproxy, and a full scan was successfully created against the live API

refs: SocketDev/socket-python-cli#164

Signed-off-by: lelia <lelia@socket.dev>
…ScanParams

Signed-off-by: lelia <lelia@socket.dev>
@lelia lelia requested a review from a team as a code owner February 24, 2026 22:37
@lelia lelia changed the title Lelia/add workspace support Add workspace param support Feb 24, 2026
@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 socketdev==3.0.31.dev1

Signed-off-by: lelia <lelia@socket.dev>
Signed-off-by: lelia <lelia@socket.dev>
@dacoburn dacoburn removed the request for review from a team February 25, 2026 02:53
Signed-off-by: lelia <lelia@socket.dev>
Signed-off-by: lelia <lelia@socket.dev>
Signed-off-by: lelia <lelia@socket.dev>
Signed-off-by: lelia <lelia@socket.dev>
@lelia lelia merged commit ec8940a into main Feb 26, 2026
6 checks passed
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