Skip to content

@qwitwa's V8 REPL concern #5156

Description

@avivkeller

@avivkeller Apologies for abusing this PR for an off topic DM, but I would appreciate some guidance in what, if anything, should be done to raise the issue I encountered, and if so, where. I had a look on the chromium issue tracker and found a few similarish discussions but was honestly intimidated by it. In my own app, I've fixed this by stripping the contents of the backticks and only completing on an empty string. That approach works for me and I expect it would work in a statistical majority of cases, but it doesn't seem actually Correct (there's nothing actually preventing an empty template argument to a tag function from side-effecting, never mind the fact that I'm assuming "my" code is handling nesting complexities right with acorn, but it's vibecoded and I haven't reviewed it properly.)

Is this worth filing a node issue for? V8 only? Is there a chat room that would make sense for me to visit to get pointed in the right direction? Feel free to ignore this if you don't have the spoons, since it isn't relevant to this PR as you've said. I just figured whilst I have the motivation for this it might be worth making it bureaucratically visible.

Originally posted by @qwitwa in nodejs/node#64034 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    needs more infoissues that need more info from the author

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions