ceos/unsloth
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat: remove seed inspect/preview and MCP tools fetch support due to backend endpoint deprecation

Browse source
This commit is contained in:
Shine1i 2026-02-15 12:45:59 +01:00
commit 6244db98d8
3 changed files with 14 additions and 148 deletions
Download patch file Download diff file Expand all files Collapse all files

31
studio/frontend/src/features/recipe-studio/api/index.ts
View file

@ -22,25 +22,6 @@ export type ValidateResponse = {
raw_detail?: string | null;
};
export type ToolsResponse = {
// biome-ignore lint/style/useNamingConvention: api schema
tools_by_provider: Record<string, string[]>;
tools: string[];
};
export type SeedInspectResponse = {
// biome-ignore lint/style/useNamingConvention: api schema
repo_id: string;
splits: string[];
// biome-ignore lint/style/useNamingConvention: api schema
globs_by_split: Record<string, string>;
columns: string[];
};
export type SeedPreviewResponse = {
rows: Record<string, unknown>[];
};
async function parseErrorResponse(response: Response): Promise<string> {
const text = (await response.text()).trim();
if (!text) {
@ -90,14 +71,4 @@ export async function validateRecipe(
return postJson<ValidateResponse>("/validate", payload);
}
export async function listRecipeTools(payload: unknown): Promise<ToolsResponse> {
return postJson<ToolsResponse>("/tools", payload);
}
export async function inspectSeedDataset(payload: unknown): Promise<SeedInspectResponse> {
return postJson<SeedInspectResponse>("/seed/inspect", payload);
}
export async function previewSeedDataset(payload: unknown): Promise<SeedPreviewResponse> {
return postJson<SeedPreviewResponse>("/seed/preview", payload);
}
// NOTE: tools + seed inspect/preview endpoints removed from harness.

28
studio/frontend/src/features/recipe-studio/dialogs/llm/mcp-tools-tab.tsx
View file

@ -6,14 +6,13 @@ import {
SelectTrigger,
SelectValue,
} from "@/components/ui/select";
import { listRecipeTools } from "../../api";
import type {
LlmConfig,
LlmMcpProviderConfig,
LlmToolConfig,
McpEnvVar,
} from "../../types";
import { toastError, toastSuccess } from "@/shared/toast";
import { toastError } from "@/shared/toast";
import { McpProvidersSection } from "./mcp-tools/mcp-providers-section";
import {
createMcpProviderId,
@ -197,11 +196,11 @@ export function LlmMcpToolsTab({
}
async function loadToolNames(): Promise<void> {
// tools endpoint removed from harness; keep UI but disable fetch.
const apiProviders = providers
.filter(isProviderReadyForToolFetch)
.map(toApiProvider)
.filter((provider) => Boolean(provider.name));
if (apiProviders.length === 0) {
toastError(
"No MCP servers configured",
@ -209,29 +208,10 @@ export function LlmMcpToolsTab({
);
return;
}
setLoadingTools(true);
try {
const response = await listRecipeTools({
recipe: {
// biome-ignore lint/style/useNamingConvention: api schema
model_providers: [],
// biome-ignore lint/style/useNamingConvention: api schema
mcp_providers: apiProviders,
// biome-ignore lint/style/useNamingConvention: api schema
model_configs: [],
// biome-ignore lint/style/useNamingConvention: api schema
tool_configs: [],
columns: [],
processors: [],
},
});
// biome-ignore lint/style/useNamingConvention: api schema
setToolsByProvider(response.tools_by_provider ?? {});
toastSuccess("Fetched MCP tools");
} catch (error) {
const message = error instanceof Error ? error.message : "Tool fetch failed.";
toastError("Failed to fetch tools", message);
toastError("Tool fetch disabled", "Backend /tools endpoint removed.");
setToolsByProvider({});
} finally {
setLoadingTools(false);
}

103
studio/frontend/src/features/recipe-studio/dialogs/seed/seed-dialog.tsx
View file

@ -14,7 +14,6 @@ import {
SelectTrigger,
SelectValue,
} from "@/components/ui/select";
import { Spinner } from "@/components/ui/spinner";
import {
Table,
TableBody,
@ -30,7 +29,6 @@ import {
TabsTrigger,
} from "@/components/ui/tabs";
import { type ReactElement, useMemo, useState } from "react";
import { inspectSeedDataset, previewSeedDataset } from "../../api";
import type {
SeedConfig,
SeedSamplingStrategy,
@ -84,17 +82,8 @@ function stringifyCell(value: unknown): string {
}
}
function parseOptionalInt(value: string | undefined): number | null {
const trimmed = value?.trim();
if (!trimmed) return null;
const num = Number(trimmed);
return Number.isFinite(num) ? num : null;
}
export function SeedDialog({ config, onUpdate }: SeedDialogProps): ReactElement {
const [inspectLoading, setInspectLoading] = useState(false);
const [inspectError, setInspectError] = useState<string | null>(null);
const [previewLoading, setPreviewLoading] = useState(false);
const [previewError, setPreviewError] = useState<string | null>(null);
const [previewRows, setPreviewRows] = useState<Record<string, unknown>[]>([]);
@ -111,84 +100,12 @@ export function SeedDialog({ config, onUpdate }: SeedDialogProps): ReactElement
const selectionId = `${config.id}-selection`;
async function onInspect(): Promise<void> {
setInspectError(null);
const repo_id = repoId;
if (!repo_id) {
setInspectError("Invalid HF dataset URL (need /datasets/org/repo).");
return;
}
setInspectLoading(true);
try {
const res = await inspectSeedDataset({
// biome-ignore lint/style/useNamingConvention: api schema
repo_id,
// biome-ignore lint/style/useNamingConvention: api schema
hf_token: config.hf_token?.trim() || null,
split: config.hf_split?.trim() || null,
});
const splits = res.splits ?? [];
const globs = res.globs_by_split ?? {};
let nextSplit = "";
if (config.hf_split && splits.includes(config.hf_split)) {
nextSplit = config.hf_split;
} else if (splits[0]) {
nextSplit = splits[0];
}
const nextPath = nextSplit ? (globs[nextSplit] ?? "") : "";
onUpdate({
hf_repo_id: res.repo_id,
seed_splits: splits,
seed_globs_by_split: globs,
seed_columns: res.columns ?? [],
hf_split: nextSplit,
hf_path: nextPath || config.hf_path,
});
} catch (err) {
setInspectError(err instanceof Error ? err.message : "Inspect failed.");
} finally {
setInspectLoading(false);
}
setInspectError("Seed inspect disabled (backend /seed/inspect removed).");
}
async function onPreview(): Promise<void> {
setPreviewError(null);
const hf_path = config.hf_path.trim();
if (!hf_path) {
setPreviewError("HF path missing (Load first).");
return;
}
setPreviewLoading(true);
try {
const res = await previewSeedDataset({
// biome-ignore lint/style/useNamingConvention: api schema
hf_path,
// biome-ignore lint/style/useNamingConvention: api schema
hf_token: config.hf_token?.trim() || null,
// biome-ignore lint/style/useNamingConvention: api schema
sampling_strategy: config.sampling_strategy,
// biome-ignore lint/style/useNamingConvention: api schema
selection_type: config.selection_type,
// biome-ignore lint/style/useNamingConvention: api schema
selection_start: parseOptionalInt(config.selection_start),
// biome-ignore lint/style/useNamingConvention: api schema
selection_end: parseOptionalInt(config.selection_end),
// biome-ignore lint/style/useNamingConvention: api schema
selection_index: parseOptionalInt(config.selection_index),
// biome-ignore lint/style/useNamingConvention: api schema
selection_num_partitions: parseOptionalInt(config.selection_num_partitions),
limit: 10,
});
const rows = res.rows ?? [];
setPreviewRows(rows);
if ((config.seed_columns?.length ?? 0) === 0 && rows[0]) {
onUpdate({ seed_columns: Object.keys(rows[0]) });
}
} catch (err) {
setPreviewError(err instanceof Error ? err.message : "Preview failed.");
} finally {
setPreviewLoading(false);
}
setPreviewError("Seed preview disabled (backend /seed/preview removed).");
setPreviewRows([]);
}
const previewColumns = useMemo(() => {
@ -227,11 +144,10 @@ export function SeedDialog({ config, onUpdate }: SeedDialogProps): ReactElement
variant="outline"
className="nodrag"
onClick={() => void onInspect()}
disabled={inspectLoading}
disabled
>
{inspectLoading ? "Loading..." : "Load"}
Load (disabled)
</Button>
{inspectLoading && <Spinner />}
</div>
<p className="text-xs text-muted-foreground">
Repo: {repoId ?? "-"}
@ -433,9 +349,9 @@ export function SeedDialog({ config, onUpdate }: SeedDialogProps): ReactElement
variant="outline"
className="nodrag"
onClick={() => void onPreview()}
disabled={previewLoading}
disabled
>
{previewLoading ? "Loading..." : "Load 10 rows"}
Load 10 rows (disabled)
</Button>
</EmptyContent>
</Empty>
@ -448,11 +364,10 @@ export function SeedDialog({ config, onUpdate }: SeedDialogProps): ReactElement
variant="outline"
className="nodrag"
onClick={() => void onPreview()}
disabled={previewLoading}
disabled
>
{previewLoading ? "Loading..." : "Reload 10 rows"}
Reload 10 rows (disabled)
</Button>
{previewLoading && <Spinner />}
</div>
<Table className="border border-border/60 rounded-xl">
<TableHeader>