• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

nielstron / opshin / 17709286207
91%
dev: 92%

Build:
Build:
LAST BUILD BRANCH: openhands-fix-issue-5-try2
DEFAULT BRANCH: dev
Ran 14 Sep 2025 09:41AM UTC
Jobs 2
Files 50
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

14 Sep 2025 09:25AM UTC coverage: 90.856%. First build
17709286207

push

github

openhands-agent
Fix issue #5 with Model B

Summary of Changes:
The changes made attempt to improve the handling of `Self` type annotations by introducing a recursive processing function `_process_annotation_for_self` in the `RewriteImportTyping` class. This function is designed to replace `Self` with the actual class name within complex type annotations such as `Dict[Self, int]` or `Union[Self, int]`. Additionally, the `type_inference.py` file was updated to check for the presence of an `idSelf` attribute on `Name` nodes that represent `Self`, and to raise a more informative error if `Self` is not properly processed.

However, the issue has not been fully resolved because:

1. In `rewrite_import_typing.py`, the code still sets `annotation.idSelf = class_name` when it should likely be setting `annotation.id = class_name` to actually replace the identifier. The attribute `idSelf` does not change the actual type reference used later in the compilation process.

2. In `type_inference.py`, the code looks for `ann.idSelf` but the rewrite phase incorrectly assigns to `idSelf` instead of modifying the `id` attribute directly. This means the type inference phase will not find the rewritten type and will either fail to resolve `Self` or fall back to the error message, which was part of the original problem.

As a result, the core issue of properly resolving `Self` in complex type annotations like `Dict[Self, int]` remains unresolved, and the error messages, while potentially improved, do not indicate that the underlying functionality now works as expected.

Duration: 5m 23s

1916 of 2318 branches covered (82.66%)

Branch coverage included in aggregate %.

6 of 13 new or added lines in 2 files covered. (46.15%)

4761 of 5031 relevant lines covered (94.63%)

1.87 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
3
90.32
opshin/type_inference.py
4
85.25
opshin/rewrite/rewrite_import_typing.py
Jobs
ID Job ID Ran Files Coverage
1 3.13 - 17709286207.1 14 Sep 2025 09:45AM UTC 50
90.98
2 3.9 - 17709286207.2 14 Sep 2025 09:45AM UTC 50
92.13
GitHub Action Run
Source Files on build 17709286207
  • Tree
  • List 50
  • Changed 0
  • Source Changed 0
  • Coverage Changed 0
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses
  • Back to Repo
  • a3c2044b on github
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc