[turtle-cli] don't require push notifications cert to build an ios app

.circleci/config.yml

@@ -1,211 +1,29 @@
-version: 2.1
+version: 2
 
-executors:
-  nix: &nix
-    docker:
-      - image: lnl7/nix:2.1.2
-    # Symlink to the readline-enabled bash which is the default command of the container
-    shell: /run/current-system/sw/bin/bash -leo pipefail
-    working_directory: ~/turtle
-  mac:
-    macos: # https://circleci.com/docs/2.0/testing-ios/#supported-xcode-versions
-      xcode: '10.1.0'
-    working_directory: /Users/distiller/project
-    shell: /bin/bash -leo pipefail
-  android:
-    <<: *nix
-    resource_class: xlarge
-
-commands:
-  install_nix:
-    steps:
-      - run:
-          name: Install nix
-          command: curl https://nixos.org/releases/nix/nix-2.2.1/install | bash -s -- --no-daemon
-      - run:
-          name: Add nix-initializing line to .bashrc
-          command: echo ". ~/.nix-profile/etc/profile.d/nix.sh" >> ~/.bashrc
-
-  setup:
-    steps:
-      - run:
-          name: Install common packages
-          command: |
-            nix-env -iA \
-              nixpkgs.binutils \
-              nixpkgs.direnv \
-              nixpkgs.findutils \
-              nixpkgs.gawk \
-              nixpkgs.git \
-              nixpkgs.gnutar \
-              nixpkgs.gzip \
-              nixpkgs.openssh \
-              nixpkgs.python \
-              nixpkgs.xz
-      - run:
-          name: Configure direnv
-          command: |
-            mkdir -p ~/.config/direnv
-            echo -e "[whitelist]\nprefix = [ \"$HOME\" ]" > ~/.config/direnv/config.toml
-            echo 'eval "$(direnv export bash)"' >> ~/.bash_profile
-      - run:
-          name: Configure yarn
-          command: echo '--frozen-lockfile true' >> ~/.yarnrc
-      - checkout
-      - run:
-          name: Configure aws-cli
-          command: |
-              aws configure set aws_access_key_id $AWS_ACCESS_KEY_ID
-              aws configure set aws_secret_access_key $AWS_SECRET_ACCESS_KEY
-
-  setup_nix:
-    steps:
-      - run:
-          name: Add cp and rm stubs
-          command: |
-            cat \<< 'EOF' >> /bin/cp
-            #! /usr/bin/env bash
-            /usr/bin/env cp "$@"
-            EOF
-            chmod +x /bin/cp
-
-            cat \<< 'EOF' >> /bin/rm
-            #! /usr/bin/env bash
-            /usr/bin/env rm "$@"
-            EOF
-            chmod +x /bin/rm
-
-  decrypt_secrets_if_possible:
-    steps:
-      - run:
-          name: Decrypt secrets if possible
-          command: |
-            if [[ -z "$TURTLE_GIT_CRYPT_KEY_BASE64" ]]; then
-              echo 'git-crypt key not present in environment'
-            else
-              git crypt unlock <(echo $TURTLE_GIT_CRYPT_KEY_BASE64 | base64 --decode)
-            fi
-
-  restore_yarn_cache:
-    parameters:
-      workingdir:
-        type: string
-        default: .
-    steps:
-      - restore_cache:
-          key: yarn-v1-{{ checksum "<< parameters.workingdir >>/yarn.lock" }}
-
-  save_yarn_cache:
-    parameters:
-      workingdir:
-        type: string
-        default: .
-    steps:
-      - save_cache:
-          key: yarn-v1-{{ checksum "<< parameters.workingdir >>/yarn.lock" }}
-          paths:
-            - ~/.cache/yarn # `yarn cache dir`
-
-  restore_workingdir_cache:
-    parameters:
-      platform:
-        type: string
-        default: ios
-      sdk_version:
-        type: string
-        default: sdk32
-    steps:
-      - restore_cache:
-          key: "workingdir-v1-<< parameters.platform >>-<< parameters.sdk_version >>"
-
-  save_workingdir_cache:
-    parameters:
-      workingdir:
-        type: string
-        default: /Users/distiller/project/workingdir
-      platform:
-        type: string
-        default: ios
-      sdk_version:
-        type: string
-        default: sdk32
-    steps:
-      - save_cache:
-          key: "workingdir-v1-<< parameters.platform >>-<< parameters.sdk_version >>"
-          paths:
-            - "<< parameters.workingdir >>/<< parameters.platform >>/<< parameters.sdk_version >>"
+machine:
+  node:
+    # Circle will use node v4.0.0
+    version: 4.0.0
 
 jobs:
   build:
     docker:
-      - image: circleci/node:8.12
+      - image: circleci/node:10.15.0
     steps:
       - checkout
       - run: yarn
-      - run: yarn lint
       - run: yarn build
-
-  unit_tests:
-    executor: nix
+  test:
+    docker:
+      - image: circleci/node:10.15.0
     steps:
-      - setup
-      - restore_yarn_cache
+      - checkout
       - run: yarn
-      - save_yarn_cache
-      - run:
-          name: Run iOS smoke tests
-          command: ./scripts/tests/runUnitTestsCI.sh
-
-  ios_smoke_tests:
-    executor: mac
-    steps:
-      - install_nix
-      - setup
-      - decrypt_secrets_if_possible
-      - restore_yarn_cache
-      - run: yarn
-      - save_yarn_cache
-      - restore_workingdir_cache:
-          sdk_version: sdk32
-          platform: ios
-      - run:
-          name: Run iOS smoke tests
-          command: ./scripts/tests/runIOSSmokeTestsCI.sh
-      - save_workingdir_cache:
-          workingdir: /Users/distiller/project/workingdir
-          sdk_version: sdk32
-          platform: ios
-
-  android_smoke_tests:
-    executor: android
-    steps:
-      - setup
-      - setup_nix
-      - restore_yarn_cache
-      - run: yarn
-      - save_yarn_cache
-      - restore_workingdir_cache:
-          sdk_version: sdk32
-          platform: android
-      - run:
-          name: Fetch Android shell app
-          command: SDK_VERSION=sdk32 ./scripts/android/fetchRemoteAndroidTarball.sh
-      - run:
-          name: Run Android smoke tests
-          command: |
-            nix-shell nix/android-shell.nix --pure \
-              --run "./scripts/tests/runAndroidSmokeTestsCI.sh"
-      - save_workingdir_cache:
-          workingdir: ~/turtle/workingdir
-          sdk_version: sdk32
-          platform: android
+      - run: ./test-ci-unit.sh
 
 workflows:
-  checks:
+  version: 2
+  build_and_test:
     jobs:
       - build
-  tests:
-    jobs:
-      - unit_tests
-      - ios_smoke_tests
-      - android_smoke_tests
+      - test

.envrc

@@ -15,6 +15,4 @@ export NODE_ENV=development
 
 export EXPO_TURTLE_TOOLS_DIR="$(pwd)/tools"
 
-if [ -z "$EXPO_UNIVERSE_DIR" ]; then
-  PATH_add $EXPO_TURTLE_TOOLS_DIR/bin
-fi
+PATH_add $EXPO_TURTLE_TOOLS_DIR/bin

.eslintrc

@@ -0,0 +1,21 @@
+{
+  "plugins": [
+    "import"
+  ],
+  "extends": ["universe", "plugin:import/errors"],
+  "rules":{
+    "import/order": [
+      "error",
+      {
+        "groups": ["builtin", "external", "parent", "sibling", "index"],
+        "newlines-between": "always"
+      }
+    ],
+    "import/no-unresolved": "off"
+  },
+  "settings": {
+    "import/resolver": {
+      "babel-module": {}
+    }
+  }
+}

.gitignore

@@ -1,13 +1,8 @@
 !/src/node_modules
 
-node_modules
-yarn-error.log
-
 /build/
 /workingdir/*
 /artifacts/*
 shell.apk
+node_modules
 .secrets*
-
-*.swp
-*.swo

.ignore

@@ -1 +0,0 @@
-fixtures/

.release-it.beta.json

@@ -1,9 +0,0 @@
-{
-  "git": {
-    "commit": false,
-    "tag": true,
-    "push": true,
-    "requireUpstream": false,
-    "tagName": "v${version}"
-  }
-}

.release-it.json

@@ -1,12 +0,0 @@
-{
-  "npm": {
-    "tag": "latest"
-  },
-  "git": {
-    "requireUpstream": false,
-    "tagName": "v${version}"
-  },
-  "github": {
-    "release": true
-  }
-}

.secrets.local-common

@@ -2,6 +2,7 @@ LOGGER_LEVEL="debug"
 API_PROTOCOL="http"
 API_HOSTNAME="localhost"
 API_PORT="3000"
+LOGGLY_TOKEN=""
 SENTRY_DSN=""
 ENVIRONMENT="development"
 REDIS_URL="redis://localhost:6379/0"

CHANGELOG.md

@@ -1,102 +0,0 @@
-# Changelog
-All notable changes to this project will be documented in this file.
-
-The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
-and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-
-## [Unreleased]
-### Changed
-- Updated iOS shell app for SDK 32 to fix AppAuth issue
-
-## [0.6.0] - 2019-04-09
-### Changed
-- Removed Branch module from Android shell apps for all SDK versions
-
-## [0.5.15] - 2019-04-03
-### Added
-- Turtle smoke tests are now being executed on CircleCI from the context of this repository.
-### Changed
-- Updated iOS shell app for SDK 32 (fixes [this loading issue for slow connections](https://github.com/expo/expo/issues/3574)).
-- Minor infra changes (doesn't affect users).
-- Error logs format - error objects are now passed in an object being passed as the first argument to logger function.
-### Removed
-- Sending logs to loggly.
-
-## [0.5.14] - 2019-03-22
-### Added
-- `--config` parameter for turtle-cli commands, which allows to specify a custom path to app.json file
-### Changed
-- Updated iOS shell app for SDK 32 (fixes [this notifications issue](https://github.com/expo/expo/issues/3663)).
-
-## [0.5.13] - 2019-03-12
-### Added
-- Android SDK version support validation.
-### Changed
-- iOS Enterprise builds don't compile bitcode now.
-
-## [0.5.12] - 2019-02-26
-### Added
-- Google Cloud logger (as a bunyan stream).
-### Changed
-- Android builder passes the manifest from a build job message if it's available.
-
-## [0.5.11] - 2019-02-22
-### Added
-- Builder for iOS client apps.
-
-## [0.5.10] - 2019-02-19
-### Added
-- Register Turtle version in Redis.
-### Changed
-- Update Android shell app for SDK 32 to fix [update](https://github.com/expo/expo/issues/3504) and [back button](https://github.com/expo/expo/issues/1786) issues.
-
-## [0.5.9] - 2019-02-06
-- Update iOS shell app for SDK 32 (fixes bare notifications [issue](https://github.com/expo/expo/issues/3223)).
-
-## [0.5.7] - 2019-01-31
-- Update Android shell app for SDK 32 to fix bundled assets issue.
-
-## [0.5.6] - 2019-01-25
-- Update iOS shell app for SDK 30, 31 and 32 (built with xcode 10.1).
-
-## [0.5.5] - 2019-01-24
-### Added
-- Sending build duration in AWS SQS messages (of `FINISHED` and `ERRORED` types).
-
-## [0.5.4] - 2019-01-22
-### Changed
-- Update iOS shell app for SDK32 to [`expo/expo#0847c10`](https://github.com/expo/expo/commit/0847c1073d38c366866f3d9279ddd662da47dc9b), i.e. v2.10.3 with fixed WebView navigation scheme and upgraded Facebook SDK.
-
-## [0.5.3] - 2019-01-21
-### Fixed
-- Building Android standalone apps ([not accepted license issue](https://github.com/expo/expo/issues/3266)) - updated Android build tools (`build-tools;28.0.3`, `platforms;android-28`).
-
-## [0.5.2] - 2019-01-18
-### Added
-- The link (in README) to the repository containing `turtle-cli` usage example.
-- [Contributing guide](CONTRIBUTING.md).
-- [Code of Conduct](CODE_OF_CONDUCT.md).
-### Removed
-- `eslint`
-### Changed
-- Fix all `tslint` errors.
-
-## [0.5.1] - 2019-01-16
-### Changed
-- Update Android shell app for SDK32 that includes `[email protected]`.
-
-## [0.5.0] - 2019-01-16
-### Added
-- Turtle version is added to the `onFinish` and `onError` build messages.
-### Changed
-- A version property value being sent to the queue. It's now the version number
-from `package.json` instead of `new`/`old` indicator.
-### Removed
-- `--push-p12-path` parameter for `turtle build:ios` command,
-a Push Notifications Certificate is no longer required to perform a successful
-iOS standalone app build.
-
-## [0.4.3] - 2019-01-09
-### Changed
-- Update iOS shell app for SDK32.
-- Update Android shell app for SDK32.

CODE_OF_CONDUCT.md

@@ -1,7 +0,0 @@
-# Community Code of Conduct
-
-The Expo team is committed to fostering an open and welcoming environment.
-
-**Our Community Guidelines can be found here:**
-
-https://expo.io/guidelines

CONTRIBUTING.md

@@ -1,33 +0,0 @@
-# Contributing to Turtle CLI
-
-## How to contribute
-You can contribute to Turtle CLI in various ways, including:
-- [Reporting bugs or issues](https://github.com/expo/turtle/issues/new) on GitHub. Please make sure to fill in all the details suggested by the template to make sure the issue can be addressed as quickly as possible.
-- [Submitting improvements to the documentation](https://github.com/expo/expo/tree/master/docs). Updates, enhancements, new guides, spelling fixes…
-- Helping other people on [Expo Forums](https://forums.expo.io/) and on [Expo Developers Slack](http://slack.expo.io).
-- Looking at existing [issues](https://github.com/expo/turtle/issues) and adding more information, particularly helping to reproduce the issues.
-- [Submitting a pull request](#submitting-a-pull-request) with a bug fix or an improvement.
-
-## Turtle CLI repository
-
-### Codebase
-This repository contains the codebase for the `turtle-cli` NPM package. The same codebase is used to run standalone app builders on the Expo servers.
-
-### Branches
-The `master` branch of the repository should be kept releasable at any time. This way we can continuously release fixes and improvements without costly managing of different branches and issues will be noticed and fixed quickly. This also ensures other contributors can check out the latest version from GitHub and work on it with minimal disruption from other features in progress.
-
-## Setting up the repository for development
-*TODO*
-
-## Submitting a pull request
-To submit a pull request:
-1. Fork the [repository](https://github.com/expo/turtle) and create a feature branch. (Existing contributors can create feature branches without forking. Prefix the branch name with `@your-github-username/`.)
-2. Write the description of your pull request. Make sure to include a test plan and test your changes.
-3. Make sure all tests pass on CircleCI.
-4. Wait for a review and adjust the code if necessary.
-
-## Publishing a release
-- To release a new version of `turtle-cli`, run `yarn release` command.
-- If you wish to release a new *beta* version of `turtle-cli`, run `yarn release:beta` instead.
-- Update [Changelog](CHANGELOG.md).
-- In a terminal, `cd` into [expo/turtle-cli-example](https://github.com/expo/turtle-cli-example) repository, run `yarn update-turtle-cli-version` (this script updates `turtle-cli` version number in CI configuration files and commits the changes) and then run `git push`. Thanks!

Dockerfile

@@ -36,34 +36,30 @@ RUN curl -SLO "https://nodejs.org/dist/v$NODE_VERSION/node-v$NODE_VERSION-linux-
 # Download and untar SDK
 ENV ANDROID_HOME /usr/local/android-sdk-linux
 ENV ANDROID_SDK /usr/local/android-sdk-linux
-ENV ANDROID_SDK_URL https://dl.google.com/android/repository/sdk-tools-linux-4333796.zip
+ENV ANDROID_SDK_URL https://dl.google.com/android/repository/sdk-tools-linux-3859397.zip
 RUN mkdir -p ${ANDROID_HOME} && \
   curl -L "${ANDROID_SDK_URL}" > ${ANDROID_HOME}/sdk.zip && \
   cd ${ANDROID_HOME} && \
   unzip -qq sdk.zip && \
   rm sdk.zip && \
+  # prevents warnings about missing repo config
   mkdir -p ${HOME}/.android && \
   touch ${HOME}/.android/repositories.cfg
 
 ENV PATH ${ANDROID_HOME}/platform-tools:${PATH}
 ENV PATH ${ANDROID_HOME}/tools:${PATH}
 ENV PATH ${ANDROID_HOME}/tools/bin:${PATH}
-ENV PATH ${ANDROID_HOME}/build-tools/28.0.3/:${PATH}
+ENV PATH ${ANDROID_HOME}/build-tools/25.0.0/:${PATH}
 
 RUN yes | sdkmanager --licenses > /dev/null
-RUN yes | sdkmanager \
+
+# Install Android SDK components
+RUN sdkmanager \
+  "platform-tools" \
   "platforms;android-23" \
-  "platforms;android-24" \
-  "platforms;android-25" \
-  "platforms;android-26" \
-  "platforms;android-27" \
-  "platforms;android-28"
-RUN yes | sdkmanager "platform-tools"
-RUN yes | sdkmanager "build-tools;28.0.3"
-RUN yes | sdkmanager \
+  "build-tools;25.0.0" \
   "extras;android;m2repository" \
-  "extras;google;m2repository" \
-  "extras;google;google_play_services"
+  "extras;google;m2repository"
 
 # Install Android NDK
 ENV ANDROID_NDK_VERSION android-ndk-r10e
@@ -102,6 +98,13 @@ ENV PATH ${GRADLE_HOME}/bin:$PATH
 # Install Gulp
 RUN npm install -g gulp-cli
 
+# Install Git
+RUN echo 'deb http://http.debian.net/debian wheezy-backports main' >> /etc/apt/sources.list && \
+  apt-get update && \
+  apt-get -t wheezy-backports install git-core && \
+  apt-get clean && \
+  rm -rf /var/lib/apt/lists/*
+
 ADD . /app
 
 RUN for SDK_VERSION in `ls /app/workingdir/android/`; do \

LICENSE

@@ -1,21 +0,0 @@
-The MIT License (MIT)
-
-Copyright (c) 2015-present 650 Industries, Inc. (aka Expo)
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.

README.md

@@ -1,39 +1,13 @@
 # Turtle CLI
 
-[![CircleCI](https://img.shields.io/circleci/project/github/expo/turtle/master.svg)](https://circleci.com/gh/expo/turtle)
-[![NPM](https://img.shields.io/npm/v/turtle-cli/latest.svg)](https://npmjs.com/package/turtle-cli)
-[![License](https://img.shields.io/badge/license-MIT-green.svg?style=flat)](https://github.com/expo/turtle/blob/master/LICENSE)
+Turtle CLI is a command line interface for building Expo standalone apps. You can use it both on your CI and your private computer.
 
-[![Expo Developers Slack](https://img.shields.io/badge/Expo%20Developers-e01563.svg?logo=slack)](http://slack.expo.io)
-[![Expo Forums](https://img.shields.io/badge/Expo%20Forums-blue.svg)](https://forums.expo.io/)
+### Glossary
 
-Turtle CLI is a command line interface for building Expo standalone apps.
-You can use it both on your CI and your private computer.
-
-- [Changelog](CHANGELOG.md)
-- [Contributing to Turtle CLI](CONTRIBUTING.md)
-- [Code of Conduct](CODE_OF_CONDUCT.md)
-
-## Glossary
 - **Turtle** - script which takes a shell app, fills it with user's code and produces a standalone app
 - **Shell app** — precompiled (in case of iOS builds) Expo app which doesn't have user's code
 - **Standalone app** - .ipa/.apk file, a user uploads standalone apps to the Google Play Store/Apple App Store
 
 ## Installation
+
 [Installation instructions and documentation here.](https://docs.expo.io/versions/latest/distribution/turtle-cli)
-
-## Usage example
-See the [expo/turtle-cli-example](https://github.com/expo/turtle-cli-example) repository to learn how to use Turtle CLI in order to build Expo standalone apps on your CI.
-
-## Do you need help?
-- Make sure you've read [the Expo Docs](https://docs.expo.io/), especially [Building Standalone Apps on Your CI
-](https://docs.expo.io/versions/latest/distribution/turtle-cli) and [Building Standalone Apps](https://docs.expo.io/versions/latest/distribution/building-standalone-apps) guides.
-- Search for existing GitHub issues [in this repository](https://github.com/expo/turtle/issues) and in [expo/expo](https://github.com/expo/expo/issues) repository.
-- Ask for help in **one** of the following ways:
-  * create [a GitHub issue](https://github.com/expo/turtle/issues/new).
-  * create a new topic on [Expo Forums](https://forums.expo.io/).
-  * ask a question on [Expo Developers Slack](http://slack.expo.io).
-
-## License
-The Turtle CLI source code is made available under the [MIT license](LICENSE).
-Some of the dependencies are licensed differently, with the BSD license, for example.

jest/integration-config.json

@@ -9,6 +9,6 @@
     "^.+\\.ts$": "ts-jest"
   },
   "testRegex": "(test|spec)\\.ts$",
-  "moduleFileExtensions": ["ts", "js", "json"],
+  "moduleFileExtensions": ["ts", "js"],
   "verbose": true
 }

jest/smoke-config.json

@@ -9,6 +9,6 @@
     "^.+\\.ts$": "ts-jest"
   },
   "testMatch": [ "**/__smoke__/*.test.ts" ],
-  "moduleFileExtensions": ["ts", "js", "json"],
+  "moduleFileExtensions": ["ts", "js"],
   "verbose": true
 }

jest/unit-config.json

@@ -9,6 +9,6 @@
     "^.+\\.ts$": "ts-jest"
   },
   "testMatch": [ "**/__tests__/*.ts" ],
-  "moduleFileExtensions": ["ts", "js", "json"],
+  "moduleFileExtensions": ["ts", "js"],
   "verbose": true
 }

jsconfig.json

@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "target": "es2017",
+    "allowSyntheticDefaultImports": false,
+    "baseUrl": "./",
+    "paths": {
+      "turtle/*": ["src/*"],
+    }
+  },
+  "exclude": ["node_modules", "dist", "build"]
+}

k8s/android/turtle-deployment.template.yml

@@ -50,6 +50,8 @@ spec:
             value: exp-shell-app-assets
           - name: AWS_S3_REGION
             value: us-west-1
+          - name: LOGGLY_SUBDOMAIN
+            value: exponent
           - name: TURTLE_TEMPORARY_FILES_DIR
             value: /tmp/turtle
           # vars from k8s secrets
@@ -98,6 +100,11 @@ spec:
               secretKeyRef:
                 name: turtle-js-config
                 key: aws_cloudwatch_region
+          - name: LOGGLY_TOKEN
+            valueFrom:
+              secretKeyRef:
+                name: turtle-js-config
+                key: loggly_token
           - name: REDIS_URL
             valueFrom:
               secretKeyRef:

nix/all-packages.nix

@@ -1,5 +0,0 @@
-self: super:
-
-{
-  nodejs = super.nodejs-8_x;
-}

nix/android-shell.nix

@@ -1,50 +0,0 @@
-with (import ./default.nix {});
-
-let
-
-   ndk = androidenv.androidndk_17c.override { fullNDK = true; };
-   ndk_root = "${ndk}/libexec/${ndk.name}";
-   sdk_path = if stdenv.isDarwin
-     then "/Library/Android/sdk"
-     else "/Android/Sdk"; # intentional Capital S
-
-in
-
-mkShell rec {
-
-  LANG="en_US.UTF-8";
-  JAVA_HOME=openjdk8;
-
-  ANDROID_SDK_ROOT=builtins.getEnv("HOME") + sdk_path;
-  ANDROID_NDK_ROOT="${ANDROID_SDK_ROOT}/ndk-bundle";
-  ANDROID_HOME=ANDROID_SDK_ROOT;
-  ANDROID_NDK=ANDROID_NDK_ROOT;
-
-  nativeBuildInputs = [
-    awscli
-    curl
-    git
-    nodejs-8_x
-    openjdk8
-    yarn
-  ];
-
-  passthru = { inherit ndk; };
-
-  shellHook = ''
-    ${./install-ndk-17c.sh} ${ndk} ${ndk_root}
-    yes | ${androidenv.androidsdk_9_0}/bin/sdkmanager --licenses --sdk_root="$ANDROID_SDK_ROOT"
-    yes | ${androidenv.androidsdk_9_0}/bin/sdkmanager --sdk_root="$ANDROID_SDK_ROOT" "platforms;android-28" | grep -v =
-    yes | ${androidenv.androidsdk_9_0}/bin/sdkmanager --sdk_root="$ANDROID_SDK_ROOT" "build-tools;28.0.3" | grep -v =
-    ${lib.optionalString stdenv.isLinux ''
-      for dep in lib lib64; do
-        if [ -L /$dep ] || [ ! -e /$dep ]; then
-          echo "Creating /$dep"
-          ln -s ${stdenv.cc.libc}/$dep /$dep
-        else
-          echo "Using existing /$dep"
-        fi
-      done
-    ''}
-  '';
-}

nix/default.nix

@@ -1,19 +0,0 @@
-let
-  pin = builtins.fromJSON (builtins.readFile ./nixpkgs.json);
-
-  nixpkgs = builtins.fetchTarball {
-    url = "https://github.com/NixOS/nixpkgs-channels/archive/${pin.rev}.tar.gz";
-    inherit (pin) sha256;
-  };
-
-in
-
-{ ... } @ args:
-
-  import nixpkgs (args // {
-    config = {
-      allowUnfree = true;
-      android_sdk.accept_license = true;
-    };
-    overlays = [ (import ./all-packages.nix) ];
-  })

nix/install-ndk-17c.sh

@@ -1,14 +0,0 @@
-#!/usr/bin/env bash
-set -euo pipefail
-
-scriptdir="$(dirname ${BASH_SOURCE[0]})"
-shellnix="$scriptdir/android-shell.nix"
-
-ndk=${1:-$(nix-build "$shellnix" -A ndk --no-out-link)}
-ndkRoot=${2:-$(nix eval --file "$shellnix" --raw ANDROID_NDK_ROOT)}
-
-mkdir -p "$ANDROID_NDK_ROOT" # Create full path if necessary
-
-# Replace any existing link or directory with link
-rm -r "$ANDROID_NDK_ROOT"
-ln -sn "$ndkRoot" "$ANDROID_NDK_ROOT"

nix/nixpkgs.json

@@ -1,7 +0,0 @@
-{
-  "url": "https://github.com/NixOS/nixpkgs-channels.git",
-  "rev": "80738ed9dc0ce48d7796baed5364eef8072c794d",
-  "date": "2018-11-17T11:18:10+01:00",
-  "sha256": "0anmvr6b47gbbyl9v2fn86mfkcwgpbd5lf0yf3drgm8pbv57c1dc",
-  "fetchSubmodules": false
-}

package.json

@@ -1,23 +1,23 @@
 {
   "name": "turtle-cli",
-  "version": "0.6.0",
+  "version": "0.4.3",
   "description": "The command-line tool for building standalone Expo apps",
   "preferGlobal": true,
   "scripts": {
-    "start": "EXPENV_OVERRIDES=local-common,local,local-queue expenv gulp",
+    "start": "EXPENV_OVERRIDES=local-common,local expenv gulp",
     "start:ios": "PLATFORM=ios yarn start",
     "start:android": "PLATFORM=android yarn start",
-    "watch": "EXPENV_OVERRIDES=local-common,local,local-queue expenv gulp watch:source",
+    "watch": "EXPENV_OVERRIDES=local-common,local expenv gulp watch:source",
     "bin:turtle": "./bin/turtle.js",
-    "lint": "tslint -c tslint.json 'src/**/*.ts'",
-    "lint:fix": "yarn lint --fix",
+    "lint": "eslint src",
     "prepare": "yarn build",
     "build": "gulp build:server",
     "build:production": "./scripts/build.sh",
     "build:android-tarball:local": "./scripts/android/buildAndroidTarballLocally.sh",
