From 0860104d68e9221220279811cd4e4de846cd7289 Mon Sep 17 00:00:00 2001 From: hatayama Date: Mon, 27 Jul 2026 11:49:30 +0900 Subject: [PATCH 1/2] Point the layout-detection test at a supported skill target The test built its fixture under .cursor, which stopped being a skill target in #1983. Detection skipped the directory entirely, so the test asserted on an empty result and failed on the count before it could check the layout state it exists to verify. .claude is the directory every other single-target test in this fixture uses. --- Assets/Tests/Editor/ToolSkillSynchronizerTests.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Assets/Tests/Editor/ToolSkillSynchronizerTests.cs b/Assets/Tests/Editor/ToolSkillSynchronizerTests.cs index 796de18007..0c14d29c0f 100644 --- a/Assets/Tests/Editor/ToolSkillSynchronizerTests.cs +++ b/Assets/Tests/Editor/ToolSkillSynchronizerTests.cs @@ -894,7 +894,7 @@ public void DetectTargets_WhenGroupedLayoutRequested_IgnoresFlatInstalledSkills( public void DetectTargets_WhenGroupedLayoutRequested_DetectsEmptyFlatManagedDirectories() { string temporaryRoot = CreateTemporaryProjectRoot(); - string targetRoot = Path.Combine(temporaryRoot, ".cursor"); + string targetRoot = Path.Combine(temporaryRoot, ".claude"); Directory.CreateDirectory(Path.Combine(targetRoot, SkillInstallLayout.SkillsDirName, "uloop-compile")); ToolSkillSynchronizer.SkillTargetInfo[] detectedTargets = SkillTargetDetector.DetectTargetsForLayoutStateAtProjectRoot( From e98bc5844d4973f9f8f926921123bd7bb68e28f4 Mon Sep 17 00:00:00 2001 From: hatayama Date: Mon, 27 Jul 2026 11:52:31 +0900 Subject: [PATCH 2/2] Drop the last removed skill-target name from the layout tests AreSkillsInstalled_WhenLegacyManagedDirectoryIsEmpty_StillDetectsFlatLayout still named .cursor. It passes either way because the API takes the directory name directly, but keeping a removed target in a fixture invites the same confusion the sibling test just caused. .codex is what the neighbouring case uses. --- Assets/Tests/Editor/SkillInstallLayoutTests.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Assets/Tests/Editor/SkillInstallLayoutTests.cs b/Assets/Tests/Editor/SkillInstallLayoutTests.cs index 2ed19de655..38442ad8c2 100644 --- a/Assets/Tests/Editor/SkillInstallLayoutTests.cs +++ b/Assets/Tests/Editor/SkillInstallLayoutTests.cs @@ -102,12 +102,12 @@ public void AreSkillsInstalled_WhenLayoutSpecified_MatchesOnlySelectedLayout() public void AreSkillsInstalled_WhenLegacyManagedDirectoryIsEmpty_StillDetectsFlatLayout() { string temporaryRoot = CreateTemporaryProjectRoot(); - string targetRoot = Path.Combine(temporaryRoot, ".cursor"); + string targetRoot = Path.Combine(temporaryRoot, ".codex"); Directory.CreateDirectory(Path.Combine(targetRoot, SkillInstallLayout.SkillsDirName, "uloop-compile")); SkillInstallationDetector detector = new(); - Assert.IsTrue(detector.AreSkillsInstalledForLayout(temporaryRoot, ".cursor", false)); - Assert.IsFalse(detector.AreSkillsInstalledForLayout(temporaryRoot, ".cursor", true)); + Assert.IsTrue(detector.AreSkillsInstalledForLayout(temporaryRoot, ".codex", false)); + Assert.IsFalse(detector.AreSkillsInstalledForLayout(temporaryRoot, ".codex", true)); } // Tests that Unity-side discovery includes CLI-only skills from the packaged core CLI.