@weareikko/code-review 0.9.1 → 0.9.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -560,7 +560,7 @@ function buildSummaryBody(summary, costFooter, options = {}) {
560
560
  return `${withFooter}\n\n${buildSummaryHistoryBlock(historyEntries)}`;
561
561
  }
562
562
  function buildReviewedCommitFooter(commitSha) {
563
- return `Reviewed by ${PRODUCT_LINK} v0.9.1 for commit ${commitSha}.`;
563
+ return `Reviewed by ${PRODUCT_LINK} v0.9.3 for commit ${commitSha}.`;
564
564
  }
565
565
  function extractReviewedCommitSha(body) {
566
566
  return REVIEWED_COMMIT_FOOTER_PATTERN.exec(body)?.[1] ?? null;
@@ -1358,7 +1358,7 @@ function createDiagnosticContext(phase, config, runId, overrides = {}) {
1358
1358
  phase,
1359
1359
  project: config.project,
1360
1360
  mr: config.mr,
1361
- gitlabUrl: config.gitlabUrl,
1361
+ gitlabUrl: config.platform === "github" ? config.githubServerUrl : config.gitlabUrl,
1362
1362
  platform: config.platform,
1363
1363
  cwd: config.cwd,
1364
1364
  model: config.model,
@@ -5240,21 +5240,20 @@ function buildAgentSubscriber(tracer, tokenUsage, operationCost, timeToFirstToke
5240
5240
  * Applies our OTLP exporter env defaults, only when the caller hasn't set them
5241
5241
  * (explicit choices, including `none`, always win).
5242
5242
  *
5243
- * Metrics temporality defaults to **delta** because code-review runs as a
5244
- * short-lived CI job. With the OTel SDK's default cumulative temporality, each
5245
- * ephemeral job's metric series starts mid-flight in the backend (its first
5246
- * export already carries accumulated counts, and per-run histograms are a single
5247
- * observation), so range aggregation of cost/token totals is unreliable —
5248
- * `sum_over_time` over-counts and `increase` under-counts or returns nothing.
5249
- * Delta makes each job's metrics self-contained deltas that sum correctly across
5250
- * runs. Override with `OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE=cumulative`
5251
- * for long-running hosts, and confirm your backend ingests delta (Grafana
5252
- * Cloud/Mimir converts to cumulative or stores it natively) before relying on it.
5243
+ * Metrics stay on the SDK/backend default **cumulative** temporality. Delta
5244
+ * would be the textbook choice for short-lived CI jobs (cumulative series from
5245
+ * ephemeral jobs start mid-flight, so range aggregation of totals is operator-
5246
+ * dependent), but 0.9.1 defaulted to delta and it was verified to BREAK ingest
5247
+ * on Grafana Cloud — the OTLP endpoint silently dropped every delta-temporality
5248
+ * metric (a review's metrics never landed), so 0.9.2 reverted it. Do not force
5249
+ * delta here unless your backend is confirmed to ingest it (e.g. an OTel
5250
+ * Collector / Alloy running `deltatocumulative` in front of the backend). To
5251
+ * aggregate a range total from these cumulative ephemeral-job metrics, use
5252
+ * per-series `max_over_time`.
5253
5253
  */
5254
5254
  function applyOtelExporterDefaults(env = process.env) {
5255
5255
  env.OTEL_METRICS_EXPORTER = env.OTEL_METRICS_EXPORTER ?? "otlp";
5256
5256
  env.OTEL_LOGS_EXPORTER = env.OTEL_LOGS_EXPORTER ?? "otlp";
5257
- env.OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE = env.OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE ?? "delta";
5258
5257
  }
5259
5258
  async function loadDefaultRuntime() {
5260
5259
  let modules;
@@ -5266,7 +5265,7 @@ async function loadDefaultRuntime() {
5266
5265
  const [sdkNode, resources, semconv] = modules;
5267
5266
  const serviceResource = resources.resourceFromAttributes({
5268
5267
  [semconv.ATTR_SERVICE_NAME ?? "service.name"]: SERVICE_NAME,
5269
- [semconv.ATTR_SERVICE_VERSION ?? "service.version"]: "0.9.1"
5268
+ [semconv.ATTR_SERVICE_VERSION ?? "service.version"]: "0.9.3"
5270
5269
  });
5271
5270
  applyOtelExporterDefaults(process.env);
5272
5271
  const sdk = new sdkNode.NodeSDK({ resource: resources.defaultResource().merge(serviceResource) });
@@ -5738,7 +5737,7 @@ function boldCommentTitle(body) {
5738
5737
  */
5739
5738
  function buildCommentBody(body, commitSha, confidence) {
5740
5739
  const confidenceLine = `_Confidence: ${confidence}._`;
5741
- const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.9.1 for commit ${commitSha}.</sub>`;
5740
+ const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.9.3 for commit ${commitSha}.</sub>`;
5742
5741
  return `${boldCommentTitle(body.trim())}\n\n${confidenceLine}\n\n---\n\n${footer}`;
5743
5742
  }
5744
5743
  function buildPayload(comment, body, refs, resolved) {
@@ -6845,10 +6844,10 @@ async function main(argv = process.argv.slice(2)) {
6845
6844
  return;
6846
6845
  }
6847
6846
  if (argv.includes("--version") || argv.includes("-v")) {
6848
- console.log("0.9.1");
6847
+ console.log("0.9.3");
6849
6848
  return;
6850
6849
  }
6851
- process.stderr.write(`[code-review] @weareikko/code-review v0.9.1\n`);
6850
+ process.stderr.write(`[code-review] @weareikko/code-review v0.9.3\n`);
6852
6851
  assertNodeVersion();
6853
6852
  applyCodeReviewEnvPrefix();
6854
6853
  applyDefaultCacheRetention();
@@ -6871,4 +6870,4 @@ if (isDirectRun()) main().catch((error) => {
6871
6870
  //#endregion
6872
6871
  export { normalizeBody as $, SUMMARY_HISTORY_END as A, buildSummaryHistoryEntries as B, createDiagnosticContext as C, traceDiagnosticPhase as D, traceDiagnostic as E, SUMMARY_MARKER as F, findExistingSummaryNoteId as G, extractSummaryHistoryEntries as H, buildArchivedSummaryEntry as I, upsertSummaryNote as J, stripSummaryHistory as K, buildReviewedCommitFooter as L, SUMMARY_HISTORY_ENTRY_START as M, SUMMARY_HISTORY_LIMIT as N, normalizeSeverity as O, SUMMARY_HISTORY_START as P, fingerprints as Q, buildSizeNoticeBlock as R, DIAGNOSTIC_CHANNEL_PREFIX as S, diagnosticChannels as T, findExistingReviewedCommitSha as U, extractReviewedCommitSha as V, findExistingSummaryNote as W, extractDiffHunkContext as X, appendFingerprintMarkers as Y, extractExistingFingerprints as Z, resolveNpmSkillDir as _, main as a, parseReviewMarkdownWithWarnings as b, buildGeneratedComments as c, startOtelBridge as d, sha256 as et, filterDiff as f, parseSkillSpec as g, loadNamedSkill as h, formatUsageLine as i, SUMMARY_HISTORY_ENTRY_END as j, toGitLabReviewSeverity as k, buildPayload as l, gitSkillCacheKey as m, formatPerModelUsage as n, run as o, runReview as p, stripSummaryMarker as q, formatSkillsFooter as r, withHttpStamping as s, countPostedBySeverity as t, isOtelEnabled as u, resolveSkillCacheDir as v, createDiagnosticRunId as w, DIAGNOSTIC_CHANNEL_NAMES as x, parseReviewMarkdown as y, buildSummaryBody as z };
6873
6872
 
6874
- //# sourceMappingURL=cli-BGn8g23f.js.map
6873
+ //# sourceMappingURL=cli-BaZloofb.js.map