chore: add Unity validation worktree workflow#273
Draft
JOY (JOY) wants to merge 3 commits into
Draft
Conversation
Contributor
Author
|
Validation hardening update:
Reason: manual opening of the fresh validation project crashed in Unity 6000.5 beta on D3D12 during import ( |
Contributor
Author
|
Local reviewer fallback: APPROVED. Reviewed scope:
Findings: no blocking issues. The update keeps Unity validation isolated in |
13 tasks
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
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.
What does this PR do?
Adds the local Unity validation worktree workflow for parallel agents. The workflow keeps feature agents in separate worktrees, uses one fixed
unity-validationworktree for Unity integration, and validates one or more refs through disposablevalidation/*branches instead of merging into rootdev.Linked issue / ADR
Refs local multi-agent Unity validation workflow.
Touched areas
Unity/)backend/nakama/)docs/design/) or ADRs (docs/adr/)Test plan
devcleaned and fast-forwarded toorigin/dev; previous dirty work preserved on local branchwip/root-dirty-20260526-141239commit52d32b49.D:\Projects\Second-Spawn\.claude\worktrees\unity-validation.tools/unity-validation/Initialize-UnityValidationWorktree.ps1is idempotent from a feature worktree and resets validation lane tovalidation/unityfromorigin/dev.tools/unity-validation/Invoke-UnityValidation.ps1 -Refs feat/facial-voice-mvp -QueueName dryrun-default-root -SkipUnitymerged PR feat: add facial voice presentation hooks #272 branch into a disposable validation branch without touching rootdev.git diff --cached --check.npx.cmd --yes markdownlint-cli2 docs/setup/unity-validation-workflow.md docs/setup/agent-handoff.md docs/setup/docs-navigation-and-gitbook.md docs/SUMMARY.md..claude/CLAUDE.mdare synced after their header line.The "path" argument must be of type string. Received undefined. No packages loaded.The script correctly fails on this log pattern instead of returning a false pass.Server-authority check (mandatory if touching gameplay)
No gameplay or state mutation path changed.
Reviewer pass
Local
code-reviewfallback verdict: APPROVED.Review summary:
dev, uses ignored worktrees, and avoids cross-repo edits..claude/CLAUDE.md, handoff rules, and GitBook navigation.