fix(storage): analytics bucket prop

[fenos]

Storage Analytics bucket

• Fixed property from id to name

[Copilot]

Pull Request Overview

This PR fixes the property name in the AnalyticBucket interface from id to name, aligning the type definition with the actual API implementation. The change updates both the interface definition and the corresponding method parameters and documentation.

• Changed AnalyticBucket.id to AnalyticBucket.name in the type definition
• Updated deleteBucket method parameter from bucketId to bucketName with corresponding documentation
• Removed unused Bucket import from StorageAnalyticsApi.ts

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
packages/core/storage-js/src/lib/types.ts	Updated AnalyticBucket interface property from id to name
packages/core/storage-js/src/packages/StorageAnalyticsApi.ts	Updated deleteBucket parameter name, documentation, and URL construction; removed unused import

Comments suppressed due to low confidence (1)

packages/core/storage-js/src/packages/StorageAnalyticsApi.ts:112

• The sortColumn option includes 'id' which is inconsistent with the AnalyticBucket interface that only has a name property (not id). Since AnalyticBucket doesn't have an id field, the 'id' sort column option should be removed to prevent runtime errors.

    sortColumn?: 'id' | 'name' | 'created_at' | 'updated_at'

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coveralls]

coverage: 95.367% (+13.8%) from 81.58%
when pulling c12f4ce on fix/analytics-bucket-id-prop
into 1371057 on master.