Cleanup content, demos, and README outline #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Pull request overview
This PR restructures the learning materials by removing the older “progressive_agents” + section README content, introducing a new demos/-based staged CLI/demo flow, and updating packaging/entrypoints and setup docs to match the new structure.
Changes:
- Remove legacy workshop/progressive_agents/notebooks section README content in favor of a new demos-first structure.
- Add
demos/staged agents (Stages 1–6) with CLIs, workflows, and supporting modules; wire them up as console scripts. - Update setup instructions and internal docstrings/comments to reference
workshop/+demos/instead ofprogressive_agents/.
Reviewed changes
Copilot reviewed 65 out of 119 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| workshop/README.md | Removed legacy workshop README content (cleanup/reorg). |
| tests/test_tools.py | Updates guidance comment to reference demos instead of progressive agents. |
| src/redis_context_course/tools.py | Updates module comments to reference workshop notebooks/demos. |
| src/redis_context_course/semantic_tool_selector.py | Updates “experimental” comment to reference workshop/demos. |
| src/redis_context_course/redis_config.py | Updates “experimental” comment to reference workshop/demos. |
| src/redis_context_course/optimization_helpers.py | Updates “experimental” comments to reference workshop/demos. |
| src/redis_context_course/hierarchical_manager.py | Updates “experimental” comment to reference workshop/demos. |
| src/redis_context_course/hierarchical_context.py | Updates “experimental” comments to reference workshop/demos. |
| src/redis_context_course/course_manager.py | Updates “experimental” comment to reference workshop/demos. |
| pyproject.toml | Adds console scripts for demo stages; packages demos into the wheel. |
| progressive_agents/stage5_working_memory/README.md | Removed legacy progressive_agents stage README. |
| progressive_agents/stage4_hybrid_search/README.md | Removed legacy progressive_agents stage README. |
| progressive_agents/README.md | Removed legacy progressive_agents top-level README. |
| notebooks/section-4-tools-and-agents/README.md | Removed legacy section README. |
| notebooks/section-3-memory-systems/README.md | Removed legacy section README. |
| notebooks/section-2-retrieved-context-engineering/README.md | Removed legacy section README. |
| notebooks/SETUP_GUIDE.md | Removed legacy notebooks setup guide. |
| demos/init.py | Introduces demos as a packaged Python module. |
| demos/CONTENT_COVERAGE_GAP_ANALYSIS.md | Updates references from progressive_agents → demos and README location. |
| demos/stage1_baseline_rag/init.py | Adds Stage 1 demo package marker. |
| demos/stage1_baseline_rag/cli.py | Updates imports/env handling to support packaged execution. |
| demos/stage1_baseline_rag/agent/init.py | Adds Stage 1 agent package exports. |
| demos/stage1_baseline_rag/agent/nodes.py | Adds Stage 1 baseline nodes (raw context). |
| demos/stage1_baseline_rag/agent/setup.py | Adds Stage 1 setup/helpers for loading/cleanup. |
| demos/stage1_baseline_rag/agent/state.py | Adds Stage 1 minimal state definition. |
| demos/stage1_baseline_rag/agent/workflow.py | Adds Stage 1 simple 2-node workflow. |
| demos/stage2_context_engineered/init.py | Adds Stage 2 demo package marker. |
| demos/stage2_context_engineered/cli.py | Updates imports/env handling to support packaged execution. |
| demos/stage2_context_engineered/agent/init.py | Adds Stage 2 agent package exports. |
| demos/stage2_context_engineered/agent/context_engineering.py | Adds Stage 2 context-engineering utilities. |
| demos/stage2_context_engineered/agent/nodes.py | Adds Stage 2 nodes (engineered context). |
| demos/stage2_context_engineered/agent/setup.py | Adds Stage 2 setup/helpers. |
| demos/stage2_context_engineered/agent/state.py | Adds Stage 2 minimal state definition. |
| demos/stage2_context_engineered/agent/workflow.py | Adds Stage 2 simple 2-node workflow. |
| demos/stage3_full_agent_without_memory/init.py | Adds Stage 3 demo package marker. |
| demos/stage3_full_agent_without_memory/cli.py | Updates imports to support packaged execution + adds run() entrypoint. |
| demos/stage3_full_agent_without_memory/debug_search.py | Fixes sys.path resolution for running from repo. |
| demos/stage3_full_agent_without_memory/test_simple.py | Updates imports/pathing to local agent module. |
| demos/stage3_full_agent_without_memory/test_linear_algebra.py | Updates imports/pathing to local agent module. |
| demos/stage3_full_agent_without_memory/agent/init.py | Adds Stage 3 agent exports. |
| demos/stage3_full_agent_without_memory/agent/edges.py | Adds Stage 3 routing helpers. |
| demos/stage3_full_agent_without_memory/agent/setup.py | Adds Stage 3 setup/helpers. |
| demos/stage3_full_agent_without_memory/agent/state.py | Adds Stage 3 state definition. |
| demos/stage3_full_agent_without_memory/agent/workflow.py | Adds Stage 3 LangGraph workflow. |
| demos/stage4_hybrid_search/init.py | Adds Stage 4 demo package marker. |
| demos/stage4_hybrid_search/cli.py | Updates imports to support packaged execution + adds run() entrypoint. |
| demos/stage4_hybrid_search/agent/init.py | Adds Stage 4 agent exports. |
| demos/stage4_hybrid_search/agent/react_agent.py | Adds Stage 4 ReAct loop implementation. |
| demos/stage4_hybrid_search/agent/react_parser.py | Adds Stage 4 ReAct parsing utilities. |
| demos/stage4_hybrid_search/agent/react_prompts.py | Adds Stage 4 ReAct prompt template. |
| demos/stage4_hybrid_search/agent/setup.py | Adds Stage 4 setup/helpers. |
| demos/stage4_hybrid_search/agent/state.py | Adds Stage 4 state definition. |
| demos/stage4_hybrid_search/agent/tools.py | Adds Stage 4 hybrid search tool w/ FilterQuery exact match. |
| demos/stage4_hybrid_search/agent/workflow.py | Adds Stage 4 workflow wrapper. |
| demos/stage5_working_memory/init.py | Adds Stage 5 demo package marker. |
| demos/stage5_working_memory/cli.py | Updates imports to support packaged execution + adds run() entrypoint. |
| demos/stage5_working_memory/debug_search.py | Fixes sys.path resolution for running from repo. |
| demos/stage5_working_memory/test_simple.py | Updates imports/pathing to local agent module. |
| demos/stage5_working_memory/test_agent_intents.py | Updates imports/pathing to local agent module. |
| demos/stage5_working_memory/test_exact_match.py | Adds a direct CourseManager “exact search” debug script. |
| demos/stage5_working_memory/test_exact_match_react.py | Updates imports to local agent module. |
| demos/stage5_working_memory/test_linear_algebra.py | Updates imports/pathing to local agent module. |
| demos/stage5_working_memory/test_react_simple.py | Updates imports to local agent module. |
| demos/stage5_working_memory/test_react_multi_turn.py | Updates imports to local agent module. |
| demos/stage5_working_memory/agent/init.py | Adds Stage 5 agent exports. |
| demos/stage5_working_memory/agent/edges.py | Adds Stage 5 routing helpers. |
| demos/stage5_working_memory/agent/react_agent.py | Adds Stage 5 ReAct agent implementation (working memory). |
| demos/stage5_working_memory/agent/react_parser.py | Adds Stage 5 ReAct parsing utilities. |
| demos/stage5_working_memory/agent/react_prompts.py | Adds Stage 5 ReAct prompt template. |
| demos/stage5_working_memory/agent/setup.py | Adds Stage 5 setup/helpers (CourseManager + memory client). |
| demos/stage5_working_memory/agent/state.py | Adds Stage 5 state definition (working memory + ReAct fields). |
| demos/stage5_working_memory/agent/workflow.py | Adds Stage 5 workflow wrapper (memory + ReAct). |
| demos/stage6_full_memory/init.py | Adds Stage 6 demo package marker. |
| demos/stage6_full_memory/cli.py | Updates imports to support packaged execution + adds run() entrypoint. |
| demos/stage6_full_memory/debug_search.py | Fixes sys.path resolution for running from repo. |
| demos/stage6_full_memory/test_simple.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_simple_memory.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_agent_intents.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_exact_match.py | Updates imports/pathing and stage naming. |
| demos/stage6_full_memory/test_linear_algebra.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_memory_tools.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_react_simple.py | Updates imports/pathing and stage naming. |
| demos/stage6_full_memory/test_react_multi_turn.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_react_linear_algebra.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/test_react_intents.py | Updates imports/pathing to local agent module. |
| demos/stage6_full_memory/agent/init.py | Adds Stage 6 agent exports (but docstring needs correction). |
| demos/stage6_full_memory/agent/edges.py | Adds Stage 6 routing helpers. |
| demos/stage6_full_memory/agent/react_agent.py | Adds Stage 6 ReAct agent node implementation. |
| demos/stage6_full_memory/agent/react_parser.py | Adds Stage 6 ReAct parsing utilities. |
| demos/stage6_full_memory/agent/setup.py | Adds Stage 6 setup/helpers (CourseManager + memory client). |
| demos/stage6_full_memory/agent/state.py | Adds Stage 6 state definition (but docstrings need correction). |
| demos/stage6_full_memory/agent/tools.py | Updates a stale path comment. |
| demos/stage6_full_memory/agent/workflow.py | Adds Stage 6 workflow wrapper (but sync runner has event-loop bug). |
| SETUP.md | Updates setup guide to reference workshop + demos and new entrypoint usage. |
Comments suppressed due to low confidence (7)
demos/stage6_full_memory/test_exact_match.py:12
- Two runtime issues introduced around these imports:
run_react_agentis imported, butdemos/stage6_full_memory/agent/react_agent.pydefinesreact_agent_node(norun_react_agenthelper), so the import will fail.- This test also calls
initialize_tools(..., user_id=...)later, butdemos/stage6_full_memory/agent/tools.pydefinesinitialize_tools(manager: CourseManager)with nouser_idparameter, which will raiseTypeError.
Align Stage 6’s public API with the Stage 5 test helper signatures (addrun_react_agent+ acceptuser_id, or update tests to use the existing Stage 6 workflow/tool APIs).
demos/stage3_full_agent_without_memory/cli.py:36
- Import of 'run_agent' is not used.
demos/stage3_full_agent_without_memory/cli.py:39 - Import of 'run_agent' is not used.
demos/stage2_context_engineered/cli.py:50 - Import of 'create_workflow' is not used.
demos/stage2_context_engineered/cli.py:53 - Import of 'create_workflow' is not used.
demos/stage1_baseline_rag/cli.py:50 - Import of 'create_workflow' is not used.
demos/stage1_baseline_rag/cli.py:53 - Import of 'create_workflow' is not used.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Initial pass at reorganizing content, removing old things, and cleaning up demo scripts and README polish.