@jphutchins/code-review 0.1.0-alpha.61 → 0.1.0-alpha.62

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jphutchins/code-review",
-  "version": "0.1.0-alpha.61",
+  "version": "0.1.0-alpha.62",
   "description": "Deterministic commenter for agentic PR review — gather, render, inline, post, adapt, extract, cost, validate, print-schema",
   "type": "module",
   "license": "MIT",

package/schema/prices.example.json

@@ -1,5 +1,5 @@
 {
-  "_comment": "USD per 1,000,000 tokens. PRICES DRIFT — keep _updated current. cost = sum over models of (input*in + output*out + cache_read*cache_read + cache_write*cache_write) / 1e6. A model's value is EITHER a flat all-day price OR a { \"slots\": [...] } set of UTC time-of-day slots for peak/off-peak pricing (issue #170), optionally with a \"weekend_slots\" partition for a provider that prices weekends differently (issue #216) — see schema/prices.schema.json, whose field descriptions are the spec: each slot is a half-open [utc_from, utc_to) UTC window, a slot whose utc_to <= utc_from wraps past midnight, and a model's slots must partition the 24h day with no gap or overlap. The CLI's vendor_cost_usd is vendor-priced and will be wrong for non-Anthropic backends; recompute from this map using the `models` array in the abstract result envelope (SPEC §6.1). Fill the zeros from your provider's pricing page.",
+  "_comment": "USD per 1,000,000 tokens. PRICES DRIFT — keep _updated current. cost = sum over models of (input*in + output*out + cache_read*cache_read + cache_write*cache_write) / 1e6. A model's value is EITHER a flat all-day price OR a { \"slots\": [...] } set of UTC time-of-day slots for peak/off-peak pricing (issue #170), optionally with a \"weekend_slots\" partition for a provider that prices weekends differently (issue #216) — see schema/prices.schema.json, whose field descriptions are the spec: each slot is a half-open [utc_from, utc_to) UTC window, a slot whose utc_to <= utc_from wraps past midnight, and a model's slots must partition the 24h day with no gap or overlap. The CLI's vendor_cost_usd is vendor-priced and will be wrong for non-Anthropic backends; recompute from this map using the `models` array in the abstract result envelope (SPEC §6.1). Fill the zeros from your provider's pricing page, then set _updated to the day you verified them — the staleness warn keys on it.",
   "_updated": "2026-07-03",
   "_unit": "USD per 1M tokens",
   "models": {

package/templates/comment.eta

@@ -12,7 +12,7 @@
 
 ## <%= it.verdictBadge(it.findings.verdict) %>
 
-<% const meta = [] %><% if (it.postedAt) { meta.push('**Reviewed** `' + it.reviewedSha.slice(0, 7) + '` at ' + it.postedAt) } %><% if (it.route) { meta.push('**route:** ' + it.route) } %><% if (it.effort) { meta.push('**effort:** ' + it.effort) } %><% if (it.hasUsage) { meta.push('**turns:** ' + it.envelope.turns); meta.push('**wall:** ' + it.formatDuration(it.envelope.duration_ms)); meta.push('**models:** ' + it.modelNames); meta.push('**cost:** ' + it.formatCost(it.costReport.totalCostUSD)) } else { meta.push('_usage unavailable_') } %><sub><%~ meta.join(' · ') %></sub>
+<% const meta = [] %><% if (it.postedAt) { meta.push('**Reviewed** `' + it.reviewedSha.slice(0, 7) + '` at ' + it.postedAt) } %><% if (it.route) { meta.push('**route:** ' + it.route) } %><% if (it.effort) { meta.push('**effort:** ' + it.effort) } %><% if (it.hasUsage) { meta.push('**turns:** ' + it.envelope.turns); meta.push('**wall:** ' + it.formatDuration(it.envelope.duration_ms)); meta.push('**models:** ' + it.modelNames); meta.push('**cost:** ' + it.formatCost(it.costReport.totalCostUSD, it.costProvenance)) } else { meta.push('_usage unavailable_') } %><sub><%~ meta.join(' · ') %></sub>
 <% if (!it.incomplete) { -%>
 <% if (it.roundsSummary) { -%>
 <sub><%~ it.roundsSummary %></sub>
@@ -229,9 +229,19 @@ All <%= it.testReport.total %> tests passed.
 > | Model | Input | Output | Cache read | Cache write | Cost |
 > |---|--:|--:|--:|--:|--:|
 <% it.costReport.lines.forEach(function(l) { -%>
-> | <%= l.model %> | <%= it.formatTokens(l.inputTokens) %> | <%= it.formatTokens(l.outputTokens) %> | <%= it.formatTokens(l.cacheReadTokens) %> | <%= it.formatTokens(l.cacheWriteTokens) %> | <%= it.formatCost(l.costUSD) %> |
+> | <%= l.model %> | <%= it.formatTokens(l.inputTokens) %> | <%= it.formatTokens(l.outputTokens) %> | <%= it.formatTokens(l.cacheReadTokens) %> | <%= it.formatTokens(l.cacheWriteTokens) %> | <%= it.formatCost(l.costUSD, l.known ? 'complete' : 'unknown') %> |
 <% }) -%>
-> | **Total** | **<%= it.formatTokens(it.costReport.totalInputTokens) %>** | **<%= it.formatTokens(it.costReport.totalOutputTokens) %>** | **<%= it.formatTokens(it.costReport.totalCacheReadTokens) %>** | **<%= it.formatTokens(it.costReport.totalCacheWriteTokens) %>** | **<%= it.formatCost(it.costReport.totalCostUSD) %>** |
+> | **Total** | **<%= it.formatTokens(it.costReport.totalInputTokens) %>** | **<%= it.formatTokens(it.costReport.totalOutputTokens) %>** | **<%= it.formatTokens(it.costReport.totalCacheReadTokens) %>** | **<%= it.formatTokens(it.costReport.totalCacheWriteTokens) %>** | **<%= it.formatCost(it.costReport.totalCostUSD, it.costProvenance) %>** |
+<% if (it.pricesProvided) { -%>
+<% if (it.stampValid) { -%>
+> Prices snapshot `_updated` <%= it.pricesUpdatedAt %>.
+<% } else { -%>
+> `_updated` is not a parseable ISO date — the staleness check is skipped.
+<% } -%>
+<% } -%>
+<% if (it.pricesProvided && it.costProvenance !== 'complete') { -%>
+> N/A rows: models missing from this price map, or slots that do not cover the run instant — check the run log for the slot warning.
+<% } -%>
 <% } else if (it.usageAvailable) { -%>
 > No per-model usage was recorded for this run.
 <% } else { -%>