[move][move-ide] Add cursor support and tests

cgswords commented 3 months ago

Description

This updates the move analyzer to have a notion of a cursor and partially revises autocompletion request handling to use that cursor. This is just a start, but hopefully paves the way for further and better autocomplete suggestions in the future.

This PR also makes a few smaller changes:

FileByteSize and FilePosition now hold a FileHash, not a FileId.
Added a way to look up a file hash from a file path on MappedFiles
Cleaned up some of the move analyzer's Symbols initialization code.
We now carry loc information on struct and enum definitions through typing (to determine if the cursor is there).

Test plan

A new test suite for cursors in the analyzer.
A new test for the autocomplete suggestion of id: UID when writing a struct with key, which appeared to be previously untested.

Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

vercel[bot] commented 3 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
sui-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 9, 2024 1:04am

3 Skipped Deployments

cgswords commented 2 months ago

Stamping but @todd-mystenlabs should probably take a look before this lands

I think you meant to tag @tnowacki

MystenLabs / sui