Skip to content

Remove aqua registry fallback - #166

Merged
jdx merged 1 commit into
jdx:mainfrom
risu729:remove-aqua-registry-fallback
May 2, 2026
Merged

jdx merged 1 commit into
jdx:mainfrom
risu729:remove-aqua-registry-fallback

Conversation

@risu729

@risu729 risu729 commented May 2, 2026 •

Copy link
Copy Markdown
Contributor

Feel free to close this. I don't know how many users will be affected by this.

Summary

  • remove the legacy /aqua-registry compatibility route from the worker app
  • remove the aqua-registry update workflow and script
  • delete the checked-in docs/aqua-registry mirror that is no longer maintained

Context

Compatibility

Legacy mise versions older than v2025.8.17 may still request /aqua-registry/... from this service and may be affected. The local mirror is out of sync and is no longer used by supported clients.

Verification

  • bunx tsc
  • ASTRO_TELEMETRY_DISABLED=1 bun run --cwd web build
  • node scripts/web-build.test.js

Notes

  • bun run test:js currently fails in scripts/generate-toml.test.js under this local Node/test setup; direct generate-toml invocation still works and this is unrelated to the aqua-registry removal.
  • cd web && bunx tsc currently fails on TS 6 deprecating baseUrl without ignoreDeprecations, also unrelated to this change.

@greptile-apps

greptile-apps Bot commented May 2, 2026

Copy link
Copy Markdown

Too many files changed for review. (2084 files found, 100 file limit)

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request involves a large-scale cleanup, specifically the deletion of numerous package configuration files and the clearing of a comprehensive registry list. These changes effectively remove definitions for hundreds of external tools from the system. I have no feedback to provide.

@risu729
risu729 marked this pull request as ready for review May 2, 2026 06:30
Copilot AI review requested due to automatic review settings May 2, 2026 06:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@jdx
jdx merged commit 87d4a32 into jdx:main May 2, 2026
4 checks passed
@risu729
risu729 deleted the remove-aqua-registry-fallback branch May 2, 2026 12:06
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.

3 participants