Skip to content

Resolve captured MyGo input to current retained node identities - #277

Closed
tiensonqin wants to merge 2 commits into
mainfrom
feature/mygo-document-capture
Closed

tiensonqin wants to merge 2 commits into
mainfrom
feature/mygo-document-capture

Conversation

@tiensonqin

@tiensonqin tiensonqin commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Add Backend.InputNode(ui.InputTarget) to resolve MyGo's actual committed hit/focus ancestry to the nearest retained LUI host binding. InputNode.Current() checks visibility and retained incarnation, so delayed pointer actions cannot target a different node that reused the same numeric ID. No geometry inference or private runtime access.

Pin the published MyGo capture API from logseq/mygo#1 at v0.0.0-20261025203821-7875529a3c4d (the exact pinned parity lineage, not upstream main). The integration owner can subsequently combine capture/geometry/theme APIs and update to that reachable lineage. docs/api-migrations.md documents the new public contract and duplicate-forwarding rule.

Validation in platform/mygo, without a workspace or temporary replacements (GOWORK=off): CGO_ENABLED=0 go test ./..., go vet ./..., and go build ./... passed against the published pin. go test -race -count=1 ./... also passed. No GUI testing was performed.

Link to Devin session: https://app.devin.ai/sessions/5024f3589d8b4e5d87cb89b53327e279
Open in Devin Desktop: https://app.devin.ai/desktop/session/5024f3589d8b4e5d87cb89b53327e279?variant=devin
Requested by: @tiensonqin

@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Closed as superseded by #276, which includes the capture API from this branch together with the theme/geometry APIs and a combined MyGo dependency pin. The published capture-only pin in this branch passed GOWORK=off tests, race tests, vet and build; the integration PR now owns subsequent CI and pin updates.

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