[Automated] Sync main after 1.9.0 release#443
Open
ballerina-bot wants to merge 5 commits intomainfrom
Open
Conversation
ballerina-bot
requested review from
daneshk,
lnash94 and
shafreenAnfar
as code owners
📝 WalkthroughWalkthroughRelease version 1.9.0 with multi-model support for persist. Updates changelog, increments gradle version to 1.9.1-SNAPSHOT, normalizes stdlib dependency versions, and bumps persist.sql-native dependency across test resource manifests. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #443 +/- ##
=======================================
Coverage 85.04% 85.04%
=======================================
Files 68 68
Lines 6808 6808
Branches 921 921
=======================================
Hits 5790 5790
Misses 730 730
Partials 288 288 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Update changelog for version 1.9.0 release with new features.
TharmiganK
approved these changes
Feb 24, 2026
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
♻️ Duplicate comments (45)
persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_92/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_92/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version ("version" key in Ballerina.toml) which should be replaced with a stable or deterministic value; edit the version entry in Ballerina.toml (the "version" key in tool_test_generate_92/Ballerina.toml) to use a fixed semantic version (e.g., 1.7.2 or a CI-friendly placeholder) instead of the timestamped pre-release string to avoid nondeterministic builds.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_120_init_params_mssql/Ballerina.toml (1)
10-10: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_120_init_params_mssql/Ballerina.toml` at line 10, Replace the timestamped pre-release version string in the version entry (the line starting with version =) with a canonical release version (e.g., "1.7.2" or the project’s agreed-upon semver) so it does not include the timestamp/hash pre-release suffix; ensure the value matches the corresponding version used in tool_test_generate_12/Ballerina.toml to avoid duplicate timestamped pre-releases.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_80/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_80/Ballerina.toml` at line 13, The version field currently uses a timestamped pre-release value ("1.7.2-20260223-082800-6b94b8c") which should not be committed; update the version key in the Ballerina.toml (the version property) to a stable release string (e.g., "1.7.2") or a CI-friendly placeholder variable instead of the timestamped pre-release value to match the change made for tool_test_generate_12.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_16/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_16/Ballerina.toml` at line 13, The version line uses a timestamped pre-release value ("version = \"1.7.2-20260223-082800-6b94b8c\"") which duplicates the same issue flagged earlier; update the "version" value in this Ballerina.toml to a stable or canonical release string (e.g., "1.7.2" or the intended non-timestamped pre-release tag) to match the other manifest and avoid embedding build timestamps in the committed file.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_123_init_params_multiple_entities/Ballerina.toml (1)
10-10: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_123_init_params_multiple_entities/Ballerina.toml` at line 10, The version field currently contains a timestamped pre-release string ("version = \"1.7.2-20260223-082800-6b94b8c\""); replace this with a stable or canonical version identifier (e.g., "1.7.2" or the appropriate release tag) so tests do not depend on ephemeral timestamped pre-release values; update the literal in the "version =" line in the Ballerina.toml used by the test (the line containing the version string) to a non-timestamped release string or a test-controlled variable.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_26/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_26/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string ("version = \"1.7.2-20260223-082800-6b94b8c\"") which is non-deterministic; update the version field to a stable deterministic semantic version (e.g., change the value in the version key from the timestamped pre-release to "1.7.2" or another fixed release tag) or adjust the generator that writes the version so it emits a consistent release version rather than the timestamped prerelease.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_13/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_12/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_13/Ballerina.toml` at line 13, The version field in Ballerina.toml contains a timestamped pre-release string (version = "1.7.2-20260223-082800-6b94b8c") which duplicates the issue in the other test; change the "version" key in this file to a stable or canonical test version (e.g., "1.7.2" or the same non-timestamped value used in tool_test_generate_12) by removing the "-20260223-082800-6b94b8c" prerelease suffix so the file uses a consistent, non-timestamped version string.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_77/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_77/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string in the version field ("version = \"1.7.2-20260223-082800-6b94b8c\"") which makes tests non-deterministic; update the version value in this Ballerina.toml (the version key) to a stable, deterministic string (e.g. "1.7.2" or a fixed test fixture value) or replace it with a placeholder used by your test harness so the generated output is repeatable across runs.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_73/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_73/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string ("version = \"1.7.2-20260223-082800-6b94b8c\"") which makes builds non-deterministic; replace the timestamped pre-release with a stable or deterministic version (e.g., "1.7.2" or a CI-controlled build number) or use the same non-timestamped convention as in the other fixture (remove the "-20260223-082800-6b94b8c" suffix) so tests remain repeatable; update the value assigned to the version key in the Ballerina.toml used by the test fixture accordingly.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_79/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_79/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string for the "version" key (version = "1.7.2-20260223-082800-6b94b8c"); replace this timestamped pre-release with the canonical project version (e.g., "1.7.2") or a CI-controlled snapshot variable used across tests so tests aren't brittle—update the "version" entry in Ballerina.toml to the stable/version variable used elsewhere (match the value used in tool_test_generate_89 or the project version source) to remove the timestamped suffix.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_backward_compat/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_backward_compat/Ballerina.toml` at line 13, The version field in Ballerina.toml uses a timestamped pre-release string ("1.7.2-20260223-082800-6b94b8c") which breaks test reproducibility; update the "version" key in the Ballerina.toml used by tool_test_generate_backward_compat to a stable expected value (e.g., "1.7.2" or the same non-timestamped version used in the corresponding tool_test_generate_89 fixture) so tests remain deterministic.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_5/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_5/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string (version = "1.7.2-20260223-082800-6b94b8c") which should be replaced with a stable, deterministic version to avoid flaky test artifacts; update the version key in this Ballerina.toml (and the duplicate in tool_test_generate_89/Ballerina.toml) to a canonical value such as "1.7.2" (or the agreed test fixture version), ensuring both files use the same non-timestamped version string so tests remain repeatable.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_28/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_28/Ballerina.toml` at line 13, The version field in Ballerina.toml is using a timestamped pre-release value ("version = \"1.7.2-20260223-082800-6b94b8c\""); replace it with a stable/consistent version string (e.g., change the version key in Ballerina.toml to "version = \"1.7.2\"") so test fixtures don’t contain timestamped pre-release versions and match the other test cases.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_111_h2/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as noted intool_test_generate_89/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_111_h2/Ballerina.toml` at line 13, The test TOML contains a non-deterministic timestamped pre-release value in the version key ("version = \"1.7.2-20260223-082800-6b94b8c\"") which can cause flaky tests; update the version value referenced by the version key to a stable, deterministic pre-release or plain semantic version (e.g., "1.7.2" or a fixed pre-release string) to match the other test fixture (tool_test_generate_89) and ensure reproducible test outputs.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_91/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_116/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_91/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string in the version key (version = "1.7.2-20260223-082800-6b94b8c"); update the version value used by the tests to a stable or canonical version (e.g., "1.7.2") or a CI-controlled placeholder so it does not include the timestamped pre-release suffix, ensuring the 'version' key in this file matches the expected non-timestamped format used elsewhere.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_59_mssql/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_116/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_59_mssql/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string in the version key ("version = \"1.7.2-20260223-082800-6b94b8c\""); replace this unstable, timestamped pre-release value with a stable, deterministic version (e.g., a release or CI tag) by updating the version key in Ballerina.toml to a non-timestamped semantic version (for consistency with other test fixtures like tool_test_generate_116), ensuring the value follows your project's release versioning convention.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_40/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_116/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_40/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string in the version key (version = "1.7.2-20260223-082800-6b94b8c") which should not be checked into tests; update the version value in this Ballerina.toml to a stable or reproducible test-friendly version (e.g., "1.7.2" or a consistent pre-release like "1.7.2-dev") so it matches the fix applied in tool_test_generate_116/Ballerina.toml; edit the version entry in this file to remove the timestamp/hash and ensure tests use the same deterministic version format.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_119_init_params_postgresql/Ballerina.toml (1)
10-10: Same timestamped pre-release version concern as flagged intool_test_generate_116/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_119_init_params_postgresql/Ballerina.toml` at line 10, The Ballerina.toml contains a timestamped pre-release version on the version key (version = "1.7.2-20260223-082800-6b94b8c"); replace this timestamped pre-release string with the canonical/stable version (e.g., "1.7.2" or the intended release tag) so the version field uses a stable semantic version without build metadata, ensuring consistency with other test fixtures that expect a non-timestamped version.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_93/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_116/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_93/Ballerina.toml` at line 13, The version line currently uses a timestamped pre-release string (version = "1.7.2-20260223-082800-6b94b8c"); change it to a stable, non-timestamped version consistent with other test fixtures (e.g., version = "1.7.2") or the canonical fixture value used in tool_test_generate_116, updating the version assignment in this Ballerina.toml so tests don't rely on ephemeral timestamps.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_1/Ballerina.toml (1)
13-13: Same timestamped pre-release version concern as flagged intool_test_generate_116/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_1/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped pre-release version string in the version field (the line with version = "1.7.2-20260223-082800-6b94b8c"); replace this timestamped pre-release value with a stable or intentionally generic version (e.g., "1.7.2" or a placeholder used by your test suite) so tests don't depend on ephemeral build timestamps; update the version value in the Ballerina.toml file accordingly to match the non-timestamped convention used elsewhere (see the version key in this file and the similar entry in tool_test_generate_116/Ballerina.toml).persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_108_h2/Ballerina.toml (1)
13-13: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_108_h2/Ballerina.toml` at line 13, Update the version field in this Ballerina.toml to match the same version bump applied in the other test file: set the version = "1.7.2-20260223-082800-6b94b8c" in the version property inside Ballerina.toml, ensuring the value and formatting are identical to the change made in tool_test_generate_15/Ballerina.toml and run the same verification steps used there to confirm consistency.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_58_mssql/Ballerina.toml (1)
13-13: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_58_mssql/Ballerina.toml` at line 13, The version field in this Ballerina.toml is out of sync; update the version = "1.7.2-20260223-082800-6b94b8c" value to match the version bump used in the corresponding tool_test_generate_15/Ballerina.toml file so both files use the identical version string, then run the same verification steps you used for tool_test_generate_15 to ensure consistency; locate the version key in this file and replace its value accordingly.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_75/Ballerina.toml (1)
13-13: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_75/Ballerina.toml` at line 13, This is a duplicate version bump: update the version key in this Ballerina.toml to match the approved value used in tool_test_generate_15/Ballerina.toml (or revert this change if it was accidental); locate the "version" entry in this file and ensure its string exactly matches the canonical version used in the other test output, then run the same verification steps you used for tool_test_generate_15 to confirm consistency.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_122_init_params_relationships/Ballerina.toml (1)
10-10: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_122_init_params_relationships/Ballerina.toml` at line 10, Update the version key in this Ballerina.toml so it matches the version used in tool_test_generate_15/Ballerina.toml (ensure the "version" value is identical across both files), remove the duplicate_comment marker from this test resource, and re-run the verification steps used for tool_test_generate_15 to confirm both files are consistent; look for the "version" key in this file and the corresponding "version" in the other Ballerina.toml to make the change.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_78/Ballerina.toml (1)
13-13: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_78/Ballerina.toml` at line 13, This is a duplicate version bump—ensure the version string "version = \"1.7.2-20260223-082800-6b94b8c\"" in tool_test_generate_78's Ballerina.toml matches the verified change in tool_test_generate_15's Ballerina.toml; update the version value here if it differs, commit the change, and run the test/verification steps you used for tool_test_generate_15 to confirm consistency.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_125_init_params_negative_invalid_model/Ballerina.toml (1)
10-10: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_125_init_params_negative_invalid_model/Ballerina.toml` at line 10, The version key in this Ballerina.toml was bumped to "1.7.2-20260223-082800-6b94b8c"; please ensure this matches the intended release used in the related test (compare with tool_test_generate_15's Ballerina.toml) and, if incorrect, change the version value back to the expected string under the version = "..." line; after updating, re-run the same verification/CI steps you used for tool_test_generate_15 to confirm tests pass.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_82/Ballerina.toml (1)
13-13: Same version bump as discussed intool_test_generate_15/Ballerina.toml— see that comment for verification steps.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_82/Ballerina.toml` at line 13, The version key currently set as version = "1.7.2-20260223-082800-6b94b8c" is a duplicate bump and should match the canonical test output; update the version value to the same version string used in the corresponding test output (the other tool_test_generate_15 TOML) so the 'version' key in this Ballerina.toml is identical across test fixtures and run the verification steps used for the other file to confirm consistency.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_hybrid/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.Notably, the AI summary for this specific file explicitly documents that
gradle.propertiesmoves from1.7.2-20260223-082800-6b94b8c→1.7.2(stable), making the version mismatch against this fixture particularly evident.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_hybrid/Ballerina.toml` at line 13, The snapshot Ballerina.toml has a mismatched version string; update the "version" key in this file (the line containing version = "1.7.2-20260223-082800-6b94b8c") to match the stable version used elsewhere (e.g., "1.7.2") so the fixture aligns with gradle.properties and the other test snapshot (tool_test_generate_34/Ballerina.toml).persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_2/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_2/Ballerina.toml` at line 13, The test fixture contains a hardcoded snapshot version string in the Ballerina.toml key "version" (current value version = "1.7.2-20260223-082800-6b94b8c") which duplicates the issue in tool_test_generate_34; update this file's "version" entry to use the canonical/normalized test snapshot value used by other fixtures (e.g., a stable semantic version or the agreed placeholder) or regenerate the fixture so the version is not a timestamped snapshot, ensuring consistency with the fixture in tool_test_generate_34 and avoiding time-dependent diffs.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_37/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_37/Ballerina.toml` at line 13, The snapshot "version" value is hardcoded and duplicates the same timestamped snapshot used elsewhere; update the version entry (the "version" key value in this Ballerina.toml) to match the expected non-volatile value used by the other test snapshot (or make it parameterized/template-driven), ensuring it no longer contains the timestamped snapshot string "1.7.2-20260223-082800-6b94b8c" and is consistent with the canonical test snapshot used by tool_test_generate_34.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_39/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.This file uses the same pre-release build string
1.7.2-20260223-082800-6b94b8c. If the fix described above is applied, this fixture should also be updated to1.7.2.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_39/Ballerina.toml` at line 13, The snapshot pre-release version string "1.7.2-20260223-082800-6b94b8c" in the Ballerina.toml fixture should be replaced with the stable release "1.7.2"; update the version entry that currently reads that pre-release build string to "1.7.2" so the test fixture matches the corrected versioning used elsewhere.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_117/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_117/Ballerina.toml` at line 13, The test snapshot file Ballerina.toml contains a non-unique version value (the line with the version key) that duplicates the snapshot used in tool_test_generate_34; update the version string in this file so each test snapshot has a unique version (e.g., regenerate the timestamp/hash portion or use a deterministic placeholder) to avoid duplicate snapshot collisions when running tests; locate the version = "1.7.2-20260223-082800-6b94b8c" entry in tool_test_generate_117/Ballerina.toml and replace it with a unique or deterministic value consistent with your snapshot generation approach.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_68_postgresql/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_68_postgresql/Ballerina.toml` at line 13, The snapshot version in Ballerina.toml is inconsistent with the other test snapshot (same concern as tool_test_generate_34/Ballerina.toml); update the version value in this file (the version = "1.7.2-20260223-082800-6b94b8c" entry) to match the canonical snapshot used by the other test or replace it with a stable/non-timestamped version placeholder used across tests so snapshots remain deterministic; modify the version string in this Ballerina.toml to the agreed common value.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_24/Ballerina.toml (1)
13-13: Same snapshot version concern astool_test_generate_34/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_24/Ballerina.toml` at line 13, The Ballerina.toml contains a hard-coded timestamped snapshot version for the "version" key which duplicates the same issue in tool_test_generate_34; update the "version" value in this file (the version key in Ballerina.toml) to use the same canonical snapshot string used by tool_test_generate_34 (or switch to a non-timestamped snapshot placeholder used across tests) so both test fixtures share a consistent, non-flaky version.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_62_mssql/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_62_mssql/Ballerina.toml` at line 13, The Ballerina.toml file contains an unintended automated native-jar version bump in the version field; revert the change so the version value (the "version" key in Ballerina.toml) matches the project's intended release version used elsewhere (or restore the previous commit's version), ensuring consistency with other manifest files and removing the automated bump.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_112_h2/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_112_h2/Ballerina.toml` at line 13, The version line in Ballerina.toml was auto-updated to a timestamped native-jar version; change the version key value back to the canonical project version used elsewhere (replace version = "1.7.2-20260223-082800-6b94b8c" with the agreed project version string used in other manifests, e.g., version = "1.7.2") so the version field in Ballerina.toml matches the rest of the repo; ensure you edit the version entry (the "version" key) in Ballerina.toml to the consistent version.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_22/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_22/Ballerina.toml` at line 13, The version line in Ballerina.toml was auto-bumped; revert or correct the "version = ..." entry so it matches the canonical/native-jar version used elsewhere in the PR (i.e., the same value as the other Ballerina.toml entries), by editing the "version" key in this file to the expected value rather than the automated bump.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_67_postgresql/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_67_postgresql/Ballerina.toml` at line 13, This file contains a duplicate automated native-jar version bump in the Ballerina.toml entry "version = \"1.7.2-20260223-082800-6b94b8c\""; remove or revert this duplicate change so the repository has a single consistent version bump (i.e., restore the prior value or remove this edit), ensuring only the intended file keeps the updated version.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_65/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_65/Ballerina.toml` at line 13, The automated native-jar version bump in Ballerina.toml should be reverted/normalized to the expected version; locate the version = "..." entry in Ballerina.toml (the version key) and restore it to the canonical/stable value used elsewhere in the PR (or make the bump consistent across other files), ensuring the version string matches the project's expected release tag so the duplicate automated bump is removed.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_29/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_29/Ballerina.toml` at line 13, The version line in this test resource (the token "version = \"1.7.2-20260223-082800-6b94b8c\"") was auto-bumped; revert it to the canonical test fixture version used elsewhere (i.e., replace the bumped value with the expected version string that matches the other test resources/previous commit) so the test snapshot matches the rest of the repo.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_50/Ballerina.toml (1)
13-13: Same automated native-jar version bump as noted above.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_50/Ballerina.toml` at line 13, The Ballerina.toml contains an automated native-jar version bump ("version" = "1.7.2-20260223-082800-6b94b8c") that should be reverted to the intended stable value; open Ballerina.toml, locate the version key and replace the timestamped snapshot string with the correct/previous version used in the repo (or remove the automated bump), ensuring the "version" entry matches the expected release version rather than the autogenerated native-jar timestamp.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_109_h2/Ballerina.toml (1)
13-13: Same timestamped snapshot version concern as flagged intool_test_generate_multimodel_1/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_109_h2/Ballerina.toml` at line 13, The version field in this Ballerina.toml uses a timestamped snapshot string (the value assigned to the version key), which should be replaced with a stable or consistent snapshot notation; update the version key in this file (the version = "1.7.2-20260223-082800-6b94b8c" entry) to a non-timestamped form used by the project (e.g., a fixed release like "1.7.2" or a standardized snapshot such as "1.7.2-SNAPSHOT" or the project-wide placeholder), ensuring it matches the same change you made for tool_test_generate_multimodel_1/Ballerina.toml.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_multimodel_2/Ballerina.toml (1)
13-13: Same timestamped snapshot version concern as flagged intool_test_generate_multimodel_1/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_multimodel_2/Ballerina.toml` at line 13, The version field uses a timestamped snapshot "1.7.2-20260223-082800-6b94b8c" which should be replaced with a non-timestamped snapshot or stable version for reproducibility; update the value in Ballerina.toml (the line containing version = "...") to the canonical form used elsewhere (e.g., "1.7.2-SNAPSHOT" or the stable "1.7.2") so it matches the fix applied for tool_test_generate_multimodel_1/Ballerina.toml.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_101/Ballerina.toml (1)
13-13: Same timestamped snapshot version concern as flagged intool_test_generate_multimodel_1/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_101/Ballerina.toml` at line 13, The Ballerina.toml contains a timestamped snapshot version string ("version = \"1.7.2-20260223-082800-6b94b8c\"") which causes non-reproducible builds; update the version value used in the Ballerina.toml entry (the line with version =) to a stable identifier (for example a released version like "1.7.2" or a canonical snapshot token such as "1.7.2-SNAPSHOT") consistent with the other tests (see the similar change applied to tool_test_generate_multimodel_1), so that the test fixture does not contain a machine/time-specific timestamped artifact.persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_105/Ballerina.toml (1)
13-13: Same timestamped snapshot version concern as flagged intool_test_generate_multimodel_1/Ballerina.toml.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_105/Ballerina.toml` at line 13, The test fixture contains a non-deterministic timestamped version string assigned to the version key ("version" = "1.7.2-20260223-082800-6b94b8c"); replace this with a stable, deterministic value (e.g., a plain release like "1.7.2" or a canonical snapshot like "1.7.2-SNAPSHOT") so tests are repeatable, and update any related test expectations to match; ensure the same non-timestamped scheme is applied consistently across the corresponding Ballerina.toml used in other tests (e.g., tool_test_generate_multimodel_1) to avoid duplicate flaky fixtures.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (101)
changelog.mdgradle.propertiespersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_1/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_101/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_105/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_106/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_107_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_108_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_109_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_11/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_110_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_111_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_112_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_113_schema/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_114/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_115/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_116/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_117/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_118_init_params_mysql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_119_init_params_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_12/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_120_init_params_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_121_init_params_h2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_122_init_params_relationships/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_123_init_params_multiple_entities/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_124_init_params_with_config/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_125_init_params_negative_invalid_model/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_13/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_15/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_16/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_17/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_19/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_22/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_24/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_25/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_26/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_27/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_28/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_29/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_3/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_31/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_33/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_34/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_35/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_36/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_37/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_39/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_40/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_41/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_46/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_5/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_50/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_51/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_58_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_59_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_60_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_61_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_62_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_63_mssql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_64/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_65/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_66_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_67_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_68_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_69_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_7/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_70_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_71_postgresql/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_72/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_73/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_74/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_75/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_76/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_77/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_78/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_79/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_8/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_80/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_81/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_82/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_83/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_84/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_85/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_86/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_87/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_88/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_89/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_9/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_90/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_91/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_92/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_93/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_94/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_backward_compat/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_hybrid/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_multimodel_1/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_multimodel_2/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_multimodel_3/Ballerina.tomlpersist-cli-tests/src/test/resources/test-src/output/tool_test_generate_multimodel_4/Ballerina.tomlpersist-tool/BalTool.toml
Comment on lines
7
to
+10
| [[platform.java21.dependency]] | ||
| groupId = "io.ballerina.stdlib" | ||
| artifactId = "persist.sql-native" | ||
| version = "1.7.1" | ||
| version = "1.7.2-20260223-082800-6b94b8c" |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
Ballerina Central io.ballerina.stdlib persist.sql-native version 1.7.2-20260223-082800-6b94b8c
💡 Result:
io.ballerina.stdlib:persist.sql-native is a Maven artifact that provides the native (JVM) implementation used by Ballerina Persist’s SQL datastore support (i.e., what the bal persist tooling wires in as a platform dependency when you use an SQL datastore). [1], [2]
The version you mentioned:
1.7.2-20260223-082800-6b94b8c
is a timestamped/commit build identifier, commonly meaning:
- base version: 1.7.2
- build date/time: 2026-02-23 08:28:00
- source revision: git commit
6b94b8c
This pattern matches “commit release” versioning schemes of the formVERSION-DATE-TIME-COMMIT. [3]
If you’re trying to download that exact version, note that public indexes (e.g., Sonatype/Maven Central UI) don’t show it as a normal release version, so it may be from a nightly/commit build repository rather than Maven Central releases. [1]
Sources:
[1] Sonatype/Maven Central artifact page for io.ballerina.stdlib:persist.sql-native
[2] Ballerina tech blog showing persist.sql-native as the platform dependency used with bal persist + SQL datastore
[3] Example documentation of VERSION-DATE-TIME-COMMIT “commit release” version format
🌐 Web query:
Ballerina persist.sql-native 1.7.2 latest version central repository
💡 Result:
The latest version of io.ballerina.stdlib:persist.sql-native available in the Maven Central repository is 1.7.2. [1]
Maven coordinates
- GroupId:
io.ballerina.stdlib - ArtifactId:
persist.sql-native - Version:
1.7.2[1]
Maven dependency
<dependency>
<groupId>io.ballerina.stdlib</groupId>
<artifactId>persist.sql-native</artifactId>
<version>1.7.2</version>
</dependency>
``` [1]
**Sources**
- [1] Maven Central (Sonatype): `io.ballerina.stdlib:persist.sql-native`
---
</details>
**Fix golden output to use stable `1.7.2` instead of nightly build version.**
The timestamped version `1.7.2-20260223-082800-6b94b8c` is not available in the public Maven Central repository; it comes from a nightly/commit build repository and may not be accessible or reproducible. Update the golden output to use the stable `1.7.2` release, or document that nightly artifact repositories must be configured for test runs to succeed.
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In
@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_33/Ballerina.toml
around lines 7 - 10, The golden output contains a timestamped nightly version
for the dependency ([[platform.java21.dependency]] with groupId
"io.ballerina.stdlib" and artifactId "persist.sql-native") which is not
reproducible; update the version field from "1.7.2-20260223-082800-6b94b8c" to
the stable release "1.7.2" in the Ballerina.toml golden output (or alternatively
add a note that nightly artifact repositories must be configured for tests).
</details>
<!-- fingerprinting:phantom:poseidon:eagle -->
<!-- This is an auto-generated comment by CodeRabbit -->
| groupId = "io.ballerina.stdlib" | ||
| artifactId = "persist.sql-native" | ||
| version = "1.7.1" | ||
| version = "1.7.2-20260223-082800-6b94b8c" |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Step 1: Check gradle.properties for stdlibPersistSqlVersion
echo "=== gradle.properties content for stdlibPersistSqlVersion ==="
fd 'gradle.properties' --max-depth 1 -x cat {} | grep -i 'persistsql'
echo ""
echo "=== Checking if gradle.properties exists and full content ==="
if [ -f gradle.properties ]; then
cat gradle.properties | grep -i persist
else
echo "gradle.properties not found in root"
fi
echo ""
echo "=== All gradle.properties files in repo ==="
fd 'gradle.properties' | head -20Repository: ballerina-platform/persist-tools
Length of output: 709
🏁 Script executed:
#!/bin/bash
# Step 2: Count Ballerina.toml files with timestamped version
echo "=== Files with timestamped version 1.7.2-20260223-082800-6b94b8c ==="
rg '1\.7\.2-20260223-082800-6b94b8c' --type toml -l | wc -l
echo ""
echo "=== Sample files with timestamped version ==="
rg '1\.7\.2-20260223-082800-6b94b8c' --type toml -l | head -10
echo ""
echo "=== Files with stable 1.7.2 ==="
rg 'version\s*=\s*"1\.7\.2"' --type toml -l | wc -l
echo ""
echo "=== Sample files with stable 1.7.2 ==="
rg 'version\s*=\s*"1\.7\.2"' --type toml -l | head -10Repository: ballerina-platform/persist-tools
Length of output: 1226
Test fixtures use timestamped native version inconsistent with stable version PR objective.
The test Ballerina.toml files reference persistSqlNativeVersion=1.7.2-20260223-082800-6b94b8c (a nightly/timestamped build), while gradle.properties declares the stable stdlibPersistSqlVersion=1.7.2. Since the PR objective is "Moved dependencies to stable versions," these 98 test resource manifests should likely use the stable 1.7.2 or match the gradle.properties declarations for consistency.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In
`@persist-cli-tests/src/test/resources/test-src/output/tool_test_generate_89/Ballerina.toml`
at line 13, Update the timestamped native version in the test Ballerina.toml
fixtures to match the stable version declared in gradle.properties: replace
occurrences of persistSqlNativeVersion=1.7.2-20260223-082800-6b94b8c with
persistSqlNativeVersion=1.7.2 (or with the value of stdlibPersistSqlVersion) in
the 98 test resource manifests so the test fixtures and gradle.properties stay
consistent.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sync main after 1.9.0 release
Overview
This pull request completes the post-release synchronization following the 1.9.0 release, updating the project to prepare for the next development cycle.
Key Changes
Version Management:
Release Documentation:
Dependency Updates:
Impact
These changes transition the codebase from a release candidate state to the next development iteration while stabilizing all external dependencies and documenting the released features.