fix(core): address locate normalization review comments (#2703)

EAGzzyCSL · web-flow · commit d8c1dcd39f65 · 2026-06-23T15:13:48.000+08:00
diff --git a/packages/core/src/ai-model/workflows/planning/locate-normalization.ts b/packages/core/src/ai-model/workflows/planning/locate-normalization.ts
@@ -34,9 +34,9 @@ export function normalizePlanningActionLocateFields(
     }
 
     debug('actionInActionSpace matched', actionInActionSpace);
-    const locateFields = actionInActionSpace
-      ? findAllMidsceneLocatorField(actionInActionSpace.paramSchema)
-      : [];
+    const locateFields = findAllMidsceneLocatorField(
+      actionInActionSpace.paramSchema,
+    );
 
     debug('locateFields', locateFields);
 
diff --git a/packages/core/tests/unit-test/locate-normalization.test.ts b/packages/core/tests/unit-test/locate-normalization.test.ts
@@ -24,7 +24,7 @@ const locateResultContext = {
 };
 
 describe('normalizePlanningActionLocateFields', () => {
-  it('leaves actions unchanged when no action schema is available for locate normalization', () => {
+  it('leaves actions unchanged when the planned action is outside the action space', () => {
     const adaptPlanningParamToPixelBbox = vi.fn();
     const actions: PlanningAction[] = [
       {

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ const locateResultContext = {`
`24`	`24`	`};`
`25`	`25`
`26`	`26`	`describe('normalizePlanningActionLocateFields', () => {`
`27`		`- it('leaves actions unchanged when no action schema is available for locate normalization', () => {`
	`27`	`+ it('leaves actions unchanged when the planned action is outside the action space', () => {`
`28`	`28`	`const adaptPlanningParamToPixelBbox = vi.fn();`
`29`	`29`	`const actions: PlanningAction[] = [`
`30`	`30`	`{`