Add claude GitHub actions 1776435485247#172
Open
Abdellah-Haddad wants to merge 12 commits intohttps-deeplearning-ai:mainfrom
Open
Add claude GitHub actions 1776435485247#172Abdellah-Haddad wants to merge 12 commits intohttps-deeplearning-ai:mainfrom
Abdellah-Haddad wants to merge 12 commits intohttps-deeplearning-ai:mainfrom
Conversation
Includes running instructions, architecture overview, and rules for uv usage, memory/file access restrictions. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add get_course_outline() to VectorStore to fetch title, link, and lesson list from course_catalog - Add CourseOutlineTool class to search_tools.py with fuzzy course name matching - Register CourseOutlineTool in RAGSystem alongside CourseSearchTool - Update AIGenerator system prompt to route outline queries to the new tool Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Support up to 2 sequential tool-call rounds in AIGenerator (tool loop replaces single-shot _handle_tool_execution); adds synthesis fallback when round cap is hit - Guard VectorStore against ChromaDB ValueError on empty collections - Add pytest dev dependency and testpaths config in pyproject.toml - Add backend/tests/ with unit tests for AIGenerator, RAGSystem, and CourseSearchTool - Move "never start server" rule from CLAUDE.md to CLAUDE.local.md Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… refactor notes Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add Prettier with project-consistent config (single quotes, 2-space indent, trailing commas, printWidth 100) as the JS/CSS/HTML formatter - Add ESLint with browser env, enforcing ===, no-var, prefer-const, and no accidental globals; marks CDN `marked` as a known global - Apply Prettier formatting to script.js: normalise indentation to 2 spaces, add trailing commas in multi-line objects, parenthesise arrow function params, collapse double blank lines - Add scripts/check-frontend.sh to run both tools from the repo root; accepts --fix flag to auto-correct all issues - Add frontend/package.json with format, lint, and quality npm scripts - Document all changes and usage instructions in frontend-changes.md Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add test_app.py with 13 tests covering POST /api/query, GET /api/courses,
and DELETE /api/session/{id} — using an inline FastAPI test app to avoid
import-time side effects from StaticFiles and RAGSystem initialization
- Add mock_rag_system fixture and build_test_app factory to conftest.py so
API test infrastructure is shared and reusable
- Add addopts = ["-v", "--tb=short"] to pytest config for cleaner output
- Add httpx as a dev dependency (required by FastAPI TestClient)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add fixed theme toggle button (top-right) with sun/moon SVG icons - Add [data-theme="light"] CSS variable block with full light palette - Apply universal 0.25s transitions for smooth theme switching - Persist theme preference in localStorage; restore on load with no flash - Fix code block and source pill contrast for light mode - Document all changes in frontend-changes.md Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Combined with quality_feature's Prettier formatting — kept 2-space indentation and included theme toggle event listener from ui_feature. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.