Remove examples directory by jackaldenryan · Pull Request #149 · getzep/zep-js

jackaldenryan · 2025-08-04T23:47:53Z

Summary

Remove all example files from the examples directory
Clean up codebase by removing graph, langgraph, memory, and user examples that are no longer needed

Test plan

Verify all example files are properly deleted
Confirm no breaking changes to core functionality
Check that repository structure is cleaned up

🤖 Generated with Claude Code

Important

Remove outdated example files from the examples directory to clean up the codebase.

Removal:
- Deletes examples/graph/conversations.ts, examples/graph/entity_type_example.ts, examples/graph/graph_example.ts, and examples/graph/user_graph_example.ts.
- Deletes examples/langgraph/.env.example, examples/langgraph/.gitignore, examples/langgraph/README.md, examples/langgraph/agent.ts, examples/langgraph/package.json, and examples/langgraph/zep-memory.ts.
- Deletes examples/memory/chat_shoe_store_history.ts and examples/memory/memory_example.ts.
- Deletes examples/users/users.ts.
Purpose:
- Cleans up the codebase by removing outdated and unnecessary example files.
- Ensures no breaking changes to core functionality.

^{This description was created by}^{for 22f8e38. You can customize this summary. It will automatically update as commits are pushed.}

Clean up codebase by removing example files that are no longer needed. This includes graph, langgraph, memory, and user examples. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>

ellipsis-dev

Important

Looks good to me! 👍

Reviewed everything up to 22f8e38 in 51 seconds. Click for details.

Reviewed 1654 lines of code in 13 files
Skipped 1 files when reviewing.
Skipped posting 13 draft comments. View those below.
Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. examples/graph/conversations.ts:1

Draft comment:
Removal of example conversation file is appropriate. Ensure no references remain in core or docs.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

2. examples/graph/entity_type_example.ts:1

Draft comment:
Entity type example removed. Confirm there are no dependent references.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

3. examples/graph/graph_example.ts:1

Draft comment:
Graph example removal looks clean; ensure any documentation is updated accordingly.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

4. examples/graph/user_graph_example.ts:1

Draft comment:
User graph example removed; verify that no external references remain.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

5. examples/langgraph/.env.example:1

Draft comment:
Removed the .env example file. Ensure that any setup documentation is updated accordingly.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

6. examples/langgraph/.gitignore:1

Draft comment:
Removal of .gitignore for langgraph example is acceptable; verify that global ignore settings cover sensitive files.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

7. examples/langgraph/README.md:1

Draft comment:
LangGraph README removed. Ensure that any references in documentation or links are updated to reflect the change.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

8. examples/langgraph/agent.ts:1

Draft comment:
Agent implementation example removed. Confirm that this removal does not affect any core workflows or docs referencing it.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

9. examples/langgraph/package.json:1

Draft comment:
Removed langgraph package.json. Ensure build or dependency references are updated if necessary.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

10. examples/langgraph/zep-memory.ts:1

Draft comment:
Zep memory adapter example removed. Verify that core memory integration remains fully covered by tests.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

11. examples/memory/chat_shoe_store_history.ts:1

Draft comment:
Chat shoe store history example is removed; ensure no lingering references exist in tests or documentation.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

12. examples/memory/memory_example.ts:1

Draft comment:
Memory example file removed. Confirm that memory functionality is otherwise properly covered.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

13. examples/users/users.ts:1

Draft comment:
User management example removed. Verify that related documentation has been updated to reflect removal.
Reason this comment was not posted:
Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_JCTOrjXmiO7S83L5

^{You can customize}^{by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}

Both were only ever used by the examples/ directory, which was deleted in #149 (commit 4b7760c, 2025-08-04), leaving them orphaned and unused for ~10 months. The current tree has zero imports of uuid. Removing them drops two packages from the dependency tree (and their future Dependabot alerts) and supersedes the uuid 10->11 bump as the fix for the uuid buffer-bounds alerts (#45/#46). The fast-uri, picomatch, and @tootallnate/once resolutions are unaffected (pulled via jest/ webpack/msw, not uuid). yarn build and the full test suite (363) pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* chore(deps): resolve all Dependabot security alerts (dev deps) Resolve all 6 open Dependabot alerts, all in development scope: - uuid ^10.0.0 -> ^11.1.1 (devDependency) — missing buffer bounds check in v3/v5/v6 (#46, #45) - fast-uri 3.0.6 -> 3.1.2 via resolutions — host confusion and path traversal via percent-encoded sequences (#42, #41) - picomatch 2.3.1 -> 2.3.2 via resolutions — method injection in POSIX character classes (#36) - @tootallnate/once 2.0.0 -> 2.0.1 via resolutions — incorrect control flow scoping (#43) Transitive deps are pinned via the existing `resolutions` block since Yarn 1's `upgrade` doesn't bump transitive-only packages. All changes are dev-scoped; `yarn build` and the full test suite (363 tests) pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * chore(deps): remove unused uuid and @types/uuid devDependencies Both were only ever used by the examples/ directory, which was deleted in #149 (commit 4b7760c, 2025-08-04), leaving them orphaned and unused for ~10 months. The current tree has zero imports of uuid. Removing them drops two packages from the dependency tree (and their future Dependabot alerts) and supersedes the uuid 10->11 bump as the fix for the uuid buffer-bounds alerts (#45/#46). The fast-uri, picomatch, and @tootallnate/once resolutions are unaffected (pulled via jest/ webpack/msw, not uuid). yarn build and the full test suite (363) pass. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Remove examples directory

22f8e38

Clean up codebase by removing example files that are no longer needed. This includes graph, langgraph, memory, and user examples. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>

ellipsis-dev Bot reviewed Aug 4, 2025

View reviewed changes

jackaldenryan merged commit 4b7760c into main Aug 4, 2025
4 checks passed

jackaldenryan deleted the jack/remove-examples branch August 4, 2025 23:52

jackaldenryan mentioned this pull request Jun 5, 2026

chore(deps): resolve all Dependabot security alerts (dev deps) #187

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove examples directory#149

Remove examples directory#149
jackaldenryan merged 1 commit into
mainfrom
jack/remove-examples

jackaldenryan commented Aug 4, 2025 •

edited by ellipsis-dev Bot

Loading

Uh oh!

ellipsis-dev Bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

jackaldenryan commented Aug 4, 2025 • edited by ellipsis-dev Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

ellipsis-dev Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

jackaldenryan commented Aug 4, 2025 •

edited by ellipsis-dev Bot

Loading