From 41bccc4aa82dc2b8cd0f70228839788007bdc34c Mon Sep 17 00:00:00 2001 From: Arvin Xu Date: Wed, 3 Jun 2026 17:02:24 +0800 Subject: [PATCH] chore: remove LOBE-XXX markers from code comments (#15422) chore: remove LOBE-XXX markers from code comments - match.test.ts: replace (LOBE-9913) marker with inline comment context - nightly-review.golden.json: replace (LOBE-9434) marker with execAgent migration context Co-authored-by: Arvin Xu --- apps/cli/e2e/fixtures/agent-signal/nightly-review.golden.json | 2 +- packages/model-runtime/src/errors/match.test.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/apps/cli/e2e/fixtures/agent-signal/nightly-review.golden.json b/apps/cli/e2e/fixtures/agent-signal/nightly-review.golden.json index b221701090..6ab4cfa230 100644 --- a/apps/cli/e2e/fixtures/agent-signal/nightly-review.golden.json +++ b/apps/cli/e2e/fixtures/agent-signal/nightly-review.golden.json @@ -1,5 +1,5 @@ { - "description": "Desensitized golden snapshot of one nightly-review self-iteration run. Used as a structural regression baseline by the execAgent migration (LOBE-9434). Assert structure, never byte-for-byte: the LLM output is non-deterministic.", + "description": "Desensitized golden snapshot of one nightly-review self-iteration run. Used as a structural regression baseline by the execAgent migration which converges all agent execution paths (chat, self-iteration, memoryWriter, skillManagement) onto a single execAgent entry point. Assert structure, never byte-for-byte: the LLM output is non-deterministic.", "finalState": { "messages": [ { diff --git a/packages/model-runtime/src/errors/match.test.ts b/packages/model-runtime/src/errors/match.test.ts index 074b986226..9ac9a8b50e 100644 --- a/packages/model-runtime/src/errors/match.test.ts +++ b/packages/model-runtime/src/errors/match.test.ts @@ -17,7 +17,7 @@ describe('matchErrorPattern', () => { ); }); - it('classifies ollamacloud "context window exceeds limit" as ExceededContextWindow, not ProviderBizError (LOBE-9913)', () => { + it('classifies ollamacloud "context window exceeds limit" as ExceededContextWindow, not ProviderBizError', () => { // ollamacloud surfaces context-window overflow as a generic 400 that the // upstream labels ProviderBizError. The ECW message pattern sits before the // 400 / ProviderBizError catch-alls, so the message wins regardless.