Skip to content

Update Graph Import in session-management.mdx#710

Merged
JackYPCOnline merged 1 commit intostrands-agents:mainfrom
mvijil910:mvijil910-patch-1
Apr 13, 2026
Merged

Update Graph Import in session-management.mdx#710
JackYPCOnline merged 1 commit intostrands-agents:mainfrom
mvijil910:mvijil910-patch-1

Conversation

@mvijil910
Copy link
Copy Markdown
Contributor

Description

Updating session-management to fix import error. Graph is not exported in multiagent's init and so is not available for import; documentation should be updated to import and use GraphBuilder instead of the Graph class. Reference: [CORRECTION] Documentation is wrong about Graph and SessionManagement #408

Related Issues

Type of Change

  • New content
    X Content update/revision
  • Structure/organization improvement
  • Typo/formatting fix
  • Bug fix
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING document
  • My changes follow the project's documentation style
  • I have tested the documentation locally using npm run dev
  • Links in the documentation are valid and working

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Updating session-management to fix import error. Graph is not exported in multiagent's init and so is not available for import; documentation should be updated to import and use GraphBuilder instead of the Graph class. Reference: [CORRECTION] Documentation is wrong about Graph and SessionManagement strands-agents#408
@mvijil910 mvijil910 temporarily deployed to manual-approval March 25, 2026 22:45 — with GitHub Actions Inactive
@mvijil910 mvijil910 changed the title Update session-management.mdx Update Import in session-management.mdx Mar 25, 2026
@mvijil910 mvijil910 changed the title Update Import in session-management.mdx Update Graph Import in session-management.mdx Mar 25, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Assessment: Request Changes

The import fix is correct — GraphBuilder is the right export from strands.multiagent. However, the rest of the code example (lines 85–91) still uses Graph(...) with a constructor API that doesn't exist, so the example would still fail at runtime.

Review Details

Thanks for addressing this issue — the fix is on the right track and just needs the full example updated!

@JackYPCOnline JackYPCOnline merged commit 827df6e into strands-agents:main Apr 13, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants