Skip to content

docs(skills): refresh ide-model-adapters after MOD-2 - #18

Merged
SaketMunda merged 1 commit into
masterfrom
plan/model-adapters-skill
Oct 9, 2026
Merged

SaketMunda merged 1 commit into
masterfrom
plan/model-adapters-skill

Conversation

@SaketMunda

Copy link
Copy Markdown
Owner

Ollama on the native API and num_ctx rules, quirks, defaultBaseUrl discovery, text tool protocol, fixture/live-test conventions, new gotchas.

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

Ollama on the native API and num_ctx rules, quirks, defaultBaseUrl discovery,
text tool protocol, fixture/live-test conventions, new gotchas.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@SaketMunda
SaketMunda merged commit 1406810 into master Oct 9, 2026
1 check passed
@SaketMunda
SaketMunda deleted the plan/model-adapters-skill branch October 9, 2026 06:38
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