-    "secrets:init-private": "./scripts/initPrivateSecrets.sh",
-    "set-workingdir:remote": "./scripts/setWorkingdirMode.sh remote",
-    "set-workingdir:local": "./scripts/setWorkingdirMode.sh local",
+    "init-workingdir:local": "./scripts/initLocalWorkingdir.sh",
+    "init-workingdir:remote": "yarn init-workingdir:remote-android && yarn init-workingdir:remote-ios",
+    "init-workingdir:remote-android": "./scripts/android/fetchRemoteAndroidTarball.sh",
+    "init-workingdir:remote-ios": "./scripts/ios/fetchRemoteIosTarball.sh",
     "docker:pull": "retry5 gcloud docker -- pull gcr.io/exponentjs/turtle-android:latest",
     "docker:pull:tag": "retry5 gcloud docker -- pull gcr.io/exponentjs/turtle-android:${TAG}",
     "docker:build": "yarn build:production android && gcloud docker -- build -t gcr.io/exponentjs/turtle-android:latest -t gcr.io/exponentjs/turtle-android:${TAG} .",
@@ -27,29 +27,17 @@
     "test:integration": "yarn test --config jest/integration-config.json",
     "test:unit": "yarn test --config jest/unit-config.json",
     "test:smoke": "yarn test --config jest/smoke-config.json --forceExit",
-    "test:smoke:android": "PLATFORM=android yarn test:smoke --testNamePattern=android",
+    "test:smoke:android": "yarn test:smoke --testNamePattern=android",
     "test:smoke:ios": "PLATFORM=ios yarn test:smoke --testNamePattern=iOS",
-    "release": "./scripts/release.sh",
-    "release:beta": "./scripts/releaseBeta.sh",
+    "secrets:init-private": "./scripts/initPrivateSecrets.sh",
     "ssh:android:pods-names": "./scripts/ssh/findAndroidPods.sh",
     "ssh:android:connect": "./scripts/ssh/connectToAndroidPod.sh",
     "ssh:ios:ips": "./scripts/ssh/findIosIps.sh",
     "set-priorities": "yarn set-priorities:local",
-    "set-priorities:local": "EXPENV_OVERRIDES=local-common,local,local-queue expenv babel-node scripts/setConfigurations.js",
+    "set-priorities:local": "EXPENV_OVERRIDES=local-common,local expenv babel-node scripts/setConfigurations.js",
     "set-priorities:staging": "expenv babel-node scripts/setConfigurations.js",
     "set-priorities:production": "EXPENV_OVERRIDES=prod expenv babel-node scripts/setConfigurations.js"
   },
-  "husky": {
-    "hooks": {
-      "pre-commit": "lint-staged"
-    }
-  },
-  "lint-staged": {
-    "src/**/*.ts": [
-      "yarn lint:fix",
-      "git add"
-    ]
-  },
   "bin": {
     "turtle": "./bin/turtle.js"
   },
@@ -72,9 +60,6 @@
     "url": "https://github.com/expo/turtle/issues"
   },
   "homepage": "https://github.com/expo/turtle#readme",
-  "publishConfig": {
-    "registry": "https://registry.npmjs.org/"
-  },
   "devDependencies": {
     "@types/decompress": "^4.2.3",
     "@types/fs-extra": "^5.0.4",
@@ -90,9 +75,13 @@
     "@types/which": "^1.3.1",
     "babel-cli": "^6.26.0",
     "babel-core": "^6.26.3",
+    "babel-eslint": "^8.2.3",
     "babel-jest": "^23.6.0",
     "babel-plugin-module-resolver": "^3.1.1",
     "babel-preset-env": "^1.6.1",
+    "eslint": "^4.19.1",
+    "eslint-config-universe": "^1.0.7",
+    "eslint-import-resolver-babel-module": "^4.0.0",
     "gulp": "github:gulpjs/gulp#4.0",
     "gulp-babel": "^7.0.1",
     "gulp-changed": "^3.2.0",
@@ -100,26 +89,21 @@
     "gulp-plumber": "^1.2.0",
     "gulp-sourcemaps": "^2.6.4",
     "gulp-typescript": "^4.0.2",
-    "husky": "^1.3.1",
     "jest": "^23.6.0",
-    "lint-staged": "^8.1.5",
     "nock": "^10.0.1",
     "nodemon": "^1.18.7",
-    "release-it": "^10.0.0",
     "rimraf": "^2.6.2",
     "source-map-loader": "^0.2.3",
     "ts-jest": "^23.10.4",
     "tslint": "^5.10.0",
-    "tslint-eslint-rules": "^5.4.0",
     "typescript": "^3.0.1"
   },
   "dependencies": {
-    "@expo/spawn-async": "^1.4.0",
-    "@google-cloud/logging-bunyan": "^0.10.1",
     "async-retry": "^1.2.1",
     "aws-sdk": "^2.226.1",
     "bunyan": "^1.8.12",
     "bunyan-debug-stream": "^1.0.8",
+    "bunyan-loggly": "^1.3.4",
     "bunyan-sentry-stream": "^1.2.1",
     "commander": "^2.15.1",
     "copy": "^0.3.1",
@@ -138,9 +122,6 @@
     "tar": "^4.4.6",
     "uuid": "^3.2.1",
     "which": "^1.3.1",
-    "xdl": "53.1.6-alpha.2"
-  },
-  "optionalDependencies": {
-    "@expo/traveling-fastlane-darwin": "^1.8.1"
+    "xdl": "52.0.14-alpha.6+371be041"
   }
 }

scripts/android/fetchRemoteAndroidTarball.sh

@@ -1,4 +1,18 @@
 #!/usr/bin/env bash
 
+set -xeo pipefail
+
 ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/../.. && pwd )"
-$ROOT_DIR/scripts/fetchRemoteTarballs.sh android
+ARTIFACTS_DIR="$ROOT_DIR/artifacts"
+WORKING_DIR="$ROOT_DIR/workingdir/android"
+ANDROID_SHELL_TARBALLS_DIRECTORY="$ROOT_DIR/shellTarballs/android"
+
+mkdir -p $ARTIFACTS_DIR
+
+for SDK_VERSION in `ls $ANDROID_SHELL_TARBALLS_DIRECTORY`; do
+  mkdir -p $WORKING_DIR/$SDK_VERSION
+  TARBALL_URI=$(head -n 1 $ANDROID_SHELL_TARBALLS_DIRECTORY/$SDK_VERSION)
+  run expo.s3env --keep AWS_ACCESS_KEY_ID --keep AWS_SECRET_ACCESS_KEY --command retry5 aws s3 cp $TARBALL_URI $ARTIFACTS_DIR/$SDK_VERSION.tar.gz --no-progress
+  tar zxf $ARTIFACTS_DIR/$SDK_VERSION.tar.gz -C $WORKING_DIR/$SDK_VERSION
+  rm -rf $TARBALL_URI $ARTIFACTS_DIR/$SDK_VERSION.tar.gz
+done

scripts/build.sh

@@ -2,8 +2,6 @@
 
 set -xeo pipefail
 
-ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/.. && pwd )"
-
 if [ -z "$1" ]; then
   echo "Please specify a platform to build"
   exit 1
@@ -13,12 +11,7 @@ PLATFORM=$1
 
 yarn
 yarn run build
-if [ "$PLATFORM" = "android" ]; then
-  $ROOT_DIR/scripts/android/fetchRemoteAndroidTarball.sh
-elif [ "$PLATFORM" = "ios" ]; then
-  $ROOT_DIR/scripts/ios/fetchRemoteIosTarball.sh
-fi
-
+yarn init-workingdir:remote-$PLATFORM
 
 if [[ -z "${TARBALL}" ]]; then
   echo "TARBALL is not set, building tarball skipped"

scripts/cli/configureAndroidSdk.sh

@@ -1,12 +1,9 @@
 #!/bin/bash
 
-SDKMANAGER=$ANDROID_HOME/tools/bin/sdkmanager
-
-yes | $SDKMANAGER --licenses > /dev/null
-yes | $SDKMANAGER "platforms;android-28"
-yes | $SDKMANAGER "platform-tools"
-yes | $SDKMANAGER "build-tools;28.0.3"
-yes | $SDKMANAGER \
+yes | $ANDROID_HOME/tools/bin/sdkmanager --licenses > /dev/null
+$ANDROID_HOME/tools/bin/sdkmanager \
+  "platform-tools" \
+  "platforms;android-23" \
+  "build-tools;25.0.0" \
   "extras;android;m2repository" \
-  "extras;google;m2repository" \
-  "extras;google;google_play_services"
+  "extras;google;m2repository"

scripts/fetchRemoteTarballs.sh

@@ -1,38 +0,0 @@
-#!/usr/bin/env bash
-
-set -xeo pipefail
-
-PLATFORM=$1
-
-ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/.. && pwd )"
-ARTIFACTS_DIR="$ROOT_DIR/artifacts"
-WORKING_DIR="$ROOT_DIR/workingdir/$PLATFORM"
-SHELL_TARBALLS_DIRECTORY="$ROOT_DIR/shellTarballs/$PLATFORM"
-
-mkdir -p $ARTIFACTS_DIR
-
-convert_s3_to_https() {
-  BUCKET=`echo $1 | sed 's/s3:\/\/\([^/]*\).*/\1/'`
-  KEY=`echo $1 | sed 's/s3:\/\/[^/]*\/\(.*\)/\1/'`
-  echo "https://s3.amazonaws.com/$BUCKET/$KEY"
-}
-
-download_shell_app() {
-  SDK_VERSION=$1
-  mkdir -p $WORKING_DIR/$SDK_VERSION
-  TARBALL_S3_URL=$(head -n 1 $SHELL_TARBALLS_DIRECTORY/$SDK_VERSION)
-  TARBALL_HTTPS_URL=$(convert_s3_to_https $TARBALL_S3_URL)
-  curl -s $TARBALL_HTTPS_URL -o $ARTIFACTS_DIR/$SDK_VERSION.tar.gz
-  tar zxf $ARTIFACTS_DIR/$SDK_VERSION.tar.gz -C $WORKING_DIR/$SDK_VERSION
-  rm -rf $ARTIFACTS_DIR/$SDK_VERSION.tar.gz
-}
-
-if [[ -z "$SDK_VERSION" ]]; then
-  for SDK_VERSION in `ls $SHELL_TARBALLS_DIRECTORY`; do
-    download_shell_app $SDK_VERSION
-  done
-else
-  if [ ! -d "$WORKING_DIR/$SDK_VERSION" ]; then
-    download_shell_app $SDK_VERSION
-  fi
-fi

scripts/initLocalWorkingdir.sh

@@ -0,0 +1,22 @@
+#!/usr/bin/env bash
+
+ensure_link_exists() {
+  if [ ! -L "$2" ]; then
+    ln -s $1 $2
+    echo "$2 link created"
+  fi
+}
+
+ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/.. && pwd )"
+WORKINGDIR="$ROOT_DIR/workingdir/local"
+
+if [ ! -d "$WORKINGDIR" ]; then
+  mkdir -p $WORKINGDIR
+  echo "$WORKINGDIR directory created"
+fi
+
+LINKS=( "template-files" "package.json" "android" "expokit-npm-package" "packages" "client-builds" "cpp" "exponent-view-template" "ios" "shellAppBase-builds" "shellAppWorkspaces")
+
+for LINKNAME in ${LINKS[@]}; do
+  ensure_link_exists "$EXPO_UNIVERSE_DIR/exponent/$LINKNAME" "$WORKINGDIR/$LINKNAME"
+done

scripts/initPrivateSecrets.sh

@@ -1,10 +1,5 @@
 #!/usr/bin/env bash
 
-if [ -z "$EXPO_UNIVERSE_DIR" ]; then
-  echo "You need to specify EXPO_UNIVERSE_DIR to use this script. Alternatively you can create .secrets.local-queue file yourself with AWS_SQS_ANDROID_QUEUE_URL, AWS_SQS_IOS_QUEUE_URL and AWS_SQS_OUT_QUEUE_URL variables."
-  exit 1
-fi
-
 if [ -z "$1" ]; then
   echo "Please specify a nickname you used to create AWS SQS queues"
   exit 1
@@ -17,9 +12,12 @@ ANDROID=`cd $TERRAFORM_DIR; terraform state show module.turtle_queues_local_$1.a
 IOS=`cd $TERRAFORM_DIR; terraform state show module.turtle_queues_local_$1.aws_sqs_queue.jobs_ios_in | head -1 | cut -d"=" -f2 | xargs`
 OUT=`cd $TERRAFORM_DIR; terraform state show module.turtle_queues_local_$1.aws_sqs_queue.jobs_out | head -1 | cut -d"=" -f2 | xargs`
 
-cat << EOF > $TURTLE_DIR/.secrets.local-queue
+cat << EOF > $TURTLE_DIR/.secrets.local
 AWS_SQS_ANDROID_QUEUE_URL="$ANDROID"
 AWS_SQS_IOS_QUEUE_URL="$IOS"
 AWS_SQS_OUT_QUEUE_URL="$OUT"
+TURTLE_FAKE_UPLOAD="1"
+TURTLE_FAKE_UPLOAD_DIR="/Users/$USER/Downloads"
+TURTLE_WORKING_DIR_PATH="$EXPO_UNIVERSE_DIR/server/turtle/workingdir"
+TURTLE_USE_LOCAL_WORKING_DIR="1"
 EOF
-

scripts/ios/fetchRemoteIosTarball.sh

@@ -1,4 +1,18 @@
 #!/usr/bin/env bash
 
+set -xeo pipefail
+
 ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/../.. && pwd )"
-$ROOT_DIR/scripts/fetchRemoteTarballs.sh ios
+ARTIFACTS_DIR="$ROOT_DIR/artifacts"
+WORKING_DIR="$ROOT_DIR/workingdir/ios"
+IOS_SHELL_TARBALLS_DIRECTORY="$ROOT_DIR/shellTarballs/ios"
+
+mkdir -p $ARTIFACTS_DIR
+
+for SDK_VERSION in `ls $IOS_SHELL_TARBALLS_DIRECTORY`; do
+  mkdir -p $WORKING_DIR/$SDK_VERSION
+  TARBALL_URI=$(head -n 1 $IOS_SHELL_TARBALLS_DIRECTORY/$SDK_VERSION)
+  run expo.s3env --keep AWS_ACCESS_KEY_ID --keep AWS_SECRET_ACCESS_KEY --command retry5 aws s3 cp $TARBALL_URI $ARTIFACTS_DIR/$SDK_VERSION.tar.gz --no-progress
+  tar zxf $ARTIFACTS_DIR/$SDK_VERSION.tar.gz -C $WORKING_DIR/$SDK_VERSION
+  rm -rf $TARBALL_URI $ARTIFACTS_DIR/$SDK_VERSION.tar.gz
+done

scripts/release.sh

@@ -1,11 +0,0 @@
-#!/usr/bin/env bash
-
-set -eo pipefail
-
-ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/.. && pwd )"
-
-$ROOT_DIR/node_modules/.bin/release-it
-echo
-echo "!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"
-echo "!!!! PLEASE REMEMBER TO UPDATE CHANGELOG !!!!"
-echo "!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"

scripts/releaseBeta.sh

@@ -1,7 +0,0 @@
-#!/usr/bin/env bash
-
-set -eo pipefail
-
-ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/.. && pwd )"
-
-$ROOT_DIR/node_modules/.bin/release-it -c $ROOT_DIR/.release-it.beta.json --preRelease=beta

scripts/setWorkingdirMode.sh

@@ -1,86 +0,0 @@
-#!/usr/bin/env bash
-
-ROOT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )"/.. && pwd )"
-
-DOWNLOAD_DIR="${EXPO_DOWNLOAD_DIR:-$HOME/Downloads}"
-
-mkdir -p $DOWNLOAD_DIR
-
-if [ ! -f "$ROOT_DIR/.secrets.local-queue" ]; then 
-  echo "You need to run \"yarn secrets:init-private [SQS_NAME]\" or create .secrets.local-queue file yourself with AWS_SQS_ANDROID_QUEUE_URL, AWS_SQS_IOS_QUEUE_URL and AWS_SQS_OUT_QUEUE_URL variables."
-  exit 1
-fi
-
-setup_local() {
-  ensure_link_exists() {
-    if [ ! -L "$2" ]; then
-      ln -s $1 $2
-      echo "$2 link created"
-    fi
-  }
-  WORKINGDIR="$ROOT_DIR/workingdir/local"
-  
-  if [[ -d $WORKINGDIR ]]; then
-    echo "Local workingdir already prepared"
-    return
-  fi
-
-  if [[ -z "$EXPO_UNIVERSE_DIR" ]]; then
-    echo "EXPO_UNIVERSE_DIR needs to be set"
-    exit -1
-  fi
-
-  cd $EXPO_UNIVERSE_DIR/exponent && yarn
-  cd $EXPO_UNIVERSE_DIR/exponent/tools-public && yarn && ./generate-dynamic-macros-android.sh
-
-  mkdir -p $WORKINGDIR
-  echo "$WORKINGDIR directory created"
-
-  LINKS=("template-files" "package.json" "android" "expokit-npm-package" "packages" "client-builds" "cpp" "exponent-view-template" "ios" "shellAppBase-builds" "shellAppWorkspaces")
-
-  for LINKNAME in ${LINKS[@]}; do
-    ensure_link_exists "$EXPO_UNIVERSE_DIR/exponent/$LINKNAME" "$WORKINGDIR/$LINKNAME"
-  done
-}
-
-setup_remote() {
-  if [[ -d $ROOT_DIR/workingdir/android && -d $ROOT_DIR/workingdir/ios ]]; then
-    echo "Shell apps already fetched"
-  else
-    echo "Fetching shell apps"
-    if [ "$PLATFORM" != 'ios' ]; then
-      $ROOT_DIR/scripts/android/fetchRemoteAndroidTarball.sh
-    fi
-    if [ "$PLATFORM" != 'android' ]; then
-      $ROOT_DIR/scripts/ios/fetchRemoteIosTarball.sh
-    fi
-  fi
-}
-
-switch_local() {
-cat << EOF > $ROOT_DIR/.secrets.local
-TURTLE_FAKE_UPLOAD="1"
-TURTLE_FAKE_UPLOAD_DIR="$HOME/Downloads"
-TURTLE_WORKING_DIR_PATH="$ROOT_DIR/workingdir"
-TURTLE_USE_LOCAL_WORKING_DIR="1"
-EOF
-}
-
-switch_remote() {
-cat << EOF > $ROOT_DIR/.secrets.local
-TURTLE_FAKE_UPLOAD="1"
-TURTLE_FAKE_UPLOAD_DIR="$HOME/Downloads"
-TURTLE_WORKING_DIR_PATH="$ROOT_DIR/workingdir"
-EOF
-}
-
-if [ "$1" == "local" ]; then
-  setup_local
-  switch_local
-elif [ "$1" == "remote" ]; then
-  setup_remote
-  switch_remote
-else
-  echo "You need to pass remote or local as argument"
-  exit -1
-fi

shellTarballs/android/sdk26

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-c19ce0d88a153f481fd790e23bdcd6f2d5ea7fd6.tar.gz
+s3://exp-artifacts/android-shell-builder-67bd02667e29221707f8c1dd11e739eada25ef55.tar.gz

shellTarballs/android/sdk27

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-c19ce0d88a153f481fd790e23bdcd6f2d5ea7fd6.tar.gz
+s3://exp-artifacts/android-shell-builder-67bd02667e29221707f8c1dd11e739eada25ef55.tar.gz

shellTarballs/android/sdk28

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-c19ce0d88a153f481fd790e23bdcd6f2d5ea7fd6.tar.gz
+s3://exp-artifacts/android-shell-builder-67bd02667e29221707f8c1dd11e739eada25ef55.tar.gz

shellTarballs/android/sdk29

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-c19ce0d88a153f481fd790e23bdcd6f2d5ea7fd6.tar.gz
+s3://exp-artifacts/android-shell-builder-67bd02667e29221707f8c1dd11e739eada25ef55.tar.gz

shellTarballs/android/sdk30

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-c19ce0d88a153f481fd790e23bdcd6f2d5ea7fd6.tar.gz
+s3://exp-artifacts/android-shell-builder-67bd02667e29221707f8c1dd11e739eada25ef55.tar.gz

shellTarballs/android/sdk31

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-89b2d24c86737d5a489c6a18467e31b2f9ce9005.tar.gz
+s3://exp-artifacts/android-shell-builder-5c339ff7bab6f2161d378cc70e1bf04d32ad98dd.tar.gz

shellTarballs/android/sdk32

@@ -1 +1 @@
-s3://exp-artifacts/android-shell-builder-0cab68670fc50ce1dc16b7d86ca7976870062c39.tar.gz
+s3://exp-artifacts/android-shell-builder-55e2cc952dc8bae3ebba8aaaab6eb1199ac7b0d0.tar.gz

shellTarballs/ios/client

@@ -1 +0,0 @@
-s3://exp-artifacts/ios-expo-client-15bb883c22173d5b01df19421580170e776041d0.tar.gz

shellTarballs/ios/sdk30

@@ -1 +1 @@
-s3://exp-artifacts/ios-shell-builder-sdk-30-6863276e6b9605e31228c845a047387571959e9b.tar.gz
+s3://exp-artifacts/ios-shell-builder-sdk-30-7d017a96d00fbf18ee7e5581ea47528b9cc78afd.tar.gz

shellTarballs/ios/sdk31

@@ -1 +1 @@
-s3://exp-artifacts/ios-shell-builder-sdk-latest-e1a8bd0ab2e1eba546792c0ffc282b5b202ccc39.tar.gz
+s3://exp-artifacts/ios-shell-builder-sdk-latest-56824ecea6ba10f5a9d8991545a1c47ee3284f5f.tar.gz

shellTarballs/ios/sdk32

@@ -1 +1 @@
-s3://exp-artifacts/ios-shell-builder-sdk-latest-dc981c7110b7d508de8e9c352064dc3d745c9f72.tar.gz
+s3://exp-artifacts/ios-shell-builder-sdk-latest-e3855783ed0618f4dcf326dc353c6e2951e05c49.tar.gz

src/__smoke__/smoke.test.ts

@@ -3,13 +3,14 @@ import fs from 'fs-extra';
 import nock from 'nock';
 
 import config from 'turtle/config';
-import { PLATFORMS } from 'turtle/constants';
 import * as jobManager from 'turtle/jobManager';
 import logger from 'turtle/logger';
+import { PLATFORMS } from 'turtle/constants';
+
 
 jest.mock('../builders/utils/uploader.ts', () => ({
   uploadBuildToS3: jest.fn(() => {
-    logger.debug('Faking uploading build');
+    console.log('Faking uploading build');
     return 'legit_path';
   }),
 }));
@@ -34,7 +35,7 @@ async function performSmokeTest(recording: string) {
     });
   }
 
-  await jobManager.doJob(logger);
+  await jobManager.doJob();
   if (!fixtureExists) {
     await fs.writeFile(recording, JSON.stringify(nock.recorder.play()));
   }
@@ -67,11 +68,11 @@ function mockFinish() {
     .post('/', (body: any) => {
       const keys = Object.keys(body);
       if (keys.includes('MessageBody')) {
-        const mb = JSON.parse(body.MessageBody);
+        const mb = JSON.parse(body['MessageBody']);
         if (mb.status === 'finished') {
           return true;
         } else if (mb.status === 'errored') {
-          fail('Build ended with error');
+          fail('Build ended with error')
         }
       }
       return false;

src/aws/cloudwatch/index.ts

@@ -1,21 +1,16 @@
 import aws from 'aws-sdk';
 import _ from 'lodash';
 
-import { EXTRA_DIMENSIONS, GROUP_HASH } from 'turtle/aws/cloudwatch/constants';
-import calculateStatistics from 'turtle/aws/cloudwatch/statistics';
-import {
-  IMetric,
-  IMetricConfiguration,
-  IMetricData,
-  IMetricsChunk,
-  MetricRegistrationObject,
-} from 'turtle/aws/cloudwatch/types';
 import config from 'turtle/config';
+import logger from 'turtle/logger';
+import calculateStatistics from 'turtle/aws/cloudwatch/statistics';
+import { EXTRA_DIMENSIONS, GROUP_HASH } from 'turtle/aws/cloudwatch/constants';
+import { Metric, MetricData, MetricConfiguration, MetricRegistrationObject, MetricsChunk } from 'turtle/aws/cloudwatch/types';
 
-interface IState {
-  metrics: IMetric[];
+interface State {
+  metrics: Array<Metric>;
   registeredMetrics: {
-    [key: string]: IMetricConfiguration;
+    [key: string]: MetricConfiguration;
   };
   intervalId: NodeJS.Timer | null;
 }
@@ -24,7 +19,7 @@ const SUCCESS_SUFFIX = '.success';
 const FAIL_SUFFIX = '.fail';
 const CHUNK_SIZE = 20;
 
-const state: IState = {
+const state: State = {
   metrics: [],
   registeredMetrics: {},
   intervalId: null,
@@ -37,12 +32,12 @@ const cloudWatch = new aws.CloudWatch({
   secretAccessKey: config.aws.secretAccessKey,
 });
 
-export function init(logger: any) {
+export function init() {
   if (config.cloudwatch.disabled) {
     return;
   }
 
-  state.intervalId = setInterval(_dumpMetrics(logger), config.cloudwatch.intervalMs);
+  state.intervalId = setInterval(_dumpMetrics, config.cloudwatch.intervalMs);
 }
 
 export function addMetricData({
@@ -51,7 +46,7 @@ export function addMetricData({
   success = true,
   explicitName = false,
   extraDimensions = [],
-}: IMetricData) {
+}: MetricData) {
   if (config.cloudwatch.disabled) {
     return;
   }
@@ -62,7 +57,7 @@ export function addMetricData({
     return;
   }
   const { unit } = registeredMetric;
-  const metric: IMetric = {
+  const metric: Metric = {
     MetricName: metricName,
     Unit: unit,
     Value: value,
@@ -110,28 +105,26 @@ function _calculateGroupHash(metricName: string, extraDimensions: any[]) {
   return `${metricName}${dimString}`;
 }
 
-function _dumpMetrics(logger: any) {
-  return () => {
-    if (!state.metrics.length && Object.keys(state.registeredMetrics).length < 0) {
-      // empty metrics list, skipping
-      return;
-    }
+function _dumpMetrics() {
+  if (!state.metrics.length && Object.keys(state.registeredMetrics).length < 0) {
+    // empty metrics list, skipping
+    return;
+  }
 
-    logger.trace(
-      '[cloudwatch] lengths for state.metrics=%d, state.registeredMetrics=%d',
-      state.metrics.length,
-      Object.keys(state.registeredMetrics).length,
-    );
+  logger.trace(
+    '[cloudwatch] lenghts for state.metrics=%d, state.registeredMetrics=%d',
+    state.metrics.length,
+    Object.keys(state.registeredMetrics).length
+  );
 
-    _addEmptyMetrics(state.metrics);
-    const formattedMetrics = _formatMetrics(state.metrics);
-    _pushMetrics(formattedMetrics, logger);
+  _addEmptyMetrics(state.metrics);
+  const formattedMetrics = _formatMetrics(state.metrics);
+  _pushMetrics(formattedMetrics);
 
-    state.metrics = [];
-  };
+  state.metrics = [];
 }
 
-function _addEmptyMetrics(metrics: IMetric[]) {
+function _addEmptyMetrics(metrics: Array<Metric>) {
   const seenMetrics = metrics.reduce((acc, i) => {
     if (!acc.has(i.MetricName)) {
       acc.add(i.MetricName);
@@ -141,46 +134,46 @@ function _addEmptyMetrics(metrics: IMetric[]) {
 
   const nonEmptyMetrics = _.omitBy(state.registeredMetrics, ({ addEmpty }) => !addEmpty);
   const toAdd = _.difference(Object.keys(nonEmptyMetrics), Array.from(seenMetrics));
-  toAdd.forEach((name) => addMetricData({ name, value: 0, explicitName: true }));
+  toAdd.forEach(name => addMetricData({ name, value: 0, explicitName: true }));
 }
 
-function _formatMetrics(metrics: IMetric[]) {
+function _formatMetrics(metrics: Array<Metric>) {
   const reducedMetrics = _reduceMetrics(metrics);
-  const metricsWithDimensions = reducedMetrics.map((metric) => {
+  const metricsWithDimensions = reducedMetrics.map(metric => {
     const { dimensions } = state.registeredMetrics[metric.MetricName];
     const joinedDimensions = [...dimensions, ...metric[EXTRA_DIMENSIONS]];
-    return joinedDimensions.map((i) => Object.assign({}, metric, { Dimensions: i }));
+    return joinedDimensions.map(i => Object.assign({}, metric, { Dimensions: i }));
   });
   const metricData = _.flatten(metricsWithDimensions);
   const chunks = _.chunk(metricData, CHUNK_SIZE);
-  return chunks.map((chunk) => ({
+  return chunks.map(chunk => ({
     Namespace: config.cloudwatch.namespace,
     MetricData: chunk,
   }));
 }
 
-function _reduceMetrics(metrics: IMetric[]) {
+function _reduceMetrics(metrics: Array<Metric>) {
   const grouped = _.groupBy(metrics, GROUP_HASH);
   return _.reduce(
     grouped,
-    (acc: IMetric[], val: any) => {
-      const { MetricName: key } = val[0] as IMetric;
+    (acc: Array<Metric>, val: any) => {
+      const { MetricName: key } = val[0] as Metric;
       const { reducer, statistics } = state.registeredMetrics[key];
       const reduced = reducer ? reducer(val) : val;
       const withStats = statistics ? calculateStatistics(reduced) : reduced;
       acc.push(...withStats);
       return acc;
     },
-    [],
+    []
   );
 }
 
-function _pushMetrics(metrics: IMetricsChunk[], logger: any) {
+function _pushMetrics(metrics: Array<MetricsChunk>) {
   logger.trace('[cloudwatch] pushing metrics', JSON.stringify(metrics, null, '\t'));
-  metrics.forEach((metricsChunk) => {
-    cloudWatch.putMetricData(metricsChunk, (err) => {
+  metrics.forEach(metricsChunk => {
+    cloudWatch.putMetricData(metricsChunk, err => {
       if (err) {
-        logger.warn({ err }, '[cloudwatch]');
+        logger.warn('[cloudwatch]', err);
       }
     });
   });

src/aws/cloudwatch/reducers.ts

@@ -1,13 +1,13 @@
-import { IMetric } from 'turtle/aws/cloudwatch/types';
+import { Metric } from 'turtle/aws/cloudwatch/types';
 
-export function sum(metrics: IMetric[]) {
+export function sum(metrics: Array<Metric>) {
   if (metrics.length === 0) {
     return [];
   }
-  const reduced = metrics.reduce((acc: IMetric | null, i: IMetric) => {
+  const reduced = metrics.reduce((acc: Metric | null, i: Metric) => {
     let result = acc;
     if (!result) {
-      result = { ...i };
+      result = {...i};
     } else {
       result.Value += i.Value;
     }
@@ -16,12 +16,12 @@ export function sum(metrics: IMetric[]) {
   return [reduced];
 }
 
-export function avg(metrics: IMetric[]) {
+export function avg(metrics: Array<Metric>) {
   if (!metrics.length) {
     return [];
   }
 
-  const metric = sum(metrics)[0] as IMetric;
+  const metric = sum(metrics)[0] as Metric;
   metric.Value = metric.Value / metrics.length;
   return [metric];
 }

src/aws/cloudwatch/statistics.ts

@@ -1,11 +1,11 @@
-import { IMetric, MetricWithStats } from 'turtle/aws/cloudwatch/types';
+import { Metric, MetricWithStats } from 'turtle/aws/cloudwatch/types';
 
-export default function calculateStatistics(metrics: IMetric[]) {
+export default function calculateStatistics(metrics: Array<Metric>) {
   if (!metrics.length) {
     return [];
   }
 
-  const reduced = metrics.reduce((acc: MetricWithStats | null, i: IMetric) => {
+  const reduced = metrics.reduce((acc: MetricWithStats | null, i: Metric) => {
     let result = acc;
     if (!result) {
       result = Object.assign({}, i, {

src/aws/cloudwatch/types.ts

@@ -1,6 +1,6 @@
 import { EXTRA_DIMENSIONS, GROUP_HASH } from 'turtle/aws/cloudwatch/constants';
 
-export interface IMetric {
+export interface Metric {
   MetricName: string;
   Unit: string;
   Value: number;
@@ -11,8 +11,8 @@ export interface IMetric {
     SampleCount: number;
     Sum: number;
   };
-  [EXTRA_DIMENSIONS]: any;
-  [GROUP_HASH]: string;
+  [EXTRA_DIMENSIONS]: any,
+  [GROUP_HASH]: string,
 }
 
 export type MetricWithStats = {
@@ -21,10 +21,10 @@ export type MetricWithStats = {
     Minimum: number;
     SampleCount: number;
     Sum: number;
-  },
-} & IMetric;
+  }
+} & Metric;
 
-export interface IMetricData {
+export interface MetricData {
   name: string;
   value: number;
   success?: boolean;
@@ -32,17 +32,17 @@ export interface IMetricData {
   extraDimensions?: any;
 }
 
-export interface IMetricConfiguration {
+export interface MetricConfiguration {
   unit: string;
   statistics?: boolean;
   reducer?: any;
   addEmpty?: boolean;
   dimensions?: any;
-}
+};
 
-export type MetricRegistrationObject = { metricName: string, explicitName?: boolean } & IMetricConfiguration;
+export type MetricRegistrationObject = { metricName: string, explicitName?: boolean } & MetricConfiguration;
 
-export interface IMetricsChunk {
+export interface MetricsChunk {
   Namespace: string;
-  MetricData: IMetric[];
-}
+  MetricData: Array<Metric>;
+};

src/aws/s3.ts

@@ -1,10 +1,10 @@
 import AWS from 'aws-sdk';
-import fs from 'fs-extra';
 import _ from 'lodash';
+import fs from 'fs-extra';
 
 import config from 'turtle/config';
 
-interface IFileUploadParams {
+interface FileUploadParams {
   bucketName?: string;
   key: string;
   srcPath: string;
@@ -17,7 +17,7 @@ const s3 = new AWS.S3({
 
 export default s3;
 
-export async function uploadFile({ bucketName = config.s3.bucket, key, srcPath }: IFileUploadParams) {
+export async function uploadFile({ bucketName = config.s3.bucket, key, srcPath }: FileUploadParams) {
   return await s3
     .upload({
       Bucket: bucketName,

src/aws/sqs.ts

@@ -2,10 +2,11 @@ import retry from 'async-retry';
 import AWS from 'aws-sdk';
 import _ from 'lodash';
 
-import { OUTPUT_QUEUE_URL, QUEUE_URL } from 'turtle/aws/utils';
 import config from 'turtle/config';
 import { JOB_UPDATE_TYPE } from 'turtle/constants/build';
+import logger from 'turtle/logger';
 import { getCurrentJobId } from 'turtle/turtleContext';
+import { QUEUE_URL, OUTPUT_QUEUE_URL } from 'turtle/aws/utils';
 
 const sqs = new AWS.SQS({
   apiVersion: '2012-11-05',
@@ -15,7 +16,7 @@ const sqs = new AWS.SQS({
 
 const VISIBILITY_TIMEOUT_SEC = 30;
 
-export async function receiveMessage(priority: string, logger: any) {
+export async function receiveMessage(priority: string) {
   const params = {
     MaxNumberOfMessages: 1,
     QueueUrl: QUEUE_URL(priority),
@@ -25,7 +26,7 @@ export async function receiveMessage(priority: string, logger: any) {
     return sqs
       .receiveMessage(params)
       .promise()
-      .then((data) => {
+      .then(data => {
         if (data.Messages) {
           logger.info(`Received a message`);
           return data.Messages[0];
@@ -34,8 +35,8 @@ export async function receiveMessage(priority: string, logger: any) {
           return null;
         }
       })
-      .catch((err) => {
-        logger.error({ err }, 'Receive error');
+      .catch(err => {
+        logger.error('Receive error', err);
         throw err;
       });
   });
@@ -61,19 +62,19 @@ export async function changeMessageVisibility(priority: string, receiptHandle: s
 }
 
 // Every VISIBILITY_TIMEOUT_SEC / 3 seconds we are telling AWS SQS
-// that we're still processing the message, so it does not
+// that we're still proccessing the message, so it does not
 // send the build job to another turtle agent
-export function changeMessageVisibilityRecurring(priority: string, receiptHandle: string, jobId: string, logger: any) {
+export function changeMessageVisibilityRecurring(priority: string, receiptHandle: string, jobId: string) {
   return setInterval(() => {
     if (getCurrentJobId() === jobId) {
-      changeMessageVisibility(priority, receiptHandle).catch((err) => {
-        logger.warn({ err }, 'Error at change msg visibility');
+      changeMessageVisibility(priority, receiptHandle).catch(err => {
+        logger.warn('Error at change msg visibility', err);
       });
     }
   }, (VISIBILITY_TIMEOUT_SEC * 1000) / 3);
 }
 
-export async function sendMessage(jobId: string, status: JOB_UPDATE_TYPE, data = {}, logger: any) {
+export async function sendMessage(jobId: string, status: JOB_UPDATE_TYPE, data = {}) {
   const params = {
     MessageBody: JSON.stringify({ jobId, status, ...data }),
     QueueUrl: OUTPUT_QUEUE_URL(),
@@ -85,6 +86,6 @@ export async function sendMessage(jobId: string, status: JOB_UPDATE_TYPE, data =
       return await sqs.sendMessage(params).promise();
     });
   } catch (err) {
-    logger.error({ err }, `Error sending SQS message: ${JSON.stringify(params)}}`);
+    logger.error(`Error sending SQS message: ${JSON.stringify(params)}}`, err);
   }
 }

src/bin/commands/ErrorWithCommandHelp.ts

@@ -1,2 +0,0 @@
-export class ErrorWithCommandHelp extends Error {
-}

src/bin/commands/ErrorWithProgramHelp.ts

@@ -1,2 +0,0 @@
-export class ErrorWithProgramHelp extends Error {
-}

src/bin/commands/build/android.ts

@@ -1,11 +1,11 @@
-import fs from 'fs-extra';
 import _ from 'lodash';
+import fs from 'fs-extra';
 import uuid from 'uuid';
 
-import { ErrorWithCommandHelp } from 'turtle/bin/commands/ErrorWithCommandHelp';
-import { createBuilderAction } from 'turtle/bin/utils/builder';
 import builder from 'turtle/builders/android';
+import { ErrorWithCommandHelp } from 'turtle/bin/commands/errors';
 import { PLATFORMS } from 'turtle/constants';
+import { createBuilderAction } from 'turtle/bin/utils/builder';
 
 export default (program: any, setCommonCommandOptions: any) => {
   const command = program.command('build:android [project-dir]');
@@ -14,12 +14,11 @@ export default (program: any, setCommonCommandOptions: any) => {
     .alias('ba')
     .option(
       '--keystore-path <app.jks>',
-      'path to your Keystore (please provide Keystore password and Key password as EXPO_ANDROID_KEYSTORE_PASSWORD'
-      + 'and EXPO_ANDROID_KEY_PASSWORD env variables)',
+      'path to your Keystore (please provide Keystore password and Key password as EXPO_ANDROID_KEYSTORE_PASSWORD and EXPO_ANDROID_KEY_PASSWORD env variables)'
     )
     .option('--keystore-alias <alias>', 'keystore Alias')
     .description(
-      'Build a standalone APK for your project, signed and ready for submission to the Google Play Store.',
+      'Build a standalone APK for your project, signed and ready for submission to the Google Play Store.'
     )
     .asyncAction(
       createBuilderAction({
@@ -30,7 +29,7 @@ export default (program: any, setCommonCommandOptions: any) => {
         builder,
         platform: PLATFORMS.ANDROID,
         os: ['darwin', 'linux'],
-      }),
+      })
     );
 };
 
@@ -58,7 +57,7 @@ const prepareCredentials = async (cmd: any) => {
   if (!credentialsExist) {
     if (someCredentialsExist) {
       throw new ErrorWithCommandHelp(
-        'Please provide all required credentials - Keystore (with password), Keystore alias and Key password',
+        'Please provide all required credentials - Keystore (with password), Keystore alias and Key password'
       );
     } else {
       return null;

src/bin/commands/build/index.ts

@@ -1,9 +1,9 @@
-import { default as buildAndroid } from 'turtle/bin/commands/build/android';
 import { default as buildIOS } from 'turtle/bin/commands/build/ios';
+import { default as buildAndroid } from 'turtle/bin/commands/build/android';
 
 function createCommand(builder: any) {
-  return (logger: any) => (program: any) => {
-    return builder(program, setCommonCommandOptions, logger);
+  return (program: any) => {
+    return builder(program, setCommonCommandOptions);
   };
 }
 
@@ -15,16 +15,15 @@ function setCommonCommandOptions(cmd: any) {
     .option('-o --output <output-file-path>', 'output file path')
     .option(
       '--public-url <url>',
-      'the URL of an externally hosted manifest (for self-hosted apps).',
+      'the URL of an externally hosted manifest (for self-hosted apps).'
     )
     .option(
       '--release-channel <channel-name>',
-      'pull from specified release channel (default: default)',
-    )
-    .option('-c --config <config-file>', 'specify a path to app.json');
+      'pull from specified release channel (default: default)'
+    );
 }
 
 export default {
   ios: createCommand(buildIOS),
   android: createCommand(buildAndroid),
-};
+}

src/bin/commands/build/ios.ts

@@ -1,11 +1,11 @@
-import fs from 'fs-extra';
 import _ from 'lodash';
 import uuid from 'uuid';
+import fs from 'fs-extra';
 
-import { ErrorWithCommandHelp } from 'turtle/bin/commands/ErrorWithCommandHelp';
-import { createBuilderAction } from 'turtle/bin/utils/builder';
 import builder from 'turtle/builders/ios';
+import { ErrorWithCommandHelp } from 'turtle/bin/commands/errors';
 import { IOS, PLATFORMS } from 'turtle/constants';
+import { createBuilderAction } from 'turtle/bin/utils/builder';
 
 export default (program: any, setCommonCommandOptions: any) => {
   const command = program.command('build:ios [project-dir]');
@@ -16,16 +16,16 @@ export default (program: any, setCommonCommandOptions: any) => {
       '-t --type <build>',
       'type of build: archive|simulator',
       /^(archive|simulator)$/i,
-      'archive',
+      'archive'
     )
     .option('--team-id <apple-teamId>', 'Apple Team ID')
     .option(
       '--dist-p12-path <dist.p12>',
-      'path to your Distribution Certificate P12 (please provide password as EXPO_IOS_DIST_P12_PASSWORD env variable)',
+      'path to your Distribution Certificate P12 (please provide password as EXPO_IOS_DIST_P12_PASSWORD env variable)'
     )
     .option('--provisioning-profile-path <.mobileprovision>', 'path to your Provisioning Profile')
     .description(
-      'Build a standalone IPA for your project, signed and ready for submission to the Apple App Store.',
+      'Build a standalone IPA for your project, signed and ready for submission to the Apple App Store.'
     )
     .asyncAction(
       createBuilderAction({
@@ -36,7 +36,7 @@ export default (program: any, setCommonCommandOptions: any) => {
         builder,
         platform: PLATFORMS.IOS,
         os: 'darwin',
-      }),
+      })
     );
 };
 
@@ -68,8 +68,7 @@ const prepareCredentials = async (cmd: any) => {
 
   if (!credentialsExist) {
     throw new ErrorWithCommandHelp(
-      'Please provide all required credentials'
-      + '- Apple Team ID, Distribution Certificate P12 (with password) and Provisioning Profile',
+      'Please provide all required credentials - Apple Team ID, Distribution Certificate P12 (with password) and Provisioning Profile'
     );
   }
 

src/bin/commands/errors.ts

@@ -0,0 +1,2 @@
+export class ErrorWithProgramHelp extends Error {}
+export class ErrorWithCommandHelp extends Error {}

src/bin/commands/index.ts

@@ -1,9 +1,8 @@
 import builders from 'turtle/bin/commands/build';
 import setup from 'turtle/bin/commands/setup';
 
-export default {
-  setupIOS: setup.ios,
-  setupAndroid: setup.android,
-  buildIOS: builders.ios,
-  buildAndroid: builders.android,
-};
+export const setupIOS = setup.ios;
+export const setupAndroid = setup.android;
+
+export const buildIOS = builders.ios;
+export const buildAndroid = builders.android;

src/bin/commands/setup.ts

@@ -1,10 +1,12 @@
 import _ from 'lodash';
 
-import { ErrorWithCommandHelp } from 'turtle/bin/commands/ErrorWithCommandHelp';
-import { ErrorWithProgramHelp } from 'turtle/bin/commands/ErrorWithProgramHelp';
+import logger from 'turtle/logger';
 import setup from 'turtle/bin/setup/setup';
+import { ErrorWithCommandHelp, ErrorWithProgramHelp } from 'turtle/bin/commands/errors';
 import { PLATFORMS } from 'turtle/constants';
 
+const l = logger.withFields({ buildPhase: 'setting up environment' });
+
 function createSetupCommand(platform: string, os?: string) {
   return (program: any) => {
     const command = program.command(`setup:${platform}`);
@@ -14,25 +16,24 @@ function createSetupCommand(platform: string, os?: string) {
       .description(`Setup environment for building ${_.capitalize(platform)} standalone apps.`)
       .option(
         '--sdk-version <sdk-version>',
-        'version of Expo SDK shell app to download (optional)',
+        'version of Expo SDK shell app to download (optional)'
       )
       .asyncAction(async (cmd: any) => await setupAction(program, cmd, platform, os));
   };
 }
 
-async function setupAction(program: any, cmd: any, platform: string, logger: any, os?: string) {
+async function setupAction(program: any, cmd: any, platform: string, os?: string) {
   try {
     if (os && process.platform !== os) {
       throw new Error('We don\'t support running standalone app builds for this platform on your operating system');
     }
 
     const { sdkVersion } = cmd;
-    const l = logger.child({ buildPhase: 'setting up environment' });
-
-    await setup(platform, l, sdkVersion);
-    l.info('it\'s all set!');
+    await setup(platform, sdkVersion);
+    l.info('it\'s all set!')
   } catch (err) {
-    logger.error({ err }, `Failed to setup environment for ${platform} builds`);
+    logger.error(`Failed to setup environment for ${platform} builds`);
+    logger.error(err.stack);
     if (err instanceof ErrorWithCommandHelp) {
       cmd.help();
     } else if (err instanceof ErrorWithProgramHelp) {

src/bin/env.ts

@@ -1,5 +1,5 @@
-import os from 'os';
 import path from 'path';
+import os from 'os';
 
 import _ from 'lodash';
 
@@ -16,8 +16,8 @@ const resolveEnv = () => {
 };
 
 const currentEnv = resolveEnv();
-const isLocal = (envName: string) => envName === 'local';
-const isStaging = (envName: string) => envName === 'staging';
+const isLocal = (env: string) => env === 'local';
+const isStaging = (env: string) => env === 'staging';
 
 const apiConfig = {
   protocol: 'https',

src/bin/index.ts

@@ -1,13 +1,11 @@
-import path from 'path';
-
-import program, { Command } from 'commander';
-import fs from 'fs-extra';
 import { LoggerDetach, ModuleVersion } from 'xdl';
+import program, { Command } from 'commander';
+import _ from 'lodash';
 
-import commands from 'turtle/bin/commands';
 import logger from 'turtle/logger';
+import * as commands from 'turtle/bin/commands';
 
-const { name, version } = JSON.parse(fs.readFileSync(path.join(__dirname, '../../package.json'), 'utf-8'));
+const { name, version } = require('../../package.json');
 
 const ModuleVersionChecker = ModuleVersion.createModuleVersionChecker(name, version);
 
@@ -17,38 +15,36 @@ Command.prototype.asyncAction = function asyncAction(asyncFn: (...asyncFnArgs: a
   return this.action(async (...args) => {
     try {
       await checkForUpdateAsync();
-    } catch (err) {
-      logger.warn({ err }, 'Failed to check for turtle-cli update.');
-    }
+    } catch (e) {}
     return await asyncFn(...args);
   });
-};
+}
 
 export function run(programName: string) {
-  runAsync(programName).catch((err) => {
-    logger.error({ err }, 'Uncaught Error');
+  runAsync(programName).catch(e => {
+    logger.error('Uncaught Error', e);
     process.exit(1);
   });
 }
 
 async function runAsync(programName: string) {
   program.version(version);
-  Object.values(commands).map((command) => command(logger)).forEach((command) => registerCommand(program, command));
+  Object.values(commands).forEach(command => registerCommand(program, command));
   program.parse(process.argv);
 
   const subCommand = process.argv[2];
   if (subCommand) {
-    const commandNames = program.commands.reduce((acc: string[], command: any) => {
-      acc.push(command._name);
-      const alias = command._alias;
+    const commands = program.commands.reduce((acc: Array<string>, command: any) => {
+      acc.push(command['_name']);
+      const alias = command['_alias'];
       if (alias) {
         acc.push(alias);
       }
       return acc;
     }, []);
-    if (!commandNames.includes(subCommand)) {
+    if (!_.includes(commands, subCommand)) {
       logger.error(
-        `"${subCommand}" is not an ${programName} command. See "${programName} --help" for the full list of commands.`,
+        `"${subCommand}" is not an ${programName} command. See "${programName} --help" for the full list of commands.`
       );
     }
   } else {
@@ -64,7 +60,7 @@ async function checkForUpdateAsync() {
     logger.warn(
       `There is a new version of ${name} available (${latest}).
 You are currently using ${name} ${current}
-Run \`npm install -g ${name}\` to get the latest version`,
+Run \`npm install -g ${name}\` to get the latest version`
     );
   }
 }

src/bin/setup/android/index.ts

@@ -2,19 +2,20 @@ import path from 'path';
 import util from 'util';
 
 import fs from 'fs-extra';
-import _which from 'which';
 import { ExponentTools } from 'xdl';
+import _which from 'which';
 
-import ensureAndroidNDKIsPresent from 'turtle/bin/setup/android/ndk';
-import ensureAndroidSDKIsPresent from 'turtle/bin/setup/android/sdk';
-import { checkSystem, ensureShellAppIsPresent } from 'turtle/bin/setup/utils/common';
-import { IToolDefinition } from 'turtle/bin/setup/utils/toolsDetector';
-import { formatShellAppDirectory } from 'turtle/builders/utils/android/workingdir';
+import logger from 'turtle/logger';
 import config from 'turtle/config';
+import { IToolDefinition } from 'turtle/bin/setup/utils/toolsDetector';
+import { checkSystem, ensureShellAppIsPresent } from 'turtle/bin/setup/utils/common';
+import { formatShellAppDirectory } from 'turtle/builders/utils/android/workingdir';
+import ensureAndroidSDKIsPresent from 'turtle/bin/setup/android/sdk';
+import ensureAndroidNDKIsPresent from 'turtle/bin/setup/android/ndk';
 import { PLATFORMS } from 'turtle/constants';
 
 const which = util.promisify(_which);
-const REQUIRED_TOOLS: IToolDefinition[] = [
+const REQUIRED_TOOLS: Array<IToolDefinition> = [
   {
     command: 'bash',
     missingDescription: 'Please install bash',
@@ -30,33 +31,34 @@ const REQUIRED_TOOLS: IToolDefinition[] = [
       const { status, stdout } = await ExponentTools.spawnAsyncThrowError(
         'javac',
         ['-version'],
-        { stdio: 'pipe' },
+        { stdio: 'pipe' }
       );
       if (stdout.match(/no java runtime present/i)) {
         return false;
       }
       return status === 0;
-    },
+    }
   },
 ];
+const LOGGER_FIELDS = { buildPhase: 'setting up environment' };
+const l = logger.withFields(LOGGER_FIELDS);
 
