Deprecate mermaid.cli or @mermaid-js/mermaid-cli?
See original GitHub issueMaybe the website is using https://www.npmjs.com/package/mermaid.cli instead of https://www.npmjs.com/package/@mermaid-js/mermaid-cli?
If anyone has access to that NPM package, maybe they can do an npm deprecate on it, something like:
npm deprecate mermaid.cli "Development has moved to the '@mermaid-js/mermaid-cli' package"
Ideally we’d also close all the open issues/PRs in https://github.com/mermaidjs/mermaid.cli and archive that repo since it looks like there are still people reporting bugs there.
Maybe with a message that says:
Development on this project has moved to https://github.com/mermaid-js/mermaid-cli
If this issue/PR is still valid, please re-create this issue in the new repository and add a link to this issue/PR.
A script like https://gist.github.com/mauritsvanrees/f0dc7d829a1957a1771adbf0e86057c5 might help, but then again, there are only ~60 or so open issues/PRs, so it might be faster to manually close every issue.
_Originally posted by @aloisklink in https://github.com/mermaid-js/mermaid-cli/issues/432#issuecomment-1325788390_
Yes, We should archive the GitHub repo, but the mermaid.cli
package name seems too good to throw away.
npm i mermaid.cli
npm i @mermaid-js/mermaid-cli
IMHO, The first one looks better, and is in line with mermaid.live
Wonder what’s keeping us from going back to that package with a new major release?
Issue Analytics
- State:
- Created 10 months ago
- Comments:11 (5 by maintainers)
I think this should allow access to publish.
Then, if we add an
mmdc
alias to the package, people would be able to run bothnpx mermaid.cli ...
andmmdc ...
if they install globally.I think it’s mainly for permissions. Most projects are instead moving to
@org/package
because that way it’s much easier to manage project permissions, see https://docs.npmjs.com/organizationsIt looks like the owner of
mermaid.cli
is @tylerlongIt’s probably easier to just ask them to run
npm deprecate mermaid.cli "Development has moved to the '@mermaid-js/mermaid-cli' package"
then to ask them to mess with NPM permissions 😃I’m not even sure if it’s possible to move ownership of a normal NPM package to an NPM
@mermaid-js
organization account.It’s part of updating the Mermaid back-end, but it’s a bit hidden, see below:
Maybe we should make that more prominent? Recommendations are welcome 😃 We could make the first line of the release notes something like
This version of mermaid-cli is bundled with v9.x.x of mermaid, see [Mermaid v9.x.x release notes][link].
We could also add it to the
mmdc --version
, so maybe it logs something like:9.2.2+mermaid-version.9.2.2
. Normally, we try to keep themermaid-cli
version identical to the bundledmermaid
version, but very occasionally, they do drift slightly.I prefer how puppeteer does it. It installs
chromium
automatically when runningnpm install
, so that way you can usemermaid-cli
offline after the first install.There’s also some docs on how to use a pre-downloaded chromium version for puppeteer (e.g. https://github.com/mermaid-js/mermaid-cli/issues/331), which is used by the Mermaid docker image and people that have strict corporate security rules. If we switch to using
playwright
, all of those instructions will also need to be updated (maybe even make it impossible since I believeplaywright
uses forked versions of browsers).I think the developer experience with
playwright
is much nicer (especially when writing tests), but themermaid-cli
use case is really basic, so I don’t see much reason to switch 😃Ideally, the main mermaid project will add server-side rendering, so that we can completely remove
chromium
(see https://github.com/mermaid-js/mermaid/issues/3650), but who knows if that is even possible!