Skip to content

Move local editor settings to a sidecar file#8588

Open
bodhibuurstede-sys wants to merge 17 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 17 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 per-project sidecars under .gdevelop/<project-name>.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 no sidecar exists.
  • Remove stale sidecar settings when a project no longer has editor settings to persist.
  • Keep sidecars distinct when multiple project files live in the same folder.
  • Add coverage for sidecar paths plus 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 sidecars are 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 via ENOENT, so normal local projects do not emit a warning before one has been created.

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 latest commit.
  • 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