extensions icon indicating copy to clipboard operation
extensions copied to clipboard

Tracking Issue: Extensions using `extension.json`

Open maxdeviant opened this issue 10 months ago • 12 comments

This is a tracking issue for extensions still using the extension.json manifest format instead of extension.toml.

Extensions using extension.json

maxdeviant avatar Feb 23 '25 22:02 maxdeviant

I run a bot script, to open a PR with the needed changes for all the above repos, hopefully this make it easier and motivate the owners to migrate. https://github.com/sigmaSd/botfixzed/blob/master/bot.ts

sigmaSd avatar Mar 06 '25 06:03 sigmaSd

I run a bot script, to open a PR with the needed changes for all the above repos, hopefully this make it easier and motivate the owners to migrate. https://github.com/sigmaSd/botfixzed/blob/master/bot.ts

The script should also remove the extension.json in the PRs that it opens.

We don't want to end up with a bunch of extensions that have both an extension.toml and and extension.json, which seems to be what is happening.

maxdeviant avatar Mar 07 '25 21:03 maxdeviant

Yes because I wanted to do the least modifications possible, and noticed in testing that zed prioritize extension.toml that's why I thought its not that important.

sigmaSd avatar Mar 07 '25 21:03 sigmaSd

Yes because I wanted to do the least modifications possible, and noticed in testing that zed prioritize extension.toml that's why I thought its not that important.

Yes, but we want the extension.json gone. That is the whole point of this migration.

And the extension CI will fail when these authors go to publish a new version if they have both.

maxdeviant avatar Mar 07 '25 21:03 maxdeviant

I see, I'll update the PRs then.

sigmaSd avatar Mar 07 '25 21:03 sigmaSd

@sigmaSd Some of the PRs have other issues as well.

The one for zed-fish contained an extension.toml that was invalid and did not make the necessary removals to other files in the repo: https://github.com/hasit/zed-fish/pull/11

maxdeviant avatar Mar 07 '25 21:03 maxdeviant

For that extension in particular , I did test manually and I noticed the grammar, but I left it because the extension still works, the PRs were meant as a minimum working changes while the author do the clean up and gets motivated to update.

I have the script running now pushing a commit to remove extension.json to all the open prs it made. but I'm not going to make it do more, the extension author have to test it manually and do the needed cleanup.

sigmaSd avatar Mar 07 '25 21:03 sigmaSd

Actually the grammars changes seems straight forward, maybe I'll add that

sigmaSd avatar Mar 07 '25 21:03 sigmaSd

@maxdeviant I'll update the script with you recommendations, currently this is what I'm going to do

  • remove grammar = type
  • remove grammer folder
  • remove languages key
  • if there is already a [grammar] dict remove it
  • add grammar to gitignore

sigmaSd avatar Mar 07 '25 22:03 sigmaSd

I update all the pr as discussed, is there more feedback ?

sigmaSd avatar Mar 07 '25 23:03 sigmaSd

I update all the pr as discussed, is there more feedback ?

Thank you! I think that should be all that is needed.

Sorry if I came off a bit blunt, I just want to make sure that we're helping out maintainers and not creating more work for them.

I do appreciate you helping out with this migration 🙂

maxdeviant avatar Mar 07 '25 23:03 maxdeviant

No problem, that makes sense thanks as well

sigmaSd avatar Mar 07 '25 23:03 sigmaSd