Skip to content

Conversation

@rtablada
Copy link
Contributor

@rtablada rtablada commented Oct 9, 2025

Description

This adds a compiler for Ember/Glimmer components as well as examples for use in both a Vite and Webpack build tooling for Ember

Linked Issues

Additional context

@NullVoxPopuli may be able to help get things over the 🏁, this was a WIP from a hackathon/side project that I have not been able to get back to.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@userquin
Copy link
Member

userquin commented Oct 9, 2025

Run the build command, ember.d.ts types should be added to the subpackage exports at package json file.

Try to reduce both ember examples to minimal required code to test the icons 🙏 (update root .gitignore to exclude ember stuff at examples).

@NullVoxPopuli
Copy link
Contributor

I can do that -- @rtablada can you add me as a collaborator on your fork? <3

@NullVoxPopuli
Copy link
Contributor

NullVoxPopuli commented Oct 9, 2025

hm, any way I can see details on why socket.io is saying my deps are bad? 😢

image

@NullVoxPopuli
Copy link
Contributor

I thiiiink this is ready for review?
Happy to do whatever else is needed though

@userquin
Copy link
Member

userquin commented Oct 9, 2025

Did you run the build command from root? ./types/ember subpackage export should be added to the package.json by tsdown.

@userquin
Copy link
Member

userquin commented Oct 9, 2025

hm, any way I can see details on why socket.io is saying my deps are bad? 😢

I'm getting also forbidden message 🤔

Check https://socket.dev/?utm_medium=gh, you have some hints at the end of the page

@userquin userquin marked this pull request as ready for review October 9, 2025 20:59
@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 9, 2025

Open in StackBlitz

npm i https://pkg.pr.new/unplugin-icons@419

commit: 2a2da4d

@userquin
Copy link
Member

userquin commented Oct 9, 2025

We still need the ember types at subpackage exports, just run pnpm build from root folder and push the changes added by tsdown build.

@userquin userquin changed the title Support For Ember with Webpack and Vite Example feat(compiler): add support for Ember Oct 9, 2025
@userquin
Copy link
Member

userquin commented Oct 9, 2025

Will do a review tmr from laptop

@userquin
Copy link
Member

userquin commented Oct 9, 2025

can you run pnpm lint from root? Maybe we need to change also eslint config file (rules)

@NullVoxPopuli
Copy link
Contributor

there is a failing lint at the moment, because one of the lint rules has assumed all .map functions are Array.prototype.map 🙈

@NullVoxPopuli
Copy link
Contributor

I pushed a little ignore with details about why in my most recent commit

@userquin
Copy link
Member

Looks like ember doing weird things without test/index.html entry point (ember plugin checking some entries in the rollup configuration entry), check the error below (dev server working, only tested with Vite):

dev server error dev server error

Copy link
Member

@userquin userquin left a comment

Choose a reason for hiding this comment

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

once released we should add a link here https://github.com/unplugin/unplugin-icons/blob/main/examples/README.md adding the corresponding StackBlitz script file and command (at least for Vite)

@NullVoxPopuli
Copy link
Contributor

check the error below (dev server working, only tested with Vite):

yea, I need to fix that 🙈 it's very easy to check if that file exists before adding the entrypoint, I just keep forgetting 🫠

we should add a link here

I can do that! <3

@NullVoxPopuli
Copy link
Contributor

thanks for the reminder! embroider-build/embroider#2584 <3

@userquin
Copy link
Member

userquin commented Oct 10, 2025

@NullVoxPopuli can you find me at discord at antfu server unplugin-icons channel? I have a few questions about Ember and IDE support...

https://discord.gg/DMn6tWHV

@NullVoxPopuli
Copy link
Contributor

The warning on vite boot has been fixed upstream

@userquin
Copy link
Member

userquin commented Oct 18, 2025

The warning on vite boot has been fixed upstream

can you update the playgrounds?

@NullVoxPopuli
Copy link
Contributor

no need, the ^ range on the dependency will cover it

@antfu antfu merged commit a27ed84 into unplugin:main Oct 19, 2025
4 checks passed
@NullVoxPopuli NullVoxPopuli deleted the ember-compiler branch October 19, 2025 22:51
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.

4 participants