fix: corrigir broadcast de blog e geração de NPCs (x-store-domain + merge de config)

[ZzPowerTech]

Resumo

Os dois bugs reportados — NPCs não sendo gerados e posts do blog não chegando como broadcast — tinham a mesma causa raiz: a API da CentralCart é multi-tenant e exige o header x-store-domain em toda chamada, mas ele estava chegando vazio em produção.

Evidência do servidor:

x-store-domain: (vazio)  →  HTTP 404 {"message":"Store not found"}

O api.store_domain estava vazio porque o plugin só fazia saveDefaultConfig(), que não mescla chaves novas em um config.yml já existente. Como store_domain e a seção blog: foram adicionados depois que o plugin já rodava, essas chaves nunca apareceram no servidor.

O que mudou

Causa raiz (corrige blog + NPCs)

• CentralCartTopPlugin#mergeConfigDefaults() — copyDefaults(true) mescla chaves novas em configs antigos, preservando os valores do usuário.
• CentralCartApiService — passa a enviar x-store-domain na busca de top doadores e a logar o corpo do erro (antes só o código HTTP, diagnóstico cego).
• BlogPostService — aborta cedo com erro explícito quando o domínio não está configurado (em vez de bater 404 três vezes).

Robustez do blog

• BlogPostCheckTask — seeding na 1ª execução (não spamma posts antigos) e anúncio de todos os posts novos, não só o último. Título sanitizado (remove # do markdown e evita quebra do MiniMessage); fallback de layout se notification.lines faltar.

Refatoração

• Novos DateTimeUtil e BlogNotifier; helpers em PluginUtils. A task automática e o /testblogpost agora compartilham o mesmo caminho de broadcast (remove duplicação).
• build.gradle — copyJar sem nome de jar hardcoded (quebrava a cada bump); versão 1.1.0 → 1.2.0.

Docs

• Adiciona CLAUDE.md, incluindo o gotcha de build: o org.gradle.java.home global aponta para JDK 25, que quebra o Gradle 8.8 — buildar com -Dorg.gradle.java.home="C:\Program Files\Java\jdk-21".

⚠️ Ação necessária pós-merge (config)

O código agora expõe a chave, mas o valor precisa ser preenchido:

1. Atualizar o jar para 1.2.0 e reiniciar o servidor (o merge de config roda no boot e adiciona api.store_domain + seção blog ao config.yml).
2. Editar config.yml: api.store_domain: "loja.austv.net" (e confirmar o api.token).
3. /centralcartreload, depois validar com /testblogpost e /spawntopnpcs.

Notas para o revisor

• O blog foi confirmado por evidência direta (testei a API: com domínio → 200; sem → 404).
• Para os NPCs não há token de produção para teste local; a correção mira a mesma causa multi-tenant e instrumenta o log. Se /spawntopnpcs ainda falhar, o console agora mostra o corpo exato do erro ([Top] ... HTTP xxx — body: ...).
• Build validado: compileJava e jar passam; plugin.yml expande para 1.2.0.
• Sem testes automatizados no repo (validação via build + comandos no servidor).

🤖 Generated with Claude Code

[github-actions]

Qodana Community for JVM

6 new problems were found

Inspection name	Severity	Problems
Constant values	🔶 Warning	6

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

• Register at Qodana Cloud and configure the action
• Use GitHub Code Scanning with Qodana
• Host Qodana report at GitHub Pages
• Inspect and use qodana.sarif.json (see the Qodana SARIF format for details)

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/qodana-action@v2025.2.3
        with:
          upload-result: true

Contact Qodana team

Contact us at qodana-support@jetbrains.com

• Or via our issue tracker: https://jb.gg/qodana-issue
• Or share your feedback: https://jb.gg/qodana-discussions

[ZzPowerTech]

Qodana — 12 dos 18 avisos resolvidos no commit 019cf9a (os que estão no escopo deste PR):

• 10× Nullability (setExecutor NPE em CentralCartTopPlugin) — extraí um helper registerCommand(name, executor) que faz null-check em getCommand() e loga um aviso se o comando faltar no plugin.yml, em vez de arriscar NPE.
• 2× Constant values que eu havia introduzido — removi as condições sempre-falsas lastSeen == null (BlogPostCheckTask) e lines == null (BlogNotifier); getString com default e getStringList nunca retornam null no Bukkit.

Os 6 avisos restantes (Constant values) estão em arquivos não tocados por este PR — MessagesManager, TopNpcManager (x2), MonthlyNpcUpdateTask, ScheduleInfoCommand, ReloadCommand. São pré-existentes; deixei-os fora para manter o PR focado na correção do blog/NPCs. Podem ser limpos num PR de manutenção separado.

🤖 Addressed by Claude Code