refactor(ai): reuse formatTelemetryError and the recordModel resolution

[evlogai]

Summary

Dedupe batch from the 2026-10-09 simplification sweep (revision a34dffd), packages/evlog/src/ai/index.ts, findings code-1 and code-2.

• recordError resolved the provider and model a second time to build its stepsUsage entry. recordModel now returns the pair it resolves, and recordError reuses it.
• The inline error instanceof Error ? error.message : String(error) expression is repeated at two sites. Both now call formatTelemetryError, which already holds that exact expression and is already used by the integration.

Behavior is unchanged. No changeset: this is an internal refactor with no consumer-visible change.

Evidence

-  recordModel(state, model.provider, model.modelId)
+  const resolved = recordModel(state, model.provider, model.modelId)
   state.lastFinishReason = 'error'
-  state.lastError = error instanceof Error ? error.message : String(error)
+  state.lastError = formatTelemetryError(error)
 
-  const resolved = resolveProviderAndModel(model.provider, model.modelId)
   state.stepsUsage.push({
...
-            streamError = chunk.error instanceof Error ? chunk.error.message : String(chunk.error)
+            streamError = formatTelemetryError(chunk.error)

Checks, run in /workspace/repo at 0ce5885 (sandbox cache enabled, read-only):

• pnpm --filter evlog exec vitest run test/ai: 103 of 103 passed.
• pnpm --filter evlog exec vitest run: 96 files, 2014 of 2014 tests passed.
• pnpm --filter evlog run lint: exit 0.
• pnpm --filter evlog run build: exit 0. That package's typecheck script is echo 'Typecheck handled by build', so the build is the typecheck.
• tsc --noEmit -p packages/evlog/tsconfig.json: reports errors only in test files and other source files, none in src/ai/index.ts. That tsconfig includes the test directory. I did not run it on main for a baseline comparison.

ELI5

The AI logger had two spots that did the same small job twice: one that figured out the model name a second time, and one that turned an error into text in three places. This change does each job once and reuses the result. What gets logged stays exactly the same.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
evi	Ready	Preview	Oct 9, 2026 6:37pm UTC
evlog-telemetry	Ready	Preview	Oct 9, 2026 6:37pm UTC

3 Skipped Deployments

Project	Deployment	Actions	Updated
evlog-docs	Skipped	v0	Oct 9, 2026 6:37pm UTC
evlog-render-lab	Skipped		Oct 9, 2026 6:37pm UTC
just-use-evlog	Skipped		Oct 9, 2026 6:37pm UTC

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4f1ea95b-d970-4b26-827c-4cc109d91bd0

---

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

---

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

Thank you for following the naming conventions! 🙏

[pkg-pr-new]

@evlog/cli

npm i https://pkg.pr.new/@evlog/cli@800

evlog

npm i https://pkg.pr.new/evlog@800

@evlog/nuxthub

npm i https://pkg.pr.new/@evlog/nuxthub@800

@evlog/signals

npm i https://pkg.pr.new/@evlog/signals@800

@evlog/telemetry

npm i https://pkg.pr.new/@evlog/telemetry@800

commit: 0ce5885