Skip to content

docs(readme): add architecture diagram#2

Merged
mbongeb merged 1 commit intomainfrom
docs/add-architecture-diagram
Feb 9, 2026
Merged

docs(readme): add architecture diagram#2
mbongeb merged 1 commit intomainfrom
docs/add-architecture-diagram

Conversation

@mbongeb
Copy link
Copy Markdown
Owner

@mbongeb mbongeb commented Feb 9, 2026

Replace the ASCII architecture diagram with the eraser.io-generated diagram showing the full system layout including deployment pipeline.

Add the eraser.io-generated architecture diagram showing the full
system: Client → Lambda Function URLs → DynamoDB, deployment
pipeline (GitHub Actions → OIDC → AWS CDK), and CloudWatch logging.
@mbongeb mbongeb merged commit af4b67c into main Feb 9, 2026
2 checks passed
@mbongeb mbongeb deleted the docs/add-architecture-diagram branch February 9, 2026 22:58
Copy link
Copy Markdown

@amazon-q-developer amazon-q-developer Bot left a comment

Choose a reason for hiding this comment

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

Review Summary

This PR replaces the ASCII architecture diagram with an image reference. However, there is a critical blocking issue:

Blocking Issue:

  • The referenced image file docs/architecture.png is not included in this PR, which will result in a broken image link in the README after merge

Action Required:
Add the docs/architecture.png file to this PR to ensure the image reference resolves correctly.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

Comment thread README.md
└──────────┘ └────────────────┘ Query │ DateTime) │
(Function URL) └───────────┘
```
![Architecture Diagram](docs/architecture.png)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🛑 Broken Link: The referenced image file docs/architecture.png does not appear to be included in this PR. Verify the image file exists at this path, or the README will display a broken image link after merge.

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