Skip to content

Support IBM Bob provider#303

Closed
ozymandiashh wants to merge 1 commit into
getagentseal:mainfrom
ozymandiashh:feat/ibm-bob-provider
Closed

Support IBM Bob provider#303
ozymandiashh wants to merge 1 commit into
getagentseal:mainfrom
ozymandiashh:feat/ibm-bob-provider

Conversation

@ozymandiashh
Copy link
Copy Markdown
Contributor

Summary

  • add an eager ibm-bob provider that discovers IBM Bob IDE task folders from both the GA IBM Bob app data directory and the preview-era Bob-IDE directory
  • reuse the shared Cline-family task parser for Bob-compatible ui_messages.json records, while adding provider-specific multi-base discovery and ibm-bob-auto fallback pricing/display
  • parse api_conversation_history.json model tags when present, use Bob-reported costs when available, and otherwise calculate cost through the configured model pricing fallback
  • wire IBM Bob into provider registry, dashboard colors/display names, macOS provider tabs, package keywords, README/provider docs, architecture docs, changelog, and provider icon assets
  • add focused tests for discovery, missing ui_messages.json, token/cost parsing, model fallback, deduplication, and metadata/display names

Closes #248.

Notes

  • This intentionally supports the IBM Bob IDE task-history surface: User/globalStorage/ibm.bob-code/tasks/<task-id>/.
  • IBM Bob Shell checkpoint data under ~/.bob is left out of scope until there is a stable usage schema fixture; parsing it now would risk reporting misleading usage.
  • Existing Roo Code and KiloCode behavior is preserved because the shared parser keeps the same public call shape and defaults the fallback model to cline-auto.

Validation

  • npm run build
  • npx vitest run tests/providers/ibm-bob.test.ts tests/providers/roo-code.test.ts tests/providers/kilo-code.test.ts tests/provider-registry.test.ts (4 files / 34 tests)
  • npx vitest run (48 files / 668 tests)
  • swift build
  • git diff --check
  • Argus local reviewer: PASS
  • Claude Opus 4.7 effort max: PASS
  • Gemini 3.1 Pro Preview: PASS

Known Local Test Note

  • swift test still fails on the existing unrelated local Swift toolchain issue in mac/Tests/CodeBurnMenubarTests/CapacityEstimatorTests.swift:2:8 (no such module Testing).

@AgentSeal AgentSeal added needs-testing needs-validation PR requires validation against real-world usage before review and removed needs-testing labels May 12, 2026
@AgentSeal
Copy link
Copy Markdown
Collaborator

Closing in favor of #316, which includes your IBM Bob provider implementation plus workspace extraction for per-project naming.

Thanks for the contribution @ozymandiashh — your provider discovery paths and shared parser reuse were solid. We added workspace extraction on top so sessions show actual folder names instead of always "IBM Bob".

@AgentSeal AgentSeal closed this May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-validation PR requires validation against real-world usage before review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: IBM Bob provider support

2 participants