🐛 Set symbol kind as Module so that is not filtered out later in the analyzer

[jmle]

See this line in the analyzer-lsp project.

https://issues.redhat.com/browse/MTA-5387

API tests PR: 343

Summary by CodeRabbit

• Bug Fixes
  • Fixed symbol classification for on-demand import matches in code analysis results.

[coderabbitai]

Walkthrough

A single symbol kind value in the SampleDelegateCommandHandler was changed from Namespace to Module for on-demand import matches. All surrounding logic, naming, location calculation, and result collection remain unchanged.

Changes

Cohort / File(s)	Summary
Symbol Kind Update java-analyzer-bundle.core/src/main/java/io/konveyor/tackle/core/internal/SampleDelegateCommandHandler.java	Changed SymbolInformation.setKind value from Namespace to Module for on-demand import match representation

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

A tiny tweak, so clean and small,
From Namespace to Module's call,
The symbol shifts, a rabbit's care,
On-demand imports, light as air! 🐰✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "Set symbol kind as Module so that is not filtered out later in the analyzer" directly aligns with the main change described in the raw summary, which states that the kind assigned to a symbol was changed from Namespace to Module. The title is specific about what was changed (symbol kind to Module) and clearly explains the purpose (to prevent filtering later in the analyzer), matching both the code modification and the PR objectives. The title is concise, single-sentence, and provides enough clarity for a teammate to understand the primary change without being vague or misleading.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[eemcmullan]

The unexpected incidents from the e2e coolstore app test look correct, so CI tests probably need to be updated