Skip to content

fix(forge): dedup de libraries evita log4j incompatível, libera v0.6.2 - #70

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 merge da versão vanilla com a do Forge/NeoForge concatenava as libraries sem remover duplicatas por coordenada Maven. Quando os dois declaravam log4j-core em versões diferentes, as duas jars entravam no classpath e o Java podia resolver a errada, causando NoSuchMethodError dentro das próprias classes de log do Forge (TransformingThrowable- PatternConverter/ThrowableProxy) — e mascarando o erro real, já que o gerador de crash report quebrava no processo. Agora load_merged_version deduplica por group:artifact:classifier, mantendo sempre a última ocorrência (a declarada pelo loader).

📝 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 NoSuchMethodError relacionado ao Log4j durante a instalação e execução de modpacks Forge/NeoForge.
    • Removidas bibliotecas duplicadas das versões mescladas, priorizando as configurações do Forge/NeoForge.
  • Lançamento

    • Aplicativo atualizado para a versão 0.6.2.
    • Publicadas as notas de lançamento, incluindo informações sobre o instalador para Windows e o changelog completo.
    • Esta versão não inclui novos recursos ou melhorias adicionais.

O merge da versão vanilla com a do Forge/NeoForge concatenava as
libraries sem remover duplicatas por coordenada Maven. Quando os dois
declaravam log4j-core em versões diferentes, as duas jars entravam no
classpath e o Java podia resolver a errada, causando NoSuchMethodError
dentro das próprias classes de log do Forge (TransformingThrowable-
PatternConverter/ThrowableProxy) — e mascarando o erro real, já que o
gerador de crash report quebrava no processo. Agora load_merged_version
deduplica por group:artifact:classifier, mantendo sempre a última
ocorrência (a declarada pelo loader).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@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: d2c3c71c-a4de-419e-a54d-6ed791a83329

📥 Commits

Reviewing files that changed from the base of the PR and between bd27d9b and 366a79f.

📒 Files selected for processing (3)
  • .github/releases/v0.6.2.md
  • src-tauri/app/infrastructure/modloader/forge_like/install.rs
  • src-tauri/tauri.conf.json

Walkthrough

A versão 0.6.2 remove bibliotecas Maven duplicadas do JSON mesclado do Forge/NeoForge. A alteração prioriza a última declaração, preserva a ordem restante e documenta a correção nas notas de lançamento.

Changes

Deduplicação de bibliotecas Forge/NeoForge

Layer / File(s) Summary
Deduplicação no carregamento da versão
src-tauri/app/infrastructure/modloader/forge_like/install.rs
load_merged_version agora remove bibliotecas duplicadas. A função usa grupo, artefato e classificador como chave, mantém a última ocorrência e trata nomes Maven inválidos.
Alinhamento da versão 0.6.2
src-tauri/tauri.conf.json, .github/releases/v0.6.2.md
A versão do aplicativo passa para 0.6.2. As notas de lançamento registram a correção do conflito de bibliotecas Log4j no Forge/NeoForge.

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

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.

@kauafpssx
kauafpssx merged commit 25632f0 into main Aug 16, 2026
3 of 4 checks passed
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