Skip to content

Conversation

@vidishagawas121
Copy link
Contributor

gh pr create
--title "Backend: Index CompilerPlugin artifacts"
--body "This PR adds backend support for indexing CompilerPlugin artifacts in Scaladex.

  • Updates indexing logic to include CompilerPlugin
  • Adds database migration if needed
  • Ensures queries return correct versions and metadata
  • Adds unit/integration tests for indexing CompilerPlugin artifacts

This PR depends on PR 1, which introduced CompilerPlugin in the models."
--base main
--head feature/compiler-plugin-backend-indexing

- Introduce CompilerPlugin in Platform.scala
- Update BinaryVersion.toString to handle CompilerPlugin
- Update all platform matches in Artifact.scala to explicitly handle CompilerPlugin
- Throw UnsupportedOperationException where features aren’t implemented
- Add unit test for creating Artifact with CompilerPlugin
- Lints clean
- Update indexing logic to include CompilerPlugin artifacts
- Add database migration for CompilerPlugin if required
- Ensure queries return correct versions and metadata
- Add unit/integration tests for CompilerPlugin indexing
@kannupriyakalra kannupriyakalra requested a review from adpi2 October 14, 2025 18:04
Copy link
Member

@adpi2 adpi2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @vidishagawas121 for your work. This is a good start but it does not yet allow parsing an artifact ID with a full Scala version as the binary version, such as kind-projector_2.13.16.

Copy link
Member

@adpi2 adpi2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is one broken test (see my comment). Also you need to run scalafmt and sbt scalafixAll.

Apart from that, it looks good to me.

for
_ <- insertAll(projects)
platforms <- searchEngine.countByPlatforms(params)
yield platforms should contain(CompilerPlugin -> 1L)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be CompilerPlugin(Version(2, 13, 16))

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants