-
- Advanced request settings
+ <>
+ -
+
+
+
+ Request tuning
- Tune the OpenAI request used by board work, chat, and memory review.
+ These settings apply to board work, chat, and memory review. Blank fields use
+ provider defaults.
-
-
-
+
);
}
function OpenAiAccountPanel(props: {
authState: OpenAiAuthState | null;
accountInfo: OpenAiAccountInfo | null;
+ selectedModel: string;
checking: boolean;
loginInput: string;
onLoginInputChange: (value: string) => void;
@@ -742,12 +1129,17 @@ function OpenAiAccountPanel(props: {
return (
-
-
OpenAI account
-
- {props.authState?.configured
- ? `Connected${props.authState.accountId ? ` as ${props.authState.accountId}` : ""}.`
- : "Connect your OpenAI account to use supported Codex models without pasting an API key."}
-
+
+ {props.authState?.configured ?
+
+ OpenAI account
+
+ {props.authState?.configured
+ ? `Connected${props.authState.accountId ? ` as ${props.authState.accountId}` : ""}.`
+ : "Connect your OpenAI account to use supported Codex models without pasting an API key."}
+
+
-
-
Finish sign-in
-
- {login.progress ?? "Open the sign-in page, then paste the returned code if prompted."}
-
+
+ Finish sign-in
+
+ {login.progress ?? "Open the sign-in page, then paste the returned code if prompted."}
+
+
@@ -799,14 +1193,18 @@ function OpenAiAccountPanel(props: {
{props.accountInfo && (
-
-
+
+
+
Model catalog
- Recommended model: {props.accountInfo.recommendedModel}. Available models:{" "}
- {props.accountInfo.models.length}.
+ Selected model: {props.selectedModel}. Recommended model:{" "}
+ {props.accountInfo.recommendedModel}. Available models:{" "}
+ {props.accountInfo.models.length}. Last refreshed:{" "}
+ {formatTimestamp(props.accountInfo.prefetchedAt)}.
-
+
)}
@@ -816,57 +1214,90 @@ function OpenAiAccountPanel(props: {
function FocusAreaEditor(props: {
area: FocusArea;
canRemove: boolean;
+ error?: string;
+ inputRef: RefCallback
;
onChange: (patch: Partial) => void;
onRemove: () => void;
}) {
const style = getFocusAreaStyle(props.area.color);
+ const inputId = `focus-area-${props.area.id}`;
+ const invalid = Boolean(props.error);
+ const removeButton = (
+
+
+ );
return (
- -
-
-
- props.onChange({ color: value as FocusAreaColor })}
- >
-
-
-
-
- {FOCUS_AREA_COLOR_OPTIONS.map((option) => (
-
-
- ))}
-
-
-
-
-
+
+
+ {props.area.label || "Focus area"} name
+
+
+ props.onChange({ label: event.target.value })}
+ aria-label={`${props.area.label || "Focus area"} name`}
+ aria-invalid={invalid}
+ />
+
+
+
+ {props.error}
+
+
+ {props.area.label || "Focus area"} color
+ props.onChange({ color: value as FocusAreaColor })}
>
-
+
+
+
+
+ {FOCUS_AREA_COLOR_OPTIONS.map((option) => (
+
+
+ ))}
+
+
+
+
+
+ {props.canRemove ? (
+ removeButton
+ ) : (
+
+
+ {removeButton}
+
+ Keep at least one focus area.
+
+ )}
);
@@ -911,6 +1342,71 @@ function normalizeFocusAreaDrafts(areas: FocusArea[]): FocusArea[] {
.filter((area): area is FocusArea => Boolean(area));
}
+function getFocusAreaErrors(areas: FocusArea[]): Map {
+ const errors = new Map();
+ for (const area of areas) {
+ if (!area.label.trim()) {
+ errors.set(area.id, "Enter a focus area name.");
+ }
+ }
+ return errors;
+}
+
+function getOpenAiNumberErrors(values: Record): OpenAiNumberErrors {
+ return {
+ maxTokens: validateOptionalNumber(values.maxTokens, {
+ label: "Max output tokens",
+ min: 1,
+ integer: true,
+ }),
+ temperature: validateOptionalNumber(values.temperature, {
+ label: "Temperature",
+ min: 0,
+ max: 2,
+ }),
+ timeoutMs: validateOptionalNumber(values.timeoutMs, {
+ label: "Request timeout",
+ min: 1000,
+ integer: true,
+ }),
+ maxRetries: validateOptionalNumber(values.maxRetries, {
+ label: "Max retries",
+ min: 0,
+ max: 10,
+ integer: true,
+ }),
+ maxRetryDelayMs: validateOptionalNumber(values.maxRetryDelayMs, {
+ label: "Retry delay",
+ min: 0,
+ integer: true,
+ }),
+ };
+}
+
+function validateOptionalNumber(
+ value: string,
+ options: { label: string; min?: number; max?: number; integer?: boolean },
+): string | undefined {
+ const trimmed = value.trim();
+ if (!trimmed) {
+ return undefined;
+ }
+ const parsed = Number(trimmed);
+ if (!Number.isFinite(parsed)) {
+ return `${options.label} must be a number.`;
+ }
+ if (options.integer && !Number.isInteger(parsed)) {
+ return `${options.label} must be a whole number.`;
+ }
+ if (options.min !== undefined && parsed < options.min) {
+ return `${options.label} must be at least ${options.min}.`;
+ }
+ if (options.max !== undefined && parsed > options.max) {
+ return `${options.label} must be ${options.max} or lower.`;
+ }
+ return undefined;
+}
+
function parseOptionalNumber(value: string): number | null {
const trimmed = value.trim();
if (!trimmed) {
@@ -930,3 +1426,16 @@ function formatOptionLabel(value: string): string {
.map((part) => (part ? `${part[0].toUpperCase()}${part.slice(1)}` : part))
.join(" ");
}
+
+function formatTimestamp(value: string): string {
+ const date = new Date(value);
+ if (Number.isNaN(date.getTime())) {
+ return "unknown";
+ }
+ return date.toLocaleString(undefined, {
+ month: "short",
+ day: "numeric",
+ hour: "numeric",
+ minute: "2-digit",
+ });
+}