Skip to content

feat(inbox): Source config updates (ET & LLMA)#1405

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

feat(inbox): Source config updates (ET & LLMA)#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), 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

Adding the error tracking source with issue_created/issue_reopened/issue_spiking toggled together.

Then, replacing the 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. Going to Cloud is quite clunky… but haven't yet focused on LLM evals here other than generally making them toggleable.

Also, adding a GitHub OAuth flow in DataSourceSetup when no integration exists (was a dead end flow previously).

TheSuggestedReviewer and SuggestedReviewersArtefact types are 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
@Twixes Twixes force-pushed the twixes/inbox-sources branch from ebdcc35 to 154149b Compare April 1, 2026 16:52
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.

2 participants