Migrate backend to FastAPI and fix e2e/script mismatches#10
Open
juma78 wants to merge 5 commits into
Open
Conversation
Replace the stdlib http.server handler with a real FastAPI app so backend/tests/test_chat.py can import it (was failing to collect). Align scripts/stop.sh's pkill pattern with the actual backend process, add sign-in flow coverage to the Playwright kanban specs, and check in CLAUDE.md. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Runs backend pytest and frontend lint/unit/e2e on push and PR, so PR ed-donner#10 (and future PRs) get automated checks instead of none. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Replace the text-only delete/add/logout/send buttons with icon-labeled ones, and rework the board layout so columns use available width (flex row with horizontal scroll) instead of being squeezed by a narrow grid alongside a proportional chat panel. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Extracts renameColumn/addCard/removeCard into lib/kanban.ts alongside the existing moveCard so board mutations are tested pure functions instead of duplicated inline reducer logic in KanbanBoard.tsx. Also dedupes icon SVG wrappers into IconBase, shares card content markup between KanbanCard and KanbanCardPreview, has ChatPanel import BoardData instead of redeclaring it, drops a no-op useMemo, extracts a signIn() test helper, and guards start.sh's venv/npm install so reruns skip redundant setup. Co-Authored-By: Claude Sonnet 5 <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.
Replaces the stdlib http.server handler with a real FastAPI app so backend/tests/test_chat.py can import it (was failing to collect). Aligns scripts/stop.sh's pkill pattern with the actual backend process, adds sign-in flow coverage to the Playwright kanban specs, and checks in CLAUDE.md.
Test plan
python -m pytest backend/tests/passesnpm run test:unitpassesnpm run test:e2epassesnpm run lintpasses