Skip to content

feat: feedback APIs#48

Open
lovestaco wants to merge 3 commits into
masterfrom
feat/feedback
Open

feat: feedback APIs#48
lovestaco wants to merge 3 commits into
masterfrom
feat/feedback

Conversation

@lovestaco
Copy link
Copy Markdown

  • Add review_feedback table with vote_type, tags, comment_content,
    code_excerpt, severity, source_type, lrc_version, and retracted_at
  • Add FeedbackStore for insert with pq.Array tag handling
  • Add POST /api/v1/feedback, GET /impact-stats, PATCH /:id/retract
    under feedbackGroup protected by APIKeyAuthMiddleware
  • Validate review ownership on submit via reviews+users join so users
    can only vote on their own reviews
  • Retract requires INNER JOIN ownership check; floating feedback
    (no review_id) cannot be retracted since it has no user attribution
  • Validate user_id from context explicitly in both handlers to prevent
    silent zero-value fallthrough
  • Impact stats reads org_id from middleware context, not query param"

lovestaco added 3 commits May 21, 2026 21:37
LiveReview Pre-Commit Check: ran (iter:8, coverage:30%)
LiveReview Pre-Commit Check: skipped (iter:1, coverage:0%)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant