Display most recently used tokens when completing about idea-conventional-commit

[Gerrit-K]

I'm not a native speaker, is "Recently used" appropriate or should I go for something else?

Me neither ;) but it sounds fine to me.

Also, beware I can't guarantee the correctness of ordering

That's a pity, but not a dealbreaker. Having suggestions in incorrect order is still better than having none at all I guess :)

Regarding your code, I'd love to help, but that would require me to learn kotlin from scratch as well as the IntelliJ plugin framework :/ Perhaps there's another open-source plugin out there that already parses the vcs log, from which you could take some bits?

[lppedd]

@Gerrit-K fantastic! Let me know of any issue in case.

[Gerrit-K]

@lppedd look awesome, thanks for picking this up!

[lppedd]

@Gerrit-K No problem 😄
I'm not a native speaker, is "Recently used" appropriate or should I go for something else?

Also, beware I can't guarantee the correctness of ordering, and with that I mean even if example-one has been used before my-scope-example, the order might be inverted (platform behavior).

[lppedd]

Testing the integration with VCS was somewhat successfull. The code would feel like an hack to a possible contributor so it needs to be rewritting using the right objects (the flow of the VCS log is horrible to debug).

private fun getVcsCommits(): List<VcsCommitMetadata> {
  val vcsLogManager = VcsProjectLog.getInstance(project).logManager!!
  val vcsLogData = vcsLogManager.dataManager.also {
    it.addDataPackChangeListener(MyDataPackChangeListener(it))
    it.refresh(it.roots)
  }

  queue.take()

  val vcsScopesWithTimes = mutableListOf<VcsCommitMetadata>()
  val allCommits = vcsLogData.dataPack.permanentGraph.allCommits

  vcsLogData.miniDetailsGetter.loadCommitsData(
    allCommits.map { it.id },
    vcsScopesWithTimes::add,
    ProgressManager.getInstance().progressIndicator,
  )

  return vcsScopesWithTimes
}

inner class MyDataPackChangeListener(private val vcsLogData: VcsLogData) : DataPackChangeListener {
  override fun onDataPackChange(newDataPack: DataPack) {
    vcsLogData.removeDataPackChangeListener(this)
    queue.put(Unit)
  }
}

[lppedd]

@Gerrit-K oh don't worry, I post code in issues only as a sort of "backup" or remainder.

[lppedd]

Implemented with commit a61f2b9.
Now there are two providers, one for recently used tokens (max 3 elements), and one for VCS-tokens (max 8 elements).

[Gerrit-K]

@lppedd I received the plugin update today and tested it briefly. Works great and I think this will be really helpful for our workflow in the future. Thanks again for addressing this so quickly! :)