@swmansion/argent 0.12.1 → 0.13.1-next.0

package/README.md

@@ -117,17 +117,18 @@ argent init
 
 `argent init` auto-detects and configures MCP for:
 
-| Editor      | Config location                                                          |
-| ----------- | ------------------------------------------------------------------------ |
-| Claude Code | `.mcp.json` (project) or `~/.claude.json` (global)                       |
-| Cursor      | `.cursor/mcp.json` (project) or `~/.cursor/mcp.json` (global)            |
-| VS Code     | `.vscode/mcp.json`                                                       |
-| Windsurf    | `~/.codeium/windsurf/mcp_config.json` (global)                           |
-| Zed         | `.zed/settings.json`                                                     |
-| Gemini CLI  | `.gemini/settings.json`                                                  |
-| Codex CLI   | `.codex/config.toml` (project) or `~/.codex/config.toml` (global)        |
-| Hermes      | `~/.hermes/config.yaml` (global)                                         |
-| opencode    | `opencode.json` (project) or `~/.config/opencode/opencode.json` (global) |
+| Editor      | Config location                                                             |
+| ----------- | --------------------------------------------------------------------------- |
+| Claude Code | `.mcp.json` (project) or `~/.claude.json` (global)                          |
+| Cursor      | `.cursor/mcp.json` (project) or `~/.cursor/mcp.json` (global)               |
+| VS Code     | `.vscode/mcp.json`                                                          |
+| Windsurf    | `~/.codeium/windsurf/mcp_config.json` (global)                              |
+| Zed         | `.zed/settings.json`                                                        |
+| Gemini CLI  | `.gemini/settings.json`                                                     |
+| Codex CLI   | `.codex/config.toml` (project) or `~/.codex/config.toml` (global)           |
+| Hermes      | `~/.hermes/config.yaml` (global)                                            |
+| opencode    | `opencode.json` (project) or `~/.config/opencode/opencode.json` (global)    |
+| Kiro        | `.kiro/settings/mcp.json` (project) or `~/.kiro/settings/mcp.json` (global) |
 
 ## Privacy
 

package/assets/queries/cpu-hotspots.sql

@@ -1,10 +1,19 @@
 -- Argent — CPU hotspots.
 --
 -- One output row per (thread_name, leaf_function) — the leaf frame name IS the
--- dominant function, so one SQL row maps 1:1 to one aggregateCpuHotspots group
--- (we drop leaf_mapping, which was unused and only fragmented the grouping).
+-- dominant function, so one SQL row maps 1:1 to one aggregateCpuHotspots group.
 -- The aggregator normalises the thread name and applies severity bands.
 --
+-- We ALSO carry the leaf frame's mapping (the loaded object the symbol lives in,
+-- e.g. `/system/lib64/libhwui.so` for app/framework code or `/kernel` for kernel
+-- frames) through to the classifier. Name patterns alone can't tell `writel`
+-- (a kernel MMIO write) from app code, but the mapping can: a leaf in `/kernel`
+-- is unambiguously system/emulator overhead. To keep one output row per
+-- (thread_name, leaf_function) — so the SQL→aggregator 1:1 mapping is preserved
+-- and the grouping is NOT fragmented — the mapping is folded in with MIN() (a
+-- given leaf symbol resolves to a single mapping in practice, so MIN is just a
+-- grouping-safe pick, not a lossy aggregate).
+--
 -- Burst windows are computed here in SQL rather than shipping every sample
 -- timestamp (the old `GROUP_CONCAT(ts_ns)` shipped ~54 KB of timestamps that
 -- JS re-parsed). A "burst" is a run of samples for the same (thread, function)
@@ -40,12 +49,14 @@ WITH samples AS (
     ps.ts            AS ts_ns,
     t.name           AS thread_name,
     t.is_main_thread AS is_main_thread,
-    spf.name         AS leaf_function
+    spf.name         AS leaf_function,
+    spm.name         AS leaf_mapping
   FROM perf_sample ps
   JOIN thread t USING (utid)
   JOIN process p USING (upid)
   LEFT JOIN stack_profile_callsite spc ON ps.callsite_id = spc.id
   LEFT JOIN stack_profile_frame    spf ON spc.frame_id   = spf.id
+  LEFT JOIN stack_profile_mapping  spm ON spf.mapping    = spm.id
   WHERE p.name = (SELECT target_process FROM _argent_args)
 ),
 -- Flag each sample whose gap to the previous sample of the same
@@ -53,7 +64,7 @@ WITH samples AS (
 -- NULL, so its CASE yields 0 — the opening sample never counts as a gap.
 flagged AS (
   SELECT
-    thread_name, is_main_thread, leaf_function, ts_ns,
+    thread_name, is_main_thread, leaf_function, leaf_mapping, ts_ns,
     CASE
       WHEN ts_ns - LAG(ts_ns) OVER w > (SELECT burst_gap_ns FROM _argent_args) THEN 1
       ELSE 0
@@ -65,7 +76,7 @@ flagged AS (
 -- each thread+function partition.
 ided AS (
   SELECT
-    thread_name, is_main_thread, leaf_function, ts_ns,
+    thread_name, is_main_thread, leaf_function, leaf_mapping, ts_ns,
     SUM(is_new_burst) OVER (
       PARTITION BY thread_name, leaf_function
       ORDER BY ts_ns
@@ -76,10 +87,13 @@ ided AS (
 -- Collapse each burst to [start_ns, end_ns, sample_count]. Every sample lands
 -- in exactly one burst, so summing burst counts == total samples and
 -- MIN/MAX of burst bounds == first/last sample of the (thread, function).
+-- leaf_mapping is grouping-invariant per (thread, function), so MIN() picks it
+-- without affecting the per-burst cardinality.
 per_burst AS (
   SELECT
     thread_name, leaf_function,
     MAX(is_main_thread) AS is_main_thread,
+    MIN(leaf_mapping)   AS leaf_mapping,
     MIN(ts_ns)          AS burst_start_ns,
     MAX(ts_ns)          AS burst_end_ns,
     COUNT(*)            AS burst_count
@@ -90,6 +104,9 @@ SELECT
   thread_name,
   MAX(is_main_thread) AS is_main_thread,
   leaf_function,
+  -- One mapping per (thread, function) group; MIN() keeps cardinality 1:1 with
+  -- the existing GROUP BY so the grouping is not fragmented.
+  MIN(leaf_mapping)   AS leaf_mapping,
   SUM(burst_count)    AS sample_count,
   MIN(burst_start_ns) AS first_ts_ns,
   MAX(burst_end_ns)   AS last_ts_ns,