[Bug]: Graph input with a field of the type "object" is not working

[valerypopoff]

What happened?

Even with the minimal graph like this:

And the code like this:

    let result = await runGraph(project, {
    //graph: '_main',
    inputs: {
      obj: {}
    },
    context: {
    },
    externalFunctions: {
    },  
    onUserEvent: {
    },
    openAiKey: process.env.openAiKey,
    openAiOrganization: '',
  })

I get this error:

Error running Rivet
Error: Graph 123 (VlYPTYilgVb_er3-AxeeQ) failed to process due to errors in nodes: Graph Input (KzXcn4Lx88xhuHDOkxxHu): TypeError: Cannot read properties of undefined (reading 'startsWith')
    at _GraphProcessor.processGraph (D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:13529:43)
    at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
    at async Object.run (D:\Programming\Rivet\node_modules\@ironclad\rivet-node\dist\cjs\bundle.cjs:147:23)
    at async runRivet (D:\Programming\Rivet\src\index.js:7:16)

P.S.: When I try to pass an object with "context" instead of "inputs", I get a similar error:

D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:471
  return type.endsWith("[]");
              ^

TypeError: Cannot read properties of undefined (reading 'endsWith')
    at isArrayDataType (D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:471:15)
    at getScalarTypeOf (D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:492:7)
    at D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:14139:31
    at Array.filter (<anonymous>)
    at #excludedDueToControlFlow (D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:14138:56)
    at #processNodeIfAllInputsAvailable (D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:13654:39)
    at D:\Programming\Rivet\node_modules\@ironclad\rivet-core\dist\cjs\bundle.cjs:13774:52
    at run (D:\Programming\Rivet\node_modules\p-queue-6\dist\index.js:157:104)
    at PQueue._tryToStartAnother (D:\Programming\Rivet\node_modules\p-queue-6\dist\index.js:105:17)
    at D:\Programming\Rivet\node_modules\p-queue-6\dist\index.js:171:18

P.P.S:
If I pass an object like this:

  let result = await runGraph(project, {
    //graph: '_main',
    inputs: {
      obj: { type: 'object', value: {} }
    },
    context: {
    },
    externalFunctions: {
    },  
    onUserEvent: {
    },
    openAiKey: process.env.openAiKey,
    openAiOrganization: '',
  })

it works.

What was the expected functionality?

The graph runs

Describe your environment

Node v22.11.0
@ironclad/rivet-node": "1.23.1"

Code of Conduct

• I agree to follow this project's Code of Conduct

[valerypopoff]

Gemini 2.0 Pro helped me find the solution and it suggests you do this (probably should do the same with "context" too):

// Inside GraphProcessor class...

  async processGraph(context, inputs = {}, contextValues = {}) {
    try {
      if (this.#running) {
        throw new Error("Cannot process graph while already processing");
      }
      this.#initProcessState();
      this.#context = context;

      // ****** MODIFICATION START ******
      // Original line:
      // this.#graphInputs = inputs;

      // New code: Iterate and wrap inputs if they are raw values
      this.#graphInputs = {}; // Initialize as an empty object
      for (const key in inputs) {
        if (Object.prototype.hasOwnProperty.call(inputs, key)) {
          const value = inputs[key];

          // Simple check: if it looks like a DataValue already, keep it.
          // A more robust check could validate value.type against known dataTypes.
          if (typeof value === 'object' && value !== null && 'type' in value && 'value' in value && dataTypes.includes(value.type)) {
             this.#graphInputs[key] = value; // Assume it's already a DataValue
          } else {
             // Otherwise, infer the type from the raw value (this handles primitives AND objects/arrays)
             this.#graphInputs[key] = inferType(value);
          }
        }
      }
      // ****** MODIFICATION END ******

      this.#contextValues ??= contextValues;

      // ... (rest of the processGraph method remains the same)

[abrenneke]

@valerypopoff are you using typescript? It shouldn't accept {} as a valid LooseDataValue for passing in like that. You have to do

inputs: {
  obj: {
    type: 'object',
    value: {}
  }
}

because otherwise it doesn't know if you're passing a DataValue or something that's not a DataValue

[valerypopoff]

@abrenneke no, I'm not using typescript. Nothing in the documentation https://rivet.ironcladapp.com/docs/api-reference/node/runGraphInFile got me thinking that I can't just pass a usual object like {}.

The documentation says that
type LooseDataValue = DataValue | string | number | boolean

But the article about DataValue is literally empty: https://rivet.ironcladapp.com/docs/api-reference/core/DataValue

As far as I understand, it's not a bug in the code, it's a bug in the documentation?

[abrenneke]

Yeah, missing from documentation. Sorry about that.

[valerypopoff]

Can we consider it to be a feature reqest then?