Shorthand link references not recognized when there's whitespace between []

[nzakas]

Initial checklist

• I read the support docs
• I read the contributing guide
• I agree to follow the code of conduct
• I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions

mdast-util-from-markdown@2.0.1

Link to runnable example

https://stackblitz.com/edit/stackblitz-starters-ifpvft?file=index.js

Steps to reproduce

1. Create a link reference with whitespace in between the []:

[foo][ ]

[foo]: https://example.com

2. Pass into fromMarkdown()

Expected behavior

The returned AST should identify [foo][ ] as a linkReference. (It appears that GitHub does this. I'm unsure if that's standard behavior for Markdown or not. The Micromark GFM extension exhibits the same behavior.)

Actual behavior

The returned AST identifies [foo][ ] as text rather than a link reference.

Affected runtime and version

node@20.13.0

Affected package manager and version

No response

Affected OS and version

No response

Build and bundle tools

No response

[ChristianMurphy]

Hey @nzakas! 👋
Commonmark specifies that spaces are significant and it would not match
https://spec.commonmark.org/dingus/?text=%5Bfoo%5D%5B%20%5D%0A%0A%5Bfoo%5D%3A%20https%3A%2F%2Fexample.com

I don't see any spec changes in https://github.github.com/gfm/#link-reference-definitions to reference links
This may be a bug in https://github.com/github/cmark-gfm

[wooorm]

This seems to be commonmark/commonmark-spec#653, which shows that most tools work differently than the spec.

I think the spec is currently saying that this is not allowed:

• what you post could be a shortcut reference, but then it would leave the [ ] in the resulting HTML, which is also not what you expected
• what you post cannot be a collapsed reference, “followed by the string []”, which this is not

I do consider this to be something that can improved in CM.

But I also consider this a current bug in GFM, which says it sticks to CM, so I don’t feel good about “solving” it in micromark-extension-gfm.

[nzakas]

To make sure I'm following, it sounds like you're saying that the spec disallows this behavior, so this is a bug in GitHub's Markdown parser?

[wooorm]

Correct

[fasttime]

I think that in this example, spaces and line breaks are not significant:

[foo][
  bar ]

[bar
]: https://example.com

https://spec.commonmark.org/dingus/?text=%5Bfoo%5D%5B%20%5D%0A%0A%5Bfoo%5D%3A%20https%3A%2F%2Fexample.com

But they are significant if there are only spacing characters?

[wooorm]

Correct. There’s indeed a “gap” between the different shortcut links and how they are defined in CM. Shortcut ([x]), collapsed ([x][]), vs. full ([x][y]). Making nzakas’s case ambiguous/vague/weird.

[wooorm]

Closing as behavior is intentional, I believe according to spec!

[github-actions]

Hi! This was closed. Team: If this was fixed, please add phase/solved. Otherwise, please add one of the no/* labels.