Skip to content

docs: correct three unverified capability claims - #78

Merged
sundargthb merged 1 commit into
mainfrom
docs/verify-capability-claims
Aug 21, 2026
Merged

docs: correct three unverified capability claims#78
sundargthb merged 1 commit into
mainfrom
docs/verify-capability-claims

Conversation

@sundargthb

Copy link
Copy Markdown
Collaborator

Docs only. Three claims in the README described behaviour the node does not have. Found while doing a pre-release check of the docs going out with the launch, and they are the same class of error as the AI Agent comparison corrected in #77.

1. "Session length: up to 8 hours"

The 28800-second figure appears twice in the SDK types, and neither is a harness session:

  • LifecycleConfiguration.maxLifetime — default 28800s, but that is the AgentCore Runtime instance lifetime, not a harness session
  • SessionConfiguration.sessionTimeoutInSeconds — max 28800s, but that is MCP Gateway session config

The node never sends either field (grep -rn "maxLifetime\|idleRuntimeSessionTimeout" nodes/ credentials/ returns nothing). The only duration it sends is the per-invocation timeoutSeconds, default 600, described in the UI as "wall-clock timeout for the entire invocation". So no user could reach eight hours through this node.

Now reads: Long-running sessions, with a per-invocation timeout you set

2. "Streaming responses" listed as a shipped feature

consumeStream() reads the event stream and accumulates — the doc comment at the top of helpers/stream.ts says so. grep -rn "sendChunk\|isStreaming" nodes/ returns nothing, so nothing reaches n8n's streaming UI. The node consumes a streaming API; it does not stream.

This matters because an n8n user reading the feature list would reasonably expect token-by-token output in the chat panel. An FSI SA asked about exactly this and offered to file a feature request.

Renamed to Structured responses, with the distinction stated explicitly, and streaming moved from the v0.1 roadmap row to later.

3. "Firecracker microVM per session"

AWS public documentation for AgentCore says "isolated microVM" and "session isolation" without naming the hypervisor. The only source for Firecracker in this repo is our own docs/SPEC.md, so the claim was circular. The comparison table and docs/SPEC.md (prose plus the ASCII diagram) now say "isolated microVM", matching the AgentCore docs.

Why now

npm currently serves the pre-#77 README, so the next release is what puts the corrected text on the npm page. That release lines up with the n8n partner blog and the AWS channel post, so this is the last chance to get the docs right before traffic arrives.

Checks run locally

typecheck, lint, format:check, test (114 passing / 12 files), secrets:check — all pass. No code touched, so no behaviour change and no node typeVersion bump.

The README comparison table and feature list described behaviour the node
does not have.

Session length "up to 8 hours" cited limits belonging to other resources.
LifecycleConfiguration.maxLifetime (default 28800s) is the AgentCore Runtime
instance lifetime, and the 28800s ceiling on
SessionConfiguration.sessionTimeoutInSeconds is for MCP Gateway sessions. The
only duration this node sends is the per-invocation timeoutSeconds, default
600, so no user could reach eight hours through it.

Streaming was listed as shipped. consumeStream() accumulates the event stream
and returns a completed result, and the node never calls n8n's sendChunk, so
nothing reaches the streaming UI. Renamed to "Structured responses" and moved
streaming to the later roadmap row.

"Firecracker microVM per session" named a hypervisor AWS public documentation
does not name. The table and docs/SPEC.md now say "isolated microVM", matching
the AgentCore docs.

Same class of error as the AI Agent comparison corrected in #77.
@sundargthb
sundargthb requested a review from a team August 20, 2026 19:04
@github-actions github-actions Bot added the size/s PR size: S label Aug 20, 2026

@notgitika notgitika left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Let's explore changelog changes being auto generated instead of manual code changes

@sundargthb
sundargthb merged commit 58b0090 into main Aug 21, 2026
15 checks passed
@sundargthb
sundargthb deleted the docs/verify-capability-claims branch August 21, 2026 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/s PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants