refactor: types for withTheme

docs/3.x upgrade guide.md

@@ -15,3 +15,25 @@ For a slightly more elaborate setup, [@babel/preset-env](https://babeljs.io/docs
 From `@babel/preset-env`'s docs
 
 > We leverage [`browserslist`, `compat-table`, and `electron-to-chromium`] to maintain mappings of which version of our supported target environments gained support of a JavaScript syntax or browser feature, as well as a mapping of those syntaxes and features to Babel transform plugins and core-js polyfills.
+
+### Typescript usage of withTheme
+
+The typings for `withTheme` have been updated to more accurately reflect its nature as a factory function that returns a `Form` component and to properly type the ref forwarded to this component. See more about [the changes here](https://github.com/rjsf-team/react-jsonschema-form/pull/2279)
+
+If you're currently using `withTheme`, the typing for the formData was always `any`, as the generated Form component was not able to accept a generic argument.
+
+In v3.x these typings will properly infer formData if no type is given, or validate it if one is, similar to how the exported Form from core would work.
+
+You can maintain the old behavior of `withTheme`, if necessary, by changing usage of your ThemedForm to override the generic for formData with `any`.
+
+```tsx 
+const schema = { ... }
+const ThemedForm = withTheme({ ... })
+const formData = { ... }
+
+// change this
+<ThemedForm schema={schema} formData={formData} />
+
+// to this to override the formData type with any and maintain the v2 behavior
+<ThemedForm<any> schema={schema} formData={formData} />
+```

packages/bootstrap-4/src/Form/Form.tsx

@@ -1,10 +1,7 @@
-import { withTheme, FormProps } from "@rjsf/core";
-
+import CoreForm from "@rjsf/core";
+import { withTheme } from "@rjsf/core";
 import Theme from "../Theme";
-import { StatelessComponent } from "react";
 
-const Form:
-  | React.ComponentClass<FormProps<any>>
-  | StatelessComponent<FormProps<any>> = withTheme(Theme);
+const Form: typeof CoreForm = withTheme(Theme);
 
 export default Form;

packages/bootstrap-4/test/Form.test.tsx

@@ -1,8 +1,8 @@
-import React from "react";
-import Form from "../src/index";
+import CoreForm, { UiSchema } from "@rjsf/core";
 import { JSONSchema7 } from "json-schema";
+import React from "react";
 import renderer from "react-test-renderer";
-import { UiSchema } from "@rjsf/core";
+import Form from "../src/index";
 
 describe("single fields", () => {
   describe("string field", () => {

packages/core/index.d.ts

@@ -61,6 +61,7 @@ declare module '@rjsf/core' {
         onChange: (formData: T, newErrorSchema: ErrorSchema) => void;
         onBlur: (id: string, value: boolean | number | string | null) => void;
         submit: () => void;
+        formElement: HTMLFormElement | null;
     }
 
     export type UiSchema = {
@@ -274,7 +275,7 @@ declare module '@rjsf/core' {
 
     export function withTheme<T = any>(
         themeProps: ThemeProps<T>,
-    ): React.ComponentClass<FormProps<T>> | React.StatelessComponent<FormProps<T>>;
+    ): typeof Form;
 
     export type AddButtonProps = {
         className: string;
@@ -436,8 +437,7 @@ declare module '@rjsf/core' {
 }
 
 declare module '@rjsf/core/lib/components/fields/SchemaField' {
-    import { JSONSchema7 } from 'json-schema';
-    import { FieldProps, UiSchema, IdSchema, FormValidation } from '@rjsf/core';
+    import { FieldProps } from '@rjsf/core';
 
     export type SchemaFieldProps<T = any> = Pick<
         FieldProps<T>,

packages/core/package.json

@@ -15,9 +15,10 @@
     "publish-to-npm": "npm run build && npm publish",
     "start": "concurrently \"npm:build:* -- --watch\"",
     "tdd": "cross-env NODE_ENV=test mocha --require @babel/register --watch --require ./test/setup-jsdom.js test/**/*_test.js",
-    "test": "cross-env BABEL_ENV=test NODE_ENV=test mocha --require @babel/register --require ./test/setup-jsdom.js test/**/*_test.js",
+    "test": "concurrently \"cross-env BABEL_ENV=test NODE_ENV=test mocha --require @babel/register --require ./test/setup-jsdom.js test/**/*_test.js\" \"npm:test-type\"",
     "test-coverage": "cross-env NODE_ENV=test nyc --reporter=lcov mocha --require @babel/register --require ./test/setup-jsdom.js test/**/*_test.js",
-    "test-debug": "cross-env NODE_ENV=test mocha --require @babel/register --require ./test/setup-jsdom.js --debug-brk --inspect test/Form_test.js"
+    "test-debug": "cross-env NODE_ENV=test mocha --require @babel/register --require ./test/setup-jsdom.js --debug-brk --inspect test/Form_test.js",
+    "test-type": "tsc"
   },
   "lint-staged": {
     "{src,test}/**/*.js": [
@@ -100,6 +101,7 @@
     "rimraf": "^2.5.4",
     "sinon": "^9.0.2",
     "style-loader": "^0.13.1",
+    "typescript": "^4.2.3",
     "webpack": "^4.42.1",
     "webpack-cli": "^3.1.2",
     "webpack-dev-middleware": "^3.4.0",

packages/core/tsconfig.json

@@ -0,0 +1,25 @@
+{
+  "compilerOptions": {
+    "module": "commonjs",
+    "lib": [
+      "es6",
+      "dom"
+    ],
+    "noImplicitAny": true,
+    "noImplicitThis": true,
+    "strictNullChecks": true,
+    "strictFunctionTypes": true,
+    "baseUrl": "../",
+    "typeRoots": [
+      "../"
+    ],
+    "types": [],
+    "noEmit": true,
+    "forceConsistentCasingInFileNames": true,
+    "jsx": "react"
+  },
+  "files": [
+    "index.d.ts",
+    "typing_test.tsx"
+  ]
+}