Private
Public Access
feat(audit): rich rollups + per-line indentation fix - 2136 total lines
Added 3 new top-level rollups (hot_paths.md, dead_fields.md, plus enriched summary.md, candidates.md, decomposition_matrix.md): - summary.md: per-aggregate memory_dim + access pattern tables, full cross-validation verdict per aggregate - decomposition_matrix.md: all 10 aggregates ranked by current cost, flagged-for-refactoring section, insufficient_data section - candidates.md: ranked optimization candidates with detail per step - hot_paths.md: top 5 hot consumers per aggregate (by field access count) - dead_fields.md: fields accessed (per-consumer breakdown) Total report: 2136 lines (was 1814).
This commit is contained in:
@@ -1,6 +1,13 @@
|
||||
# Code Path & Data Pipeline Audit Summary
|
||||
|
||||
Generated for 13 aggregates
|
||||
Generated for 13 aggregates on 2026-06-22
|
||||
|
||||
- **Real aggregates (in scope):** 10
|
||||
- **Candidate aggregates (placeholders):** 3
|
||||
- **Total producers:** 78
|
||||
- **Total consumers:** 41
|
||||
- **Total current cost (us/turn):** 4950
|
||||
- **Total actionable savings (us/turn):** 140
|
||||
|
||||
## 4-mem-dim rollup
|
||||
|
||||
@@ -9,18 +16,82 @@ Generated for 13 aggregates
|
||||
- **discussion** (6): Metadata, CommsLogEntry, CommsLog, HistoryMessage, History, ChatMessage
|
||||
- **unknown** (2): ToolSpec, ProviderHistory
|
||||
|
||||
## Per-aggregate memory_dim + access pattern
|
||||
|
||||
| Aggregate | Kind | Memory dim | Access pattern | Producers | Consumers |
|
||||
|---|---|---|---|---|---|
|
||||
| `CommsLog` | `typealias` | `discussion` | `mixed` | 0 | 0 |
|
||||
| `CommsLogEntry` | `typealias` | `discussion` | `mixed` | 0 | 0 |
|
||||
| `FileItem` | `typealias` | `curation` | `mixed` | 0 | 0 |
|
||||
| `FileItems` | `typealias` | `curation` | `whole_struct` | 0 | 3 |
|
||||
| `History` | `typealias` | `discussion` | `mixed` | 0 | 0 |
|
||||
| `HistoryMessage` | `typealias` | `discussion` | `mixed` | 0 | 2 |
|
||||
| `Metadata` | `typealias` | `discussion` | `whole_struct` | 77 | 35 |
|
||||
| `Result` | `typealias` | `control` | `mixed` | 0 | 0 |
|
||||
| `ToolCall` | `typealias` | `control` | `whole_struct` | 1 | 1 |
|
||||
| `ToolDefinition` | `typealias` | `control` | `mixed` | 0 | 0 |
|
||||
| `ChatMessage` | `candidate_dataclass` | `discussion` | `mixed` | 0 | 0 |
|
||||
| `ProviderHistory` | `candidate_dataclass` | `unknown` | `mixed` | 0 | 0 |
|
||||
| `ToolSpec` | `candidate_dataclass` | `unknown` | `mixed` | 0 | 0 |
|
||||
|
||||
## Cross-validation verdict
|
||||
|
||||
- **Metadata**: result_coverage=77 producers, 35 consumers; type_alias_coverage=130 sites; 0 typed (0%); 130 untyped (100%)
|
||||
- **FileItem**: result_coverage=0 producers, 0 consumers; type_alias_coverage=0 sites
|
||||
- **FileItems**: result_coverage=0 producers, 3 consumers; type_alias_coverage=0 sites
|
||||
- **CommsLogEntry**: result_coverage=0 producers, 0 consumers; type_alias_coverage=0 sites
|
||||
- **CommsLog**: result_coverage=0 producers, 0 consumers; type_alias_coverage=0 sites
|
||||
- **HistoryMessage**: result_coverage=0 producers, 2 consumers; type_alias_coverage=4 sites; 0 typed (0%); 4 untyped (100%)
|
||||
- **History**: result_coverage=0 producers, 0 consumers; type_alias_coverage=0 sites
|
||||
- **ToolDefinition**: result_coverage=0 producers, 0 consumers; type_alias_coverage=0 sites
|
||||
- **ToolCall**: result_coverage=1 producers, 1 consumers; type_alias_coverage=1 sites; 0 typed (0%); 1 untyped (100%)
|
||||
- **Result**: result_coverage=0 producers, 0 consumers; type_alias_coverage=0 sites
|
||||
- **ToolSpec**: result_coverage=; type_alias_coverage=
|
||||
- **ChatMessage**: result_coverage=; type_alias_coverage=
|
||||
- **ProviderHistory**: result_coverage=; type_alias_coverage=
|
||||
### `CommsLog`
|
||||
|
||||
- **Result coverage:** 0 producers, 0 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `CommsLogEntry`
|
||||
|
||||
- **Result coverage:** 0 producers, 0 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `FileItem`
|
||||
|
||||
- **Result coverage:** 0 producers, 0 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `FileItems`
|
||||
|
||||
- **Result coverage:** 0 producers, 3 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `History`
|
||||
|
||||
- **Result coverage:** 0 producers, 0 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `HistoryMessage`
|
||||
|
||||
- **Result coverage:** 0 producers, 2 consumers
|
||||
- **Type alias coverage:** 4 sites; 0 typed (0%); 4 untyped (100%)
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `Metadata`
|
||||
|
||||
- **Result coverage:** 77 producers, 35 consumers
|
||||
- **Type alias coverage:** 130 sites; 0 typed (0%); 130 untyped (100%)
|
||||
- **Cross-audit findings (total sites):** 1
|
||||
|
||||
### `Result`
|
||||
|
||||
- **Result coverage:** 0 producers, 0 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `ToolCall`
|
||||
|
||||
- **Result coverage:** 1 producers, 1 consumers
|
||||
- **Type alias coverage:** 1 sites; 0 typed (0%); 1 untyped (100%)
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
### `ToolDefinition`
|
||||
|
||||
- **Result coverage:** 0 producers, 0 consumers
|
||||
- **Type alias coverage:** 0 sites
|
||||
- **Cross-audit findings (total sites):** 0
|
||||
|
||||
Reference in New Issue
Block a user