@hncode/hncode 0.7.0 → 0.7.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hncode/hncode",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "hncode — A cyan-blue TUI AI coding agent with zero runtime dependencies.",
   "type": "module",
   "main": "src/index.js",

package/src/config.js

@@ -370,6 +370,27 @@ const b = baseUrl.replace(/\/+$/, '');
     || root.max_output_size
     || (modelMaxOutput > 0 ? modelMaxOutput : DEFAULTS.maxOutputTokens),
   );
+  // A NUMBER OR NOTHING: an unparseable value falls back to "not set", which means the provider decides.
+  const numOrNull = (v, min, max) => {
+    if (v == null || v === '') return null;
+    const n = Number(v);
+    if (!Number.isFinite(n)) return null;
+    return Math.max(min, Math.min(max, n));
+  };
+  // `sampling` gates the whole group. A provider that rejects `temperature` outright — several refuse it for
+  // reasoning models — needs a switch, not a magic value nobody can find.
+  const samplingOn = process.env.HNCODE_SAMPLING
+    ? !/^(0|false|no|off)$/i.test(process.env.HNCODE_SAMPLING)
+    : root.sampling !== false;
+  const temperature = samplingOn
+    ? numOrNull(process.env.HNCODE_TEMPERATURE ?? root.temperature, 0, 2)
+    : null;
+  const topP = samplingOn
+    ? numOrNull(process.env.HNCODE_TOP_P ?? root.top_p, 0, 1)
+    : null;
+  const presencePenalty = samplingOn
+    ? numOrNull(process.env.HNCODE_PRESENCE_PENALTY ?? root.presence_penalty, -2, 2)
+    : null;
   const reasoning = process.env.HNCODE_REASONING ? /^(1|true|yes)$/i.test(process.env.HNCODE_REASONING) : !!root.reasoning;
   const allowExternal = process.env.HNCODE_ALLOW_EXTERNAL === '1' || !!root.tool_allow_external_paths;
 
@@ -377,6 +398,11 @@ const b = baseUrl.replace(/\/+$/, '');
     model, provider: providerName, innerModel,
     baseUrl: b, endpoint, apiKey, protocol,
     maxContextTokens: maxContext, maxOutputTokens: maxOutput, reasoning,
+    // Sampling. `undefined`/null means "do not send it", which is what lets a provider apply its own
+    // default — and what keeps a rejection from becoming a hard failure.
+    temperature: temperature == null ? undefined : temperature,
+    topP: topP == null ? undefined : topP,
+    presencePenalty: presencePenalty == null ? undefined : presencePenalty,
     workspace: process.env.HNCODE_WORKSPACE || root.workspace || DEFAULTS.workspace,
     allowExternal,
     // Standing permission rules (see permissions.js). `[permissions]` in config.toml:

package/src/llm.js

@@ -233,6 +233,8 @@ function authHeaders(cfg) {
 function buildBody(cfg, messages, streaming = true, opts = {}) {
   const common = { stream: streaming, max_tokens: cfg.maxOutputTokens };
   if (cfg.temperature != null) common.temperature = cfg.temperature;
+  if (cfg.topP != null) common.top_p = cfg.topP;
+  if (cfg.presencePenalty != null) common.presence_penalty = cfg.presencePenalty;
   // Ask for the token accounting on the FINAL stream frame. Without this the
   // OpenAI-compatible stream carries no `usage` at all, so the only token numbers
   // available were our own estimates (see /usage, which divided the transcript's
@@ -265,6 +267,9 @@ function buildBody(cfg, messages, streaming = true, opts = {}) {
       ...common, ...wire,
     };
     if (!body.tools || !body.tools.length) delete body.tools;
+    // NO PRESENCE PENALTY ON THIS PROTOCOL: it has no equivalent, and an unknown field is an error
+    // here rather than something quietly ignored.
+    delete body.presence_penalty;
     // Prompt caching: mark the stable prefix (tools + system + conversation head)
     // so the provider bills it as a cache READ instead of fresh input. The key is
     // the SESSION id, so it stays identical across turns and never thrashes.
@@ -287,6 +292,8 @@ function buildBody(cfg, messages, streaming = true, opts = {}) {
     };
     if (instructions) rbody.instructions = instructions;
     if (cfg.temperature != null) rbody.temperature = cfg.temperature;
+    if (cfg.topP != null) rbody.top_p = cfg.topP;
+    // No presence penalty on this protocol: sending it is an unknown-parameter error, not a hint.
     if (!opts.noTools) {
       const defsR = Object.prototype.hasOwnProperty.call(opts, 'tools')
         ? responsesToolDefs(opts.tools) : responsesToolDefs(llmToolsList);

package/src/settings-schema.js

@@ -125,6 +125,18 @@ export const SETTINGS_SCHEMA = [
     ui: { tab: 'model', group: 'Thinking', label: 'Context window', hint: 'Override the model window when the catalog is wrong; 0 means use it' } },
   { key: 'prompt_cache', ...bool(true),
     ui: { tab: 'model', group: 'Provider', label: 'Prompt caching', hint: 'Ask the provider to cache the request prefix' } },
+  { key: 'sampling', ...bool(true),
+    label: 'Send sampling parameters',
+    help: 'Off sends no temperature/top_p at all, for a provider that rejects them (several do, for reasoning models).' },
+  { key: 'temperature', ...str(''),
+    label: 'Temperature',
+    help: '0 is deterministic, 2 is wild. Empty leaves the provider default. Accepts 0-2 with one decimal.' },
+  { key: 'top_p', ...str(''),
+    label: 'Top P',
+    help: 'Nucleus sampling 0-1. Empty leaves the provider default. Usually set INSTEAD of temperature, not both.' },
+  { key: 'presence_penalty', ...str(''),
+    label: 'Presence penalty',
+    help: 'OpenAI-compatible only: -2 to 2, positive discourages repeating topics. Anthropic has no equivalent and never receives it.' },
 
   // ---- context ------------------------------------------------------------
   { key: 'auto_compact', ...bool(true),

package/src/tui.js

@@ -187,6 +187,7 @@ export const COMMANDS = [
   { name: 'permission', desc: 'Select permission mode', priority: 100 },
   { name: 'settings', aliases: ['config'], desc: 'Browse and change every setting, grouped by area', priority: 100, argumentHint: '[tab|filter]' },
   { name: 'set', desc: 'Set one config value by key, or list what differs from the default', priority: 100, argumentHint: '[<key> [value]]' },
+  { name: 'sampling', desc: 'Show or set temperature, top_p and presence_penalty', priority: 100, argumentHint: '[on|off|temperature <n>|top_p <n>|presence_penalty <n>]' },
   { name: 'plan', desc: 'Toggle plan mode', priority: 100, argumentHint: '[on|off|clear]' },
   { name: 'websearch-api', desc: 'Edit the WebSearch API keys (one per line, tried in order)', priority: 96 },
   { name: 'focus', desc: 'Toggle Focus mode (minimal tools first, full tools after)', priority: 98, argumentHint: '[on|off]' },
@@ -8688,12 +8689,26 @@ export async function dispatch(cmdRaw, arg, state, cfg, session, h, submit, stdo
       if (key === 'calm_mode') cfg.calmMode = v;
       if (key === 'swarm_mode') cfg.swarm = v;
       if (key === 'tool_allow_external_paths') cfg.allowExternal = v;
+      if (key === 'sampling' && !v) {
+        // Off means NONE of them is sent — not their defaults, which is what a provider rejecting them
+        // would receive otherwise.
+        cfg.temperature = undefined; cfg.topP = undefined; cfg.presencePenalty = undefined;
+      }
     } else {
       cfg[key] = raw; cfg.raw[key] = raw;
       if (key === 'compact_threshold') cfg.compactThreshold = Number(raw) / 100;
       if (key === 'compact_keep_ratio') cfg.compactKeepRatio = Number(raw) / 100;
       if (key === 'trim_threshold') cfg.trimThreshold = Number(raw) / 100;
       if (key === 'trim_keep_ratio') cfg.trimKeepRatio = Number(raw) / 100;
+      // SAMPLING: a NUMBER, or gone. The generic writer stores the typed string, and `llm.js` sends it
+      // straight into the request — a provider given `"0.3"` rejects it or ignores it.
+      if (key === 'temperature' || key === 'top_p' || key === 'presence_penalty') {
+        const field = { temperature: 'temperature', top_p: 'topP', presence_penalty: 'presencePenalty' }[key];
+        const n = raw === '' ? null : Number(raw);
+        // Clearing the field removes the parameter, so the provider applies its own default — which is not
+        // the same thing as sending 0.
+        cfg[field] = n == null || !Number.isFinite(n) ? undefined : n;
+      }
       if (key === 'max_context_tokens') cfg.maxContextTokens = Number(raw) || cfg.maxContextTokens;
       if (key === 'provider') cfg.provider = raw;
       if (key === 'model') { cfg.model = raw; state.modelLabel = raw; }
@@ -8942,6 +8957,42 @@ export async function dispatch(cmdRaw, arg, state, cfg, session, h, submit, stdo
     }
 
 
+
+    case 'sampling': {
+      const parts = String(raw || '').trim().split(/\s+/).filter(Boolean);
+      const show = () => {
+        const row = (label, v, note) => `  ${label.padEnd(18)} ${v == null ? '(provider default)' : v}${note ? '   ' + note : ''}`;
+        sayPanel([
+          'Sampling parameters',
+          '',
+          row('send them', cfg.sampling === false ? 'off' : 'on', cfg.sampling === false ? 'nothing is sent' : ''),
+          row('temperature', cfg.temperature),
+          row('top_p', cfg.topP),
+          row('presence_penalty', cfg.presencePenalty, 'OpenAI-compatible only'),
+          '',
+          'Set with /sampling temperature 0.3, /sampling top_p 0.9, /sampling off',
+        ].join('\n'));
+      };
+      if (!parts.length) { show(); return; }
+      if (parts[0] === 'on' || parts[0] === 'off') {
+        applySetting('sampling', parts[0] === 'on' ? 'true' : 'false');
+        app(`Sampling: ${parts[0]}`);
+        show();
+        return;
+      }
+      const key = parts[0] === 'top-p' ? 'top_p' : parts[0] === 'presence-penalty' ? 'presence_penalty' : parts[0];
+      if (!['temperature', 'top_p', 'presence_penalty'].includes(key)) {
+        appErr('Usage: /sampling [on|off] | temperature <0-2> | top_p <0-1> | presence_penalty <-2-2>');
+        return;
+      }
+      if (parts.length === 1) { show(); return; }
+      // An empty value clears it: the field means "let the provider decide", and `0` is a real temperature.
+      const value = parts.slice(1).join(' ').trim();
+      if (!applySetting(key, value)) return;
+      app(value === '' ? `${key} cleared` : `${key} = ${value}`);
+      show();
+      return;
+    }
     case 'set': {
       // `/set <key> <value>` — the direct path, so any setting is reachable without
       // remembering which command owns it. `/set` alone lists what is changed.
@@ -20567,6 +20618,7 @@ function wheelStep(state, family) {
     hub.actions.setConfig = async (key, value) => {
       const ALLOWED = new Set([
         'calm_mode', 'auto_update', 'auto_compact', 'prompt_cache',
+        'temperature', 'top_p', 'presence_penalty', 'sampling',
         'tool_allow_external_paths', 'subagent_model', 'swarm_mode', 'theme',
       ]);
       const k = String(key || '');