@bastani/atomic 0.9.16-alpha.1 → 0.9.16-alpha.3

package/dist/builtin/workflows/skills/why/references/source-playbook.md

@@ -0,0 +1,17 @@
+# Source playbooks
+
+The why skill spawns one investigator per available evidence category, each reading a single source-specific playbook below. The playbooks are concrete examples for common MCPs; adapt them for a different MCP in the same category.
+
+| Category | Playbook | Example MCP it documents |
+|---|---|---|
+| Source control history | [`code-archaeology.md`](./sources/code-archaeology.md) | git, `gh` |
+| Issue / ticket tracker | [`linear.md`](./sources/linear.md) | Linear (adapt for Jira, GitHub Issues, Plane, Shortcut) |
+| Long-form documents | [`notion.md`](./sources/notion.md) | Notion (adapt for Confluence, Google Docs, Coda) |
+| Real-time team chat | [`slack.md`](./sources/slack.md) | Slack (adapt for Discord, Microsoft Teams, Mattermost) |
+| Infrastructure observability | [`datadog.md`](./sources/datadog.md) | Datadog (adapt for New Relic, Honeycomb, Grafana, Splunk) |
+| Error / exception tracking | [`sentry.md`](./sources/sentry.md) | Sentry (adapt for Rollbar, Bugsnag, Airbrake) |
+| Product analytics warehouse | [`databricks.md`](./sources/databricks.md) | Databricks SQL (adapt for Snowflake, BigQuery, ClickHouse, dbt) |
+
+Cross-cutting:
+
+- [`incident-postmortem.md`](./sources/incident-postmortem.md). Add this if the target code looks defensive (null checks, retry, timeout, rate limit, feature flag, egress guard, OOM handler).

package/dist/builtin/workflows/skills/why/references/sources/code-archaeology.md

@@ -0,0 +1,88 @@
+# Code Archaeology (git + in-repo)
+
+## What this source contains
+
+- Commit history (messages, dates, authors, diffs)
+- PR descriptions, review comments, and discussion threads (via `gh`)
+- Inline code comments, TODOs, FIXMEs, deprecation notes
+- ADRs (architectural decision records) if the repo keeps them
+- Tests. Names and assertions often encode the edge cases that motivated a change
+- Related files modified in the same commits (co-change signal)
+- CHANGELOG entries, release notes in the repo
+- Issue/ticket IDs mentioned in commit messages and PR bodies
+
+The most trustworthy source, tied directly to the code, and the most complete. Everything that went through the repo should be here.
+
+## How to search it
+
+Expand the seed commit list:
+
+```bash
+# Full history of the file through renames
+git log --follow --oneline -- <file>
+
+# Pickaxe: commits that added or removed this exact text
+git log -S '<exact_string_from_code>' -- <file>
+
+# Or for patterns:
+git log -G '<regex>' -- <file>
+
+# Who wrote each line and when
+git blame -L <start>,<end> <file>
+
+# The full diff of a specific commit
+git show <hash>
+
+# Commits between two points affecting this file
+git log <old>..<new> -p -- <file>
+```
+
+For each substantive commit, pull the PR context:
+
+```bash
+# Find the PR number from the merge commit or branch
+git log -1 --format=%B <hash>
+
+# Full PR context: body, review comments, linked issues
+gh pr view <number> --json title,body,author,createdAt,mergedAt,labels,closingIssuesReferences,comments,reviews,files
+
+# The --json reviews and comments fields are where the real signal is
+```
+
+Look for out-of-band docs:
+
+```bash
+# ADRs often live in docs/adr/ or similar
+rg -l -i 'architecture.decision' --glob '*.md'
+
+# TODOs and FIXMEs near the target
+rg -n -C2 '(TODO|FIXME|HACK|XXX|NOTE)' <target_file>
+
+# Related tests. Names often encode the "why"
+rg -l '<symbol>' --glob '*test*'
+```
+
+## What good evidence looks like here
+
+- A PR description that explains the problem being solved, not just the change ("This fixes the pagination bug that caused X")
+- A long review thread where alternatives were debated
+- An inline comment near the target line that explains a non-obvious constraint
+- A test named `test_handles_edge_case_when_X` that reveals an edge case motivating the code
+- A commit message that references a ticket or incident ID
+- A CHANGELOG entry that summarizes the user-visible rationale
+
+## Common pitfalls
+
+- **Squash-merge flatlands.** If the repo squashes PRs, individual commits in the branch history are lost. Fall back to PR body and comments.
+- **Misleading commit messages.** "Small refactor" sometimes hides an intentional behavior change. Look at the diff, not the message.
+- **Cargo-culted patterns.** The author may have copied a pattern without understanding why. Check if the pattern originated earlier in the codebase and investigate *that* commit.
+- **Bot commits and auto-merges.** Dependabot, Renovate, and automated backports usually don't carry motivation. Skip them when trying to find intent.
+- **Treating code as evidence of intent.** The code itself isn't evidence for why it exists. Evidence comes from commit messages, PRs, comments, tests, docs. Don't cite "the function is named X" as evidence of intent.
+
+## What to return
+
+Every commit/PR/comment that bears on the question, with:
+- The exact text (quoted)
+- The hash / PR number / file:line
+- Author and date
+- Whether it's direct (explicitly addresses the question) or circumstantial

package/dist/builtin/workflows/skills/why/references/sources/databricks.md

@@ -0,0 +1,70 @@
+# Databricks Analytics & System Tables
+
+## What this source contains
+
+Databricks is the product-analytics, data-pipeline, and warehouse-telemetry layer. It complements Datadog: Datadog is the *infra/runtime* view, Databricks is the *product/data* view (what users did, which experiments ran, how feature usage evolved, where a threshold constant came from).
+
+- **Product analytics events.** `your_warehouse.events.analytics_track_event` (raw) and typed, deduplicated per-event dbt models in `<your_analytics_db>.<schema>.<table>`. User behavior: feature invocations, clicks, accepts/rejects, submissions, client-reported errors.
+- **Usage & billing events.** `your_warehouse.events.usage_event` / `<your_analytics_db>.<schema>.stg_usage_events`; `your_warehouse.events.raw_model_event` / `<your_analytics_db>.<schema>.stg_raw_model_events`. For cost- or volume-driven decisions.
+- **Experiment / feature-flag data.** Exposure and outcome tables. **Schema is company-specific.** Probe with `SHOW TABLES` before assuming names.
+- **System tables.** `system.query.history`, `system.compute.warehouses`, `system.billing.*`, `system.access.audit`. Answer "was this query expensive?", "how often did anyone run this?", "when did warehouse load spike?"
+- **dbt lineage.** Models in `<your_analytics_db>.<schema>` reveal what pipelines depend on a table/field; upstream changes frequently motivate consumer-code changes.
+- **Databricks notebooks.** Exploratory analyses engineers wrote before code changes. **Not queryable via the SQL MCP.** If you suspect the rationale lives in a notebook, name it as a gap.
+
+## How to search it
+
+Use the Databricks SQL MCP. Primary tool: `execute_sql_read_only`. If it returns a `statement_id`, poll with `poll_sql_result` rather than re-running.
+
+**Orient before querying.** Schemas are company-specific; probe before trusting a table name:
+
+```sql
+SHOW TABLES IN <your_analytics_db>.<schema> LIKE '*<keyword>*';
+DESCRIBE TABLE <your_analytics_db>.<schema>.stg_<event>;
+```
+
+**Time-bound every query.** These tables are huge and unconstrained scans time out. Filter on `_timestamp` (events) or `start_time` (`system.query.history`) with a window bracketing the ship date, typically ~30 days before and after, wider only for strong reason.
+
+**Prefer typed dbt models over the raw table.** `<your_analytics_db>.<schema>.<table>` is deduplicated, typed, and liquid-clustered; `your_warehouse.events.analytics_track_event` has duplicates and untyped `properties_json`. Model-name pattern: `stg_<source>_<event_name_with_underscores>`, where `<source>` is `app`, `backend`, `website`, or `cli`; confirm the exact model name with `SHOW TABLES` when the pattern alone doesn't resolve it. Drop to the raw table only when there's no dbt model yet, or you need events from inside the dbt refresh lag.
+
+**Column conventions on the typed dbt models** (knowing these avoids a `DESCRIBE` round-trip):
+
+- `_timestamp`, `_id`, `_auth_id`, `_request_id`, `event_name`. Standard on every model
+- `properties_<name>`. Typed, underscore-cased event properties (`properties_entrypoint`, `properties_size_bytes`, …)
+- `context_team_id`, `context_client_version`, `context_country`, `context_client_os`. Pre-extracted client context
+
+### Investigation patterns that tend to pay off
+
+Pick the table + column combination that matches the target:
+
+1. **Event usage trajectory.** Daily counts on the relevant `stg_*` model across a ±30d window around the PR merge. A step function from zero to steady volume within a day or two of the merge is strong circumstantial evidence the PR launched the feature. A decay to zero suggests a deprecation or deletion.
+2. **Guard-rail / defensive-check origin.** Distribution (median / p99 / max) of the relevant `properties_<name>` column in the 14 days *before* the PR. A p99 that matches the target's threshold constant suggests the number was chosen from data.
+3. **Experiment / feature-flag lookup.** `SHOW TABLES ... LIKE '*experiment*'` to find the exposure table, then pull exposure counts by variant for the relevant flag key near the PR date.
+4. **Query-history evidence for migrations, backfills, or perf rewrites.** `system.query.history` filtered by `statement_text ILIKE '%<table_or_symbol>%'` with a tight `start_time` window surfaces the expensive queries that likely motivated the change (sort by `total_duration_ms` or aggregate `SUM(read_bytes)`, `COUNT(*)`).
+5. **dbt lineage.** If the target reads from or writes into a `<your_analytics_db>.<schema>` model, the model's own git history (in this repo) often carries the rationale. Hand that lead back to the git investigator rather than chasing it yourself.
+
+## What good evidence looks like here
+
+Beyond the pattern shapes above:
+
+- An error-classifying event's count drops to near zero in the days after a defensive-code PR. Suggests the PR resolved that error class
+- An exposure table row names the target's feature-flag key with a "shipped" / "concluded" decision around the PR ship date
+
+## Common pitfalls
+
+- **Instrumented ≠ caused.** An event's existence means someone cared enough to log it, not that the target code exists *because* of it. Pair with a PR/commit citation from the git investigator before claiming causation.
+- **Silent instrumentation changes.** A step function in event volume may mean a new event started being logged, not that user behavior changed. Check for instrumentation PRs in the same window before reading the ramp as a feature-launch signal.
+- **Schema drift.** Event properties evolve; a column on the typed dbt model today may not have existed when the target was written. Older data may carry the property only inside raw `properties_json`.
+- **dbt refresh lag.** `<your_analytics_db>.<schema>.*` is rebuilt on a schedule (often hourly/daily). For events from the last few hours, fall back to `your_warehouse.events.*` and deduplicate by `_id`.
+- **Company-specific tables.** Experiment, feature-flag, billing, and usage tables vary. Reporting a result from a table whose existence you never confirmed is a classic failure mode. Probe with `SHOW TABLES` / `DESCRIBE TABLE` first.
+- **Retention cliff.** If the relevant window predates the table's retention or the dbt model's creation date, that's a *gap*, not a null result. Name it explicitly so the synthesizer doesn't read "no results" as "no activity."
+- **Notebooks aren't queryable.** The SQL MCP can't see Databricks notebooks. If you suspect the rationale lives in one, return a gap.
+
+## What to return
+
+For each relevant finding:
+- Type (product event / experiment exposure / usage or billing event / system-table row / dbt model)
+- Fully-qualified table name and the exact query you ran
+- Time window queried
+- Compact numeric summary (counts, percentiles, first/last-seen timestamps). **Don't dump raw rows.**
+- Temporal correlation with the target's ship date (e.g., "first row 2024-08-15; PR #49074 merged 2024-08-14")
+- Relevance + strength: direct / circumstantial / weak

package/dist/builtin/workflows/skills/why/references/sources/datadog.md

@@ -0,0 +1,99 @@
+# Datadog Telemetry
+
+## What this source contains
+
+Datadog holds the runtime record: what actually happened in production, as opposed to what was planned or discussed.
+
+- **Metrics.** Counters, gauges, histograms instrumented by the team. A metric's *presence* is itself evidence: someone thought this number worth watching.
+- **Monitors & alerts.** Conditions the team decided warranted waking someone up. A monitor firing on `rate_limit_hit > 10/min` is direct evidence the team worried about that threshold.
+- **Dashboards.** Curated views. The charts tell you what the team considers important for a subsystem.
+- **APM traces & spans.** Request-level runtime data. Useful for "why is this slow" / "why is there a timeout here" questions.
+- **Logs.** High-volume event records. Often contain the error conditions that motivated defensive code.
+- **Incidents.** Formal incident records with timelines and linked postmortems.
+- **Notebooks.** Exploratory investigations; often contain hypotheses and analyses.
+
+Datadog answers "what was the production reality around the time this code was written?", which often explains the code's shape.
+
+## How to search it
+
+Use the Datadog MCP. Start broad, then narrow.
+
+1. **Identify the owning service(s).**
+
+   ```
+   search_datadog_services (filter by name or team)
+   search_datadog_service_dependencies (see upstream/downstream)
+   ```
+
+2. **Dashboards and monitors first. They tell you what the team cares about.**
+
+   ```
+   search_datadog_dashboards (query: feature name, service name, symbol)
+   search_datadog_monitors   (same queries)
+   ```
+
+   When a dashboard or monitor covers the target, note its queries and watched thresholds. The threshold is frequently the answer to "why is this clamped at N?"
+
+3. **Metrics around the target.**
+
+   ```
+   search_datadog_metrics (by name pattern, e.g., the feature or symbol)
+   get_datadog_metric_context (metadata: description, units, tags)
+   get_datadog_metric (timeseries; "was there a spike around the PR date?")
+   ```
+
+   Correlating a metric's trajectory with the target's add/change date is strong supporting evidence: "the `payment_timeout` metric spiked 2023-11-03, and the retry logic merged 2023-11-06."
+
+4. **Logs. Narrow, don't dump.**
+
+   ```
+   search_datadog_logs (raw log patterns near the target, set use_log_patterns=true)
+   analyze_datadog_logs (SQL-style aggregations, only when you need counts)
+   ```
+
+   Search with symbols, error strings, or feature names. **Strongly prefer time-bounded queries** (e.g., 30 days before/after the change). Log volume is huge; unconstrained searches waste time and may time out.
+
+5. **APM spans and traces.**
+
+   ```
+   aggregate_spans    (stats: "how often does this endpoint fail?")
+   search_datadog_spans (inspect individual spans)
+   get_datadog_trace  (a specific trace ID)
+   ```
+
+   Useful for timeouts, retries, slow paths, and cross-service behavior.
+
+6. **Incidents.**
+
+   ```
+   search_datadog_incidents (by title, team, date range)
+   get_datadog_incident     (full detail for a specific incident)
+   ```
+
+   If the target looks defensive, search for incidents around the time it was added. An incident whose timeline includes "added defensive check for X" is near-direct evidence.
+
+## What good evidence looks like here
+
+- A monitor whose query and threshold match the constraint the code enforces (code clamps to 100; monitor alerts when requests exceed 100/min)
+- A dashboard created by the target's author, with widgets that correspond to what the code measures or guards against
+- A metric showing a production spike immediately before the code was merged, and stable values after
+- An incident record referencing the target code, the same symbols, or the same error strings
+- Logs showing a specific error pattern the defensive code would prevent, timestamped in the window before the change
+
+## Common pitfalls
+
+- **Correlation is not causation.** A spike before a PR and stabilization after is suggestive, not definitive. Other changes may have landed in the same window. Check neighboring PRs.
+- **Overfitting to the chart you found.** Datadog visualizations are *made* by humans and reflect that human's framing. A chart named "retry success rate" is evidence the team cared about retry success, not that it's why a specific line of code exists.
+- **Vanished telemetry.** Metrics can be renamed, deleted, or have short retention. If you can't find data from the relevant window, that's a gap, not a null result.
+- **Noise at scale.** Searching logs for a common string returns thousands of matches. Narrow by service, tag, and time aggressively. Use `analyze_datadog_logs` to aggregate rather than dumping raw logs.
+- **Instrumented != caused.** A metric's existence tells you someone cared enough to measure something, not that the code was added *because* of it. Cross-reference with commit/PR dates.
+
+## What to return
+
+For each relevant item:
+- Type (dashboard / monitor / metric / log pattern / trace / incident / notebook)
+- Title or name
+- Link or identifier (dashboard ID, monitor ID, metric name, incident ID)
+- Owner/author and created/modified date
+- The specific condition, query, or quote that bears on the question (verbatim where possible)
+- Relevance: what this suggests about the target code, and how strong the connection is

package/dist/builtin/workflows/skills/why/references/sources/incident-postmortem.md

@@ -0,0 +1,15 @@
+# Incident & Postmortem Context
+
+Not a separate source, a **cross-cutting angle**. Incidents often motivate defensive code ("we added this check after the X outage"), so if the target looks defensive (null checks, retry logic, timeout handling, rate limiting, feature flags), specifically hunt for incident history across every available source:
+
+- **Notion**: search for postmortems mentioning the target file, feature, or error string
+- **Linear**: look for tickets labeled `incident`, `sev-*`, `postmortem-action-item`, `reliability`
+- **Slack**: search `#sev-*` and `#incident-*` channels around the dates the target code was added
+- **Git**: commits with messages like "fix for incident", "add defensive check", "revert" followed by "re-apply with..." are strong signals
+- **Datadog**: `search_datadog_incidents` for formal incident records with timelines; dashboards and monitors created as postmortem action items
+- **Sentry**: issues whose first-seen/last-seen window aligns with the target's PR ship date; stack traces through the target
+- **Databricks**: product-analytics events that classify an error condition (client-reported failures, user-visible retry events, etc.) often spike during an incident window. A drop in that event count after the target PR ships is circumstantial support that the target code resolved the user-visible symptom, even when Datadog/Sentry signal is noisy.
+
+If you find an incident link, fetch the full postmortem. Postmortems typically have an "Action Items" section that ties directly to code changes. When multiple sources corroborate (a Datadog incident ID appears in a Linear ticket, which appears in a Notion postmortem, which appears in a Slack thread that links to the target PR, and the Databricks error-event count drops after the fix), the evidence is especially strong.
+
+Worth spending time on when the code's defensive character makes an incident-driven origin plausible. Skip it for code that doesn't look defensive.

