Skip to content

Remove experimental annotations from a lot of API - #14106

Merged
kennytv merged 9 commits into
mainfrom
exp-annotations
Aug 6, 2026
Merged

Remove experimental annotations from a lot of API#14106
kennytv merged 9 commits into
mainfrom
exp-annotations

Conversation

@kennytv

@kennytv kennytv commented Jul 26, 2026

Copy link
Copy Markdown
Member

Mainly:

  • Data components
  • Bed enter actions
  • Connection
  • Inventory view and menu builders (except for the location subtype given block entity behavior has issues)
  • Combat tracker
  • Server links
  • Dialogs

It's not a goal to add the new annotation everywhere that it would realistically fit here, just to where it replaces existing experimental annotations.

Most notably includes the data component API, where it is instead replaced with package-wide MinecraftVersionDependent annotations
@kennytv
kennytv requested a review from a team as a code owner July 26, 2026 17:53
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Jul 26, 2026
@GliczDev

Copy link
Copy Markdown
Contributor

What about Position? Also, what will happen to the Paper Dialog API once Adventure eventually has one?

@Doc94

Doc94 commented Jul 27, 2026

Copy link
Copy Markdown
Member

Not sure if can be part of this PR... but with the new annotation can be good include in the generator for all the fields added? (like ItemType or Tags)

Comment thread paper-api/src/main/java/org/bukkit/tag/package-info.java Outdated
@kennytv

kennytv commented Jul 28, 2026

Copy link
Copy Markdown
Member Author

This PR isn't meant to add the annotation everywhere that fits, that's a large task and can be done separately/in smaller steps. This is mainly about removing the experimental annotation.

Position isn't all too useful yet and with Location implementing it doesn't have its immutability promise

Comment thread paper-api/src/main/java/org/bukkit/block/TrialSpawner.java
Comment thread paper-api/src/main/java/org/bukkit/util/BlockTransformer.java
Comment thread paper-api/src/main/java/org/bukkit/util/EntityTransformer.java
Comment thread paper-api/src/main/java/org/bukkit/MusicInstrument.java
Comment thread paper-api/src/main/java/org/bukkit/structure/Structure.java
@papermc-projects papermc-projects Bot moved this from Awaiting review to Full approval in Paper PR Queue Aug 5, 2026
@kennytv
kennytv merged commit 7731202 into main Aug 6, 2026
7 checks passed
@kennytv
kennytv deleted the exp-annotations branch August 6, 2026 10:24
@github-project-automation github-project-automation Bot moved this from Full approval to Merged in Paper PR Queue Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

6 participants