Skip to content

feat: Console SDK update for version 3.2.0#64

Closed
premtsd-code wants to merge 2 commits intomainfrom
dev
Closed

feat: Console SDK update for version 3.2.0#64
premtsd-code wants to merge 2 commits intomainfrom
dev

Conversation

@premtsd-code
Copy link
Copy Markdown
Contributor

@premtsd-code premtsd-code commented Feb 23, 2026

This PR contains updates to the Console SDK for version 3.2.0.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1fb3834 and 9e27c04.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • src/client.ts
✅ Files skipped from review due to trivial changes (1)
  • src/client.ts
🚧 Files skipped from review as they are similar to previous changes (1)
  • CHANGELOG.md

Walkthrough

This pull request bumps the SDK version to 3.2.0 (package.json, README CDN snippet, CHANGELOG) and updates the client header x-sdk-version to 3.2.0. It adds three new Deno runtime entries (deno-1.21, deno-1.24, deno-1.35) to src/enums/build-runtime.ts, src/enums/runtime.ts, and src/enums/runtimes.ts. No other logic, control-flow, or public API removals were introduced.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: updating the SDK to version 3.2.0, which is evident from version bumps across package.json, src/client.ts, CHANGELOG.md, and README.md.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@CHANGELOG.md`:
- Around line 113-198: Remove the stray changelog block that was copied from
another SDK: delete everything starting at the header "## 22.3.1" through the
end of the file (the block that includes version headings down to "## 17.0.0").
Ensure only the legitimate changelog entries for this SDK (up to its actual
latest version, e.g., 1.5.0) remain and verify there are no leftover lines from
the "22.x.x"–"17.x.x" block.
- Around line 3-6: The changelog entry contains internal CI context ("for the CI
failure in sdk-generator"); remove that note and rewrite the entry to be
consumer-facing—e.g., state that the SDK was regenerated to version 1.8.x and
that the Health and Projects services now include the new methods
getConsolePausing, updateConsoleAccess, and updateStatus—so the entry describes
what changed for SDK consumers without mentioning CI or internal tooling.

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 40d238d and 1fb3834.

📒 Files selected for processing (7)
  • CHANGELOG.md
  • README.md
  • package.json
  • src/client.ts
  • src/enums/build-runtime.ts
  • src/enums/runtime.ts
  • src/enums/runtimes.ts

Comment on lines +3 to +6
## 3.2.0

* Regenerate SDK with version 1.8.x to add `getConsolePausing`, `updateConsoleAccess` and `updateStatus` to `Health` and `Projects` service for the CI failure in sdk-generator

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Internal CI context doesn't belong in a user-facing changelog entry.

"for the CI failure in sdk-generator" is an internal engineering note. The changelog entry should describe what the change means to SDK consumers, not why it was triggered internally.

✏️ Suggested rewrite
-* Regenerate SDK with version 1.8.x to add `getConsolePausing`, `updateConsoleAccess` and `updateStatus` to `Health` and `Projects` service for the CI failure in sdk-generator
+* Add `getConsolePausing`, `updateConsoleAccess`, and `updateStatus` methods to `Health` and `Projects` services
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
## 3.2.0
* Regenerate SDK with version 1.8.x to add `getConsolePausing`, `updateConsoleAccess` and `updateStatus` to `Health` and `Projects` service for the CI failure in sdk-generator
## 3.2.0
* Add `getConsolePausing`, `updateConsoleAccess`, and `updateStatus` methods to `Health` and `Projects` services
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@CHANGELOG.md` around lines 3 - 6, The changelog entry contains internal CI
context ("for the CI failure in sdk-generator"); remove that note and rewrite
the entry to be consumer-facing—e.g., state that the SDK was regenerated to
version 1.8.x and that the Health and Projects services now include the new
methods getConsolePausing, updateConsoleAccess, and updateStatus—so the entry
describes what changed for SDK consumers without mentioning CI or internal
tooling.

@premtsd-code premtsd-code deleted the dev branch February 24, 2026 11:34
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