/${cmd.command} 123`);
+ return null;
+ }
+ return cmd.prNumber;
+ };
+ switch (cmd.command) {
+ case 'start':
+ case 'help':
+ await this.telegram.sendMessage(HELP);
+ return;
+ case 'status':
+ await this.sendStatus();
+ return;
+ case 'scan': {
+ const r = await this.scanOnce('command');
+ await this.telegram.sendMessage(
+ `Scanned: ${r.queued.length ? `queued ${r.queued.map((n) => `#${n}`).join(', ')}` : 'nothing new'}${
+ r.closed.length ? `; closed ${r.closed.map((n) => `#${n}`).join(', ')}` : ''
+ }.`
+ );
+ return;
+ }
+ case 'review':
+ case 'rescan': {
+ const n = need();
+ if (n === null) return;
+ await this.queueByCommand(n);
+ return;
+ }
+ case 'report': {
+ const n = need();
+ if (n !== null) await this.sendReport(n);
+ return;
+ }
+ case 'summary': {
+ const n = need();
+ if (n === null) return;
+ const rec = this.store.pr(n);
+ if (!rec?.report) await this.telegram.sendMessage(`No review of #${n} yet.`);
+ else await this.sendSummary(rec);
+ return;
+ }
+ case 'draft': {
+ const n = need();
+ if (n !== null) await this.sendDraft(n);
+ return;
+ }
+ case 'post': {
+ const n = need();
+ if (n !== null) await this.startConfirm('post', n);
+ return;
+ }
+ case 'merge': {
+ const n = need();
+ if (n !== null) await this.startConfirm('merge', n);
+ return;
+ }
+ case 'close': {
+ const n = need();
+ if (n === null) return;
+ if (cmd.rest) await this.startConfirm('close', n, cmd.rest);
+ else await this.askCloseReason(n);
+ return;
+ }
+ case 'approve':
+ case 'approveci': {
+ const n = need();
+ if (n !== null) await this.approveCi(n);
+ return;
+ }
+ case 'ask': {
+ const n = need();
+ if (n === null) return;
+ if (!cmd.rest) {
+ await this.telegram.sendMessage(`Ask what? /ask ${n} does this handle X?`);
+ return;
+ }
+ this.startFollowup(n, cmd.rest, msg.message_id);
+ return;
+ }
+ case 'pause':
+ this.store.state.paused = true;
+ this.store.save();
+ await this.telegram.sendMessage('⏸ Auto-review paused. /review N still works; /resume to continue.');
+ return;
+ case 'resume': {
+ this.store.state.paused = false;
+ this.store.save();
+ const r = await this.scanOnce('resume');
+ await this.telegram.sendMessage(
+ `▶️ Auto-review resumed${r.queued.length ? `; queued ${r.queued.map((n) => `#${n}`).join(', ')}` : ''}.`
+ );
+ return;
+ }
+ default:
+ await this.telegram.sendMessage(`Unknown command /${escapeHtml(cmd.command)}. See /help.`);
+ }
+ }
+
+ private async handleCallback(cb: TelegramCallbackQuery): Promise/close ${n} reason.`
+ );
+ this.store.state.reasonPrompts[String(id)] = n;
+ this.store.save();
+ }
+
+ private async approveCi(n: number): Promise${escapeHtml(f.file)}${f.line ? `:${f.line}` : ''}` : '';
+ return `${SEVERITY_ICON[f.severity]} ${escapeHtml(f.title)}${where}`;
+}
+
+function checksLine(checks: CheckResult[]): string {
+ if (!checks.length) return '';
+ const parts = checks.map((c) => {
+ const icon = c.result === 'pass' ? '✅' : c.result === 'fail' ? '❌' : '⏭';
+ return `${escapeHtml(c.name)} ${icon}`;
+ });
+ return `Checks: ${parts.join(' · ')}`;
+}
+
+function truncate(text: string, max: number): string {
+ if (text.length <= max) return text;
+ return text.slice(0, Math.max(0, max - 1)).trimEnd() + '…';
+}
+
+export interface SummaryMeta {
+ ci: CiState;
+ /** Time the review took, for the footer. */
+ durationMin?: number;
+}
+
+/** The message the maintainer reads on the phone. HTML parse mode. */
+export function formatTelegramSummary(pr: PrSummary, report: ReviewReport, meta: SummaryMeta): string {
+ const header =
+ `🔍 PR #${pr.number} · ${escapeHtml(truncate(pr.title, 120))}\n` +
+ `by ${escapeHtml(pr.author)} · +${pr.additions}/−${pr.deletions} · ${pr.changedFiles} files · ${CI_LABEL[meta.ci]} · ${
+ pr.mergeable === 'CONFLICTING'
+ ? 'conflicts ⚠️'
+ : pr.mergeable === 'MERGEABLE'
+ ? 'mergeable'
+ : 'mergeability unknown'
+ }${pr.isDraft ? ' · draft' : ''}\n` +
+ `${escapeHtml(pr.url)}\n`;
+ const verdict = `\n${VERDICT_LABEL[report.verdict]} (confidence ${report.confidence}${report.scope === 'mixed' ? ', mixed scope' : ''})\n`;
+ const summary = report.summary ? `\n${escapeHtml(report.summary)}\n` : '';
+
+ const counts = countBySeverity(report.findings);
+ const countStr = SEVERITY_ORDER.filter((s) => counts[s] > 0)
+ .map((s) => `${counts[s]} ${s}${counts[s] === 1 ? '' : 's'}`)
+ .join(', ');
+ const findingsHeader = report.findings.length ? `\nFindings (${countStr}):\n` : '\nFindings: none\n';
+
+ const checks = checksLine(report.checks);
+ const recommendation = report.recommendation ? `\nRecommendation: ${escapeHtml(report.recommendation)}\n` : '';
+ const footer = meta.durationMin !== undefined ? `\nreview took ${meta.durationMin} min` : '';
+
+ const fixed = header + verdict + summary + findingsHeader;
+ const tail = (checks ? `\n${checks}\n` : '') + recommendation + footer;
+ let budget = SUMMARY_BUDGET - fixed.length - tail.length;
+
+ const lines: string[] = [];
+ let shown = 0;
+ for (const f of report.findings) {
+ const line = findingLine(f) + '\n';
+ if (line.length > budget) break;
+ lines.push(line);
+ budget -= line.length;
+ shown++;
+ }
+ const hidden = report.findings.length - shown;
+ const more = hidden > 0 ? `… ${hidden} more in the full report\n` : '';
+ return fixed + lines.join('') + more + tail;
+}
+
+export function formatReviewFailure(
+ pr: PickAdd Case
+
+
@@ -2644,6 +2651,7 @@
Runs this case in a hardened, isolated container. The base image is built automatically on first use. Docker/Podman must be installed.
+ Already have a container running? Codeman only runs docker exec into it and never touches its lifecycle.
Run the case inside a container: one per case, shared by all its sessions.
+
+
+ On: Codeman only runs docker exec into a container you already built and run — it never creates, starts, stops or removes it. The CLIs must already be installed and logged in inside it.
+
+
+
+
+
+ Pick from the running containers or type a name.
+
+
+
+
+
+
+
+ A path that already exists inside the container. Adoption mounts nothing, so this need not match the host workspace path.
+
@@ -2854,7 +2880,10 @@
-
+
+
+
+
Absolute HOST directory, bind-mounted into the container. Codeman scaffolds CLAUDE.md + hooks into it.
@@ -2862,12 +2891,12 @@
A reusable docker host profile. Reuse the same ID across cases to share settings.
-
+
Build it once with
- node scripts/build-agent-image.mjs. Contains node + claude/codex/gemini/opencode/agy/pi/grok/dsh + tmux.
+
-
+
Advanced container settings
diff --git a/src/web/public/keyboard-accessory.js b/src/web/public/keyboard-accessory.js
index 35b67c3e..8cdc16ba 100644
--- a/src/web/public/keyboard-accessory.js
+++ b/src/web/public/keyboard-accessory.js
@@ -185,9 +185,13 @@ const PathPicker = {
if (this._options.sessionId) params.set('sessionId', this._options.sessionId);
if (this._showHidden) params.set('showHidden', 'true');
try {
- const response = await fetch(`/api/filesystem/browse?${params.toString()}`);
- const result = await response.json();
- if (!response.ok || !result.success) throw new Error(result.error || 'Failed to browse this folder');
+ // A caller may supply its own source (the container-workdir picker browses
+ // INSIDE a container, which the host filesystem endpoint cannot answer).
+ // It returns the same shape, so everything below is unchanged.
+ const result = this._options.fetchListing
+ ? await this._options.fetchListing(path)
+ : await (await fetch(`/api/filesystem/browse?${params.toString()}`)).json();
+ if (!result?.success) throw new Error(result?.error || 'Failed to browse this folder');
if (!this.overlay || loadSequence !== this._loadSequence) return;
this.render(result.data);
} catch (error) {
@@ -304,7 +308,7 @@ const PathPicker = {
// A hidden file is only reachable while the toggle is on, and the preview
// endpoint re-resolves the path independently, so it needs the flag too.
if (this._showHidden) params.set('showHidden', 'true');
- const previewUrl = `/api/filesystem/preview?${params.toString()}`;
+ const previewUrl = (window.CodemanBase?.url || ((p) => p))(`/api/filesystem/preview?${params.toString()}`);
const overlay = document.createElement('div');
overlay.className = 'path-preview-overlay';
diff --git a/src/web/public/manifest.json b/src/web/public/manifest.json
index 79b6ea64..8ce935b0 100644
--- a/src/web/public/manifest.json
+++ b/src/web/public/manifest.json
@@ -2,7 +2,8 @@
"name": "Codeman",
"short_name": "Codeman",
"description": "Claude Code session manager",
- "start_url": "/",
+ "start_url": "./",
+ "scope": "./",
"display": "standalone",
"orientation": "any",
"background_color": "#0a0a0a",
diff --git a/src/web/public/mobile.css b/src/web/public/mobile.css
index c9b0838d..b313f097 100644
--- a/src/web/public/mobile.css
+++ b/src/web/public/mobile.css
@@ -3333,8 +3333,10 @@ html:is([data-skin="paper-gray"], [data-skin="solarized-light"], [data-skin="cat
recessed tray and matching pill geometry instead of reading as a fat
accent pill parked beside a stray × glyph. Tray colors come from skin
tokens, never a hardcoded black alpha, or the light skins get a grey slab.
- `:has()` keeps the tray off the two sheets that carry a lone × (Session
- Options and Add Case save from inside their own forms). */
+ `:has()` keeps the tray off Session Options, the one sheet left carrying a
+ lone × because it saves from inside its own per-section forms. Add Case
+ now has a header save of its own (its footer is hidden below 860px, so
+ that button is the only way to submit it there), and picks up the tray. */
:is(#appSettingsModal, #sessionOptionsModal, #createCaseModal) .set-head-actions:has(.set-head-save) {
padding: 3px;
border: 1px solid var(--border);
@@ -3353,6 +3355,15 @@ html:is([data-skin="paper-gray"], [data-skin="solarized-light"], [data-skin="cat
font-size: 0.86rem;
}
+ /* Add Case's pending state has to show on the header button: below 860px it is
+ the only submit control (the footer is hidden), and the #caseModalSubmit
+ .loading rule in the phone block dims a button nobody can see. Measured at
+ 390px before this: header opacity 1 for the whole clone, hidden footer 0.6. */
+ #createCaseModal .set-head-save.loading {
+ opacity: 0.6;
+ pointer-events: none;
+ }
+
:is(#appSettingsModal, #sessionOptionsModal, #createCaseModal) .set-head-actions .modal-close {
width: 36px;
height: 36px;
diff --git a/src/web/public/notification-manager.js b/src/web/public/notification-manager.js
index 2d7f4d9f..c1f7f665 100644
--- a/src/web/public/notification-manager.js
+++ b/src/web/public/notification-manager.js
@@ -390,7 +390,7 @@ class NotificationManager {
const notif = new Notification(`${this.originalTitle}: ${localizedTitle}`, {
body: localizedBody,
tag, // Groups same-tag notifications
- icon: '/favicon.ico',
+ icon: (window.CodemanBase?.url || ((p) => p))('/favicon.ico'),
silent: true, // We handle audio ourselves
});
diff --git a/src/web/public/panels-ui.js b/src/web/public/panels-ui.js
index c035272a..e1f0e8e1 100644
--- a/src/web/public/panels-ui.js
+++ b/src/web/public/panels-ui.js
@@ -3430,7 +3430,7 @@ Object.assign(CodemanApp.prototype, {
const nameClass = isDir ? 'file-tree-name directory' : 'file-tree-name';
const downloadBtn = !isDir
- ? `⬇`
+ ? `⬇`
: '';
html.push(`
@@ -3605,7 +3605,7 @@ Object.assign(CodemanApp.prototype, {
: '';
const nameClass = isDir ? 'file-tree-name directory' : 'file-tree-name';
const downloadBtn = !isDir
- ? `⬇`
+ ? `⬇`
: '';
return `
@@ -4004,18 +4004,20 @@ Object.assign(CodemanApp.prototype, {
// (html/htm arrive as a download there by design — file-raw serves them
// attachment-only so widening READ never widens RUN.)
const officeDoc = ext === 'docx' || ext === 'pptx';
- this.filePreviewDetachUrl = attachmentId
- ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/${officeDoc ? 'preview' : 'raw'}`
- : officeDoc
- ? `/api/sessions/${sessionId}/file-preview?path=${encodeURIComponent(filePath)}`
- : `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`;
+ this.filePreviewDetachUrl = CodemanBase.url(
+ attachmentId
+ ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/${officeDoc ? 'preview' : 'raw'}`
+ : officeDoc
+ ? `/api/sessions/${sessionId}/file-preview?path=${encodeURIComponent(filePath)}`
+ : `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`
+ );
if (detachBtn) detachBtn.hidden = false;
// Registered attachment: render straight from its by-id routes — images and
// PDFs inline, Office docs via the server-converted PDF preview, text fetched
// raw. (Workspace-path previews fall through to the file-content endpoint.)
if (attachmentId) {
- const base = `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}`;
+ const base = CodemanBase.url(`/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}`);
const IMAGE_EXTS = new Set(['png', 'jpg', 'jpeg', 'gif', 'webp', 'bmp', 'svg']);
// VIDEO/AUDIO mirror VIDEO_ATTACHMENT_EXTENSIONS/AUDIO_ATTACHMENT_EXTENSIONS
// (src/attachment-registry.ts, the single source); the frontend cannot import
@@ -4073,13 +4075,13 @@ Object.assign(CodemanApp.prototype, {
// to file-content below, which would dump the binary bytes as mojibake.
if (ext === 'docx' || ext === 'pptx') {
footerEl.textContent = ext.toUpperCase();
- const previewSrc = `/api/sessions/${sessionId}/file-preview?path=${encodeURIComponent(filePath)}`;
+ const previewSrc = CodemanBase.url(`/api/sessions/${sessionId}/file-preview?path=${encodeURIComponent(filePath)}`);
bodyEl.innerHTML = ``;
return;
}
if (ext === 'pdf') {
footerEl.textContent = 'PDF';
- const rawSrc = `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`;
+ const rawSrc = CodemanBase.url(`/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`);
bodyEl.innerHTML = ``;
return;
}
@@ -4113,19 +4115,19 @@ Object.assign(CodemanApp.prototype, {
const data = result.data;
if (data.type === 'image') {
- bodyEl.innerHTML = `
`;
+ bodyEl.innerHTML = `)})
`;
footerEl.textContent = `${this.formatFileSize(data.size)} \u2022 ${data.extension}`;
} else if (data.type === 'video') {
// playsinline: iOS otherwise hijacks playback into its fullscreen
// player, which leaves the overlay behind it and its own close button
// as the only way back.
- bodyEl.innerHTML = ``;
+ bodyEl.innerHTML = ``;
footerEl.textContent = `${this.formatFileSize(data.size)} \u2022 ${data.extension}`;
} else if (data.type === 'audio') {
- bodyEl.innerHTML = ``;
+ bodyEl.innerHTML = ``;
footerEl.textContent = `${this.formatFileSize(data.size)} \u2022 ${data.extension}`;
} else if (data.type === 'binary') {
- const downloadHref = `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}&download=true`;
+ const downloadHref = CodemanBase.url(`/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}&download=true`);
bodyEl.innerHTML = ``;
footerEl.textContent = data.extension || 'binary';
} else {
@@ -4422,9 +4424,11 @@ Object.assign(CodemanApp.prototype, {
},
openAttachmentInNewTab(sessionId, filePath, attachmentId = null) {
- const url = attachmentId
- ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/raw`
- : `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`;
+ const url = CodemanBase.url(
+ attachmentId
+ ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/raw`
+ : `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`
+ );
window.open(url, '_blank');
},
@@ -4460,19 +4464,22 @@ Object.assign(CodemanApp.prototype, {
const stack = this.ensureAttachmentCardStack();
const session = this.sessions.get(sessionId);
const sessionName = session?.name || sessionId.substring(0, 8);
- const attachmentRawUrl =
+ const attachmentRawUrl = CodemanBase.url(
rawUrl ||
- (attachmentId
- ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/raw`
- : `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`);
- const attachmentPreviewUrl =
+ (attachmentId
+ ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/raw`
+ : `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(filePath)}`)
+ );
+ const attachmentPreviewUrl = CodemanBase.url(
previewUrl ||
- (attachmentId ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/preview` : null);
- const attachmentThumbnailUrl =
+ (attachmentId ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/preview` : null)
+ );
+ const attachmentThumbnailUrl = CodemanBase.url(
thumbnailUrl ||
- (attachmentId
- ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/thumbnail`
- : `/api/sessions/${sessionId}/file-thumbnail?path=${encodeURIComponent(filePath)}`);
+ (attachmentId
+ ? `/api/sessions/${sessionId}/attachments/${encodeURIComponent(attachmentId)}/thumbnail`
+ : `/api/sessions/${sessionId}/file-thumbnail?path=${encodeURIComponent(filePath)}`)
+ );
const downloadUrl = attachmentId ? `${attachmentRawUrl}?download=true` : `${attachmentRawUrl}&download=true`;
const typeLabel = (extension || attachmentType || 'file').toUpperCase();
@@ -4736,7 +4743,7 @@ Object.assign(CodemanApp.prototype, {
.join(' • ');
const thumb =
item.thumbnailUrl && !item.missing
- ? `})
`
+ ? `)})
`
: '';
const disabled = item.missing ? 'disabled aria-disabled="true"' : '';
return `
@@ -4776,7 +4783,7 @@ Object.assign(CodemanApp.prototype, {
const item = this.getAttachmentHistoryItem(itemId);
if (!item || item.missing) return;
if (item.rawUrl || item.url) {
- window.open(item.rawUrl || item.url, '_blank');
+ window.open(CodemanBase.url(item.rawUrl || item.url), '_blank');
return;
}
this.openAttachmentInNewTab(item.sessionId, item.relativePath || item.fileName, item.attachmentId || null);
@@ -4785,7 +4792,7 @@ Object.assign(CodemanApp.prototype, {
downloadAttachmentHistoryItem(itemId) {
const item = this.getAttachmentHistoryItem(itemId);
if (!item || item.missing || !item.downloadUrl) return;
- window.open(item.downloadUrl, '_blank');
+ window.open(CodemanBase.url(item.downloadUrl), '_blank');
},
reshowAttachmentCard(itemId) {
@@ -4931,7 +4938,7 @@ Object.assign(CodemanApp.prototype, {
// Connect to SSE stream
const eventSource = new EventSource(
- `/api/sessions/${sessionId}/tail-file?path=${encodeURIComponent(filePath)}&lines=50`
+ CodemanBase.url(`/api/sessions/${sessionId}/tail-file?path=${encodeURIComponent(filePath)}&lines=50`)
);
eventSource.onmessage = (e) => {
@@ -5073,7 +5080,7 @@ Object.assign(CodemanApp.prototype, {
// Build image URL using the existing file-raw endpoint
// Use relativePath (path from working dir) instead of fileName (basename) for subdirectory images
- const imageUrl = `/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(relativePath || fileName)}`;
+ const imageUrl = CodemanBase.url(`/api/sessions/${sessionId}/file-raw?path=${encodeURIComponent(relativePath || fileName)}`);
// Create window element
const win = document.createElement('div');
diff --git a/src/web/public/session-ui.js b/src/web/public/session-ui.js
index aa9535a4..75083430 100644
--- a/src/web/public/session-ui.js
+++ b/src/web/public/session-ui.js
@@ -187,6 +187,13 @@ Object.assign(CodemanApp.prototype, {
this.closeCasePicker();
this.updateDirDisplayForCase(select.value);
this.updateMobileCaseLabel(select.value);
+ // Warm the container's CLI list HERE rather than when the run menu opens.
+ // The probe is a `docker exec` round trip, so gating it on the menu meant the
+ // menu painted every mode first and only narrowed a moment later — which
+ // reads as "it shows all of them" and lets a mode be picked that the
+ // container does not have.
+ const picked = (this.cases || []).find((c) => c.name === select.value);
+ if (picked?.location === 'docker') void this._probeDockerCaseModes(picked, null);
if (save) {
this.saveLastUsedCase(select.value);
}
@@ -477,10 +484,40 @@ Object.assign(CodemanApp.prototype, {
* run modes like the rest, and neither `agy` nor `pi` is likely to be installed.
*/
_refreshRunModeAvailability(menu) {
+ // A DOCKER case runs its agents INSIDE the container, so host CLI
+ // availability answers the wrong question: the host may have no claude at
+ // all while the container ships one, and gating on the host hides a mode
+ // that would have worked. Adoption records what the container really has
+ // (`availableModes`); an owned container runs our base image, which ships
+ // every CLI, so an absent list means "do not gate" rather than "nothing".
+ // Same source every run* path reads the selected case from.
+ const caseName = document.getElementById('quickStartCase')?.value;
+ const activeCase = caseName ? (this.cases || []).find((c) => c.name === caseName) : null;
+ const isDocker = activeCase?.location === 'docker';
+ // Prefer a LIVE probe over the value stored at attach time: a container's
+ // CLIs can be installed or removed long after the case was linked, and a
+ // case linked before that field existed has none at all.
+ const containerModes = isDocker
+ ? this._dockerCaseModes?.[caseName] || activeCase.docker?.availableModes || null
+ : null;
+ if (isDocker && !this._dockerCaseModes?.[caseName]) void this._probeDockerCaseModes(activeCase, menu);
+ // An unreachable container hides every agent mode and explains why, instead
+ // of silently offering modes that cannot start.
+ //
+ // ⚠️ ADOPTED cases only. For an OWNED case a missing container is the normal
+ // state before the first session — the launch chain creates and starts it — so
+ // reporting it as a fault hid every agent mode on a freshly linked Docker case
+ // behind "start it yourself first", for a container Codeman was about to create.
+ const probeError = isDocker ? this._dockerCaseProbeError?.[caseName] : null;
for (const mode of ['claude', 'opencode', 'codex', 'gemini', 'antigravity', 'pi', 'grok', 'deepseek', 'omp']) {
const btn = menu.querySelector(`.run-mode-option[data-mode="${mode}"]`);
- if (btn) btn.style.display = this.isCliAvailable(mode) ? 'flex' : 'none';
+ if (!btn) continue;
+ let available;
+ if (isDocker) available = probeError ? false : containerModes ? containerModes.includes(mode) : true;
+ else available = this.isCliAvailable(mode);
+ btn.style.display = available ? 'flex' : 'none';
}
+ this._renderRunModeNotice(menu, probeError);
// DeepSeek is the one mode whose availability has two halves: `dsh` can be
// perfectly installed while no pane-capable profile exists, because DeepSeek
// ships no terminal front door. In that state the honest offer is "add one",
@@ -622,6 +659,81 @@ Object.assign(CodemanApp.prototype, {
}
},
+ /**
+ * One-line explanation at the top of the run menu. Only a container that could
+ * not be read produces one; everything else removes it, so a stale reason can
+ * never outlive the condition that caused it.
+ */
+ _renderRunModeNotice(menu, message) {
+ if (!menu) return;
+ let el = menu.querySelector('.run-mode-notice');
+ if (!message) {
+ el?.remove();
+ return;
+ }
+ if (!el) {
+ el = document.createElement('div');
+ el.className = 'run-mode-notice';
+ menu.prepend(el);
+ }
+ // Server-supplied text: set it, never parse it as markup.
+ el.textContent = message;
+ },
+
+ /**
+ * Ask the container which CLIs it actually has, and re-gate the menu once the
+ * answer lands. Cached per case for the page's lifetime: the menu re-opens
+ * often and the probe is a `docker exec` round trip.
+ *
+ * Best-effort by design — an unreachable daemon or a stopped container leaves
+ * the cache empty, which the caller reads as "unknown" and therefore does not
+ * gate. Hiding every mode because a probe failed would be worse than showing
+ * one that turns out to be missing, which the launch path already refuses with
+ * a specific message.
+ */
+ async _probeDockerCaseModes(activeCase, menu) {
+ const name = activeCase?.name;
+ const container = activeCase?.docker?.container;
+ const hostId = activeCase?.docker?.hostId;
+ if (!name || !container || !hostId) return;
+ this._dockerCaseModes = this._dockerCaseModes || {};
+ if (this._dockerModeProbeInFlight?.[name]) return;
+ this._dockerModeProbeInFlight = this._dockerModeProbeInFlight || {};
+ this._dockerModeProbeInFlight[name] = true;
+ try {
+ // ⚠️ _api serializes `body` and sets Content-Type itself. Passing an
+ // already-stringified body double-encodes it and the server rejects a
+ // JSON string where it expects an object (400 INVALID_INPUT).
+ const probe = await this._apiJson('/api/docker-cases/adopt-preflight', {
+ method: 'POST',
+ body: { hostId, container },
+ });
+ if (probe?.ok && Array.isArray(probe.availableModes)) {
+ this._dockerCaseModes[name] = probe.availableModes;
+ delete this._dockerCaseProbeError?.[name];
+ } else {
+ // An ADOPTED container that cannot be probed — recreated, stopped, engine
+ // down — must NOT fall through to "show everything". Offering claude on a
+ // container that is not running is a click that can only fail, with the
+ // reason visible nowhere. Record the reason and say it in the menu.
+ //
+ // ⚠️ An OWNED container gets no error: it does not exist until the first
+ // session launches it, so "not found" is the expected answer for every
+ // newly linked Docker case, and gating on it made those cases unusable.
+ // Leaving the cache empty reads as "unknown", which does not gate.
+ if (activeCase?.docker?.owned === false) {
+ this._dockerCaseProbeError = this._dockerCaseProbeError || {};
+ this._dockerCaseProbeError[name] = probe?.error || `Could not read container "${container}".`;
+ }
+ delete this._dockerCaseModes[name];
+ }
+ // Only repaint while the menu the user opened is still on screen.
+ if (menu?.classList.contains('active')) this._refreshRunModeAvailability(menu);
+ } finally {
+ delete this._dockerModeProbeInFlight[name];
+ }
+ },
+
async _loadRunModeHistory() {
const container = document.getElementById('runModeHistory');
if (!container) return;
@@ -2387,6 +2499,28 @@ Object.assign(CodemanApp.prototype, {
modal.querySelectorAll('.set-rail-item').forEach(btn => {
btn.onclick = () => this.switchCaseModalTab(btn.dataset.tab);
});
+ // Adopt-an-existing-container toggle + its read-only preflight. Assigned (not
+ // addEventListener) so reopening the modal cannot stack duplicate handlers,
+ // matching the rail wiring right above.
+ const adoptToggle = document.getElementById('dockerAdoptExisting');
+ if (adoptToggle) adoptToggle.onchange = () => this._syncDockerAdoptMode();
+ const adoptCheck = document.getElementById('dockerAdoptCheckBtn');
+ if (adoptCheck) adoptCheck.onclick = () => this._dockerAdoptPreflight();
+ const adoptJump = document.getElementById('dockerAdoptJumpBtn');
+ if (adoptJump) adoptJump.onclick = () => this.jumpToDockerAdopt();
+ // Containers come from the host profile, so switching Host ID invalidates the
+ // suggestions. Dropping the marker (rather than refetching here) keeps the
+ // fetch lazy — it happens when adopt mode is actually on.
+ const hostIdInput = document.getElementById('dockerHostId');
+ if (hostIdInput) {
+ hostIdInput.onchange = () => {
+ delete document.getElementById('dockerContainerList')?.dataset.loadedFor;
+ if (document.getElementById('dockerAdoptExisting')?.checked) void this._loadDockerContainerOptions();
+ };
+ }
+ // A fresh open re-reads the engine: containers start and stop between visits.
+ delete document.getElementById('dockerContainerList')?.dataset.loadedFor;
+ this._syncDockerAdoptMode();
// Scroll-into-view on focus for mobile keyboard visibility
modal.querySelectorAll('input[type="text"]').forEach(input => {
if (!input._mobileScrollWired) {
@@ -2416,15 +2550,19 @@ Object.assign(CodemanApp.prototype, {
// A switched-to panel starts at its own top.
const doc = document.getElementById('createCaseDoc');
if (doc) doc.scrollTop = 0;
- // Update submit button (hide for manage tab)
- const submitBtn = document.getElementById('caseModalSubmit');
+ // Update submit buttons (hide for manage tab). Two of them: mobile.css hides
+ // this modal's .set-foot, so phones submit through the header button instead.
+ const submitBtns = ['caseModalSubmit', 'caseModalSubmitMobile']
+ .map((id) => document.getElementById(id))
+ .filter(Boolean);
if (tabName === 'case-manage') {
- submitBtn.style.display = 'none';
+ submitBtns.forEach((btn) => {
+ btn.style.display = 'none';
+ });
this.renderCaseManageList();
this.refreshDockerExports();
} else {
- submitBtn.style.display = '';
- submitBtn.textContent =
+ const label =
tabName === 'case-create'
? 'Create'
: tabName === 'case-clone'
@@ -2434,6 +2572,10 @@ Object.assign(CodemanApp.prototype, {
: tabName === 'case-docker'
? 'Link Docker'
: 'Link';
+ submitBtns.forEach((btn) => {
+ btn.style.display = '';
+ btn.textContent = label;
+ });
}
// Focus appropriate input
if (tabName === 'case-create') {
@@ -2454,14 +2596,19 @@ Object.assign(CodemanApp.prototype, {
},
async submitCaseModal() {
- const btn = document.getElementById('caseModalSubmit');
- const originalText = btn.textContent;
- btn.classList.add('loading');
- btn.textContent =
+ // Both submit buttons move together: whichever one the user pressed, the
+ // other must show the same pending state and be equally unclickable.
+ const btns = ['caseModalSubmit', 'caseModalSubmitMobile'].map((id) => document.getElementById(id)).filter(Boolean);
+ const originalText = btns.map((btn) => btn.textContent);
+ const pendingText =
this.caseModalTab === 'case-create' ? 'Creating...' : this.caseModalTab === 'case-clone' ? 'Cloning...' : 'Linking...';
// A clone holds this request open for minutes; without disabling the button a
// second click fires a second clone (the loser then fails on ALREADY_EXISTS).
- btn.disabled = true;
+ btns.forEach((btn) => {
+ btn.classList.add('loading');
+ btn.textContent = pendingText;
+ btn.disabled = true;
+ });
try {
if (this.caseModalTab === 'case-create') {
await this.createCase();
@@ -2475,9 +2622,11 @@ Object.assign(CodemanApp.prototype, {
await this.linkCase();
}
} finally {
- btn.classList.remove('loading');
- btn.disabled = false;
- btn.textContent = originalText;
+ btns.forEach((btn, index) => {
+ btn.classList.remove('loading');
+ btn.disabled = false;
+ btn.textContent = originalText[index];
+ });
}
},
@@ -2862,6 +3011,61 @@ Object.assign(CodemanApp.prototype, {
});
},
+ /** HOST workspace directory — the same picker Link Existing uses. */
+ openDockerWorkspacePathPicker() {
+ const pathInput = document.getElementById('dockerWorkspacePath');
+ PathPicker.open({
+ title: 'Select Host Workspace Folder',
+ initialPath: pathInput.value.trim(),
+ directoriesOnly: true,
+ onSelect: (path) => {
+ pathInput.value = path;
+ const nameInput = document.getElementById('dockerCaseName');
+ if (nameInput && !nameInput.value.trim()) {
+ const folder = path.split('/').filter(Boolean).pop() || '';
+ if (/^[a-zA-Z0-9_-]+$/.test(folder)) nameInput.value = folder;
+ }
+ },
+ });
+ },
+
+ /**
+ * Container workdir. Browses INSIDE the container, because for an adopted
+ * container nothing is mounted at a matching host path — the host picker would
+ * be listing a different filesystem, and typing this field blind is exactly
+ * what makes the launch fail with an OCI chdir error.
+ */
+ openDockerWorkdirPicker() {
+ const pathInput = document.getElementById('dockerAdoptWorkdir');
+ const container = document.getElementById('dockerContainerName')?.value.trim();
+ const hostId = document.getElementById('dockerHostId')?.value.trim() || 'local';
+ if (!container) {
+ this.showToast('Enter the container name first', 'error');
+ return;
+ }
+ PathPicker.open({
+ title: `Select Folder Inside ${container}`,
+ initialPath: pathInput.value.trim() || '/',
+ directoriesOnly: true,
+ fetchListing: async (path) => {
+ const data = await this._apiJson('/api/docker-cases/browse', {
+ method: 'POST',
+ body: { hostId, container, path: path || '/' },
+ });
+ if (!data) return { success: false, error: `Could not read ${container}. Is it running?` };
+ if (data.error) return { success: false, error: data.error };
+ // Shape it like the host endpoint: one root, so Up/Location behave.
+ return {
+ success: true,
+ data: { ...data, root: '/', roots: [{ label: container, path: '/' }], truncated: false },
+ };
+ },
+ onSelect: (path) => {
+ pathInput.value = path;
+ },
+ });
+ },
+
async linkRemoteCase() {
const name = document.getElementById('remoteCaseName').value.trim();
const remotePath = document.getElementById('remoteCasePath').value.trim();
@@ -2943,10 +3147,107 @@ Object.assign(CodemanApp.prototype, {
}
},
+ /**
+ * Reflect the "attach to an existing container" checkbox onto the modal so CSS
+ * can swap which half of the Docker panel applies. An attribute rather than
+ * per-row inline styles: the panel is rebuilt by nothing, but the create-time
+ * rows are a SET (image, network, advanced block) and one attribute keeps them
+ * in lockstep with the container-name row.
+ */
+ _syncDockerAdoptMode() {
+ const modal = document.getElementById('createCaseModal');
+ if (!modal) return;
+ const adopting = document.getElementById('dockerAdoptExisting')?.checked;
+ if (adopting) modal.setAttribute('data-docker-adopt', '1');
+ else modal.removeAttribute('data-docker-adopt');
+ if (adopting) void this._loadDockerContainerOptions();
+ },
+
+ /**
+ * Fill the container-name `