Skip to content

fix(identity): add missing utils.py and sys.path setup#1227

Open
madhununna wants to merge 1 commit intoawslabs:mainfrom
madhununna:fix/issue-969-missing-utils
Open

fix(identity): add missing utils.py and sys.path setup#1227
madhununna wants to merge 1 commit intoawslabs:mainfrom
madhununna:fix/issue-969-missing-utils

Conversation

@madhununna
Copy link
Copy Markdown

Fixes #969

Concise description of the PR

Added missing utils.py to 03-AgentCore-identity/ and added sys.path setup to two notebooks that were missing it, because all three identity tutorial notebooks import setup_cognito_user_pool and reauthenticate_user from utils but no utils.py existed in that directory.

User experience

Before: Running any of the three identity tutorial notebooks fails immediately with ModuleNotFoundError: No module named 'utils' before any code executes.

After: The import succeeds and notebooks proceed normally past the setup cell.

Checklist

  • I have reviewed the contributing guidelines
  • Add your name to CONTRIBUTORS.md
  • Are you uploading a dataset?
  • Have you documented Introduction, Architecture Diagram, Prerequisites, Usage, Sample Prompts, and Clean Up steps in your example README?
  • I agree to resolve any issues created for this example in the future
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

Notebooks import setup_cognito_user_pool and reauthenticate_user from utils but no utils.py existed, causing ModuleNotFoundError.

  • Add shared utils.py to 03-AgentCore-identity/ with both functions
  • Add sys.path cell to the two 3LO notebooks that were missing it (inbound auth notebook already had the sys.path setup)

Fixes #969

Amazon Bedrock AgentCore Samples Pull Request

Important

  1. We strictly follow a issue-first approach, please first open an issue relating to this Pull Request.
  2. Once this Pull Request is ready for review please attach review ready label to it. Only PRs with review ready will be reviewed.

Issue number:

Concise description of the PR

Changes to ..., because ...

User experience

Please share what the user experience looks like before and after this change

Checklist

If your change doesn't seem to apply, please leave them unchecked.

  • I have reviewed the contributing guidelines
  • Add your name to CONTRIBUTORS.md
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Are you uploading a dataset?
  • Have you documented Introduction, Architecture Diagram, Prerequisites, Usage, Sample Prompts, and Clean Up steps in your example README?
  • I agree to resolve any issues created for this example in the future.
  • I have performed a self-review of this change
  • Changes have been tested
  • Changes are documented

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.

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions github-actions bot added 01-tutorials 01-tutorials 03-AgentCore-identity 01-tutorials/03-AgentCore-identity labels Mar 31, 2026
Notebooks import setup_cognito_user_pool and reauthenticate_user
from utils but no utils.py existed, causing ModuleNotFoundError.

- Add shared utils.py to 03-AgentCore-identity/ with both functions
- Add sys.path cell to the two 3LO notebooks that were missing it
  (inbound auth notebook already had the sys.path setup)

Fixes awslabs#969
@madhununna madhununna force-pushed the fix/issue-969-missing-utils branch from 7d19f58 to dc5c8ca Compare March 31, 2026 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

01-tutorials 01-tutorials 03-AgentCore-identity 01-tutorials/03-AgentCore-identity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

01-tutorials - [Bug] ModuleNotFoundError

1 participant