-
Notifications
You must be signed in to change notification settings - Fork 420
agentcore memory browser #422
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Anil Gurrala <[email protected]>
} | ||
|
||
// Get current AWS region | ||
let region = 'us-east-1'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like its hard coded, please confirm.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the hardcode logic
> | ||
<div className="namespace-type"> | ||
<span className={`type-badge ${ns.type.toLowerCase()}`}> | ||
{ns.type === 'SEMANTIC' ? 'Facts' : |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about if there is a custom strategy? Will this work or its hard coded?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes it still works for custom strategy. But added few more lines to make it more user friendly
return { | ||
"memories": [], | ||
"total_count": 0, | ||
"source": "list_memory_records", | ||
"memory_id": memory_id, | ||
"error": f"Failed to access namespace '{query.namespace}': {str(e)}" | ||
} |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eliminated all information exposure through exceptions
return { | ||
"memory_id": memory_id, | ||
"namespaces": found_namespaces, | ||
"total_found": len(found_namespaces), | ||
"method": "fallback_pattern_based", | ||
"message": f"Found {len(found_namespaces)} namespaces using fallback method (get_memory_strategies failed: {str(e)})" | ||
} |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eliminated all information exposure through exceptions
return { | ||
"memory_id": memory_id, | ||
"namespaces": [], | ||
"total_found": 0, | ||
"error": f"Both get_memory_strategies and fallback failed: {str(e)} / {str(e2)}" | ||
} |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
Stack trace information
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eliminated all information exposure through exceptions
return { | ||
"valid": False, | ||
"memory_id": query.memory_id, | ||
"accessible": False, | ||
"message": f"Memory ID validation failed: {str(e)}" | ||
} |
Check warning
Code scanning / CodeQL
Information exposure through an exception Medium
Stack trace information
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eliminated all information exposure through exceptions
Latest scan for commit: Security Scan ResultsScan Metadata
SummaryScanner ResultsThe table below shows findings by scanner, with status based on severity thresholds and dependencies: Column Explanations: Severity Levels (S/C/H/M/L/I):
Other Columns:
Scanner Results:
Severity Thresholds (Thresh Column):
Threshold Source: Values in parentheses indicate where the threshold is configured:
Statistics calculation:
Detailed FindingsShow 1 actionable findingsFinding 1: B104
Description: Code Snippet:
Report generated by Automated Security Helper (ASH) at 2025-10-02T23:52:09+00:00 |
* Transactional search images fixed, delete scripts * Add name to contributors * Renamed notebook, confirmed it works * Simplify README, get rid of local collector, change nb name * Update README.md Signed-off-by: seanslavichaws <[email protected]> * README update and fixed extra jinja2 spans * Add exclusions change to env example --------- Signed-off-by: seanslavichaws <[email protected]> Co-authored-by: Sean Slavich <[email protected]>
awslabs#423) * Adding LangGraph notebooks for agentcore checkpointer and memory store * fixing pip imports * feat: update math agent checkpointing file * docs: update documentation for human-in-the-loop sample * chore: rename math_agent notebook * refactor: update support-agent notebook structure * feat: update nutrition-assistant with custom strategy * feat: update nutrition assistant * docs: update readme * Fixing namespace in memory creation --------- Co-authored-by: Jack Gordley <[email protected]> Co-authored-by: Akarsha Sehwag <[email protected]> Co-authored-by: Akarsha Sehwag <[email protected]>
* Changes to support latest version of browser-use (0.7.9) * Changes based on comments and added name in contributors
…slabs#435) * Updated architecture diagram links in notebooks * fix: added agentcore-starterkit in requirements & updated memory manager notebook with cleaner docs --------- Co-authored-by: George Belsian <[email protected]>
* Updated observability samples to for Langfuse and Braintrust * Updated observability samples to for Langfuse and Braintrust --------- Signed-off-by: Rajesh Kumar Ravi <[email protected]> Co-authored-by: Rajesh Kumar Ravi <[email protected]>
…wslabs#415) * sagemakerai mlflow readme * sagemakerai mlflow readme * update readme --------- Signed-off-by: sandeepraveesh <[email protected]> Co-authored-by: Sandeep Raveesh <[email protected]>
3fa6c02
to
7a40903
Compare
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Amazon Bedrock AgentCore Samples Pull Request
Important
review ready
label to it. Only PRs withreview ready
will be reviewed.Issue number:
Concise description of the PR
User experience
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.