fable 3.1.68 → 3.1.69

package/docs/services/anticipate.md

@@ -109,7 +109,7 @@ tmpAnticipate.anticipate(function (fCallback)
 ```
 
 The step function receives one parameter:
-- `fCallback` — Callback to signal completion. Pass an error to bail out: `fCallback(new Error('something failed'))`
+- `fCallback` -- Callback to signal completion. Pass an error to bail out: `fCallback(new Error('something failed'))`
 
 ### wait(fCallback)
 
@@ -185,7 +185,7 @@ tmpAnticipate.anticipate(function (fCallback)
 	catch (pError)
 	{
 		tmpRecoveredError = pError;
-		fCallback();  // No error passed — pipeline continues
+		fCallback();  // No error passed -- pipeline continues
 	}
 });
 tmpAnticipate.anticipate(function (fCallback)

package/docs/services/csv-parser.md

@@ -149,4 +149,4 @@ If a row has more columns than the header, extra columns are keyed by their nume
 - All values are returned as strings
 - The parser is designed for streaming/line-by-line use, not whole-file-at-once parsing
 - Carriage returns (`\r`) are automatically stripped from fields
-- The parser is stateful — create a new instance for each file you parse
+- The parser is stateful -- create a new instance for each file you parse

package/docs/services/environment-data.md

@@ -23,8 +23,8 @@ fable.EnvironmentData.Environment
 
 Fable automatically uses the appropriate implementation:
 
-- **Node.js**: `Fable-Service-EnvironmentData.js` — sets `Environment` to `'node.js'`
-- **Browser**: `Fable-Service-EnvironmentData-Web.js` — sets `Environment` to `'web'`
+- **Node.js**: `Fable-Service-EnvironmentData.js` -- sets `Environment` to `'node.js'`
+- **Browser**: `Fable-Service-EnvironmentData-Web.js` -- sets `Environment` to `'web'`
 
 ## Use Cases
 

package/docs/services/expression-parser-functions/matrixinverse.md

@@ -42,7 +42,7 @@ Inverse = MATRIXINVERSE(Matrix3x3)
 ### Solving Equations
 
 ```expression
-// Solve Ax = b  →  x = A⁻¹b
+// Solve Ax = b  ->  x = A⁻¹b
 Solution = MATRIXVECTORMULTIPLY(MATRIXINVERSE(A), B)
 ```
 

package/docs/services/expression-parser-functions/tofixed.md

@@ -70,6 +70,6 @@ FormattedPercent = TOFIXED(PERCENT(Completed, Total) * 100, 1)
 ## Notes
 
 - Always returns specified number of decimal places
-- Pads with zeros if necessary (19.9 → "19.90")
+- Pads with zeros if necessary (19.9 -> "19.90")
 - Uses the Math service's `toFixedPrecise` method
 - Returns result as string

package/docs/services/expression-parser.md

@@ -28,11 +28,11 @@ parser.solve('2 ^ 10');       // Returns '1024'
 parser.solve(expression, dataObject, resultObject, manifest, destinationObject);
 ```
 
-- **expression** — the expression string to evaluate
-- **dataObject** — object containing variable values (optional)
-- **resultObject** — object to store solver metadata/logs (optional)
-- **manifest** — a Manyfest instance for address resolution, or `false` (optional)
-- **destinationObject** — object where named results are written (optional)
+- **expression** -- the expression string to evaluate
+- **dataObject** -- object containing variable values (optional)
+- **resultObject** -- object to store solver metadata/logs (optional)
+- **manifest** -- a Manyfest instance for address resolution, or `false` (optional)
+- **destinationObject** -- object where named results are written (optional)
 
 ### Assigning Results
 
@@ -59,12 +59,12 @@ parser.solve('Name ?= GETVALUE("AppData.Students[0]")', data, {}, false, dest);
 
 | Operator | Description | Example |
 |----------|-------------|---------|
-| `+` | Addition | `5 + 3` → `'8'` |
-| `-` | Subtraction | `5 - 3` → `'2'` |
-| `*` | Multiplication | `5 * 3` → `'15'` |
-| `/` | Division | `15 / 3` → `'5'` |
-| `^` | Power | `2 ^ 3` → `'8'` |
-| `%` | Modulus | `10 % 3` → `'1'` |
+| `+` | Addition | `5 + 3` -> `'8'` |
+| `-` | Subtraction | `5 - 3` -> `'2'` |
+| `*` | Multiplication | `5 * 3` -> `'15'` |
+| `/` | Division | `15 / 3` -> `'5'` |
+| `^` | Power | `2 ^ 3` -> `'8'` |
+| `%` | Modulus | `10 % 3` -> `'1'` |
 
 ### Comparison Operators
 
@@ -72,12 +72,12 @@ Comparison operators evaluate to `'1'` (true) or `'0'` (false). They bind looser
 
 | Operator | Description | Example |
 |----------|-------------|---------|
-| `>` | Greater than | `5 > 3` → `'1'` |
-| `>=` | Greater than or equal | `5 >= 5` → `'1'` |
-| `<` | Less than | `3 < 5` → `'1'` |
-| `<=` | Less than or equal | `3 <= 3` → `'1'` |
-| `==` | Equal | `5 == 5` → `'1'` |
-| `!=` | Not equal | `5 != 3` → `'1'` |
+| `>` | Greater than | `5 > 3` -> `'1'` |
+| `>=` | Greater than or equal | `5 >= 5` -> `'1'` |
+| `<` | Less than | `3 < 5` -> `'1'` |
+| `<=` | Less than or equal | `3 <= 3` -> `'1'` |
+| `==` | Equal | `5 == 5` -> `'1'` |
+| `!=` | Not equal | `5 != 3` -> `'1'` |
 
 ```javascript
 parser.solve('Result = 5 > 3', {}, {}, false, dest);
@@ -386,14 +386,14 @@ parser.solve('Sampled = MULTIROWMAP ROWS FROM Rows SERIESSTEP 2 VAR v FROM Value
 ```
 
 **Available variables in MULTIROWMAP expressions:**
-- `stepIndex` — iteration counter (0, 1, 2, ...)
-- `rowIndex` — actual array index of the current row
+- `stepIndex` -- iteration counter (0, 1, 2, ...)
+- `rowIndex` -- actual array index of the current row
 - Any VAR-mapped variable names
 
 **OFFSET values:**
-- `0` (default) — current row
-- `-1` — previous row, `-2` — two rows back, `-3` — three back, etc.
-- `1` — next row, `2` — two ahead, etc.
+- `0` (default) -- current row
+- `-1` -- previous row, `-2` -- two rows back, `-3` -- three back, etc.
+- `1` -- next row, `2` -- two ahead, etc.
 
 ### ITERATIVESERIES
 

package/docs/services/object-cache.md

@@ -138,9 +138,9 @@ Get the full linked list node (including metadata) for a hash:
 
 ```javascript
 const node = cache.getNode('my-key');
-// node.Datum — the stored data
-// node.Hash — the hash key
-// node.Metadata.Created — timestamp (ms) when the entry was created
+// node.Datum -- the stored data
+// node.Hash -- the hash key
+// node.Metadata.Created -- timestamp (ms) when the entry was created
 ```
 
 ### RecordMap

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fable",
-  "version": "3.1.68",
+  "version": "3.1.69",
   "description": "A service dependency injection, configuration and logging library.",
   "main": "source/Fable.js",
   "scripts": {
@@ -51,7 +51,8 @@
   },
   "homepage": "https://github.com/stevenvelozo/fable",
   "devDependencies": {
-    "quackage": "^1.0.68"
+    "pict-docuserve": "^0.1.5",
+    "quackage": "^1.1.0"
   },
   "dependencies": {
     "async.eachlimit": "^0.5.2",

package/source/services/Fable-Service-RestClient.js

@@ -2,6 +2,8 @@ const libFableServiceBase = require('fable-serviceproviderbase');
 
 const libSimpleGet = require('simple-get');
 const libCookie = require('cookie');
+const libHttp = require('http');
+const libHttps = require('https');
 
 class FableServiceRestClient extends libFableServiceBase
 {
@@ -24,6 +26,44 @@ class FableServiceRestClient extends libFableServiceBase
 		// This is a function that can be overridden, to allow the management
 		// of the request options before they are passed to the request library.
 		this.prepareRequestOptions = (pOptions) => { return pOptions; };
+
+		// Check for keep-alive configuration (from options or fable settings)
+		// Useful in environments where connections are slow to establish
+		// (e.g. inside poorly configured customer networks / VPNs)
+		let tmpKeepAlive = this.options.KeepAlive || this.fable.settings.RestClientKeepAlive;
+		if (tmpKeepAlive)
+		{
+			this.initializeKeepAliveAgent();
+		}
+	}
+
+	/**
+	 * Initialize HTTP keep-alive agents and wire them into prepareRequestOptions.
+	 * Creates both an HTTP and HTTPS agent so the correct one is selected per-request
+	 * based on the URL protocol.
+	 */
+	initializeKeepAliveAgent()
+	{
+		let tmpAgentOptions = { keepAlive: true };
+
+		this.httpAgent = new libHttp.Agent(tmpAgentOptions);
+		this.httpsAgent = new libHttps.Agent(tmpAgentOptions);
+
+		// Capture any previously set prepareRequestOptions so we can chain
+		let tmpPreviousPrepareRequestOptions = this.prepareRequestOptions;
+
+		this.prepareRequestOptions = (pOptions) =>
+		{
+			if (typeof pOptions.url === 'string' && pOptions.url.startsWith('http:'))
+			{
+				pOptions.agent = this.httpAgent;
+			}
+			else
+			{
+				pOptions.agent = this.httpsAgent;
+			}
+			return tmpPreviousPrepareRequestOptions(pOptions);
+		};
 	}
 
 	get simpleGet()

package/test/RestClient_test.js

@@ -146,6 +146,136 @@ suite
 					}
 				);
 
+				suite
+					(
+						'KeepAlive Agent',
+						function ()
+						{
+							test
+								(
+									'Initialize keep-alive agents via options.',
+									function ()
+									{
+										let testFable = new libFable();
+										let tmpRestClient = testFable.instantiateServiceProvider('RestClient', { KeepAlive: true }, 'RestClient-KeepAlive-Options');
+
+										Expect(tmpRestClient.httpAgent).to.be.an('object');
+										Expect(tmpRestClient.httpsAgent).to.be.an('object');
+										Expect(tmpRestClient.httpAgent.keepAlive).to.equal(true);
+										Expect(tmpRestClient.httpsAgent.keepAlive).to.equal(true);
+									}
+								);
+							test
+								(
+									'Initialize keep-alive agents via fable settings.',
+									function ()
+									{
+										let testFable = new libFable({ RestClientKeepAlive: true });
+										let tmpRestClient = testFable.instantiateServiceProvider('RestClient', {}, 'RestClient-KeepAlive-Settings');
+
+										Expect(tmpRestClient.httpAgent).to.be.an('object');
+										Expect(tmpRestClient.httpsAgent).to.be.an('object');
+										Expect(tmpRestClient.httpAgent.keepAlive).to.equal(true);
+										Expect(tmpRestClient.httpsAgent.keepAlive).to.equal(true);
+									}
+								);
+							test
+								(
+									'Do not create agents when KeepAlive is not set.',
+									function ()
+									{
+										let testFable = new libFable();
+										let tmpRestClient = testFable.instantiateServiceProvider('RestClient', {}, 'RestClient-NoKeepAlive');
+
+										Expect(tmpRestClient.httpAgent).to.equal(undefined);
+										Expect(tmpRestClient.httpsAgent).to.equal(undefined);
+									}
+								);
+							test
+								(
+									'Inject HTTP agent into request options for http URLs.',
+									function (fTestComplete)
+									{
+										var tmpServer = libHTTP.createServer(function (pReq, pRes)
+										{
+											pRes.writeHead(200, { 'Content-Type': 'application/json' });
+											pRes.end(JSON.stringify({ OK: true }));
+										});
+
+										tmpServer.listen(0, function ()
+										{
+											var tmpPort = tmpServer.address().port;
+											var testFable = new libFable();
+											var tmpRestClient = testFable.instantiateServiceProvider('RestClient', { KeepAlive: true }, 'RestClient-KeepAlive-HTTP');
+
+											// Wrap prepareRequestOptions to capture the final options
+											var tmpOriginalPrepare = tmpRestClient.prepareRequestOptions;
+											var tmpCapturedAgent = null;
+											tmpRestClient.prepareRequestOptions = function (pOptions)
+											{
+												let tmpResult = tmpOriginalPrepare(pOptions);
+												tmpCapturedAgent = tmpResult.agent;
+												return tmpResult;
+											};
+
+											tmpRestClient.getJSON('http://localhost:' + tmpPort + '/test',
+												function (pError, pResponse, pBody)
+												{
+													Expect(tmpCapturedAgent).to.equal(tmpRestClient.httpAgent);
+													Expect(pBody).to.be.an('object');
+													Expect(pBody.OK).to.equal(true);
+													tmpServer.close();
+													fTestComplete();
+												});
+										});
+									}
+								);
+							test
+								(
+									'Chain with previously set prepareRequestOptions.',
+									function (fTestComplete)
+									{
+										var tmpServer = libHTTP.createServer(function (pReq, pRes)
+										{
+											pRes.writeHead(200, { 'Content-Type': 'application/json' });
+											pRes.end(JSON.stringify({ CustomHeader: pReq.headers['x-custom'] || '' }));
+										});
+
+										tmpServer.listen(0, function ()
+										{
+											var tmpPort = tmpServer.address().port;
+											var testFable = new libFable();
+											var tmpRestClient = testFable.instantiateServiceProvider('RestClient', { KeepAlive: true }, 'RestClient-KeepAlive-Chain');
+
+											// Set a custom prepareRequestOptions AFTER keep-alive init to verify chaining
+											var tmpKeepAlivePrepare = tmpRestClient.prepareRequestOptions;
+											tmpRestClient.prepareRequestOptions = function (pOptions)
+											{
+												// Apply keep-alive first
+												let tmpResult = tmpKeepAlivePrepare(pOptions);
+												// Then add custom header
+												if (!tmpResult.headers)
+												{
+													tmpResult.headers = {};
+												}
+												tmpResult.headers['X-Custom'] = 'test-value';
+												return tmpResult;
+											};
+
+											tmpRestClient.getJSON('http://localhost:' + tmpPort + '/test',
+												function (pError, pResponse, pBody)
+												{
+													Expect(pBody).to.be.an('object');
+													Expect(pBody.CustomHeader).to.equal('test-value');
+													tmpServer.close();
+													fTestComplete();
+												});
+										});
+									}
+								);
+						}
+					);
+
 				suite
 					(
 						'Error Handling',