Reimplement dangerous request parameter check with visitor

[simolus3]

The usesAuthenticatedRequestParameters and usesUnauthenticatedRequestParameters properties are currently computed whenever we're constructing or composing a clause in the sync-rules compiler.

While that's a fairly simple implementation, it doesn't scale well if we're trying to infer more information about used clauses in the future. In particular, I'm looking at a lint that would require knowing whether a query uses stream parameters. Adding that information (or more features in the future) to all clauses would complicate the implementation.

So, this refactors the check to use the visitor pattern:

• When a clause has child clauses, it exposes a function for traversing them.
• The request function and parameter table ref clauses have special identifiers to recognize them.
• A visitor traverses all clauses to check whether authenticated or unauthenticated request parameters are used.
• The ORcombinator for parameter match clauses also has a special key so that we can apply the special behavior (it only counts if both sides are using authenticated parameters).

[changeset-bot]

🦋 Changeset detected

Latest commit: d4df5b8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages

Name	Type
@powersync/service-sync-rules	Patch
@powersync/service-jpgwire	Patch
@powersync/service-core-tests	Patch
@powersync/service-core	Patch
@powersync/lib-services-framework	Patch
@powersync/service-module-mongodb-storage	Patch
@powersync/service-module-mongodb	Patch
@powersync/service-module-mysql	Patch
@powersync/service-module-postgres-storage	Patch
@powersync/service-module-postgres	Patch
@powersync/lib-service-postgres	Patch
@powersync/service-module-core	Patch
@powersync/service-image	Patch
test-client	Patch
@powersync/service-rsocket-router	Patch
@powersync/lib-service-mongodb	Patch
@powersync/service-schema	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR