Skip to content

feat: Console SDK update for version 8.1.0#74

Merged
ChiragAgg5k merged 2 commits intomainfrom
dev
Mar 31, 2026
Merged

feat: Console SDK update for version 8.1.0#74
ChiragAgg5k merged 2 commits intomainfrom
dev

Conversation

@ChiragAgg5k
Copy link
Copy Markdown
Member

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

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

Warning

Rate limit exceeded

@ChiragAgg5k has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 16 minutes and 39 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 16 minutes and 39 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a9b4c80d-330a-454c-b245-78d024ebf16d

📥 Commits

Reviewing files that changed from the base of the PR and between c4178fc and 6d0d431.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (8)
  • .github/workflows/publish.yml
  • .gitignore
  • CHANGELOG.md
  • README.md
  • package.json
  • src/client.ts
  • src/models.ts
  • src/services/vcs.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • 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.

@greptile-apps
Copy link
Copy Markdown

greptile-apps bot commented Mar 31, 2026

Greptile Summary

This PR updates the Console SDK from 8.0.0 to 8.1.0, adding runtime detection support alongside the existing framework detection, exposing client headers, and hardening the CI pipeline.

Key changes:

  • getHeaders() on Client: Returns a shallow copy of the statically-configured header map. Uses the project-local Headers type (a plain string-keyed record), so there is no collision with the Web Fetch API's Headers class.
  • New models (ProviderRepositoryRuntime, ProviderRepositoryRuntimeList, DetectionRuntime): Parallel to the existing framework variants and consistent in style. The variables field in list-level models uses string[] (matching ProviderRepositoryFramework) while detection-level models use DetectionVariable[] (matching DetectionFramework) — the asymmetry is intentional and consistent.
  • VCS service return types: Both createRepositoryDetection() and listRepositories() overload signatures (including the deprecated positional-argument overloads) and the implementation signature are all updated to the union return types uniformly.
  • CI fix: npm installnpm ci ensures the publish workflow uses the locked dependency tree.
  • New package-lock.json: Correctly identifies the package as @appwrite.io/console@8.1.0. Two transitively-included, deprecated dev-only packages (glob@8.1.0, inflight@1.0.6) are present but do not affect the published SDK bundle.

Confidence Score: 5/5

Safe to merge — no P0 or P1 issues found; all changes are consistent and well-scoped to the 8.1.0 feature set.

All findings from the previous review round have been addressed. The new models are internally consistent, the union return types are applied uniformly across all overloads, and the CI pipeline improvement is correct. Only transitive dev-dependency deprecation warnings remain, which do not affect the published SDK.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/publish.yml Switched from npm install to npm ci for reproducible CI builds — correct improvement.
src/client.ts Adds getHeaders() method returning a shallow copy of the local Headers plain-object type; bumps SDK version string to 8.1.0. No issues.
src/models.ts Adds ProviderRepositoryRuntime, ProviderRepositoryRuntimeList, DetectionRuntime models and extends UsageOrganization/UsageOrganizationProject with realtime stats fields. Models are consistent with existing patterns.
src/services/vcs.ts Updates return types of createRepositoryDetection() and listRepositories() to union types. Overload signatures, deprecated overloads, and implementation signature are all updated consistently.
package-lock.json New lock file correctly reflects @appwrite.io/console@8.1.0; contains two transitively-included deprecated dev-only packages (glob@8.1.0, inflight@1.0.6) that don't affect the published SDK.

Reviews (2): Last reviewed commit: "chore: update Console SDK to 8.1.0" | Re-trigger Greptile

@ChiragAgg5k ChiragAgg5k merged commit f7af193 into main Mar 31, 2026
2 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