@ngockhoale/ukit 3.1.5 → 3.1.6

package/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes to UKit are documented here.
 
+## 3.1.6 - 2026-09-26
+
+- **Decision endpoint URL normalization**: a `UKIT_DECISION_BASE_URL` that already ends in `/v1` (OpenAI convention) no longer produces `…/v1/v1/chat/completions`; the client strips a trailing `/v1` before joining. Applied to `src/decision/client.js` and mirrored in `unic-decision.mjs`.
+
 ## 3.1.5 - 2026-09-26
 
 - **Install/update warns when the decision lane has no API key.** After a project refresh, `ukit install` (and `ukit update`'s refresh) prints a bold-red warning with setup instructions — `ukit decision --base-url <url> --key <key>`, `~/.ukit/storage/gatewayDecision.json`, or `UKIT_DECISION_API_KEY` — whenever no dedicated decision key and no shared UNIC token are resolvable. When a key (or the shared UNIC token fallback) is found, a one-line confirmation is printed instead.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ngockhoale/ukit",
-  "version": "3.1.5",
+  "version": "3.1.6",
   "description": "Install/update an index-first AI workspace for Claude Code, OpenAI Codex and omp (Oh My Pi).",
   "license": "MIT",
   "type": "module",

package/src/decision/client.js

@@ -56,6 +56,15 @@ const DEFAULT_CHECKPOINTS = {
 const FALLBACK_CHECKPOINT = 'unic-decision';
 
 const DEFAULT_TIMEOUT_MS = 5000;
+
+// Join `<baseUrl>/v1/chat/completions` tolerantly: users reasonably paste a
+// base that already ends in /v1 (OpenAI convention). Strip a trailing /v1
+// before appending so `.../v1` and `.../` yield the same endpoint.
+function decisionCompletionsUrl(baseUrl) {
+  const trimmed = String(baseUrl).replace(/\/+$/, '').replace(/\/v1$/i, '');
+  return `${trimmed}/v1/chat/completions`;
+}
+
 const DEFAULT_MAX_RETRIES = 1;
 const DEFAULT_BREAKER = { failureThreshold: 3, cooldownMs: 30_000 };
 
@@ -218,7 +227,7 @@ export function createDecisionClient({
     if (!base?.baseUrl) return null;
     const key = await resolveDecisionApiKey({ projectRoot, homeDir, env });
     return {
-      url: `${String(base.baseUrl).replace(/\/+$/, '')}/v1/chat/completions`,
+      url: `${decisionCompletionsUrl(base.baseUrl)}`,
       headers: buildAuthHeaders(key),
     };
   }

package/template_project/.claude/ukit/index/unic-decision.mjs

@@ -486,6 +486,15 @@ const DEFAULT_CHECKPOINTS = {
 
 const DEFAULT_TIMEOUT_MS = 5000;
 const DEFAULT_MAX_RETRIES = 1;
+
+// Join `<baseUrl>/v1/chat/completions` tolerantly: users reasonably paste a
+// base that already ends in /v1 (OpenAI convention). Strip a trailing /v1
+// before appending so `.../v1` and `.../` yield the same endpoint.
+function decisionCompletionsUrl(baseUrl) {
+  const trimmed = String(baseUrl).replace(/\/+$/, '').replace(/\/v1$/i, '');
+  return `${trimmed}/v1/chat/completions`;
+}
+
 const DEFAULT_STATE_LIMITS = { multilingual: 1024, english: 512 };
 
 function decisionPlaneConfig(config) {
@@ -671,7 +680,7 @@ export async function requestBatch(batch, {
   }
   const key = resolveDecisionApiKey({ projectRoot, homeDir, env });
   const endpoint = {
-    url: `${String(base.baseUrl).replace(/\/+$/, '')}/v1/chat/completions`,
+    url: decisionCompletionsUrl(base.baseUrl),
     headers: buildAuthHeaders(key),
   };
 
@@ -801,7 +810,7 @@ export async function runHealthProbe({
   try {
     const res = await postWithDeadline(
       transport,
-      `${String(base.baseUrl).replace(/\/+$/, '')}/v1/chat/completions`,
+      decisionCompletionsUrl(base.baseUrl),
       buildAuthHeaders(key),
       body,
       timeoutMs,