dsh-ide-git 0.23.0 → 0.25.0

package/src/host/commit-message.js

@@ -0,0 +1,354 @@
+/**
+ * dsh-ide-git host: AI commit messages (issue #6). The llm + sessions services
+ * are injected by the plugin entry; this module never reaches for them itself.
+ */
+import { PanelError, badRequest, cwdOf } from './core.js'
+import { repoRootOf } from './git.js'
+import { summary, diff } from './read.js'
+
+/* ============================== ai commit message ============================== */
+/* Issue #6: write the commit message with the session's own model route. The
+   settings arrive WITH the request (the browser reads them from the same row
+   Config document it renders), so a setting change takes effect on the next
+   click without the host caching anything. */
+const COMMIT_MAX_PROMPT_CHARS = 2000
+const COMMIT_TOTAL_BYTES = 12 * 1024
+const COMMIT_FILE_BYTES = 2048
+const COMMIT_FILE_LINES = 80
+const COMMIT_MAX_FILES = 30
+const COMMIT_BINARY = /\.(png|jpe?g|gif|webp|ico|pdf|zip|gz|tgz|bz2|xz|7z|jar|war|woff2?|ttf|otf|eot|mp[34]|mov|avi|mkv|so|dylib|dll|exe|bin|wasm|class|o|a)$/i
+
+/** One model call's text, plus how it ENDED (a failure arrives as a finish chunk). */
+async function collectCommitStream(stream) {
+  let text = ''
+  let finish = null
+  let sawReasoning = false
+  for await (const chunk of stream) {
+    if (chunk === null || chunk === undefined) continue
+    if (chunk.type === 'text-delta' && typeof chunk.text === 'string') text += chunk.text
+    else if (chunk.type === 'reasoning-delta') sawReasoning = true
+    else if (chunk.type === 'finish') finish = chunk.reason
+  }
+  return { text: text, finish: finish, sawReasoning: sawReasoning }
+}
+
+/** The change set, trimmed to a budget the model can actually be asked about. */
+function commitInputOf(stagedPatch, unstagedPatch) {
+  const patch = (stagedPatch === '' ? '' : stagedPatch) + (unstagedPatch === '' ? '' : (stagedPatch === '' ? '' : '\n') + unstagedPatch)
+  const files = []
+  let bytes = 0
+  let dropped = 0
+  let truncated = false
+  const parts = patch.split(/^diff --git /m).slice(1)
+  for (const part of parts) {
+    const head = /^a\/(.+?) b\//.exec(part)
+    const name = head === null ? '(unknown)' : head[1]
+    if (files.length >= COMMIT_MAX_FILES) { dropped += 1; continue }
+    if (COMMIT_BINARY.test(name)) { dropped += 1; continue }
+    if (/(^|\/)(package-lock\.json|pnpm-lock\.yaml|yarn\.lock|Cargo\.lock|composer\.lock)$/.test(name)) { dropped += 1; continue }
+    let body = 'diff --git ' + part
+    let trimmed = false
+    const lines = body.split('\n')
+    if (lines.length > COMMIT_FILE_LINES) { body = lines.slice(0, COMMIT_FILE_LINES).join('\n'); trimmed = true }
+    if (body.length > COMMIT_FILE_BYTES) { body = body.slice(0, COMMIT_FILE_BYTES); trimmed = true }
+    /* A file the total budget then drops was never sent, so it must not report
+       as "truncated": truncated means "the model saw a shortened version". */
+    if (bytes + body.length > COMMIT_TOTAL_BYTES) { dropped += 1; continue }
+    if (trimmed === true) truncated = true
+    bytes += body.length
+    files.push({ path: name, body: body })
+  }
+  return { files: files, bytes: bytes, dropped: dropped, truncated: truncated }
+}
+
+/** Built-in instruction + the change set + the user's own addition, as DATA. */
+function commitMessagesOf(input, userPrompt) {
+  const system = [
+    'You write git commit messages.',
+    'Reply with the commit message ONLY: no prose, no code fences, no quotes, no explanation.',
+    'First line: a concise subject, at most 72 characters, no trailing period.',
+    'Then one blank line and a short body explaining what changed and why, when it adds value.',
+    'Describe only what the diff shows. Never invent files, features or issue numbers.',
+  ].join('\n')
+  const user = [
+    'Write the commit message for these changes.',
+    '',
+    input.files.map((file) => file.body).join('\n'),
+  ].join('\n')
+  /* The user's text is DATA in the same message, never a system instruction: it
+     steers style (format, language, length) but cannot replace the rules above. */
+  const addition = typeof userPrompt === 'string' && userPrompt.trim() !== ''
+    ? '\n\nAdditional requirements from the user (style only; the rules above still apply):\n' + userPrompt.trim()
+    : ''
+  return { system: system, user: user + addition }
+}
+
+/** Strip what a model habitually wraps a message in; keep the text itself. */
+function cleanupCommitMessage(text) {
+  let out = String(text === null || text === undefined ? '' : text).trim()
+  const fence = /^```[a-zA-Z]*\n([\s\S]*?)\n?```$/.exec(out)
+  if (fence !== null) out = fence[1].trim()
+  if (out.length > 1 && ((out.startsWith('"') && out.endsWith('"')) || (out.startsWith('“') && out.endsWith('”')))) out = out.slice(1, -1).trim()
+  return out
+}
+
+/* ---- ai commit message: the route handler ---- */
+let commitServices = null
+let commitBusy = false
+
+function gatewayError(code, message) {
+  return new PanelError(code, message, 502)
+}
+
+function failureTextOf(failure) {
+  if (failure === null || failure === undefined) return ''
+  if (typeof failure.message === 'string' && failure.message !== '') return failure.message
+  return String(failure.code === undefined ? failure : failure.code)
+}
+
+/** The no-route fallback: the first provider this host serves that lists at
+    least one model, and that provider's first model. Issue #19 — a fresh
+    session (no model picked yet) used to fail the panel button with 'no-route'
+    before the user ever saw a message; a commit message is too small a task to
+    lecture anyone about routing. Errors per provider stay silent on purpose:
+    a provider that fails its catalog simply is not a fallback candidate. */
+async function fallbackRouteOf() {
+  const llm = commitServices.llm
+  for (const row of llm.listProviders()) {
+    try {
+      const models = await llm.listModels(row.id)
+      const model = models[0]
+      if (model !== undefined && model !== null && typeof model.id === 'string' && model.id !== '') {
+        return { provider: row.id, model: model.id }
+      }
+    } catch (error) {
+      void error
+    }
+  }
+  return null
+}
+
+/** Route the call exactly like the session does when it talks to the model. */
+async function commitRouteOf(payload) {
+  const wanted = typeof payload.model === 'string' ? payload.model.trim() : ''
+  if (wanted === '') {
+    const id = typeof payload.sessionId === 'string' ? payload.sessionId.trim() : ''
+    if (id === '') throw new PanelError('no-session', 'no session id: cannot follow the session model route', 502)
+    const list = commitServices.sessions.list()
+    const session = list.filter((row) => String(row.id) === id)[0]
+    if (session === undefined) throw new PanelError('no-session', 'unknown session "' + id + '"', 502)
+    let header = null
+    try { header = session.requestHeader() } catch (error) { header = null; void error }
+    const config = header === null || header === undefined || header.config === undefined || header.config === null ? {} : header.config
+    if (typeof config.provider !== 'string' || typeof config.model !== 'string') {
+      /* Issue #19: 'no-route' is the failure every blank session hit. The
+         served fallback below keeps the button useful on a brand-new session;
+         the panel's model picker (this release) is the user's visible way to
+         steer it, and it drives the SAME session route once a model is picked. */
+      const fallback = await fallbackRouteOf()
+      if (fallback !== null) return { provider: fallback.provider, model: fallback.model, source: 'fallback' }
+      throw new PanelError('no-models', 'no model route and no served models to fall back to; pick a model in the session first', 502)
+    }
+    return { provider: config.provider, model: config.model, source: 'session' }
+  }
+  const slash = wanted.indexOf('/')
+  if (slash <= 0 || slash === wanted.length - 1 || wanted.indexOf('/', slash + 1) >= 0) {
+    throw badRequest('model must look like "provider/model"')
+  }
+  const provider = wanted.slice(0, slash)
+  const model = wanted.slice(slash + 1)
+  const known = commitServices.llm.listProviders().map((row) => row.id)
+  if (known.indexOf(provider) < 0) {
+    throw new PanelError('unknown-provider', 'unknown provider "' + provider + '"; this host serves: ' + known.join(', '))
+  }
+  /* The catalog is advisory (routing accepts unlisted ids), so an empty catalog
+     cannot prove a model wrong — only a non-empty one without this id can. */
+  const models = await commitServices.llm.listModels(provider)
+  if (models.length > 0 && !models.some((row) => row.id === model)) {
+    throw new PanelError('unknown-model', 'unknown model "' + model + '" for ' + provider + '; try: ' + models.map((row) => row.id).slice(0, 8).join(', '))
+  }
+  return { provider: provider, model: model, source: 'setting' }
+}
+
+/** The unset reasoning default: 'off' when this route offers it, '' otherwise.
+    Failures stay silent on purpose — this is a preference, not a contract: an
+    unreachable capability lookup must not turn a working model call into an
+    error; it just leaves the effort to the adapter's own default. */
+async function quietOffEffortOf(route) {
+  try {
+    const llm = commitServices.llm
+    if (typeof llm.resolveModelInfo !== 'function') return ''
+    const info = await llm.resolveModelInfo(route.provider, route.model)
+    const reasoning = info !== null && typeof info === 'object' && info.reasoning !== null && typeof info.reasoning === 'object' ? info.reasoning : null
+    const efforts = reasoning !== null && Array.isArray(reasoning.efforts) ? reasoning.efforts : []
+    return efforts.some((row) => row !== null && typeof row === 'object' && row.id === 'off') ? 'off' : ''
+  } catch (error) {
+    void error
+    return ''
+  }
+}
+
+/** The settings card's model picker: every provider route this host serves and
+    the models each one lists. The catalog is advisory (routing accepts
+    unlisted ids, see commitRouteOf), so a stored hand-typed value stays valid
+    even when it is not in this list. */
+async function commitModels() {
+  if (commitServices === null) {
+    throw new PanelError('no-host-service', 'this host does not serve llm + sessions, so there is no model catalog here', 501)
+  }
+  const providers = []
+  for (const row of commitServices.llm.listProviders()) {
+    /* One provider's discovery failing must not hide the others: its entry
+       ships with an empty model list and the picker still works. */
+    let models = []
+    try { models = await commitServices.llm.listModels(row.id) } catch (error) { models = []; void error }
+    providers.push({
+      id: row.id,
+      name: typeof row.name === 'string' && row.name !== '' ? row.name : row.id,
+      models: models
+        .filter((model) => model !== null && typeof model === 'object' && typeof model.id === 'string' && model.id !== '')
+        .map((model) => ({ id: model.id, name: typeof model.name === 'string' && model.name !== '' ? model.name : model.id })),
+    })
+  }
+  return { providers: providers }
+}
+
+/** The settings card's reasoning picker: the efforts one exact provider/model
+    route accepts. An unreachable capability lookup answers with an EMPTY list
+    (the browser falls back to its text field) instead of an error — the picker
+    is a convenience, not a gate. */
+async function commitEfforts(payload) {
+  if (commitServices === null) {
+    throw new PanelError('no-host-service', 'this host does not serve llm + sessions, so there are no reasoning efforts here', 501)
+  }
+  const wanted = typeof payload.model === 'string' ? payload.model.trim() : ''
+  const slash = wanted.indexOf('/')
+  if (slash <= 0 || slash === wanted.length - 1 || wanted.indexOf('/', slash + 1) >= 0) {
+    throw badRequest('model must look like "provider/model"')
+  }
+  const provider = wanted.slice(0, slash)
+  const model = wanted.slice(slash + 1)
+  const known = commitServices.llm.listProviders().map((row) => row.id)
+  if (known.indexOf(provider) < 0) {
+    throw new PanelError('unknown-provider', 'unknown provider "' + provider + '"; this host serves: ' + known.join(', '))
+  }
+  if (typeof commitServices.llm.resolveModelInfo !== 'function') {
+    return { provider: provider, model: model, efforts: [], defaultEffort: null }
+  }
+  try {
+    const info = await commitServices.llm.resolveModelInfo(provider, model)
+    const reasoning = info !== null && typeof info === 'object' && info.reasoning !== null && typeof info.reasoning === 'object' ? info.reasoning : null
+    const efforts = reasoning !== null && Array.isArray(reasoning.efforts) ? reasoning.efforts : []
+    return {
+      provider: provider,
+      model: model,
+      efforts: efforts
+        .filter((row) => row !== null && typeof row === 'object' && typeof row.id === 'string' && row.id !== '')
+        .map((row) => ({ id: row.id, name: typeof row.name === 'string' && row.name !== '' ? row.name : row.id })),
+      defaultEffort: reasoning !== null && typeof reasoning.defaultEffort === 'string' ? reasoning.defaultEffort : null,
+    }
+  } catch (error) {
+    void error
+    return { provider: provider, model: model, efforts: [], defaultEffort: null }
+  }
+}
+
+async function commitMessage(payload) {
+  if (commitServices === null) {
+    throw new PanelError('no-host-service', 'this host does not serve llm + sessions, so the message cannot be written here', 501)
+  }
+  /* The single-flight window opens BEFORE the first await and covers the route
+     lookup, the git reads and the model call. Claiming it only around the stream
+     let four simultaneous clicks all pass the check and spend four model calls
+     (measured: 4x200 on one click burst), which is four times the user's quota. */
+  if (commitBusy) throw new PanelError('busy', 'a commit message is already being written', 409)
+  commitBusy = true
+  try {
+    const cwd = cwdOf(payload)
+    /* The picked repository travels as `repoRoot`, and every other git call
+       threads it through `repoRootOf`. Resolving the root from `cwd` alone made
+       this the one route that answered not-a-repo in a workspace whose own root
+       is not a repository — a workspace holding several sibling repos, which is
+       exactly what the repo picker exists for. */
+    const repoRoot = await repoRootOf(cwd, payload)
+    const scope = { cwd: cwd, repoRoot: repoRoot }
+    const route = await commitRouteOf(payload)
+    const sums = await summary(scope)
+    const stagedPatch = sums.changes.staged.length > 0 ? (await diff(Object.assign({}, scope, { staged: true }))).patch : ''
+    const input = commitInputOf(stagedPatch, (await diff(scope)).patch)
+    if (input.files.length === 0) {
+      if (input.dropped > 0) {
+        throw new PanelError('only-ignored-changes', 'the only changes are lock files or binary files, which are not sent to the model; stage a source change first')
+      }
+      throw badRequest('nothing to describe: stage or edit something first')
+    }
+    const asked = typeof payload.prompt === 'string' ? payload.prompt : ''
+    const prompt = asked.slice(0, COMMIT_MAX_PROMPT_CHARS)
+    const reasoning = typeof payload.reasoningEffort === 'string' ? payload.reasoningEffort.trim() : ''
+    const spoken = commitMessagesOf(input, prompt)
+    const options = {
+      provider: route.provider,
+      model: route.model,
+      system: spoken.system,
+      messages: [{ role: 'user', content: [{ type: 'text', text: spoken.user }] }],
+      temperature: 0.2,
+    }
+    /* No maxTokens on purpose: omitted, the runtime materializes the selected
+       model's own configured output cap (resolveModelInfo().defaultMaxTokens) —
+       exactly what a session call gets. The hardcoded 512 here was half of
+       issue #7: with thinking enabled, max_tokens is the budget for reasoning
+       AND message together, so 512 died inside the reasoning phase before one
+       word of the message existed. */
+    /* The other half of issue #7: an UNSET effort is not a neutral "model
+       default" — the DeepSeek adapter resolves "unspecified" to reasoning ON at
+       "high". A commit message is a formatting task, so the unset default now
+       asks for "off" whenever the route offers it. The id is read from the
+       route's own efforts table (resolveModelInfo), never assumed: the runtime
+       rejects an effort the route does not list, so a blind 'off' would trade
+       one loud failure for another. The user's explicit choice always wins. */
+    const effectiveReasoning = reasoning !== '' ? reasoning : await quietOffEffortOf(route)
+    if (effectiveReasoning !== '') options.reasoningEffort = effectiveReasoning
+    if (typeof payload.sessionId === 'string' && payload.sessionId !== '') options.sessionId = payload.sessionId
+    const started = Date.now()
+    const collected = await collectCommitStream(commitServices.llm.stream(options))
+    /* An adapter failure is a terminal finish, not a throw: a missing check here
+       would turn "the call failed" into "the model said nothing" (假成功). */
+    const finish = collected.finish
+    if (finish === null || finish === undefined) {
+      throw gatewayError('llm-no-finish', 'the model stream ended without a finish reason; treat the result as unknown, not as success')
+    }
+    if (finish.kind === 'error') throw gatewayError('llm-error', failureTextOf(finish.failure))
+    if (finish.kind === 'aborted') throw gatewayError('llm-aborted', 'the call was aborted: ' + failureTextOf(finish.failure))
+    if (finish.kind !== 'stop') {
+      /* The max-tokens + no-text + reasoning-delta shape is the exact signature
+         of issue #7: the budget (reasoning included) ran out mid-thought. Say
+         what happened and what to do, instead of a bare finish kind. */
+      if (finish.kind === 'max-tokens' && collected.text.trim() === '' && collected.sawReasoning === true) {
+        throw gatewayError('llm-unfinished', 'the model stopped with "max-tokens" before writing the message: its reasoning spent the whole output budget; lower the reasoning effort in the plugin settings or pick a lighter model')
+      }
+      throw gatewayError('llm-unfinished', 'the model stopped with "' + String(finish.kind) + '"; the message may be incomplete')
+    }
+    const message = cleanupCommitMessage(collected.text)
+    if (message === '') throw gatewayError('empty-output', 'the model returned no text')
+    return {
+      message: message,
+      provider: route.provider,
+      model: route.model,
+      routeSource: route.source,
+      reasoningEffort: effectiveReasoning === '' ? null : effectiveReasoning,
+      elapsedMs: Date.now() - started,
+      input: { files: input.files.length, bytes: input.bytes, dropped: input.dropped, truncated: input.truncated },
+      promptTruncated: asked.length > COMMIT_MAX_PROMPT_CHARS,
+    }
+  } finally {
+    commitBusy = false
+  }
+}
+
+export function setCommitServices(services) {
+
+  commitServices = services
+
+}
+
+export { commitMessage, commitModels, commitEfforts }

package/src/host/config.js

@@ -0,0 +1,71 @@
+/** dsh-ide-git host: row Config (rail switches + commit-message settings). */
+
+/* Row Config = the plugin's settings surface on dsh >= 0.1.7: the host persists
+   these values under the loader row id ('ide-git', see cordis.patch.yml) and the
+   browser reads them through ctx.configForms, so the settings card on the
+   plugin's page and the action rail read ONE document. The rail's edit is one
+   switch per action (`railRefresh` … `railPush`); a missing field means "default"
+   (the action is shown), which is what keeps an action added by a later version
+   visible on an older profile.
+   `@deepseek-ai/schemastery` is resolved LAZILY: it is a host-provided module
+   (every sibling plugin declares its row Config the same way), and on a host that
+   cannot resolve it the row config is passed through unvalidated instead of the
+   whole row disappearing — so the browser keeps its localStorage home there. */
+let Schema = null
+try {
+  Schema = (await import('@deepseek-ai/schemastery')).default
+} catch (error) {
+  console.warn('[dsh-ide-git] @deepseek-ai/schemastery is not resolvable here; the row Config surface is absent'
+    + ' (the browser falls back to its localStorage rail config): ' + (error && error.message ? error.message : String(error)))
+}
+
+/** Rail action ids, in rail order. MUST mirror RAIL_SPECS in src/client.js. */
+const RAIL_CONFIG_IDS = [
+  'refresh', 'tree', 'worktrees', 'float', 'split', 'fullscreen', 'newBranch', 'checkout', 'delete',
+  'compare', 'diff', 'stash', 'tag', 'favorite', 'fetch', 'pull', 'push',
+]
+
+/** Volatile refs (dsh >= 0.1.7 settings contract) or plain values on older hosts. */
+function live(schema) {
+  return typeof schema.volatile === 'function' ? schema.volatile() : schema
+}
+
+function railConfigSchema(schema) {
+  const shape = {}
+  for (const id of RAIL_CONFIG_IDS) {
+    shape['rail' + id.charAt(0).toUpperCase() + id.slice(1)] = live(schema.boolean().default(true))
+  }
+  /* AI commit message (issue #6), the three settings the user asked for. Empty
+     string means "follow whatever the session is using" / "no extra prompt", so
+     an untouched profile behaves exactly like the feature's default. */
+  shape.commitModel = live(schema.string().default(''))
+  shape.commitReasoning = live(schema.string().default(''))
+  shape.commitPrompt = live(schema.string().default(''))
+  /* Conversation-area view tab (issue #8): on by default, switchable off from
+     the plugin settings card. false means "do not register the
+     conversation.view seat at all" — the sidebar and dock seats are not
+     affected; a missing field means the default (shown). */
+  shape.conversationTab = live(schema.boolean().default(true))
+  /* Immersive conversation seat (issue #8 round 3): on by default. false means
+     the seat falls back to the pin-to-scrollport layout with a visible
+     composer; a missing field means the default (on). NOTE: the field MUST be
+     declared here — the Host rejects row-Config writes for paths the schema
+     does not mark volatile ("Config field … is not volatile"), and the settings
+     switch then rolls back silently. */
+  shape.immersive = live(schema.boolean().default(true))
+  /* Composer status pill (v0.19.0): the settings card's "输入框 Git 状态" switch
+     and the pill itself read this field. v0.19.0-v0.19.6 shipped the switch and
+     the reader WITHOUT this declaration, so the Host rejected every write with
+     `Config field "composerStatus" is not volatile` — the card rolled the switch
+     back and said "设置没写进去". Any new row-Config field the client writes MUST
+     be declared (and volatile) here, or the write is refused before it lands. */
+  shape.composerStatus = live(schema.boolean().default(true))
+  /* Inline diff preview (issue #22): ON by default so the panel's own diff
+     pane — with its split / wrap switches — stays reachable even when
+     dsh-better-sidebar or the native preview seat is installed. OFF hands the
+     preview back to those external tabs, exactly the pre-0.23 behaviour. */
+  shape.inlineDiff = live(schema.boolean().default(true))
+  return schema.object(shape)
+}
+
+export const Config = Schema === null ? undefined : railConfigSchema(Schema)

