Skip to content

Commit 857ea7c

Browse files
tastybentoclaude
andcommitted
Fix ExecutionException handling and extend CI to master branch
- Log async chat errors via addon.logError instead of rethrowing as RuntimeException - Add master to CI push/PR triggers so release PRs are validated Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent d43ec29 commit 857ea7c

2 files changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/build.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,11 @@ on:
44
push:
55
branches:
66
- develop
7+
- master
78
pull_request:
89
branches:
910
- develop
11+
- master
1012

1113
jobs:
1214
build-and-test:

src/main/java/world/bentobox/chat/listeners/ChatListener.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ public void onChat(final AsyncPlayerChatEvent e) {
121121
} catch (InterruptedException ex) {
122122
Thread.currentThread().interrupt();
123123
} catch (java.util.concurrent.ExecutionException ex) {
124-
throw new RuntimeException("Failed to process async chat on the main thread", ex);
124+
addon.logError("Failed to process async chat for " + p.getName() + ": " + ex.getCause());
125125
}
126126
} else if (handleChatSync(p, message)) {
127127
e.setCancelled(true);

0 commit comments

Comments
 (0)