Skip to content

feat(inbox): error tracking source, LLM evaluations, source config improvements#1405

Draft
Twixes wants to merge 1 commit intomainfrom
twixes/inbox-sources
Draft

feat(inbox): error tracking source, LLM evaluations, source config improvements#1405
Twixes wants to merge 1 commit intomainfrom
twixes/inbox-sources

Conversation

@Twixes
Copy link
Copy Markdown
Member

@Twixes Twixes commented Apr 1, 2026

Problem

Signal source config in Code is missing error tracking (which Cloud has with 3 sub-types), shows a non-functional LLM analytics toggle (backend hardcodes return True), and has re-render issues when toggling sources due to useAuthenticatedMutation state churn.

Changes

  • Add error tracking source with issue_created/issue_reopened/issue_spiking toggled together
  • Replace LLM analytics toggle with a list of evaluations fetched from the API, each individually toggleable – polls every 5s when visible, links to Cloud for full management
  • Drop useAuthenticatedMutation in favor of direct API calls with per-source optimistic state – fixes the frame drop on toggle
  • Change SignalSourceToggles from onChange(values) to onToggle(source, enabled) with React.memo cards
  • Add GitHub OAuth flow in DataSourceSetup when no integration exists (was a dead-end message)
  • Add SuggestedReviewer and SuggestedReviewersArtefact types for use in later PRs

…provements

- Add error tracking as signal source with 3 sub-types toggled together
- Replace non-functional LLM analytics toggle with evaluations list (polls 5s)
- Evaluations link to Cloud for management (region-aware)
- Fix re-render cascade: direct API calls, per-source optimistic state
- Per-source onToggle API with memoized cards
- Rounded toggle cards, GitHub OAuth flow
- Add suggested reviewer + artefact types
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