Skip to content

fix(sites): remove deployment metrics card and bump immutable audit override#3136

Merged
HarshMN2345 merged 3 commits into
mainfrom
fix-remove-site-deployment-metrics-usage
Jul 22, 2026
Merged

fix(sites): remove deployment metrics card and bump immutable audit override#3136
HarshMN2345 merged 3 commits into
mainfrom
fix-remove-site-deployment-metrics-usage

Conversation

@HarshMN2345

@HarshMN2345 HarshMN2345 commented Jul 22, 2026

Copy link
Copy Markdown
Member

Summary

  • sites.getUsage() is being removed from the SDK in appwrite/sdk-for-console#105, part of the same usage-endpoint removal covered by fix: remove remaining tabs/routes and clear high-severity audit #3134.
  • That cleanup missed one live consumer: the inline build-metrics card (deploymentMetrics.svelte) rendered on the site deployments page, which called sites.getUsage() directly (separate from the top-level Usage tab already removed).
  • Removes the card and its now-dead component file.
  • Also bumps the immutable override from ^5.1.5 to ^5.1.8 (pulled in via sass) to clear a bun audit --audit-level high failure — two DoS advisories, pre-existing on main, unrelated to the sites fix but needed for CI to pass.

Test plan

  • bun audit --audit-level high — 0 vulnerabilities
  • bun run check — 0 errors
  • bun run lint — 0 errors
  • bun run test:unit — 239 passed
  • Site deployments page renders correctly without the metrics card

sites.getUsage() is being dropped from the SDK (appwrite/sdk-for-console#105),
along with the other usage endpoints removed in #3134. This inline
build-metrics card on the site deployments page was the one live
consumer that cleanup missed.
@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes the site deployment metrics card that used a deprecated SDK endpoint. The main changes are:

  • Removes the metrics card from the site deployments page.
  • Deletes the card's self-contained Svelte component.
  • Updates the immutable dependency override and lockfile resolution.

Confidence Score: 5/5

This looks safe to merge.

  • The deleted component has no remaining imports or render sites.
  • The deprecated sites.getUsage() call is removed.
  • The dependency override and resolved lockfile version are consistent.
  • No blocking issues were found in the updated code.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/sites/site-[site]/deployments/+page.svelte Removes the metrics component import and render site while preserving the deployment list layout.
src/routes/(console)/project-[region]-[project]/sites/site-[site]/deployments/deploymentMetrics.svelte Deletes the self-contained component that called the deprecated sites.getUsage() endpoint.
package.json Raises the immutable override from ^5.1.5 to ^5.1.8.
bun.lock Updates the override and resolves immutable to version 5.1.9.

Reviews (2): Last reviewed commit: "Merge branch 'fix-audit-immutable' into ..." | Re-trigger Greptile

bun audit --audit-level high was failing on immutable (pulled in via
sass) for two DoS advisories (32-bit trie overflow, hash-collision
complexity). Bumped the override from ^5.1.5 to ^5.1.8.
@HarshMN2345 HarshMN2345 changed the title fix(sites): remove deployment metrics card calling sites.getUsage() fix(sites): remove deployment metrics card and bump immutable audit override Jul 22, 2026
@HarshMN2345
HarshMN2345 merged commit c424116 into main Jul 22, 2026
3 of 4 checks passed
@HarshMN2345
HarshMN2345 deleted the fix-remove-site-deployment-metrics-usage branch July 22, 2026 10:51
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