Skip to content

Commit 77c841b

Browse files
committed
Tests: switch NL suite to ClaudeTests/longUnityScript-claudeTest.cs; enrich spec examples; add routing metadata; add standalone long test script
1 parent 3943abd commit 77c841b

File tree

5 files changed

+2055
-18
lines changed

5 files changed

+2055
-18
lines changed

.claude/prompts/nl-unity-suite.md

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ You are running in CI at the repository root. Use only the tools that are allowe
77
- MCP tools from server "unity" (exposed as mcp__unity__*).
88

99
## Test target
10-
- Primary file: `Assets/Scripts/Interaction/SmartReach.cs`
10+
- Primary file: `ClaudeTests/longUnityScript-claudeTest.cs`
1111
- For each operation, prefer structured edit tools (`replace_method`, `insert_method`, `delete_method`, `anchor_insert`, `apply_text_edits`, `regex_replace`) via the MCP server.
1212
- Include `precondition_sha256` for any text path write.
1313

@@ -23,8 +23,8 @@ You are running in CI at the repository root. Use only the tools that are allowe
2323
- If a write fails midway, ensure the file is restored before proceeding.
2424

2525
## NL-0. Sanity Reads (windowed)
26-
- Tail 120 lines of SmartReach.cs.
27-
- Show 40 lines around method `DeactivateIK`.
26+
- Tail 120 lines of `ClaudeTests/longUnityScript-claudeTest.cs`.
27+
- Show 40 lines around method `Update`.
2828
- **Pass** if both windows render with expected anchors present.
2929

3030
## NL-1. Method replace/insert/delete (natural-language)
@@ -35,11 +35,11 @@ You are running in CI at the repository root. Use only the tools that are allowe
3535
- **Pass** if diffs match and verification windows show expected content.
3636

3737
## NL-2. Anchor comment insertion
38-
- Add a comment `Build marker OK` immediately above `TestSelectObjectToPlace` attribute line.
39-
- **Pass** if the comment appears directly above `[ContextMenu("Test SelectObjectToPlace")]`.
38+
- Add a comment `Build marker OK` immediately above the `Update` method.
39+
- **Pass** if the comment appears directly above the `public void Update()` line.
4040

4141
## NL-3. End-of-class insertion
42-
- Insert a 3-line comment `Tail test A/B/C` before the last method (preview, then apply).
42+
- Insert a 3-line comment `Tail test A/B/C` before the last method or immediately before the final class brace (preview, then apply).
4343
- **Pass** if windowed read shows the three lines at the intended location.
4444

4545
## NL-4. Compile trigger
@@ -56,9 +56,9 @@ You are running in CI at the repository root. Use only the tools that are allowe
5656
- Identify `HasTarget` body lines; single `replace_range` to change only inside braces; then revert.
5757
- **Pass** on exact-range change + revert.
5858

59-
## T-C. Attribute preservation
60-
- For `DumpTargetingSnapshot`, change only interior `Debug.Log` lines via `replace_range`; attributes must remain untouched (inline or previous-line variants).
61-
- **Pass** if attributes unchanged.
59+
## T-C. Header/region preservation
60+
- For `ApplyBlend`, change only interior lines via `replace_range`; the method signature and surrounding `#region`/`#endregion` markers must remain untouched.
61+
- **Pass** if signature and region markers unchanged.
6262

6363
## T-D. End-of-class insertion (anchor)
6464
- Find final class brace; `position: before` to append a temporary helper; then remove.
@@ -73,8 +73,8 @@ You are running in CI at the repository root. Use only the tools that are allowe
7373
- **Pass** if either all 3 apply or none.
7474

7575
## T-G. Path normalization
76-
- Run the same edit once with `unity://path/Assets/...` and once with `Assets/...` (if supported).
77-
- **Pass** if both target the same file and no `Assets/Assets` duplication.
76+
- Run the same edit once with `unity://path/ClaudeTests/longUnityScript-claudeTest.cs` and once with `ClaudeTests/longUnityScript-claudeTest.cs` (if supported).
77+
- **Pass** if both target the same file and no path duplication.
7878

7979
## T-H. Validation levels
8080
- After edits, run `validate` with `level: "standard"`, then `"basic"` for temporarily unbalanced text ops; final state must be valid.

0 commit comments

Comments
 (0)