tejpratap46 · tejpratap46 · Jul 4, 2026 · Jul 3, 2026 · coderabbitai · Jul 4, 2026
diff --git a/.editorconfig b/.editorconfig
@@ -0,0 +1,2 @@
+[*.{kt,kts}]
+ktlint_standard_function-naming = disabled
diff --git a/.github/workflows/ktlint.yml b/.github/workflows/ktlint.yml
@@ -27,3 +27,9 @@ jobs:
           report-path: build/*.xml # Support glob patterns by https://www.npmjs.com/package/@actions/glob
           ignore-warnings: true # Ignore Lint Warnings
         continue-on-error: false # If annotations contain error of severity, action-ktlint exit 1.
+      - name: Upload ktlint report
+        uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: ktlint-report
+          path: build/ktlint-report.xml
diff --git a/.idea/deploymentTargetSelector.xml b/.idea/deploymentTargetSelector.xml
diff --git a/modules/app/src/main/java/com/tejpratapsingh/animator/ui/view/RenaultCar.kt b/modules/app/src/main/java/com/tejpratapsingh/animator/ui/view/RenaultCar.kt
@@ -20,7 +20,7 @@ class RenaultCar(
     effects: List<MotionEffect> = emptyList(),
 ) : BaseContourMotionView(context, startFrame, endFrame, effects = effects) {
     companion object {
-        const val imageAssetSubFolder = "renault_kiger_bg"
+        const val IMAGE_ASSET_SUB_FOLDER = "renault_kiger_bg"
     }
 
     private val imageView: ImageView =
@@ -78,7 +78,7 @@ class RenaultCar(
         )
 
         // Determine which image to show based on the current frame
-        val imageName = "$imageAssetSubFolder/$frame.png"
+        val imageName = "$IMAGE_ASSET_SUB_FOLDER/$frame.png"
 
         try {
             val inputStream: InputStream = assetManager.open(imageName)

diff --git a/modules/core/src/main/java/com/tejpratapsingh/motionlib/core/MotionLayoutInfo.kt b/modules/core/src/main/java/com/tejpratapsingh/motionlib/core/MotionLayoutInfo.kt
@@ -12,20 +12,20 @@ data class MotionLayoutInfo(
     val height: Int = WRAP_CONTENT,
     val padding: Padding = Padding(),
     val margin: Margin = Margin(),
-    val gravity: Int = Gravity.NO_GRAVITY
+    val gravity: Int = Gravity.NO_GRAVITY,
 ) {
     data class Padding(
         val left: Int = 0,
         val top: Int = 0,
         val right: Int = 0,
-        val bottom: Int = 0
+        val bottom: Int = 0,
     )
 
     data class Margin(
         val left: Int = 0,
         val top: Int = 0,
         val right: Int = 0,
-        val bottom: Int = 0
+        val bottom: Int = 0,
     )
 
     companion object {

diff --git a/modules/core/src/main/java/com/tejpratapsingh/motionlib/core/VideoAspectRatio.kt b/modules/core/src/main/java/com/tejpratapsingh/motionlib/core/VideoAspectRatio.kt
@@ -1,5 +1,6 @@
 package com.tejpratapsingh.motionlib.core
 
+@Suppress("ClassName")
 sealed class VideoAspectRatio(
     val width: Int,
     val height: Int,

diff --git a/modules/core/src/test/java/com/tejpratapsingh/motionlib/core/MotionTextSizeProviderTest.kt b/modules/core/src/test/java/com/tejpratapsingh/motionlib/core/MotionTextSizeProviderTest.kt
@@ -5,7 +5,6 @@ import org.junit.Before
 import org.junit.Test
 
 class MotionTextSizeProviderTest {
-
     @Before
     fun setup() {
         // Reset baseTextScale before each test
@@ -15,33 +14,33 @@ class MotionTextSizeProviderTest {
     @Test
     fun testDefaultFontSizes() {
         val aspectRatio = VideoAspectRatio.Ratio16x9_1080 // 1920x1080, min is 1080, scale is 1.0
-        
+
         // H1 should be 160f * 1.0 = 160f
         assertEquals(160f, MotionTextSizeProvider.getFontSize(aspectRatio, MotionTextVariant.H1), 0.01f)
-        
+
         // P should be 48f * 1.0 = 48f
         assertEquals(48f, MotionTextSizeProvider.getFontSize(aspectRatio, MotionTextVariant.P), 0.01f)
     }
 
     @Test
     fun testBaseTextScale() {
         val aspectRatio = VideoAspectRatio.Ratio16x9_1080 // 1920x1080, min is 1080, scale is 1.0
-        
+
         MotionTextSizeProvider.baseTextScale = 2.0f
-        
+
         // H1 should be 160f * 2.0 = 320f
         assertEquals(320f, MotionTextSizeProvider.getFontSize(aspectRatio, MotionTextVariant.H1), 0.01f)
-        
+
         // P should be 48f * 2.0 = 96f
         assertEquals(96f, MotionTextSizeProvider.getFontSize(aspectRatio, MotionTextVariant.P), 0.01f)
     }
 
     @Test
     fun testScalingWithAspectRatio() {
-        // 480p 16:9 is 854x480, min is 480. 
+        // 480p 16:9 is 854x480, min is 480.
         // Scale = 480 / 1080 = 0.4444...
         val aspectRatio = VideoAspectRatio.Ratio16x9_480
-        
+
         val expectedH1 = 160f * (480f / 1080f)
         assertEquals(expectedH1, MotionTextSizeProvider.getFontSize(aspectRatio, MotionTextVariant.H1), 0.01f)
     }

diff --git a/modules/ivi-demo/build.gradle b/modules/ivi-demo/build.gradle
@@ -3,11 +3,11 @@ plugins {
 }
 
 android {
-    namespace 'com.tejpratapsingh.ivi_demo'
+    namespace 'com.tejpratapsingh.ividemo'
     compileSdk 36
 
     defaultConfig {
-        applicationId "com.tejpratapsingh.ivi_demo"
+        applicationId "com.tejpratapsingh.ividemo"
         minSdk 28
         targetSdk 36
         versionCode 1

diff --git a/...singh/ivi_demo/ExampleInstrumentedTest.kt → ...psingh/ividemo/ExampleInstrumentedTest.kt b/...singh/ivi_demo/ExampleInstrumentedTest.kt → ...psingh/ividemo/ExampleInstrumentedTest.kt
@@ -1,4 +1,4 @@
-package com.tejpratapsingh.ivi_demo
+package com.tejpratapsingh.ividemo
 
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import androidx.test.platform.app.InstrumentationRegistry

diff --git a/...m/tejpratapsingh/ivi_demo/MainActivity.kt → ...om/tejpratapsingh/ividemo/MainActivity.kt b/...m/tejpratapsingh/ivi_demo/MainActivity.kt → ...om/tejpratapsingh/ividemo/MainActivity.kt
@@ -1,9 +1,8 @@
-package com.tejpratapsingh.ivi_demo
+package com.tejpratapsingh.ividemo
 
-import RenaultCar
-import android.os.Build
 import android.os.Bundle
-import com.tejpratapsingh.ivi_demo.extension.enableSwipeSeekReverse
+import com.tejpratapsingh.ividemo.extension.enableSwipeSeekReverse
+import com.tejpratapsingh.ividemo.motion.RenaultCar
 import com.tejpratapsingh.motionlib.activities.PreviewActivity
 import com.tejpratapsingh.motionlib.core.MotionConfig
 import com.tejpratapsingh.motionlib.core.VideoAspectRatio
@@ -39,9 +38,7 @@ class MainActivity : PreviewActivity() {
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
-        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
-            motionVideoPlayer.seekBar.min = 1
-        }
+        motionVideoPlayer.seekBar.min = 1
 
         motionView.enableSwipeSeekReverse(
             maxProgress = video.totalFrames,

diff --git a/...ingh/ivi_demo/extension/ViewExtensions.kt → ...singh/ividemo/extension/ViewExtensions.kt b/...ingh/ivi_demo/extension/ViewExtensions.kt → ...singh/ividemo/extension/ViewExtensions.kt
@@ -1,4 +1,4 @@
-package com.tejpratapsingh.ivi_demo.extension
+package com.tejpratapsingh.ividemo.extension
 
 import android.annotation.SuppressLint
 import android.view.MotionEvent

diff --git a/...pratapsingh/ivi_demo/motion/RenaultCar.kt → ...jpratapsingh/ividemo/motion/RenaultCar.kt b/...pratapsingh/ivi_demo/motion/RenaultCar.kt → ...jpratapsingh/ividemo/motion/RenaultCar.kt
@@ -1,3 +1,5 @@
+package com.tejpratapsingh.ividemo.motion
+
 import android.content.Context
 import android.graphics.BitmapFactory
 import android.widget.ImageView
@@ -22,8 +24,8 @@ class RenaultCar(
     effects: List<MotionEffect> = emptyList(),
 ) : BaseContourMotionView(context, startFrame, endFrame, effects = effects) {
     companion object {
-        const val imageAssetSubFolder = "renault_kiger_bg"
-        const val roadAssetSubFolder = "road"
+        const val IMAGE_ASSET_SUB_FOLDER = "renault_kiger_bg"
+        const val ROAD_ASSET_SUB_FOLDER = "road"
     }
 
     private val imageViewBg: ImageView =
@@ -37,8 +39,8 @@ class RenaultCar(
         }
 
     private val assetManager = context.assets
-    private val files = assetManager.list(imageAssetSubFolder)
-    private val roadFiles = assetManager.list(roadAssetSubFolder)
+    private val files = assetManager.list(IMAGE_ASSET_SUB_FOLDER)
+    private val roadFiles = assetManager.list(ROAD_ASSET_SUB_FOLDER)
 
     init {
         imageViewBg.layoutBy(
@@ -120,7 +122,7 @@ class RenaultCar(
 //        val road = String.format(
 //            Locale.getDefault(),
 //            "%s/%02d.png",
-//            roadAssetSubFolder,
+//            ROAD_ASSET_SUB_FOLDER,
 //            min(roadInterpolator, (roadFiles?.size ?: 1) - 1)
 //        )
 //
@@ -137,7 +139,7 @@ class RenaultCar(
             String.format(
                 Locale.getDefault(),
                 "%s/%d.png",
-                imageAssetSubFolder,
+                IMAGE_ASSET_SUB_FOLDER,
                 min(frame, (files?.size ?: 1) - 1),
             )
 

diff --git a/...om/tejpratapsingh/ivi_demo/motion/Road.kt → ...com/tejpratapsingh/ividemo/motion/Road.kt b/...om/tejpratapsingh/ivi_demo/motion/Road.kt → ...com/tejpratapsingh/ividemo/motion/Road.kt
@@ -1,4 +1,4 @@
-package com.tejpratapsingh.ivi_demo.motion
+package com.tejpratapsingh.ividemo.motion
 
 import android.content.Context
 import android.graphics.BitmapFactory
@@ -18,7 +18,7 @@ class Road(
     effects: List<MotionEffect> = emptyList(),
 ) : BaseContourMotionView(context, startFrame, endFrame, effects = effects) {
     companion object {
-        const val imageAssetSubFolder = "road"
+        const val IMAGE_ASSET_SUB_FOLDER = "road"
     }
 
     private val imageView: ImageView =
@@ -60,7 +60,7 @@ class Road(
         super.forFrame(frame)
 
         // Determine which image to show based on the current frame
-        val imageName = "$imageAssetSubFolder/$frame.png"
+        val imageName = "$IMAGE_ASSET_SUB_FOLDER/$frame.png"
 
         try {
             val inputStream: InputStream = assetManager.open(imageName)

diff --git a/...ingh/ivi_demo/sequence/RenaultSequence.kt → ...singh/ividemo/sequence/RenaultSequence.kt b/...ingh/ivi_demo/sequence/RenaultSequence.kt → ...singh/ividemo/sequence/RenaultSequence.kt
@@ -1,7 +1,7 @@
-package com.tejpratapsingh.ivi_demo.sequence
+package com.tejpratapsingh.ividemo.sequence
 
-import RenaultCar
 import android.content.Context
+import com.tejpratapsingh.ividemo.motion.RenaultCar
 import com.tejpratapsingh.motionlib.core.MotionConfig
 import com.tejpratapsingh.motionlib.core.VideoAspectRatio
 import com.tejpratapsingh.motionlib.core.motion.BaseContourMotionView
@@ -18,7 +18,7 @@ fun sampleMotionVideo(applicationContext: Context): MotionVideoProducer {
     setCurrentConfig(motionConfig)
 
     val assetManager = applicationContext.assets
-    val files = assetManager.list(RenaultCar.imageAssetSubFolder)
+    val files = assetManager.list(RenaultCar.IMAGE_ASSET_SUB_FOLDER)
 
     val motionView: BaseContourMotionView =
         RenaultCar(

diff --git a/...singh/ivi_demo/view/TrapezoidImageView.kt → ...psingh/ividemo/view/TrapezoidImageView.kt b/...singh/ivi_demo/view/TrapezoidImageView.kt → ...psingh/ividemo/view/TrapezoidImageView.kt
@@ -1,4 +1,4 @@
-package com.tejpratapsingh.ivi_demo.view
+package com.tejpratapsingh.ividemo.view
 
 import android.content.Context
 import android.graphics.Bitmap

diff --git a/...ejpratapsingh/ivi_demo/ExampleUnitTest.kt → ...tejpratapsingh/ividemo/ExampleUnitTest.kt b/...ejpratapsingh/ivi_demo/ExampleUnitTest.kt → ...tejpratapsingh/ividemo/ExampleUnitTest.kt
@@ -1,4 +1,4 @@
-package com.tejpratapsingh.ivi_demo
+package com.tejpratapsingh.ividemo
 
 import org.junit.Assert.assertEquals
 import org.junit.Test

diff --git a/...Test/java/com/tejpratapsingh/lyricsmaker/presentation/compose/ProjectDetailsScreenTest.kt b/...Test/java/com/tejpratapsingh/lyricsmaker/presentation/compose/ProjectDetailsScreenTest.kt
@@ -28,7 +28,6 @@ import org.junit.runner.RunWith
  */
 @RunWith(AndroidJUnit4::class)
 class ProjectDetailsScreenTest {
-
     @get:Rule
     val composeTestRule = createComposeRule()
 
@@ -72,6 +71,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = {},
+                onEditClick = {},
                 onShareClick = {},
             )
         }
