mmuddasani

Results 11 comments of mmuddasani

Has there been any status update or progress on this bug lately? I find it to be quite an annoying developer experience with ESM not yet doing the right thing...

Microsoft's jsonc module that @farsightsoftware found appears incredibly comprehensive. Someone has even ported it as a third-party Deno module: https://deno.land/x/jsonc It has an MIT license, so I'm wondering if Deno's...

I imagine the only downside with incorporating the API of [npm:comment-json](https://www.npmjs.com/package/comment-json) as-is into deno std is the pollution of the global Symbol registry. Symbols are a very neat idea, and...

@ayame113 I am in agreement with both of your suggestions. I would also propose (crudely atm) something as follows: ```ts export const comments_api: unique symbol = Symbol("comments api") export interface...

I discovered that `deno fmt` also loses some JSONC comments! I made minimal reproducible examples [here](https://github.com/mmuddasani/deno-fmt-comment-loss). For instance, compare [unformatted object](https://raw.githubusercontent.com/mmuddasani/deno-fmt-comment-loss/main/testdata/object.jsonc) with [formatted object](https://raw.githubusercontent.com/mmuddasani/deno-fmt-comment-loss/main/generated/object.jsonc). This bug is probably for a...

@kt3k @ayame113 Is it possible and/or worthwhile to implement the core business logic in Rust as part of the `deno` CLI, while somehow having Deno standard library expose a convenient...

Is the [unicodeSets](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/unicodeSets) feature (`v`-mode RegExp) allowed in Deno standard library? I don't see any current usages from a rudimentary code [search](https://github.com/search?q=repo%3Adenoland%2Fdeno_std+lang%3Ats+-path%3A_test.ts+%2F%5C%2F%5Bdgimsuvy%5D%2B%5Cb%2F&type=code). At the very least, I've developed an appreciation...

To avoid comment loss by default, perhaps the below scenarios provide a sensible default mode. In order to assess for any unforeseen ambiguity/unknowns, I will try to prototype an API...

The error can be reproduced by running `./test/bootstrap.ts --clean issue-502`. I've shared test output below when I executed from my local machine. For some odd reason, there is a null...

@farsightsoftware I'm an interested spectator here, and it seems your proposed route will be faster to do than adding a JSONC AST parser to deno std itself (which I am...