style: fix types

Author: gajus

.eslintrc

@@ -1,6 +1,7 @@
 {
   "extends": [
-    "canonical"
+    "canonical",
+    "canonical/flowtype"
   ],
   "root": true
 }

src/conditionalClassMerge.js

@@ -6,6 +6,8 @@ import {
   stringLiteral
 } from 'babel-types';
 
+/* eslint-disable flowtype/no-weak-types */
+
 export default (
   classNameExpression: any,
   styleNameExpression: any,

src/index.js

@@ -104,7 +104,7 @@ export default ({
     }
   };
 
-  const getTargetResourcePath = (path: Object, stats:Object) => {
+  const getTargetResourcePath = (path: *, stats: *) => {
     const targetFileDirectoryPath = dirname(stats.file.opts.filename);
 
     if (path.node.source.value.startsWith('.')) {
@@ -114,7 +114,7 @@ export default ({
     return require.resolve(path.node.source.value);
   };
 
-  const notForPlugin = (path: Object, stats: Object) => {
+  const notForPlugin = (path: *, stats: *) => {
     stats.opts.filetypes = stats.opts.filetypes || {};
 
     const extension = path.node.source.value.lastIndexOf('.') > -1 ? path.node.source.value.substr(path.node.source.value.lastIndexOf('.')) : null;
@@ -133,7 +133,7 @@ export default ({
   return {
     inherits: babelPluginJsxSyntax,
     visitor: {
-      ImportDeclaration (path: Object, stats: Object): void {
+      ImportDeclaration (path: *, stats: *): void {
         if (notForPlugin(path, stats)) {
           return;
         }
@@ -169,7 +169,7 @@ export default ({
           path.remove();
         }
       },
-      JSXElement (path: Object, stats: Object): void {
+      JSXElement (path: *, stats: *): void {
         const filename = stats.file.opts.filename;
         const styleNameAttribute = path.node.openingElement.attributes
           .find((attribute) => {
@@ -203,7 +203,7 @@ export default ({
           );
         }
       },
-      Program (path: Object, stats: Object): void {
+      Program (path: *, stats: *): void {
         if (!validate(stats.opts)) {
           // eslint-disable-next-line no-console
           console.error(validate.errors);

src/replaceJsxExpressionContainer.js

@@ -13,6 +13,7 @@ import conditionalClassMerge from './conditionalClassMerge';
 
 export default (
   t: BabelTypes,
+  // eslint-disable-next-line flowtype/no-weak-types
   path: Object,
   styleNameAttribute: JSXAttribute,
   importedHelperIndentifier: Identifier,

src/requireCssModule.js

@@ -18,20 +18,24 @@ import type {
   StyleModuleMapType
 } from './types';
 
-type FileTypeOptions = {|
+type FiletypeOptionsType = {|
   +syntax: string,
-  // eslint-disable-next-line no-undef
   +plugins?: $ReadOnlyArray<string>
 |};
 
-const getFiletypeOptions = (cssSourceFilePath: string, filetypes: {[key: string]: FileTypeOptions}): ?FileTypeOptions => {
+type FiletypesConfigurationType = {
+  [key: string]: FiletypeOptionsType
+};
+
+const getFiletypeOptions = (cssSourceFilePath: string, filetypes: FiletypesConfigurationType): ?FiletypeOptionsType => {
   const extension = cssSourceFilePath.substr(cssSourceFilePath.lastIndexOf('.'));
   const filetype = filetypes ? filetypes[extension] : null;
 
   return filetype;
 };
 
-const getSyntax = (filetypeOptions: FileTypeOptions): ?(Function|Object) => {
+// eslint-disable-next-line flowtype/no-weak-types
+const getSyntax = (filetypeOptions: FiletypeOptionsType): ?(Function | Object) => {
   if (!filetypeOptions || !filetypeOptions.syntax) {
     return null;
   }
@@ -40,8 +44,8 @@ const getSyntax = (filetypeOptions: FileTypeOptions): ?(Function|Object) => {
   return require(filetypeOptions.syntax);
 };
 
-// eslint-disable-next-line no-undef
-const getExtraPlugins = (filetypeOptions: ?FileTypeOptions): $ReadOnlyArray<any> => {
+// eslint-disable-next-line flowtype/no-weak-types
+const getExtraPlugins = (filetypeOptions: ?FiletypeOptionsType): $ReadOnlyArray<*> => {
   if (!filetypeOptions || !filetypeOptions.plugins) {
     return [];
   }
@@ -52,7 +56,8 @@ const getExtraPlugins = (filetypeOptions: ?FileTypeOptions): $ReadOnlyArray<any>
   });
 };
 
-const getTokens = (runner, cssSourceFilePath: string, filetypeOptions: ?FileTypeOptions): StyleModuleMapType => {
+const getTokens = (runner, cssSourceFilePath: string, filetypeOptions: ?FiletypeOptionsType): StyleModuleMapType => {
+  // eslint-disable-next-line flowtype/no-weak-types
   const options: Object = {
     from: cssSourceFilePath
   };
@@ -75,8 +80,8 @@ const getTokens = (runner, cssSourceFilePath: string, filetypeOptions: ?FileType
 };
 
 type OptionsType = {|
-  filetypes: Object,
   generateScopedName?: string,
+  filetypes: FiletypesConfigurationType,
   context?: string
 |};
 

src/resolveStringLiteral.js

@@ -15,7 +15,7 @@ import type {
 /**
  * Updates the className value of a JSX element using a provided styleName attribute.
  */
-export default (path: Object, styleModuleImportMap: StyleModuleImportMapType, styleNameAttribute: JSXAttribute): void => {
+export default (path: *, styleModuleImportMap: StyleModuleImportMapType, styleNameAttribute: JSXAttribute): void => {
   const classNameAttribute = path.node.openingElement.attributes
     .find((attribute) => {
       return typeof attribute.name !== 'undefined' && attribute.name.name === 'className';

src/schemas/optionsSchema.json

@@ -11,19 +11,19 @@
       "additionalProperties": false,
       "patternProperties": {
         "\\..*": {
-          "type": "object",
           "additionalProperties": false,
           "properties": {
-            "syntax": {
-              "type": "string"
-            },
             "plugins": {
-              "type": "array",
               "items": {
                 "type": "string"
-              }
+              },
+              "type": "array"
+            },
+            "syntax": {
+              "type": "string"
             }
-          }
+          },
+          "type": "object"
         }
       },
       "type": "object"
@@ -39,4 +39,4 @@
     }
   },
   "type": "object"
-}
+}