Skip to content

fix: app startup failure for name mismatches in separate venvs#778

Open
yaseminozkut wants to merge 3 commits into
pollen-robotics:mainfrom
yaseminozkut:fix/app-name-mismatch
Open

fix: app startup failure for name mismatches in separate venvs#778
yaseminozkut wants to merge 3 commits into
pollen-robotics:mainfrom
yaseminozkut:fix/app-name-mismatch

Conversation

@yaseminozkut
Copy link
Copy Markdown
Contributor

Fixes issue #777

  • added _extract_package_info_from_pyproject function to extract the real entry point from pyproject.toml during installation
  • use the stored entry_point_name instead of the space name when loading apps from separate venvs
  • add fallback to app name for the apps installed before this fix so that users won't have problem (at least for the apps that don't have mismatch. otherwise, they will fail after this fix and they would need to uninstall and install the app because we are checking different variable from the metadata)

@CarolinePascal
Copy link
Copy Markdown
Collaborator

Hi @yaseminozkut,

Thank you for the contribution ! However, after discussing with the team, we would rather enforce that the hugging face space name always matches the entry point name. It's already the case when generating an app with reachy-mini-app-assistant create and we will make the app checks more strict to ensure the names are the same.

Let us know if you still think there is an issue to solve there.

Best,

Caroline.

@CarolinePascal CarolinePascal added the enhancement New feature or request label Feb 9, 2026
@Jacid23
Copy link
Copy Markdown

Jacid23 commented Feb 14, 2026

problem is there needs to be a name entry point registry because now the ui has no clue what one install the main or the fork. What a nightmare trying to figure out why diff space names showed up as different space install in the ui. People fork these things and honestly shouldn't be able to publish with the same name or something gotta change

@yaseminozkut
Copy link
Copy Markdown
Contributor Author

Hi @CarolinePascal,
Thanks for your comment! Yes checking the Hugging Face space name make sense. But if the entry point name for the package and the space name does not match, then even though the desktop app installs, it cannot find it and fails to start. I saw few apps had this error when I tried to start them. Maybe they are old (made before the enforcing) and the apps itself need to be fixed anyway.

@Jacid23 also raised a good point about a different problem for forks sharing the same entry point names. I haven't tested this scenario myself, maybe @Jacid23 can give more explanation. Does it fail to start or start the wrong app, or is it just a UI issue (not clear who's app is it)? If its just a UI issue, maybe usernames from Hugging Face can be added in the UI.

I also saw this discussion https://huggingface.co/datasets/pollen-robotics/reachy-mini-official-app-store/discussions/20 that @pierre-rouanet mentioned (maybe its before the enforcing with the reachy-mini-app-assistant check ?)

Best,
Yasemin

@CarolinePascal
Copy link
Copy Markdown
Collaborator

Also pinging @apirrone and @RemiFabre for visibility !

@FabienDanieau FabienDanieau changed the base branch from develop to main March 3, 2026 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants