@weareikko/code-review 0.9.1 → 0.9.2

package/dist/cli.js

@@ -1,2 +1,2 @@
-import { C as createDiagnosticContext, D as traceDiagnosticPhase, E as traceDiagnostic, S as DIAGNOSTIC_CHANNEL_PREFIX, T as diagnosticChannels, a as main, d as startOtelBridge, i as formatUsageLine, n as formatPerModelUsage, o as run, r as formatSkillsFooter, s as withHttpStamping, t as countPostedBySeverity, u as isOtelEnabled, w as createDiagnosticRunId, x as DIAGNOSTIC_CHANNEL_NAMES } from "./cli-BGn8g23f.js";
+import { C as createDiagnosticContext, D as traceDiagnosticPhase, E as traceDiagnostic, S as DIAGNOSTIC_CHANNEL_PREFIX, T as diagnosticChannels, a as main, d as startOtelBridge, i as formatUsageLine, n as formatPerModelUsage, o as run, r as formatSkillsFooter, s as withHttpStamping, t as countPostedBySeverity, u as isOtelEnabled, w as createDiagnosticRunId, x as DIAGNOSTIC_CHANNEL_NAMES } from "./cli-D47EQ1qq.js";
 export { DIAGNOSTIC_CHANNEL_NAMES, DIAGNOSTIC_CHANNEL_PREFIX, countPostedBySeverity, createDiagnosticContext, createDiagnosticRunId, diagnosticChannels, formatPerModelUsage, formatSkillsFooter, formatUsageLine, isOtelEnabled, main, run, startOtelBridge, traceDiagnostic, traceDiagnosticPhase, withHttpStamping };

package/dist/otel.d.ts

@@ -112,16 +112,16 @@ export declare function startOtelBridge(options?: OtelBridgeOptions): Promise<Ot
  * Applies our OTLP exporter env defaults, only when the caller hasn't set them
  * (explicit choices, including `none`, always win).
  *
- * Metrics temporality defaults to **delta** because code-review runs as a
- * short-lived CI job. With the OTel SDK's default cumulative temporality, each
- * ephemeral job's metric series starts mid-flight in the backend (its first
- * export already carries accumulated counts, and per-run histograms are a single
- * observation), so range aggregation of cost/token totals is unreliable —
- * `sum_over_time` over-counts and `increase` under-counts or returns nothing.
- * Delta makes each job's metrics self-contained deltas that sum correctly across
- * runs. Override with `OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE=cumulative`
- * for long-running hosts, and confirm your backend ingests delta (Grafana
- * Cloud/Mimir converts to cumulative or stores it natively) before relying on it.
+ * Metrics stay on the SDK/backend default **cumulative** temporality. Delta
+ * would be the textbook choice for short-lived CI jobs (cumulative series from
+ * ephemeral jobs start mid-flight, so range aggregation of totals is operator-
+ * dependent), but 0.9.1 defaulted to delta and it was verified to BREAK ingest
+ * on Grafana Cloud — the OTLP endpoint silently dropped every delta-temporality
+ * metric (a review's metrics never landed), so 0.9.2 reverted it. Do not force
+ * delta here unless your backend is confirmed to ingest it (e.g. an OTel
+ * Collector / Alloy running `deltatocumulative` in front of the backend). To
+ * aggregate a range total from these cumulative ephemeral-job metrics, use
+ * per-series `max_over_time`.
  */
 export declare function applyOtelExporterDefaults(env?: NodeJS.ProcessEnv): void;
 //# sourceMappingURL=otel.d.ts.map

package/dist/otel.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"otel.d.ts","sourceRoot":"","sources":["../src/otel.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4CG;AAEH,OAAO,KAAK,EAMV,aAAa,EAGb,cAAc,EACf,MAAM,oBAAoB,CAAC;AAE5B,OAAO,KAAK,EAAU,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAQtE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AACpD,OAAO,EAAc,KAAK,gBAAgB,EAAE,MAAM,YAAY,CAAC;AAO/D,MAAM,WAAW,UAAU;IACzB,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1B;;;;;;OAMG;IACH,oBAAoB,CAAC,KAAK,EAAE,MAAM,GAAG,CAAC,CAAC,KAAK,EAAE,SAAS,KAAK,MAAM,IAAI,CAAC,GAAG,SAAS,CAAC;IACpF;;;;;OAKG;IACH,WAAW,CAAC,QAAQ,EAAE,gBAAgB,EAAE,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;CAChE;AAED,MAAM,WAAW,WAAW;IAC1B,cAAc,EAAE,cAAc,CAAC;IAC/B,aAAa,EAAE,aAAa,CAAC;IAC7B,cAAc,EAAE,cAAc,CAAC;IAC/B,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CAC3B;AAED,MAAM,WAAW,iBAAiB;IAChC;;;;;;OAMG;IACH,OAAO,CAAC,EAAE,WAAW,CAAC;IACtB;;;OAGG;IACH,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;IACxB;;;;;;;OAOG;IACH,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AA4DD,wBAAgB,aAAa,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,OAAO,CAE3E;AAED;;;;;;;;;GASG;AACH,wBAAgB,uBAAuB,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,OAAO,CAIrF;AAED,wBAAsB,eAAe,CAAC,OAAO,GAAE,iBAAsB,GAAG,OAAO,CAAC,UAAU,GAAG,IAAI,CAAC,CA6VjG;AAmWD;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,yBAAyB,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,IAAI,CAQpF"}
+{"version":3,"file":"otel.d.ts","sourceRoot":"","sources":["../src/otel.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4CG;AAEH,OAAO,KAAK,EAMV,aAAa,EAGb,cAAc,EACf,MAAM,oBAAoB,CAAC;AAE5B,OAAO,KAAK,EAAU,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAQtE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AACpD,OAAO,EAAc,KAAK,gBAAgB,EAAE,MAAM,YAAY,CAAC;AAO/D,MAAM,WAAW,UAAU;IACzB,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1B;;;;;;OAMG;IACH,oBAAoB,CAAC,KAAK,EAAE,MAAM,GAAG,CAAC,CAAC,KAAK,EAAE,SAAS,KAAK,MAAM,IAAI,CAAC,GAAG,SAAS,CAAC;IACpF;;;;;OAKG;IACH,WAAW,CAAC,QAAQ,EAAE,gBAAgB,EAAE,EAAE,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;CAChE;AAED,MAAM,WAAW,WAAW;IAC1B,cAAc,EAAE,cAAc,CAAC;IAC/B,aAAa,EAAE,aAAa,CAAC;IAC7B,cAAc,EAAE,cAAc,CAAC;IAC/B,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CAC3B;AAED,MAAM,WAAW,iBAAiB;IAChC;;;;;;OAMG;IACH,OAAO,CAAC,EAAE,WAAW,CAAC;IACtB;;;OAGG;IACH,GAAG,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;IACxB;;;;;;;OAOG;IACH,cAAc,CAAC,EAAE,OAAO,CAAC;CAC1B;AA4DD,wBAAgB,aAAa,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,OAAO,CAE3E;AAED;;;;;;;;;GASG;AACH,wBAAgB,uBAAuB,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,OAAO,CAIrF;AAED,wBAAsB,eAAe,CAAC,OAAO,GAAE,iBAAsB,GAAG,OAAO,CAAC,UAAU,GAAG,IAAI,CAAC,CA6VjG;AAmWD;;;;;;;;;;;;;;GAcG;AACH,wBAAgB,yBAAyB,CAAC,GAAG,GAAE,MAAM,CAAC,UAAwB,GAAG,IAAI,CAMpF"}

package/dist/review.js

@@ -1,2 +1,2 @@
-import { $ as normalizeBody, A as SUMMARY_HISTORY_END, B as buildSummaryHistoryEntries, C as createDiagnosticContext, D as traceDiagnosticPhase, E as traceDiagnostic, F as SUMMARY_MARKER, G as findExistingSummaryNoteId, H as extractSummaryHistoryEntries, I as buildArchivedSummaryEntry, J as upsertSummaryNote, K as stripSummaryHistory, L as buildReviewedCommitFooter, M as SUMMARY_HISTORY_ENTRY_START, N as SUMMARY_HISTORY_LIMIT, O as normalizeSeverity, P as SUMMARY_HISTORY_START, Q as fingerprints, R as buildSizeNoticeBlock, S as DIAGNOSTIC_CHANNEL_PREFIX, T as diagnosticChannels, U as findExistingReviewedCommitSha, V as extractReviewedCommitSha, W as findExistingSummaryNote, X as extractDiffHunkContext, Y as appendFingerprintMarkers, Z as extractExistingFingerprints, _ as resolveNpmSkillDir, b as parseReviewMarkdownWithWarnings, c as buildGeneratedComments, d as startOtelBridge, et as sha256, f as filterDiff, g as parseSkillSpec, h as loadNamedSkill, j as SUMMARY_HISTORY_ENTRY_END, k as toGitLabReviewSeverity, l as buildPayload, m as gitSkillCacheKey, o as run, p as runReview, q as stripSummaryMarker, u as isOtelEnabled, v as resolveSkillCacheDir, w as createDiagnosticRunId, x as DIAGNOSTIC_CHANNEL_NAMES, y as parseReviewMarkdown, z as buildSummaryBody } from "./cli-BGn8g23f.js";
+import { $ as normalizeBody, A as SUMMARY_HISTORY_END, B as buildSummaryHistoryEntries, C as createDiagnosticContext, D as traceDiagnosticPhase, E as traceDiagnostic, F as SUMMARY_MARKER, G as findExistingSummaryNoteId, H as extractSummaryHistoryEntries, I as buildArchivedSummaryEntry, J as upsertSummaryNote, K as stripSummaryHistory, L as buildReviewedCommitFooter, M as SUMMARY_HISTORY_ENTRY_START, N as SUMMARY_HISTORY_LIMIT, O as normalizeSeverity, P as SUMMARY_HISTORY_START, Q as fingerprints, R as buildSizeNoticeBlock, S as DIAGNOSTIC_CHANNEL_PREFIX, T as diagnosticChannels, U as findExistingReviewedCommitSha, V as extractReviewedCommitSha, W as findExistingSummaryNote, X as extractDiffHunkContext, Y as appendFingerprintMarkers, Z as extractExistingFingerprints, _ as resolveNpmSkillDir, b as parseReviewMarkdownWithWarnings, c as buildGeneratedComments, d as startOtelBridge, et as sha256, f as filterDiff, g as parseSkillSpec, h as loadNamedSkill, j as SUMMARY_HISTORY_ENTRY_END, k as toGitLabReviewSeverity, l as buildPayload, m as gitSkillCacheKey, o as run, p as runReview, q as stripSummaryMarker, u as isOtelEnabled, v as resolveSkillCacheDir, w as createDiagnosticRunId, x as DIAGNOSTIC_CHANNEL_NAMES, y as parseReviewMarkdown, z as buildSummaryBody } from "./cli-D47EQ1qq.js";
 export { DIAGNOSTIC_CHANNEL_NAMES, DIAGNOSTIC_CHANNEL_PREFIX, SUMMARY_HISTORY_END, SUMMARY_HISTORY_ENTRY_END, SUMMARY_HISTORY_ENTRY_START, SUMMARY_HISTORY_LIMIT, SUMMARY_HISTORY_START, SUMMARY_MARKER, appendFingerprintMarkers, buildArchivedSummaryEntry, buildGeneratedComments, buildPayload, buildReviewedCommitFooter, buildSizeNoticeBlock, buildSummaryBody, buildSummaryHistoryEntries, createDiagnosticContext, createDiagnosticRunId, diagnosticChannels, extractDiffHunkContext, extractExistingFingerprints, extractReviewedCommitSha, extractSummaryHistoryEntries, filterDiff, findExistingReviewedCommitSha, findExistingSummaryNote, findExistingSummaryNoteId, fingerprints, gitSkillCacheKey, isOtelEnabled, loadNamedSkill, normalizeBody, normalizeSeverity, parseReviewMarkdown, parseReviewMarkdownWithWarnings, parseSkillSpec, resolveNpmSkillDir, resolveSkillCacheDir, run, runReview, sha256, startOtelBridge, stripSummaryHistory, stripSummaryMarker, toGitLabReviewSeverity, traceDiagnostic, traceDiagnosticPhase, upsertSummaryNote };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@weareikko/code-review",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Agent-driven code review for GitLab merge requests and GitHub pull requests, with deduplicated inline comments and summary notes.",
   "keywords": [
     "ci",