GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — feat(user-agent): rename the client to socket-python-cli by jdalton · Pull Request #320 · SocketDev/socket-python-cli

Conversation

@jdalton

@jdalton John-David Dalton (jdalton) commented Aug 27, 2026 •
edited by lelia
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Use one lowercase socket-python-cli User-Agent name across direct API requests and requests the CLI sends through the Python SDK, so attribution stays consistent. Pin the published Python SDK (socketdev) to 3.6.2, which rejects retired API credentials before use.

The backend accepts both SocketPythonCLI/ and socket-python-cli/ as py-cli, and that support is deployed. This CLI rename can ship independently of the later standalone SDK/Basics attribution work.

Both changes are included in 2.11.11. The changelog and package-constant merge conflicts are resolved, and the existing Coana 15.14.1 release notes are preserved.

Validation

  • All 733 unit and core tests passed in an isolated environment installed from uv.lock, using the published SDK 3.6.2. Prepared-request regression coverage verifies the lowercase CLI identity on both direct and SDK calls.
  • Runtime version, pyproject.toml, and uv.lock all agree on 2.11.11.
  • uv lock --locked --offline, repository-wide Ruff, the CI import smoke check, and git diff --check passed.
  • The Docker release dependency export selects socketdev==3.6.2 with hashes. The lockfile update changes only the SDK artifacts and the project's SDK requirement; other package versions are preserved.
  • The fourth-wall audit found no added code comments or docstrings to revise.

Release readiness

SDK 3.6.2 is published and pinned; no SDK version selection remains outstanding. Backend compatibility is already deployed. Merge after the required review approval and CI checks for the latest commit pass, then tag v2.11.11 for release.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@jdalton
John-David Dalton (jdalton) requested a review from a team as a code owner August 27, 2026 20:04
@lelia lelia self-assigned this Aug 28, 2026
lelia
lelia previously requested changes Sep 2, 2026
View reviewed changes

@lelia lelia left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

John-David Dalton (@jdalton) thanks for adding this, but I think we need to implement backend changes first before we rename on the client side.

I confirmed that backend attribution currently matches user-agents on exact, case sensitive prefixes, and currently only SocketPythonCLI/ is registered, meaning that this rename will cause us to silently lose Python CLI attribution as users upgrade.

By the way, the same class of change needs to take place on the SDK side, too - in fact, the SocketSDKPython/ user-agent appears to have never been registered on the backend. We may as well fix the gap everywhere in one go, if you're up for it.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@jdalton
John-David Dalton (jdalton) force-pushed the ua/lowercase-client-name branch from d79c264 to a4cc539 Compare October 8, 2026 15:45
@jdalton

This comment was marked as outdated.

Sign in to view
@jdalton
John-David Dalton (jdalton) temporarily deployed to socket-firewall October 12, 2026 05:27 — with GitHub Actions Inactive
@jdalton
John-David Dalton (jdalton) force-pushed the ua/lowercase-client-name branch from c77895c to a2e8bc6 Compare October 12, 2026 05:39
@jdalton
John-David Dalton (jdalton) temporarily deployed to socket-firewall October 12, 2026 05:39 — with GitHub Actions Inactive
@jdalton
John-David Dalton (jdalton) requested a review from lelia October 12, 2026 05:43
@jdalton
feat(user-agent): rename client to socket-python-cli
ffcc07b 
Use one client name for CLI and SDK requests. Preserve the current release version and record the change under Unreleased.
@jdalton
John-David Dalton (jdalton) force-pushed the ua/lowercase-client-name branch from a2e8bc6 to ffcc07b Compare October 29, 2026 15:37
@jdalton

John-David Dalton (jdalton) commented Oct 29, 2026

Copy link
Copy Markdown
Contributor Author

The backend now recognizes both CLI prefixes and SocketSDKPython/, with regression coverage. I refreshed this branch onto main; all 582 unit tests pass. Please take another look. Production deployment still needs confirmation before release.
lelia reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@jdalton
John-David Dalton (jdalton) dismissed lelia’s stale review October 29, 2026 15:51

Backend support for both CLI names and the SDK name has landed. The branch is refreshed and CI passes. Requesting a fresh review.

@jdalton
John-David Dalton (jdalton) requested review from lelia and removed request for lelia October 29, 2026 15:52
This was referenced Nov 3, 2026
fix(user-agent): identify the Python SDK consistently SocketDev/socket-sdk-python#127
Draft
fix(user-agent): identify Basics API requests SocketDev/socket-basics#133
Draft
@lelia
fix(user-agent): refresh CLI for 2.11.11
820ea6f 
Merge current main, preserve the 2.11.11 Coana release notes,
and use the lowercase CLI identity for direct and SDK calls.
Verify both request paths at the HTTP transport boundary.

Assisted-by: Codex:GPT-6
lelia
lelia reviewed Nov 3, 2026
View reviewed changes

@lelia lelia left a comment •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

[agent] Refreshed this PR onto current main and resolved the merge conflicts. The User-Agent change is included in 2.11.11; the runtime version, pyproject.toml, and uv.lock all agree, and the existing Coana release notes are preserved. All 584 unit tests passed, including prepared-request coverage for both direct API calls and SDK calls. Ruff and the locked dependency check also passed.

Backend deployment evidence: I verified that the depscan change accepting both SocketPythonCLI/ and socket-python-cli/ as py-cli was confirmed deployed. The CLI rename therefore has no dependency on the upcoming SDK/Basics attribution rollout (TBD).

The SDK dependency bump remains pending selection of the target version; this PR currently retains socketdev==3.6.1. After that update, rerun CI before tagging v2.11.11.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
lelia
lelia approved these changes Nov 3, 2026
View reviewed changes
@lelia
fix(deps): use Python SDK 3.6.2
390dba8 
Pin the published SDK and refresh its locked artifacts.
Include the upgrade in the 2.11.11 release notes.

Assisted-by: Codex:GPT-6
@lelia
lelia temporarily deployed to socket-firewall November 3, 2026 23:23 — with GitHub Actions Inactive
@socket-security

socket-security Bot commented Nov 3, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updatedpypi/​socketdev@​3.6.1 ⏵ 3.6.298 +1100100100100

View full report
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@socket-security-staging

socket-security-staging Bot commented Nov 3, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updatedpypi/​socketdev@​3.6.1 ⏵ 3.6.298 +1100100100100

View full report
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@lelia
lelia merged commit 911331c into main Nov 3, 2026
32 checks passed

Uh oh!

There was an error while loading. Please reload this page.

This branch was previously deployed

1 inactive deployment
socket-firewall — 390dba83 Deployed Nov 3, 2026 by lelia via python-sfw-smoke-enterprise #549
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lelia lelia lelia approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@lelia lelia

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@jdalton @lelia
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.