Skip to content

Add collision tile information to korge fleks gradle plugin#2356

Merged
jobe-m merged 7 commits intomainfrom
feature/Add-collision-tile-information-to-korge-fleks-gradle-plugin
Feb 21, 2026
Merged

Add collision tile information to korge fleks gradle plugin#2356
jobe-m merged 7 commits intomainfrom
feature/Add-collision-tile-information-to-korge-fleks-gradle-plugin

Conversation

@jobe-m
Copy link
Member

@jobe-m jobe-m commented Feb 21, 2026

No description provided.

@augmentcode
Copy link

augmentcode bot commented Feb 21, 2026

🤖 Augment PR Summary

Summary: This PR enhances the KoRGE Fleks Gradle exporter to include per-tile collision metadata when exporting LDtk level chunks.

Changes:

  • Extends TileSetData to carry a per-tileset map of tile IDs to collision IDs.
  • Parses an LDtk enum named collisionConfig and builds a collision-ID → shape/rect lookup table.
  • Reads tileset enumTags to associate tiles with collision definitions.
  • Encodes collision info into the exported tile int (using dedicated bit ranges) and emits collision shape data into common.json.
  • Copies the collision-shapes tileset image into level_data/ to make collision shape graphics available at runtime.
  • Tightens LDtk JSON parsing by turning several previously-optional fields into required definitions with explicit errors.

Technical Notes: Collision is represented as an 8-bit value in the tile encoding, while common.json carries the collision ID → (x,y) mapping for lookup.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@jobe-m jobe-m merged commit d84cff5 into main Feb 21, 2026
6 checks passed
@jobe-m jobe-m deleted the feature/Add-collision-tile-information-to-korge-fleks-gradle-plugin branch February 21, 2026 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant