-
-
Notifications
You must be signed in to change notification settings - Fork 52
[tutor] Add AI-Powered Installation Tutor #566
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
Closed
srikrishnavansi
wants to merge
75
commits into
cortexlinux:main
from
srikrishnavansi:feature/ai-tutor
Closed
Changes from all commits
Commits
Show all changes
75 commits
Select commit
Hold shift + click to select a range
1fbed6a
[tutor] Add AI-Powered Installation Tutor (Issue #131)
srikrishnavansi e0dd20b
Merge branch 'main' into feature/ai-tutor
Anshgrover23 4b524e2
Merge branch 'main' into feature/ai-tutor
Anshgrover23 e8b3314
Merge branch 'main' into feature/ai-tutor
Anshgrover23 654370c
fix(tutor): address PR review comments from Gemini and CodeRabbit
srikrishnavansi bb8c491
Merge branch 'feature/ai-tutor' of https://github.com/srikrishnavansi…
srikrishnavansi aeef51c
Merge branch 'main' into feature/ai-tutor
srikrishnavansi 03c11ad
Merge branch 'main' into feature/ai-tutor
Anshgrover23 d95ee00
style(tutor): fix ruff lint errors
srikrishnavansi b8cedf0
Merge branch 'feature/ai-tutor' of https://github.com/srikrishnavansi…
srikrishnavansi a799d27
test(tutor): move tests to standard location for CI coverage
srikrishnavansi 049185d
fix(tutor): address CodeRabbit review comments
srikrishnavansi 4e86d28
fix(security): use tempfile.gettempdir() instead of hardcoded /tmp path
srikrishnavansi bce506a
fix: address SonarQube code quality issues
srikrishnavansi f4beddd
style: fix black formatting in cli.py
srikrishnavansi 29a7a8b
refactor: remove duplicate tests from cortex/tutor/tests
srikrishnavansi 4ff4eb6
fix: remaining floating point comparisons in test_integration.py
srikrishnavansi 2fab6a5
fix: address reviewer feedback and CodeRabbit suggestions
srikrishnavansi 79bc930
chore: remove unused imports (CodeRabbit nitpicks)
srikrishnavansi 92df93b
fix: correct return types and add input validation (CodeRabbit)
srikrishnavansi 24bd578
fix: refactor QAHandlerTool with lazy init and input validation
srikrishnavansi 5fe8a89
style: fix black formatting in qa_handler.py
srikrishnavansi 0d0b495
fix: resolve Pydantic forward reference and type annotation issues
srikrishnavansi 814018b
fix: add safe access pattern for result["answer"] in ConversationHandler
srikrishnavansi 49830cb
refactor: cleanup code quality issues from review
srikrishnavansi fff6646
Merge upstream/main: migrate deps to pyproject.toml
srikrishnavansi 2bf80bb
style: fix import sorting in cli.py (ruff I001)
srikrishnavansi b481abd
fix: address CodeRabbit review feedback
srikrishnavansi 8ce6277
refactor: remove LangChain/LangGraph, use existing llm_router
srikrishnavansi 0329f5b
fix: remove langchain imports from deterministic tools
srikrishnavansi 1b9c88a
fix: black formatting and add exception logging
srikrishnavansi 7f0b30d
fix: handle zero score correctly in progress tracker
srikrishnavansi a813fd9
Merge branch 'main' into feature/ai-tutor
srikrishnavansi b0a19c5
refactor: fix unused kwargs warnings and nested conditionals
srikrishnavansi 284f119
fix: allow explicit completed=False in update_progress
srikrishnavansi db6a101
docs: update AI_TUTOR.md to reflect simplified architecture
srikrishnavansi c382393
Merge branch 'main' into feature/ai-tutor
Anshgrover23 5f93fd7
Merge branch 'main' into feature/ai-tutor
srikrishnavansi ec37b80
Merge branch 'main' into feature/ai-tutor
srikrishnavansi bbe7029
test: add llm.py tests, update coverage docs
srikrishnavansi bc773d3
fix: use pytest.approx for floats, suppress config warning
srikrishnavansi 859cde2
Merge branch 'main' into feature/ai-tutor
srikrishnavansi f42f286
merge: sync with upstream/main and update docs
srikrishnavansi 6fe19cd
Merge branch 'feature/ai-tutor' of https://github.com/srikrishnavansi…
srikrishnavansi 1c08c7f
docs: fix placeholder URL in AI_TUTOR.md
srikrishnavansi 1e12ac3
docs: remove hardcoded model versions from AI_TUTOR.md
srikrishnavansi a96a303
docs: add TUTOR_OFFLINE to config section and fix placeholders
srikrishnavansi 67c9461
Merge branch 'main' into feature/ai-tutor
srikrishnavansi 82ebb5c
refactor: flatten tutor module structure and add layered prompts
srikrishnavansi a122bee
[autofix.ci] apply automated fixes
autofix-ci[bot] fa6a4c4
fix: use Pydantic models for structured LLM output parsing
srikrishnavansi fdb7478
fix: use pytest.approx for floating point comparison
srikrishnavansi 4f69c2b
refactor: use structured outputs and remove dead code
srikrishnavansi aa9531f
fix: use conlist for proper Pydantic list length validation
srikrishnavansi 1d41fca
fix: replace conlist with plain list types in contracts
srikrishnavansi 662d48f
docs: update AI_TUTOR.md to reflect structured outputs architecture
srikrishnavansi 9bd9f32
Merge branch 'main' into feature/ai-tutor
srikrishnavansi ab99dc2
Merge branch 'main' into feature/ai-tutor
Anshgrover23 5502b85
Merge branch 'main' into feature/ai-tutor
Anshgrover23 b525eb8
Merge branch 'main' into feature/ai-tutor
srikrishnavansi 808b1ff
refactor: address reviewer comments
srikrishnavansi 496c220
fix: resolve merge conflict - include both tutor and i18n packages
srikrishnavansi 1bea813
fix: merge upstream/main and resolve pyproject.toml conflict
srikrishnavansi 7d40a30
[autofix.ci] apply automated fixes
autofix-ci[bot] ced43c9
docs: clarify clear_cache docstring behavior
srikrishnavansi f0e8bc8
docs: add TUTOR_MODEL_NAME and TUTOR_CACHE_TTL_HOURS to configuration
srikrishnavansi a1ec904
Merge branch 'main' into feature/ai-tutor
Anshgrover23 78846db
fix: add exception logging to LessonLoaderTool and cache_lesson
srikrishnavansi 6dc0764
fix: address CodeRabbit review - truthiness check and logging
srikrishnavansi 933fd4d
Merge branch 'main' into feature/ai-tutor
srikrishnavansi 3d2eda4
Merge branch 'main' into feature/ai-tutor
Anshgrover23 470f7db
Merge branch 'main' into feature/ai-tutor
Anshgrover23 2fc3360
Merge branch 'main' into feature/ai-tutor
srikrishnavansi e1d8cbe
Merge branch 'main' into feature/ai-tutor
srikrishnavansi 169426c
Merge branch 'main' into feature/ai-tutor
Anshgrover23 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| """ | ||
| Intelligent Tutor - AI-Powered Installation Tutor for Cortex Linux. | ||
|
|
||
| An interactive AI tutor that teaches users about packages and best practices. | ||
| """ | ||
|
|
||
| from cortex.tutor import agent | ||
| from cortex.tutor.branding import console, tutor_print | ||
| from cortex.tutor.config import Config | ||
|
|
||
| __all__ = ["Config", "agent", "console", "tutor_print"] |
Oops, something went wrong.
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.
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.
Avoid unintended “reset all” when a package positional is supplied with
--reset.With
--resetusingnargs="?"andconst="__all__",cortex tutor docker --resetsetsreset="__all__"andpackage="docker", but the current logic ignores the positional and resets all. Consider honoring the positional package whenreset_target == "__all__"and a package is present (or require an explicit reset target).🛠️ Proposed fix
🤖 Prompt for AI Agents