Add non-null assertion to codeEventEntity for consistent fail-fast behavior

[Copilot]

Addresses review feedback on #602 to add non-null assertion to codeEventEntity, ensuring it fails fast if the entity doesn't exist rather than silently accepting undefined.

Changes

• Refactored codeEventEntity from conditional assignment in forEach loop to .find()! pattern matching pinActiveEntity
• Added ESLint disable comments to both non-null assertions documenting intentional fail-fast behavior
• Excluded CODE_EVENT_KEY from mainEntities array in forEach loop

Before:

let codeEventEntity: LockCodeManagerEntityEntry;
lockCodeManagerEntities.forEach((entity) => {
    // ...
    else if (entity.key === CODE_EVENT_KEY) {
        codeEventEntity = entity;  // May remain undefined
    }
    // ...
});

After:

// eslint-disable-next-line @typescript-eslint/no-unnecessary-type-assertion
const codeEventEntity = lockCodeManagerEntities.find(
    (entity) => entity.slotNum === slotNum && entity.key === CODE_EVENT_KEY
)!;  // Explicit fail-fast if missing

Both pinActiveEntity and codeEventEntity now use identical patterns, asserting entities always exist per the Entity Creation Blocking Issue fix.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[raman325]

Already incorporated into the parent branch