package/dist/builtin/workflows/skills/why/references/sources/linear.md

@@ -0,0 +1,48 @@
+# Linear Tickets
+
+## What this source contains
+
+- Issues describing features, bugs, and their motivation
+- Project docs attached to issues (often PRDs or specs)
+- Parent/sub-issue relationships (broader initiative → specific tickets)
+- Comments on issues (clarifications, scope changes, "why we're doing this" rationale)
+- Labels (e.g., `compliance`, `customer-request`, `perf`) that signal the type of motivation
+- Status updates that explain scope changes
+- Attachments and linked GitHub PRs
+
+Linear is where the product/business context often lives: the "we're doing this because customer X asked" or "this is for the Q3 compliance initiative" layer.
+
+## How to search it
+
+Use the Linear MCP.
+
+1. **Start with linked tickets.** If the seed commits or PRs reference ticket IDs (e.g., `ENG-1234`, `[BUG-567]`), fetch those first with `get_issue`. Read the full issue including comments.
+2. **List related issues by keyword.** Use `list_issues` with text search for the feature name, key symbol, or business term. Try multiple phrasings.
+3. **Walk the issue tree.** If you land on a sub-issue, fetch its parent. Sub-issues are tactical; parents often carry the "why."
+4. **Read project docs.** If the issue belongs to a project, use `get_project` and check attached docs. Project-level documents are where specs and rationale are most often captured.
+5. **Check labels and milestones.** Labels hint at the category of motivation (customer-request, incident-followup, compliance). Milestones tie work to deadlines, which often reveal motivation.
+
+## What good evidence looks like here
+
+- An issue description stating the business problem: "Customer Acme needs X because of their SOC2 audit"
+- A comment recording a decision: "We decided to go with approach B because approach A would require touching the billing service"
+- A parent issue titled like an initiative: "Q3 Enterprise Readiness" or "Reduce Payment Failures"
+- An attached PRD or spec
+- Labels like `customer:acme`, `incident-followup`, `compliance`, `perf-regression`
+
+## Common pitfalls
+
+- **Scope drift.** The ticket the PR references may have been closed and reopened with a different scope. Read the whole history.
+- **Mechanical templates.** Some teams require "Why" sections but fill them with boilerplate. Generic text ("improve user experience") is probably not a real answer.
+- **Stale tickets.** Old tickets often reflect a version of the plan that changed. Check dates and cross-reference with the code's ship date.
+- **Closed-as-duplicate chains.** Follow the duplicate-of relationships back to the canonical ticket.
+- **Private workspace content.** If you can't access an issue, note that as a gap rather than guessing.
+
+## What to return
+
+For each relevant ticket:
+- Ticket ID and title
+- The problem/motivation quoted from the description or comments (not paraphrased; the synthesizer needs the exact text to cite)
+- Labels, parent issue, project
+- Author, created date, closed date
+- Link to the ticket if available

package/dist/builtin/workflows/skills/why/references/sources/notion.md

@@ -0,0 +1,55 @@
+# Notion Docs
+
+## What this source contains
+
+- PRDs (product requirement documents)
+- Technical specs and RFCs
+- Architectural decision records (ADRs)
+- Meeting notes from design reviews
+- Team pages with domain context
+- Postmortems from incidents
+- Runbooks that may explain defensive code
+- Strategy documents that set priorities
+
+Notion is where "why" often lives in long-form before it becomes code. A significant feature usually has a doc.
+
+## How to search it
+
+Use the Notion MCP.
+
+1. **Keyword searches with `notion-search`.** Try:
+   - The feature name
+   - Key symbols / class names from the target code
+   - Author handles (design docs are often authored before the code lands)
+   - Error strings or user-visible terms
+   - Time-bounded queries if you know when the code shipped
+2. **Fetch candidate pages with `notion-fetch`.** Read the full content, not the preview; rationale is often buried mid-document.
+3. **Follow backlinks and child pages.** Design docs often have sub-pages for alternatives considered, appendices, or implementation notes.
+4. **Check related databases.** `notion-query-data-sources` and `notion-query-meeting-notes` can surface meeting notes that discussed the decision.
+5. **Search author-specific spaces.** If the PR author has a personal notebook (common at some companies), it may hold exploratory thinking that preceded the code.
+
+## What good evidence looks like here
+
+- A PRD with a "Problem statement" or "Motivation" section that matches the target code's purpose
+- An "Alternatives considered" or "Rejected approaches" section
+- A postmortem that names the target code as the fix for a specific incident
+- Meeting notes that record "we decided X because Y" and tie to the same author/date range as the PR
+- An ADR template filled out non-trivially (status, context, decision, consequences)
+
+## Common pitfalls
+
+- **Outdated docs.** Specs are often written before implementation and not updated; the doc may describe a plan that changed. Cross-check against the actual PR.
+- **Doc vs. reality drift.** A spec may say "we'll do X" but the code actually does Y. Flag the divergence; the synthesizer will surface the contradiction.
+- **Boilerplate templates.** Some orgs require a "Why" section that gets filled with fluff. Look for specificity.
+- **Unlinked docs.** The most relevant doc may not be linked from anywhere. Broad keyword searches help.
+- **Multiple drafts.** If a topic has multiple docs, find the one that was finalized or most recently updated. Check dates.
+- **Access-restricted pages.** If you can't access a page, note it as a gap.
+
+## What to return
+
+For each relevant doc:
+- Title and URL
+- Authors and last-updated date
+- The motivation text (verbatim quote), with page/section location
+- Relevant linked pages (so the synthesizer can cite them)
+- Whether the doc was finalized or draft

package/dist/builtin/workflows/skills/why/references/sources/sentry.md

@@ -0,0 +1,100 @@
+# Sentry Error History
+
+## What this source contains
+
+Sentry is the archive of things that went wrong. For defensive, corrective, or error-handling code, it often holds the direct motivation: the specific exceptions, stack traces, and frequencies that pushed someone to add a check, catch, retry, or fallback.
+
+- **Issues.** Grouped errors with counts, first/last seen timestamps, affected releases, and comments
+- **Events.** Individual error instances within an issue (stack traces, tags, user context)
+- **Releases.** Deployment records with associated issues (useful for "which version fixed this?")
+- **Replays.** Session recordings of user-facing errors (if enabled)
+- **Profiles.** Performance profiling data (less useful for "why"; more for "how slow")
+- **Issue comments & assignments.** Sometimes contain engineer notes on root cause
+
+The most valuable thing Sentry provides is **temporal correlation**: "issue X was created 2024-01-02, peaked at 500 events/day, stopped appearing after release v2.14.0 on 2024-01-15, the release that shipped the defensive check."
+
+## How to search it
+
+Use the Sentry MCP.
+
+1. **Orient.** If you don't know the project slug and organization:
+
+   ```
+   find_organizations
+   find_projects
+   ```
+
+2. **Search for issues related to the target.**
+
+   ```
+   search_issues (natural language, e.g., "errors in PaymentService timeout", "unhandled exceptions in uploadFile")
+   ```
+
+   Good query components: exception class names the target handles, the function or class name of the target, error message strings the target checks for, the file path of the target.
+
+3. **Narrow by release and time window.**
+
+   ```
+   search_issue_events (filter by release, time, environment, trace ID, tags)
+   get_issue_tag_values (for an issue, see distribution across versions, users, environments)
+   ```
+
+   For a suspected issue, check:
+   - **First seen.** When did the error start appearing?
+   - **Last seen.** When did it stop? Does it line up with the target's ship date?
+   - **Affected releases.** Which versions saw it? Which was the fix?
+   - **Frequency trajectory.** Did it spike, then get resolved?
+
+4. **Pull the full event for context.**
+
+   ```
+   get_sentry_resource (pass a Sentry URL or type+ID)
+   ```
+
+   Does the stack trace pass through the target code? Do the tags and breadcrumbs match the conditions the target defends against?
+
+5. **Check releases that landed near the target.**
+
+   ```
+   find_releases (around the commit date of the target)
+   ```
+
+   Cross-reference release version with the PR's merge date.
+
+6. **Use Seer sparingly.**
+
+   ```
+   analyze_issue_with_seer
+   ```
+
+   Seer produces AI root-cause analyses. Useful as a hypothesis generator, but treat them as inference, not authoritative. The actual events and stack traces are the primary evidence; Seer's narrative is secondary.
+
+## What good evidence looks like here
+
+- An issue whose **first seen** is shortly before the target's PR and **last seen** shortly after, suggesting the target addressed this error
+- Stack traces that pass through or land on the target function, showing the exact failure mode being defended against
+- A comment on the issue from the PR author describing the fix
+- The target's PR description or commit message referencing a Sentry issue URL or ID
+- An issue with high event counts that stops after the release containing the target
+
+## Common pitfalls
+
+- **Grouping drift.** Sentry groups errors by fingerprint. Refactors or renames can track the "same" error under a new issue ID. If an issue ends abruptly, the error may have just been regrouped. Check for new issues immediately after.
+- **Release correlation is noisy.** A release contains many commits. An issue stopping at v2.14.0 doesn't prove the target fixed it; another change in the same release might have. Cross-reference with the target's exact commit.
+- **Silent fixes.** Sometimes the error stops because upstream changed, not because of the defensive code. The correlation suggests the fix; it doesn't prove authorship.
+- **Resolved != fixed.** Issues can be marked "resolved" manually without any code change. Treat `resolved` as a human marker, not evidence that code fixed it.
+- **Seer hallucinations.** Seer can generate confident-sounding explanations that aren't right. Fall back to the actual events, stack traces, and timestamps when making claims.
+- **Sampling.** Some projects sample events aggressively. A low event count may just mean high sampling, not a rare error. If in doubt, note the gap.
+
+## What to return
+
+For each relevant issue:
+- Issue ID and title
+- Project and organization
+- First seen / last seen timestamps
+- Event count (and sampling rate if known)
+- Affected releases
+- A representative stack trace snippet showing relevance to the target (verbatim excerpt, not summary)
+- First/last-seen correlation with the target's ship date
+- Link to the issue
+- Any author comments or resolution notes

