Autosize v8 heap

Also expose $NPM_CLI_OPTIONS for customizing npm

VERSION

@@ -1 +1 @@
-1.5.32
+1.5.32-1492485633

bin/release

@@ -4,5 +4,5 @@
 cat << EOF
 addons: []
 default_process_types:
-  web: npm start
+  web: npm start --max-old-space-size=$(( $(jq -r '.limits.mem' <<<"$VCAP_APPLICATION") * 75 / 100 )) $NPM_CLI_OPTIONS
 EOF

cf_spec/fixtures/without_procfile/.gitignore

@@ -0,0 +1 @@
+node_modules/

cf_spec/fixtures/without_procfile/.npmrc

@@ -0,0 +1 @@
+ca=

cf_spec/fixtures/without_procfile/README.md

@@ -0,0 +1 @@
+This file here to suppress "npm WARN package.json node_web_app@0.0.0 No README data"

cf_spec/fixtures/without_procfile/package.json

@@ -0,0 +1,22 @@
+{
+  "name": "node_web_app",
+  "version": "0.0.0",
+  "description": "hello, world",
+  "main": "server.js",
+  "scripts": {
+    "test": "echo \"Error: no test specified\" && exit 1"
+  },
+  "author": "",
+  "license": "BSD-2-Clause",
+  "dependencies": {
+    "logfmt": "~1.1.2",
+    "express": "~4.0.0"
+  },
+  "engines" : {
+  },
+  "repository" :
+  {
+    "type" : "git",
+    "url" : "http://github.com/cloudfoundry/nodejs-buildpack.git"
+  }
+}

cf_spec/fixtures/without_procfile/package.sh

@@ -0,0 +1,4 @@
+#!/bin/bash -l
+
+rm -rf node_modules
+npm install

cf_spec/fixtures/without_procfile/resolve

@@ -0,0 +1 @@
+0.10.26

cf_spec/fixtures/without_procfile/server.js

@@ -0,0 +1,18 @@
+// web.js
+
+console.log("***************************I'm running!********************************")
+
+var express = require("express");
+var logfmt = require("logfmt");
+var app = express();
+
+app.use(logfmt.requestLogger());
+
+app.get('/', function(req, res) {
+  res.send('Hello, World!');
+});
+
+var port = Number(process.env.PORT || 5000);
+app.listen(port, function() {
+  console.log("Listening on " + port);
+});

cf_spec/integration/deploy_a_node_app_spec.rb

@@ -74,6 +74,28 @@
     end
   end
 
+  context 'with no Procfile and no NPM_CLI_OPTIONS env var' do
+    let (:app_name) { 'without_procfile' }
+
+    it 'is running with autosized max_old_space_size' do
+      expect(app).to be_running
+      expect(app).to have_logged("--max_old_space_size=768")
+    end
+  end
+
+  context 'with no Procfile and with NPM_CLI_OPTIONS env var' do
+    let (:app_name) { 'without_procfile' }
+
+    subject(:app) do
+      Machete.deploy_app(app_name, env: {'NPM_CLI_OPTIONS' => '--trace-sync-io'})
+    end
+
+    it 'is running with autosized max_old_space_size and appended NPM_CLI_OPTIONS' do
+      expect(app).to be_running
+      expect(app).to have_logged("--max_old_space_size=768 --trace-sync-io")
+    end
+  end
+
   context 'with an app that has vendored dependencies' do
     let(:app_name) { 'vendored_dependencies' }