Skip to content

fix: remove remaining tabs/routes and clear high-severity audit#3134

Merged
HarshMN2345 merged 11 commits into
mainfrom
fix-remove-remaining-usage-tabs-consolidated
Jul 21, 2026
Merged

fix: remove remaining tabs/routes and clear high-severity audit#3134
HarshMN2345 merged 11 commits into
mainfrom
fix-remove-remaining-usage-tabs-consolidated

Conversation

@HarshMN2345

@HarshMN2345 HarshMN2345 commented Jul 21, 2026

Copy link
Copy Markdown
Member

No description provided.

Same cleanup as #3128, one level down: the per-table Usage tab still
called tablesDB.getTableUsage(), which is going away with the backend
usage-endpoint removal and is already showing empty data on cloud.
The "Go to usage" (g u) shortcut pointed at the table usage route
removed earlier in this branch, leaving a broken command.
Only the table usage page consumed this component; it's now dead
code after removing that route.
Same cleanup as #3128/#3130: removes the per-database Usage tab
(database-[database]/usage) and its command-center shortcut. It
called tablesDB.getUsage(), which is going away with the backend
usage-endpoint removal.
Removes the Usage tab/route and its g-u command-center shortcut for
buckets, functions, sites, and auth — the same cleanup already done
for databases (#3128/#3130/#3131). Project-level billing usage
(settings/usage) is untouched.
bun audit --audit-level high was failing CI on transitive deps pulled
in by eslint and @sentry/vite-plugin. Existing brace-expansion
override didn't cover the patched range; added a js-yaml override.
…nto fix-remove-remaining-usage-tabs-consolidated
…age-tabs' into fix-remove-remaining-usage-tabs-consolidated
…' into fix-remove-remaining-usage-tabs-consolidated
Removing every remaining usage/[[period]] route in this branch left
the shared Usage/UsageMultiple components (src/lib/layout) with no
consumers, which also broke type-checking since no route defines a
`period` param anymore. Extracted the still-used helper functions
(periodToDates, total, etc.) to a plain usageHelpers.ts and deleted
the dead components. Also removed a leftover "Usage" sub-navigation
entry in the database store.
@appwrite

appwrite Bot commented Jul 21, 2026

Copy link
Copy Markdown

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Environment variable changes require redeployment to take effect

@HarshMN2345 HarshMN2345 changed the title fix: remove remaining Usage tabs/routes and clear high-severity audit fix: remove remaining tabs/routes and clear high-severity audit Jul 21, 2026
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes deprecated usage routes and clears dependency audit issues. The main changes are:

  • Removed service-level usage pages, tabs, and command palette links.
  • Moved shared usage date and metric helpers into usageHelpers.ts.
  • Updated Bun dependency overrides and lockfile entries for patched packages.
  • Restored integrity metadata for remote pkg.vc dependencies.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
bun.lock Updates patched dependency resolutions and keeps integrity checksums for remote package entries.
package.json Aligns dependency overrides with the updated lockfile for brace-expansion and js-yaml.
src/lib/layout/usageHelpers.ts Adds the shared usage helper module used by the remaining usage and overview pages.
src/lib/layout/index.ts Removes deleted usage component exports while preserving the shared UsagePeriods type export.
src/routes/(console)/project-[region]-[project]/functions/function-[function]/usage/[[period]]/+page.ts Removes the function usage route.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/usage/[[period]]/+page.ts Removes the database usage route.
src/routes/(console)/project-[region]-[project]/auth/usage/[[period]]/+page.ts Removes the auth usage route.

Reviews (2): Last reviewed commit: "fix: restore integrity checksums for pkg..." | Re-trigger Greptile

A prior bun update/install cycle on this branch stripped the sha512
checksums for @appwrite.io/console, pink-icons-svelte, and
pink-svelte (flagged in review). Restored them to match main's
pinned versions.
@HarshMN2345
HarshMN2345 merged commit a2a7e1a into main Jul 21, 2026
3 of 4 checks passed
@HarshMN2345
HarshMN2345 deleted the fix-remove-remaining-usage-tabs-consolidated branch July 21, 2026 12:42
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