Replace `pack.jsonc` with `pack.ts` #468

New issue

Open

opened 2026-01-30 08:11:53 +00:00 by LunarTides · 0 comments

LunarTides commented

2026-01-30 08:11:53 +00:00

(Migrated from github.com)

Alternative to #467.

There are a few problems with this. Mainly that parsing becomes a lot harder. I can't import the file since it could be malicious, therefore I have to parse the content of the file like I do the cards. It should be possible though.

I could actually import the file if I can make sure it doesn't run any malicious code. E.g. Import whitelist, no code at the top of the file, etc... Hmm... Maybe make use of #469?

Alternative to #467. There are a few problems with this. Mainly that parsing becomes a lot harder. I can't import the file since it could be malicious, therefore I have to parse the content of the file like I do the cards. It should be possible though. I could actually import the file if I can make sure it doesn't run any malicious code. E.g. Import whitelist, no code at the top of the file, etc... Hmm... Maybe make use of #469?

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

LunarTides/Hearthstone.js#468

No description provided.

Rows
Columns

Replace pack.jsonc with pack.ts #468

Replace `pack.jsonc` with `pack.ts` #468