Run tsc against generated code (connectrpc#214)

This makes sure that the generated code doesn't have any finicky TS
errors. This change would have caught connectrpc#209

Also remove workspace designation on example so it can be independently
cloned.

examples/react/basic/package.json

@@ -19,7 +19,7 @@
     "@connectrpc/connect-query": "workspace:*",
     "@connectrpc/connect-web": "^1.1.2",
     "@connectrpc/protoc-gen-connect-es": "^1.1.2",
-    "@connectrpc/protoc-gen-connect-query": "workspace:*",
+    "@connectrpc/protoc-gen-connect-query": "^0.5.1",
     "@tanstack/react-query": "^4.35.7",
     "@tanstack/react-query-devtools": "^4.35.7",
     "@testing-library/jest-dom": "^6.1.3",

packages/connect-query/src/create-unary-functions.test.ts

@@ -538,9 +538,15 @@ describe("createUnaryFunctions", () => {
         expect(onError).toHaveBeenCalledTimes(0);
         expect(consoleErrorSpy).not.toHaveBeenCalled();
 
-        await sleep(10);
+        await waitFor(
+          () => {
+            expect(result.current.error).toStrictEqual("error");
+          },
+          {
+            timeout: 300,
+          },
+        );
 
-        expect(result.current.error).toStrictEqual("error");
         expect(result.current.isError).toStrictEqual(true);
         expect(onError).toHaveBeenCalledTimes(1);
         expect(consoleErrorSpy).toHaveBeenCalledWith("error");
@@ -1191,7 +1197,14 @@ describe("createUnaryFunctions", () => {
         expect(onError).toHaveBeenCalledTimes(0);
         expect(consoleErrorSpy).not.toHaveBeenCalled();
 
-        await sleep(10);
+        await waitFor(
+          () => {
+            expect(result.current.error).toStrictEqual("error");
+          },
+          {
+            timeout: 300,
+          },
+        );
 
         expect(result.current.error).toStrictEqual("error");
         expect(result.current.isError).toStrictEqual(true);

packages/protoc-gen-connect-query/.gitignore

@@ -0,0 +1,3 @@
+# A directory that exists solely to hold the result of tsc since --noEmit doesn't discover to protability issue found
+# by https://github.com/connectrpc/connect-query-es/issues/209
+.type-dump

packages/protoc-gen-connect-query/package.json

@@ -22,13 +22,17 @@
     "clean": "rm -rf dist",
     "build": "pnpm clean && tsc --project tsconfig.build.json",
     "format": "prettier . --write --ignore-path ./.eslintignore && eslint . --fix && license-header",
-    "generate": "rm -rf snapshots/gen && cd snapshots && buf generate buf.build/connectrpc/eliza:8b8b971d6fde4dc8ba5d96f9fda7d53c"
+    "generate": "rm -rf snapshots/gen && cd snapshots && buf generate buf.build/connectrpc/eliza:8b8b971d6fde4dc8ba5d96f9fda7d53c",
+    "test": "tsc --declaration --declarationDir .type-dump --emitDeclarationOnly"
   },
   "preferUnplugged": true,
   "devDependencies": {
     "@bufbuild/buf": "1.27.0",
     "@bufbuild/protoc-gen-es": "^1.3.3",
+    "@connectrpc/connect": "^1.1.2",
+    "@connectrpc/connect-query": "workspace:*",
     "@connectrpc/protoc-gen-connect-es": "^1.1.2",
+    "@tanstack/react-query": "^4.35.7",
     "typescript": "^5.2.2"
   },
   "dependencies": {

packages/protoc-gen-connect-query/snapshots/buf.gen.yaml

@@ -13,3 +13,4 @@ plugins:
     opt:
       - target=ts+dts+js
       - import_extension=none
+      - ts_nocheck=false

packages/protoc-gen-connect-query/snapshots/gen/connectrpc/eliza/v1/eliza-ElizaService_connectquery.d.ts

@@ -12,10 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-connect-query v0.5.2 with parameter "target=ts+dts+js,import_extension=none"
+// @generated by protoc-gen-connect-query v0.5.2 with parameter "target=ts+dts+js,import_extension=none,ts_nocheck=false"
 // @generated from file connectrpc/eliza/v1/eliza.proto (package connectrpc.eliza.v1, syntax proto3)
 /* eslint-disable */
-// @ts-nocheck
 
 import {
   ConverseRequest,

packages/protoc-gen-connect-query/snapshots/gen/connectrpc/eliza/v1/eliza-ElizaService_connectquery.js

@@ -12,10 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-connect-query v0.5.2 with parameter "target=ts+dts+js,import_extension=none"
+// @generated by protoc-gen-connect-query v0.5.2 with parameter "target=ts+dts+js,import_extension=none,ts_nocheck=false"
 // @generated from file connectrpc/eliza/v1/eliza.proto (package connectrpc.eliza.v1, syntax proto3)
 /* eslint-disable */
-// @ts-nocheck
 
 import {
   ConverseRequest,

packages/protoc-gen-connect-query/snapshots/gen/connectrpc/eliza/v1/eliza-ElizaService_connectquery.ts

@@ -12,10 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// @generated by protoc-gen-connect-query v0.5.2 with parameter "target=ts+dts+js,import_extension=none"
+// @generated by protoc-gen-connect-query v0.5.2 with parameter "target=ts+dts+js,import_extension=none,ts_nocheck=false"
 // @generated from file connectrpc/eliza/v1/eliza.proto (package connectrpc.eliza.v1, syntax proto3)
 /* eslint-disable */
-// @ts-nocheck
 
 import {
   ConverseRequest,

packages/protoc-gen-connect-query/tsconfig.json

@@ -1,4 +1,7 @@
 {
   "extends": "./tsconfig.build.json",
-  "include": ["src", "snapshots"]
+  "include": ["src", "snapshots", "./*.config.ts"],
+  "compilerOptions": {
+    "lib": ["DOM", "ESNext"]
+  }
 }