Skip to content

Mod/mod 2 OpenAI compat adapter - #17

Merged
SaketMunda merged 4 commits into
masterfrom
mod/MOD-2-openai-compat-adapter
Oct 9, 2026
Merged

SaketMunda merged 4 commits into
masterfrom
mod/MOD-2-openai-compat-adapter

Conversation

@SaketMunda

Copy link
Copy Markdown
Owner

Module

Brief: roadmap/modules/<ID>-*.md

What changed

Definition of Done

  • Every item in the brief's Acceptance criteria is met and demonstrated (test or recorded manual check).
  • scripts/verify.sh is green (typecheck, lint, unit tests).
  • New logic in packages/* has unit tests (target ≥80% line coverage on new files). UI logic (state, reducers, parsers) is tested. Pure presentation isn't required to be.
  • No any / as unknown as at package boundaries. Every external input is validated with zod.
  • No new network destinations except ones the user configured. No telemetry.
  • Errors are handled: user-facing failures show an actionable message, and nothing fails silently.
  • Brief updated: Status, Handoff notes (what was built, deviations, known gaps, follow-ups).
  • PR opened from mod/<ID>-<slug>, description links the brief, CI green.

Acceptance criteria evidence

# Criterion Evidence

SaketMunda and others added 4 commits October 8, 2026 17:34
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- OpenAICompatAdapter: SSE chat, tool calls accumulated by index/id, usage via
  stream_options, quirks table + one-shot adaptive retry
- Ollama on the native API (/v1 ignores num_ctx): /api/show probe, explicit
  num_ctx, /api/tags discovery, 'Is Ollama running?' and pull hints
- Text-based tool protocol for models without native tool calls
- Registry: ProviderDefinition.defaultBaseUrl; discover probes a loopback default
- Recorded fixtures + contract suite for both providers

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…-port test

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@SaketMunda
SaketMunda merged commit 8f03634 into master Oct 9, 2026
1 check passed
@SaketMunda
SaketMunda deleted the mod/MOD-2-openai-compat-adapter branch October 9, 2026 05:42
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