Skip to content

Rename entityConfig to entityBlueprint in Korge-fleks gradle task#2358

Merged
jobe-m merged 2 commits intomainfrom
feature/Rename-entityConfig-to-entityBlueprint
Mar 25, 2026
Merged

Rename entityConfig to entityBlueprint in Korge-fleks gradle task#2358
jobe-m merged 2 commits intomainfrom
feature/Rename-entityConfig-to-entityBlueprint

Conversation

@jobe-m
Copy link
Copy Markdown
Member

@jobe-m jobe-m commented Mar 25, 2026

No description provided.

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Mar 25, 2026

🤖 Augment PR Summary

Summary: Updates the Korge-fleks Gradle exporter to use the LDtk field name entityBlueprint instead of entityConfig when building chunk entity metadata.
Changes: Renames the identifier checks/reads and the derived type value to match the new naming, keeping the exported JSON structure consistent with the updated field.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@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. 1 suggestion posted.

Fix All in Augment

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

val entityConfigField = ldtkEntities.first { it["__identifier"] == "entityConfig" }
val entityConfigType = entityConfigField["__value"]?.let { it as String } ?: error("Entity config field '__value' is null for entity '$entityName' in chunk '$chunkName'!")
val entityBlueprintField = ldtkEntities.first { it["__identifier"] == "entityBlueprint" }
val entityBlueprintType = entityBlueprintField["__value"]?.let { it as String } ?: error("Entity config field '__value' is null for entity '$entityName' in chunk '$chunkName'!")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The thrown error message still says "Entity config field" even though the identifier is now entityBlueprint, which could be confusing when debugging null __value cases.

Severity: low

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@jobe-m jobe-m merged commit a69a5e9 into main Mar 25, 2026
6 checks passed
@jobe-m jobe-m deleted the feature/Rename-entityConfig-to-entityBlueprint branch March 25, 2026 16:29
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