Use native scheduler if defined

.codesandbox/ci.json

@@ -1,7 +1,7 @@
 {
   "packages": ["packages/react", "packages/react-dom", "packages/scheduler"],
   "buildCommand": "download-build-in-codesandbox-ci",
-  "node": "14",
+  "node": "18",
   "publishDirectory": {
     "react": "build/oss-experimental/react",
     "react-dom": "build/oss-experimental/react-dom",

.eslintrc.js

@@ -8,23 +8,26 @@ const {
 const restrictedGlobals = require('confusing-browser-globals');
 
 const OFF = 0;
+const WARNING = 1;
 const ERROR = 2;
 
 module.exports = {
-  extends: ['fbjs', 'prettier'],
+  extends: ['prettier'],
 
   // Stop ESLint from looking for a configuration file in parent folders
   root: true,
 
   plugins: [
+    'babel',
+    'ft-flow',
     'jest',
     'no-for-of-loops',
     'no-function-declare-after-return',
     'react',
     'react-internal',
   ],
 
-  parser: 'babel-eslint',
+  parser: 'hermes-eslint',
   parserOptions: {
     ecmaVersion: 9,
     sourceType: 'script',
@@ -33,6 +36,190 @@ module.exports = {
   // We're stricter than the default config, mostly. We'll override a few rules
   // and then enable some React specific ones.
   rules: {
+    'ft-flow/array-style-complex-type': [OFF, 'verbose'],
+    'ft-flow/array-style-simple-type': [OFF, 'verbose'], // TODO should be WARNING
+    'ft-flow/boolean-style': ERROR,
+    'ft-flow/no-dupe-keys': ERROR,
+    'ft-flow/no-primitive-constructor-types': ERROR,
+    'ft-flow/no-types-missing-file-annotation': OFF, // TODO should be ERROR
+    'ft-flow/no-unused-expressions': ERROR,
+    // 'ft-flow/no-weak-types': WARNING,
+    // 'ft-flow/require-valid-file-annotation': ERROR,
+
+    'no-cond-assign': OFF,
+    'no-constant-condition': OFF,
+    'no-control-regex': OFF,
+    'no-debugger': ERROR,
+    'no-dupe-args': ERROR,
+    'no-dupe-keys': ERROR,
+    'no-duplicate-case': WARNING,
+    'no-empty-character-class': WARNING,
+    'no-empty': OFF,
+    'no-ex-assign': WARNING,
+    'no-extra-boolean-cast': WARNING,
+    'no-func-assign': ERROR,
+    'no-invalid-regexp': WARNING,
+    'no-irregular-whitespace': WARNING,
+    'no-negated-in-lhs': ERROR,
+    'no-obj-calls': ERROR,
+    'no-regex-spaces': WARNING,
+    'no-sparse-arrays': ERROR,
+    'no-unreachable': ERROR,
+    'use-isnan': ERROR,
+    'valid-jsdoc': OFF,
+    'block-scoped-var': OFF,
+    complexity: OFF,
+    'default-case': OFF,
+    'guard-for-in': OFF,
+    'no-alert': OFF,
+    'no-caller': ERROR,
+    'no-case-declarations': OFF,
+    'no-div-regex': OFF,
+    'no-else-return': OFF,
+    'no-empty-pattern': WARNING,
+    'no-eq-null': OFF,
+    'no-eval': ERROR,
+    'no-extend-native': WARNING,
+    'no-extra-bind': WARNING,
+    'no-fallthrough': WARNING,
+    'no-implicit-coercion': OFF,
+    'no-implied-eval': ERROR,
+    'no-invalid-this': OFF,
+    'no-iterator': OFF,
+    'no-labels': [ERROR, {allowLoop: true, allowSwitch: true}],
+    'no-lone-blocks': WARNING,
+    'no-loop-func': OFF,
+    'no-magic-numbers': OFF,
+    'no-multi-str': ERROR,
+    'no-native-reassign': [ERROR, {exceptions: ['Map', 'Set']}],
+    'no-new-func': ERROR,
+    'no-new': WARNING,
+    'no-new-wrappers': WARNING,
+    'no-octal-escape': WARNING,
+    'no-octal': WARNING,
+    'no-param-reassign': OFF,
+    'no-process-env': OFF,
+    'no-proto': ERROR,
+    'no-redeclare': OFF, // TODO should be WARNING?
+    'no-return-assign': OFF,
+    'no-script-url': ERROR,
+    'no-self-compare': WARNING,
+    'no-sequences': WARNING,
+    'no-throw-literal': ERROR,
+    'no-useless-call': WARNING,
+    'no-void': OFF,
+    'no-warning-comments': OFF,
+    'no-with': OFF,
+    radix: WARNING,
+    'vars-on-top': OFF,
+    yoda: OFF,
+    'init-declarations': OFF,
+    'no-catch-shadow': ERROR,
+    'no-delete-var': ERROR,
+    'no-label-var': WARNING,
+    'no-shadow-restricted-names': WARNING,
+    'no-undef-init': OFF,
+    'no-undef': ERROR,
+    'no-undefined': OFF,
+    'callback-return': OFF,
+    'global-require': OFF,
+    'handle-callback-err': OFF,
+    'no-mixed-requires': OFF,
+    'no-new-require': OFF,
+    'no-path-concat': OFF,
+    'no-process-exit': OFF,
+    'no-restricted-modules': OFF,
+    'no-sync': OFF,
+    camelcase: [OFF, {properties: 'always'}],
+    'consistent-this': [OFF, 'self'],
+    'func-names': OFF,
+    'func-style': [OFF, 'declaration'],
+    'id-length': OFF,
+    'id-match': OFF,
+    'max-depth': OFF,
+    'max-nested-callbacks': OFF,
+    'max-params': OFF,
+    'max-statements': OFF,
+    'new-cap': OFF,
+    'newline-after-var': OFF,
+    'no-array-constructor': ERROR,
+    'no-continue': OFF,
+    'no-inline-comments': OFF,
+    'no-lonely-if': OFF,
+    'no-negated-condition': OFF,
+    'no-nested-ternary': OFF,
+    'no-new-object': WARNING,
+    'no-plusplus': OFF,
+    'no-ternary': OFF,
+    'no-underscore-dangle': OFF,
+    'no-unneeded-ternary': WARNING,
+    'one-var': [WARNING, {initialized: 'never'}],
+    'operator-assignment': [WARNING, 'always'],
+    'require-jsdoc': OFF,
+    'sort-vars': OFF,
+    'spaced-comment': [
+      OFF,
+      'always',
+      {exceptions: ['jshint', 'jslint', 'eslint', 'global']},
+    ],
+    'constructor-super': ERROR,
+    'no-class-assign': WARNING,
+    'no-const-assign': ERROR,
+    'no-dupe-class-members': ERROR,
+    'no-this-before-super': ERROR,
+    'object-shorthand': OFF,
+    'prefer-const': OFF,
+    'prefer-spread': OFF,
+    'prefer-reflect': OFF,
+    'prefer-template': OFF,
+    'require-yield': OFF,
+    'babel/generator-star-spacing': OFF,
+    'babel/new-cap': OFF,
+    'babel/array-bracket-spacing': OFF,
+    'babel/object-curly-spacing': OFF,
+    'babel/object-shorthand': OFF,
+    'babel/arrow-parens': OFF,
+    'babel/no-await-in-loop': OFF,
+    'babel/flow-object-type': OFF,
+    'react/display-name': OFF,
+    'react/forbid-prop-types': OFF,
+    'react/jsx-closing-bracket-location': OFF,
+    'react/jsx-curly-spacing': OFF,
+    'react/jsx-equals-spacing': WARNING,
+    'react/jsx-filename-extension': OFF,
+    'react/jsx-first-prop-new-line': OFF,
+    'react/jsx-handler-names': OFF,
+    'react/jsx-indent': OFF,
+    'react/jsx-indent-props': OFF,
+    'react/jsx-key': OFF,
+    'react/jsx-max-props-per-line': OFF,
+    'react/jsx-no-bind': OFF,
+    'react/jsx-no-duplicate-props': ERROR,
+    'react/jsx-no-literals': OFF,
+    'react/jsx-no-target-blank': OFF,
+    'react/jsx-pascal-case': OFF,
+    'react/jsx-sort-props': OFF,
+    'react/jsx-uses-vars': ERROR,
+    'react/no-comment-textnodes': OFF,
+    'react/no-danger': OFF,
+    'react/no-deprecated': OFF,
+    'react/no-did-mount-set-state': OFF,
+    'react/no-did-update-set-state': OFF,
+    'react/no-direct-mutation-state': OFF,
+    'react/no-multi-comp': OFF,
+    'react/no-render-return-value': OFF,
+    'react/no-set-state': OFF,
+    'react/no-string-refs': OFF,
+    'react/no-unknown-property': OFF,
+    'react/prefer-es6-class': OFF,
+    'react/prefer-stateless-function': OFF,
+    'react/prop-types': OFF,
+    'react/require-extension': OFF,
+    'react/require-optimization': OFF,
+    'react/require-render-return': OFF,
+    'react/sort-comp': OFF,
+    'react/sort-prop-types': OFF,
+
     'accessor-pairs': OFF,
     'brace-style': [ERROR, '1tbs'],
     'consistent-return': OFF,
@@ -51,7 +238,6 @@ module.exports = {
     'no-restricted-globals': [ERROR].concat(restrictedGlobals),
     'no-restricted-syntax': [ERROR, 'WithStatement'],
     'no-shadow': ERROR,
-    'no-unused-expressions': ERROR,
     'no-unused-vars': [ERROR, {args: 'none'}],
     'no-use-before-define': OFF,
     'no-useless-concat': OFF,
@@ -74,8 +260,6 @@ module.exports = {
     // deal. But I turned it off because loading the plugin causes some obscure
     // syntax error and it didn't seem worth investigating.
     'max-len': OFF,
-    // Prettier forces semicolons in a few places
-    'flowtype/object-type-delimiter': OFF,
 
     // React & JSX
     // Our transforms set this automatically
@@ -141,6 +325,7 @@ module.exports = {
         'packages/react-native-renderer/**/*.js',
         'packages/eslint-plugin-react-hooks/**/*.js',
         'packages/jest-react/**/*.js',
+        'packages/internal-test-utils/**/*.js',
         'packages/**/__tests__/*.js',
         'packages/**/npm/*.js',
       ],
@@ -166,7 +351,7 @@ module.exports = {
       // We apply these settings to the source files that get compiled.
       // They can use all features including JSX (but shouldn't use `var`).
       files: esNextPaths,
-      parser: 'babel-eslint',
+      parser: 'hermes-eslint',
       parserOptions: {
         ecmaVersion: 8,
         sourceType: 'module',
@@ -204,14 +389,6 @@ module.exports = {
           ERROR,
           {isProductionUserAppCode: false},
         ],
-
-        // Disable accessibility checks
-        'jsx-a11y/aria-role': OFF,
-        'jsx-a11y/no-noninteractive-element-interactions': OFF,
-        'jsx-a11y/no-static-element-interactions': OFF,
-        'jsx-a11y/role-has-required-aria-props': OFF,
-        'jsx-a11y/no-noninteractive-tabindex': OFF,
-        'jsx-a11y/tabindex-no-positive': OFF,
       },
     },
     {
@@ -253,10 +430,71 @@ module.exports = {
     },
   ],
 
+  env: {
+    browser: true,
+    es6: true,
+    node: true,
+    jest: true,
+  },
+
   globals: {
+    $Call: 'readonly',
+    $ElementType: 'readonly',
+    $Flow$ModuleRef: 'readonly',
+    $FlowFixMe: 'readonly',
+    $Keys: 'readonly',
+    $NonMaybeType: 'readonly',
+    $PropertyType: 'readonly',
+    $ReadOnly: 'readonly',
+    $ReadOnlyArray: 'readonly',
+    $Shape: 'readonly',
+    AnimationFrameID: 'readonly',
+    // For Flow type annotation. Only `BigInt` is valid at runtime.
+    bigint: 'readonly',
+    BigInt: 'readonly',
+    Class: 'readonly',
+    ClientRect: 'readonly',
+    CopyInspectedElementPath: 'readonly',
+    DOMHighResTimeStamp: 'readonly',
+    EventListener: 'readonly',
+    Iterable: 'readonly',
+    Iterator: 'readonly',
+    JSONValue: 'readonly',
+    JSResourceReference: 'readonly',
+    MouseEventHandler: 'readonly',
+    PropagationPhases: 'readonly',
+    PropertyDescriptor: 'readonly',
+    React$AbstractComponent: 'readonly',
+    React$Component: 'readonly',
+    React$ComponentType: 'readonly',
+    React$Config: 'readonly',
+    React$Context: 'readonly',
+    React$Element: 'readonly',
+    React$ElementConfig: 'readonly',
+    React$ElementProps: 'readonly',
+    React$ElementRef: 'readonly',
+    React$ElementType: 'readonly',
+    React$Key: 'readonly',
+    React$Node: 'readonly',
+    React$Portal: 'readonly',
+    React$Ref: 'readonly',
+    React$StatelessFunctionalComponent: 'readonly',
+    ReadableStreamController: 'readonly',
+    RequestInfo: 'readonly',
+    RequestOptions: 'readonly',
+    ResponseState: 'readonly',
+    StoreAsGlobal: 'readonly',
+    symbol: 'readonly',
+    SyntheticEvent: 'readonly',
+    SyntheticMouseEvent: 'readonly',
+    Thenable: 'readonly',
+    TimeoutID: 'readonly',
+    WheelEventHandler: 'readonly',
+
     spyOnDev: 'readonly',
     spyOnDevAndProd: 'readonly',
     spyOnProd: 'readonly',
+    __DEV__: 'readonly',
     __EXPERIMENTAL__: 'readonly',
     __EXTENSION__: 'readonly',
     __PROFILE__: 'readonly',

.github/PULL_REQUEST_TEMPLATE.md

@@ -9,7 +9,7 @@
   3. If you've fixed a bug or added code that should be tested, add tests!
   4. Ensure the test suite passes (`yarn test`). Tip: `yarn test --watch TestName` is helpful in development.
   5. Run `yarn test --prod` to test in the production environment. It supports the same options as `yarn test`.
-  6. If you need a debugger, run `yarn debug-test --watch TestName`, open `chrome://inspect`, and press "Inspect".
+  6. If you need a debugger, run `yarn test --debug --watch TestName`, open `chrome://inspect`, and press "Inspect".
   7. Format your code with [prettier](https://github.com/prettier/prettier) (`yarn prettier`).
   8. Make sure your code lints (`yarn lint`). Tip: `yarn linc` to only check changed files.
   9. Run the [Flow](https://flowtype.org/) type checks (`yarn flow`).