Skip to content

Conversation

@MarcinVaadin
Copy link
Member

Summary

This PR removes all Hilla framework support from the Eclipse plugin's new project wizard, simplifying the codebase to focus exclusively on Vaadin Flow.

Changes Made

  • UI Changes:

    • Removed Hilla checkbox from Starter Project wizard
    • Removed "Hilla / React" option from Hello World framework dropdown
    • Updated help text to only describe Flow framework
  • Model Changes:

    • Removed includeHilla field and related getter/setter methods from StarterProjectModel
    • Simplified URL generation logic to only support Flow framework
    • Updated HelloWorldProjectModel to remove Hilla comment reference
  • Validation Logic:

    • Removed Hilla-specific validation constraints
    • Simplified architecture combo enablement logic
  • Documentation:

    • Removed Hilla limitations from AGENTS.md
    • Removed Hilla endpoint discovery from future enhancements
  • Tests:

    • Updated StarterProjectModelTest to remove Hilla-related test cases
    • Updated HelloWorldProjectModelTest to use Flow instead of Hilla

Test Results

✅ All tests pass (19 tests, 0 failures)

  • StarterProjectModelTest: 9 tests passed
  • HelloWorldProjectModelTest: 10 tests passed

Impact

  • Users will no longer see Hilla as an option in the new project wizard
  • All existing projects remain unaffected
  • The plugin now only supports Flow framework for new project creation

🤖 Generated with Claude Code

MarcinVaadin and others added 2 commits November 13, 2025 13:53
Removed all Hilla-related functionality from the Eclipse plugin's new
project wizard to simplify the codebase and focus on Flow framework only.

Changes:
- Removed Hilla checkbox from Starter Project wizard UI
- Removed Hilla option from Hello World project framework selection
- Updated StarterProjectModel to remove includeHilla field and methods
- Simplified URL generation to only support Flow framework
- Updated validation logic to remove Hilla-specific constraints
- Updated documentation to remove Hilla references
- Updated all test cases to remove Hilla-related assertions

All tests pass successfully (19 tests, 0 failures).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Since Hilla has been removed, the framework selection combo in the
Hello World project wizard now only had a single option (Flow). This
commit removes the unnecessary framework selection UI element as Flow
is now the only and default framework.

Changes:
- Removed frameworkCombo field from VaadinProjectWizardPage
- Removed framework selection UI from Hello World wizard options
- Removed framework combo listener registration

Framework is hardcoded to "flow" in getProjectModel() method.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@github-actions
Copy link

Artifact build on last commit: repository.zip.

Copy link

@mgarciavaadin mgarciavaadin left a comment

Choose a reason for hiding this comment

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

Would make sense to remove also the "Pure Java" option?

I think it can be confusing for users. Otherwise we should add an explanation about option for typescript views etc... something that explains why there is a "Pure Java" option

Copy link

@mgarciavaadin mgarciavaadin left a comment

Choose a reason for hiding this comment

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

Needs to update NewProjectWizard

                properties.put("include_hilla", starterModel.isIncludeHilla());

Somehow I was able to compile it and test it without issues... weird

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.

3 participants