@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.
- package/dist/{cli-BGn8g23f.js → cli-BaZloofb.js} +17 -18
- package/dist/cli-BaZloofb.js.map +1 -0
- package/dist/cli.js +1 -1
- package/dist/diagnostics.d.ts +5 -0
- package/dist/diagnostics.d.ts.map +1 -1
- package/dist/otel.d.ts +10 -10
- package/dist/otel.d.ts.map +1 -1
- package/dist/review.js +1 -1
- package/package.json +1 -1
- package/dist/cli-BGn8g23f.js.map +0 -1
|
@@ -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.
|
|
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
|
|
5244
|
-
*
|
|
5245
|
-
* ephemeral
|
|
5246
|
-
*
|
|
5247
|
-
*
|
|
5248
|
-
*
|
|
5249
|
-
*
|
|
5250
|
-
*
|
|
5251
|
-
*
|
|
5252
|
-
*
|
|
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.
|
|
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.
|
|
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.
|
|
6847
|
+
console.log("0.9.3");
|
|
6849
6848
|
return;
|
|
6850
6849
|
}
|
|
6851
|
-
process.stderr.write(`[code-review] @weareikko/code-review v0.9.
|
|
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-
|
|
6873
|
+
//# sourceMappingURL=cli-BaZloofb.js.map
|