Files
oh-my-pi/packages/coding-agent
metaphorics 2a7f716386 fix(coding-agent): route xAI image edits to /v1/images/edits; honor image_size
The xAI image branch on origin/main always posts to /v1/images/generations
and hard-codes `resolution: "1k"`, contradicting two advertised contracts:

  P1: generate_image schema declares `input: z.array(inputImageSchema)`
  globally; resolvedImages was populated for every provider but the xAI
  branch POST body only forwarded text fields. Image-edit and
  multi-reference prompts silently degraded to text-only.

  P2: user-supplied image_size was ignored on the xAI path even though
  every other provider honors it via resolveOpenAIImageSize /
  imageConfig.imageSize.

Fix:

  * Add XAIImageReference and XAIImageRequestBase typed interfaces;
    combine into a discriminated XAIImageRequestBody union with mutually-
    exclusive image / images fields (text-only branch carries
    `image?: never; images?: never`).
  * Route to POST /v1/images/edits when resolvedImages.length > 0; map
    1 source -> `image: {url, type}`, 2-3 sources -> `images: [{url,
    type}, ...]` per docs.x.ai. Cap at 3 with a tool-level error; xAI
    documents that limit.
  * Reuse the existing toDataUrl(InlineImageData) helper for the `url`
    field (data: URIs are accepted alongside public URLs per docs.x.ai).
    `type: "image_url"` is the OpenAI-compat discriminator every official
    xAI code example sends.
  * Add resolveXAIResolution(image_size): map OpenAI-style pixel size to
    xAI's discrete "1k" | "2k" tier. 1024x1024 -> 1k; anything wider ->
    2k. Absent image_size still defaults to "1k", matching hermes-agent
    DEFAULT_RESOLUTION (plugins/image_gen/xai/__init__.py:71).
  * buildXAIEditPayload uses tuple destructure + explicit guard rather
    than `resolvedImages[0]`, staying safe under future
    noUncheckedIndexedAccess: true.

No effect on the OpenAI / OpenAI-codex / antigravity / gemini / openrouter
branches.

Op: correct
Restores: ref:feat/xai-grok-oauth@ecedf7c7e
2026-05-27 15:49:23 +00:00
..
2026-05-27 15:04:44 +02:00
2026-05-09 03:11:18 +02:00

@oh-my-pi/pi-coding-agent

Core implementation package for the omp coding agent in the oh-my-pi monorepo.

For installation, setup, provider configuration, model roles, slash commands, and full CLI reference, see:

Package-specific references:

Memory backends

The agent supports three mutually-exclusive memory backends, selected via the memory.backend setting (Settings → Memory tab, or ~/.omp/config.yml):

  • off (default) — no memory subsystem runs.
  • local — existing rollout-summarisation pipeline; writes memory_summary.md and consolidated artifacts under the agent dir.
  • hindsight — talks to a Hindsight server (Cloud or self-hosted Docker), retains transcripts every Nth user turn, recalls memories on the first turn of a session, and exposes retain, recall, and reflect.

Hindsight quickstart

  1. Run a Hindsight server (Cloud or docker run -p 8888:8888 ghcr.io/vectorize-io/hindsight:latest).
  2. Set memory.backend = "hindsight" and hindsight.apiUrl = "http://localhost:8888" (or your Cloud URL).
  3. Optional environment overrides (env wins over settings):
    • HINDSIGHT_API_URL, HINDSIGHT_API_TOKEN — connection
    • HINDSIGHT_BANK_ID, HINDSIGHT_DYNAMIC_BANK_ID, HINDSIGHT_AGENT_NAME — bank addressing
    • HINDSIGHT_AUTO_RECALL, HINDSIGHT_AUTO_RETAIN, HINDSIGHT_RETAIN_MODE — lifecycle
    • HINDSIGHT_RECALL_BUDGET, HINDSIGHT_RECALL_MAX_TOKENS — recall sizing
    • HINDSIGHT_BANK_MISSION, HINDSIGHT_DEBUG

Switching backends mid-session is honoured on the next system-prompt rebuild and the next /memory slash command. Existing users with memories.enabled = true|false are migrated to memory.backend = "local"|"off" exactly once on first launch.