package/src/host/core.js

@@ -0,0 +1,121 @@
+/**
+ * dsh-ide-git host: shared errors, payload validation and the per-repository
+ * write queue. No git process runs here; see ./git.js.
+ */
+import path from 'node:path'
+
+/* ============================== errors ============================== */
+
+class PanelError extends Error {
+  constructor(code, message, status = 400) {
+    super(message)
+    this.code = code
+    this.status = status
+  }
+}
+
+function badRequest(message) {
+  return new PanelError('bad-request', message, 400)
+}
+
+function messageOf(error) {
+  return error instanceof Error ? error.message : String(error)
+}
+
+/* ============================== payload helpers ============================== */
+
+function requireAbsolute(value, label) {
+  if (typeof value !== 'string' || value.trim() === '') throw badRequest(label + ' is required')
+  if (!path.isAbsolute(value)) throw badRequest(label + ' must be an absolute path')
+  return path.resolve(value)
+}
+
+/** A ref/branch/tag name: no whitespace, no leading dash, no control characters. */
+function requireRef(value, label) {
+  if (typeof value !== 'string' || value.trim() === '') throw badRequest(label + ' is required')
+  const name = value.trim()
+  if (name.startsWith('-')) throw badRequest(label + ' must not start with "-"')
+  if (/[\u0000-\u001f\u007f\s]/.test(name)) throw badRequest(label + ' contains invalid characters')
+  return name
+}
+
+function optionalRevs(value, label) {
+  if (value === undefined || value === null) return []
+  if (!Array.isArray(value)) throw badRequest(label + ' must be an array')
+  return value.map((entry) => requireRef(entry, label + ' entry'))
+}
+
+function requireMessage(value) {
+  if (typeof value !== 'string' || value.trim() === '') throw badRequest('message is required')
+  return value
+}
+
+function cwdOf(payload) {
+  return requireAbsolute(payload === undefined ? undefined : payload.cwd, 'cwd')
+}
+
+/* ============================== safety ============================== */
+
+/**
+ * Write operations on one repository run one at a time. The client greys its
+ * actions out while a request is in flight, but a stale tab, a second window or
+ * a double click could still start two writers, and git dislikes concurrent
+ * index writers — so the host serialises writes per repository.
+ */
+const repoQueues = new Map()
+
+function withRepoLock(key, work) {
+  const previous = repoQueues.get(key) === undefined ? Promise.resolve() : repoQueues.get(key)
+  const next = previous.then(work, work)
+  const settled = next.then(() => undefined, () => undefined)
+  repoQueues.set(key, settled)
+  settled.then(() => { if (repoQueues.get(key) === settled) repoQueues.delete(key) })
+  return next
+}
+
+const WRITE_METHODS = new Set([
+  'stage', 'unstage', 'discard', 'commit', 'checkout', 'branchCreate', 'branchRename', 'branch-upstream',
+  'branchDelete', 'worktreeAdd', 'worktreeRemove', 'worktreePrune', 'merge', 'rebase', 'cherryPick', 'revert', 'reset', 'fetch', 'pull',
+  'push', 'stashPush', 'stashApply', 'stashDrop', 'stashClear', 'tagCreate', 'tagDelete', 'tagDeleteRemote', 'pushTags', 'clone', 'remoteAdd', 'remoteRemove', 'branchDeleteRemote', 'undoApply',
+])
+
+function lockKeyOf(payload) {
+  if (typeof payload.repoRoot === 'string' && payload.repoRoot !== '') return payload.repoRoot
+  if (typeof payload.cwd === 'string' && payload.cwd !== '') return payload.cwd
+  return 'global'
+}
+
+/* ============================== path helpers ============================== */
+
+function requireRelativePath(value) {
+  if (typeof value !== 'string' || value.trim() === '') throw badRequest('path is required')
+  const trimmed = value.trim()
+  if (path.isAbsolute(trimmed)) return trimmed
+  if (trimmed.split('/').includes('..')) throw badRequest('path must stay inside the repository')
+  return trimmed
+}
+
+function optionalPaths(payload, label) {
+  if (payload === undefined || payload[label] === undefined || payload[label] === null) return []
+  if (!Array.isArray(payload[label])) throw badRequest(label + ' must be an array')
+  if (payload[label].length > 5000) throw badRequest(label + ' has too many entries')
+  return payload[label].map((entry) => requireRelativePath(entry))
+}
+
+function clampInt(value, min, max, fallback) {
+  const parsed = typeof value === 'number' ? value : Number.parseInt(value, 10)
+  if (!Number.isFinite(parsed)) return fallback
+  return Math.min(max, Math.max(min, Math.trunc(parsed)))
+}
+
+export {
+
+  PanelError, badRequest, messageOf,
+
+  requireAbsolute, requireRef, optionalRevs, requireMessage, cwdOf,
+
+  requireRelativePath, optionalPaths, clampInt,
+
+  withRepoLock, WRITE_METHODS, lockKeyOf,
+
+}