fix: move AV to a standalone file to avoid circular dependencies (#316)

React Native packager can not deel with such situation:

1. A -> B -> A
2. A export something by overwriting module.exports

B will get an empty export by `requir('A')`

fix #313

Author: leeyeh

gulpfile.babel.js

@@ -22,14 +22,12 @@ import browserify from 'browserify';
 import browserSync from 'browser-sync';
 import sourcemaps from 'gulp-sourcemaps';
 import babel from 'gulp-babel';
-import lc from './src/av';
+import { version } from './package.json';
 
 const reload = browserSync.reload;
 
 // 获取当前版本号
-const getAVVersion = () => {
-  return lc.version.replace('js', '');
-};
+const getAVVersion = () => version;
 
 const uploadCDN = (file, version, cb) => {
   qiniu.conf.ACCESS_KEY = process.env.CDN_QINIU_KEY;
@@ -65,7 +63,7 @@ gulp.task('clean-dist', () => {
 
 gulp.task('browserify', ['clean-dist'], () => {
   const bundle = browserify({
-    entries: './src/av.js',
+    entries: './src/index.js',
     standalone: 'AV'
   });
   return bundle.bundle()

package.json

@@ -1,7 +1,7 @@
 {
   "name": "leancloud-storage",
   "version": "1.1.0",
-  "main": "./dist/node/av.js",
+  "main": "./dist/node/index.js",
   "description": "LeanCloud JavaScript SDK.",
   "repository": {
     "type": "git",

src/av.js

@@ -1,50 +1,2 @@
-/*!
- * LeanCloud JavaScript SDK
- * https://leancloud.cn
- *
- * Copyright 2016 LeanCloud.cn, Inc.
- * The LeanCloud JavaScript SDK is freely distributable under the MIT license.
- */
+module.exports = global.AV || {};
 
-/**
- * 每位工程师都有保持代码优雅的义务
- * Each engineer has a duty to keep the code elegant
-**/
-
-const AV = module.exports = global.AV || {};
-AV._ = require('underscore');
-AV.version = require('./version');
-AV.Promise = require('./promise');
-AV.localStorage = require('./localstorage');
-AV.Cache = require('./cache');
-
-// All internal configuration items
-AV._config = AV._config || {};
-
-require('./utils').init(AV);
-
-require('./event')(AV);
-require('./geopoint')(AV);
-require('./acl')(AV);
-require('./op')(AV);
-require('./relation')(AV);
-require('./file')(AV);
-require('./object')(AV);
-require('./role')(AV);
-require('./user')(AV);
-require('./query')(AV);
-require('./cloudfunction')(AV);
-require('./push')(AV);
-require('./status')(AV);
-require('./search')(AV);
-require('./insight')(AV);
-
-// TODO: deprecated AV.Error()
-const AVError = require('./error');
-/**
- * @deprecated AV.Error() is deprecated, and will be removed in next release.
- */
-AV.Error = (...args) => {
-  console.warn('AV.Error() is deprecated, and will be removed in next release.');
-  new AVError(...args);
-};

src/index.js

@@ -0,0 +1,51 @@
+/*!
+ * LeanCloud JavaScript SDK
+ * https://leancloud.cn
+ *
+ * Copyright 2016 LeanCloud.cn, Inc.
+ * The LeanCloud JavaScript SDK is freely distributable under the MIT license.
+ */
+
+/**
+ * 每位工程师都有保持代码优雅的义务
+ * Each engineer has a duty to keep the code elegant
+**/
+
+const AV = module.exports = require('./av');
+
+AV._ = require('underscore');
+AV.version = require('./version');
+AV.Promise = require('./promise');
+AV.localStorage = require('./localstorage');
+AV.Cache = require('./cache');
+
+// All internal configuration items
+AV._config = AV._config || {};
+
+require('./utils').init(AV);
+
+require('./event')(AV);
+require('./geopoint')(AV);
+require('./acl')(AV);
+require('./op')(AV);
+require('./relation')(AV);
+require('./file')(AV);
+require('./object')(AV);
+require('./role')(AV);
+require('./user')(AV);
+require('./query')(AV);
+require('./cloudfunction')(AV);
+require('./push')(AV);
+require('./status')(AV);
+require('./search')(AV);
+require('./insight')(AV);
+
+// TODO: deprecated AV.Error()
+const AVError = require('./error');
+/**
+ * @deprecated AV.Error() is deprecated, and will be removed in next release.
+ */
+AV.Error = (...args) => {
+  console.warn('AV.Error() is deprecated, and will be removed in next release.');
+  new AVError(...args);
+};

test/test.js

@@ -4,7 +4,7 @@ if (typeof require !== 'undefined') {
   GLOBAL.debug = require('debug')('test');
   GLOBAL.expect = require('expect.js');
   GLOBAL.serverURL = 'http://192.168.1.216:3000';
-  GLOBAL.AV = require('../dist/node/av');
+  GLOBAL.AV = require('..');
 }
 
 // AV._config.APIServerURL = 'https://cn-stg1.avoscloud.com';