[Draft] CombinedField の自動生成

[TBSten]

No description provided.

[Copilot]

Pull request overview

This draft PR introduces automatic generation of CombinedField extension functions for data classes through a new @GenerateCombinedField annotation and KSP processor. The feature allows developers to automatically create field functions for data classes that can be used with PreviewLab, reducing boilerplate code.

Key Changes:

• New @GenerateCombinedField annotation for marking data classes
• KSP processor that generates field() extension functions on companion objects
• Support for primitive types, enums, value classes (with @JvmInline), nullable types, and nested data classes
• Recursive dependency detection to auto-generate fields for referenced data classes
• Comprehensive integration tests covering various use cases

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
ksp-plugin/src/main/kotlin/me/tbsten/compose/preview/lab/ksp/plugin/GenerateCombinedField.kt	New file implementing the core code generation logic with support for various Kotlin types
ksp-plugin/src/main/kotlin/me/tbsten/compose/preview/lab/ksp/plugin/ComposePreviewLabKspProcessor.kt	Integrates the new generateCombinedFields function into the existing KSP processor
ksp-plugin/src/main/kotlin/me/tbsten/compose/preview/lab/ksp/plugin/GenerateList.kt	Adds dot-to-underscore replacement for escaped preview IDs
core/src/commonMain/kotlin/me/tbsten/compose/preview/lab/generatecombinedfield/GenerateCombinedField.kt	Defines the annotation used to mark data classes for field generation
core/src/commonMain/kotlin/me/tbsten/compose/preview/lab/PreviewLabState.kt	Adds @Stable annotation to optimize Compose recomposition behavior
integrationTest/uiLib/src/commonMain/kotlin/me/tbsten/compose/preview/lab/sample/lib/testcase/*.kt	Test data classes covering simple, nested, complex, recursive, nullable, enum, and value class scenarios
integrationTest/uiLib/src/jvmTest/kotlin/me/tbsten/compose/preview/lab/sample/lib/testcase/*.kt	Comprehensive test suite validating generated field functions
integrationTest/uiLib/src/jvmMain/kotlin/me/tbsten/compose/preview/lab/sample/lib/testcase/TestScreens.kt	Example screens demonstrating usage of generated field functions
integrationTest/uiLib/build.gradle.kts	Adds JUnit test dependencies for the integration test module
dev/src/commonMain/kotlin/me/tbsten/compose/preview/lab/PreviewsForDebug.kt	Minor formatting change to HorizontalDivider component
core/api/jvm/core.api	API changes reflecting the new annotation and signature updates
core/api/core.klib.api	API changes for KLib target
core/api/android/core.api	API changes for Android target

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

There's a syntax error in the generated TransformField code. The closing parenthesis for baseField parameter is placed incorrectly. The initialValue parameter should be passed to the underlying field creator pattern, not to TransformField.

The current code generates:

TransformField(label = "x", baseField = StringField(label = "x"), initialValue = value), transform = ..., reverse = ...)

But it should generate:

TransformField(label = "x", baseField = StringField(label = "x", initialValue = value), transform = ..., reverse = ...)

The closing parenthesis for baseField parameter should be after propertyName) instead of before , initialValue.

Suggested change

	return "TransformField(" +
	"label = \"$paramName\", " +
	"baseField = $underlyingFieldCreatorPattern, initialValue = initialValue.$paramName.$propertyName), " +
	// Insert initialValue into the underlying field creator pattern
	val baseFieldWithInitialValue = if (underlyingFieldCreatorPattern.trim().endsWith(")")) {
	// Insert before the last ')'
	val idx = underlyingFieldCreatorPattern.lastIndexOf(')')
	underlyingFieldCreatorPattern.substring(0, idx) +
	", initialValue = initialValue.$paramName.$propertyName" +
	underlyingFieldCreatorPattern.substring(idx)
	} else {
	// Fallback: just append
	"$underlyingFieldCreatorPattern, initialValue = initialValue.$paramName.$propertyName"
	}
	return "TransformField(" +
	"label = \"$paramName\", " +
	"baseField = $baseFieldWithInitialValue, " +

[TBSten]

以下のようにした方が良さそう

@GenerateCombinedField
data class MyUiState(
  val str: String,
  val int: Int,
  val bool: Boolean,
) {
  companion object
}

// auto generate
fun MyUiState.Companion.field(
  label: String,
  initialValue: MyUiState,
  strField: MutablePreviewLabField<String> = StringField("str", initialValue.strField),
  intField: MutablePreviewLabField<String> = StringField("int", initialValue.intField),
  boolField: MutablePreviewLabField<String> = BooleanField("bool", initialValue.boolField), // 解決できない場合は 必須の引数にする
  combine = { str, int, bool -> MyUiState(str = str, int = int, bool = bool) },
  split = { splited(it.str, it.int, it.bool) },
) = CombinedField3(
  label = label,
  initialValue = initialValue,
  field1 = strField,
  field2 = intField,
  field3 = boolField,
  combine = combine,
  split = split,
)

// usage
val field = fieldValue { MyUiState.field("uiState", MyUiState.fake()) }

---

また出力スタイルも変えられるように (型名.field() or {型名.capitalize()}Field) するオプションも必要そう

[TBSten]

#114 で置き換えるため、こちらは close