BedrockAgentCoreClient throws error when undefined actorId is passed

[sjoerdvanBommel]

Checkboxes for prior research

• I've gone through Developer Guide and API reference
• I've checked AWS Forums and StackOverflow.
• I've searched for previous similar issues and didn't find any solution.

Describe the bug

I'm trying to get a list of events and do not want to filter on actor. The documentation specifies:

The identifier of the actor for which to list events. **If specified**, only events from this actor are returned.

However, when I send this ListEventsCommand:

  const res = await client.send(
    new ListEventsCommand({
      memoryId: DEFAULT_MEMORY_ID,
      sessionId,
      actorId: undefined,
      includePayloads: true,
      maxResults: limit,
    }),

I receive this error:

"Error: No value provided for input HTTP label: actorId.\n    at resolvedPath (/Users/sjoerdvanbommel/workspace/cimpress/ags-ai-agent/node_modules/@smithy/core/dist-cjs/submodules/protocols/index.js:486:11)\n    at /Users/sjoerdvanbommel/workspace/cimpress/ags-ai-agent/node_modules/@smithy/core/dist-cjs/submodules/protocols/index.js:545:19\n    at RequestBuilder.build (/Users/sjoerdvanbommel/workspace/cimpress/ags-ai-agent/node_modules/@smithy/core/dist-cjs/submodules/protocols/index.js:511:7)\n    at process.processTicksAndRejections (node:internal/process/task_queues:105:5)"

Is this expected? If so, I would expect the type of actorId to be string instead of string | undefined

Regression Issue

• Select this option if this issue appears to be a regression.

SDK version number

@aws-sdk/client-bedrock-agentcore@3.873.0

Which JavaScript Runtime is this issue in?

Node.js

Details of the browser/Node.js/ReactNative version

v22.13.1

Reproduction Steps

See description

Observed Behavior

See description

Expected Behavior

See description

Possible Solution

No response

Additional Information/Context

No response

[aBurmeseDev]

Hi @sjoerdvanBommel - thanks for checking in.

I had reached out to appropriate internal service team to verify this behavior of ListEvents API and will get back to you once we hear back. (ref P297083653)

[aBurmeseDev]

Bedrock team member just confirmed that actorId is a required field for this operation and that the documentation needs to be updated.

[aBurmeseDev]

Just heard back from service team member that the docs has been updated to address the confusion around actorId. Closing issue.

[github-actions]

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.