Skip to content

fix(cli): name the ECMA-427 identity grammar in the usage text - #3

Merged
HuiJun merged 1 commit into
developfrom
fix/help-identity-grammar
Oct 30, 2026
Merged

HuiJun merged 1 commit into
developfrom
fix/help-identity-grammar

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

Summary

syspi -h still advertised pkg:<vcs>/<host>/<path>[@<version>][#<subpath>], the pre-ECMA-427 grammar: since #2, pkg:svn/… and pkg:hg/… written that way are refused, so the help contradicted the tool. The usage text now shows the canonical form and the two input spellings origin.Spellings already names in error messages:

pkg:generic/<name>?vcs_url=<vcs>%2B<transport>:%2F%2F<host>%2F<path>[%40<version>][%23<subpath>]
Accepted as input and normalised to it:
  pkg:git/<host>/<path>[@<version>][#<subpath>]
  <vcs>+<transport>://<host>/<path>[@<version>][#<subpath>]

The lock.Entry.Resource comment carried the same stale grammar and is corrected. No behaviour change; changes/unreleased/help-identity-grammar.fixed.md records it.

The usage text still advertised pkg:<vcs>/<host>/<path>, which svn and hg identities now refuse; it and the lock's Resource comment name the pkg:generic form and the two input spellings.

Co-Authored-By: jason.han <hanhuijun@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration
devin-ai-integration Bot marked this pull request as ready for review October 30, 2026 00:46

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@HuiJun
HuiJun merged commit 3ff8498 into develop Oct 30, 2026
3 checks passed
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.

1 participant