Skip to content

Add grid/table cell addressing by row/col#335

Merged
JE-Chen merged 1 commit into
devfrom
feat/grid-locator-batch
Jun 22, 2026
Merged

Add grid/table cell addressing by row/col#335
JE-Chen merged 1 commit into
devfrom
feat/grid-locator-batch

Conversation

@JE-Chen

@JE-Chen JE-Chen commented Jun 22, 2026

Copy link
Copy Markdown
Member

Summary

Round-11, automation-core (locate lane). anchor_locator does pairwise spatial relations but nothing addresses a 2-D grid — "the cell at row 3, column 2" of a table.

Layers

  • Headless core: je_auto_control/utils/grid_locator/grid_locator.pycluster_grid (boxes → rows by centre-y within row_tolerance, cells by centre-x) and locate_cell (centre of the 0-based (row,col) cell). Pure; box enumeration stays the caller's job (no real screen). No PySide6.
  • Facade: both re-exported + __all__.
  • Executor: AC_grid_cell (JSON boxes + row/col).
  • MCP: ac_grid_cell.
  • Script Builder: one CommandSpec under Mouse.

Tests / docs

  • test/unit_test/headless/test_grid_locator_batch.py — 8 headless tests (cluster rows/cols, cell centre, out-of-range, row_tolerance merge/split, empty + executor round-trip + wiring + facade). Fully pure.
  • EN/Zh feature docs v125_features_doc.rst + toctrees; 3 WHATS_NEW entries.

Note: Codacy may show action_required from its coverage gate only (issues=0); SonarCloud + all GitHub Actions are the substantive gates.

@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@JE-Chen JE-Chen merged commit fbce852 into dev Jun 22, 2026
15 of 16 checks passed
@JE-Chen JE-Chen deleted the feat/grid-locator-batch branch June 22, 2026 18:28
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant