From e69c5955b5b26faa0205a7904de811b48cdb29ce Mon Sep 17 00:00:00 2001
From: satoshi komatsu <84182110+satoshi7190@users.noreply.github.com>
Date: Tue, 2 Jul 2024 09:05:17 +0900
Subject: [PATCH] =?UTF-8?q?sink=E3=81=AEparameters=E3=81=8B=E3=82=89?=
 =?UTF-8?q?=E5=A4=89=E6=8F=9B=E3=82=AA=E3=83=97=E3=82=B7=E3=83=A7=E3=83=B3?=
 =?UTF-8?q?=E3=81=AEGUI=E3=82=92=E7=94=9F=E6=88=90=E3=81=99=E3=82=8B(Poc)?=
 =?UTF-8?q?=20(#563)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

<!-- Close or Related Issues -->
Close #334

### Description（変更内容）
<!-- Please describe the motivation behind this PR and the changes it
introduces. -->
<!-- 何のために、どのような変更をしますか？ -->


![image](https://github.com/MIERUNE/plateau-gis-converter/assets/84182110/fc68c1d1-24bf-4fac-a6d2-e05ecacc130a)


- sinkのparametersをUIから変更できるようにしました（MVTのみ）
- UIはIntegerのみ対応。StringとBooleanは仮実装してますが、まだ使用してないのでコメントアウトしてます。
- 入力ファイルと出力ファイルが指定されてない場合は変換ボタンを押せないようにしました。

### Manual Testing（手動テスト）
<!-- If manual testing is required, please describe the procedure. -->
<!-- 手動での動作確認が必要なら、そのやり方を記述してください。-->

- MVTのズームレベルを変更して出力できるか確認
---
 app/src-tauri/src/main.rs             |  18 +++-
 app/src/app.css                       |  13 +++
 app/src/lib/sinkparams.ts             |  55 +++++++++++
 app/src/routes/+page.svelte           |  33 ++++---
 app/src/routes/InputSelector.svelte   |   4 +-
 app/src/routes/OutputSelector.svelte  |   4 +-
 app/src/routes/SettingSelector.svelte | 129 ++++++++++++++++++++------
 nusamai/src/parameters/mod.rs         |  23 +++--
 nusamai/src/sink/cesiumtiles/mod.rs   |   2 +
 nusamai/src/sink/czml/mod.rs          |   1 +
 nusamai/src/sink/geojson/mod.rs       |   1 +
 nusamai/src/sink/gltf/mod.rs          |   1 +
 nusamai/src/sink/gpkg/mod.rs          |   1 +
 nusamai/src/sink/kml/mod.rs           |   1 +
 nusamai/src/sink/minecraft/mod.rs     |   1 +
 nusamai/src/sink/mvt/mod.rs           |   3 +
 nusamai/src/sink/noop/mod.rs          |   1 +
 nusamai/src/sink/ply/mod.rs           |   1 +
 nusamai/src/sink/serde/mod.rs         |   1 +
 nusamai/src/sink/shapefile/mod.rs     |   3 +-
 20 files changed, 238 insertions(+), 58 deletions(-)
 create mode 100644 app/src/lib/sinkparams.ts

diff --git a/app/src-tauri/src/main.rs b/app/src-tauri/src/main.rs
index 93e5f59d..4c5000d9 100644
--- a/app/src-tauri/src/main.rs
+++ b/app/src-tauri/src/main.rs
@@ -9,6 +9,7 @@ use std::{
 };
 
 use log::LevelFilter;
+use nusamai::parameters::Parameters;
 use nusamai::{
     pipeline::{feedback, Canceller},
     sink::{
@@ -57,6 +58,7 @@ fn main() {
         .invoke_handler(tauri::generate_handler![
             run_conversion,
             cancel_conversion,
+            get_parameter,
             get_transform
         ])
         .run(tauri::generate_context!())
@@ -134,6 +136,7 @@ fn run_conversion(
     epsg: u16,
     rules_path: String,
     transformer_options: Vec<TransformerOption>,
+    sink_parameters: Parameters,
     tasks_state: tauri::State<ConversionTasksState>,
     window: tauri::Window,
 ) -> Result<(), Error> {
@@ -174,7 +177,7 @@ fn run_conversion(
             Error::InvalidSetting(msg)
         })?;
 
-        let mut sink_params = sink_provider.parameters();
+        let mut sink_params = sink_parameters;
         if let Err(err) = sink_params.update_values_with_str(&sinkopt) {
             let msg = format!("Error parsing sink options: {:?}", err);
             log::error!("{}", msg);
@@ -307,3 +310,16 @@ fn get_transform(filetype: String) -> Result<TransformerRegistry, Error> {
 
     Ok(transformer_registry)
 }
+
+/// Get the configurable parameters of the sink
+#[tauri::command]
+fn get_parameter(filetype: String) -> Result<Parameters, Error> {
+    let sink_provider = select_sink_provider(&filetype).ok_or_else(|| {
+        let msg = format!("Invalid sink type: {}", filetype);
+        log::error!("{}", msg);
+        Error::InvalidSetting(msg)
+    })?;
+    let sink_params = sink_provider.parameters();
+
+    Ok(sink_params)
+}
diff --git a/app/src/app.css b/app/src/app.css
index e8ce3746..56329e65 100644
--- a/app/src/app.css
+++ b/app/src/app.css
@@ -5,3 +5,16 @@
 body {
 	@apply text-base;
 }
+
+body::-webkit-scrollbar {
+	width: 9px;
+	height: 9px;
+}
+
+body::-webkit-scrollbar-track {
+	background-color: #d9d9d9;
+}
+
+body::-webkit-scrollbar-thumb {
+	background-color: rgb(0 190 190);
+}
diff --git a/app/src/lib/sinkparams.ts b/app/src/lib/sinkparams.ts
new file mode 100644
index 00000000..51bf3db4
--- /dev/null
+++ b/app/src/lib/sinkparams.ts
@@ -0,0 +1,55 @@
+export interface IntegerParameter {
+	Integer: {
+		value: number;
+		min?: number;
+		max?: number;
+	};
+}
+
+export interface StringParameter {
+	String: {
+		value: string;
+	};
+}
+
+export interface BooleanParameter {
+	Boolean: {
+		value: boolean;
+	};
+}
+
+type Parameter = IntegerParameter | StringParameter | BooleanParameter;
+
+interface ParamsOptionItem {
+	parameter: Parameter;
+	description: string;
+	label?: string;
+}
+
+export interface SinkParameters {
+	items: {
+		[key: string]: ParamsOptionItem;
+	};
+}
+
+// Check the parameter type
+export function isIntegerParameter(
+	parameter: Parameter
+): parameter is { Integer: { value: number; min: number; max: number } } {
+	return (parameter as { Integer?: unknown }).Integer !== undefined;
+}
+export function isStringParameter(
+	parameter: Parameter
+): parameter is { String: { value: string } } {
+	return (parameter as { String?: unknown }).String !== undefined;
+}
+export function isBooleanParameter(
+	parameter: Parameter
+): parameter is { Boolean: { value: boolean } } {
+	return (parameter as { Boolean?: unknown }).Boolean !== undefined;
+}
+
+export function createRangeArray(min: number, max: number): number[] {
+	if (min > max) throw new Error('min should be less than or equal to max');
+	return Array.from({ length: max - min + 1 }, (_, i) => min + i);
+}
diff --git a/app/src/routes/+page.svelte b/app/src/routes/+page.svelte
index 30cc9116..bf7893b9 100644
--- a/app/src/routes/+page.svelte
+++ b/app/src/routes/+page.svelte
@@ -2,6 +2,7 @@
 	import { message } from '@tauri-apps/api/dialog';
 	import { invoke } from '@tauri-apps/api/tauri';
 	import { attachConsole } from 'tauri-plugin-log-api';
+	import type { SinkParameters } from '$lib/sinkparams';
 
 	import Icon from '@iconify/svelte';
 	import InputSelector from './InputSelector.svelte';
@@ -16,20 +17,14 @@
 	let epsg: number;
 	let rulesPath = '';
 	let outputPath = '';
+	let sinkParameters = {} as SinkParameters;
 	let isRunning = false;
+	let isConvertButtonDisabled = true;
+
+	$: isConvertButtonDisabled = !inputPaths.length || !outputPath || isRunning;
 	let transformerRegistry: { key: string; label: string; is_enabled: boolean }[];
 
 	async function convertAndSave() {
-		if (!inputPaths) {
-			await message('入力フォルダ/ファイルを選択してください', { type: 'warning' });
-			return;
-		}
-
-		if (!outputPath) {
-			await message('出力先を選択してください', { type: 'warning' });
-			return;
-		}
-
 		isRunning = true;
 
 		const transformerOptions = transformerRegistry.map((transformerConfig) => {
@@ -46,7 +41,8 @@
 				filetype,
 				epsg,
 				rulesPath,
-				transformerOptions
+				transformerOptions,
+				sinkParameters
 			});
 			isRunning = false;
 			await message(`変換が完了しました。\n'${outputPath}' に出力しました。`, { type: 'info' });
@@ -63,7 +59,7 @@
 </script>
 
 {#if isRunning}
-	<div class="absolute inset-0 bg-black/70 backdrop-blur-[2px] z-20">
+	<div class="fixed inset-0 bg-black/70 backdrop-blur-[2px] z-20 h-screen">
 		<LoadingAnimation />
 	</div>
 {/if}
@@ -79,14 +75,23 @@
 
 		<InputSelector bind:inputPaths />
 
-		<SettingSelector bind:filetype bind:epsg bind:rulesPath bind:transformerRegistry />
+		<SettingSelector
+			bind:filetype
+			bind:epsg
+			bind:rulesPath
+			bind:sinkParameters
+			bind:transformerRegistry
+		/>
 
 		<OutputSelector {filetype} bind:outputPath />
 
 		<div class="flex justify-end">
 			<button
 				on:click={convertAndSave}
-				class="bg-accent1 flex items-center font-bold py-1.5 pl-3 pr-5 rounded-full gap-1 shawdow-2xl hover:opacity-75"
+				disabled={isConvertButtonDisabled}
+				class="bg-accent1 flex items-center font-bold py-1.5 pl-3 pr-5 rounded-full gap-1 shawdow-2xl {isConvertButtonDisabled
+					? 'opacity-50'
+					: ''}"
 			>
 				<Icon class="text-lg" icon="ic:baseline-play-arrow" />
 				変換
diff --git a/app/src/routes/InputSelector.svelte b/app/src/routes/InputSelector.svelte
index e086985e..4f8d6f33 100644
--- a/app/src/routes/InputSelector.svelte
+++ b/app/src/routes/InputSelector.svelte
@@ -90,7 +90,7 @@
 			<div class="text-sm">
 				{#if isFolderMode}
 					{#if inputFolders.length === 0}
-						<p class="opacity-50">フォルダが選択されていません</p>
+						<p class="text-red-400">フォルダが選択されていません</p>
 					{:else}
 						<div class="flex items-center gap-1">
 							<p>
@@ -112,7 +112,7 @@
 						</div>
 					{/if}
 				{:else if inputPaths.length === 0}
-					<p class="opacity-50">ファイルが選択されていません</p>
+					<p class="text-red-400">ファイルが選択されていません</p>
 				{:else}
 					<div class="flex items-center gap-1">
 						<p><b>{inputPaths.length}</b> ファイル</p>
diff --git a/app/src/routes/OutputSelector.svelte b/app/src/routes/OutputSelector.svelte
index b2f23594..257563a5 100644
--- a/app/src/routes/OutputSelector.svelte
+++ b/app/src/routes/OutputSelector.svelte
@@ -43,7 +43,7 @@
 				on:click={openOutputDialog}
 				class="bg-accent1 font-semibold rounded px-4 py-0.5 shadow hover:opacity-75">選択</button
 			>
-			<div class="text-sm" class:opacity-50={!outputPath}>
+			<div class="text-sm">
 				{#if outputPath}
 					<div class="flex justify-center items-center gap-1.5">
 						<p><code>{abbreviatePath(outputPath, 40)}</code></p>
@@ -52,7 +52,7 @@
 						</button>
 					</div>
 				{:else}
-					<p>出力先が選択されていません</p>
+					<p class="text-red-400">出力先が選択されていません</p>
 				{/if}
 			</div>
 		</div>
diff --git a/app/src/routes/SettingSelector.svelte b/app/src/routes/SettingSelector.svelte
index 4a20fb29..2e030e3e 100644
--- a/app/src/routes/SettingSelector.svelte
+++ b/app/src/routes/SettingSelector.svelte
@@ -1,12 +1,20 @@
 <script lang="ts">
 	import { filetypeOptions } from '$lib/settings';
+	import { invoke } from '@tauri-apps/api/tauri';
+	import type { SinkParameters } from '$lib/sinkparams';
+	import {
+		isIntegerParameter,
+		isStringParameter,
+		isBooleanParameter,
+		createRangeArray
+	} from '$lib/sinkparams';
 	import Icon from '@iconify/svelte';
 	import { dialog } from '@tauri-apps/api';
-	import { invoke } from '@tauri-apps/api/tauri';
 
 	export let filetype: string;
 	export let epsg: number = 4979;
 	export let rulesPath: string;
+	export let sinkParameters: SinkParameters;
 	export let transformerRegistry: { key: string; label: string; is_enabled: boolean }[];
 
 	$: epsgOptions = filetypeOptions[filetype]?.epsg || [];
@@ -19,6 +27,10 @@
 		}
 	}
 
+	function clearRulesPath() {
+		rulesPath = '';
+	}
+
 	async function openRulesPathDialog() {
 		const res = await dialog.open({
 			filters: [
@@ -32,10 +44,6 @@
 		rulesPath = Array.isArray(res) ? res[0] : res;
 	}
 
-	function clearRulesPath() {
-		rulesPath = '';
-	}
-
 	async function getTransformerRegistry(filetype: string) {
 		const registry = (await invoke('get_transform', { filetype })) as any;
 
@@ -51,6 +59,21 @@
 	}
 
 	$: getTransformerRegistry(filetype);
+
+	let sinkOptionKeys: string[] = [];
+
+	// Get the parameter options for the selected filetype
+	async function setSinkParameter(filetype: string) {
+		const parameters = (await invoke('get_parameter', { filetype })) as SinkParameters;
+
+		// Exclude '@output' and 'transform' from the parameter options list
+		sinkOptionKeys = Object.keys(parameters.items).filter(
+			(item) => item !== '@output' && item !== 'transform'
+		);
+		sinkParameters = parameters;
+	}
+
+	$: setSinkParameter(filetype);
 </script>
 
 <div>
@@ -85,35 +108,81 @@
 				{/each}
 			</select>
 		</div>
-		{#if transformerRegistry && transformerRegistry.length > 0}
+
+		{#if (transformerRegistry && transformerRegistry.length > 0) || sinkOptionKeys.length > 0}
 			<div class="flex flex-col gap-1.5">
 				<label for="transform-select" class="font-bold">出力の詳細設定</label>
-				{#each transformerRegistry as config}
-					<div class="inline-flex items-center gap-6">
-						<label
-							for={config.key}
-							class="mt-px mb-0 ml-3 font-light text-gray-700 cursor-pointer select-none text-sm w-52"
-						>
-							{config.label}
-						</label>
-						<div class="relative inline-block w-10 h-6 rounded-full cursor-pointer">
-							<input
-								bind:checked={config.is_enabled}
-								id={config.key}
-								type="checkbox"
-								class="absolute w-10 h-6 transition-colors duration-300 rounded-full appearance-none cursor-pointer peer bg-gray-200 checked:bg-accent1 peer-checked:before:bg-accent1"
-							/>
-							<label
-								for={config.key}
-								class="before:content[''] absolute top-2/4 -left-1 h-6 w-6 -translate-y-2/4 cursor-pointer rounded-full border border-blue-gray-100 bg-white shadow-md transition-all duration-300 peer-checked:translate-x-full"
-							>
-								<div
-									class="inline-block p-5 rounded-full top-2/4 left-2/4 -translate-x-2/4 -translate-y-2/4"
-								></div>
+
+				<!-- Transformer options -->
+				{#if transformerRegistry && transformerRegistry.length > 0}
+					{#each transformerRegistry as config}
+						<div class="flex gap-2 w-80">
+							<label for={config.key} class="w-3/4">
+								{config.label}
 							</label>
+							<div class="relative inline-block w-10 h-6 rounded-full cursor-pointer">
+								<input
+									bind:checked={config.is_enabled}
+									id={config.key}
+									type="checkbox"
+									class="absolute w-10 h-6 transition-colors duration-300 rounded-full appearance-none cursor-pointer peer bg-gray-200 checked:bg-accent1 peer-checked:before:bg-accent1"
+								/>
+								<label
+									for={config.key}
+									class="before:content[''] absolute top-2/4 -left-1 h-6 w-6 -translate-y-2/4 cursor-pointer rounded-full border border-blue-gray-100 bg-white shadow-md transition-all duration-300 peer-checked:translate-x-full"
+								>
+									<div
+										class="inline-block p-5 rounded-full top-2/4 left-2/4 -translate-x-2/4 -translate-y-2/4"
+									></div>
+								</label>
+							</div>
 						</div>
+					{/each}
+				{/if}
+
+				<!-- Sink options -->
+				{#if sinkOptionKeys.length > 0}
+					<div class="flex flex-col gap-2">
+						{#each sinkOptionKeys as key}
+							{#if isIntegerParameter(sinkParameters.items[key].parameter)}
+								<div class="flex gap-2 w-80">
+									<label for={key} class="w-3/4">{sinkParameters.items[key].label}</label>
+									<select
+										bind:value={sinkParameters.items[key].parameter.Integer.value}
+										id={key}
+										class="w-1/4 border-2 border-gray-300 px-2 rounded-md"
+									>
+										{#each createRangeArray(sinkParameters.items[key].parameter.Integer.min, sinkParameters.items[key].parameter.Integer.max) as value}
+											<option {value} class="text-center">{value}</option>
+										{/each}
+									</select>
+								</div>
+							{:else if isStringParameter(sinkParameters.items[key].parameter)}
+								<!-- TODO String input -->
+								<!-- <div class="flex gap-2 w-80">
+								<label for={key} class="w-3/4">{sinkParameters.items[key].label}</label>
+								<input
+									type="text"
+									id={key}
+									bind:value={sinkParameters.items[key].parameter.String.value}
+									class="w-1/4"
+								/>
+							</div> -->
+							{:else if isBooleanParameter(sinkParameters.items[key].parameter)}
+								<!-- TODO Boolean input -->
+								<!-- <div class="flex gap-2 w-80">
+								<label for={key} class="w-3/4">{sinkParameters.items[key].label}</label>
+								<input
+									type="checkbox"
+									id={key}
+									bind:checked={sinkParameters.items[key].parameter.Boolean.value}
+									class="w-1/4"
+								/>
+							</div> -->
+							{/if}
+						{/each}
 					</div>
-				{/each}
+				{/if}
 			</div>
 		{/if}
 
@@ -133,7 +202,7 @@
 							</button>
 						</div>
 					{:else}
-						<p>設定ファイルが選択されていません</p>
+						<p>設定ファイルを選択してください（任意）</p>
 					{/if}
 				</div>
 			</div>
diff --git a/nusamai/src/parameters/mod.rs b/nusamai/src/parameters/mod.rs
index d9b1239e..9ce20f8a 100644
--- a/nusamai/src/parameters/mod.rs
+++ b/nusamai/src/parameters/mod.rs
@@ -3,7 +3,7 @@
 use std::collections::HashMap;
 
 use indexmap::{map::Entry, IndexMap};
-use serde::Serialize;
+use serde::{Deserialize, Serialize};
 use thiserror::Error;
 
 #[macro_export]
@@ -27,7 +27,7 @@ pub enum Error {
     UnknownParameter(String),
 }
 
-#[derive(Debug, Default, Serialize)]
+#[derive(Debug, Default, Serialize, Deserialize)]
 pub struct Parameters {
     items: IndexMap<String, ParameterEntry>,
 }
@@ -103,11 +103,12 @@ impl Parameters {
     }
 }
 
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Deserialize)]
 pub struct ParameterEntry {
     pub description: String,
     pub required: bool,
     pub parameter: ParameterType,
+    pub label: Option<String>,
 }
 
 impl ParameterEntry {
@@ -141,7 +142,7 @@ impl ParameterEntry {
     }
 }
 
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Deserialize)]
 pub enum ParameterType {
     FileSystemPath(FileSystemPathParameter),
     String(StringParameter),
@@ -150,7 +151,7 @@ pub enum ParameterType {
     // and so on ...
 }
 
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Deserialize)]
 pub struct FileSystemPathParameter {
     pub value: Option<std::path::PathBuf>,
     pub must_exist: bool,
@@ -184,7 +185,7 @@ impl FileSystemPathParameter {
     }
 }
 
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Deserialize)]
 pub struct StringParameter {
     pub value: Option<String>,
 }
@@ -217,7 +218,7 @@ impl StringParameter {
     }
 }
 
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Deserialize)]
 pub struct BooleanParameter {
     pub value: Option<bool>,
 }
@@ -252,7 +253,7 @@ impl BooleanParameter {
     }
 }
 
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Deserialize)]
 pub struct IntegerParameter {
     pub value: Option<i64>,
     pub min: Option<i64>,
@@ -324,6 +325,7 @@ mod tests {
                 description: "test".into(),
                 required: true,
                 parameter: ParameterType::String(StringParameter { value: None }),
+                label: None,
             },
         );
         params.define(
@@ -332,6 +334,7 @@ mod tests {
                 description: "test2".into(),
                 required: false,
                 parameter: ParameterType::String(StringParameter { value: None }),
+                label: Some("test2".into()),
             },
         );
 
@@ -352,6 +355,7 @@ mod tests {
                 description: "test".into(),
                 required: true,
                 parameter: ParameterType::Boolean(BooleanParameter { value: None }),
+                label: None,
             },
         );
         params.define(
@@ -360,6 +364,7 @@ mod tests {
                 description: "test2".into(),
                 required: false,
                 parameter: ParameterType::Boolean(BooleanParameter { value: None }),
+                label: Some("test2".into()),
             },
         );
 
@@ -393,6 +398,7 @@ mod tests {
                     min: Some(4),
                     max: Some(10),
                 }),
+                label: None,
             },
         );
         params.define(
@@ -405,6 +411,7 @@ mod tests {
                     min: Some(4),
                     max: Some(10),
                 }),
+                label: Some("test2".into()),
             },
         );
 
