azu
Another Discussion Points - Is the naming correct? - `metadata` property is another option, but `id` is simpler. - `metdata: {}` or `id: string` - Should `id` is required property...
I agree with you. ## TODO - [ ] add `id` to messaging as required property - [ ] fix usage https://github.com/search?q=repo%3Atextlint%2Feditor%20postMessage&type=code Welcome to PR
Yeah, `error` response is reasonable.
I think that worker should just post `error` message when `id` is missing. It is global error like [Window: error event](https://developer.mozilla.org/en-US/docs/Web/API/Window/error_event). If the command has `id` and some error on...
It may be better to proceed separately with the PR to add the `id` and the PR to make the `id` mandatory. The latter is a breaking change, so there...
Ah, I did not know that catch the exception on the worker via `worker.addEventListener("error", ...`. It is better than `command === "error"`.
https://github.com/textlint-ja/textlint-rule-max-ten とはまた違う感じですか? 文節ごとの長さなんですかね?
Failed test case is here https://github.com/textlint/textlint/blob/82edb8d675f56c468da045e6236bf3ee98486417/packages/textlint/test/cli/cli-test.ts#L439-L452
https://github.com/secretlint/secretlint/blob/410818b45c8dfad5bf53e0238c027f66126ea65e/packages/%40secretlint/config-loader/src/SecretLintModuleResolver.ts#L10-L17 Probably, we can replace it with plain function.
https://github.com/apache/arrow/releases/tag/apache-arrow-19.0.0 Apache-arrow 19 is released