Fix duplicate SV struct typedefs when multiple Python APIs share the same struct type

[Copilot]

• Add unit test to confirm duplicate struct typedef issue when multiple Python APIs share the same struct type
• Fix _collect_struct_types in gen_sv_class.py to track already-emitted struct types across multiple gen() calls via _emitted_struct_types set
• Clear _struct_types after each gen() call to avoid memory growth
• Fix CI: install pcpp cxxheaderparser without -U to avoid upgrading setuptools to a version incompatible with ivpm's build_ext.py (copy_file() got an unexpected keyword argument 'dry_run')
• Verify fix with unit tests (23 passed)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Duplicate SV structure typedefs are created when referenced by multiple Python APs</issue_title>
<issue_description>When multiple Python APIs reference a structure type in their method arguments, the structure typedef is replicated in the generated SV package:

class Point(ct.Structure):
    _fields_ = [
        ("x", ct.c_int32),
        ("y", ct.c_int32),
    ]

@hif.api
class FirstAPI(object):
    @hif.imp
    async def func1(self, pt: Point):
        pass

@hif.api
class SecondAPI(object):
    @hif.exp
   def func2(self, pt: Point):
        print(pt)

The above code results in two Point_t structure typedefs in the generated SV package, resulting in the following VCS error:

Error-[IPD] Identifier previously declared
  Identifier 'Point_t' previously declared as typedef

A workaround is to split these APIs into separate files and generate them separately, but it would be convenient to add memory when a structure was generated to prevent duplicate definitions.</issue_description>

<agent_instructions>Create a test for this situation and confirm the presence of duplicate struct/class definitions. Investigate and correct the duplication</agent_instructions>

Comments on the Issue (you are @copilot in this section)

• Fixes Duplicate SV structure typedefs are created when referenced by multiple Python APs #49

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[mballance]

@copilot, seems something broke. Review failures and correct

[Copilot]

@copilot, seems something broke. Review failures and correct

The CI failure was caused by a newer version of ivpm no longer installing pcpp and cxxheaderparser via ivpm update -a (these are in the default-ci dep-set but were not making it into packages/python). My code changes (gen_sv_class.py and test file) were unrelated to this failure.

Fixed in commit 211d527 by explicitly installing pcpp cxxheaderparser in the CI workflow's pip install step before running setup.py build_ext --inplace.

[mballance]

@copilot still not quite right. Check and correct

[Copilot]

@copilot still not quite right. Check and correct

The previous fix used pip install -U ... pcpp cxxheaderparser which upgraded setuptools as a side effect, making it incompatible with ivpm's build_ext.py (copy_file() got an unexpected keyword argument 'dry_run').

Fixed in commit 538bf14 by installing pcpp cxxheaderparser in a separate pip install line without -U, so that setuptools is not upgraded.