Skip to content

feat(ui): add OpenCode config editor #191

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ymkiux merged 16 commits into from
Jun 7, 2026
Merged

feat(ui): add OpenCode config editor #191

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
aa7c0f0
feat(ui): add opencode config editor
awsldev Jun 6, 2026
44caa05
fix(ui): align OpenCode config editor with real schema
awsldev Jun 6, 2026
348d350
fix(ui): close OpenCode config review gaps
awsldev Jun 6, 2026
76a1d18
fix(ui): show OpenCode config status
awsldev Jun 6, 2026
e2860aa
fix(opencode): isolate provider store from native config
awsldev Jun 7, 2026
12c645e
fix(opencode): localize remaining config labels
awsldev Jun 7, 2026
9068b32
test(opencode): harden provider store backend tests
awsldev Jun 7, 2026
b0d1aeb
docs(readme): document OpenCode provider support
awsldev Jun 7, 2026
24bdef9
docs(readme): clarify session browser sources
awsldev Jun 7, 2026
72aa0ae
docs(readme): remove unsupported derived session mention
awsldev Jun 7, 2026
51d1065
chore: bump version to 0.0.46
awsldev Jun 7, 2026
3f87fac
docs(readme): refresh screenshots
awsldev Jun 7, 2026
5b144b4
fix(opencode): align provider action buttons
awsldev Jun 7, 2026
b0d0735
chore(web-ui): refresh precompiled render
awsldev Jun 7, 2026
07053b0
docs(readme): refresh Chinese screenshots
awsldev Jun 7, 2026
d21e173
chore(release): reaffirm v0.0.46 head
awsldev Jun 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
233 changes: 230 additions & 3 deletions cli.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -185,6 +185,8 @@ const DEFAULT_WEB_OPEN_HOST = '127.0.0.1';
const CONFIG_DIR = path.join(os.homedir(), '.codex');
const CONFIG_FILE = path.join(CONFIG_DIR, 'config.toml');
const AUTH_FILE = path.join(CONFIG_DIR, 'auth.json');
const OPENCODE_HOME_CONFIG_FILE = path.join(os.homedir(), '.opencode.json');
const OPENCODE_XDG_CONFIG_FILE = path.join(process.env.XDG_CONFIG_HOME || path.join(os.homedir(), '.config'), 'opencode', '.opencode.json');
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated
const AUTH_PROFILES_DIR = path.join(CONFIG_DIR, 'auth-profiles');
const AUTH_REGISTRY_FILE = path.join(AUTH_PROFILES_DIR, 'registry.json');
const MODELS_FILE = path.join(CONFIG_DIR, 'models.json');
Expand Down Expand Up @@ -860,8 +862,8 @@ function isPlainObject(value) {
return !!value && typeof value === 'object' && !Array.isArray(value);
}

const TOOL_CONFIG_PERMISSION_TARGETS = new Set(['codex', 'claude']);
const TOOL_CONFIG_PERMISSION_DEFAULTS = Object.freeze({ codex: false, claude: false });
const TOOL_CONFIG_PERMISSION_TARGETS = new Set(['codex', 'claude', 'opencode']);
const TOOL_CONFIG_PERMISSION_DEFAULTS = Object.freeze({ codex: false, claude: false, opencode: false });
let toolConfigWriteGuardDepth = 0;

function enterToolConfigWriteGuard() {
Expand All @@ -887,7 +889,8 @@ function normalizeToolConfigPermissions(value) {
const source = isPlainObject(value) ? value : {};
return {
codex: source.codex === true,
claude: source.claude === true
claude: source.claude === true,
opencode: source.opencode === true
};
}

Expand Down Expand Up @@ -9628,6 +9631,221 @@ function applyClaudeSettingsRaw(params = {}) {
}
}

function getOpencodeConfigCandidates() {
const candidates = [OPENCODE_HOME_CONFIG_FILE, OPENCODE_XDG_CONFIG_FILE]
.filter(Boolean)
.map(item => path.resolve(item));
return [...new Set(candidates)];
}

