Skip to content

Move local editor settings to a sidecar file#8588

Open
bodhibuurstede-sys wants to merge 8 commits into
4ian:masterfrom
bodhibuurstede-sys:codex/separate-editor-settings
Open

Move local editor settings to a sidecar file#8588
bodhibuurstede-sys wants to merge 8 commits into
4ian:masterfrom
bodhibuurstede-sys:codex/separate-editor-settings

Conversation

@bodhibuurstede-sys
Copy link
Copy Markdown

@bodhibuurstede-sys bodhibuurstede-sys commented May 11, 2026

Addresses #1983.

Summary

  • Persist local editor UI settings in .gdevelop/editor-settings.json instead of writing them into the project JSON.
  • Restore sidecar settings when opening local projects, while keeping existing embedded settings working when the sidecar is absent.
  • Remove stale sidecar settings when a project no longer has editor settings to persist.
  • Add coverage for extracting/applying layout, external layout, and custom object variant editor settings.
  • Make the pre-built libGD.js downloader try matching master/commit/<hash> builds before master/latest, so detached PR CI does not pick an incompatible generated file.

Notes

  • The sidecar is intentionally local/user-specific, so teams can gitignore .gdevelop/ to avoid collaboration conflicts.
  • I kept this scoped to the local file storage path where the Git conflict problem happens.
  • Missing sidecar files are ignored before readJSONFile, so normal local projects do not emit a warning when .gdevelop/editor-settings.json has not been created yet.

Verification

  • Semaphore Fast tests (not building GDevelop.js - can have false negatives): passed on the latest commit.
  • Semaphore Flow: passed with No errors! on the earlier run before the CI-only fallback update.
  • Semaphore newIDE auto-formatting and newIDE linting: passed on the earlier run before the CI-only fallback update.
  • Not run locally: this Codex environment does not have the repository dependency setup needed for the full GDevelop test suite.

@bodhibuurstede-sys bodhibuurstede-sys requested a review from 4ian as a code owner May 11, 2026 16:52
@4ian 4ian added the AI label May 11, 2026
Copy link
Copy Markdown
Author

Hi @4ian, the Semaphore PR check is green now after the libGD.js fallback update. This should be ready for review when you have a chance; happy to adjust anything.

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