Skip to content

Fix SDK DuckDB lock, LiteLLM model prefix, sidebar SVG logo#14

Merged
anilmurty merged 1 commit into
mainfrom
fix-sdk-lock-litellm-logo
Apr 8, 2026
Merged

Fix SDK DuckDB lock, LiteLLM model prefix, sidebar SVG logo#14
anilmurty merged 1 commit into
mainfrom
fix-sdk-lock-litellm-logo

Conversation

@anilmurty
Copy link
Copy Markdown
Contributor

Summary

Three bug fixes that were pushed to PR #12's branch after it was already merged.

1. SDK DuckDB lock (critical)

SDK bootstrap now detects running ocw serve and sends spans via HTTP (OcwHttpExporter) instead of opening DuckDB directly. Fixes the "Could not set lock on file" error when agents run alongside ocw serve.

2. LiteLLM model prefix

Strips openai/ prefix from model names so pricing lookup finds gpt-4o-mini not openai/gpt-4o-mini.

3. Sidebar SVG logo

Embeds opencla.watch icon inline in web UI sidebar with currentColor.

Test plan

  • All 232 tests pass
  • ruff check ocw/ clean
  • Run ocw serve & then agent — spans arrive via HTTP
  • ocw cost shows gpt-4o-mini not openai/gpt-4o-mini

🤖 Generated with Claude Code

- SDK bootstrap detects running ocw serve and sends spans via HTTP
  instead of opening DuckDB directly (fixes lock error)
- New OcwHttpExporter: OTel SpanExporter that POSTs OTLP JSON to
  /api/v1/spans with ingest secret auth
- LiteLLM integration strips provider prefix from model name
  (openai/gpt-4o-mini -> gpt-4o-mini) for consistent pricing lookup
- Embed opencla.watch SVG icon inline in web UI sidebar

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@anilmurty anilmurty force-pushed the fix-sdk-lock-litellm-logo branch from 731ba02 to 99afecb Compare April 8, 2026 16:41
@anilmurty anilmurty merged commit 032f48d into main Apr 8, 2026
4 checks passed
@anilmurty anilmurty deleted the fix-sdk-lock-litellm-logo branch April 8, 2026 16:44
anilmurty added a commit that referenced this pull request Apr 15, 2026
- pyproject.toml: 0.1.7 → 0.1.8
- sdk-ts/package.json: 0.1.7 → 0.1.8
- CLAUDE.md: add critical rule #14 — both version files must be bumped
  before creating a GitHub release, or publish workflows will 403

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
anilmurty added a commit that referenced this pull request Apr 15, 2026
README:
- Remove "Works with Claude Code" from The Problem section
- Remove "Why not LangSmith / Langfuse / Datadog" section
- Remove inline Architecture mermaid diagram (moved to docs/)
- Replace inline Claude Code, NemoClaw, OpenClaw, and framework
  details with hyperlinks to dedicated docs
- Slim framework support to a summary table linking to docs/

New docs:
- docs/claude-code-integration.md — full Claude Code setup, MCP, uninstall
- docs/nemoclaw-integration.md — NemoClaw sandbox event observability
- docs/framework-support.md — provider patches, framework patches, OTLP, TS SDK

Updated docs:
- docs/architecture.md — replace ASCII diagram with mermaid flowchart from README

Also:
- pyproject.toml: 0.1.7 → 0.1.8
- sdk-ts/package.json: 0.1.7 → 0.1.8
- CLAUDE.md: add critical rule #14 (version bump on release)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@anilmurty anilmurty mentioned this pull request Apr 15, 2026
3 tasks
@anilmurty anilmurty mentioned this pull request May 18, 2026
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant