refactor(ChunkScanner): skip empty chunk section, clear before load

[MukjepScarlet]

No description provided.

[liquidbounce-support-bot]

Thanks for the refactor, @MukjepScarlet! A couple of things worth a look before a maintainer reviews:

• scanChunkSections no longer awaits the section scans. The old code collected the jobs and .joinAll()ed them, so the suspend fun only returned once every section had been scanned. The new version wraps the loop in coroutineScope { … } but still launches each job on the object-level scope (scope.launch { … }), so those coroutines aren't children of the coroutineScope and aren't awaited — the function returns as soon as the loop finishes dispatching. If the intent is to keep the join behaviour, launch on the coroutineScope receiver (or collect + joinAll); if fire-and-forget is intended, the coroutineScope wrapper is a no-op and can go.

• The ChunkLoad change is broader than the title. Adding it.clearChunk(chunk.pos) before chunkUpdate (for the replaced-chunk case) is a behaviour change beyond "skip empty chunk section" — might be worth calling out in the PR description so reviewers don't miss it.

Everything else (the hasOnlyAir() skip and the suspend (CoroutineScope) -> Unit interface) reads cleanly. Leaving the real review to the maintainers.

---

🤖 Automated support reply — a human maintainer will review if this doesn't help.