Skip to content

Comments

refactor(web): convert main correction-search method to accept multiple search-spaces 🚂#14903

Merged
jahorton merged 2 commits intoepic/autocorrectfrom
refactor/web/multi-space-correction-search
Jan 28, 2026
Merged

refactor(web): convert main correction-search method to accept multiple search-spaces 🚂#14903
jahorton merged 2 commits intoepic/autocorrectfrom
refactor/web/multi-space-correction-search

Conversation

@jahorton
Copy link
Contributor

@jahorton jahorton commented Oct 7, 2025

This PR's changes allow the main suggestion-generation method to source suggestions from multiple different search-space objects, rather than just one. This, in turn, will help the engine to consider possible alternate word-breaking scenarios - though much more work is needed to actually do this.

Build-bot: skip build:web
Test-bot: skip

@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Oct 7, 2025

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

  • Web
    • KeymanWeb Test Home - build : all tests passed (no artifacts on BuildLevel "build")

@keymanapp-test-bot keymanapp-test-bot bot changed the title refactor(web): convert main correction-search method to accept multiple search-spaces refactor(web): convert main correction-search method to accept multiple search-spaces 🚂 Oct 7, 2025
@keymanapp-test-bot keymanapp-test-bot bot added this to the A19S13 milestone Oct 7, 2025
@darcywong00 darcywong00 modified the milestones: A19S13, A19S14 Oct 11, 2025
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 5032faf to 0d6ed48 Compare October 17, 2025 15:09
@jahorton jahorton changed the base branch from change/web/remove-tracker-analyze-state to feat/web/search-space-identifiers October 17, 2025 15:09
@jahorton jahorton force-pushed the feat/web/search-space-identifiers branch from 11cf559 to ff8d585 Compare October 20, 2025 15:37
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from fcab0aa to 02b9293 Compare October 20, 2025 16:08
@jahorton jahorton changed the base branch from feat/web/search-space-identifiers to refactor/web/search-batcher October 20, 2025 16:09
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 02b9293 to 96a4b99 Compare October 21, 2025 19:14
@jahorton jahorton changed the base branch from refactor/web/search-batcher to refactor/web/use-interface-as-search-parent October 21, 2025 19:14
@jahorton jahorton force-pushed the refactor/web/use-interface-as-search-parent branch from 30f660a to 604ee6c Compare October 22, 2025 21:04
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 96a4b99 to 3fe8645 Compare October 23, 2025 15:45
@jahorton jahorton force-pushed the refactor/web/use-interface-as-search-parent branch from 604ee6c to 1f69bba Compare October 23, 2025 16:11
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 3fe8645 to de3d86c Compare October 23, 2025 21:36
@darcywong00 darcywong00 modified the milestones: A19S14, A19S15 Oct 24, 2025
@jahorton jahorton force-pushed the refactor/web/use-interface-as-search-parent branch from 1f69bba to 7b67bc5 Compare October 27, 2025 20:17
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from de3d86c to 0fdd522 Compare October 27, 2025 20:20
@keyman-server keyman-server modified the milestones: A19S15, A19S16 Nov 8, 2025
@jahorton jahorton force-pushed the refactor/web/use-interface-as-search-parent branch from f0978aa to c28bb20 Compare November 10, 2025 19:21
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 0fdd522 to 9507c71 Compare November 10, 2025 19:24
@keyman-server keyman-server added this to the A19S18 milestone Dec 6, 2025
@keyman-server keyman-server modified the milestones: A19S18, A19S19 Dec 21, 2025
@keyman-server keyman-server modified the milestones: A19S19, A19S20 Jan 3, 2026
@jahorton jahorton force-pushed the refactor/web/use-interface-as-search-parent branch 7 times, most recently from 28ded16 to 3a56e23 Compare January 9, 2026 17:32
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 851fb77 to 1ee5cc2 Compare January 9, 2026 20:48
@jahorton jahorton changed the base branch from refactor/web/use-interface-as-search-parent to refactor/web/differentiate-module-types January 9, 2026 20:48
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 1ee5cc2 to c81f1f7 Compare January 14, 2026 22:12
@keyman-server keyman-server modified the milestones: A19S20, A19S21 Jan 16, 2026
Base automatically changed from refactor/web/differentiate-module-types to epic/autocorrect January 22, 2026 16:21
…le spaces

Build-bot: skip build:web
Test-bot: skip
@jahorton jahorton force-pushed the refactor/web/multi-space-correction-search branch from 52aca11 to ceabe5a Compare January 22, 2026 18:27
@jahorton jahorton marked this pull request as ready for review January 22, 2026 19:43
@jahorton jahorton requested a review from ermshiperete January 26, 2026 14:18
@jahorton jahorton merged commit cb63d93 into epic/autocorrect Jan 28, 2026
7 of 8 checks passed
@jahorton jahorton deleted the refactor/web/multi-space-correction-search branch January 28, 2026 22:20
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants