Enable editor automation and faster checks - #880
Merged
Eli Pinkerton (wallstop) merged 24 commits intoSep 28, 2026
Merged
Conversation
Commit the existing exact tool versions as one resolved graph. This lets local checks and CI use npm ci and gives CI a stable cache key. Keep the lock out of the Unity package and teach release tooling users how to update it. Refs #879
Use isolated runspaces for pure-content fixtures so repeated child PowerShell starts no longer dominate local checks. Keep CLI binding smoke tests and require exact exits and diagnostics. Refs #869
Editor scripts need to inspect and repair missing prefab components from explicit folders. Move the repair out of the window and report save and cleanup failures. Cover nested components, batch saves, and invalid input in Unity EditMode tests. Refs #821
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit fc1832f. Configure here.
The whole-tree Repo Lint gate found two comparisons in the new prefab API that used the unsupported direction. Swap their operands without changing behavior. Refs #821
Eli Pinkerton (wallstop)
changed the title
Eli Pinkerton (wallstop)
changed the title
Eli Pinkerton (wallstop)
changed the title
Eli Pinkerton (wallstop)
changed the title
Eli Pinkerton (wallstop)
deleted the
codex/goal-sep28-issues-ci-speed
branch
September 28, 2026 19:03
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DISCLOSURE: LLM-GENERATED TEXT
Editor scripts can now repair missing prefab scripts, create sprite atlas configs, apply batch texture and sprite settings, and enable or disable analyzer policies without opening windows. The windows use the same APIs, so interactive and scripted actions share importer, asset-save, and partial-failure behavior.
The PR also normalizes Windows asset folder paths across editor discovery and repair, saves atlas config state before refresh, and guards batched atlas creation. Local contract checks run directly and use isolated PowerShell runspaces for synthetic fixtures, including EOL mutation cases. Git staging retries honor the configured lock timeout. The root npm lockfile is again ignored and absent from the final PR tree so Git URL installs receive the intended package layout; CI uses its existing no-lock install path.
Validation: Head
771ae636passed local Unity 6000.4.6f1 sprite-settings tests 165/165 and progress-cleanup tests 14/14. Full Unity typecheck, both editor-test variants, 342 compiled documentation samples, strict MkDocs, spelling, formatting, preflight, no-lock npm install, and npm package validation passed. All nine hosted workflows passed on head771ae636, including 93/93 contract checks and the full Unity matrix. EditMode totals increased by four on every selected Unity version and the single-threaded run; PlayMode and standalone totals held steady, with zero failures. The preceding head8c0a1abbalso passed all nine workflows. The EOL contract check on an earlier head took 5.5s versus 16.7s on its prior green head with 31 versus 27 assertions; this timing comparison is observational.Editor APIs mutate assets and importers and can report partial failure after a write. The analyzer ruleset API documents that file changes are outside Unity Undo.
Fixes #879. Refs #821 and #869.