Skip to content

fix(daedalus): recover NeoForge index from existing artifacts - #7751

Closed
waxifyx wants to merge 1 commit into
modrinth:mainfrom
waxifyx:fix/neoforge-manifest-recovery
Closed

waxifyx wants to merge 1 commit into
modrinth:mainfrom
waxifyx:fix/neoforge-manifest-recovery

Conversation

@waxifyx

@waxifyx waxifyx commented Sep 29, 2026

Copy link
Copy Markdown

Summary

  • Keep a published loader index when Maven metadata shrinks sharply rather than overwriting it with an incomplete index.
  • Rebuild missing NeoForge index entries from the existing Maven version list, reusing version JSON already published on launcher-meta instead of downloading every installer again.
  • Publish the rebuilt NeoForge index only on a later run after any missing version JSON has been uploaded.

Why

The live neo/v0/manifest.json currently contains only Minecraft 1.20.1 and 26.3, while NeoForge Maven metadata lists 1759 versions and older neo/v0/versions/*.json files remain available. Previously the truncated index caused the generator to reprocess all of those installers; one failure prevented rebuilding it. This should allow the scheduled generator to recover the index without a client-side fallback. Related: #4530.

Validation

  • rustfmt --check --edition 2024 apps/daedalus_client/src/forge.rs
  • cargo test -p daedalus_client --bin daedalus_client (5 tests passed)
  • cargo clippy -p daedalus_client --bin daedalus_client --all-targets

Cargo checks used a matching temporary workspace because the local clone was sparse; only apps/daedalus_client/src/forge.rs is changed in this PR.

Preserve the published index when upstream version metadata shrinks and reuse available version manifests during recovery. Publish recovered indexes only after missing version artifacts are present.
@github-actions

Copy link
Copy Markdown
Contributor

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@waxifyx

waxifyx commented Sep 29, 2026

Copy link
Copy Markdown
Author

Closing at the repository owner’s request. Please disregard this proposal.

@waxifyx waxifyx closed this Sep 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant