You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With #668 we got basic support for textDocument/documentSymbol, which means we currently iterate over all the "public", or top-level identifiers, like packages, rules, functions. This seems to cover the most important aspect of this, which is to quickly get an overview of a package and to be able to navigate around between the rules/functions. Regal doesn't settle for basic though, so we should look into extending our coverage here, and include things like values and symbols inside of rule bodies.
I think objects and arrays in particular would be helpful, as it would be helpful to see where in deeply nested constructs the cursor is. But I'm sure we'll find other useful aspects of this as we make progress.
The text was updated successfully, but these errors were encountered:
One thing to note is that the implementation of workspace/symbol fetches document symbols from each document. We probably don't want things like values to be seen in the workspace symbol search, so filtering out anything but the top level attributes from there is probably a good idea.
With #668 we got basic support for
textDocument/documentSymbol
, which means we currently iterate over all the "public", or top-level identifiers, like packages, rules, functions. This seems to cover the most important aspect of this, which is to quickly get an overview of a package and to be able to navigate around between the rules/functions. Regal doesn't settle for basic though, so we should look into extending our coverage here, and include things like values and symbols inside of rule bodies.I think objects and arrays in particular would be helpful, as it would be helpful to see where in deeply nested constructs the cursor is. But I'm sure we'll find other useful aspects of this as we make progress.
The text was updated successfully, but these errors were encountered: