Skip to content

chore: bump gateway-cli version 0.1.2#1031

Merged
slavastartsev merged 1 commit intomasterfrom
chore/bump-gateway-cli-0.1.2
Apr 7, 2026
Merged

chore: bump gateway-cli version 0.1.2#1031
slavastartsev merged 1 commit intomasterfrom
chore/bump-gateway-cli-0.1.2

Conversation

@slavastartsev
Copy link
Copy Markdown
Contributor

No description provided.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bob-docs Ready Ready Preview, Comment Apr 7, 2026 3:14pm

Request Review

@slavastartsev slavastartsev merged commit 9d80805 into master Apr 7, 2026
6 checks passed
@slavastartsev slavastartsev deleted the chore/bump-gateway-cli-0.1.2 branch April 7, 2026 15:15
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the @gobob/gateway-cli version to 0.1.2 and bumps the @gobob/bob-sdk dependency to 5.3.2. A critical issue was identified regarding the SDK update, which incorrectly introduces linting-related packages into the production dependency graph and references non-existent versions (e.g., globals@17.4.0), which will likely cause installation failures for external users.

},
"dependencies": {
"@gobob/bob-sdk": "^5.3.1",
"@gobob/bob-sdk": "^5.3.2",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

high

The update to @gobob/bob-sdk@5.3.2 introduces several linting-related packages (@eslint/eslintrc, @eslint/js, and globals) into the production dependency graph. These are incorrectly listed as dependencies in the SDK's package.json instead of devDependencies. Furthermore, the version 17.4.0 for globals (and 10.0.1 for @eslint/js seen in the lockfile) appears to be non-standard or hallucinated, as these versions do not exist on the public npm registry. This will likely cause installation failures for external users and unnecessarily bloats the production environment. Please verify the SDK configuration before proceeding with this bump.

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