Repository navigation
chore(deps): bump appwrite from 21.5.0 to 29.1.0 - #22
dependabot[bot] wants to merge 1 commit into
Conversation
🟡 Tier B · Needs changes before merging
The dependency update raises Appwrite to 29.1.0 and synchronizes the manifest and npm lockfile. The lockfile now includes Appwrite's added runtime dependencies, including MQTT and its supporting packages.
Fix with agent prompt### Issue 1
package-lock.json:5908
**Appwrite 29 breaks the project's TypeScript check**
In 29.1.0, `Client.ping()` is typed as `Promise<unknown>`; 21.5.0 declared `Promise<string>`. The existing assignment at `app/index.tsx:73` passes that value to `Log.response: string`, so `npx tsc --noEmit` fails with TS2322 after `npm ci` and the CI typecheck cannot pass until the value is handled.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.📂 Walkthrough · 2
⏳ Still open from earlier reviews · 1
Reviewed |
d976dda to
7b4e074
Compare
| "resolved": "https://registry.npmjs.org/appwrite/-/appwrite-21.5.0.tgz", | ||
| "integrity": "sha512-643bMRZVYXMluXvSXbdaLAi9qqTJLWbVGguKH4vH6IdKHur6gGIirhCOqAEt33pV4TOFJ55VBu8c/+Ft1ke2SA==", | ||
| "license": "BSD-3-Clause" | ||
| "version": "29.1.0", |
There was a problem hiding this comment.
Appwrite 29 breaks the project's TypeScript check
In 29.1.0, Client.ping() is typed as Promise<unknown>; 21.5.0 declared Promise<string>. The existing assignment at app/index.tsx:73 passes that value to Log.response: string, so npx tsc --noEmit fails with TS2322 after npm ci and the CI typecheck cannot pass until the value is handled.
Prompt To Fix With AI
This is a comment left during a code review.
Path: package-lock.json
Line: 5908
Comment:
**Appwrite 29 breaks the project's TypeScript check**
In 29.1.0, `Client.ping()` is typed as `Promise<unknown>`; 21.5.0 declared `Promise<string>`. The existing assignment at `app/index.tsx:73` passes that value to `Log.response: string`, so `npx tsc --noEmit` fails with TS2322 after `npm ci` and the CI typecheck cannot pass until the value is handled.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.🟠 Major · bug · Reply if this doesn't apply.
Bumps [appwrite](https://github.com/appwrite/sdk-for-web) from 21.5.0 to 29.1.0. - [Release notes](https://github.com/appwrite/sdk-for-web/releases) - [Changelog](https://github.com/appwrite/sdk-for-web/blob/main/CHANGELOG.md) - [Commits](appwrite/sdk-for-web@21.5.0...29.1.0) --- updated-dependencies: - dependency-name: appwrite dependency-version: 29.1.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
7b4e074 to
845a36b
Compare
Bumps appwrite from 21.5.0 to 29.1.0.
Release notes
Sourced from appwrite's releases.
... (truncated)
Changelog
Sourced from appwrite's changelog.
... (truncated)
Commits
8ba5db3Merge pull request #192 from appwrite/dev43c953echore: update Web SDK to 29.1.00fb781eMerge remote-tracking branch 'origin/dev' into dev1699d10Merge pull request #191 from appwrite/dev66fd0c8docs: create tablesDB in the JavaScript typing example73da3f7docs: refresh README4110020Merge commit '03267bc28324fc3c186d4e76683b19cd90022adf' into devaf41861Merge pull request #190 from appwrite/dev03267bcfeat: SDK update for the 2.5.x majors188db0cMerge remote-tracking branch 'origin/dev' into devMaintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for appwrite since your current version.