Repository navigation
fix(self-driving): enable Logs and Endpoints signal sources in setup #412
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -35,6 +35,8 @@ Reach the source-config tools through the PostHog `exec` tool — `info` then `c | |
| | Scout gate | **On by default — never create a row.** The server lets scout findings into the inbox with no config row; a `signals_scout` / `cross_source_issue` row only exists to opt OUT. If you find one with `enabled: false` (an earlier opt-out), flip it back on with `inbox-source-configs-partial-update`; with no row, do nothing and record "on by default" | `signals_scout` / `cross_source_issue` — existing disabled row only | | ||
| | Health checks | **Always** — instrumentation issues (missing events, proxy gaps, outdated SDKs) are always actionable and a good thing for the agent to fix | `health_checks` / `health_issue` | | ||
| | Error tracking | **Enable by default**, even with no current signal — teams adopt error tracking sooner or later, and with no errors there are no findings and no cost. Evidence (report, exception autocapture ON, or error issues from the step-2 probe) only raises confidence; its absence is **not** a reason to skip | **All three rows**: `error_tracking` / `issue_created`, `error_tracking` / `issue_reopened`, `error_tracking` / `issue_spiking` — the product UI treats them as one switch | | ||
| | Logs | **Only when the project uses Logs**, per your step-2 checklist. The pair carries alerts that fire and alerts the server auto-disables, so a project with no logs never produces one | `logs` / `alert_state_change` | | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When Logs ranks among a project's most-used products, this enables the native responder while Useful? React with 👍 / 👎. |
||
| | Endpoints | **Only when the project has endpoints**, per the step-2 `endpoints-get-all` probe. The pair carries failed executions and breakdown-limit overflows | **Both rows**: `endpoints` / `endpoint_execution_failed` and `endpoints` / `endpoint_breakdown_limit_exceeded` | | ||
| | Support | **Enable by default** — step 3 turned the Conversations product ON, so wire its source. It stays idle until an inbound channel (email / inbox / Slack) is connected, so record that channel connection as a follow-up — but enabling the source now means tickets reach the inbox automatically once a channel exists, with no second setup. Don't gate on profile evidence. | `conversations` / `ticket` | | ||
|
|
||
| ## Nothing to create here | ||
|
|
@@ -46,7 +48,7 @@ The rest of the pairs have no row for this run to write. Each one's coverage liv | |
| | `session_replay` / `session_analysis_cluster` | Step 6c's Replay Vision scanners. This pair is **retired** — the session summarization feature behind it is gone, PostHog deleted the existing rows, and the server now skips the pair when it checks whether a team has a working source, so a row written here is a dead switch in the user's inbox. | | ||
| | `replay_vision` | The scanners themselves. Replay Vision is **self-authorizing**: `emits_signals` on each scanner *is* its per-source config, so step 6c's writes are the whole story. | | ||
| | `llm_analytics` | Nowhere — it's internal, with no user-facing responder behind it. | | ||
| | `logs`, plus any `source_type` of `evaluation` or `alert_state_change` | Nowhere yet — no v1 responder. | | ||
| | `source_type` of `evaluation` | Nowhere — no emitter produces this type any more. The taxonomy keeps the value so older signals still resolve to a label. | | ||
| | `github`, `linear`, `zendesk`, `pganalyze`, `jira`, `google_search_console`, … | Step 5, on the user's answer, once each tool's warehouse source exists. | | ||
|
|
||
| Record every source decision with its reason — the report needs them. | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On a newly integrated project where Logs was successfully configured,
context/skills/integration-v2/report/description.mdexplicitly requires that outcome inposthog-setup-report.md, but step 1 limits report-derived facts to events, error tracking, and feature flags while this line relies only on the profile. Because a profile 404 is explicitly expected on first-run teams, Logs becomesunknownand step 4 skips the new source row, continuing to drop Logs findings despite the report already proving usage; treat a successful Logs result in the setup report as affirmative evidence before falling back to the profile.Useful? React with 👍 / 👎.