factMapId instead of id - id (and similar names) seems like it would be common enough to be passed in context, a gross name like factMapId should be less bothersome

Author: akmjenkins

README.md

@@ -233,7 +233,7 @@ The `then` or `otherwise` property can consist of either `actions`, but it can a
 const myRule = {
   when: [
     {
-      id: 'weatherCondition',
+      factMapId: 'weatherCondition',
       weather: {
         params: {
           query: '{{city}}',
@@ -302,7 +302,7 @@ const myRule = {
 
 A fact map is a plain object whose keys are facts (static or functional) and values are [`Evaluator`'s](#evaluator).
 
-NOTE: `id` is a reserved word in a `FactMap`. It is used internally to allow easy access to the results of a `FactMap` for interpolation in the `then` or `otherwise` clauses.
+**NOTE: `factMapId` is a reserved word in a `FactMap`. It is used internally to allow easy access to the results of a `FactMap` for interpolation in the `then` or `otherwise` clauses. For this reason `factMapId` _CANNOT_ be given as a fact or context**.
 
 #### Evaluator
 

src/evaluator.js

@@ -1,8 +1,8 @@
 export const createEvaluator =
   (validator, opts, emit, rule) =>
-  (id) =>
+  (factMapId) =>
   async ([factName, { params, path, is }]) => {
-    const onError = (params) => emit('error', { ...params, rule, id });
+    const onError = (params) => emit('error', { ...params, rule, factMapId });
 
     const fact = opts.facts[factName] || opts.context[factName];
     try {

src/fact.map.processor.js

@@ -3,16 +3,16 @@ import { createEvaluator } from './evaluator';
 export const createFactMapProcessor = (validator, opts, emit) => (rule) => {
   const evaluator = createEvaluator(validator, opts, emit, rule);
   return async (factMap, index) => {
-    // id is a reserved word
-    const { id = index, ...map } = factMap;
+    // factMapId is a reserved word
+    const { factMapId = index, ...map } = factMap;
 
     // flags for if there was an error processing the fact map
     // and if all evaluations in the fact map passed
     let error = false;
     let passed = true;
 
     const results = (
-      await Promise.all(Object.entries(map).map(evaluator(id)))
+      await Promise.all(Object.entries(map).map(evaluator(factMapId)))
     ).reduce((acc, { factName, ...rest }) => {
       if (error) return acc;
       error = error || !!rest.error;
@@ -23,7 +23,7 @@ export const createFactMapProcessor = (validator, opts, emit) => (rule) => {
 
     // return the results in the same form they were passed in
     return {
-      [id]: {
+      [factMapId]: {
         ...results,
         __passed: passed,
         __error: error,