Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

Not sure why Leil Selichot has torah reading.

Because it's on shabbos (unless I understood the code wrong)

A separate question: do events like Shabbat Chazon need a torah flag? They don't actually affect torah reading at all, but rather the haftarah. Special haftarah feels like a different indicator.

Proposal: modify "TORAH_READING" to indicate that the main laining changes or that an additional sefer torah is required.
( Nobody is proposing that we add TORAH_READING to every Monday, Thursday and Saturday)
SPECIAL_HAFTARAH might be a good idea for things like shabbat chazon, but that will be a much messier category with fuzzier boundaries, and will be, I think, harder to pin down actual practice, since it varies.

Because it's on shabbos (unless I understood the code wrong)

nobody says selichot on shabbat, they say it on saturday night.

Nobody is proposing that we add TORAH_READING to every Monday, Thursday and Saturday

Actually, it's in the PR.

nobody says selichot on shabbat, they say it on saturday night.

True. But the event is marked for a certain day, not a time (un/fortunately), and so the flag still applies. If you're worried about duplicates, the API already returns multiple events for a given day in certain instances, so one would need to dedup by date anyway. Or perhaps the module should be adjusted to only return the requested flags for a given day, but that's another conversation.

The intent of this PR is to flag every instance that the Torah is read (this was something I had to do by hand for a previous project with this module, which is why I proposed it). Other markers for a (special) haftorah being read could be great too.

True. But the event is marked for a certain day, not a time (un/fortunately), and so the flag still applies. If you're worried about duplicates, the API already returns multiple events for a given day in certain instances, so one would need to dedup by date anyway. Or perhaps the module should be adjusted to only return the requested flags for a given day, but that's another conversation.

Perhaps I don't understand how this flag is to be used. Can you describe what would be a real-world negative consequence of not having the flag on the leil selichot event? I'm asking because it seems like semantic harm is being done when storing false data ( "Leil selichot needs a torah").

For what it's worth, I found that maintaining the special leyning rules over the last few years in the JavaScript version of Hebcal was complex enough that it was worth splitting out into a separate package.

If there is a need for things like special Haftarot and maftir and such in Golang, we could consider porting https://github.com/hebcal/hebcal-leyning from JS to Golang.

Note there is also a REST version of that leyning API

https://www.hebcal.com/home/4277/leyning-torah-reading-api