@@ -91,6 +91,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = {},
+                onEditClick = {},
                 onShareClick = {},
             )
         }
@@ -113,6 +114,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = {},
+                onEditClick = {},
                 onShareClick = {},
             )
         }
@@ -133,6 +135,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = {},
+                onEditClick = {},
                 onShareClick = {},
             )
         }
@@ -149,6 +152,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = {},
+                onEditClick = {},
                 onShareClick = { sharedProject = it },
             )
         }
@@ -170,6 +174,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = {},
+                onEditClick = {},
                 onShareClick = {},
             )
         }
@@ -186,6 +191,7 @@ class ProjectDetailsScreenTest {
             ProjectDetailsScreen(
                 project = project,
                 onBackClick = { backClicked = true },
+                onEditClick = {},
                 onShareClick = {},
             )
         }
@@ -194,4 +200,43 @@ class ProjectDetailsScreenTest {
 
         assertEquals(true, backClicked)
     }
-}
+
+    // ------------------------------------------------------------------
+    // Edit button
+    // ------------------------------------------------------------------
+
+    @Test
+    fun editButton_isDisplayed() {
+        val project = buildProject()
+
+        composeTestRule.setContent {
+            ProjectDetailsScreen(
+                project = project,
+                onBackClick = {},
+                onEditClick = {},
+                onShareClick = {},
+            )
+        }
+
+        composeTestRule.onNodeWithText("Edit Project").assertIsDisplayed()
+    }
+
+    @Test
+    fun editButton_invokesOnEditClickWithProject() {
+        val project = buildProject(id = "edit-test")
+        var editedProject: MotionProject? = null
+
+        composeTestRule.setContent {
+            ProjectDetailsScreen(
+                project = project,
+                onBackClick = {},
+                onEditClick = { editedProject = it },
+                onShareClick = {},
+            )
+        }
+
+        composeTestRule.onNodeWithText("Edit Project").performClick()
+
+        assertEquals(project.id, editedProject?.id)
+    }
+}
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		[*.{kt,kts}]
		ktlint_standard_function-naming = disabled