Pre-beta changes

[blaine-arcjet]

• Remove createRemoteClient in arcjet and defer to adapters #46
• Remove rateLimit alias before 1.0.0 #191
• Remove logger dep from arcjet core #922 (only technically breaking since it could be deduped into the tree)
• Context should be frozen #930
• Remove ability to construct multiple rules with one constructor #1397
• Missing refillRate on TokenBucket configuration causes error #992
• Refactor analyze into a wasm loader package and a logic package #1448
• Consider making validate an async function #1391
• Validate characteristics #1801
• Remove match from rate limit rules #1810
• Remove Enum-like objects #1833
• Rename validateEmail options to align with detectBot and sensitiveInfo options #1834
• Use kebab-case instead of SCREAMING_SNAKE_CASE for our string unions #1835
• Consider removing TS files from published packages #1836
• Connect RPC & Buf v2 upgrades
  • deps: bump @bufbuild/protobuf from 1.10.0 to 2.2.2 #2133
  • deps: Bump @connectrpc/connect-web from 1.6.1 to 2.0.0 #2286
  • deps: Bump @connectrpc/connect from 1.6.1 to 2.0.0 #2288
  • deps: Bump @connectrpc/connect-node from 1.6.1 to 2.0.0 #2290

[davidmytton]

All these make sense to me except #539 unless we also do #782. eslint 9 has version conflicts with other packages so we should avoid forcing a breaking change release just for our internal packages.

[blaine-arcjet]

I'm fine to skip the eslint upgrade but I think we're in agreement to decouple the version numbers of our devDeps so we can make breaking changes in the future without affecting the SDK version numbers.

[davidmytton]

Yes, agreed.

[blaine-arcjet]

We're putting this on hold until we're ready to approach beta again.

[e-moran]

We should rename the block field in validateEmail to align with the allow/deny naming of other rules.

[blaine-arcjet]

I've been discussing with @e-moran about making the wholesale switch from SCREAMING_SNAKE_CASE to kebab-case for our enum-like values. This would match the output of jco and require less translation when calling Wasm.

[blaine-arcjet]

Remove the ability to construct multiple rules with one constructor: #1397

[e-moran]

I've been discussing with @e-moran about making the wholesale switch from SCREAMING_SNAKE_CASE to kebab-case for our enum-like values. This would match the output of jco and require less translation when calling Wasm.

Do you think we should do this before the beta as its a big breaking change? I could have time to do it at the end of the week after the docs.

[blaine-arcjet]

@e-moran I think we need to make this backwards compatible with warning logs for deprecated format. Would you be able to do that in the timeline suggested?

[blaine-arcjet]

I think we should also consider removing the .ts files from our published packages. It is becoming frustrating that some frameworks/build tools process the .ts files even when our package manifest indicates that we provide the .js and .d.ts files.

[blaine-arcjet]

I believe I've migrated all suggestions in this thread to issues and update the root checklist.