Skip to content

Conversation

@seveibar
Copy link
Contributor

@seveibar seveibar commented Jul 7, 2025

Summary

  • implement LQFN footprint
  • expose lqfn builder and schema
  • include lqfn in helper types
  • add a simple LQFN56 test and snapshot
  • update zod to latest

Testing

  • BUN_UPDATE_SNAPSHOTS=1 bun test tests/lqfn.test.ts

https://chatgpt.com/codex/tasks/task_b_686b1007c33c832e9946acce16b9b1a2

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.

Bug: Direct Mutation Causes Unintended Side Effects

The lqfn function directly mutates its input parameters object by setting parameters.legsoutside = false. This modifies the caller's object, which can cause unexpected side effects if the original object is reused. The function should instead create a copy of the parameters, for example, using object spreading: return quad({ ...parameters, legsoutside: false }).

src/fn/lqfn.ts#L9-L10

): { circuitJson: AnySoupElement[]; parameters: any } => {
parameters.legsoutside = false

Fix in CursorFix in Web


BugBot free trial expires on July 22, 2025
You have used $0.00 of your $50.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants