Skip to content

Mark Bukkit plugin descriptor as Folia-supported#1

Merged
Chwitst187 merged 1 commit intomainfrom
codex/fix-fastasyncworldedit-plugin-loading-error
Feb 25, 2026
Merged

Mark Bukkit plugin descriptor as Folia-supported#1
Chwitst187 merged 1 commit intomainfrom
codex/fix-fastasyncworldedit-plugin-loading-error

Conversation

@Chwitst187
Copy link
Copy Markdown
Owner

Motivation

  • Add folia-supported: true to the Bukkit plugin.yml so Folia/Paper recognizes the plugin as explicitly supported and does not reject it at runtime.

Description

  • Added folia-supported: true to worldedit-bukkit/src/main/resources/plugin.yml to mark the Bukkit plugin as Folia-supported.

Testing

  • Ran ./gradlew :worldedit-bukkit:processResources --no-daemon and the task completed successfully.

Codex Task

Copilot AI review requested due to automatic review settings February 25, 2026 19:48
@Chwitst187 Chwitst187 merged commit d116c46 into main Feb 25, 2026
@Chwitst187 Chwitst187 deleted the codex/fix-fastasyncworldedit-plugin-loading-error branch February 25, 2026 19:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds the folia-supported: true flag to the Bukkit plugin descriptor (plugin.yml) to indicate that FastAsyncWorldEdit (FAWE) is compatible with Folia, Paper's region-based multi-threaded server software.

Changes:

  • Added folia-supported: true field to plugin.yml to mark the plugin as Folia-compatible

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

version: "${internalVersion}"
load: STARTUP
api-version: 1.20
folia-supported: true
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding folia-supported: true indicates that the plugin is compatible with Folia's region-based threading model, but the codebase still uses legacy Bukkit scheduler APIs that are incompatible with Folia.

Folia requires using region-based schedulers (e.g., RegionScheduler, EntityScheduler, AsyncScheduler) instead of the global Bukkit.getScheduler() API. The current implementation in files like BukkitTaskManager.java uses methods such as scheduleSyncRepeatingTask, runTask, and runTaskAsynchronously which won't work correctly on Folia servers.

Additionally, code in BukkitBlockCommandSender.java uses Bukkit.getScheduler().callSyncMethod() which is also incompatible with Folia's threading model.

Before marking the plugin as Folia-supported, the scheduler usage needs to be adapted to work with Folia's region-based threading, or the plugin needs verified testing on Folia to ensure it actually works despite using legacy APIs (Paper's compatibility layer may handle some cases, but this needs verification).

Suggested change
folia-supported: true
folia-supported: false

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants