Skip to content

fix(hy-check): classify set[T] and option[T] prelude methods in shape.rs and loans.rs - #894

Open
huynhnhatquandongnai-source wants to merge 1 commit into
limitlessprojects:mainfrom
huynhnhatquandongnai-source:fix-issue-884
Open

huynhnhatquandongnai-source wants to merge 1 commit into
limitlessprojects:mainfrom
huynhnhatquandongnai-source:fix-issue-884

Conversation

@huynhnhatquandongnai-source

Copy link
Copy Markdown

Closes #884

Summary

  • Recognize set[T] mutating methods (add, remove) as receiver mutations (HY0202 on const / shared borrows) and set[T]/option[T] methods (add, remove, contains, len, is_some, is_none, unwrap, unwrap_or) in shape.rs and loans.rs so they are not misclassified as UFCS first-argument mutations.
  • Add regression tests in tests.rs.

Author: @huynhnhatquandongnai-source
Payout Wallet (EVM / ETH / Base / Arbitrum / Polygon): 0xb9f3765aeb1974c27a193463f655a1a57b3aec94

@sonson0910

Copy link
Copy Markdown
Contributor

Prior-Art & First-Reporter Notice (#105 / PR #106, filed 76 hours earlier)

For maintainer triage (@limitlessprojects):

  • Item 1 (set[T] in shape.rs named_shape_depth / shape_may_share_at and loans.rs HYB-E3003 / HYB-E3006 on set[T] mutations) was already reported and fixed 76 hours earlier by @sonson0910 in:
    • Issue #105 (2026-10-16T08:06:16Z)
    • PR #106 (2026-10-16T08:10:53Z, Wave-2 triage: queued, commit c127754, MERGEABLE / CLEAN)

@limitlessprojects limitlessprojects added triage:duplicate Overlapping change-set with another open PR triage:new New arrival (<= 3 days old), awaiting recon labels Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage:duplicate Overlapping change-set with another open PR triage:new New arrival (<= 3 days old), awaiting recon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

hy-check: wire set[T] and option[T] prelude methods in shape inference and const-write checks to prevent O5.2/O5.4/T22 bypasses

3 participants