Skip to content

fix(forge): corrige UnsatisfiedLinkError do lwjgl.dll e libera v0.6.1 - #69

Merged
kauafpssx merged 1 commit into
mainfrom
feat/v0.6.0
Aug 16, 2026
Merged

kauafpssx merged 1 commit into
mainfrom
feat/v0.6.0

Conversation

@kauafpssx

@kauafpssx kauafpssx commented Aug 16, 2026 •

Copy link
Copy Markdown
Owner

O comando de launch do Forge/NeoForge apontava java.library.path pra instance_dir/natives, mas o mc-launcher-core sempre extrai as natives em versions/<version_id>/natives — o Java procurava lwjgl.dll (e as demais natives) num diretório vazio e o jogo caía com UnsatisfiedLinkError ao abrir a janela. Removido o override, o comando agora usa o mesmo diretório default da crate (onde a extração real acontece). Vanilla não era afetado.

📝 Descrição

Descreva brevemente o que este PR faz e por quê.

🧩 Mudanças

  • Funcionalidade nova
  • Correção de bug
  • Refatoração
  • Documentação
  • Dependências / CI

✅ Checklist

Antes de enviar, confirme:

  • Branch criada a partir da main atualizada
  • Código segue os padrões do projeto (máx. 200 linhas por arquivo, erros com thiserror, código em inglês)
  • npm run lint passou sem erros
  • npm run build (typecheck) passou
  • Testei manualmente o fluxo alterado
  • Atualizei as notas de release (.github/releases/vX.Y.Z.md) quando necessário

🖼️ Evidências (opcional)

🔗 Issues relacionadas

Summary by CodeRabbit

  • Correções

    • Corrigido o erro que impedia a inicialização de instâncias Forge/NeoForge no Windows por não localizar a biblioteca lwjgl.dll.
  • Atualizações

    • Aplicativo atualizado para a versão 0.6.1.
    • Instalador do Windows atualizado.
    • Notas de versão e link para o changelog completo adicionados.

O comando de launch do Forge/NeoForge apontava java.library.path pra
instance_dir/natives, mas o mc-launcher-core sempre extrai as natives
em versions/<version_id>/natives — o Java procurava lwjgl.dll (e as
demais natives) num diretório vazio e o jogo caía com
UnsatisfiedLinkError ao abrir a janela. Removido o override, o comando
agora usa o mesmo diretório default da crate (onde a extração real
acontece). Vanilla não era afetado.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@kauafpssx
kauafpssx merged commit bd27d9b into main Aug 16, 2026
3 of 4 checks passed
@coderabbitai

coderabbitai Bot commented Aug 16, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: f21bec53-8099-489f-89aa-9a9bb5578c04

📥 Commits

Reviewing files that changed from the base of the PR and between 111ec00 and 23dc786.

📒 Files selected for processing (4)
  • .github/releases/v0.6.1.md
  • src-tauri/app/application/use_cases/launch_instance.rs
  • src-tauri/app/infrastructure/modloader/forge_like/command.rs
  • src-tauri/tauri.conf.json

Walkthrough

A correção remove o diretório explícito de natives do lançamento Forge/NeoForge. O construtor passa a usar o caminho padrão. A versão do aplicativo é atualizada para 0.6.1, com novas notas de release.

Changes

Lançamento Forge/NeoForge

Layer / File(s) Summary
Uso do caminho padrão de natives
src-tauri/app/infrastructure/modloader/forge_like/command.rs
build_command não recebe mais natives_dir. LaunchOptions não define mais natives_directory.
Atualização da versão e notas de release
src-tauri/tauri.conf.json, .github/releases/v0.6.1.md
A versão passa para 0.6.1. As notas documentam a correção do erro relacionado a lwjgl.dll em instâncias Forge/NeoForge.

Estimated code review effort: 2 (Simple) | ~10 minutos

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/v0.6.0

Comment @coderabbitai help to get the list of available commands.

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.

1 participant