Improve the extract plugin so it throws a warning/error when it tries to extract message with undefine id.

[semoal]

Also we should improve the extract plugin so it throws a warning/error when it tries to extract message with undefine id.

Originally posted by @tricoder42 in #862 (comment)

[semoal]

I close this, because already does it. https://github.com/lingui/js-lingui/blob/main/packages/babel-plugin-extract-messages/src/index.ts#L134

[tricoder42]

Yeah, but that's only Trans macro. I guess we simply need to copy the same working to lines 162

js-lingui/packages/babel-plugin-extract-messages/src/index.ts

Lines 159 to 163 in 2bf9ebf

	const props = {
	id: path.node.arguments[0].value,
	}
	const copyOptions = ["message", "comment"]

and 195:

js-lingui/packages/babel-plugin-extract-messages/src/index.ts

Lines 192 to 196 in 2bf9ebf

	const props = {
	id: path.node.value,
	}
	collectMessage(path, file, props)

the bug in js macro caused that undefined got extracted. That shouldn't happen, i.e. if id is undefined, we shouldn't extract any message but rather show a warning.