Optimize retained line rendering and renderer state handling#58
Conversation
…g mode feat(lang): add debug message for rebased shape count in English, Simplified Chinese, and Traditional Chinese
…s for entity counts
… update, and add unit tests
…ission logic, and add unit tests
…bled side effect, and add unit tests
twme-ai
left a comment
twme-ai
left a comment
There was a problem hiding this comment.
我以最新 head 重新審查並驗證：GitHub CI 與 mvn clean verify 都顯示成功，但正常建置實際執行 0 個測試；指定新版 Surefire 後，新增的 26 個 JUnit 5 測試才有執行且全部通過。另外，新的 view 子權限分流在外層 gate 漏掉 hide 權限。詳見逐行留言。
| <artifactId>junit-jupiter</artifactId> | ||
| <version>5.11.4</version> | ||
| <scope>test</scope> | ||
| </dependency> |
There was a problem hiding this comment.
[P1] 請設定支援 JUnit 5 的 Maven Surefire 版本。目前只加入 junit-jupiter 依賴，但 Maven 實際使用預設的 Surefire 2.17；我執行 mvn clean verify 得到 Tests run: 0，所以目前綠色 CI 沒有跑這次新增的任何測試。改用 maven-surefire-plugin:3.5.4:test 後才真正執行 26 個測試（全部通過）。建議在 <build><plugins> 明確加入近期的 Surefire 3.x，讓 PR CI 真正涵蓋這批測試。
| case "share" -> sender.hasPermission("worldeditdisplay.use.share"); | ||
| case "view" -> sender.hasPermission("worldeditdisplay.use.view") | ||
| || sender.hasPermission("worldeditdisplay.use.view.list") | ||
| || sender.hasPermission("worldeditdisplay.use.view.label"); |
There was a problem hiding this comment.
[P1] 這個外層 view gate 也需要包含 worldeditdisplay.use.view.hide。目前 ViewCommand 已允許只有該權限的玩家使用 hide / hideall / unhide，plugin.yml 也把它定義成獨立權限，但請求會先在這裡被拒絕，因此只授予 hide 權限的角色實際無法使用這三個命令。也建議補一個只有 use.view.hide 時 canUseSubCommand(sender, "view") 為 true 的測試。
|
I opened #59 as a reviewed replacement for this PR. It contains all commits from #58 through |
2 participants
Reduce display entity churn by retaining line shapes across renders, expose rendering and rebase statistics in debug output, and improve renderer decisions, permissions, persistence, error handling, and test coverage.