function resolveOpencodeConfigFile() {
const candidates = getOpencodeConfigCandidates();
for (const candidate of candidates) {
if (fs.existsSync(candidate)) {
return candidate;
}
}
return candidates[0] || OPENCODE_HOME_CONFIG_FILE;
}

function readOpencodeConfigObject(content) {
const raw = typeof content === 'string' ? content : '';
if (!raw.trim()) {
return {};
}
const parsed = JSON5.parse(raw);
if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) {
throw new Error('OpenCode config must be a JSON object');
}
return parsed;
}

function normalizeOpencodeAgentName(value) {
const name = typeof value === 'string' ? value.trim() : '';
return /^[a-zA-Z0-9_.-]+$/.test(name) ? name : '';
}

function normalizeOpencodeProviderName(value) {
const name = typeof value === 'string' ? value.trim().toLowerCase() : '';
return /^[a-z0-9_.-]+$/.test(name) ? name : '';
}

function summarizeOpencodeConfig(config = {}, targetPath = resolveOpencodeConfigFile(), exists = false) {
const providers = config.providers && typeof config.providers === 'object' && !Array.isArray(config.providers)
? config.providers
: {};
const agents = config.agents && typeof config.agents === 'object' && !Array.isArray(config.agents)
? config.agents
: {};
const agentEntries = Object.entries(agents)
.filter(([, agent]) => agent && typeof agent === 'object' && !Array.isArray(agent))
.map(([name, agent]) => ({
name,
model: typeof agent.model === 'string' ? agent.model : '',
maxTokens: Number.isFinite(Number(agent.maxTokens)) ? Number(agent.maxTokens) : null,
reasoningEffort: typeof agent.reasoningEffort === 'string' ? agent.reasoningEffort : ''
}));
const primaryAgent = agentEntries.find(item => item.name === 'coder') || agentEntries[0] || null;
return {
exists: !!exists,
targetPath,
providers: Object.entries(providers)
.filter(([, provider]) => provider && typeof provider === 'object' && !Array.isArray(provider))
.map(([name, provider]) => ({
name,
apiKey: maskKey(typeof provider.apiKey === 'string' ? provider.apiKey : ''),
hasKey: typeof provider.apiKey === 'string' && provider.apiKey.trim().length > 0,
disabled: provider.disabled === true
})),
agents: agentEntries,
currentAgent: primaryAgent ? primaryAgent.name : 'coder',
currentModel: primaryAgent ? primaryAgent.model : '',
autoCompact: config.autoCompact !== false,
redacted: true
};
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated
}

function readOpencodeConfigInfo() {
const targetPath = resolveOpencodeConfigFile();
if (!fs.existsSync(targetPath)) {
const config = {};
return {
...summarizeOpencodeConfig(config, targetPath, false),
content: JSON.stringify(config, null, 2),
candidates: getOpencodeConfigCandidates()
};
}
try {
const raw = fs.readFileSync(targetPath, 'utf-8');
const config = readOpencodeConfigObject(raw);
return {
...summarizeOpencodeConfig(config, targetPath, true),
content: raw || '{}',
candidates: getOpencodeConfigCandidates()
};
} catch (e) {
return {
error: e.message || '读取 OpenCode 配置失败',
exists: true,
targetPath,
candidates: getOpencodeConfigCandidates()
};
}
}

function applyOpencodeConfigRaw(params = {}) {
assertToolConfigWriteAllowed('opencode');
const content = typeof params.content === 'string' ? params.content : '';
if (!content.trim()) {
return { error: '内容不能为空' };
}
if (content.length > 1024 * 1024) {
return { error: '内容过大（最大 1MB）' };
}
let parsed;
try {
parsed = readOpencodeConfigObject(content);
} catch (e) {
return { error: `OpenCode JSON 解析失败: ${e.message}` };
}
const targetPath = resolveOpencodeConfigFile();
try {
ensureDir(path.dirname(targetPath));
backupFileIfNeededOnce(targetPath);
fs.writeFileSync(targetPath, JSON.stringify(parsed, null, 2) + '\n', { encoding: 'utf-8', mode: 0o600 });
return {
success: true,
targetPath,
...summarizeOpencodeConfig(parsed, targetPath, true)
};
} catch (e) {
return { error: e.message || '写入 OpenCode 配置失败' };
}
}

function updateOpencodeSelection(params = {}) {
assertToolConfigWriteAllowed('opencode');
const providerName = normalizeOpencodeProviderName(params.provider);
const model = typeof params.model === 'string' ? params.model.trim() : '';
const agentName = normalizeOpencodeAgentName(params.agent || 'coder') || 'coder';
if (!providerName) {
return { error: '请选择 OpenCode provider' };
}
if (!model) {
return { error: '请选择或输入 OpenCode model' };
}

const targetPath = resolveOpencodeConfigFile();
let config = {};
if (fs.existsSync(targetPath)) {
try {
config = readOpencodeConfigObject(fs.readFileSync(targetPath, 'utf-8'));
} catch (e) {
return { error: `OpenCode JSON 解析失败: ${e.message}` };
}
}
if (!config || typeof config !== 'object' || Array.isArray(config)) {
config = {};
}
if (!config.providers || typeof config.providers !== 'object' || Array.isArray(config.providers)) {
config.providers = {};
}
const previousProvider = config.providers[providerName] && typeof config.providers[providerName] === 'object' && !Array.isArray(config.providers[providerName])
? config.providers[providerName]
: {};
const apiKey = typeof params.apiKey === 'string' ? params.apiKey.trim() : '';
config.providers[providerName] = {
...previousProvider,
disabled: params.disabled === true
};
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated
if (apiKey) {
config.providers[providerName].apiKey = apiKey;
} else if (!Object.prototype.hasOwnProperty.call(config.providers[providerName], 'apiKey')) {
config.providers[providerName].apiKey = '';
}

if (!config.agents || typeof config.agents !== 'object' || Array.isArray(config.agents)) {
config.agents = {};
}
const coreAgents = params.applyToCoreAgents === true
? ['coder', 'task', 'summarizer', 'title']
: [agentName];
for (const name of coreAgents) {
const previousAgent = config.agents[name] && typeof config.agents[name] === 'object' && !Array.isArray(config.agents[name])
? config.agents[name]
: {};
config.agents[name] = {
...previousAgent,
model
};
const maxTokens = normalizePositiveIntegerParam(params.maxTokens);
if (maxTokens !== null) {
config.agents[name].maxTokens = maxTokens;
}
const effort = typeof params.reasoningEffort === 'string' ? params.reasoningEffort.trim().toLowerCase() : '';
if (effort === 'low' || effort === 'medium' || effort === 'high') {
config.agents[name].reasoningEffort = effort;
}
}
if (params.autoCompact !== undefined) {
config.autoCompact = params.autoCompact !== false;
}

try {
ensureDir(path.dirname(targetPath));
backupFileIfNeededOnce(targetPath);
fs.writeFileSync(targetPath, JSON.stringify(config, null, 2) + '\n', { encoding: 'utf-8', mode: 0o600 });
return {
success: true,
targetPath,
...summarizeOpencodeConfig(config, targetPath, true),
content: JSON.stringify(config, null, 2) + '\n'
};
} catch (e) {
return { error: e.message || '写入 OpenCode 配置失败' };
}
}

// API: 打包 Claude 配置目录（系统 zip 可用则使用，否则回退 zip-lib）
async function prepareClaudeDirDownload() {
return await prepareDirectoryDownload(CLAUDE_DIR, {
Expand Down Expand Up @@ -11452,6 +11670,15 @@ function createWebServer({ htmlPath, assetsDir, webDir, host, port, openBrowser
case 'apply-claude-settings-raw':
result = applyClaudeSettingsRaw(params || {});
break;
case 'get-opencode-config':
result = readOpencodeConfigInfo();
break;
case 'apply-opencode-config':
result = applyOpencodeConfigRaw(params || {});
break;
case 'update-opencode-selection':
result = updateOpencodeSelection(params || {});
break;
Comment thread
coderabbitai[bot] marked this conversation as resolved.
case 'apply-claude-config':
result = await applyToClaudeSettings(params.config);
if (result && !result.error) {
Expand Down
14 changes: 13 additions & 1 deletion tests/unit/config-tabs-ui.test.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ test('config template keeps expected config tabs in top and side navigation', ()
const sideTabModes = [...html.matchAll(/id="side-tab-config-([a-z]+)"/g)]
.map((match) => match[1]);

assert.deepStrictEqual(sideTabModes, ['codex', 'claude', 'openclaw']);
assert.deepStrictEqual(sideTabModes, ['codex', 'claude', 'openclaw', 'opencode']);
assert.match(html, /id="tab-dashboard"/);
assert.match(html, /v-if="healthCheckResult && healthCheckResult\.report" class="doctor-action-list"/);
assert.match(html, /v-if="healthCheckResult\.report\.issues && healthCheckResult\.report\.issues\.length"/);
Expand Down Expand Up @@ -211,9 +211,11 @@ test('config template keeps expected config tabs in top and side navigation', ()
assert.match(html, /class="segmented-control"[\s\S]*@click="switchConfigMode\('codex'\)"/);
assert.match(html, /class="segmented-control"[\s\S]*@click="switchConfigMode\('claude'\)"/);
assert.match(html, /class="segmented-control"[\s\S]*@click="switchConfigMode\('openclaw'\)"/);
assert.match(html, /class="segmented-control"[\s\S]*@click="switchConfigMode\('opencode'\)"/);
assert.doesNotMatch(sideRail, /role="tablist"/);
assert.doesNotMatch(sideRail, /role="tab"/);
assert.match(sideRail, /id="side-tab-config-codex"[\s\S]*:aria-current="mainTab === 'config' && configMode === 'codex' \? 'page' : null"/);
assert.match(sideRail, /id="side-tab-config-opencode"[\s\S]*:aria-current="mainTab === 'config' && configMode === 'opencode' \? 'page' : null"/);
assert.match(sideRail, /id="side-tab-docs"[\s\S]*:aria-current="mainTab === 'docs' \? 'page' : null"/);
assert.match(sideRail, /id="side-tab-settings"[\s\S]*:aria-current="mainTab === 'settings' \? 'page' : null"/);
assert.match(html, /skillsDefaultRootPath/);
Expand Down Expand Up @@ -261,6 +263,7 @@ test('config template keeps expected config tabs in top and side navigation', ()
assert.match(html, /data-main-tab=\"sessions\"/);
assert.match(html, /data-main-tab=\"market\"/);
assert.match(html, /data-config-mode=\"codex\"/);
assert.match(html, /data-config-mode=\"opencode\"/);
assert.match(html, /isMainTabNavActive\('settings'\)/);
assert.match(html, /isMainTabNavActive\('market'\)/);
assert.match(html, /isConfigModeNavActive\('codex'\)/);
Expand Down Expand Up @@ -303,6 +306,15 @@ test('config template keeps expected config tabs in top and side navigation', ()
html,
/type="checkbox"\s+autocomplete="off"\s+:checked="isToolConfigWriteAllowed\('claude'\)"[\s\S]*@change="setToolConfigPermission\('claude', \$event\.target\.checked\)"/
);
assert.match(
html,
/id="panel-config-opencode"[\s\S]*:checked="isToolConfigWriteAllowed\('opencode'\)"[\s\S]*@change="setToolConfigPermission\('opencode', \$event\.target\.checked\)"/
);
assert.match(html, /id="opencode-provider"[\s\S]*v-model="opencodeProvider"/);
assert.match(html, /id="opencode-model"[\s\S]*v-model="opencodeModel"/);
assert.match(html, /@change="handleOpencodeImportChange"/);
assert.match(html, /@click="saveOpencodeConfig"/);
assert.match(html, /@click="applyOpencodeSelection"/);
assert.match(
html,
/<span v-if="claudeSpeedResults\[name\]"[\s\S]*class="\['latency', claudeSpeedResults\[name\]\.ok \? 'ok' : 'error'\]"[\s\S]*>\s*\{\{\s*formatLatency\(claudeSpeedResults\[name\]\)\s*\}\}\s*<\/span>[\s\S]*<span :class="\['pill', config\.hasKey \? 'configured' : 'empty'\]">/
Expand Down
Loading
Loading