You won’t break your game, but you will remove the main reason to be running GrimArmory. 
I’ve confirmed the presence of this issue. Working on identifying why it occurred - to see if anything else might be affected.
For now, if you’d like to ‘patch’ this yourself, adding the following to the end of the DeathHooks script file (and rebuilding your mod) should resolve this issue:
function gd.DeathHooks.chthonianrylok_ekketzul(objectID)
gd.quests.areaEQuestVoid.ulgrimBossKilled(objectID)
end
But items dropping are determined in loottable files or am i mistaking here?
Augury cards drop directly from enemy files, not from loottables.
Yep, path opens now as it should, thanks!
Well you’ll be happy to know I’ve identified the triggering problem.
You’ll be less happy to know that I’ll need a bit to update the mod even still.
This issue occurs for any enemies who have a quest trigger behind them AND who are present in the Shattered Realm as unique boss spawns. There are 33 such problem cases.
It’s an odd edge-case which actually only occurs because of some code I wrote to facilitate modding as a whole. Working on adjusting that code currently.
function gd.DeathHooks.aetherialcolossus_gatekeeper(objectID)
gd.quests.areaFQuestMalmouthBridge.unlockLever(objectID)
end
Fixed this one, but yeah, i noticed that it was 2nd quest trigger in row already.
Yet another Hotfix has been released for Grim Armory Version 1.5:
A minor update has been released for Grim Armory:
A(nother) minor update has been released for Grim Armory:
Can we get much higher?
Yes, unfortunately.
Another minor update to Grim Armory has been released, involving some user-feedback-based balance changes and an aesthetic edit for the future:
GrimArmory is looking to the future, and wants your feedback!
It’s been an ongoing issue in GrimArmory that enemy files with loot specific to them (which is basically every item in this mod) run into problems when Crate makes updates to those enemies. This is an issue that DeathHooks was made to tackle, though I never really capitalized on its potential…until now.
In the (hopefully near) future, my goal for GrimArmory is to make its mod-specific loot drop via the DeathHooks script, rather than finagling with enemy loot weights/tables. This will consolidate things internally for me, from a development standpoint, and will also make GrimArmory easier to update to parity with vanilla Grim Dawn content when Crate makes changes to their enemies.
As far as I am aware, and this functionality may change in the future, an item cannot “drop” on the ground via Script. Instead, items can only be directly added to a player’s inventory. (e.g., some vanilla secret quest items) Do you think it would be problematic if Augury cards were added to a player’s inventory rather than first appearing on the ground? On the one hand, Augury cards are components, and therefore are automatically schlurped up when a player walks over them. But you still get to see them drop and be made aware of having received the loot. Direct inventory insertion does not have this form of visual communication.
A script could make use of UI notifications, such as SR Timer increases or the “A Nemesis Approaches” message, for when an Augury card is looted. But I worry about these notifications being too frequent and, subsequently, not paid attention to. I’d like your thoughts on this, too.
While I’m making updates to the looting functionality of the game, I also want to consider changes to the functionality of Augury cards themselves. Currently, they are “partial” components, and you need to assemble a full stack of them to get the completed, “usable” version of the item that can be “spent” at Inusat. Back in the day, partial components could be auto-combined via a simple button on the UI screen, but that button no longer exists and its functionality appears to have been removed from the game.
So, the question then…should I make all Augury Cards ONLY drop as completed versions, similar to the way components function today?
Were this to happen, there would be two immediately-noticeable changes that would come with it:
It sounds to me like you’re knocking down 2 birds with one stone! I’m in favor of the cards being straight added to the player’s inventory instead since that helps remove the need of modifying enemy tables.
I’m not entirely sure how the drop rate currently works as to whether it’s for Heroes or Bosses and whether the cards are random or based off the types of the monsters (like Chaos monsters dropping cards for Chaos sets), but considering that Grimarillion multiplies the number of Hero monsters and people are already annoyed by the presence of “Enemy Hero beaten! Bonus XP!” messages from that, I can see two options:
Add the UI notifications, but only if the card was not previously in your inventory. That way people know to look in their inventory if they’re trying to farm the cards for their desired set but won’t be burdened by the extra messages for remaining card additions.
It’s definitely better to have fewer, but rarer components. That way, players don’t feel they’re wasting their time because they have some, but not all of the requirements. It’s better for peace of mind to not have that 3/4 presence constantly annoying them knowing that last component is out there somewhere but RNG simply decides not to drop it for a lengthy period of time.
Increase the crafting formula to require the original amount of cards required to ‘complete’ the stack.
Change the cards to be quest items. I’m not sure multiple components can be added into the primary reagent slot, though I just tested it briefly. Quest items work for sure though. Use a color code ^o to make the items appear orange instead of purple like quest items, if necessary.
This seems like the simplest change with the least amount of headache and side effects.