Conversation
- Add SPA fallback in Fiber static handler so /web/* deep links resolve - Add react-router, @tanstack/react-query, axios; Vite dev proxy to :8080 - Add typed API client (envelope unwrap + error normalization) and query hooks - Replace state-switcher with react-router (basename /web) - Repositories screen: provider OAuth setup, connect/redirect, live repo list, disconnect - AI Connectors screen: full CRUD (list/create/edit/delete) - Drop repoRows/aiConnectors mocks (inline remaining overview placeholders)
miqdadyyy
changed the title
… and handlers Adds coverage for review/git/overview usecases, review & overview repositories, and HTTP/worker handlers.
✅ Coverage Report
|
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.
Overview
End-to-end refactor of the PR review platform: the legacy
vcs/aiconnector/prfetcher/repostack is replaced with a cleaner domain model built aroundgit,llm_provider,repository,review,setting, andoverview, plus a pluggable review-method engine. The web app is reworked to match the new API surface with a React Query data layer.Backend
Domain restructure
services/vcswithservices/git— GitHub, GitLab, and Bitbucket implementations behind anIGitService+IGitFactory, with shared base/diff/oauth helpers.git,llm_provider,repository,review,setting,overview.aiconnector,connectedrepo,aiprompttemplate,prfetcher, andrepo.Review engine
services/reviewwith pluggable review-method strategies behindIReviewMethodService+ factory:diff_only,file_by_file,two_pass,semantic_chunk, anddependency_aware.usecase/reviewfor the end-to-end PR review workflow, including review result items and repository-level review rules.Platform / infra
cmd/cli/runner/nats.go) and removed the unused cron handler.anthropic,openai) extended for the new review methods.config/production.yaml,.dockerignore, and updated the Dockerfile.git,llm_provider,overview,repository_rule,setting, and updatedreview.docs/superpowers/.Frontend
web/src/lib/queries/(connectors, overview, repos, review-methods, reviews, rules) plus sharedapi.tsandtypes.ts.Notes
make wire) for the new provider/repository/usecase sets.