Skip to content

Conversation

@akhilles
Copy link
Contributor

@akhilles akhilles commented Feb 4, 2026

Note

Medium Risk
Changes the shape of emitted properties by removing keys that were previously present with None, which may affect downstream consumers expecting those fields to exist.

Overview
Updates multiple generics/*.zen component templates to only add optional entries to properties when the corresponding config value is set, instead of always emitting keys with None.

This normalization is applied across Bjt, Capacitor, Crystal, Diode, FerriteBead, Inductor, Led, Mosfet, MountingHole, Resistor, and Thermistor, with NetTie also avoiding legacy fields when variant is used. Tvs is refactored to build a properties dict similarly and pass it into Component.

Written by Cursor Bugbot for commit 676613a. This will update automatically on new commits. Configure here.

Cursor Bugbot found 1 potential issue for commit 676613a

@akhilles akhilles requested review from LK and hexdae February 4, 2026 21:17
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@akhilles akhilles merged commit 89664b4 into main Feb 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant