@eagleoutice/flowr 2.6.2 → 2.7.0

package/dataflow/internal/process/functions/call/built-in/built-in-source.js

@@ -3,9 +3,9 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getSourceProvider = getSourceProvider;
-exports.setSourceProvider = setSourceProvider;
 exports.inferWdFromScript = inferWdFromScript;
+exports.platformBasename = platformBasename;
+exports.platformDirname = platformDirname;
 exports.findSource = findSource;
 exports.processSourceCall = processSourceCall;
 exports.sourceRequest = sourceRequest;
@@ -21,7 +21,6 @@ const logger_1 = require("../../../../../logger");
 const type_1 = require("../../../../../../r-bridge/lang-4.x/ast/model/type");
 const overwrite_1 = require("../../../../../environments/overwrite");
 const log_1 = require("../../../../../../util/log");
-const fs_1 = __importDefault(require("fs"));
 const parser_1 = require("../../../../../../r-bridge/lang-4.x/ast/parser/json/parser");
 const shell_executor_1 = require("../../../../../../r-bridge/shell-executor");
 const assert_1 = require("../../../../../../util/assert");
@@ -30,40 +29,36 @@ const general_1 = require("../../../../../eval/values/general");
 const r_value_1 = require("../../../../../eval/values/r-value");
 const unknown_side_effect_1 = require("../../../../../graph/unknown-side-effect");
 const alias_tracking_1 = require("../../../../../eval/resolve/alias-tracking");
-let sourceProvider = (0, retriever_1.requestProviderFromFile)();
-/**
- * Returns the current (global) source provider
- */
-function getSourceProvider() {
-    return sourceProvider;
-}
-/**
- * Sets the current (global) source provider
- */
-function setSourceProvider(provider) {
-    sourceProvider = provider;
-}
 /**
  * Infers working directories based on the given option and reference chain
  */
 function inferWdFromScript(option, referenceChain) {
     switch (option) {
         case config_1.InferWorkingDirectory.MainScript:
-            return referenceChain[0]?.request === 'file' ? [platformDirname(referenceChain[0].content)] : [];
-        case config_1.InferWorkingDirectory.ActiveScript:
-            return referenceChain[referenceChain.length - 1] ? [platformDirname(referenceChain[referenceChain.length - 1].content)] : [];
+            return referenceChain[0] ? [platformDirname(referenceChain[0])] : [];
+        case config_1.InferWorkingDirectory.ActiveScript: {
+            const secondToLast = referenceChain[referenceChain.length - 1];
+            return secondToLast ? [platformDirname(secondToLast)] : [];
+        }
         case config_1.InferWorkingDirectory.AnyScript:
-            return referenceChain.filter(e => e.request === 'file').map(e => platformDirname(e.content));
+            return referenceChain.filter(assert_1.isNotUndefined).map(e => platformDirname(e));
         case config_1.InferWorkingDirectory.No:
         default:
             return [];
     }
 }
 const AnyPathSeparator = /[/\\]/g;
+/**
+ * Return the basename of a path in a platform-agnostic way
+ * @see {@link platformDirname} - for the dirname counterpart
+ */
 function platformBasename(p) {
     const normalized = p.replaceAll(path_1.default.win32.sep, path_1.default.posix.sep);
     return path_1.default.posix.basename(normalized);
 }
+/**
+ * Return the dirname of a path in a platform-agnostic way
+ */
 function platformDirname(p) {
     const normalized = p.replaceAll(path_1.default.win32.sep, path_1.default.posix.sep);
     return path_1.default.posix.dirname(normalized);
@@ -87,10 +82,7 @@ function applyReplacements(path, replacements) {
  */
 function findSource(resolveSource, seed, data) {
     const capitalization = resolveSource?.ignoreCapitalization ?? false;
-    const explorePaths = [
-        ...(resolveSource?.searchPath ?? []),
-        ...(inferWdFromScript(resolveSource?.inferWorkingDirectory ?? config_1.InferWorkingDirectory.No, data.referenceChain))
-    ];
+    const explorePaths = (resolveSource?.searchPath ?? []).concat(inferWdFromScript(resolveSource?.inferWorkingDirectory ?? config_1.InferWorkingDirectory.No, data.referenceChain));
     let tryPaths = [seed];
     switch (resolveSource?.dropPaths ?? config_1.DropPathsOption.No) {
         case config_1.DropPathsOption.Once: {
@@ -122,7 +114,8 @@ function findSource(resolveSource, seed, data) {
     for (const explore of [undefined, ...explorePaths]) {
         for (const tryPath of tryPaths) {
             const effectivePath = explore ? path_1.default.join(explore, tryPath) : tryPath;
-            const get = sourceProvider.exists(effectivePath, capitalization) ?? sourceProvider.exists(returnPlatformPath(effectivePath), capitalization);
+            const context = data.ctx.files;
+            const get = context.exists(effectivePath, capitalization) ?? context.exists(returnPlatformPath(effectivePath), capitalization);
             if (get && !found.includes(effectivePath)) {
                 found.push(returnPlatformPath(effectivePath));
             }
@@ -145,7 +138,7 @@ function processSourceCall(name, args, rootId, data, config) {
         (0, known_call_handling_1.processKnownFunctionCall)({ name, args, rootId, data, origin: 'builtin:source' }).information
         : (0, info_1.initializeCleanDataflowInformation)(rootId, data);
     const sourceFileArgument = args[0];
-    if (!config.forceFollow && data.flowrConfig.ignoreSourceCalls) {
+    if (!config.forceFollow && data.ctx.config.ignoreSourceCalls) {
         (0, log_1.expensiveTrace)(logger_1.dataflowLogger, () => `Skipping source call ${JSON.stringify(sourceFileArgument)} (disabled in config file)`);
         (0, unknown_side_effect_1.handleUnknownSideEffect)(information.graph, information.environment, rootId);
         return information;
@@ -155,26 +148,25 @@ function processSourceCall(name, args, rootId, data, config) {
         sourceFile = [(0, retriever_1.removeRQuotes)(sourceFileArgument.lexeme)];
     }
     else if (sourceFileArgument !== r_function_call_1.EmptyArgument) {
-        const resolved = (0, general_1.valueSetGuard)((0, alias_tracking_1.resolveIdToValue)(sourceFileArgument.info.id, { environment: data.environment, idMap: data.completeAst.idMap, resolve: data.flowrConfig.solver.variables }));
+        const resolved = (0, general_1.valueSetGuard)((0, alias_tracking_1.resolveIdToValue)(sourceFileArgument.info.id, { environment: data.environment, idMap: data.completeAst.idMap, resolve: data.ctx.config.solver.variables, ctx: data.ctx }));
         sourceFile = resolved?.elements.map(r => r.type === 'string' && (0, r_value_1.isValue)(r.value) ? r.value.str : undefined).filter(assert_1.isNotUndefined);
     }
     if (sourceFile && sourceFile.length === 1) {
         const path = (0, retriever_1.removeRQuotes)(sourceFile[0]);
-        let filepath = path ? findSource(data.flowrConfig.solver.resolveSource, path, data) : path;
+        let filepath = path ? findSource(data.ctx.config.solver.resolveSource, path, data) : path;
         if (Array.isArray(filepath)) {
             filepath = filepath?.[0];
         }
         if (filepath !== undefined) {
-            const request = sourceProvider.createRequest(filepath);
             // check if the sourced file has already been dataflow analyzed, and if so, skip it
-            const limit = data.flowrConfig.solver.resolveSource?.repeatedSourceLimit ?? 0;
-            const findCount = data.referenceChain.filter(e => e.request === request.request && e.content === request.content).length;
+            const limit = data.ctx.config.solver.resolveSource?.repeatedSourceLimit ?? 0;
+            const findCount = data.referenceChain.filter(e => e !== undefined && filepath === e).length;
             if (findCount > limit) {
-                logger_1.dataflowLogger.warn(`Found cycle (>=${limit + 1}) in dataflow analysis for ${JSON.stringify(request)}: ${JSON.stringify(data.referenceChain)}, skipping further dataflow analysis`);
+                logger_1.dataflowLogger.warn(`Found cycle (>=${limit + 1}) in dataflow analysis for ${JSON.stringify(filepath)}: ${JSON.stringify(data.referenceChain)}, skipping further dataflow analysis`);
                 (0, unknown_side_effect_1.handleUnknownSideEffect)(information.graph, information.environment, rootId);
                 return information;
             }
-            return sourceRequest(rootId, request, data, information, (0, decorate_1.sourcedDeterministicCountingIdGenerator)((findCount > 0 ? findCount + '::' : '') + path, name.location));
+            return sourceRequest(rootId, { request: 'file', content: filepath }, data, information, (0, decorate_1.sourcedDeterministicCountingIdGenerator)((findCount > 0 ? findCount + '::' : '') + path, name.location));
         }
     }
     (0, log_1.expensiveTrace)(logger_1.dataflowLogger, () => `Non-constant argument ${JSON.stringify(sourceFile)} for source is currently not supported, skipping`);
@@ -183,30 +175,48 @@ function processSourceCall(name, args, rootId, data, config) {
 }
 /**
  * Processes a source request with the given dataflow processor information and existing dataflow information
+ * Otherwise, this can be an {@link RProjectFile} representing a standalone source file
  */
 function sourceRequest(rootId, request, data, information, getId) {
-    if (request.request === 'file') {
-        /* check if the file exists and if not, fail */
-        if (!fs_1.default.existsSync(request.content)) {
+    // parse, normalize and dataflow the sourced file
+    let dataflow;
+    let fst;
+    let filePath;
+    if ('root' in request) {
+        fst = request;
+        filePath = request.filePath;
+    }
+    else {
+        const textRequest = data.ctx.files.resolveRequest(request);
+        if (textRequest === undefined && request.request === 'file') {
+            // if translation failed there is nothing we can do!!
             logger_1.dataflowLogger.warn(`Failed to analyze sourced file ${JSON.stringify(request)}: file does not exist`);
             (0, unknown_side_effect_1.handleUnknownSideEffect)(information.graph, information.environment, rootId);
             return information;
         }
+        else {
+            (0, assert_1.guard)(textRequest !== undefined, `Expected text request to be defined for sourced file ${JSON.stringify(request)}`);
+        }
+        const parsed = (!data.parser.async ? data.parser : new shell_executor_1.RShellExecutor()).parse(textRequest.r);
+        const normalized = (typeof parsed !== 'string' ?
+            (0, parser_1.normalizeTreeSitter)({ files: [{ parsed, filePath: textRequest.path }] }, getId, data.ctx.config)
+            : (0, parser_1.normalize)({ files: [{ parsed, filePath: textRequest.path }] }, getId));
+        fst = normalized.ast.files[0];
+        // this can be improved, see issue #628
+        for (const [k, v] of normalized.idMap) {
+            data.completeAst.idMap.set(k, v);
+        }
+        // add to the main ast
+        if (!data.completeAst.ast.files.find(f => f.filePath === fst.filePath)) {
+            data.completeAst.ast.files.push(fst);
+        }
+        filePath = textRequest.path;
     }
-    // parse, normalize and dataflow the sourced file
-    let normalized;
-    let dataflow;
     try {
-        const file = request.request === 'file' ? request.content : undefined;
-        const parsed = (!data.parser.async ? data.parser : new shell_executor_1.RShellExecutor()).parse(request);
-        normalized = (typeof parsed !== 'string' ?
-            (0, parser_1.normalizeTreeSitter)({ parsed }, getId, data.flowrConfig, file)
-            : (0, parser_1.normalize)({ parsed }, getId, file));
-        dataflow = (0, processor_1.processDataflowFor)(normalized.ast, {
+        dataflow = (0, processor_1.processDataflowFor)(fst.root, {
             ...data,
-            currentRequest: request,
             environment: information.environment,
-            referenceChain: [...data.referenceChain, request]
+            referenceChain: [...data.referenceChain, fst.filePath]
         });
     }
     catch (e) {
@@ -216,47 +226,40 @@ function sourceRequest(rootId, request, data, information, getId) {
         return information;
     }
     // take the entry point as well as all the written references, and give them a control dependency to the source call to show that they are conditional
-    if (dataflow.graph.hasVertex(dataflow.entryPoint)) {
-        dataflow.graph.addControlDependency(dataflow.entryPoint, rootId, true);
-    }
-    for (const out of dataflow.out) {
-        dataflow.graph.addControlDependency(out.nodeId, rootId, true);
+    if (!String(rootId).startsWith('file-')) {
+        if (dataflow.graph.hasVertex(dataflow.entryPoint)) {
+            dataflow.graph.addControlDependency(dataflow.entryPoint, rootId, true);
+        }
+        for (const out of dataflow.out) {
+            dataflow.graph.addControlDependency(out.nodeId, rootId, true);
+        }
     }
-    dataflow.graph.addFile(request.request === 'file' ? request.content : '<inline>');
+    data.ctx.files.addConsideredFile(filePath ?? '<inline>');
     // update our graph with the sourced file's information
-    const newInformation = { ...information };
-    newInformation.environment = (0, overwrite_1.overwriteEnvironment)(information.environment, dataflow.environment);
-    newInformation.graph.mergeWith(dataflow.graph);
-    // this can be improved, see issue #628
-    for (const [k, v] of normalized.idMap) {
-        data.completeAst.idMap.set(k, v);
-    }
     return {
-        ...newInformation,
-        in: newInformation.in.concat(dataflow.in),
-        out: newInformation.out.concat(dataflow.out),
-        unknownReferences: newInformation.unknownReferences.concat(dataflow.unknownReferences),
+        ...information,
+        environment: (0, overwrite_1.overwriteEnvironment)(information.environment, dataflow.environment),
+        graph: information.graph.mergeWith(dataflow.graph),
+        in: information.in.concat(dataflow.in),
+        out: information.out.concat(dataflow.out),
+        unknownReferences: information.unknownReferences.concat(dataflow.unknownReferences),
         exitPoints: dataflow.exitPoints
     };
 }
 /**
  * Processes a standalone source file (i.e., not from a source function call)
  */
-function standaloneSourceFile(inputRequest, data, uniqueSourceId, information) {
-    const path = inputRequest.request === 'file' ? inputRequest.content : '-inline-';
-    /* this way we can still pass content */
-    const request = inputRequest.request === 'file' ? sourceProvider.createRequest(inputRequest.content) : inputRequest;
+function standaloneSourceFile(idx, file, data, information) {
     // check if the sourced file has already been dataflow analyzed, and if so, skip it
-    if (data.referenceChain.find(e => e.request === request.request && e.content === request.content)) {
-        logger_1.dataflowLogger.info(`Found loop in dataflow analysis for ${JSON.stringify(request)}: ${JSON.stringify(data.referenceChain)}, skipping further dataflow analysis`);
-        (0, unknown_side_effect_1.handleUnknownSideEffect)(information.graph, information.environment, uniqueSourceId);
+    if (data.referenceChain.find(e => e !== undefined && e === file.filePath)) {
+        logger_1.dataflowLogger.info(`Found loop in dataflow analysis for ${JSON.stringify(file.filePath)}: ${JSON.stringify(data.referenceChain)}, skipping further dataflow analysis`);
+        (0, unknown_side_effect_1.handleUnknownSideEffect)(information.graph, information.environment, file.root.info.id);
         return information;
     }
-    return sourceRequest(uniqueSourceId, request, {
+    return sourceRequest('file-' + idx, file, {
         ...data,
-        currentRequest: request,
         environment: information.environment,
-        referenceChain: [...data.referenceChain, inputRequest]
-    }, information, (0, decorate_1.deterministicPrefixIdGenerator)(path + '::' + uniqueSourceId));
+        referenceChain: [...data.referenceChain, file.filePath]
+    }, information);
 }
 //# sourceMappingURL=built-in-source.js.map

package/dataflow/internal/process/functions/call/built-in/built-in-special-bin-op.d.ts

@@ -6,7 +6,9 @@ import type { RSymbol } from '../../../../../../r-bridge/lang-4.x/ast/model/node
 import type { NodeId } from '../../../../../../r-bridge/lang-4.x/ast/model/processing/node-id';
 import type { ForceArguments } from '../common';
 /**
- *
+ * Process a special built-in binary operator, possibly lazily.
+ * For example, the logical AND `&&` and OR `||` operators only evaluate their right-hand side if necessary.
+ * Please note that this is not (directly) related to R's special binary operators like `%in%`.
  */
 export declare function processSpecialBinOp<OtherInfo>(name: RSymbol<OtherInfo & ParentInformation>, args: readonly RFunctionArgument<OtherInfo & ParentInformation>[], rootId: NodeId, data: DataflowProcessorInformation<OtherInfo & ParentInformation>, config: {
     lazy: boolean;

package/dataflow/internal/process/functions/call/built-in/built-in-special-bin-op.js

@@ -5,7 +5,9 @@ const known_call_handling_1 = require("../known-call-handling");
 const logger_1 = require("../../../../../logger");
 const edge_1 = require("../../../../../graph/edge");
 /**
- *
+ * Process a special built-in binary operator, possibly lazily.
+ * For example, the logical AND `&&` and OR `||` operators only evaluate their right-hand side if necessary.
+ * Please note that this is not (directly) related to R's special binary operators like `%in%`.
  */
 function processSpecialBinOp(name, args, rootId, data, config) {
     if (!config.lazy) {

package/dataflow/internal/process/functions/call/built-in/built-in-vector.js

@@ -16,7 +16,7 @@ const containers_1 = require("../../../../../../util/containers");
  */
 function processVector(name, args, rootId, data) {
     const fnCall = (0, known_call_handling_1.processKnownFunctionCall)({ name, args, rootId, data, origin: 'builtin:vector' });
-    if (!data.flowrConfig.solver.pointerTracking) {
+    if (!data.ctx.config.solver.pointerTracking) {
         return fnCall.information;
     }
     let vectorArgs = [];
@@ -52,7 +52,7 @@ function processVector(name, args, rootId, data) {
             vectorArgs = vectorArgs.concat(flattenedIndices);
         }
     }
-    if ((0, config_1.isOverPointerAnalysisThreshold)(data.flowrConfig, vectorArgs.length)) {
+    if ((0, config_1.isOverPointerAnalysisThreshold)(data.ctx.config, vectorArgs.length)) {
         return fnCall.information;
     }
     const indices = {

package/dataflow/internal/process/functions/call/built-in/built-in-while-loop.js

@@ -8,11 +8,11 @@ const assert_1 = require("../../../../../../util/assert");
 const unpack_argument_1 = require("../argument/unpack-argument");
 const r_function_call_1 = require("../../../../../../r-bridge/lang-4.x/ast/model/nodes/r-function-call");
 const logger_1 = require("../../../../../logger");
-const environment_1 = require("../../../../../environments/environment");
 const edge_1 = require("../../../../../graph/edge");
 const identifier_1 = require("../../../../../environments/identifier");
 const general_1 = require("../../../../../eval/values/general");
 const alias_tracking_1 = require("../../../../../eval/resolve/alias-tracking");
+const reference_to_maybe_1 = require("../../../../../environments/reference-to-maybe");
 /**
  *
  */
@@ -21,13 +21,13 @@ function processWhileLoop(name, args, rootId, data) {
         logger_1.dataflowLogger.warn(`While-Loop ${name.content} does not have 2 arguments, skipping`);
         return (0, known_call_handling_1.processKnownFunctionCall)({ name, args, rootId, data, origin: 'default' }).information;
     }
-    const unpackedArgs = args.map(e => (0, unpack_argument_1.unpackArgument)(e));
+    const unpackedArgs = args.map(e => (0, unpack_argument_1.unpackNonameArg)(e));
     if (unpackedArgs.some(assert_1.isUndefined)) {
         logger_1.dataflowLogger.warn(`While-Loop ${name.content} has empty arguments in ${JSON.stringify(args)}, skipping`);
         return (0, known_call_handling_1.processKnownFunctionCall)({ name, args, rootId, data, origin: 'default' }).information;
     }
     // we should defer this to the abstract interpretation
-    const values = (0, alias_tracking_1.resolveIdToValue)(unpackedArgs[0]?.info.id, { environment: data.environment, idMap: data.completeAst.idMap, resolve: data.flowrConfig.solver.variables });
+    const values = (0, alias_tracking_1.resolveIdToValue)(unpackedArgs[0]?.info.id, { environment: data.environment, idMap: data.completeAst.idMap, resolve: data.ctx.config.solver.variables, ctx: data.ctx });
     const conditionIsAlwaysFalse = (0, general_1.valueSetGuard)(values)?.elements.every(d => d.type === 'logical' && d.value === false) ?? false;
     //We don't care about the body if it never executes
     if (conditionIsAlwaysFalse) {
@@ -69,7 +69,7 @@ function processWhileLoop(name, args, rootId, data) {
         return condition;
     }
     const cdTrue = { id: name.info.id, when: true };
-    const remainingInputs = (0, linker_1.linkInputs)((0, environment_1.makeAllMaybe)(body.unknownReferences, information.graph, information.environment, false, cdTrue).concat((0, environment_1.makeAllMaybe)(body.in, information.graph, information.environment, false, cdTrue)), information.environment, condition.in.concat(condition.unknownReferences), information.graph, true);
+    const remainingInputs = (0, linker_1.linkInputs)((0, reference_to_maybe_1.makeAllMaybe)(body.unknownReferences, information.graph, information.environment, false, cdTrue).concat((0, reference_to_maybe_1.makeAllMaybe)(body.in, information.graph, information.environment, false, cdTrue)), information.environment, condition.in.concat(condition.unknownReferences), information.graph, true);
     (0, linker_1.linkCircularRedefinitionsWithinALoop)(information.graph, (0, linker_1.produceNameSharedIdMap)((0, linker_1.findNonLocalReads)(information.graph, condition.in)), body.out);
     (0, linker_1.reapplyLoopExitPoints)(body.exitPoints, body.in.concat(body.out, body.unknownReferences));
     // as the while-loop always evaluates its condition
@@ -77,7 +77,7 @@ function processWhileLoop(name, args, rootId, data) {
     return {
         unknownReferences: [],
         in: [{ nodeId: name.info.id, name: name.lexeme, controlDependencies: originalDependency, type: identifier_1.ReferenceType.Function }, ...remainingInputs],
-        out: condition.out.concat((0, environment_1.makeAllMaybe)(body.out, information.graph, information.environment, true, cdTrue)),
+        out: condition.out.concat((0, reference_to_maybe_1.makeAllMaybe)(body.out, information.graph, information.environment, true, cdTrue)),
         entryPoint: name.info.id,
         exitPoints: (0, info_1.filterOutLoopExitPoints)(body.exitPoints),
         graph: information.graph,

package/dataflow/internal/process/functions/call/common.d.ts

@@ -30,7 +30,7 @@ export interface ProcessAllArgumentResult {
     readonly processedArguments: (DataflowInformation | undefined)[];
 }
 /**
- *
+ * Processes all arguments for a function call, updating the given final graph and environment.
  */
 export declare function processAllArguments<OtherInfo>({ functionName, args, data, finalGraph, functionRootId, forceArgs, patchData }: ProcessAllArgumentInput<OtherInfo>): ProcessAllArgumentResult;
 export interface PatchFunctionCallInput<OtherInfo> {

package/dataflow/internal/process/functions/call/common.js

@@ -29,8 +29,7 @@ function forceVertexArgumentValueReferences(rootId, value, graph, env) {
             }
         }
     }
-    const containedSubflowIn = graph.vertices(true)
-        .filter(([, info]) => (0, vertex_1.isFunctionDefinitionVertex)(info))
+    const containedSubflowIn = graph.verticesOfType(vertex_1.VertexType.FunctionDefinition)
         .flatMap(([, info]) => info.subflow.in)
         .toArray();
     // try to resolve them against the current environment
@@ -44,9 +43,9 @@ function forceVertexArgumentValueReferences(rootId, value, graph, env) {
     }
 }
 /**
- *
+ * Processes all arguments for a function call, updating the given final graph and environment.
  */
-function processAllArguments({ functionName, args, data, finalGraph, functionRootId, forceArgs = [], patchData = d => d }) {
+function processAllArguments({ functionName, args, data, finalGraph, functionRootId, forceArgs = [], patchData }) {
     let finalEnv = functionName.environment;
     // arg env contains the environments with other args defined
     let argEnv = functionName.environment;
@@ -56,7 +55,7 @@ function processAllArguments({ functionName, args, data, finalGraph, functionRoo
     let i = -1;
     for (const arg of args) {
         i++;
-        data = patchData(data, i);
+        data = patchData?.(data, i) ?? data;
         if (arg === r_function_call_1.EmptyArgument) {
             callArgs.push(r_function_call_1.EmptyArgument);
             processedArguments.push(undefined);
@@ -70,7 +69,7 @@ function processAllArguments({ functionName, args, data, finalGraph, functionRoo
         finalEnv = (0, overwrite_1.overwriteEnvironment)(finalEnv, processed.environment);
         finalGraph.mergeWith(processed.graph);
         // resolve reads within argument, we resolve before adding the `processed.environment` to avoid cyclic dependencies
-        for (const ingoing of [...processed.in, ...processed.unknownReferences]) {
+        for (const ingoing of processed.in.concat(processed.unknownReferences)) {
             // check if it is called directly
             const vtx = finalGraph.getVertex(ingoing.nodeId);
             const tryToResolve = ingoing.name ? (0, resolve_by_name_1.resolveByName)(ingoing.name, argEnv, vtx?.tag === vertex_1.VertexType.FunctionCall ? identifier_1.ReferenceType.Function : identifier_1.ReferenceType.Unknown) : undefined;
@@ -122,7 +121,7 @@ function patchFunctionCall({ nextGraph, rootId, name, data, argumentProcessResul
         args: argumentProcessResult.map(arg => arg === undefined ? r_function_call_1.EmptyArgument : { nodeId: arg.entryPoint, controlDependencies: undefined, call: undefined, type: identifier_1.ReferenceType.Argument }),
         origin: [origin],
         link
-    }, !nextGraph.hasVertex(rootId) || nextGraph.isRoot(rootId), true);
+    }, data.ctx.env.makeCleanEnv(), !nextGraph.hasVertex(rootId) || nextGraph.isRoot(rootId), true);
     for (const arg of argumentProcessResult) {
         if (arg) {
             nextGraph.addEdge(rootId, arg.entryPoint, edge_1.EdgeType.Argument);

package/dataflow/internal/process/functions/call/default-call-handling.d.ts

@@ -3,6 +3,8 @@ import type { DataflowProcessorInformation } from '../../../../processor';
 import type { ParentInformation } from '../../../../../r-bridge/lang-4.x/ast/model/processing/decorate';
 import type { RFunctionCall } from '../../../../../r-bridge/lang-4.x/ast/model/nodes/r-function-call';
 /**
- *
+ * Processes a function call, either named or unnamed.
+ * @see {@link processNamedCall}
+ * @see {@link processUnnamedFunctionCall}
  */
 export declare function processFunctionCall<OtherInfo>(functionCall: RFunctionCall<OtherInfo & ParentInformation>, data: DataflowProcessorInformation<OtherInfo & ParentInformation>): DataflowInformation;

package/dataflow/internal/process/functions/call/default-call-handling.js

@@ -4,7 +4,9 @@ exports.processFunctionCall = processFunctionCall;
 const named_call_handling_1 = require("./named-call-handling");
 const unnamed_call_handling_1 = require("./unnamed-call-handling");
 /**
- *
+ * Processes a function call, either named or unnamed.
+ * @see {@link processNamedCall}
+ * @see {@link processUnnamedFunctionCall}
  */
 function processFunctionCall(functionCall, data) {
     if (functionCall.named) {

package/dataflow/internal/process/functions/call/known-call-handling.js

@@ -36,7 +36,7 @@ function processKnownFunctionCall({ name, args, rootId, data, reverseOrder = fal
     const finalGraph = new graph_1.DataflowGraph(data.completeAst.idMap);
     const functionCallName = name.content;
     (0, log_1.expensiveTrace)(logger_1.dataflowLogger, () => `Processing known function call ${functionCallName} with ${args.length} arguments`);
-    const processArgs = reverseOrder ? [...args].reverse() : args;
+    const processArgs = reverseOrder ? args.toReversed() : args;
     const { finalEnv, callArgs, remainingReadInArgs, processedArguments } = (0, common_1.processAllArguments)({ functionName, args: processArgs, data, finalGraph, functionRootId: rootId, patchData, forceArgs });
     if (markAsNSE) {
         markNonStandardEvaluationEdges(markAsNSE, processedArguments, finalGraph, rootId);
@@ -49,10 +49,10 @@ function processKnownFunctionCall({ name, args, rootId, data, reverseOrder = fal
         /* will be overwritten accordingly */
         onlyBuiltin: false,
         cds: data.controlDependencies,
-        args: reverseOrder ? [...callArgs].reverse() : callArgs,
+        args: reverseOrder ? callArgs.reverse() : callArgs,
         indicesCollection: indicesCollection,
         origin: origin === 'default' ? ['function'] : [origin]
-    });
+    }, data.ctx.env.makeCleanEnv());
     if (hasUnknownSideEffect) {
         (0, unknown_side_effect_1.handleUnknownSideEffect)(finalGraph, data.environment, rootId);
     }
@@ -70,7 +70,7 @@ function processKnownFunctionCall({ name, args, rootId, data, reverseOrder = fal
             entryPoint: rootId,
             exitPoints: [{ nodeId: rootId, type: 0 /* ExitPointType.Default */, controlDependencies: data.controlDependencies }]
         },
-        processedArguments: reverseOrder ? [...processedArguments].reverse() : processedArguments,
+        processedArguments: reverseOrder ? processedArguments.reverse() : processedArguments,
         fnRef
     };
 }

package/dataflow/internal/process/functions/call/named-call-handling.js

@@ -13,13 +13,13 @@ function mergeInformation(info, newInfo) {
         return newInfo;
     }
     return {
-        unknownReferences: [...info.unknownReferences, ...newInfo.unknownReferences],
-        in: [...info.in, ...newInfo.in],
-        out: [...info.out, ...newInfo.out],
+        unknownReferences: info.unknownReferences.concat(newInfo.unknownReferences),
+        in: info.in.concat(newInfo.in),
+        out: info.out.concat(newInfo.out),
         graph: info.graph.mergeWith(newInfo.graph),
         environment: (0, append_1.appendEnvironment)(info.environment, newInfo.environment),
         entryPoint: newInfo.entryPoint,
-        exitPoints: [...info.exitPoints, ...newInfo.exitPoints],
+        exitPoints: info.exitPoints.concat(newInfo.exitPoints),
     };
 }
 function processDefaultFunctionProcessor(information, name, args, rootId, data) {

package/dataflow/internal/process/functions/call/unnamed-call-handling.js

@@ -45,7 +45,7 @@ function processUnnamedFunctionCall(functionCall, data) {
         cds: data.controlDependencies,
         args: callArgs, // same reference
         origin: [exports.UnnamedFunctionCallOrigin]
-    });
+    }, data.ctx.env.makeCleanEnv());
     let inIds = remainingReadInArgs;
     inIds.push({ nodeId: functionRootId, name: functionCallName, controlDependencies: data.controlDependencies, type: identifier_1.ReferenceType.Function });
     if (functionCall.calledFunction.type === type_1.RType.FunctionDefinition) {

package/dataflow/internal/process/functions/process-argument.js

@@ -34,7 +34,7 @@ function processFunctionArgument(argument, data) {
             tag: vertex_1.VertexType.Use,
             id: argument.info.id,
             cds: data.controlDependencies
-        });
+        }, data.ctx.env.makeCleanEnv());
         entryPoint = argument.info.id;
     }
     const ingoingRefs = [...value?.unknownReferences ?? [], ...value?.in ?? [], ...(name === undefined ? [] : [...name.in])];

package/dataflow/internal/process/functions/process-parameter.js

@@ -23,13 +23,13 @@ function processFunctionParameter(parameter, data) {
     for (const writtenNode of writtenNodes) {
         log_1.log.trace(`parameter ${writtenNode.name} (${writtenNode.nodeId}) is defined at id ${writtenNode.definedAt} with ${defaultValue === undefined ? 'no default value' : ' no default value'}`);
         graph.setDefinitionOfVertex(writtenNode);
-        environment = (0, define_1.define)(writtenNode, false, environment, data.flowrConfig);
+        environment = (0, define_1.define)(writtenNode, false, environment, data.ctx.config);
         if (defaultValue !== undefined) {
             if (parameter.defaultValue?.type === type_1.RType.FunctionDefinition) {
                 graph.addEdge(writtenNode, parameter.defaultValue.info.id, edge_1.EdgeType.DefinedBy);
             }
             else {
-                const definedBy = [...defaultValue.in, ...defaultValue.unknownReferences];
+                const definedBy = defaultValue.in.concat(defaultValue.unknownReferences);
                 for (const node of definedBy) {
                     graph.addEdge(writtenNode, node, edge_1.EdgeType.DefinedBy);
                 }
@@ -38,8 +38,8 @@ function processFunctionParameter(parameter, data) {
     }
     return {
         unknownReferences: [],
-        in: defaultValue === undefined ? [] : [...defaultValue.in, ...defaultValue.unknownReferences, ...name.in],
-        out: [...(defaultValue?.out ?? []), ...name.out, ...name.unknownReferences],
+        in: defaultValue === undefined ? [] : defaultValue.in.concat(defaultValue.unknownReferences, name.in),
+        out: (defaultValue?.out ?? []).concat(name.out, name.unknownReferences),
         graph: graph,
         environment: environment,
         entryPoint: parameter.info.id,

package/dataflow/internal/process/process-symbol.js

@@ -22,7 +22,7 @@ function processSymbol(symbol, data) {
             tag: vertex_1.VertexType.Use,
             id: symbol.info.id,
             cds: data.controlDependencies
-        }),
+        }, data.ctx.env.makeCleanEnv()),
         entryPoint: symbol.info.id,
         exitPoints: [{ nodeId: symbol.info.id, type: 0 /* ExitPointType.Default */, controlDependencies: data.controlDependencies }]
     };

package/dataflow/internal/process/process-value.d.ts

@@ -4,4 +4,4 @@ import type { RNodeWithParent } from '../../../r-bridge/lang-4.x/ast/model/proce
 /**
  *
  */
-export declare function processValue<OtherInfo>({ info: { id } }: RNodeWithParent, data: DataflowProcessorInformation<OtherInfo>): DataflowInformation;
+export declare function processValue<OtherInfo>({ info: { id } }: RNodeWithParent, { controlDependencies, completeAst: { idMap }, ctx: { env }, environment }: DataflowProcessorInformation<OtherInfo>): DataflowInformation;

package/dataflow/internal/process/process-value.js

@@ -7,18 +7,18 @@ const identifier_1 = require("../../environments/identifier");
 /**
  *
  */
-function processValue({ info: { id } }, data) {
+function processValue({ info: { id } }, { controlDependencies, completeAst: { idMap }, ctx: { env }, environment }) {
     return {
         unknownReferences: [],
-        in: [{ nodeId: id, name: undefined, controlDependencies: data.controlDependencies, type: identifier_1.ReferenceType.Constant }],
+        in: [{ nodeId: id, name: undefined, controlDependencies, type: identifier_1.ReferenceType.Constant }],
         out: [],
-        environment: data.environment,
-        graph: new graph_1.DataflowGraph(data.completeAst.idMap).addVertex({
+        environment,
+        graph: new graph_1.DataflowGraph(idMap).addVertex({
             tag: vertex_1.VertexType.Value,
             id: id,
-            cds: data.controlDependencies
-        }),
-        exitPoints: [{ nodeId: id, type: 0 /* ExitPointType.Default */, controlDependencies: data.controlDependencies }],
+            cds: controlDependencies
+        }, env.makeCleanEnv()),
+        exitPoints: [{ nodeId: id, type: 0 /* ExitPointType.Default */, controlDependencies }],
         entryPoint: id
     };
 }

package/dataflow/origin/dfg-get-symbol-refs.d.ts

@@ -14,6 +14,6 @@ import type { DataflowGraph } from '../graph/graph';
  * @example getAllRefsToSymbol('3\@y') returns ['3\@y', '4\@y']
  * @param graph  - Dataflow Graph
  * @param nodeId - NodeId of Symbol to resolve
- * @returns List including the Definitions and Refereneces to that definition
+ * @returns List including the Definitions and References to that definition
  */
 export declare function getAllRefsToSymbol(graph: DataflowGraph, nodeId: NodeId): NodeId[] | undefined;

package/dataflow/origin/dfg-get-symbol-refs.js

@@ -18,7 +18,7 @@ const dfg_get_origin_1 = require("./dfg-get-origin");
  * @example getAllRefsToSymbol('3\@y') returns ['3\@y', '4\@y']
  * @param graph  - Dataflow Graph
  * @param nodeId - NodeId of Symbol to resolve
- * @returns List including the Definitions and Refereneces to that definition
+ * @returns List including the Definitions and References to that definition
  */
 function getAllRefsToSymbol(graph, nodeId) {
     // Get all origins and filter for ones that happen for sure