From a3a88c7631d1877c80f4366dd704f6de9ea0c413 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 11:29:02 +0530 Subject: [PATCH 01/15] added extra logs --- .../lyricsmaker/presentation/activity/LyricsActivity.kt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt index 23bf144e..f31dca8c 100644 --- a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt +++ b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt @@ -4,6 +4,7 @@ import android.content.Context import android.content.Intent import android.os.Build import android.os.Bundle +import android.util.Log import com.google.android.material.dialog.MaterialAlertDialogBuilder import com.tejpratapsingh.lyricsmaker.data.lrc.SyncedLyricFrame import com.tejpratapsingh.lyricsmaker.presentation.motion.getLyricsVideoProducer @@ -31,6 +32,7 @@ class LyricsActivity : PreviewActivity() { lyrics: ArrayList, socialMeta: SocialMeta? = null, ) { + Log.d(TAG,"start") context.startActivity( Intent(context, LyricsActivity::class.java).also { it.putExtra(SONG, song) @@ -71,7 +73,7 @@ class LyricsActivity : PreviewActivity() { override fun onCreate(savedInstanceState: Bundle?) { super.onCreate(savedInstanceState) - + Log.d("LyricsActivity","onCreate") val start = lyrics.minBy { it.frame }.frame val end = lyrics.maxBy { it.frame }.frame From 56f603a6c3ce554eb72bc028fa4ceb7c4435e468 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 13:45:21 +0530 Subject: [PATCH 02/15] Add Kotlin PR review workflow configuration --- kotlin-review.yml | 43 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) create mode 100644 kotlin-review.yml diff --git a/kotlin-review.yml b/kotlin-review.yml new file mode 100644 index 00000000..0de902ee --- /dev/null +++ b/kotlin-review.yml @@ -0,0 +1,43 @@ +name: Kotlin PR Review + +on: + pull_request: + types: [opened, reopened, synchronize] + +jobs: + kotlin_review: + runs-on: code-review # ← matches the label you gave your runner + + timeout-minutes: 15 + + steps: + - name: Checkout + uses: actions/checkout@v4 + with: + fetch-depth: 0 # full history needed for git diff + + - name: Run Kotlin Review Agent + env: + PLATFORM: github + REVIEWER_TOKEN: ${{ secrets.REVIEWER_TOKEN }} + OLLAMA_HOST: ${{ secrets.OLLAMA_HOST }} + OLLAMA_MODEL: ${{ secrets.OLLAMA_MODEL }} + # Map GitHub context vars to the names review_ci.py expects + CI_PROJECT_DIR: ${{ github.workspace }} + CI_PROJECT_ID: ${{ github.repository }} # e.g. hemusimple/AndroidVideoMotion + CI_MERGE_REQUEST_IID: ${{ github.event.pull_request.number }} + CI_MERGE_REQUEST_TARGET_BRANCH_NAME: ${{ github.event.pull_request.base.ref }} + CI_MERGE_REQUEST_SOURCE_BRANCH_NAME: ${{ github.event.pull_request.head.ref }} + CI_MERGE_REQUEST_TITLE: ${{ github.event.pull_request.title }} + CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} + run: | + pip install -q -r /home/rntbci/AIML/code-reviewer/requirements.txt + python /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log + + - name: Upload review log + uses: actions/upload-artifact@v4 + if: always() + with: + name: review-output + path: review_output.log + retention-days: 7 From 04cab6638a961cfd701fdadd3df6699a05da3f97 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 14:19:17 +0530 Subject: [PATCH 03/15] Add Kotlin PR review workflow configuration --- .github/workflows/kotlin-review.yml | 43 +++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) create mode 100644 .github/workflows/kotlin-review.yml diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml new file mode 100644 index 00000000..0de902ee --- /dev/null +++ b/.github/workflows/kotlin-review.yml @@ -0,0 +1,43 @@ +name: Kotlin PR Review + +on: + pull_request: + types: [opened, reopened, synchronize] + +jobs: + kotlin_review: + runs-on: code-review # ← matches the label you gave your runner + + timeout-minutes: 15 + + steps: + - name: Checkout + uses: actions/checkout@v4 + with: + fetch-depth: 0 # full history needed for git diff + + - name: Run Kotlin Review Agent + env: + PLATFORM: github + REVIEWER_TOKEN: ${{ secrets.REVIEWER_TOKEN }} + OLLAMA_HOST: ${{ secrets.OLLAMA_HOST }} + OLLAMA_MODEL: ${{ secrets.OLLAMA_MODEL }} + # Map GitHub context vars to the names review_ci.py expects + CI_PROJECT_DIR: ${{ github.workspace }} + CI_PROJECT_ID: ${{ github.repository }} # e.g. hemusimple/AndroidVideoMotion + CI_MERGE_REQUEST_IID: ${{ github.event.pull_request.number }} + CI_MERGE_REQUEST_TARGET_BRANCH_NAME: ${{ github.event.pull_request.base.ref }} + CI_MERGE_REQUEST_SOURCE_BRANCH_NAME: ${{ github.event.pull_request.head.ref }} + CI_MERGE_REQUEST_TITLE: ${{ github.event.pull_request.title }} + CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} + run: | + pip install -q -r /home/rntbci/AIML/code-reviewer/requirements.txt + python /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log + + - name: Upload review log + uses: actions/upload-artifact@v4 + if: always() + with: + name: review-output + path: review_output.log + retention-days: 7 From 3a6ea4f504bfbba171fbf6806b7e6677c2305ebe Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 14:19:41 +0530 Subject: [PATCH 04/15] Delete kotlin-review.yml --- kotlin-review.yml | 43 ------------------------------------------- 1 file changed, 43 deletions(-) delete mode 100644 kotlin-review.yml diff --git a/kotlin-review.yml b/kotlin-review.yml deleted file mode 100644 index 0de902ee..00000000 --- a/kotlin-review.yml +++ /dev/null @@ -1,43 +0,0 @@ -name: Kotlin PR Review - -on: - pull_request: - types: [opened, reopened, synchronize] - -jobs: - kotlin_review: - runs-on: code-review # ← matches the label you gave your runner - - timeout-minutes: 15 - - steps: - - name: Checkout - uses: actions/checkout@v4 - with: - fetch-depth: 0 # full history needed for git diff - - - name: Run Kotlin Review Agent - env: - PLATFORM: github - REVIEWER_TOKEN: ${{ secrets.REVIEWER_TOKEN }} - OLLAMA_HOST: ${{ secrets.OLLAMA_HOST }} - OLLAMA_MODEL: ${{ secrets.OLLAMA_MODEL }} - # Map GitHub context vars to the names review_ci.py expects - CI_PROJECT_DIR: ${{ github.workspace }} - CI_PROJECT_ID: ${{ github.repository }} # e.g. hemusimple/AndroidVideoMotion - CI_MERGE_REQUEST_IID: ${{ github.event.pull_request.number }} - CI_MERGE_REQUEST_TARGET_BRANCH_NAME: ${{ github.event.pull_request.base.ref }} - CI_MERGE_REQUEST_SOURCE_BRANCH_NAME: ${{ github.event.pull_request.head.ref }} - CI_MERGE_REQUEST_TITLE: ${{ github.event.pull_request.title }} - CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} - run: | - pip install -q -r /home/rntbci/AIML/code-reviewer/requirements.txt - python /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log - - - name: Upload review log - uses: actions/upload-artifact@v4 - if: always() - with: - name: review-output - path: review_output.log - retention-days: 7 From 7b9d97b28c9a2dc8f2e62eba38568440b0ec9039 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 14:23:12 +0530 Subject: [PATCH 05/15] Update Python command to use python3 in CI workflow --- .github/workflows/kotlin-review.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index 0de902ee..818deaee 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -32,7 +32,7 @@ jobs: CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} run: | pip install -q -r /home/rntbci/AIML/code-reviewer/requirements.txt - python /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log + python3 /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log - name: Upload review log uses: actions/upload-artifact@v4 From aca19ec7329b0f408837428d14167b29275c5ec2 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 14:26:44 +0530 Subject: [PATCH 06/15] Change pip install command to pip3 with specific packages Updated Python package installation in CI workflow. --- .github/workflows/kotlin-review.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index 818deaee..978432c5 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -31,7 +31,7 @@ jobs: CI_MERGE_REQUEST_TITLE: ${{ github.event.pull_request.title }} CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} run: | - pip install -q -r /home/rntbci/AIML/code-reviewer/requirements.txt + pip3 install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity python3 /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log - name: Upload review log From 6b998e775fb26fbb46d2ed72f9e70135ffca53ea Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 14:34:04 +0530 Subject: [PATCH 07/15] Add getData function to LyricsActivity Added getData function to LyricsActivity. --- .../lyricsmaker/presentation/activity/LyricsActivity.kt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt index f31dca8c..593d038a 100644 --- a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt +++ b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt @@ -74,6 +74,7 @@ class LyricsActivity : PreviewActivity() { override fun onCreate(savedInstanceState: Bundle?) { super.onCreate(savedInstanceState) Log.d("LyricsActivity","onCreate") + val data = getdata() val start = lyrics.minBy { it.frame }.frame val end = lyrics.maxBy { it.frame }.frame @@ -100,4 +101,6 @@ class LyricsActivity : PreviewActivity() { } override fun getMotionVideo(): MotionVideoProducer = video + + fun getData() = "user data" } From 49e27b63812b3fcc83eb5628d56f03f59519ba3e Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 14:50:37 +0530 Subject: [PATCH 08/15] Update getData method to return string representation --- .../lyricsmaker/presentation/activity/LyricsActivity.kt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt index 593d038a..6b5ef45f 100644 --- a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt +++ b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt @@ -102,5 +102,5 @@ class LyricsActivity : PreviewActivity() { override fun getMotionVideo(): MotionVideoProducer = video - fun getData() = "user data" + fun getData() = "user data".toString() } From 360576e089f56780a36b3c46a02056d45208a5bc Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Thu, 16 Apr 2026 15:03:34 +0530 Subject: [PATCH 09/15] Add song property to LyricsActivity --- .../lyricsmaker/presentation/activity/LyricsActivity.kt | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt index 6b5ef45f..a550afc9 100644 --- a/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt +++ b/modules/lyrics-maker/src/main/java/com/tejpratapsingh/lyricsmaker/presentation/activity/LyricsActivity.kt @@ -43,6 +43,7 @@ class LyricsActivity : PreviewActivity() { } } + private val song: String get() = intent.getStringExtra(SONG) ?: "" From 571eb7e039935060a0e18e1391c551a06dfd4f39 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Sun, 19 Apr 2026 00:39:26 +0530 Subject: [PATCH 10/15] Update Python installation and script path in workflow --- .github/workflows/kotlin-review.yml | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index 978432c5..c6e27dab 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -30,9 +30,12 @@ jobs: CI_MERGE_REQUEST_SOURCE_BRANCH_NAME: ${{ github.event.pull_request.head.ref }} CI_MERGE_REQUEST_TITLE: ${{ github.event.pull_request.title }} CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} + run: | - pip3 install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity - python3 /home/rntbci/AIML/code-reviewer/review_ci.py 2>&1 | tee review_output.log + pip install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity + python D:\AIML\code-reviewer\review_ci.py 2>&1 | Tee-Object -FilePath review_output.log + shell: pwsh + - name: Upload review log uses: actions/upload-artifact@v4 From cb731eef5f513b7387c95a49c1ce1bacfbbdb999 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Sun, 19 Apr 2026 00:48:39 +0530 Subject: [PATCH 11/15] Activate virtual environment in CI workflow Activate Python virtual environment before running the review script. --- .github/workflows/kotlin-review.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index c6e27dab..fb6c356c 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -32,6 +32,7 @@ jobs: CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} run: | + D:\AIML\code-reviewer\myvenv\Scripts\activate.ps1 pip install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity python D:\AIML\code-reviewer\review_ci.py 2>&1 | Tee-Object -FilePath review_output.log shell: pwsh From 4e1e7d0a26b2bd6ff191291d621968dddbc163ca Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Sun, 19 Apr 2026 00:50:43 +0530 Subject: [PATCH 12/15] Change shell from pwsh to powershell in workflow --- .github/workflows/kotlin-review.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index fb6c356c..ada4d5d4 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -35,7 +35,7 @@ jobs: D:\AIML\code-reviewer\myvenv\Scripts\activate.ps1 pip install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity python D:\AIML\code-reviewer\review_ci.py 2>&1 | Tee-Object -FilePath review_output.log - shell: pwsh + shell: powershell - name: Upload review log From 8fd40d013ae99414516f2d45a5f41b04ee9e9ee7 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Sun, 19 Apr 2026 00:53:37 +0530 Subject: [PATCH 13/15] Fix output redirection in Kotlin review workflow --- .github/workflows/kotlin-review.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index ada4d5d4..731e1c3a 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -34,7 +34,8 @@ jobs: run: | D:\AIML\code-reviewer\myvenv\Scripts\activate.ps1 pip install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity - python D:\AIML\code-reviewer\review_ci.py 2>&1 | Tee-Object -FilePath review_output.log + python D:\AIML\code-reviewer\review_ci.py > review_output.log 2>&1 + exit 0 shell: powershell From 57d9a7832ec428063ac661b1c165daac4e78533f Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Sun, 19 Apr 2026 00:55:21 +0530 Subject: [PATCH 14/15] Change activation script and shell for CI workflow --- .github/workflows/kotlin-review.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index 731e1c3a..928ffdaf 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -32,11 +32,11 @@ jobs: CI_COMMIT_AUTHOR: ${{ github.event.pull_request.user.login }} run: | - D:\AIML\code-reviewer\myvenv\Scripts\activate.ps1 + call D:\AIML\code-reviewer\myvenv\Scripts\activate.bat pip install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity python D:\AIML\code-reviewer\review_ci.py > review_output.log 2>&1 exit 0 - shell: powershell + shell: cmd - name: Upload review log From c5c17e8531430722ad06ad851cdd70e0a147c0e7 Mon Sep 17 00:00:00 2001 From: hemanthnaidu Date: Sun, 19 Apr 2026 01:00:52 +0530 Subject: [PATCH 15/15] Modify review script execution in workflow Removed output redirection from the review script execution. --- .github/workflows/kotlin-review.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/kotlin-review.yml b/.github/workflows/kotlin-review.yml index 928ffdaf..1438d07b 100644 --- a/.github/workflows/kotlin-review.yml +++ b/.github/workflows/kotlin-review.yml @@ -34,7 +34,7 @@ jobs: run: | call D:\AIML\code-reviewer\myvenv\Scripts\activate.bat pip install -q httpx python-dotenv pydantic pydantic-settings ollama loguru tenacity - python D:\AIML\code-reviewer\review_ci.py > review_output.log 2>&1 + python D:\AIML\code-reviewer\review_ci.py exit 0 shell: cmd