diff --git a/nusamai/src/sink/cesiumtiles/mod.rs b/nusamai/src/sink/cesiumtiles/mod.rs
index 79fae4fb..3a9a51ad 100644
--- a/nusamai/src/sink/cesiumtiles/mod.rs
+++ b/nusamai/src/sink/cesiumtiles/mod.rs
@@ -59,6 +59,7 @@ impl DataSinkProvider for CesiumTilesSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         // TODO: min Zoom
@@ -70,6 +71,7 @@ impl DataSinkProvider for CesiumTilesSinkProvider {
                 description: "transform option".into(),
                 required: false,
                 parameter: ParameterType::String(StringParameter { value: None }),
+                label: None,
             },
         );
 
diff --git a/nusamai/src/sink/czml/mod.rs b/nusamai/src/sink/czml/mod.rs
index f2d57134..3d197127 100644
--- a/nusamai/src/sink/czml/mod.rs
+++ b/nusamai/src/sink/czml/mod.rs
@@ -47,6 +47,7 @@ impl DataSinkProvider for CzmlSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/geojson/mod.rs b/nusamai/src/sink/geojson/mod.rs
index 9262df42..2a6ecf8e 100644
--- a/nusamai/src/sink/geojson/mod.rs
+++ b/nusamai/src/sink/geojson/mod.rs
@@ -48,6 +48,7 @@ impl DataSinkProvider for GeoJsonSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/gltf/mod.rs b/nusamai/src/sink/gltf/mod.rs
index dbd5a255..3983409f 100644
--- a/nusamai/src/sink/gltf/mod.rs
+++ b/nusamai/src/sink/gltf/mod.rs
@@ -48,6 +48,7 @@ impl DataSinkProvider for GltfSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/gpkg/mod.rs b/nusamai/src/sink/gpkg/mod.rs
index ae2c1458..697b2ec9 100644
--- a/nusamai/src/sink/gpkg/mod.rs
+++ b/nusamai/src/sink/gpkg/mod.rs
@@ -49,6 +49,7 @@ impl DataSinkProvider for GpkgSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/kml/mod.rs b/nusamai/src/sink/kml/mod.rs
index 11a75bc6..10ad8ba7 100644
--- a/nusamai/src/sink/kml/mod.rs
+++ b/nusamai/src/sink/kml/mod.rs
@@ -47,6 +47,7 @@ impl DataSinkProvider for KmlSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/minecraft/mod.rs b/nusamai/src/sink/minecraft/mod.rs
index 0ca581f2..f281d6f6 100644
--- a/nusamai/src/sink/minecraft/mod.rs
+++ b/nusamai/src/sink/minecraft/mod.rs
@@ -53,6 +53,7 @@ impl DataSinkProvider for MinecraftSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/mvt/mod.rs b/nusamai/src/sink/mvt/mod.rs
index 63c4b230..8409a2a9 100644
--- a/nusamai/src/sink/mvt/mod.rs
+++ b/nusamai/src/sink/mvt/mod.rs
@@ -53,6 +53,7 @@ impl DataSinkProvider for MvtSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params.define(
@@ -65,6 +66,7 @@ impl DataSinkProvider for MvtSinkProvider {
                     min: Some(0),
                     max: Some(20),
                 }),
+                label: Some("最小ズームレベル".into()),
             },
         );
         params.define(
@@ -77,6 +79,7 @@ impl DataSinkProvider for MvtSinkProvider {
                     min: Some(0),
                     max: Some(20),
                 }),
+                label: Some("最大ズームレベル".into()),
             },
         );
         params
diff --git a/nusamai/src/sink/noop/mod.rs b/nusamai/src/sink/noop/mod.rs
index f7d19ed8..40096423 100644
--- a/nusamai/src/sink/noop/mod.rs
+++ b/nusamai/src/sink/noop/mod.rs
@@ -39,6 +39,7 @@ impl DataSinkProvider for NoopSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         // REMOVE: deprecated
diff --git a/nusamai/src/sink/ply/mod.rs b/nusamai/src/sink/ply/mod.rs
index 0ec16161..72305fb7 100644
--- a/nusamai/src/sink/ply/mod.rs
+++ b/nusamai/src/sink/ply/mod.rs
@@ -56,6 +56,7 @@ impl DataSinkProvider for StanfordPlySinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/serde/mod.rs b/nusamai/src/sink/serde/mod.rs
index c596917a..53664399 100644
--- a/nusamai/src/sink/serde/mod.rs
+++ b/nusamai/src/sink/serde/mod.rs
@@ -40,6 +40,7 @@ impl DataSinkProvider for SerdeSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params
diff --git a/nusamai/src/sink/shapefile/mod.rs b/nusamai/src/sink/shapefile/mod.rs
index 69fa059f..d21e5e76 100644
--- a/nusamai/src/sink/shapefile/mod.rs
+++ b/nusamai/src/sink/shapefile/mod.rs
@@ -28,7 +28,7 @@ use crate::{
     pipeline::{Feedback, PipelineError, Receiver, Result},
     sink::{DataRequirements, DataSink, DataSinkProvider, SinkInfo},
     transformer,
-    transformer::{TransformerRegistry, TransformerOption},
+    transformer::{TransformerOption, TransformerRegistry},
 };
 
 pub struct ShapefileSinkProvider {}
@@ -52,6 +52,7 @@ impl DataSinkProvider for ShapefileSinkProvider {
                     value: None,
                     must_exist: false,
                 }),
+                label: None,
             },
         );
         params