package/dist/builtin/workflows/skills/why/references/sources/slack.md

@@ -0,0 +1,54 @@
+# Slack Conversations
+
+## What this source contains
+
+- Real-time discussions of problems and decisions
+- Incident channels where fire-drill decisions were made
+- Design discussion threads where tradeoffs were debated
+- Questions answered by senior engineers that didn't make it into docs
+- Post-merge discussions that explain why something was revisited
+- DMs (usually not searchable, scope accordingly)
+
+Slack is frequently where the *real* decisions got made, especially for smaller changes that didn't warrant a doc. It's also the most ephemeral source: threads get deleted, channels get archived, and search quality degrades over time.
+
+## How to search it
+
+Slack MCP tools vary. Check which Slack MCP is available and inspect its tool schema first. It may require `mcp_auth`. If authentication fails, stop and report the gap.
+
+1. **Author-bounded search.** Messages from the PR author around the PR merge date. Limits scope dramatically and often hits gold.
+2. **Keyword search for the feature name and key symbols.** Include misspellings and casual phrasings.
+3. **PR URL search.** Slack often links PRs when they're reviewed or discussed. Search for the PR URL (or just `/pull/<number>`).
+4. **Error string search.** If the code handles a specific error, search for the error string. Incident threads often surface.
+5. **Channel-scoped search.** Narrow to likely channels:
+   - `#eng-*`. Engineering discussions
+   - `#proj-*`. Project channels
+   - `#incident-*` / `#sev-*`. Incident channels
+   - Team-specific channels for the owning team
+   - Design review channels
+6. **Thread traversal.** When you find a relevant message, fetch the whole thread. The decision often lives in the replies.
+
+## What good evidence looks like here
+
+- A thread where tradeoffs were explicitly debated ("I was going to use A but B is better because...")
+- An incident channel message describing the bug the code prevents
+- A question from a reviewer and an authoritative answer from the author or lead
+- A reference to a meeting where a decision was made
+- A message from a product manager or customer-facing engineer explaining a customer ask
+
+## Common pitfalls
+
+- **Channel archaeology limits.** Very old messages may be gone due to retention policies. If you can't find anything before a certain date, note the retention cliff.
+- **Unsearched DMs.** Many decisions happen in DMs that aren't searchable. You'll miss them; that's a known limitation.
+- **Speculative jokes as "decisions."** Slack is casual. "Lol just do the thing" isn't a decision, even if it preceded the commit. Look for considered discussion.
+- **Context collapse in single messages.** Without the thread, a single message often reads differently than in context. Always fetch threads.
+- **Auth failures.** If the MCP isn't authenticated, stop. Don't make up findings. Report that Slack wasn't searchable.
+
+## What to return
+
+For each relevant thread:
+- Channel name
+- Permalink or thread ID
+- Participants
+- Date range of the discussion
+- The key quotes (verbatim) with attribution
+- Context: what thread/incident/discussion this was part of