Skip to content

Fix script links after Explorer renames and moves - #36

Merged
JamieW105 merged 1 commit into
mainfrom
fix/script-links-follow-renames
Oct 14, 2026
Merged

JamieW105 merged 1 commit into
mainfrom
fix/script-links-follow-renames

Conversation

@JamieW105

@JamieW105 JamieW105 commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

Renaming an attached script in Explorer previously left its registry path pointing to the old filename, so Play could no longer find the source. Explorer now updates the script registry while keeping the attachment ID, language, and properties intact. File moves and folder renames/moves update all affected script paths. Failed registry saves roll back the file and metadata move. Invalid script destinations and registry collisions are rejected.

Companion documentation: Rustic-Game-Engine/docs#43

Validation:

  • cargo test -p rustic-editor explorer_tests --no-fail-fast: 20 passed, including script identity, nested relocations, failed-save rollback, and invalid destinations.
  • cargo fmt --all --check: passed.
  • cargo clippy --locked -p rustic-editor --all-targets -- -D warnings: passed.
  • Documentation lint, production build, TypeScript, and npm audit: passed.
  • Windows installer build and packaged Setup tests: passed; a fresh dist/RusticGameEngine-Setup-*.exe was uploaded as RusticGameEngine-Windows-Installer: https://github.com/Rustic-Game-Engine/engine/actions/runs/38538271564.
  • Full Linux CI (workspace lint, tests, rendered viewport verification, target checks, and application smoke tests): passed.

@JamieW105
JamieW105 merged commit 76ae507 into main Oct 14, 2026
9 checks passed
@github-actions
github-actions Bot deleted the fix/script-links-follow-renames branch October 14, 2026 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants