feat(ui): integrate list_tags api call into machine detail

This commit is contained in:
Brian McGee
2025-08-12 14:31:07 +01:00
parent ea75c9bfa9
commit 92e9bb2ed8
3 changed files with 53 additions and 15 deletions

View File

@@ -11,7 +11,7 @@ import { Label } from "@/src/components/Form/Label";
import { Orienter } from "@/src/components/Form/Orienter"; import { Orienter } from "@/src/components/Form/Orienter";
import { CollectionNode } from "@kobalte/core"; import { CollectionNode } from "@kobalte/core";
interface MachineTag { export interface MachineTag {
value: string; value: string;
disabled?: boolean; disabled?: boolean;
new?: boolean; new?: boolean;
@@ -24,11 +24,10 @@ export type MachineTagsProps = FieldProps & {
disabled?: boolean; disabled?: boolean;
required?: boolean; required?: boolean;
defaultValue?: string[]; defaultValue?: string[];
defaultOptions?: string[];
readonlyOptions?: string[];
}; };
// tags which are applied to all machines and cannot be removed
const staticOptions = [{ value: "all", disabled: true }];
const uniqueOptions = (options: MachineTag[]) => { const uniqueOptions = (options: MachineTag[]) => {
const record: Record<string, MachineTag> = {}; const record: Record<string, MachineTag> = {};
options.forEach((option) => { options.forEach((option) => {
@@ -39,13 +38,8 @@ const uniqueOptions = (options: MachineTag[]) => {
return Object.values(record); return Object.values(record);
}; };
const sortedOptions = (options: MachineTag[]) => { const sortedOptions = (options: MachineTag[]) =>
return options.sort((a, b) => { options.sort((a, b) => a.value.localeCompare(b.value));
if (a.new && !b.new) return -1;
if (a.disabled && !b.disabled) return -1;
return a.value.localeCompare(b.value);
});
};
const sortedAndUniqueOptions = (options: MachineTag[]) => const sortedAndUniqueOptions = (options: MachineTag[]) =>
sortedOptions(uniqueOptions(options)); sortedOptions(uniqueOptions(options));
@@ -72,9 +66,15 @@ export const MachineTags = (props: MachineTagsProps) => {
(props.defaultValue || []).map((value) => ({ value })), (props.defaultValue || []).map((value) => ({ value })),
); );
// todo this should be the superset of tags used across the entire clan and be passed in via a prop // convert default options string[] into MachineTag[]
const [availableOptions, setAvailableOptions] = createSignal<MachineTag[]>( const [availableOptions, setAvailableOptions] = createSignal<MachineTag[]>(
sortedAndUniqueOptions([...staticOptions, ...defaultValue]), sortedAndUniqueOptions([
...(props.readonlyOptions || []).map((value) => ({
value,
disabled: true,
})),
...(props.defaultOptions || []).map((value) => ({ value })),
]),
); );
const onKeyDown = (event: KeyboardEvent) => { const onKeyDown = (event: KeyboardEvent) => {

View File

@@ -13,11 +13,13 @@ export type FieldSchema<T> = {
}; };
}; };
export type Tags = SuccessData<"list_tags">;
export type Machine = SuccessData<"get_machine">; export type Machine = SuccessData<"get_machine">;
export type ListMachines = SuccessData<"list_machines">; export type ListMachines = SuccessData<"list_machines">;
export type MachineDetails = SuccessData<"get_machine_details">; export type MachineDetails = SuccessData<"get_machine_details">;
export interface MachineDetail { export interface MachineDetail {
tags: Tags;
machine: Machine; machine: Machine;
fieldsSchema: FieldSchema<Machine>; fieldsSchema: FieldSchema<Machine>;
} }
@@ -50,7 +52,12 @@ export const useMachineQuery = (clanURI: string, machineName: string) => {
return useQuery<MachineDetail>(() => ({ return useQuery<MachineDetail>(() => ({
queryKey: ["clans", encodeBase64(clanURI), "machine", machineName], queryKey: ["clans", encodeBase64(clanURI), "machine", machineName],
queryFn: async () => { queryFn: async () => {
const [machineCall, schemaCall] = await Promise.all([ const [tagsCall, machineCall, schemaCall] = await Promise.all([
client.fetch("list_tags", {
flake: {
identifier: clanURI,
},
}),
client.fetch("get_machine", { client.fetch("get_machine", {
name: machineName, name: machineName,
flake: { flake: {
@@ -67,6 +74,11 @@ export const useMachineQuery = (clanURI: string, machineName: string) => {
}), }),
]); ]);
const tags = await tagsCall.result;
if (tags.status === "error") {
throw new Error("Error fetching tags: " + tags.errors[0].message);
}
const machine = await machineCall.result; const machine = await machineCall.result;
if (machine.status === "error") { if (machine.status === "error") {
throw new Error("Error fetching machine: " + machine.errors[0].message); throw new Error("Error fetching machine: " + machine.errors[0].message);
@@ -81,6 +93,7 @@ export const useMachineQuery = (clanURI: string, machineName: string) => {
} }
return { return {
tags: tags.data,
machine: machine.data, machine: machine.data,
fieldsSchema: writeSchema.data, fieldsSchema: writeSchema.data,
}; };

View File

@@ -4,7 +4,8 @@ import { MachineDetail } from "@/src/hooks/queries";
import { SidebarSectionForm } from "@/src/components/Sidebar/SidebarSectionForm"; import { SidebarSectionForm } from "@/src/components/Sidebar/SidebarSectionForm";
import { pick } from "@/src/util"; import { pick } from "@/src/util";
import { UseQueryResult } from "@tanstack/solid-query"; import { UseQueryResult } from "@tanstack/solid-query";
import { MachineTags } from "@/src/components/Form/MachineTags"; import { MachineTag, MachineTags } from "@/src/components/Form/MachineTags";
import { machineNameParam } from "@/src/hooks/clan";
const schema = v.object({ const schema = v.object({
tags: v.pipe(v.optional(v.array(v.string()))), tags: v.pipe(v.optional(v.array(v.string()))),
@@ -30,6 +31,28 @@ export const SectionTags = (props: SectionTags) => {
return pick(machineQuery.data.machine, ["tags"]) satisfies FormValues; return pick(machineQuery.data.machine, ["tags"]) satisfies FormValues;
}; };
const readonlyOptions = () => {
if (!machineQuery.isSuccess) {
return [];
}
const result: string[] = ["all"];
if (machineQuery.data.machine.machineClass) {
result.push(machineQuery.data.machine.machineClass);
}
return result;
};
const defaultOptions = () => {
if (!machineQuery.isSuccess) {
return [];
}
return machineQuery.data.tags?.options ?? [];
};
return ( return (
<Show when={machineQuery.isSuccess}> <Show when={machineQuery.isSuccess}>
<SidebarSectionForm <SidebarSectionForm
@@ -50,6 +73,8 @@ export const SectionTags = (props: SectionTags) => {
readOnly={!editing} readOnly={!editing}
orientation="horizontal" orientation="horizontal"
defaultValue={field.value} defaultValue={field.value}
defaultOptions={defaultOptions()}
readonlyOptions={readonlyOptions()}
input={input} input={input}
/> />
)} )}