-export default async function setup(logger: any, sdkVersion?: string) {
-  await checkSystem(REQUIRED_TOOLS, logger);
-  await prepareAndroidEnv(logger);
+export default async function setup(sdkVersion?: string) {
+  await checkSystem(REQUIRED_TOOLS);
+  await prepareAndroidEnv();
   if (sdkVersion) {
     await ensureShellAppIsPresent(
       sdkVersion,
       { formatShellAppDirectory, formatShellAppTarballUriPath },
-      logger,
-      _shellAppPostDownloadAction,
+      _shellAppPostDownloadAction
     );
   }
 }
 
-async function prepareAndroidEnv(logger: any) {
+async function prepareAndroidEnv() {
   await fs.ensureDir(config.directories.androidDependenciesDir);
-  const sdkConfig = await ensureAndroidSDKIsPresent(logger);
-  const ndkConfig = await ensureAndroidNDKIsPresent(logger);
+  const sdkConfig = await ensureAndroidSDKIsPresent();
+  const ndkConfig = await ensureAndroidNDKIsPresent();
   _setEnvVars({ ...sdkConfig.envVars, ...ndkConfig.envVars });
   _alterPath([...sdkConfig.path, ...sdkConfig.path]);
 }
@@ -65,23 +67,21 @@ function formatShellAppTarballUriPath(sdkMajorVersion: string) {
   return path.join(config.directories.shellTarballsDir, PLATFORMS.ANDROID, `sdk${sdkMajorVersion}`);
 }
 
-async function _shellAppPostDownloadAction(workingdir: string, logger: any) {
+async function _shellAppPostDownloadAction(workingdir: string) {
   const inWorkingdir = (filename: string) => path.join(workingdir, filename);
 
   await fs.move(inWorkingdir('package.json'), inWorkingdir('exponent-package.json'));
   await fs.move(inWorkingdir('universe-package.json'), inWorkingdir('package.json'));
-  await _installNodeModules(workingdir, logger);
+  await _installNodeModules(workingdir);
   await fs.move(inWorkingdir('package.json'), inWorkingdir('universe-package.json'));
   await fs.move(inWorkingdir('exponent-package.json'), inWorkingdir('package.json'));
 
   // TODO: remove following lines after upgrading android shell app
   const toolsPublicDir = path.join(workingdir, 'tools-public');
-  await _installNodeModules(toolsPublicDir, logger);
+  await _installNodeModules(toolsPublicDir);
 }
 
-async function _installNodeModules(cwd: string, logger: any) {
-  const LOGGER_FIELDS = { buildPhase: 'setting up environment' };
-  const l = logger.child(LOGGER_FIELDS);
+async function _installNodeModules(cwd: string) {
   l.info(`installing dependencies in ${cwd} directory...`);
   const command = await _shouldUseYarnOrNpm();
   await ExponentTools.spawnAsyncThrowError(command, ['install'], {
@@ -107,7 +107,7 @@ function _setEnvVars(envVars: object) {
     .forEach(([key, value]) => process.env[key] = value);
 }
 
-function _alterPath(newPaths: string[]) {
+function _alterPath(newPaths: Array<string>) {
   const currentPaths = process.env.PATH ? process.env.PATH.split(':') : [];
   const paths = [...newPaths, ...currentPaths];
   process.env.PATH = paths.join(':');

src/bin/setup/android/ndk.ts

@@ -4,24 +4,27 @@ import path from 'path';
 import decompress from 'decompress';
 import fs from 'fs-extra';
 
-import { formatArtifactDownloadPath } from 'turtle/bin/setup/utils/common';
-import download from 'turtle/bin/setup/utils/downloader';
 import config from 'turtle/config';
+import logger from 'turtle/logger';
+import download from 'turtle/bin/setup/utils/downloader';
+import { formatArtifactDownloadPath } from 'turtle/bin/setup/utils/common';
 
 const ANDROID_NDK_URL = `https://dl.google.com/android/repository/android-ndk-r10e-${os.platform()}-x86_64.zip`;
 
-export default async function ensureAndroidNDKIsPresent(logger: any) {
+const LOGGER_FIELDS = { buildPhase: 'setting up environment' };
+const l = logger.withFields(LOGGER_FIELDS);
+
+export default async function ensureAndroidNDKIsPresent() {
   const androidNdkDir = path.join(config.directories.androidDependenciesDir, 'ndk');
   if (!(await fs.pathExists(androidNdkDir))) {
     await fs.ensureDir(androidNdkDir);
     const androidNdkDownloadPath = formatArtifactDownloadPath(ANDROID_NDK_URL);
 
     try {
-      const l = logger.child({ buildPhase: 'setting up environment' });
       l.info('Downloading Android NDK');
       await fs.ensureDir(config.directories.artifactsDir);
       await download(ANDROID_NDK_URL, androidNdkDownloadPath);
-      l.info('Decompressing Android NDK');
+      l.info('Decompressing Android NDK')
       await decompress(androidNdkDownloadPath, androidNdkDir, { strip: 1 });
       l.info('Android NDK installed successfully');
     } catch (err) {
@@ -44,5 +47,5 @@ function _createEnvVars(androidNdkDir: string) {
 }
 
 function _createPaths(androidNdkDir: string) {
-  return [androidNdkDir];
+  return [androidNdkDir]
 }

src/bin/setup/android/sdk.ts

@@ -5,25 +5,28 @@ import decompress from 'decompress';
 import fs from 'fs-extra';
 import { ExponentTools } from 'xdl';
 
-import { formatArtifactDownloadPath } from 'turtle/bin/setup/utils/common';
-import download from 'turtle/bin/setup/utils/downloader';
 import config from 'turtle/config';
+import logger from 'turtle/logger';
+import download from 'turtle/bin/setup/utils/downloader';
+import { formatArtifactDownloadPath } from 'turtle/bin/setup/utils/common';
+
+const ANDROID_SDK_URL = 'https://dl.google.com/android/repository/sdk-tools-linux-3859397.zip';
 
-const ANDROID_SDK_URL = 'https://dl.google.com/android/repository/sdk-tools-linux-4333796.zip';
 const LOGGER_FIELDS = { buildPhase: 'setting up environment' };
-export default async function ensureAndroidSDKIsPresent(logger: any) {
+const l = logger.withFields(LOGGER_FIELDS);
+
+export default async function ensureAndroidSDKIsPresent() {
   const androidSdkDir = path.join(config.directories.androidDependenciesDir, 'sdk');
   if (!(await fs.pathExists(androidSdkDir))) {
     await fs.ensureDir(androidSdkDir);
     const androidSdkDownloadPath = formatArtifactDownloadPath(ANDROID_SDK_URL);
 
     try {
-      const l = logger.child(LOGGER_FIELDS);
       l.info('Downloading Android SDK');
       await fs.ensureDir(config.directories.artifactsDir);
       await download(ANDROID_SDK_URL, androidSdkDownloadPath);
       await fs.ensureDir(androidSdkDir);
-      l.info('Decompressing Android SDK');
+      l.info('Decompressing Android SDK')
       await decompress(androidSdkDownloadPath, androidSdkDir);
       await fs.remove(androidSdkDownloadPath);
       l.info('Configuring Android SDK, this may take a while');
@@ -70,6 +73,6 @@ function _createPaths(androidSdkDir: string) {
     path.join(androidSdkDir, 'platform-tools'),
     path.join(androidSdkDir, 'tools'),
     path.join(androidSdkDir, 'tools/bin'),
-    path.join(androidSdkDir, 'build-tools/28.0.3'),
+    path.join(androidSdkDir, 'build-tools/25.0.0'),
   ];
 }

src/bin/setup/ios.ts

@@ -2,13 +2,13 @@ import path from 'path';
 
 import { ExponentTools } from 'xdl';
 
+import config from 'turtle/config';
 import { checkSystem, ensureShellAppIsPresent } from 'turtle/bin/setup/utils/common';
 import { IToolDefinition } from 'turtle/bin/setup/utils/toolsDetector';
 import { formatShellAppDirectory } from 'turtle/builders/utils/ios/workingdir';
-import config from 'turtle/config';
 import { PLATFORMS } from 'turtle/constants';
 
-const REQUIRED_TOOLS: IToolDefinition[] = [
+const REQUIRED_TOOLS: Array<IToolDefinition> = [
   {
     command: 'bash',
     missingDescription: 'Please install bash',
@@ -24,23 +24,23 @@ const REQUIRED_TOOLS: IToolDefinition[] = [
       const { status, stdout } = await ExponentTools.spawnAsyncThrowError(
         'xcodebuild',
         ['-version'],
-        { stdio: 'pipe' },
+        { stdio: 'pipe' }
       );
       if (stdout.match(/requires xcode/i)) {
         return false;
       }
       return status === 0;
-    },
+    }
   },
 ];
 
-export default async function setup(logger: any, sdkVersion?: string) {
-  await checkSystem(REQUIRED_TOOLS, logger);
+export default async function setup(sdkVersion?: string) {
+  await checkSystem(REQUIRED_TOOLS);
   if (sdkVersion) {
     await ensureShellAppIsPresent(sdkVersion, {
       formatShellAppDirectory,
       formatShellAppTarballUriPath,
-    }, logger);
+    });
   }
 }
 

src/bin/setup/setup.ts

@@ -1,18 +1,18 @@
 import isSemver from 'is-semver';
 
-import setupAndroid from 'turtle/bin/setup/android';
 import setupIos from 'turtle/bin/setup/ios';
+import setupAndroid from 'turtle/bin/setup/android';
 import { PLATFORMS } from 'turtle/constants';
 
-export default async function setup(platform: string, logger: any, sdkVersion?: string) {
+export default async function setup(platform: string, sdkVersion?: string) {
   if (sdkVersion && !isSemver(sdkVersion)) {
     throw new Error('SDK version is not valid.');
   }
 
   if (platform === PLATFORMS.IOS) {
-    return await setupIos(logger, sdkVersion);
+    return await setupIos(sdkVersion);
   } else if (platform === PLATFORMS.ANDROID) {
-    return await setupAndroid(logger, sdkVersion);
+    return await setupAndroid(sdkVersion);
   } else {
     throw new Error('This should never happen :(');
   }

src/bin/setup/utils/common.ts

@@ -1,63 +1,56 @@
 import path from 'path';
 
+import { ExponentTools } from 'xdl';
 import fs from 'fs-extra';
 import tar from 'tar';
-import { ExponentTools } from 'xdl';
+
+import config from 'turtle/config';
+import logger from 'turtle/logger';
+import { IToolDefinition, ensureToolsAreInstalled } from 'turtle/bin/setup/utils/toolsDetector';
 
 import download from 'turtle/bin/setup/utils/downloader';
-import { ensureToolsAreInstalled, IToolDefinition } from 'turtle/bin/setup/utils/toolsDetector';
-import { IShellAppDirectoryConfig } from 'turtle/builders/utils/workingdir';
-import config from 'turtle/config';
+
+const l = logger.withFields({ buildPhase: 'setting up environment' });
 
 interface IShellAppFormaters {
-  formatShellAppDirectory: (config: IShellAppDirectoryConfig) => string;
+  formatShellAppDirectory: (sdkVersion: string) => string;
   formatShellAppTarballUriPath: (sdkMajorVersion: string) => string;
 }
 
-type PostDownloadAction = (workingdir: string, logger: any) => void;
+type PostDownloadAction = (workingdir: string) => void;
 
-export async function checkSystem(requiredTools: IToolDefinition[], logger: any) {
-  await ensureToolsAreInstalled(requiredTools, logger);
+export async function checkSystem(requiredTools: Array<IToolDefinition>) {
+  await ensureToolsAreInstalled(requiredTools);
 }
 
-export async function ensureShellAppIsPresent(
-  sdkVersion: string,
-  formatters: IShellAppFormaters,
-  logger: any,
-  postDownloadAction?: PostDownloadAction,
-) {
-  const workingdir = formatters.formatShellAppDirectory({ sdkVersion });
-  const l = logger.child({ buildPhase: 'setting up environment' });
+export async function ensureShellAppIsPresent(sdkVersion: string, formatters: IShellAppFormaters, postDownloadAction?: PostDownloadAction) {
+  const workingdir = formatters.formatShellAppDirectory(sdkVersion);
   if (await fs.pathExists(workingdir)) {
     return;
   }
   l.info(`shell app for SDK ${sdkVersion} doesn't exist, downloading...`);
-  await _downloadShellApp(sdkVersion, workingdir, formatters, l);
+  await _downloadShellApp(sdkVersion, workingdir, formatters);
   if (postDownloadAction) {
-    await postDownloadAction(workingdir, logger);
+    await postDownloadAction(workingdir);
   }
 }
 
-async function _downloadShellApp(
-  sdkVersion: string,
-  targetDirectory: string,
-  formatters: IShellAppFormaters,
-  logger: any,
-) {
+async function _downloadShellApp(sdkVersion: string, targetDirectory: string, formatters: IShellAppFormaters) {
   const shellAppTarballS3Uri = await _readShellAppTarballS3Uri(sdkVersion, formatters);
   const tarballDownloadTargetPath = formatArtifactDownloadPath(shellAppTarballS3Uri);
   await fs.ensureDir(config.directories.artifactsDir);
   await download(shellAppTarballS3Uri, tarballDownloadTargetPath);
-  logger.info('shell app has been downloaded');
+  l.info('shell app has been downloaded');
   await fs.ensureDir(targetDirectory);
-  logger.info('extracting shell app (this may take a while)...');
+  l.info('extracting shell app (this may take a while)...');
   await tar.x({
     file: tarballDownloadTargetPath,
-    C: targetDirectory,
+    C: targetDirectory
   });
-  logger.info('shell app extracted');
+  l.info('shell app extracted');
 }
 
+
 export function formatArtifactDownloadPath(uri: string) {
   const { base } = path.parse(uri);
   return path.join(config.directories.artifactsDir, base);

src/bin/setup/utils/downloader.ts

@@ -1,9 +1,9 @@
-import fs from 'fs';
 import { URL } from 'url';
+import fs from 'fs';
 
-import ProgressBar from 'progress';
 import request from 'request';
 import requestProgress from 'request-progress';
+import ProgressBar from 'progress';
 
 const S3_PROTOCOL = 's3:';
 const PROGRESS_BAR_CONFIG = {
@@ -38,6 +38,7 @@ export default async function download(url: string, destPath: string) {
   });
 }
 
+
 function convertS3ToHttpsUrl(s3UrlRaw: string) {
   const { host: s3Bucket, pathname: s3Path, protocol } = new URL(s3UrlRaw);
   if (protocol !== S3_PROTOCOL) {

src/bin/setup/utils/toolsDetector.ts

@@ -1,7 +1,10 @@
 import util from 'util';
 import _which from 'which';
 
+import logger from 'turtle/logger';
+
 const which = util.promisify(_which);
+const l = logger.withFields({ buildPhase: 'setting up environment' });
 
 export interface IToolDefinition {
   command: string;
@@ -9,7 +12,7 @@ export interface IToolDefinition {
   testFn?: () => Promise<boolean>;
 }
 
-export async function ensureToolsAreInstalled(tools: IToolDefinition[], logger: any) {
+export async function ensureToolsAreInstalled(tools: Array<IToolDefinition>) {
   let isAnyToolMissing = false;
   for (const { command, missingDescription, testFn } of tools) {
     try {
@@ -21,12 +24,11 @@ export async function ensureToolsAreInstalled(tools: IToolDefinition[], logger:
         await which(command);
       }
     } catch (err) {
-      const l = logger.child({ buildPhase: 'setting up environment' });
       isAnyToolMissing = true;
       if (!testFn) {
-        l.error({ err }, `${command} is missing in your $PATH`);
+        l.error(`${command} is missing in your $PATH`);
       }
-      l.error({ err }, missingDescription);
+      l.error(missingDescription);
     }
   }
   if (isAnyToolMissing) {

src/bin/utils/builder.ts

@@ -1,14 +1,13 @@
 import _ from 'lodash';
 
-import { ErrorWithCommandHelp } from 'turtle/bin/commands/ErrorWithCommandHelp';
-import { ErrorWithProgramHelp } from 'turtle/bin/commands/ErrorWithProgramHelp';
-import setup from 'turtle/bin/setup/setup';
-import * as ProjectUtils from 'turtle/bin/utils/project';
-import * as UserUtils from 'turtle/bin/utils/user';
+import logger from 'turtle/logger';
 import { sanitizeJob } from 'turtle/validator';
+import * as UserUtils from 'turtle/bin/utils/user';
+import * as ProjectUtils from 'turtle/bin/utils/project';
+import { ErrorWithCommandHelp, ErrorWithProgramHelp } from 'turtle/bin/commands/errors';
+import setup from 'turtle/bin/setup/setup';
 
 export function createBuilderAction({
-  logger,
   program,
   command,
   prepareCredentials,
@@ -30,9 +29,7 @@ export function createBuilderAction({
       }
 
       if (cmd.buildDir && cmd.output) {
-        throw new ErrorWithCommandHelp(
-          'You can\'t provide both --build-dir and --output parameters, please choose one',
-        );
+        throw new ErrorWithCommandHelp('You can\'t provide both --build-dir and --output parameters, please choose one');
       }
 
       const userData = {
@@ -47,11 +44,10 @@ export function createBuilderAction({
           throw new ErrorWithCommandHelp('You must provide both Expo username and password');
         }
       } else if (!cmd.publicUrl) {
-        throw new ErrorWithCommandHelp(
-          'You must provide your Expo username and password unless you specify --public-url to your project manifest.',
-        );
+        throw new ErrorWithCommandHelp('You must provide your Expo username and password unless you specify --public-url to your project manifest.');
       }
 
+
       const args = {
         releaseChannel: cmd.releaseChannel || 'default',
         buildType: cmd.type,
@@ -60,9 +56,9 @@ export function createBuilderAction({
         publicUrl: cmd.publicUrl,
       };
 
-      const appJSON = await ProjectUtils.loadAppJSON(projectDirArg, cmd.config);
+      const appJSON = await ProjectUtils.loadAppJSON(projectDirArg);
       const sdkVersion = _.get(appJSON, 'expo.sdkVersion');
-      await setup(platform, logger, sdkVersion);
+      await setup(platform, sdkVersion);
       const credentials = await prepareCredentials(cmd);
       const rawJob = {
         ...buildJobObject(appJSON, args, credentials),
@@ -72,7 +68,8 @@ export function createBuilderAction({
       const job = await sanitizeJob(rawJob);
       await builder(job);
     } catch (err) {
-      logger.error({ err }, 'Failed to build standalone app');
+      logger.error(`Failed to build standalone app: ${err.message}`);
+      logger.error(err.stack);
       if (err instanceof ErrorWithCommandHelp) {
         command.help();
       } else if (err instanceof ErrorWithProgramHelp) {

src/bin/utils/project.ts

@@ -1,17 +1,12 @@
-import fs from 'fs-extra';
 import path from 'path';
+import fs from 'fs-extra';
 
-export async function loadAppJSON(projectDirArg: string, config: string | null) {
-  let appJSONPath;
-  if (config) {
-    appJSONPath = resolveAbsoluteDir(config);
-  } else {
-    const projectDir = resolveAbsoluteDir(projectDirArg);
-    appJSONPath = path.join(projectDir, 'app.json');
-  }
+export async function loadAppJSON(projectDirArg: string) {
+  const projectDir = resolveAbsoluteDir(projectDirArg);
+  const appJSONPath = path.join(projectDir, 'app.json');
   const appJSONExists = await fs.pathExists(appJSONPath);
   if (!appJSONExists) {
-    throw new Error(`Couldn't find app.json.`);
+    throw new Error(`Couldn't find app.json in ${projectDir} directory.`);
   } else {
     return require(appJSONPath);
   }

src/bin/utils/user.ts

@@ -1,6 +1,6 @@
 import { User as UserManager } from 'xdl';
 
-import { ErrorWithProgramHelp } from 'turtle/bin/commands/ErrorWithProgramHelp';
+import { ErrorWithProgramHelp } from 'turtle/bin/commands/errors';
 
 interface IUserData {
   username?: string;

src/builders/BuildError.ts

@@ -1,17 +0,0 @@
-class BuildError extends Error {
-  public reason: BuildErrorReason;
-
-  constructor(reason: BuildErrorReason, message: string) {
-    super(message);
-    this.name = 'BuildError';
-    this.reason = reason;
-  }
-}
-
-enum BuildErrorReason {
-  SESSION_EXPIRED = 'session-expired',
-  PROVISIONING_PROFILE_MISSING = 'provisioning-profile-missing',
-}
-
-export default BuildError;
-export { BuildErrorReason };

src/builders/android.ts

@@ -2,35 +2,30 @@ import path from 'path';
 
 import fs from 'fs-extra';
 import _ from 'lodash';
-import uuidv4 from 'uuid/v4';
 import { AndroidShellApp, ImageUtils } from 'xdl';
+import uuidv4 from 'uuid/v4';
 
 import getOrCreateCredentials from 'turtle/builders/utils/android/credentials';
-import { formatShellAppDirectory } from 'turtle/builders/utils/android/workingdir';
 import * as commonUtils from 'turtle/builders/utils/common';
 import * as imageHelpers from 'turtle/builders/utils/image';
 import { uploadBuildToS3 } from 'turtle/builders/utils/uploader';
-import { ensureCanBuildSdkVersion } from 'turtle/builders/utils/version';
 import config from 'turtle/config';
+import logger from 'turtle/logger';
 import { IAndroidCredentials, IJob, IJobResult } from 'turtle/job';
+import { formatShellAppDirectory } from 'turtle/builders/utils/android/workingdir';
 
-export default async function buildAndroid(jobData: IJob, logger: any): Promise<IJobResult> {
-  await ensureCanBuildSdkVersion(jobData);
-  const credentials = await getOrCreateCredentials(jobData, logger);
-  const apkFilePath = await runShellAppBuilder(jobData, credentials, logger);
+export default async function buildAndroid(jobData: IJob): Promise<IJobResult> {
+  const credentials = await getOrCreateCredentials(jobData);
+  const apkFilePath = await runShellAppBuilder(jobData, credentials);
   const randomHex = uuidv4().replace(/-/g, '');
   const s3Filename = `${jobData.experienceName}-${randomHex}-signed.apk`;
   const s3FileKey = `android/${s3Filename}`;
   const fakeUploadFilename = s3Filename.replace('/', '\\');
   const artifactUrl = await uploadBuildToS3({
-    logger,
     uploadPath: apkFilePath,
     s3FileKey,
     ...config.builder.fakeUpload && {
-      fakeUploadBuildPath:
-        jobData.fakeUploadBuildPath
-        ? jobData.fakeUploadBuildPath
-        : path.join(jobData.fakeUploadDir || config.directories.fakeUploadDir, fakeUploadFilename),
+      fakeUploadBuildPath: jobData.fakeUploadBuildPath ? jobData.fakeUploadBuildPath : path.join(jobData.fakeUploadDir || config.directories.fakeUploadDir, fakeUploadFilename)
     },
   });
 
@@ -40,7 +35,6 @@ export default async function buildAndroid(jobData: IJob, logger: any): Promise<
 async function runShellAppBuilder(
   jobData: IJob,
   credentials: IAndroidCredentials,
-  logger: any,
 ): Promise<string> {
   const { temporaryFilesRoot } = config.directories;
   await fs.ensureDir(temporaryFilesRoot);
@@ -63,26 +57,25 @@ async function runShellAppBuilder(
 
   const outputFilePath = path.join(temporaryFilesRoot, `shell-signed-${jobData.id}.apk`);
 
-  const { config: jobConfig, manifest, sdkVersion: sdkVersionFromJob } = jobData;
-  const sdkVersion = _.get(manifest, 'sdkVersion', sdkVersionFromJob);
-  const workingDir = formatShellAppDirectory({ sdkVersion });
+  const { manifest: { sdkVersion: sdkVersionFromManifest = null } = {}, sdkVersion: sdkVersionFromJob } = jobData;
+  const sdkVersion = sdkVersionFromJob || sdkVersionFromManifest;
+  const workingDir = formatShellAppDirectory(sdkVersion);
 
   try {
     await AndroidShellApp.createAndroidShellAppAsync({
       url: commonUtils.getExperienceUrl(jobData),
-      sdkVersion,
+      sdkVersion: _.get(jobData, 'manifest.sdkVersion') || jobData.sdkVersion,
       keystore: tempKeystorePath,
-      manifest,
       alias: credentials.keystoreAlias,
       keystorePassword: credentials.keystorePassword,
       keyPassword: credentials.keyPassword,
       privateConfigFile: tempShellAppConfigPath,
-      releaseChannel: jobConfig.releaseChannel,
+      releaseChannel: jobData.config.releaseChannel,
       workingDir,
       outputFile: outputFilePath,
     });
   } catch (err) {
-    commonUtils.logErrorOnce(err, logger);
+    commonUtils.logErrorOnce(err);
     throw err;
   } finally {
     if (!config.builder.skipCleanup) {

src/builders/index.ts

@@ -1,10 +1,7 @@
-import { IJob, IJobResult } from 'turtle/job';
-import buildAndroid from './android';
 import buildIOS from './ios';
+import buildAndroid from './android';
 
-const builders: { [index: string]: (job: IJob, logger: any) => Promise<IJobResult>} = {
+export default {
   ios: buildIOS,
   android: buildAndroid,
 };
-
-export default builders;

src/builders/ios/archive.ts

@@ -1,11 +1,12 @@
 import _ from 'lodash';
 import { IosIPABuilder as createIPABuilder } from 'xdl';
 
-import { IContext } from 'turtle/builders/ios/context';
+import runShellAppBuilder from 'turtle/builders/utils/ios/shellAppBuilder';
 import { logErrorOnce } from 'turtle/builders/utils/common';
 import * as fileUtils from 'turtle/builders/utils/file';
 import * as keychain from 'turtle/builders/utils/ios/keychain';
-import runShellAppBuilder from 'turtle/builders/utils/ios/shellAppBuilder';
+import logger from 'turtle/logger/index';
+import { IContext } from 'turtle/builders/ios/context';
 import { IJob } from 'turtle/job';
 
 export default async function buildArchive(ctx: IContext, job: IJob) {
@@ -18,7 +19,7 @@ export default async function buildArchive(ctx: IContext, job: IJob) {
     const manifest = await runShellAppBuilder(ctx, job);
     await buildAndSignIPA(ctx, job, keychainInfo.path, manifest);
   } catch (err) {
-    logErrorOnce(err, ctx.logger);
+    logErrorOnce(err);
     throw err;
   } finally {
     if (keychainInfo) {
@@ -28,20 +29,20 @@ export default async function buildArchive(ctx: IContext, job: IJob) {
 }
 
 async function buildAndSignIPA(ctx: IContext, job: IJob, keychainPath: string, manifest: any) {
-  ctx.logger = ctx.logger.child({ buildPhase: 'building and signing IPA' });
-  ctx.logger.info('building and signing IPA');
+  const l = logger.withFields({ buildPhase: 'building and signing IPA' });
+  l.info('building and signing IPA');
 
   const {
-    credentials: { provisioningProfile, certPassword, teamId },
-    config: { bundleIdentifier: bundleIdentifierFromConfig },
+    credentials: { provisioningProfile, certPassword, teamId, password },
+    config: { bundleIdentifier: _bundleIdentifierFromConfig },
   } = job;
 
-  const bundleIdentifierFromManifest = _.get(manifest, 'ios.bundleIdentifier');
-  const bundleIdentifier = bundleIdentifierFromConfig || bundleIdentifierFromManifest;
+  const _bundleIdentifierFromManifest = _.get(manifest, 'ios.bundleIdentifier');
+  const bundleIdentifier = _bundleIdentifierFromConfig || _bundleIdentifierFromManifest;
 
   const { provisioningProfilePath } = ctx;
   await fileUtils.writeBase64ToBinaryFile(provisioningProfilePath, provisioningProfile as string);
-  ctx.logger.info('saved provisioning profile to temporary path');
+  l.info('saved provisioning profile to temporary path');
 
   const ipaBuilder = createIPABuilder({
     keychainPath,
@@ -50,11 +51,12 @@ async function buildAndSignIPA(ctx: IContext, job: IJob, keychainPath: string, m
     certPath: ctx.tempCertPath,
     certPassword,
     teamID: teamId,
+    password,
     bundleIdentifier,
-    workspacePath: ctx.workspacePath,
+    workingDir: ctx.workingDir,
     manifest,
   });
   await ipaBuilder.build();
 
-  ctx.logger.info(`done building and signing IPA`);
+  l.info(`done building and signing IPA`);
 }

src/builders/ios/context.ts

@@ -3,10 +3,10 @@ import * as path from 'path';
 import { v4 as uuidv4 } from 'uuid';
 import { IosShellApp } from 'xdl';
 
-import { formatShellAppDirectory } from 'turtle/builders/utils/ios/workingdir';
 import config from 'turtle/config';
 import { IOS, PLATFORMS } from 'turtle/constants/index';
 import { IJob } from 'turtle/job';
+import { formatShellAppDirectory } from 'turtle/builders/utils/ios/workingdir';
 
 export interface IContext {
   appDir: string;
@@ -15,7 +15,6 @@ export interface IContext {
   baseArchiveDir: string;
   buildDir: string;
   fakeUploadBuildPath?: string;
-  logger: any;
   outputPath: string;
   provisioningProfilePath: string;
   provisioningProfileDir: string;
@@ -23,99 +22,51 @@ export interface IContext {
   tempCertPath: string;
   uploadPath: string;
   workingDir: string;
-  workspacePath: string;
-  applicationFilesSrc: string;
 }
 
-const { EXPOKIT_APP, EXPONENT_APP } = IosShellApp;
+const { DEFAULT_EXPOKIT_WORKSPACE_NAME } = IosShellApp;
 const { BUILD_TYPES } = IOS;
 
-export function createBuilderContext(job: IJob, logger: any): IContext {
+export function createBuilderContext(job: IJob): IContext {
   const { join } = path;
   const appUUID = uuidv4();
-  const {
-    manifest: {
-      sdkVersion: sdkVersionFromManifest = null,
-    } = {},
-    sdkVersion: sdkVersionFromJob,
-    config: { buildType },
-  } = job;
+  const { manifest: { sdkVersion: _sdkVersionFromManifest = null } = {}, sdkVersion: _sdkVersionFromJob, config: { buildType } } = job;
 
-  const sdkVersion = sdkVersionFromJob || sdkVersionFromManifest;
-  const workingDir = formatShellAppDirectory({ sdkVersion, buildType: buildType! });
+  const sdkVersion = _sdkVersionFromJob || _sdkVersionFromManifest;
+  const workingDir = formatShellAppDirectory(sdkVersion);
 
   const context: any = {
     appDir: join(config.directories.temporaryFilesRoot, appUUID),
     appUUID,
     workingDir,
-    logger,
   };
   context.buildDir = join(context.appDir, 'build');
   context.provisioningProfileDir = join(context.appDir, 'provisioning');
   context.provisioningProfilePath = join(
     context.provisioningProfileDir,
-    `${appUUID}.mobileprovision`,
+    `${appUUID}.mobileprovision`
   );
   context.tempCertPath = join(context.appDir, 'cert.p12');
   context.baseArchiveDir = join(context.appDir, 'archive');
 
-  if (buildType === BUILD_TYPES.CLIENT) {
-    context.applicationFilesSrc = join(
-      workingDir,
-      'expo-client-build',
-      '**',
-      '*',
-    );
-  } else {
-    context.applicationFilesSrc = join(
-      workingDir,
-      'shellAppBase-builds',
-      buildType as string,
-      '**',
-      '*',
-    );
-  }
-
-  if (buildType === BUILD_TYPES.ARCHIVE) {
-    context.outputPath = join(context.appDir, 'archive.xcarchive');
-    context.uploadPath = join(context.buildDir, 'archive.ipa');
-    context.archiveDir = join(
-      context.baseArchiveDir,
-      'Release',
-      `${EXPOKIT_APP}.xcarchive`,
-      'Products',
-      'Applications',
-      `${EXPOKIT_APP}.app`,
-    );
-    context.workspacePath = path.join(
-      workingDir,
-      'shellAppWorkspaces',
-      'ios',
-      'default',
-      `${EXPOKIT_APP}.xcworkspace`,
-    );
-  } else if (buildType === BUILD_TYPES.CLIENT) {
-    context.outputPath = join(context.appDir, 'archive.xcarchive');
-    context.uploadPath = join(context.buildDir, 'archive.ipa');
-    context.archiveDir = join(
-      context.baseArchiveDir,
-      `${EXPONENT_APP}.xcarchive`,
-      'Products',
-      'Applications',
-      `${EXPONENT_APP}.app`,
-    );
-    context.workspacePath = path.join(
-      workingDir,
-      'ios',
-      `${EXPONENT_APP}.xcworkspace`,
-    );
-  } else if (buildType === BUILD_TYPES.SIMULATOR) {
+  if (buildType === BUILD_TYPES.SIMULATOR) {
     context.outputPath = join(context.appDir, 'archive.tar.gz');
     context.uploadPath = join(context.appDir, 'archive.tar.gz');
     context.archiveDir = join(
       context.baseArchiveDir,
       'Release',
-      `${EXPOKIT_APP}.app`,
+      `${DEFAULT_EXPOKIT_WORKSPACE_NAME}.app`
+    );
+  } else if (buildType === BUILD_TYPES.ARCHIVE) {
+    context.outputPath = join(context.appDir, 'archive.xcarchive');
+    context.uploadPath = join(context.buildDir, 'archive.ipa');
+    context.archiveDir = join(
+      context.baseArchiveDir,
+      'Release',
+      `${DEFAULT_EXPOKIT_WORKSPACE_NAME}.xcarchive`,
+      'Products',
+      'Applications',
+      `${DEFAULT_EXPOKIT_WORKSPACE_NAME}.app`
     );
   }
 
@@ -123,9 +74,7 @@ export function createBuilderContext(job: IJob, logger: any): IContext {
   const s3Filename = `${job.experienceName}-${appUUID}-${buildType}.${s3FileExtension}`;
   if (config.builder.fakeUpload) {
     const fakeUploadFilename = s3Filename.replace('/', '\\');
-    context.fakeUploadBuildPath = job.fakeUploadBuildPath
-      ? job.fakeUploadBuildPath
-      : join(job.fakeUploadDir || config.directories.fakeUploadDir, fakeUploadFilename);
+    context.fakeUploadBuildPath = job.fakeUploadBuildPath ? job.fakeUploadBuildPath : join(job.fakeUploadDir || config.directories.fakeUploadDir, fakeUploadFilename);
   } else {
     context.s3FileKey = join(PLATFORMS.IOS, s3Filename);
   }

src/builders/ios/index.ts

@@ -1,39 +1,35 @@
+import path from 'path';
+
 import fs from 'fs-extra';
 import pick from 'lodash/pick';
+import { ExponentTools } from 'xdl';
 
 import buildArchive from 'turtle/builders/ios/archive';
 import { createBuilderContext, IContext } from 'turtle/builders/ios/context';
 import buildSimulator from 'turtle/builders/ios/simulator';
 import { logErrorOnce } from 'turtle/builders/utils/common';
-import prepareAdHocBuildCredentials from 'turtle/builders/utils/ios/adhocBuild';
 import { uploadBuildToS3 } from 'turtle/builders/utils/uploader';
-import { ensureCanBuildSdkVersion } from 'turtle/builders/utils/version';
 import config from 'turtle/config';
-import { IOS } from 'turtle/constants/index';
+import { IOS, PLATFORMS } from 'turtle/constants/index';
 import { IJob, IJobResult } from 'turtle/job';
 
 const { BUILD_TYPES } = IOS;
 
-export default async function iosBuilder(job: IJob, logger: any): Promise<IJobResult> {
-  if (job.config.buildType !== BUILD_TYPES.CLIENT) {
-    await ensureCanBuildSdkVersion(job);
-  }
+let SUPPORTED_SDK_VERSIONS: Array<number>;
 
-  const ctx = createBuilderContext(job, logger);
+export default async function iosBuilder(job: IJob): Promise<IJobResult> {
+  await ensureCanBuildSdkVersion(job);
+
+  const ctx = createBuilderContext(job);
 
   try {
     await initBuilder(ctx);
 
     const { buildType } = job.config;
-
-    if (buildType === BUILD_TYPES.CLIENT) {
-      await prepareAdHocBuildCredentials(job, ctx.logger);
-    }
-
-    if ([BUILD_TYPES.ARCHIVE, BUILD_TYPES.CLIENT].includes(buildType!)) {
-      await buildArchive(ctx, job);
-    } else if (buildType === BUILD_TYPES.SIMULATOR) {
+    if (buildType === BUILD_TYPES.SIMULATOR) {
       await buildSimulator(ctx, job);
+    } else if (buildType === BUILD_TYPES.ARCHIVE) {
+      await buildArchive(ctx, job);
     } else {
       throw new Error(`Unsupported iOS build type: ${buildType}`);
     }
@@ -41,7 +37,7 @@ export default async function iosBuilder(job: IJob, logger: any): Promise<IJobRe
     const artifactUrl = await uploadBuildToS3(ctx);
     return { artifactUrl };
   } catch (err) {
-    logErrorOnce(err, ctx.logger);
+    logErrorOnce(err);
     throw err;
   } finally {
     await cleanup(ctx);
@@ -64,3 +60,25 @@ async function cleanup(ctx: IContext) {
 
 const getTemporaryDirs = (ctx: IContext) =>
   Object.values(pick(ctx, ['appDir', 'provisioningProfileDir']));
+
+async function ensureCanBuildSdkVersion(job: IJob) {
+  if (config.builder.useLocalWorkingDir) {
+    return;
+  }
+
+  if (!SUPPORTED_SDK_VERSIONS) {
+    SUPPORTED_SDK_VERSIONS = await findSupportedSdkVersions();
+  }
+
+  const targetMajorSdkVersion = ExponentTools.parseSdkMajorVersion(job.sdkVersion || job.manifest.sdkVersion);
+  if (!SUPPORTED_SDK_VERSIONS.includes(targetMajorSdkVersion)) {
+    throw new Error(`Unsupported SDK Version!`);
+  }
+}
+
+async function findSupportedSdkVersions(): Promise<Array<number>> {
+  const SDK_DIR_PREFIX = 'sdk';
+  const files = await fs.readdir(path.join(config.directories.workingDir, PLATFORMS.IOS));
+  const sdks = files.filter(file => file.startsWith(SDK_DIR_PREFIX));
+  return sdks.map(sdk => parseInt(sdk.substr(SDK_DIR_PREFIX.length)));
+}

src/builders/ios/simulator.ts

@@ -1,10 +1,11 @@
-import { IContext } from 'turtle/builders/ios/context';
 import runShellAppBuilder from 'turtle/builders/utils/ios/shellAppBuilder';
+import logger from 'turtle/logger';
+import { IContext } from 'turtle/builders/ios/context';
 import { IJob } from 'turtle/job';
 
 export default async function buildSimulator(ctx: IContext, job: IJob) {
-  ctx.logger = ctx.logger.child({ buildPhase: 'running simulator builder' });
-  ctx.logger.info('running simulator build');
+  const l = logger.withFields({ buildPhase: 'running simulator builder' });
+  l.info('running simulator build');
   await runShellAppBuilder(ctx, job);
-  ctx.logger.info('build complete');
+  l.info('build complete');
 }

src/builders/utils/android/credentials.ts

@@ -1,17 +1,18 @@
 import path from 'path';
 
-import fs from 'fs-extra';
 import uuidv4 from 'uuid/v4';
-import { Credentials } from 'xdl';
+import fs from 'fs-extra';
+import { AndroidKeystore } from 'xdl';
 
 import * as sqs from 'turtle/aws/sqs';
 import { UPDATE_CREDENTIALS } from 'turtle/constants/build';
-import { IAndroidCredentials, IJob } from 'turtle/job';
+import logger from 'turtle/logger';
 import { isOffline } from 'turtle/turtleContext';
+import { IAndroidCredentials, IJob } from 'turtle/job';
 
-async function getOrCreateCredentials(jobData: IJob, logger: any): Promise<IAndroidCredentials> {
+async function getOrCreateCredentials(jobData: IJob): Promise<IAndroidCredentials> {
   if (!jobData.credentials) {
-    const l = logger.child({ buildPhase: 'generating keystore' });
+    const l = logger.withFields({ buildPhase: 'generating keystore' });
     const credentials: any = {};
     l.info('Creating keystore');
     const keystoreFilename = `/tmp/keystore-${jobData.id}.tmp.jks`;
@@ -20,15 +21,13 @@ async function getOrCreateCredentials(jobData: IJob, logger: any): Promise<IAndr
     credentials.keystoreAlias = Buffer.from(jobData.experienceName).toString('base64');
     const androidPackage = jobData.manifest.android.package;
 
-    await Credentials.Android.createKeystore(
-      {
-        keystorePath: keystoreFilename,
-        keystorePassword: credentials.keystorePassword,
-        keyPassword: credentials.keyPassword,
-        keyAlias: credentials.keystoreAlias,
-      },
+    await AndroidKeystore.createKeystore({
+      keystorePassword: credentials.keystorePassword,
+      keyPassword: credentials.keyPassword,
+      keystoreFilename,
+      keystoreAlias: credentials.keystoreAlias,
       androidPackage,
-    );
+    });
 
     credentials.keystore = (await fs.readFile(keystoreFilename)).toString('base64');
     l.info('Keystore created successfully');
@@ -43,7 +42,8 @@ async function getOrCreateCredentials(jobData: IJob, logger: any): Promise<IAndr
     } else {
       await sqs.sendMessage(jobData.id, UPDATE_CREDENTIALS, {
         ...credentials,
-      }, l);
+        turtleVersion: jobData.config.turtleVersion,
+      });
       l.info('Keystore sent to storage successfully');
     }
     await fs.unlink(keystoreFilename);

src/builders/utils/android/workingdir.ts

@@ -1,12 +1,9 @@
 import path from 'path';
-
-import { ExponentTools } from 'xdl';
-
-import { IShellAppDirectoryConfig } from 'turtle/builders/utils/workingdir';
 import config from 'turtle/config';
+import { ExponentTools } from 'xdl';
 import { PLATFORMS } from 'turtle/constants';
 
-export function formatShellAppDirectory({ sdkVersion }: IShellAppDirectoryConfig) {
+export function formatShellAppDirectory(sdkVersion: string) {
   const majorSdkVersion = ExponentTools.parseSdkMajorVersion(sdkVersion);
   return config.builder.useLocalWorkingDir
     ? path.join(config.directories.workingDir, 'local')

src/builders/utils/common.ts

@@ -1,6 +1,7 @@
 import * as url from 'url';
 
 import config from 'turtle/config';
+import logger from 'turtle/logger';
 import { IJob } from 'turtle/job';
 
 export function getExperienceUrl(job: IJob) {
@@ -17,7 +18,7 @@ export function getExperienceUrl(job: IJob) {
 
 const alreadyLoggedError = Symbol('alreadyLoggedError');
 
-export function logErrorOnce(err: any, logger: any) {
+export function logErrorOnce(err: any) {
   if (!err[alreadyLoggedError]) {
     logger.error(err.stack);
     err[alreadyLoggedError] = true;

src/builders/utils/image.ts

@@ -18,15 +18,15 @@ export async function resizeIconWithSharpAsync(iconSizePx: number, iconFilename:
   await fs.writeFile(filename, buffer);
 }
 
-export async function getImageDimensionsWithSharpAsync(dirname: string, filename: string) {
-  const filepath = path.join(dirname, filename);
+export async function getImageDimensionsWithSharpAsync(basename: string, dirname: string) {
+  const filename = path.join(dirname, basename);
 
   // PLEASE DON'T REMOVE THE FOLLOWING LINE (sharp caches files taking path as the cache key)
   sharp.cache(false);
 
   try {
-    const { width, height } = await sharp(filepath).metadata();
-    return { width, height };
+    const meta = await sharp(filename).metadata();
+    return [meta.width, meta.height];
   } catch (e) {
     return null;
   }

src/builders/utils/ios/adhocBuild.ts

@@ -1,98 +0,0 @@
-import path from 'path';
-
-import spawnAsync from '@expo/spawn-async';
-import fs from 'fs-extra';
-import isEmpty from 'lodash/isEmpty';
-import { IosCodeSigning } from 'xdl';
-
-import * as sqs from 'turtle/aws/sqs';
-import BuildError, { BuildErrorReason } from 'turtle/builders/BuildError';
-import { UPDATE_CREDENTIALS } from 'turtle/constants/build';
-import { IJob } from 'turtle/job';
-import { isOffline } from 'turtle/turtleContext';
-
-// tslint:disable-next-line:no-var-requires
-const travelingFastlane = process.platform === 'darwin' ? require('@expo/traveling-fastlane-darwin')() : null;
-
-async function prepareAdHocBuildCredentials(job: IJob, logger: any) {
-  if (process.platform !== 'darwin') {
-    throw new Error('This function should be called only on macOS!');
-  }
-
-  const { bundleIdentifier } = job.config;
-  const { certP12, certPassword, teamId, appleSession, udids } = job.credentials;
-
-  const certSerialNumber = IosCodeSigning.findP12CertSerialNumber(certP12, certPassword);
-  const args = [
-    teamId,
-    udids!.join(','),
-    bundleIdentifier,
-    certSerialNumber || '__last__',
-  ];
-
-  try {
-    const credentials = await runFastlaneAction(
-      travelingFastlane.manageAdHocProvisioningProfile,
-      args,
-      { env: { FASTLANE_SESSION: appleSession } },
-    );
-    if (isEmpty(credentials)) {
-      if (!job.credentials.provisioningProfile) {
-        // a user has generated the provisioning profile for his app, but he probably deleted it from Expo servers, ...
-        throw new BuildError(
-          BuildErrorReason.PROVISIONING_PROFILE_MISSING,
-          `Provisioning profile is missing on Expo servers.`,
-        );
-      }
-      return;
-    }
-
-    logger.info('New ad hoc provisioning profile successfully created');
-    job.credentials.provisioningProfile = credentials.provisioningProfile;
-
-    if (isOffline()) {
-      const provisioningProfilePath = path.join(job.projectDir, `adhoc.mobileprovision`);
-      await fs.writeFile(provisioningProfilePath, Buffer.from(credentials.provisioningProfile, 'base64'));
-      logger.info(`Saved provisioning profile to ${provisioningProfilePath}`);
-    } else {
-      await sqs.sendMessage(job.id, UPDATE_CREDENTIALS, {
-        provisioningProfileId: credentials.provisioningProfileId,
-        provisioningProfile: credentials.provisioningProfile,
-      }, logger);
-      logger.info('Ad Hoc provisioning profile sent to storage successfully');
-    }
-  } catch (e) {
-    if (e instanceof BuildError) {
-      logger.error('Apple Session expired! Please authenticate again using expo-cli.');
-    }
-    throw e;
-  }
-}
-
-async function runFastlaneAction(action: string, args: string[], { env }: any) {
-  const { stderr } = await spawnAsync(
-    action,
-    args,
-    {
-      stdio: ['inherit', 'pipe', 'pipe'],
-      env: { ...process.env, ...env },
-    },
-  );
-  const { result, ...rest } = JSON.parse(stderr.trim());
-  if (result === 'failure') {
-    const { reason, rawDump, type } = rest;
-    let errorMsg = `Reason: ${reason}`;
-    if (rawDump) {
-      errorMsg += `, raw: ${JSON.stringify(rawDump)}`;
-    }
-    if (type === 'session-expired') {
-      throw new BuildError(BuildErrorReason.SESSION_EXPIRED, errorMsg);
-    } else {
-      throw new Error(errorMsg);
-    }
-  } else {
-    return rest;
-  }
-}
-
-export default prepareAdHocBuildCredentials;

src/builders/utils/ios/keychain.ts

@@ -1,27 +1,29 @@
 import { IosKeychain } from 'xdl';
 
-import { IContext } from 'turtle/builders/ios/context';
 import * as fileUtils from 'turtle/builders/utils/file';
+import logger from 'turtle/logger';
+import { IContext } from 'turtle/builders/ios/context';
 
 interface IKeychain {
   path: string;
 }
 
+
 export async function create(ctx: IContext): Promise<IKeychain> {
-  const l = ctx.logger.child({ buildPhase: 'creating keychain' });
+  const l = logger.withFields({ buildPhase: 'creating keychain' });
   try {
     l.info('creating keychain...');
     const keychainInfo = await IosKeychain.createKeychain(ctx.appUUID, false);
     l.info('done creating keychain');
     return keychainInfo;
   } catch (err) {
-    l.error({ err }, 'unable to create keychain');
+    l.error('unable to create keychain');
     throw err;
   }
 }
 
 export async function remove(ctx: IContext, keychainPath: string) {
-  const l = ctx.logger.child({ buildPhase: 'deleting keychain' });
+  const l = logger.withFields({ buildPhase: 'deleting keychain' });
   try {
     l.info('delete keychain...');
     const keychainInfo = await IosKeychain.deleteKeychain({
@@ -31,7 +33,7 @@ export async function remove(ctx: IContext, keychainPath: string) {
     l.info('done deleting keychain');
     return keychainInfo;
   } catch (err) {
-    l.error({ err }, 'unable to delete keychain');
+    l.error('unable to delete keychain');
     throw err;
   }
 }
@@ -48,7 +50,7 @@ export async function importCert(
     certPassword,
   }: { keychainPath: string; certP12: string; certPassword: string },
 ) {
-  const l = ctx.logger.child({ buildPhase: 'importing certificate into keychain' });
+  const l = logger.withFields({ buildPhase: 'importing certificate into keychain' });
   try {
     l.info('importing distribution certificate into keychain...');
     const { tempCertPath: certPath } = ctx;
@@ -56,7 +58,7 @@ export async function importCert(
     await IosKeychain.importIntoKeychain({ keychainPath, certPath, certPassword });
     l.info('done importing distribution certificate into keychain');
   } catch (err) {
-    l.error({ err }, 'unable to import distribution certificate into keychain');
+    l.error('unable to import distribution certificate into keychain');
     throw err;
   }
 }

src/builders/utils/ios/shellAppBuilder.ts

@@ -2,52 +2,51 @@ import path from 'path';
 import util from 'util';
 
 import copy from 'copy';
-import _ from 'lodash';
 import { ImageUtils, IosShellApp } from 'xdl';
+import _ from 'lodash';
 
-import { IContext } from 'turtle/builders/ios/context';
 import * as commonUtils from 'turtle/builders/utils/common';
 import * as imageHelpers from 'turtle/builders/utils/image';
-import { IOS } from 'turtle/constants/index';
+import logger from 'turtle/logger/index';
+import { IContext } from 'turtle/builders/ios/context';
 import { IJob } from 'turtle/job';
+import { PLATFORMS } from 'turtle/constants';
 
 const copyAsync = util.promisify(copy);
 
 export default async function runShellAppBuilder(ctx: IContext, job: IJob): Promise<any> {
-  const { config: jobConfig, manifest, sdkVersion: sdkVersionFromJob } = job;
+  const { config: jobConfig, manifest, sdkVersion: _sdkVersionFromJob } = job;
   const { buildType, releaseChannel } = jobConfig;
-  const sdkVersion = _.get(manifest, 'sdkVersion', sdkVersionFromJob);
+  const sdkVersion = _.get(manifest, 'sdkVersion', _sdkVersionFromJob);
+  const applicationFilesSrc = path.join(
+    ctx.workingDir,
+    'shellAppBase-builds',
+    buildType as string,
+    '**',
+    '*',
+  );
+  await copyAsync(applicationFilesSrc, ctx.baseArchiveDir);
 
-  await copyAsync(ctx.applicationFilesSrc, ctx.baseArchiveDir);
-
-  ctx.logger.info(
+  logger.info(
     { buildPhase: 'icons setup' },
     'ImageUtils: setting image functions to alternative sharp implementations',
   );
   ImageUtils.setResizeImageFunction(imageHelpers.resizeIconWithSharpAsync);
   ImageUtils.setGetImageDimensionsFunction(imageHelpers.getImageDimensionsWithSharpAsync);
   const shellAppParams = {
+    url: commonUtils.getExperienceUrl(job),
+    sdkVersion,
     action: 'configure',
     type: buildType,
+    releaseChannel,
     archivePath: ctx.archiveDir,
     privateConfigData: job.config,
-    expoSourcePath: path.join(ctx.workingDir, 'ios'),
-    manifest,
-    output: ctx.outputPath,
     verbose: true,
+    output: ctx.outputPath,
+    expoSourcePath: path.join(ctx.workingDir, PLATFORMS.IOS),
+    manifest,
   };
-  if (buildType === IOS.BUILD_TYPES.CLIENT) {
-    Object.assign(shellAppParams, {
-      workspacePath: path.join(ctx.workingDir, 'ios'),
-    });
-  } else {
-    Object.assign(shellAppParams, {
-      url: commonUtils.getExperienceUrl(job),
-      releaseChannel,
-      sdkVersion,
-    });
-  }
 
-  ctx.logger.info({ buildPhase: 'configuring NSBundle' }, 'configuring NSBundle...');
+  logger.info({ buildPhase: 'configuring NSBundle' }, 'configuring NSBundle...');
   return await IosShellApp.configureAndCopyArchiveAsync(shellAppParams);
 }

src/builders/utils/ios/workingdir.ts

@@ -1,19 +1,11 @@
 import path from 'path';
-
 import { ExponentTools } from 'xdl';
-
-import { IShellAppDirectoryConfig } from 'turtle/builders/utils/workingdir';
 import config from 'turtle/config';
-import { IOS, PLATFORMS } from 'turtle/constants';
+import { PLATFORMS } from 'turtle/constants';
 
-export function formatShellAppDirectory({ sdkVersion, buildType }: IShellAppDirectoryConfig) {
-  const { workingDir } = config.directories;
-  if (buildType === IOS.BUILD_TYPES.CLIENT) {
-    return path.join(workingDir, PLATFORMS.IOS, 'client');
-  } else if (config.builder.useLocalWorkingDir) {
-    return path.join(workingDir, 'local');
-  } else {
-    const majorVersion = ExponentTools.parseSdkMajorVersion(sdkVersion);
-    return path.join(workingDir, PLATFORMS.IOS, `sdk${majorVersion}`);
-  }
+export function formatShellAppDirectory(sdkVersion: string) {
+  const majorVersion = ExponentTools.parseSdkMajorVersion(sdkVersion);
+  return config.builder.useLocalWorkingDir
+    ? path.join(config.directories.workingDir, 'local')
+    : path.join(config.directories.workingDir, PLATFORMS.IOS, `sdk${majorVersion}`);
 }

src/builders/utils/uploader.ts

@@ -2,24 +2,24 @@ import * as fs from 'fs-extra';
 
 import { uploadFile } from 'turtle/aws/s3';
 import config from 'turtle/config';
+import logger from 'turtle/logger';
 
 interface IUploadCtx {
   fakeUploadBuildPath?: string;
   s3FileKey?: string;
   uploadPath: string;
-  logger: any;
 }
 
 export async function uploadBuildToS3(ctx: IUploadCtx) {
   if (config.builder.fakeUpload) {
-    const l = ctx.logger.child({ buildPhase: 'copying build artifact' });
+    const l = logger.withFields({ buildPhase: 'copying build artifact' });
     const { fakeUploadBuildPath, uploadPath } = ctx;
     l.info('copying build to fake upload directory');
     await fs.copy(uploadPath, fakeUploadBuildPath as string);
     l.info(`copied build to ${fakeUploadBuildPath}`);
     return fakeUploadBuildPath as string;
   } else {
-    const l = ctx.logger.child({ buildPhase: 'uploading to S3' });
+    const l = logger.withFields({ buildPhase: 'uploading to S3' });
     l.info('uploading build artifact to S3');
     const { Location: fileLocation } = await uploadFile({
       key: ctx.s3FileKey as string,

src/builders/utils/version.ts

@@ -1,30 +0,0 @@
-import fs from 'fs-extra';
-import path from 'path';
-import { ExponentTools } from 'xdl';
-
-import config from 'turtle/config';
-import { IJob } from 'turtle/job';
-
-let SUPPORTED_SDK_VERSIONS: number[];
-
-export async function ensureCanBuildSdkVersion(job: IJob) {
-  if (config.builder.useLocalWorkingDir) {
-    return;
-  }
-
-  if (!SUPPORTED_SDK_VERSIONS) {
-    SUPPORTED_SDK_VERSIONS = await findSupportedSdkVersions(job.platform);
-  }
-
-  const targetMajorSdkVersion = ExponentTools.parseSdkMajorVersion(job.sdkVersion || job.manifest.sdkVersion);
-  if (!SUPPORTED_SDK_VERSIONS.includes(targetMajorSdkVersion)) {
-    throw new Error(`Unsupported SDK Version!`);
-  }
-}
-
-async function findSupportedSdkVersions(platform: 'android' | 'ios'): Promise<number[]> {
-  const SDK_DIR_PREFIX = 'sdk';
-  const files = await fs.readdir(path.join(config.directories.workingDir, platform));
-  const sdks = files.filter((file) => file.startsWith(SDK_DIR_PREFIX));
-  return sdks.map((sdk) => parseInt(sdk.substr(SDK_DIR_PREFIX.length), 10));
-}

src/builders/utils/workingdir.ts

@@ -1,6 +0,0 @@
-import { IOS } from 'turtle/constants';
-
-export interface IShellAppDirectoryConfig {
-  sdkVersion: string;
-  buildType?: IOS.BUILD_TYPES;
-}

src/config.ts

@@ -1,14 +1,14 @@
 import os from 'os';
 import path from 'path';
 
-import { PLATFORMS } from 'turtle/constants';
 import { env, envNum, envOptional, envTransform } from 'turtle/utils/env';
+import { PLATFORMS } from 'turtle/constants';
 
 export default {
   env: env('NODE_ENV'),
   hostname: env('HOSTNAME', os.hostname()),
   deploymentEnv: env('ENVIRONMENT'),
-  platform: env('PLATFORM', PLATFORMS.IOS) as PLATFORMS,
+  platform: env('PLATFORM', PLATFORMS.IOS),
   aws: {
     accessKeyId: env('AWS_ACCESS_KEY_ID'),
     secretAccessKey: env('AWS_SECRET_ACCESS_KEY'),
@@ -47,9 +47,11 @@ export default {
     client: {
       level: env('CLIENT_LOGGER_LEVEL', 'info'),
     },
-  },
-  google: {
-    credentials: env('GOOGLE_APPLICATION_CREDENTIALS', ''),
+    loggly: {
+      token: env('LOGGLY_TOKEN', ''),
+      subdomain: env('LOGGLY_SUBDOMAIN', ''),
+      buffer: envNum('LOGGLY_BUFFER', 100),
+    },
   },
   sentry: {
     dsn: env('SENTRY_DSN', ''),
@@ -64,7 +66,7 @@ export default {
     skipCleanup: envTransform(
       'TURTLE_SKIP_CLEANUP',
       '0',
-      (val: string) => env('NODE_ENV') === 'development' && val === '1',
+      (val: string) => env('NODE_ENV') === 'development' && val === '1'
     ),
     fakeUpload: envTransform('TURTLE_FAKE_UPLOAD', '0', (val: string) => val === '1'),
     maxJobTimeMs: envNum('TURTLE_MAX_JOB_TIME_MS', 60 * 60 * 1000),
@@ -75,15 +77,9 @@ export default {
     workingDir: env('TURTLE_WORKING_DIR_PATH', path.join(__dirname, '../workingdir')),
     shellTarballsDir: env('TURTLE_SHELL_TARBALLS_DIR', path.join(__dirname, '../shellTarballs')),
     artifactsDir: env('TURTLE_ARTIFACTS_DIR', path.join(__dirname, '../artifacts')),
-    androidDependenciesDir: env(
-      'TURTLE_ANDROID_DEPENDENCIES_DIR',
-      path.join(os.homedir(), '.turtle/androidDependencies'),
-      ),
+    androidDependenciesDir: env('TURTLE_ANDROID_DEPENDENCIES_DIR', path.join(os.homedir(), '.turtle/androidDependencies')),
     tempS3LogsDir: env('TURTLE_TEMP_S3_LOGS_DIR', '/tmp/logs'),
     fakeUploadDir: envOptional('TURTLE_FAKE_UPLOAD_DIR'),
-    temporaryFilesRoot: env(
-      'TURTLE_TEMPORARY_FILES_DIR',
-      path.join(os.platform() === 'darwin' ? '/private' : '/', 'tmp', 'turtle'),
-    ),
+    temporaryFilesRoot: env('TURTLE_TEMPORARY_FILES_DIR', path.join(os.platform() === 'darwin' ? '/private' : '/', 'tmp', 'turtle')),
   },
 };

src/constants/build.ts

@@ -5,7 +5,7 @@ export enum JOB_STATES {
   FINISHED = 'finished',
   ERRORED = 'errored',
   SENT_TO_QUEUE = 'sent-to-queue',
-}
+};
 
 export const UPDATE_CREDENTIALS = 'update-credentials';
 

src/constants/index.ts

@@ -1,9 +1,15 @@
 import * as BUILD from './build';
 import * as IOS from './ios';
 
-export enum PLATFORMS {
-  IOS = 'ios',
-  ANDROID = 'android',
-}
+export const PLATFORMS = {
+  IOS: 'ios',
+  ANDROID: 'android',
+};
 
 export { IOS, BUILD };
+
+export const TURTLE_VERSIONS = {
+  NEW: 'new',
+  OLD: 'old',
+  BOTH: 'both',
+};

src/constants/ios.ts

@@ -1,5 +1,4 @@
-export enum BUILD_TYPES {
-  ARCHIVE = 'archive',
-  CLIENT = 'client',
-  SIMULATOR = 'simulator',
-}
+export const BUILD_TYPES = {
+  ARCHIVE: 'archive',
+  SIMULATOR: 'simulator',
+};

src/job.ts

@@ -1,5 +1,3 @@
-import { IOS } from 'turtle/constants';
-
 export interface IJob {
   platform: 'android' | 'ios';
   experienceName: string;
@@ -8,9 +6,10 @@ export interface IJob {
   sdkVersion: string;
   config: {
     releaseChannel: string;
+    turtleVersion?: string;
     publicUrl?: string;
     // ios
-    buildType?: IOS.BUILD_TYPES;
+    buildType?: string;
     bundleIdentifier?: string;
   };
   credentials: {
@@ -18,14 +17,13 @@ export interface IJob {
     keystore?: string;
     keystoreAlias?: string;
     keystorePassword?: string;
-    keyPassword?: string;
     // ios
+    keyPassword?: string;
     certP12?: string;
     certPassword?: string;
     provisioningProfile?: string;
+    password?: string;
     teamId?: string;
-    appleSession?: string;
-    udids?: string[];
   };
   projectDir: string;
   fakeUploadDir?: string;

src/jobManager.ts

@@ -1,90 +1,90 @@
 import _ from 'lodash';
+import { logErrorOnce } from 'turtle/builders/utils/common';
+import logger from 'turtle/logger';
 
 import * as sqs from 'turtle/aws/sqs';
-import builders from 'turtle/builders';
-import BuildError from 'turtle/builders/BuildError';
-import { logErrorOnce } from 'turtle/builders/utils/common';
-import config from 'turtle/config';
 import { BUILD } from 'turtle/constants/index';
-import * as buildDurationMetric from 'turtle/metrics/buildDuration';
-import * as buildStatusMetric from 'turtle/metrics/buildStatus';
-import { checkShouldExit, setCurrentJobId, turtleVersion } from 'turtle/turtleContext';
-import { getPriorities } from 'turtle/utils/priorities';
-import * as redis from 'turtle/utils/redis';
 import { sanitizeJob } from 'turtle/validator';
+import builders from 'turtle/builders';
+import config from 'turtle/config';
+import * as redis from 'turtle/utils/redis';
+import { checkShouldExit, setCurrentJobId } from 'turtle/turtleContext';
+import * as buildStatusMetric from 'turtle/metrics/buildStatus';
+import * as buildDurationMetric from 'turtle/metrics/buildDuration';
+import { getPriorities } from 'turtle/utils/priorities';
 
-function _maybeExit(logger: any) {
+function _maybeExit() {
   if (checkShouldExit()) {
     logger.warn('Exiting due to previously received termination signal.');
     process.exit();
   }
 }
 
-export async function doJob(logger: any) {
-  const jobData = await getJob(logger);
-  await processJob(jobData, logger);
-  _maybeExit(logger);
+export async function doJob() {
+  const jobData = await getJob();
+  await processJob(jobData);
+  _maybeExit();
 }
 
-export async function getJob(logger: any) {
-  _maybeExit(logger);
+export async function getJob() {
+  _maybeExit();
   logger.info('Fetching job');
+  // eslint-disable-next-line no-constant-condition
   while (true) {
     try {
       let job = null;
-      const priorities = await getPriorities(logger);
+      const priorities = await getPriorities();
       for (const priority of priorities) {
-        job = await sqs.receiveMessage(priority, logger);
-        _maybeExit(logger);
+        job = await sqs.receiveMessage(priority);
+        _maybeExit();
         if (job != null) {
           job.priority = priority;
           break;
         }
       }
-      _maybeExit(logger);
+      _maybeExit();
       if (job !== null) {
         return job;
       }
     } catch (err) {
-      logger.error({ err }, 'Error at receiving messages');
+      logger.error('Error at receiving messages', err);
     }
   }
 }
 
-async function processJob(jobData: any, logger: any) {
+async function processJob(jobData: any) {
   const receiptHandle = jobData.ReceiptHandle;
   const { priority } = jobData;
   let timeoutId;
   try {
     const rawJob = JSON.parse(jobData.Body);
-    timeoutId = failAfterMaxJobTime(priority, receiptHandle, rawJob, logger);
+    timeoutId = failAfterMaxJobTime(priority, receiptHandle, rawJob);
     const job = await sanitizeJob(rawJob);
 
     setCurrentJobId(job.id);
-    const pingerHandle = sqs.changeMessageVisibilityRecurring(priority, jobData.ReceiptHandle, job.id, logger);
+    const pingerHandle = sqs.changeMessageVisibilityRecurring(priority, jobData.ReceiptHandle, job.id);
 
     logger.info(`Doing job MessageId=${jobData.MessageId} BuildId=${job.id} ${Date.now()}`);
 
-    const cancelled = await redis.checkIfCancelled(job.id, logger);
+    const cancelled = await redis.checkIfCancelled(job.id);
     if (cancelled) {
       logger.info('The job has been cancelled');
     } else {
-      redis.registerListener(job.id, () => deleteMessage(priority, receiptHandle, logger), logger);
-      const startTimestamp = Date.now();
+      redis.registerListener(job.id, () => deleteMessage(priority, receiptHandle));
+      const startTimestamp = +new Date();
       let buildFailed = false;
       const buildType = _.get(job, 'config.buildType', 'default');
       try {
-        const status = await build(job, logger);
-        buildStatusMetric.add(buildType, status);
-        buildFailed = !status;
+        const success = await build(job);
+        buildStatusMetric.add(buildType, success);
+        buildFailed = !success;
       } catch (err) {
         buildStatusMetric.add(buildType, false);
         buildFailed = true;
         throw err;
       } finally {
-        const endTimestamp = Date.now();
+        const endTimestamp = +new Date();
         const turtleBuildDurationSecs = Math.ceil((endTimestamp - startTimestamp) / 1000);
-        logger.info(`BuildID=${job.id} Build duration=${turtleBuildDurationSecs}`);
         buildDurationMetric.addTurtleDuration(buildType, turtleBuildDurationSecs, !buildFailed);
         if (job.messageCreatedTimestamp) {
           const totalBuildDurationSecs = Math.ceil(
@@ -92,13 +92,13 @@ async function processJob(jobData: any, logger: any) {
           );
           buildDurationMetric.addTotalDuration(buildType, totalBuildDurationSecs, !buildFailed);
         }
-        redis.unregisterListeners(logger);
+        redis.unregisterListeners();
       }
       logger.info(`Job done MessageId=${jobData.MessageId} BuildId=${job.id} ${Date.now()}`);
     }
     clearInterval(pingerHandle);
   } finally {
-    await deleteMessage(priority, receiptHandle, logger);
+    await deleteMessage(priority, receiptHandle);
     if (timeoutId) {
       logger.info('Clearing job failer timeout...');
       clearTimeout(timeoutId);
@@ -106,11 +106,13 @@ async function processJob(jobData: any, logger: any) {
   }
 }
 
-function failAfterMaxJobTime(priority: string, receiptHandle: string, job: any, logger: any) {
+function failAfterMaxJobTime(priority: string, receiptHandle: string, job: any) {
   return setTimeout(async () => {
     try {
-      sqs.sendMessage(job.id, BUILD.JOB_STATES.ERRORED, { turtleVersion }, logger);
-      await deleteMessage(priority, receiptHandle, logger);
+      sqs.sendMessage(job.id, BUILD.JOB_STATES.ERRORED, {
+        turtleVersion: job.config.turtleVersion,
+      });
+      await deleteMessage(priority, receiptHandle);
     } finally {
       logger.error('Going to terminate turtle agent, just in case...');
       process.exit(1);
@@ -118,48 +120,33 @@ function failAfterMaxJobTime(priority: string, receiptHandle: string, job: any,
   }, config.builder.maxJobTimeMs);
 }
 
-async function build(job: any, logger: any) {
-  const startTimestamp = Date.now();
+async function build(job: any) {
   const s3Url = await logger.init(job);
-  const buildLogger = logger.child({ jobID: job.id, experienceName: job.experienceName });
-  const calculateBuildDuration = () => Math.ceil((Date.now() - startTimestamp) / 1000);
+  const { turtleVersion } = job.config;
 
   try {
     await sqs.sendMessage(job.id, BUILD.JOB_STATES.IN_PROGRESS, {
       logUrl: s3Url,
       logFormat: 'json',
-    }, buildLogger);
-    const result = await builders[job.platform](job, buildLogger);
-    const buildDuration = calculateBuildDuration();
-    sqs.sendMessage(job.id, BUILD.JOB_STATES.FINISHED, {
-      ...result,
       turtleVersion,
-      buildDuration,
-    }, buildLogger);
+    });
+    const result = await (builders as any)[job.platform](job);
+    sqs.sendMessage(job.id, BUILD.JOB_STATES.FINISHED, { ...result, turtleVersion });
     return true;
   } catch (err) {
-    logErrorOnce(err, buildLogger);
-    const buildDuration = calculateBuildDuration();
-    let reason;
-    if (err instanceof BuildError) {
-      reason = err.reason;
-    }
-    sqs.sendMessage(job.id, BUILD.JOB_STATES.ERRORED, {
-      turtleVersion,
-      buildDuration,
-      ...reason && { reason },
-    }, buildLogger);
+    logErrorOnce(err);
+    sqs.sendMessage(job.id, BUILD.JOB_STATES.ERRORED, { turtleVersion });
     return false;
   } finally {
     await logger.cleanup(job);
   }
 }
 
-async function deleteMessage(priority: string, receiptHandle: string, logger: any) {
+async function deleteMessage(priority: string, receiptHandle: string) {
   try {
     setCurrentJobId(null);
     await sqs.deleteMessage(priority, receiptHandle);
   } catch (err) {
-    logger.error({ err }, 'Error at deleting msg');
+    logger.error('Error at deleting msg', err);
   }
 }